Release 6.1.0 (#185)

* fix: bug with `kebab-case` path params (issue #184, thanks @Mr-sgreen)

* fix: typings for --js option; feat: --silent option (output only errors to console)

* bump: up version to 6.1.0

Author: js2me

CHANGELOG.md

@@ -1,5 +1,12 @@
 # next release  
 
+Features:  
+- `--silent` option. Output only errors to console (default: false)  
+
+Fixes:  
+- Bug with `kebab-case` path params (issue #184, thanks @Mr-sgreen)  
+- Typings for `--js` option  
+
 # 6.0.0  
 
 BREAKING_CHANGES:  

README.md

@@ -57,6 +57,7 @@ Options:
   --clean-output                clean output folder before generate api. WARNING: May cause data loss (default: false)
   --axios                       generate axios http client (default: false)
   --single-http-client          Ability to send HttpClient instance to Api constructor (default: false)
+  --silent                      Output only errors to console (default: false)
   --default-response <type>     default type for empty response schema (default: "void")
   -h, --help                    display help for command
 ```

index.js

@@ -64,6 +64,7 @@ program
   .option("--disableStrictSSL", "disabled strict SSL", false)
   .option("--axios", "generate axios http client", false)
   .option("--single-http-client", "Ability to send HttpClient instance to Api constructor", false)
+  .option("--silent", "Output only errors to console", false)
   .option("--default-response <type>", "default type for empty response schema", TS_KEYWORDS.VOID)
   .option(
     "--clean-output",
@@ -93,6 +94,7 @@ const {
   defaultResponse,
   singleHttpClient,
   axios,
+  silent,
 } = program;
 
 generateApi({
@@ -116,4 +118,5 @@ generateApi({
   disableStrictSSL: !!disableStrictSSL,
   singleHttpClient: !!singleHttpClient,
   cleanOutput: !!cleanOutput,
+  silent: !!silent,
 });

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "swagger-typescript-api",
-  "version": "6.0.0",
+  "version": "6.1.0",
   "description": "Create typescript api module from swagger schema",
   "scripts": {
     "cli:json": "node index.js -r -d -p ./swagger-test-cli.json -n swagger-test-cli.ts --extract-request-params --enum-names-as-values",
@@ -30,6 +30,7 @@
     "test:--enum-names-as-values": "node tests/spec/enumNamesAsValues/test.js",
     "test:--default-response": "node tests/spec/defaultResponse/test.js",
     "test:--js": "node tests/spec/js/test.js",
+    "test:--js--axios": "node tests/spec/jsAxios/test.js",
     "test:--axios": "node tests/spec/axios/test.js",
     "test:partialBaseTemplate": "node tests/spec/partialBaseTemplate/test.js",
     "test:partialDefaultTemplate": "node tests/spec/partialDefaultTemplate/test.js"

package.json

@@ -64,6 +64,8 @@ const config = {
   },
   /** Record<templateName, templateContent> */
   templatesToRender: {},
+  toJS: false,
+  silent: false,
 };
 
 /** needs to use data everywhere in project */

src/config.js

@@ -26,7 +26,7 @@ module.exports = {
     url,
     spec,
     name: fileName,
-    toJS: translateToJavaScript,
+    toJS: translateToJavaScript = config.toJS,
     modular,
     templates,
     generateResponses = config.generateResponses,
@@ -45,6 +45,7 @@ module.exports = {
     enumNamesAsValues,
     disableStrictSSL = config.disableStrictSSL,
     cleanOutput,
+    silent = config.silent,
   }) =>
     new Promise((resolve, reject) => {
       addToConfig({
@@ -66,6 +67,8 @@ module.exports = {
         defaultResponseType,
         singleHttpClient,
         constants,
+        silent,
+        toJS: translateToJavaScript,
       });
       (spec ? convertSwaggerObject(spec) : getSwaggerObject(input, url, disableStrictSSL))
         .then(({ usageSchema, originalSchema }) => {
@@ -75,7 +78,7 @@ module.exports = {
 
           const templatesToRender = getTemplates(config);
 
-          console.log("☄️  start generating your typescript api");
+          if (!config.silent) console.log("☄️  start generating your typescript api");
 
           fixSwaggerScheme(usageSchema, originalSchema);
 
@@ -148,10 +151,12 @@ module.exports = {
             if (translateToJavaScript) {
               createFile(output, file.name, file.content);
               createFile(output, file.declaration.name, file.declaration.content);
-              console.log(`✔️  your javascript api file created in "${output}"`);
+              if (!config.silent)
+                console.log(`✔️  your javascript api file created in "${output}"`);
             } else {
               createFile(output, file.name, file.content);
-              console.log(`✔️  your typescript api file created in "${output}"`);
+              if (!config.silent)
+                console.log(`✔️  your typescript api file created in "${output}"`);
             }
 
             return file;

src/index.js

@@ -1,12 +1,13 @@
 const _ = require("lodash");
+const { config } = require("./config");
 
 const isValidName = (name) => /^([A-Za-z$_]{1,})$/g.test(name);
 
 const formattedModelNamesMap = new Map();
 
 const checkAndRenameModelName = (name) => {
   if (typeof name !== "string") {
-    console.warn("🔨 wrong name of the model name", name);
+    if (!config.silent) console.warn("🔨 wrong name of the model name", name, config.silent);
 
     return name;
   }

src/modelNames.js

@@ -22,12 +22,13 @@ const getRouteName = (routeInfo) => {
       duplicateIdentifier,
       routeNameDuplicatesMap.get(duplicateIdentifier) + 1,
     );
-    console.warn(
-      `🥵  Module "${moduleName}" already have method "${routeName}()"`,
-      `\n🥵  This method has been renamed to "${
-        routeName + routeNameDuplicatesMap.get(duplicateIdentifier)
-      }()" to solve conflict names.`,
-    );
+    if (!config.silent)
+      console.warn(
+        `🥵  Module "${moduleName}" already have method "${routeName}()"`,
+        `\n🥵  This method has been renamed to "${
+          routeName + routeNameDuplicatesMap.get(duplicateIdentifier)
+        }()" to solve conflict names.`,
+      );
   } else {
     routeNameDuplicatesMap.set(duplicateIdentifier, 1);
   }

src/routeNames.js

@@ -132,14 +132,14 @@ const parseRoute = (route) => {
       if (!paramName) return pathParams;
 
       if (_.includes(paramName, "-")) {
-        console.warn("🔨 wrong path param name", paramName);
+        if (!config.silent) console.warn("🔨 wrong path param name", paramName);
       }
 
       return [
         ...pathParams,
         {
           $match: match,
-          name: paramName,
+          name: _.camelCase(paramName),
           required: true,
           type: "string",
           description: "",
@@ -162,6 +162,7 @@ const parseRoute = (route) => {
   );
 
   return {
+    originalRoute: route || "",
     route: fixedRoute,
     pathParams,
   };
@@ -206,6 +207,10 @@ const getRouteParams = (routeInfo, pathParams) => {
       };
     }
 
+    if (routeParam.in === "path" && routeParam.name) {
+      routeParam.name = _.camelCase(routeParam.name);
+    }
+
     if (routeParam) {
       routeParams[routeParam.in].push(routeParam);
     }