@volter/twin-openai 2.0.6 → 2.0.7

package/dist/src/openai-scenario.js

@@ -57,6 +57,8 @@ export const openaiScenarioAdapter = {
         modelEquals: (req, cond) => nonEmptyString(cond) && req.model === cond,
         userTextIncludes: (req, cond) => nonEmptyString(cond) && lastUserText(req.messages).toLowerCase().includes(cond.toLowerCase()),
         anyTextIncludes: (req, cond) => nonEmptyString(cond) && req.messages.map((m) => contentToText(m.content)).join('\n').toLowerCase().includes(cond.toLowerCase()),
+        // the latest message alone (a tool's result just returned, not one from earlier in the conversation)
+        lastMessageTextIncludes: (req, cond) => nonEmptyString(cond) && contentToText(req.messages[req.messages.length - 1]?.content).toLowerCase().includes(cond.toLowerCase()),
         lastMessageIsToolResult: (req, cond) => typeof cond === 'boolean' && (req.messages[req.messages.length - 1]?.role === 'tool') === cond,
         toolResultFor: (req, cond) => nonEmptyString(cond) && lastToolResultNames(req.messages).has(cond),
         hasTool: (req, cond) => nonEmptyString(cond) && toolNames(req.tools).includes(cond),
@@ -66,7 +68,7 @@ export const openaiScenarioAdapter = {
     },
     text: (req) => req.messages.map((m) => contentToText(m.content)).join('\n'),
     validateOn: (on) => {
-        for (const k of ['modelEquals', 'userTextIncludes', 'anyTextIncludes', 'toolResultFor', 'hasTool'])
+        for (const k of ['modelEquals', 'userTextIncludes', 'anyTextIncludes', 'lastMessageTextIncludes', 'toolResultFor', 'hasTool'])
             if (on[k] !== undefined && (typeof on[k] !== 'string' || !on[k]))
                 return `on.${k} is a non-empty string`;
         if (on.lastMessageIsToolResult !== undefined && typeof on.lastMessageIsToolResult !== 'boolean')

package/dist/src/openai-stub.d.ts

@@ -20,8 +20,12 @@ export declare function stubAssistantText(messages: ChatMessageParam[], model: s
 /**
  * Build a deterministic stub argument string for a tool. When the tool declares a JSON-schema
  * `parameters` object (especially with `strict:true`), real models emit arguments that validate
- * against the schema; the twin synthesizes a deterministic object containing every declared
- * property with a type-appropriate placeholder so `strict` callers parse it cleanly. A required
+ * against the schema; the twin synthesizes a deterministic object with a type-appropriate
+ * placeholder for each property it sets, so `strict` callers parse it cleanly. A strict tool, or a
+ * schema that names no `required` list, gets every declared property (strict makes every property
+ * required); otherwise only the required ones: a model leaves an optional parameter it has no reason
+ * to set unset, and a placeholder there is a choice nobody made (LibreChat's web_search took an
+ * optional `date` enum's first value, "past hour", and its search was refused). A required
  * string is never left empty: an empty value is schema-valid in type but names nothing, so an
  * application's tool could not act on it and a story could not follow the call. It takes what the
  * user's message names (`namedIn`), else a deterministic value of the parameter's own name; an enum

package/dist/src/openai-stub.js

@@ -95,8 +95,12 @@ function namedIn(userText) {
 /**
  * Build a deterministic stub argument string for a tool. When the tool declares a JSON-schema
  * `parameters` object (especially with `strict:true`), real models emit arguments that validate
- * against the schema; the twin synthesizes a deterministic object containing every declared
- * property with a type-appropriate placeholder so `strict` callers parse it cleanly. A required
+ * against the schema; the twin synthesizes a deterministic object with a type-appropriate
+ * placeholder for each property it sets, so `strict` callers parse it cleanly. A strict tool, or a
+ * schema that names no `required` list, gets every declared property (strict makes every property
+ * required); otherwise only the required ones: a model leaves an optional parameter it has no reason
+ * to set unset, and a placeholder there is a choice nobody made (LibreChat's web_search took an
+ * optional `date` enum's first value, "past hour", and its search was refused). A required
  * string is never left empty: an empty value is schema-valid in type but names nothing, so an
  * application's tool could not act on it and a story could not follow the call. It takes what the
  * user's message names (`namedIn`), else a deterministic value of the parameter's own name; an enum
@@ -109,9 +113,13 @@ export function stubToolArguments(tool, userText = '') {
     if (!props || typeof props !== 'object')
         return '{}';
     const required = new Set(Array.isArray(schema.required) ? schema.required.map(String) : []);
+    const strict = tool?.function?.strict === true || tool?.strict === true;
+    const onlyRequired = !strict && Array.isArray(schema.required);
     const words = new Set(userText.toLowerCase().match(/[a-z0-9-]+/g) ?? []);
     const out = {};
     for (const [key, def] of Object.entries(props)) {
+        if (onlyRequired && !required.has(key))
+            continue;
         const d = def;
         const named = Array.isArray(d?.enum) ? d.enum.find((e) => typeof e === 'string' && words.has(e.toLowerCase())) : undefined;
         out[key] = named ?? (required.has(key) && d?.type === 'string' && !Array.isArray(d?.enum) ? namedIn(userText) ?? `twin-${key}` : placeholderForSchema(def));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/twin-openai",
-  "version": "2.0.6",
+  "version": "2.0.7",
   "description": "Local OpenAI twin — a faithful, stateful local OpenAI API your real `openai` SDK talks to unmodified. The model is stubbed (deterministic) and embeddings are deterministic pseudo-vectors, but the protocol envelope (chat completions/streaming/tool_calls, the Responses API, embeddings, models, files, batches, moderations, fine-tuning, vector stores) is vendor-faithful. Built on @volter/world-core.",
   "keywords": [
     "twin",
@@ -54,10 +54,10 @@
     "postpack": "node ../../../scripts/publish/prepare-publish.mjs postpack"
   },
   "peerDependencies": {
-    "@volter/world-core": "2.0.6"
+    "@volter/world-core": "2.0.7"
   },
   "devDependencies": {
-    "@volter/world-core": "2.0.6",
+    "@volter/world-core": "2.0.7",
     "@volter/world-tooling": "0.1.0",
     "openai": "^4.104.0",
     "@types/bun": "^1.2.20",
@@ -68,6 +68,6 @@
     "node": ">=22.3"
   },
   "dependencies": {
-    "@volter/world-ui": "0.1.6"
+    "@volter/world-ui": "0.1.7"
   }
 }

package/src/openai-scenario.ts

@@ -77,6 +77,8 @@ export const openaiScenarioAdapter: PackScenarioAdapter<OpenAIScenarioRequest> =
     modelEquals: (req, cond) => nonEmptyString(cond) && req.model === cond,
     userTextIncludes: (req, cond) => nonEmptyString(cond) && lastUserText(req.messages).toLowerCase().includes(cond.toLowerCase()),
     anyTextIncludes: (req, cond) => nonEmptyString(cond) && req.messages.map((m) => contentToText(m.content)).join('\n').toLowerCase().includes(cond.toLowerCase()),
+    // the latest message alone (a tool's result just returned, not one from earlier in the conversation)
+    lastMessageTextIncludes: (req, cond) => nonEmptyString(cond) && contentToText((req.messages[req.messages.length - 1] as { content?: unknown } | undefined)?.content as never).toLowerCase().includes(cond.toLowerCase()),
     lastMessageIsToolResult: (req, cond) => typeof cond === 'boolean' && ((req.messages[req.messages.length - 1] as { role?: string } | undefined)?.role === 'tool') === cond,
     toolResultFor: (req, cond) => nonEmptyString(cond) && lastToolResultNames(req.messages).has(cond),
     hasTool: (req, cond) => nonEmptyString(cond) && toolNames(req.tools).includes(cond),
@@ -86,7 +88,7 @@ export const openaiScenarioAdapter: PackScenarioAdapter<OpenAIScenarioRequest> =
   },
   text: (req) => req.messages.map((m) => contentToText(m.content)).join('\n'),
   validateOn: (on) => {
-    for (const k of ['modelEquals', 'userTextIncludes', 'anyTextIncludes', 'toolResultFor', 'hasTool'] as const) if (on[k] !== undefined && (typeof on[k] !== 'string' || !on[k])) return `on.${k} is a non-empty string`;
+    for (const k of ['modelEquals', 'userTextIncludes', 'anyTextIncludes', 'lastMessageTextIncludes', 'toolResultFor', 'hasTool'] as const) if (on[k] !== undefined && (typeof on[k] !== 'string' || !on[k])) return `on.${k} is a non-empty string`;
     if (on.lastMessageIsToolResult !== undefined && typeof on.lastMessageIsToolResult !== 'boolean') return 'on.lastMessageIsToolResult is a boolean';
     for (const k of ['maxTokensBelow', 'maxTokensAtLeast'] as const) if (on[k] !== undefined && !positiveNumber(on[k])) return `on.${k} is a positive number`;
     return null;

package/src/openai-stub.ts

@@ -100,8 +100,12 @@ function namedIn(userText: string): string | undefined {
 /**
  * Build a deterministic stub argument string for a tool. When the tool declares a JSON-schema
  * `parameters` object (especially with `strict:true`), real models emit arguments that validate
- * against the schema; the twin synthesizes a deterministic object containing every declared
- * property with a type-appropriate placeholder so `strict` callers parse it cleanly. A required
+ * against the schema; the twin synthesizes a deterministic object with a type-appropriate
+ * placeholder for each property it sets, so `strict` callers parse it cleanly. A strict tool, or a
+ * schema that names no `required` list, gets every declared property (strict makes every property
+ * required); otherwise only the required ones: a model leaves an optional parameter it has no reason
+ * to set unset, and a placeholder there is a choice nobody made (LibreChat's web_search took an
+ * optional `date` enum's first value, "past hour", and its search was refused). A required
  * string is never left empty: an empty value is schema-valid in type but names nothing, so an
  * application's tool could not act on it and a story could not follow the call. It takes what the
  * user's message names (`namedIn`), else a deterministic value of the parameter's own name; an enum
@@ -113,9 +117,12 @@ export function stubToolArguments(tool: unknown, userText = ''): string {
   const props = schema && typeof schema === 'object' ? schema.properties : undefined;
   if (!props || typeof props !== 'object') return '{}';
   const required = new Set(Array.isArray(schema!.required) ? schema!.required.map(String) : []);
+  const strict = (tool as { function?: { strict?: unknown }; strict?: unknown } | undefined)?.function?.strict === true || (tool as { strict?: unknown } | undefined)?.strict === true;
+  const onlyRequired = !strict && Array.isArray(schema!.required);
   const words = new Set(userText.toLowerCase().match(/[a-z0-9-]+/g) ?? []);
   const out: Record<string, unknown> = {};
   for (const [key, def] of Object.entries(props)) {
+    if (onlyRequired && !required.has(key)) continue;
     const d = def as { type?: unknown; enum?: unknown[] } | undefined;
     const named = Array.isArray(d?.enum) ? d!.enum!.find((e) => typeof e === 'string' && words.has(e.toLowerCase())) : undefined;
     out[key] = named ?? (required.has(key) && d?.type === 'string' && !Array.isArray(d?.enum) ? namedIn(userText) ?? `twin-${key}` : placeholderForSchema(def));