wingbot 3.75.9-alpha.1 → 3.75.9-alpha.2

package/.claude/settings.local.json

@@ -0,0 +1,10 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(npx mocha:*)",
+      "Bash(npx tsc *)",
+      "Bash(npm test *)",
+      "Bash(node -e \"const p = require\\('./bot/plugins/BTMLLM'\\); console.log\\('factory type:', typeof p\\); console.log\\('factory name:', p.name\\);\")"
+    ]
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wingbot",
-  "version": "3.75.9-alpha.1",
+  "version": "3.75.9-alpha.2",
   "description": "Enterprise Messaging Bot Conversation Engine",
   "main": "index.js",
   "type": "commonjs",

package/src/LLMSession.js

@@ -118,7 +118,6 @@ const stateData = require('./utils/stateData');
  * @prop {string} name - The function name
  * @prop {string} [description] - What the function does
  * @prop {FnParamsObject} [parameters] - Parameter schema
- * @prop {boolean} [strict]
  */
 
 /**
@@ -351,19 +350,15 @@ class LLMSession {
             .map(({
                 name,
                 description = null,
-                parameters = {},
-                strict = true
+                parameters = {}
             }) => ({
                 name,
                 ...(description && { description }),
-                ...(typeof strict === 'boolean' ? { strict } : {}),
+                strict: true,
                 parameters: {
                     type: 'object',
                     properties: {},
                     additionalProperties: false,
-                    required: 'properties' in parameters
-                        ? Object.keys(parameters.properties)
-                        : [],
                     ...parameters
                 }
             }));

package/src/LLMType.js

@@ -196,10 +196,11 @@ class LLMType {
     }
 
     /**
-     * Marks field as optional when used as an object property.
+     * Marks field as optional when used as an object property. In the
+     * generated schema this surfaces as a nullable type (e.g. `['string', 'null']`)
+     * — strict mode still requires the key to be present, just allows `null`.
      *
-     * When used outside object-property context, it has no practical effect,
-     * because requiredness is evaluated by parent object schemas.
+     * Outside an object-property context it has no practical effect.
      *
      * @example
      * const input = LLMType.object({
@@ -215,10 +216,9 @@ class LLMType {
     }
 
     /**
-     * Returns JSON Schema representation.
-     *
-     * For object schemas, nested properties are resolved recursively by calling
-     * `toJSON()` on each nested `LLMType` instance.
+     * Returns JSON Schema representation in OpenAI strict-mode form:
+     * every property is listed in `required`, and properties marked
+     * `optional()` are made nullable instead of being dropped from `required`.
      *
      * @example
      * const schema = LLMType.object({
@@ -246,18 +246,18 @@ class LLMType {
                 const properties = {};
                 const required = [];
                 Object.entries(this._properties).forEach(([key, child]) => {
-                    properties[key] = child.toJSON();
-                    if (child._required) {
-                        required.push(key);
+                    const childSchema = child.toJSON();
+                    if (!child._required) {
+                        childSchema.type = [childSchema.type, 'null'];
                     }
+                    properties[key] = childSchema;
+                    required.push(key);
                 });
 
                 schema.type = 'object';
                 schema.properties = properties;
                 schema.additionalProperties = false;
-                if (required.length > 0) {
-                    schema.required = required;
-                }
+                schema.required = required;
                 break;
             }