@lazyingart/agintiflow 0.20.261 → 0.20.262

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lazyingart/agintiflow",
-  "version": "0.20.261",
+  "version": "0.20.262",
   "type": "module",
   "description": "AgInTiFlow is a project-aware agent workspace for hybrid wet-dry R&D, hardware-aware intelligence, software automation, and industrial workflows.",
   "license": "Apache-2.0",

package/scripts/smoke-deep-research.js

@@ -189,6 +189,28 @@ async function main() {
     )) === JSON.stringify({ type: "function", function: { name: "apply_patch" } }),
     "DeepSeek action recovery did not force the only executable tool"
   );
+  assert(
+    JSON.stringify(toolChoiceForProvider(
+      { provider: "deepseek", completionFreshMutationRequired: true },
+      [{ role: "user", content: "Repair the retained source from the exact failure evidence." }],
+      [
+        { type: "function", function: { name: "apply_patch" } },
+        { type: "function", function: { name: "finish" } },
+      ]
+    )) === JSON.stringify({ type: "function", function: { name: "apply_patch" } }),
+    "DeepSeek constrained recovery relied on a prompt phrase instead of the offered tool contract"
+  );
+  assert(
+    toolChoiceForProvider(
+      { provider: "deepseek" },
+      [{ role: "user", content: "Explain what this source does." }],
+      [
+        { type: "function", function: { name: "read_file" } },
+        { type: "function", function: { name: "finish" } },
+      ]
+    ) === "auto",
+    "DeepSeek ordinary chat forced a tool merely because few tools were enabled"
+  );
   assert(
     JSON.stringify(providerStructuredOutputAttempts("deepseek")) === JSON.stringify(["json_object", "prompt"]),
     "DeepSeek structured extraction still probes an unsupported JSON Schema mode"
@@ -241,7 +263,7 @@ async function main() {
       completionFreshMutationNeedsSourceRead: false,
       completionFreshMutationPaths: ["service.py"],
     },
-    [{ role: "user", content: "Emit exactly one enabled tool call that performs the next concrete action." }]
+    [{ role: "user", content: "Repair the retained source from the exact failure evidence." }]
   );
   assert(
     deepSeekActionPayload?.thinking?.type === "disabled",

package/src/model-client.js

@@ -348,27 +348,50 @@ function requiresConcreteToolContinuation(messages = []) {
   );
 }
 
-function deepSeekActionOnlyRequest(config = {}, messages = []) {
-  return normalizeProviderId(config.provider, "") === "deepseek" &&
+function singleExecutableToolName(tools = []) {
+  const names = (Array.isArray(tools) ? tools : [])
+    .filter((tool) => tool?.type === "function")
+    .map((tool) => String(tool.function?.name || ""))
+    .filter(Boolean);
+  const actionable = [...new Set(names.filter((name) => name !== "finish"))];
+  if (actionable.length !== 1) return "";
+  return names.every((name) => name === "finish" || name === actionable[0])
+    ? actionable[0]
+    : "";
+}
+
+function deepSeekActionOnlyTool(config = {}, messages = [], tools = []) {
+  if (normalizeProviderId(config.provider, "") !== "deepseek") return "";
+  const constrainedRecovery = config.completionFreshMutationRequired === true ||
     requiresConcreteToolContinuation(messages);
+  if (!constrainedRecovery) return "";
+  const constrainedTool = singleExecutableToolName(tools);
+  if (constrainedTool) return constrainedTool;
+  const actionable = (Array.isArray(tools) ? tools : [])
+    .filter((tool) => tool?.type === "function" && tool.function?.name !== "finish")
+    .map((tool) => String(tool.function?.name || ""))
+    .filter(Boolean);
+  return actionable.length === 1 ? actionable[0] : "";
+}
+
+function deepSeekActionOnlyRequest(config = {}, messages = [], tools = []) {
+  return normalizeProviderId(config.provider, "") === "deepseek" &&
+    Boolean(
+      requiresConcreteToolContinuation(messages) ||
+      (config.completionFreshMutationRequired === true && singleExecutableToolName(tools))
+    );
 }
 
 export function toolChoiceForProvider(config, messages = [], tools = []) {
   const requiresConcreteContinuation = requiresConcreteToolContinuation(messages);
+  const deepSeekActionTool = deepSeekActionOnlyTool(config, messages, tools);
+  if (deepSeekActionTool) {
+    return {
+      type: "function",
+      function: { name: deepSeekActionTool },
+    };
+  }
   if (requiresConcreteContinuation) {
-    const actionable = (Array.isArray(tools) ? tools : [])
-      .filter((tool) => tool?.type === "function" && tool.function?.name !== "finish")
-      .map((tool) => String(tool.function?.name || ""))
-      .filter(Boolean);
-    if (
-      normalizeProviderId(config.provider, "") === "deepseek" &&
-      actionable.length === 1
-    ) {
-      return {
-        type: "function",
-        function: { name: actionable[0] },
-      };
-    }
     return "required";
   }
   if (config.provider !== "venice") return "auto";
@@ -2675,7 +2698,7 @@ export async function requestNextStep(client, config, messages) {
     parallel_tool_calls: false,
     messages: prepareMessages(config, messages),
     tools: requestTools,
-    ...(deepSeekActionOnlyRequest(config, messages)
+    ...(deepSeekActionOnlyRequest(config, messages, requestTools)
       ? { thinking: { type: "disabled" } }
       : {}),
     ...(Number(config.maxOutputTokens || 0) > 0 ? { max_tokens: Number(config.maxOutputTokens) } : {}),