address CR

Author: mfori

packages/input_schema/src/input_schema.ts

@@ -270,7 +270,7 @@ function validateField(validator: Ajv, fieldSchema: Record<string, unknown>, fie
     if ('propertyNames' in fieldSchema && fieldSchema.propertyNames?.pattern) {
         validateRegexpPattern(fieldSchema.propertyNames.pattern, `${fieldKey}.propertyNames.pattern`);
     }
-    if ('patternProperties' in fieldSchema && fieldSchema.patternProperties?.['.*'].pattern) {
+    if ('patternProperties' in fieldSchema && fieldSchema.patternProperties?.['.*']?.pattern) {
         validateRegexpPattern(fieldSchema.patternProperties['.*'].pattern, `${fieldKey}.patternProperties.*.pattern`);
     }
 }

packages/input_schema/src/types.ts

@@ -57,7 +57,7 @@ export type ObjectFieldDefinition = CommonFieldDefinition<object> & {
     required?: string[];
     additionalProperties?: boolean;
     propertyNames?: { pattern: string };
-    patternProperties?: { '.*': { type: 'string', pattern: string } };
+    patternProperties?: { '.*': { type: 'string'; pattern: string; } };
 }
 
 export type ArrayFieldDefinition = CommonFieldDefinition<unknown[]> & {

test/input_schema.test.ts

@@ -1567,7 +1567,7 @@ describe('input_schema.json', () => {
                                 patternProperties: {
                                     '^[a-zA-Z_][a-zA-Z0-9_]*$': {
                                         type: 'string',
-                                        regex: '^[0-9]+$',
+                                        pattern: '^[0-9]+$',
                                     },
                                 },
                             },

test/utilities.client.test.ts

@@ -1787,7 +1787,6 @@ describe('utilities.client', () => {
                         type: 'object',
                         editor: 'schemaBased',
                         propertyNames: {
-                            type: 'string',
                             pattern: '^key_\\d+$',
                         },
                     },