@north-light/crouter 0.3.191 → 0.3.192

package/dist/pi-extensions/__tests__/canvas-structured-output.test.js

@@ -1,6 +1,6 @@
 // Run with: node --import tsx/esm --test src/pi-extensions/__tests__/canvas-structured-output.test.ts
 //
-// Regression: Codex strict function schemas require every property in `required`.
+// Regression: strict function schemas require all properties and closed object shapes.
 import { after, test } from 'node:test';
 import assert from 'node:assert/strict';
 import { mkdtempSync, rmSync } from 'node:fs';
@@ -37,9 +37,15 @@ test('optional submit fields remain optional through a strict provider tool sche
             commit: { type: 'string' },
             verification: { type: 'string' },
             blocker: { type: 'string' },
+            details: {
+                type: 'object',
+                properties: {
+                    summary: { type: 'string' },
+                },
+                required: ['summary'],
+            },
         },
-        required: ['status', 'verification'],
-        additionalProperties: false,
+        required: ['status', 'verification', 'details'],
     };
     writeOutputSchema('structured-output-test', 'oneoff', schema);
     let definition;
@@ -51,10 +57,12 @@ test('optional submit fields remain optional through a strict provider tool sche
     });
     assert.ok(definition, 'the pending request registers submit');
     const parameters = definition['parameters'];
-    assert.deepEqual(parameters.required, ['status', 'commit', 'verification', 'blocker']);
+    assert.deepEqual(parameters.required, ['status', 'commit', 'verification', 'blocker', 'details']);
     assert.deepEqual(parameters.properties.commit, { anyOf: [{ type: 'string' }, { type: 'null' }] });
     assert.deepEqual(parameters.properties.blocker, { anyOf: [{ type: 'string' }, { type: 'null' }] });
-    assert.deepEqual(schema.required, ['status', 'verification'], 'the stored request keeps its original optionality');
+    assert.equal(parameters.additionalProperties, false);
+    assert.equal(parameters.properties.details.additionalProperties, false);
+    assert.deepEqual(schema.required, ['status', 'verification', 'details'], 'the stored request keeps its original optionality');
     const [providerTool] = convertResponsesTools([definition], {
         strict: false,
         supportsStrictMode: true,

package/dist/pi-extensions/canvas-structured-output.js

@@ -47,6 +47,10 @@ function strictToolSchema(schema) {
             }
         }
         const properties = transformed['properties'];
+        const isObjectSchema = transformed['type'] === 'object'
+            || (properties !== null && typeof properties === 'object' && !Array.isArray(properties));
+        if (isObjectSchema)
+            transformed['additionalProperties'] = false;
         if (properties === null || typeof properties !== 'object' || Array.isArray(properties))
             return transformed;
         const required = new Set(Array.isArray(source['required']) ? source['required'].filter((name) => typeof name === 'string') : []);
@@ -125,9 +129,9 @@ export function registerCanvasStructuredOutput(pi) {
                         'Use submit to answer the pending one-off structured-output request before stopping or going dormant.',
                         'After one-off submit succeeds, continue with any remaining work normally.',
                     ],
-                // OpenAI strict tools require every property to be required. Optional fields
-                // become nullable in the provider-only schema and null is removed before the
-                // original result is persisted.
+                // Strict provider tools require every property to be required and every
+                // object to reject undeclared properties. Optional fields become nullable
+                // in the provider-only schema and null is removed before persistence.
                 parameters: strictToolSchema(request.schema),
                 constrainedSampling: { type: 'json_schema', strict: 'prefer' },
                 async execute(_toolCallId, params, _signal, _onUpdate, ctx) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.191",
+  "version": "0.3.192",
   "description": "crtr — agent runtime with memory, plugins, and marketplaces",
   "type": "module",
   "main": "dist/index.js",

package/runtime.lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.191",
+  "version": "0.3.192",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@north-light/crouter",
-      "version": "0.3.191",
+      "version": "0.3.192",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {