Merge pull request #20 from dev-five-git/add-wrap-target

Add warp guard

Author: owjs3901

.changepacks/changepack_log_s2ifFSZjZS9Av0TpjQsEW.json

@@ -0,0 +1 @@
+{"changes":{"packages/generator/package.json":"Patch"},"note":"Add warp guard","date":"2025-12-12T03:17:58.146361300Z"}

packages/generator/src/__tests__/wrap-interface-key-guard.test.ts

@@ -40,3 +40,18 @@ test.each([
 ] as const)('wrapInterfaceKeyGuard wraps key with multiple slashes: %s -> %s', (key, expected) => {
   expect(wrapInterfaceKeyGuard(key)).toBe(expected)
 })
+
+test.each([
+  ['field"name', '[`field"name`]'],
+  ["field'name", "[`field'name`]"],
+  ['field`name', '[`field`name`]'],
+  ['field-name', '[`field-name`]'],
+  ['field name', '[`field name`]'],
+  ['field@name', '[`field@name`]'],
+  ['field#name', '[`field#name`]'],
+  ['field$name', 'field$name'], // $ is valid in identifiers
+  ['field_name', 'field_name'], // _ is valid in identifiers
+  ['123field', '[`123field`]'], // cannot start with number
+] as const)('wrapInterfaceKeyGuard wraps key with forbidden characters: %s -> %s', (key, expected) => {
+  expect(wrapInterfaceKeyGuard(key)).toBe(expected)
+})

packages/generator/src/wrap-interface-key-guard.ts

@@ -1,5 +1,19 @@
 export function wrapInterfaceKeyGuard(key: string): string {
-  if (key.includes('/') || key.includes('.')) {
+  // Empty string should be returned as-is
+  if (key === '') {
+    return key
+  }
+
+  // Check if key contains forbidden characters that require wrapping
+  // TypeScript identifier pattern: starts with letter/underscore/dollar, followed by letters/numbers/underscore/dollar
+  const isValidIdentifier = /^[a-zA-Z_$][a-zA-Z0-9_$]*$/.test(key)
+
+  if (
+    !isValidIdentifier ||
+    key.includes('"') ||
+    key.includes("'") ||
+    key.includes('`')
+  ) {
     return `[\`${key}\`]`
   }
   return key