reasonix 0.38.0 → 0.39.0

package/dist/cli/{chat-FPEYKTMI.js → chat-QCY6CH7O.js}

@@ -1,41 +1,42 @@
 #!/usr/bin/env node
 import {
   chatCommand
-} from "./chunk-4PNXH2MH.js";
+} from "./chunk-TPK2CHWR.js";
 import "./chunk-BQNUJJN7.js";
-import "./chunk-RFX7TYVV.js";
+import "./chunk-7G3SESEU.js";
 import "./chunk-MRLXEMZ7.js";
 import "./chunk-CPOV2O73.js";
-import "./chunk-YJKLNYCP.js";
-import "./chunk-UNMYFZPZ.js";
+import "./chunk-5ZCRXN7S.js";
+import "./chunk-SJNIIH5W.js";
 import "./chunk-XJLZ4HKU.js";
 import "./chunk-XHQIK7B6.js";
-import "./chunk-A63QT566.js";
-import "./chunk-3VTV4WAH.js";
-import "./chunk-FB46F6H4.js";
+import "./chunk-DDA76P44.js";
+import "./chunk-NLV2YORE.js";
+import "./chunk-SUZRC4NC.js";
 import "./chunk-MHDNZXJJ.js";
-import "./chunk-T5U5JO7Q.js";
+import "./chunk-TGO7X47P.js";
+import "./chunk-AFFZF3MW.js";
 import "./chunk-DAEAAVDF.js";
 import "./chunk-KMWKGPFZ.js";
 import "./chunk-3Q3C4W66.js";
 import "./chunk-4DCHFFEY.js";
 import "./chunk-WJ3YX4PZ.js";
-import "./chunk-NYP2DDDV.js";
+import "./chunk-TPDWAMG6.js";
 import "./chunk-SOZE7V7V.js";
 import "./chunk-6NMWJSES.js";
-import "./chunk-LY352GTC.js";
-import "./chunk-XOIDSPMQ.js";
+import "./chunk-NTVW2TWO.js";
+import "./chunk-6DR4F3MC.js";
 import "./chunk-FM57FNPJ.js";
-import "./chunk-JOFZ6AW5.js";
-import "./chunk-LMNAMITH.js";
+import "./chunk-4D662BWT.js";
+import "./chunk-AKDDHHE6.js";
 import "./chunk-5X7LZJDE.js";
 import "./chunk-6CXT5JRM.js";
-import "./chunk-FYKZB6TX.js";
-import "./chunk-BW2HWSYH.js";
+import "./chunk-AJGLCSZS.js";
+import "./chunk-SWLIVNTP.js";
 import "./chunk-ZTLZO42A.js";
 import "./chunk-ORM6PK57.js";
 import "./chunk-CRPQUBP6.js";
 export {
   chatCommand
 };
-//# sourceMappingURL=chat-FPEYKTMI.js.map
+//# sourceMappingURL=chat-QCY6CH7O.js.map

package/dist/cli/{chunk-JOFZ6AW5.js → chunk-4D662BWT.js}

@@ -7,7 +7,7 @@ import {
   compileFilters,
   defaultIndexConfig,
   resolveSemanticEmbeddingConfig
-} from "./chunk-BW2HWSYH.js";
+} from "./chunk-SWLIVNTP.js";
 
 // src/index/semantic/builder.ts
 import { promises as fs3 } from "fs";
@@ -948,4 +948,4 @@ export {
   startOllamaDaemon,
   pullOllamaModel
 };
-//# sourceMappingURL=chunk-JOFZ6AW5.js.map
+//# sourceMappingURL=chunk-4D662BWT.js.map

package/dist/cli/{chunk-YJKLNYCP.js → chunk-5ZCRXN7S.js}

