fix(js/core): allow Gemini propertyOrdering in schema validation

js/core/src/schema.ts

@@ -25,6 +25,10 @@ import { GenkitError } from './error.js';
 import type { Registry } from './registry.js';
 const ajv = new Ajv();
 addFormats(ajv);
+// Gemini structured output uses the non-standard `propertyOrdering` keyword as a
+// field-order hint. Register it so Ajv strict mode does not reject those schemas.
+// See https://ai.google.dev/gemini-api/docs/structured-output#property-ordering
+ajv.addVocabulary(['propertyOrdering']);
 
 export { z }; // provide a consistent zod to use throughout genkit
 

js/core/tests/schema_test.ts

@@ -105,6 +105,38 @@ describe('validate()', () => {
         { path: '(root)', message: "must have required property 'foo'" },
       ],
     },
+    {
+      it: 'should allow Gemini propertyOrdering keyword in json schema',
+      jsonSchema: {
+        type: 'object',
+        properties: {
+          answer: { type: 'number' },
+          x: { type: 'number' },
+          y: { type: 'number' },
+        },
+        required: ['x', 'y', 'answer'],
+        propertyOrdering: ['x', 'y', 'answer'],
+        additionalProperties: false,
+      },
+      data: { x: 1, y: 2, answer: 3 },
+      valid: true,
+    },
+    {
+      it: 'should still validate data when propertyOrdering is present',
+      jsonSchema: {
+        type: 'object',
+        properties: {
+          x: { type: 'number' },
+          y: { type: 'number' },
+        },
+        required: ['x', 'y'],
+        propertyOrdering: ['x', 'y'],
+        additionalProperties: false,
+      },
+      data: { x: 'nope', y: 2 },
+      valid: false,
+      errors: [{ path: 'x', message: 'must be number' }],
+    },
   ];
   for (const test of tests) {
     it(test.it, () => {
@@ -375,6 +407,27 @@ describe('disableSchemaCodeGeneration()', () => {
     assert.strictEqual(result.valid, true);
   });
 
+  it('should allow Gemini propertyOrdering keyword in interpret mode', () => {
+    disableSchemaCodeGeneration();
+    const result = validateSchema(
+      { x: 1, y: 2, answer: 3 },
+      {
+        jsonSchema: {
+          type: 'object',
+          properties: {
+            answer: { type: 'number' },
+            x: { type: 'number' },
+            y: { type: 'number' },
+          },
+          required: ['x', 'y', 'answer'],
+          propertyOrdering: ['x', 'y', 'answer'],
+          additionalProperties: false,
+        },
+      }
+    );
+    assert.strictEqual(result.valid, true);
+  });
+
   it('should strip undefined values recursively', () => {
     disableSchemaCodeGeneration();
     const result = validateSchema(