fix: Properly escape property keys (#2)

denosaurs · Jul 24, 2024 · 5779444 · 5779444
1 parent 7487b43
commit 5779444
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/deno.json b/deno.json
@@ -1,6 +1,6 @@
 {
   "name": "@denosaurs/typefetch",
-  "version": "0.0.8",
+  "version": "0.0.9",
   "exports": {
     ".": "./main.ts"
   },

diff --git a/mod.ts b/mod.ts
@@ -31,6 +31,13 @@ export interface Options {
   includeServerUrls?: boolean;
 }
 
+export function escapeObjectKey(key: string): string {
+  if (/^[a-zA-Z_$][0-9a-zA-Z_$]*$/.test(key)) {
+    return key;
+  }
+  return `"${key}"`;
+}
+
 export function toSchemaType(
   document: OpenAPI.Document,
   schema?:
@@ -125,7 +132,7 @@ export function toSchemaType(
         return `{${
           Object.entries(schema.properties)
             .map(([property, type]) =>
-              `${property}:${toSchemaType(document, type)}`
+              `${escapeObjectKey(property)}:${toSchemaType(document, type)}`
             )
             .join(";")
         }}`;