fix: Ensure that getShortType regex wildcard is non-greedy (#135)

7nohe · Aug 7, 2024 · 8eb0f33 · 8eb0f33
1 parent c6e980b
commit 8eb0f33
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/common.mts b/src/common.mts
@@ -116,7 +116,7 @@ export function extractPropertiesFromObjectParam(param: ParameterDeclaration) {
  * TODO: Replace with a more robust solution.
  */
 export function getShortType(type: string) {
-  return type.replaceAll(/import\(".*"\)\./g, "");
+  return type.replaceAll(/import\(".*?"\)\./g, "");
 }
 
 export function getClassesFromService(node: SourceFile) {

diff --git a/tests/common.test.ts b/tests/common.test.ts
@@ -80,6 +80,15 @@ describe("common", () => {
       const result = getShortType(type);
       expect(result).toBe("MyType");
     });
+
+    test("json", () => {
+      const type =
+        '{ import1?: import("/path/to/import1").Import1; import2: import("/path/to/import2").Import2; import3: import("/path/to/import3").Import3; }';
+      const result = getShortType(type);
+      expect(result).toBe(
+        "{ import1?: Import1; import2: Import2; import3: Import3; }",
+      );
+    });
   });
 
   test("formatOptions - converts string boolean to boolean (false)", () => {