@@ -2,7 +2,7 @@
 import {
   MemoryStore,
   sanitizeMemoryName
-} from "./chunk-A63QT566.js";
+} from "./chunk-DDA76P44.js";
 import {
   countTokens,
   estimateConversationTokens,
@@ -13,15 +13,15 @@ import {
 } from "./chunk-KMWKGPFZ.js";
 import {
   pauseGate
-} from "./chunk-LY352GTC.js";
+} from "./chunk-NTVW2TWO.js";
 import {
   NEGATIVE_CLAIM_RULE,
   TUI_FORMATTING_RULES
-} from "./chunk-XOIDSPMQ.js";
+} from "./chunk-6DR4F3MC.js";
 import {
   formatHookOutcomeMessage,
   runHooks
-} from "./chunk-LMNAMITH.js";
+} from "./chunk-AKDDHHE6.js";
 import {
   ignoredByLayers,
   loadGitignoreAt,
@@ -36,12 +36,12 @@ import {
 } from "./chunk-6CXT5JRM.js";
 import {
   t
-} from "./chunk-FYKZB6TX.js";
+} from "./chunk-AJGLCSZS.js";
 import {
   DEFAULT_INDEX_EXCLUDES,
   webSearchEndpoint,
   webSearchEngine
-} from "./chunk-BW2HWSYH.js";
+} from "./chunk-SWLIVNTP.js";
 import {
   DEEPSEEK_CONTEXT_TOKENS,
   DEFAULT_CONTEXT_TOKENS,
@@ -287,6 +287,8 @@ var ToolRegistry = class {
   _interceptor = null;
   _auditListener = null;
   _resultAugmenter = null;
+  /** Per-tool fingerprint of the last call that failed schema validation. Cleared by any successful validation for that tool. */
+  _lastMalformed = /* @__PURE__ */ new Map();
   constructor(opts = {}) {
     this._autoFlatten = opts.autoFlatten !== false;
   }
@@ -361,17 +363,29 @@ var ToolRegistry = class {
     if (!tool) {
       return JSON.stringify({ error: `unknown tool: ${name}` });
     }
+    const fingerprint = fingerprintArgs(argumentsRaw);
     let args;
     try {
       args = typeof argumentsRaw === "string" ? argumentsRaw.trim() ? JSON.parse(argumentsRaw) ?? {} : {} : argumentsRaw ?? {};
     } catch (err) {
-      return JSON.stringify({
-        error: `invalid tool arguments JSON: ${err.message}`
-      });
+      return this._noteMalformed(
+        name,
+        fingerprint,
+        `invalid tool arguments JSON: ${err.message}`
+      );
     }
     if (tool.flatSchema && args && typeof args === "object" && hasDotKey(args)) {
       args = nestArguments(args);
     }
+    const missing = tool.parameters ? missingRequiredParam(tool.parameters, args) : null;
+    if (missing) {
+      return this._noteMalformed(
+        name,
+        fingerprint,
+        `missing required parameter "${missing}". Retry with all required parameters filled.`
+      );
+    }
+    this._lastMalformed.delete(name);
     if (this._planMode && !isReadOnlyCall(tool, args)) {
       return JSON.stringify({
         error: `${name}: unavailable in plan mode \u2014 this is a read-only exploration phase. Use read_file / list_directory / search_files / directory_tree / web_search / allowlisted shell commands to investigate. Call submit_plan with your proposed plan when you're ready for the user's review.`,
@@ -427,6 +441,18 @@ var ToolRegistry = class {
     }
     return finalResult;
   }
+  /** Records the failed call's fingerprint; on the 2nd consecutive identical malformed call to the same tool, returns a sharper error that tells the model to stop retrying. */
+  _noteMalformed(name, fingerprint, detail) {
+    const prev = this._lastMalformed.get(name);
+    this._lastMalformed.set(name, fingerprint);
+    if (prev === fingerprint) {
+      return JSON.stringify({
+        error: `${name}: same call just failed validation (${detail}) \u2014 DO NOT retry with identical args. Either fix the call (read the schema in the tool spec) or pick a different tool.`,
+        consecutiveMalformed: true
+      });
+    }
+    return JSON.stringify({ error: `${name}: ${detail}` });
+  }
 };
 function isReadOnlyCall(tool, args) {
   if (tool.readOnlyCheck) {
@@ -444,6 +470,22 @@ function hasDotKey(obj) {
   }
   return false;
 }
+function fingerprintArgs(argumentsRaw) {
+  if (typeof argumentsRaw === "string") return argumentsRaw;
+  try {
+    return JSON.stringify(argumentsRaw);
+  } catch {
+    return "";
+  }
+}
+function missingRequiredParam(schema, args) {
+  const required = schema.required;
+  if (!required || required.length === 0) return null;
+  for (const key of required) {
+    if (args[key] === void 0) return key;
+  }
+  return null;
+}
 
 // src/memory/runtime.ts
 import { createHash } from "crypto";
@@ -3435,7 +3477,7 @@ function registerChoiceTool(registry, opts = {}) {
 
 // src/tools/plan-core.ts
 var SUBMIT_PLAN_DESCRIPTION = "Submit ONE concrete plan you've already decided on. Use this for tasks that warrant a review gate \u2014 multi-file refactors, architecture changes, anything that would be expensive or confusing to undo. Skip it for small fixes (one-line typo, obvious bug with a clear fix) \u2014 just make the change. The user will either approve (you then implement it), ask for refinement, or cancel. If the user has already enabled /plan mode, writes are blocked at dispatch and you MUST use this. CRITICAL: do NOT use submit_plan to present alternative routes (A/B/C, option 1/2/3) for the user to pick from \u2014 the picker only exposes approve/refine/cancel, so a menu plan strands the user with no way to choose. For branching decisions, call `ask_choice` instead; only call submit_plan once the user has picked a direction and you have a single actionable plan. Write the plan as markdown with a one-line summary, a bulleted list of files to touch and what will change, and any risks or open questions. STRONGLY PREFERRED: pass `steps` \u2014 an array of {id, title, action, risk?} \u2014 so the UI renders a structured step list above the approval picker and tracks per-step progress. Use risk='high' for steps that touch prod data / break public APIs / are hard to undo; 'med' for non-trivial but reversible (multi-file edits, schema tweaks); 'low' for safe local work. After each step, call `mark_step_complete` so the user sees progress ticks.";
-var MARK_STEP_COMPLETE_DESCRIPTION = "Mark one step of the approved plan as done. Call this after finishing each step, then immediately continue with the NEXT step \u2014 do not stop or wait for the user. The TUI updates the plan card's progress in place. After the FINAL step, write a brief reply summarizing what was done and end the turn. Pass the `stepId` from the plan's steps array, a short `result` (what you did), and optional `notes` for anything surprising (errors, scope changes, follow-ups). This tool doesn't change any files. Don't call it if the plan didn't include structured steps, and don't invent ids that weren't in the original plan.";
+var MARK_STEP_COMPLETE_DESCRIPTION = "Mark one step of the approved plan as done. MANDATORY: call this exactly once after finishing each step, before starting the next one \u2014 skipping it leaves the user staring at `0/N done` on the resume banner even when the work is finished, and they have no way to know which steps actually ran. The TUI updates the plan card's progress in place; the count is persisted to disk so it survives session resume. After the FINAL step, write a brief reply summarizing what was done and end the turn. Pass the `stepId` from the plan's steps array, a short `result` (what you did), and optional `notes` for anything surprising (errors, scope changes, follow-ups). This tool doesn't change any files. Don't call it if the plan didn't include structured steps, and don't invent ids that weren't in the original plan. If you only realized at the end that you skipped marking steps, mark them then \u2014 late is still better than never.";
 var REVISE_PLAN_DESCRIPTION = "Surgically replace the REMAINING steps of an in-flight plan. Call this when the user has given feedback at a checkpoint that warrants a structured plan change \u2014 skip a step, swap two steps, add a new step, change risk, etc. Pass: `reason` (one sentence why), `remainingSteps` (the new tail of the plan, replacing whatever steps haven't been done yet), and optional `summary` (updated one-line plan summary). Done steps are NEVER touched \u2014 keep them out of `remainingSteps`. The TUI shows a diff (removed in red, kept in gray, added in green) and the user accepts or rejects. Don't call this for trivial mid-step adjustments \u2014 just keep executing. Don't call submit_plan for revisions either \u2014 that resets the whole plan including completed steps. Use submit_plan only when the entire approach has changed; use revise_plan when the tail needs editing.";
 var STEP_ITEM_SCHEMA = {
   type: "object",
@@ -5166,4 +5208,4 @@ export {
   snapshotBeforeEdits,
   restoreSnapshots
 };
-//# sourceMappingURL=chunk-YJKLNYCP.js.map
+//# sourceMappingURL=chunk-5ZCRXN7S.js.map