Merge pull request #910 from acacode/ts-migrate

TypeScript

Author: smorimoto

README.md

@@ -39,8 +39,6 @@ Options:
   --js                          generate js api module with declaration file (default: false)
   --module-name-index <number>  determines which path index should be used for routes separation (example: GET:/fruits/getFruit -> index:0 -> moduleName -> fruits) (default: 0)
   --module-name-first-tag       splits routes based on the first tag (default: false)
-  --disableStrictSSL            disabled strict SSL (default: false)
-  --disableProxy                disabled proxy (default: false)
   --axios                       generate axios http client (default: false)
   --unwrap-response-data        unwrap the data item from the response (default: false)
   --disable-throw-on-error      Do not throw an error when response.ok is not true (default: false)
@@ -403,7 +401,7 @@ type PrimitiveTypeStructValue =
   | string
   | ((
       schema: Record<string, any>,
-      parser: import("./src/schema-parser/schema-parser").SchemaParser,
+      parser: import("./src/schema-parser/schema-parser").SchemaParser
     ) => string);
 
 type PrimitiveTypeStruct = Record<
@@ -416,7 +414,7 @@ type PrimitiveTypeStruct = Record<
 >;
 
 declare const primitiveTypeConstructs: (
-  struct: PrimitiveTypeStruct,
+  struct: PrimitiveTypeStruct
 ) => Partial<PrimitiveTypeStruct>;
 
 generateApi({

index.ts

@@ -175,27 +175,17 @@ const generateCommand = defineCommand({
       type: "string",
       description:
         "determines which path index should be used for routes separation (example: GET:/fruits/getFruit -> index:0 -> moduleName -> fruits)",
-      default: codeGenBaseConfig.moduleNameIndex,
+      default: codeGenBaseConfig.moduleNameIndex.toString(),
     },
     "module-name-first-tag": {
       type: "boolean",
       description: "splits routes based on the first tag",
       default: codeGenBaseConfig.moduleNameFirstTag,
     },
-    disableStrictSSL: {
-      type: "boolean",
-      description: "disabled strict SSL",
-      default: codeGenBaseConfig.disableStrictSSL,
-    },
-    disableProxy: {
-      type: "boolean",
-      description: "disabled proxy",
-      default: codeGenBaseConfig.disableProxy,
-    },
     axios: {
       type: "boolean",
       description: "generate axios http client",
-      default: codeGenBaseConfig.httpClientType === HTTP_CLIENT.AXIOS,
+      default: false,
     },
     "unwrap-response-data": {
       type: "boolean",
@@ -223,12 +213,12 @@ const generateCommand = defineCommand({
       default: codeGenBaseConfig.defaultResponseType,
     },
     "type-prefix": {
-      type: "boolean",
+      type: "string",
       description: "data contract name prefix",
       default: codeGenBaseConfig.typePrefix,
     },
     "type-suffix": {
-      type: "boolean",
+      type: "string",
       description: "data contract name suffix",
       default: codeGenBaseConfig.typeSuffix,
     },
@@ -277,7 +267,6 @@ const generateCommand = defineCommand({
     "custom-config": {
       type: "string",
       description: "custom config: primitiveTypeConstructs, hooks, ... ",
-      default: "",
     },
   },
   run: async ({ args }) => {
@@ -306,8 +295,6 @@ const generateCommand = defineCommand({
       debug: args.debug,
       defaultResponseAsSuccess: args["default-as-success"],
       defaultResponseType: args["default-response"],
-      disableProxy: args.disableProxy,
-      disableStrictSSL: args.disableStrictSSL,
       disableThrowOnError: args["disable-throw-on-error"],
       enumNamesAsValues: args["enum-names-as-values"],
       extractEnums: args["extract-enums"],
@@ -325,11 +312,11 @@ const generateCommand = defineCommand({
         args["http-client"] || args.axios
           ? HTTP_CLIENT.AXIOS
           : HTTP_CLIENT.FETCH,
-      input: path.resolve(process.cwd(), args.path),
+      input: path.resolve(process.cwd(), args.path as string),
       modular: args.modular,
       moduleNameFirstTag: args["module-name-first-tag"],
       moduleNameIndex: +args["module-name-index"] || 0,
-      output: path.resolve(process.cwd(), args.output || "."),
+      output: path.resolve(process.cwd(), (args.output as string) || "."),
       patch: args.patch,
       silent: args.silent,
       singleHttpClient: args["single-http-client"],

src/code-formatter.js renamed to src/code-formatter.ts

@@ -1,17 +1,15 @@
 import * as prettier from "prettier";
 import * as typescript from "typescript";
+import type { CodeGenConfig } from "./configuration.js";
 
-class CodeFormatter {
-  /**
-   * @type {CodeGenConfig}
-   */
-  config;
+export class CodeFormatter {
+  config: CodeGenConfig;
 
-  constructor({ config }) {
+  constructor(config: CodeGenConfig) {
     this.config = config;
   }
 
-  removeUnusedImports = (content) => {
+  removeUnusedImports = (content: string) => {
     const tempFileName = "file.ts";
 
     const host = new TsLanguageServiceHost(tempFileName, content);
@@ -20,6 +18,7 @@ class CodeFormatter {
     const fileTextChanges = languageService.organizeImports(
       { type: "file", fileName: tempFileName },
       { newLineCharacter: typescript.sys.newLine },
+      undefined,
     )[0];
 
     if (fileTextChanges?.textChanges.length) {
@@ -35,11 +34,7 @@ class CodeFormatter {
     return content;
   };
 
-  /**
-   * @param content
-   * @returns {Promise<string>}
-   */
-  prettierFormat = async (content) => {
+  prettierFormat = async (content: string) => {
     const formatted = await prettier.format(
       content,
       this.config.prettierOptions,
@@ -48,7 +43,7 @@ class CodeFormatter {
   };
 
   formatCode = async (
-    code,
+    code: string,
     { removeUnusedImports = true, prettierFormat = true } = {},
   ) => {
     if (removeUnusedImports) {
@@ -62,22 +57,24 @@ class CodeFormatter {
 }
 
 class TsLanguageServiceHost {
-  constructor(fileName, content) {
+  fileName: string;
+  content: string;
+  compilerOptions: typescript.CompilerOptions;
+
+  constructor(fileName: string, content: string) {
+    this.fileName = fileName;
+    this.content = content;
     const tsconfig = typescript.findConfigFile(
       fileName,
       typescript.sys.fileExists,
     );
-
-    Object.assign(this, {
-      fileName,
-      content,
-      compilerOptions: tsconfig
-        ? typescript.convertCompilerOptionsFromJson(
-            typescript.readConfigFile(tsconfig, typescript.sys.readFile).config
-              .compilerOptions,
-          ).options
-        : typescript.getDefaultCompilerOptions(),
-    });
+    this.compilerOptions = tsconfig
+      ? typescript.convertCompilerOptionsFromJson(
+          typescript.readConfigFile(tsconfig, typescript.sys.readFile).config
+            .compilerOptions,
+          "",
+        ).options
+      : typescript.getDefaultCompilerOptions();
   }
 
   getNewLine() {
@@ -101,16 +98,14 @@ class TsLanguageServiceHost {
   getScriptSnapshot() {
     return typescript.ScriptSnapshot.fromString(this.content);
   }
-  readFile(fileName, encoding) {
+  readFile(fileName: string, encoding: string) {
     if (fileName === this.fileName) {
       return this.content;
     }
 
     return typescript.sys.readFile(fileName, encoding);
   }
-  fileExists(path) {
+  fileExists(path: string) {
     return typescript.sys.fileExists(path);
   }
 }
-
-export { CodeFormatter };