fix: replace special characters using SPECIAL_CHARACTER_MAP for dup…

…licate-identifiers
openapi-ts · Aug 24, 2024 · 3f679a6 · 3f679a6
1 parent 2a4b067
commit 3f679a6
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/packages/openapi-typescript/src/lib/ts.ts b/packages/openapi-typescript/src/lib/ts.ts
@@ -4,6 +4,10 @@ import ts, { type LiteralTypeNode, type TypeLiteralNode } from "typescript";
 export const JS_PROPERTY_INDEX_RE = /^[A-Za-z_$][A-Za-z_$0-9]*$/;
 export const JS_ENUM_INVALID_CHARS_RE = /[^A-Za-z_$0-9]+(.)?/g;
 export const JS_PROPERTY_INDEX_INVALID_CHARS_RE = /[^A-Za-z_$0-9]+/g;
+export const SPECIAL_CHARACTER_MAP: Record<string, string> = {
+  "+": "Plus",
+  // Add more mappings as needed
+};
 
 export const BOOLEAN = ts.factory.createKeywordTypeNode(ts.SyntaxKind.BooleanKeyword);
 export const FALSE = ts.factory.createLiteralTypeNode(ts.factory.createFalse());
@@ -308,7 +312,7 @@ export function tsEnumMember(value: string | number, metadata: { name?: string;
       if (invalidCharMatch[0] === name) {
         name = `"${name}"`;
       } else {
-        name = name.replace(JS_PROPERTY_INDEX_INVALID_CHARS_RE, "_");
+        name = name.replace(JS_PROPERTY_INDEX_INVALID_CHARS_RE, (substring) => SPECIAL_CHARACTER_MAP[substring] || "_");
       }
     }
   }

diff --git a/packages/openapi-typescript/test/lib/ts.test.ts b/packages/openapi-typescript/test/lib/ts.test.ts
@@ -208,6 +208,14 @@ describe("tsEnum", () => {
     NotFound = 101,
     // User doesn't have permissions
     PermissionDenied = 102
+}`);
+  });
+
+  test("replace special character", () => {
+    expect(astToString(tsEnum("FOO_ENUM", ['Etc/GMT+0', "Etc/GMT+1", "Etc/GMT-1"])).trim()).toBe(`enum FOO_ENUM {
+    Etc_GMTPlus0 = "Etc/GMT+0",
+    Etc_GMTPlus1 = "Etc/GMT+1",
+    Etc_GMT_1 = "Etc/GMT-1"
 }`);
   });
 });