agim-cli 1.2.107 → 1.2.109

package/CHANGELOG.md

@@ -4,7 +4,66 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
-## [1.2.107] - 2026-05-29
+## [1.2.109] - 2026-05-29
+
+### Changed (native agent stability, batch P0b — gap ② closed)
+
+- **agent-loop 支持工具调用并行（只读类按 caller 声明的白名单）。** 此前
+  `for (const call of result.toolCalls) { await dispatch(...) }` 强制把模型一次返回
+  的多个工具调用串行起来——5 个独立 `native_read_file` 也要排队跑，长 turn 时延被
+  线性放大。现在 `AgentLoopInput.parallelSafeTools?: ReadonlySet<string>` 让 caller
+  声明哪些工具名是 parallel-safe（无副作用、不共享可变状态、顺序无关）；同一迭代
+  内声明安全的调用走 `Promise.allSettled` 并发，非声明的工具仍串行并在自身派发前
+  把累计 pending 全部 drain（serial barrier，让串行调用看到一个 settled world）。
+- **history 与 toolCalls 的发射严格按原始 call 顺序**——采用 slot 数组在派发完成
+  后一次性按 index 提交，避免完成顺序与 tool_call_id 对齐失序（部分 OpenAI 兼容
+  provider 对此敏感）。
+- **stuck-loop 和 goal-critic 检查相应移到批次发射之后**：仍正确触发（last-3
+  比对 + critic 新增 `lastCriticAtToolCalls` 节流，按"至少每 N 次工具调用一次"
+  的语义保持，不会因为一次 iteration 多个调用而误增触发频率）。
+- **native 内置声明的 parallel-safe 集合**（`NATIVE_PARALLEL_SAFE_TOOLS`）：
+  `native_echo` / `native_now` / `native_random_uuid` / `native_read_file` /
+  `native_list_dir` / `native_glob` / `native_grep` / `native_web_fetch` /
+  `native_web_search` / `mcp__imhub__{read_skill,list_skills,search_memos,
+  memory_list,memory_query}`。`native_exec` / `native_write_file` / 任何写类
+  memo / `push_message` / `ask_user` / `call_agent` / `long_task` /
+  `complete_goal` / 其他 MCP 均不在内（保持串行 + barrier）。
+
+### Tests
+
+4 个集成测试覆盖：并发耗时≈max（非 sum）、缺省 parallelSafeTools 完全串行（backwards-
+compat）、完成顺序 ≠ 原始顺序时 history 仍按原始顺序、serial barrier 在 mixed 序列
+正确 drain。整套 P0a+P0b 测试 45/45 全绿；typecheck 干净。
+
+## [1.2.108] - 2026-05-29
+
+### Changed (native agent stability, batch P0a — 3 of 7 gaps closed)
+
+- **provider 重试现在尊重 `Retry-After` 头 + 指数退避 + jitter（默认 3 次）。**
+  之前是固定 250ms × attempt 线性 backoff、retry=1、不读 `Retry-After`，导致
+  DeepSeek / Moonshot 等 OpenAI-compat 提供方返回 429 时把服务器要求的几秒等待
+  完全无视，250ms 后再撞墙、跳到 fallback provider，吃 cold-start + 模型语义切换。
+  现在默认 4 次尝试（base 250ms、cap 30s、jitter ±20%），`Retry-After` 作为硬下限
+  （秒/HTTP-date 两种 RFC 7231 格式都支持）。`IMHUB_LLM_RETRY_MAX` 覆盖默认（clamp
+  [0,10]）。导出 `parseRetryAfter` / `computeBackoffMs` 两个纯函数 + 13 个回归测试。
+- **stuck-loop 早停升级为 `(name, argsKey, errorType)` 三元组。** v1.2.95 用的是
+  `(name, isError, preview[0..200])`：模型只要让结果文本带个时间戳/换行/计数器就
+  绕过、绕到 max-iter 才停。新版用 `stableStringify(arguments)`（递归排序键，确定性
+  哈希）+ `extractErrorType`（错误首行 ≤80 字符指纹）。同样的工具同样的参数同样的
+  失败 3 次 → 停。导出 `stableStringify` / `extractErrorType` / `isStuckLoop` +
+  16 个回归测试（含明确的"结果抖动不再隐藏死循环"对照）。
+- **MCP tools 缓存加 TTL（默认 5 min）+ 重连改指数退避（base 5s, cap 60s,
+  成功重置）。** v1.2.96 的 cachedTools 永不刷新——上游 MCP server 加新工具/改
+  schema 必须重启 agim 才看得见；重连冷却也是固定 5s，永远 broken 的 server 被
+  5s 一次永久敲。现在：`McpClient.cachedTools` 带 `fetchedAt`，
+  `IMHUB_MCP_TOOLS_TTL_MS` 控制 TTL（0=不缓存），新增 `refresh()` 主动清缓存；
+  `mcp-registry.tryReconnect` 按 `state.reconnectFailures` 指数翻倍冷却，成功
+  立即 reset 为 0。导出 `computeReconnectCooldownMs` + 8 个回归测试。
+
+### Tests
+
+37 new unit tests across 3 files；41 个 P0a 相关测试全绿；typecheck 干净。
+
 
 ### CI
 

package/dist/cli-ui/env-file.js

@@ -198,9 +198,9 @@ export function updateEnvFile(updates) {
 }
 /** Wipe a single key. No-op if missing. */
 export function unsetEnvKey(key) {
+    assertEnvWriteAllowed();
     if (!existsSync(ENV_FILE))
         return;
-    assertEnvWriteAllowed();
     const current = readEnvFile();
     if (!(key in current))
         return;

package/dist/cli-ui/env-file.js.map

@@ -1 +1 @@
-{"version":3,"file":"env-file.js","sourceRoot":"","sources":["../../src/cli-ui/env-file.ts"],"names":[],"mappings":"AAAA,sEAAsE;AACtE,EAAE;AACF,0EAA0E;AAC1E,oEAAoE;AACpE,+DAA+D;AAC/D,EAAE;AACF,8EAA8E;AAC9E,0EAA0E;AAE1E,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,SAAS,EAAE,SAAS,EAAE,UAAU,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,SAAS,CAAA;AACjH,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAChC,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,GAAG,EAAE,MAAM,WAAW,CAAA;AAC9C,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAA;AAEjD,MAAM,CAAC,MAAM,QAAQ,GAAG,IAAI,CAAC,SAAS,EAAE,KAAK,CAAC,CAAA;AAE9C;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,MAAM,UAAU,qBAAqB,CAAC,SAAiB,QAAQ;IAC7D,MAAM,SAAS,GACb,OAAO,CAAC,GAAG,CAAC,QAAQ,KAAK,MAAM;QAC/B,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,MAAM;QACpB,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAA;IAC9B,IAAI,CAAC,SAAS;QAAE,OAAM;IAEtB,uEAAuE;IACvE,uEAAuE;IACvE,wEAAwE;IACxE,yEAAyE;IACzE,IAAI,IAAY,CAAA;IAChB,IAAI,CAAC;QAAC,IAAI,GAAG,YAAY,CAAC,MAAM,EAAE,CAAC,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC;QAAC,IAAI,GAAG,MAAM,EAAE,CAAA;IAAC,CAAC;IAC/D,IAAI,GAAG,GAAG,OAAO,CAAC,MAAM,CAAC,CAAA;IACzB,IAAI,CAAC;QAAC,GAAG,GAAG,YAAY,CAAC,GAAG,CAAC,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC,CAAC,iCAAiC,CAAC,CAAC;IAE3E,IAAI,GAAG,KAAK,IAAI,IAAI,GAAG,CAAC,UAAU,CAAC,IAAI,GAAG,GAAG,CAAC;QAAE,OAAM;IAEtD,MAAM,IAAI,KAAK,CACb,iCAAiC,MAAM,iCAAiC;QACxE,6BAA6B,IAAI,kCAAkC;QACnE,uEAAuE;QACvE,oEAAoE;QACpE,uEAAuE;QACvE,8CAA8C,CAC/C,CAAA;AACH,CAAC;AAED;0EAC0E;AAC1E,MAAM,UAAU,WAAW;IACzB,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC;QAAE,OAAO,EAAE,CAAA;IACpC,MAAM,MAAM,GAA2B,EAAE,CAAA;IACzC,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,YAAY,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAC3C,KAAK,MAAM,IAAI,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YACnC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAA;YAC3B,IAAI,CAAC,OAAO,IAAI,OAAO,CAAC,UAAU,CAAC,GAAG,CAAC;gBAAE,SAAQ;YACjD,MAAM,EAAE,GAAG,OAAO,CAAC,OAAO,CAAC,GAAG,CAAC,CAAA;YAC/B,IAAI,EAAE,IAAI,CAAC;gBAAE,SAAQ;YACrB,MAAM,GAAG,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAA;YACvC,MAAM,GAAG,GAAG,OAAO,CAAC,KAAK,CAAC,EAAE,GAAG,CAAC,CAAC,CAAA;YACjC,MAAM,CAAC,GAAG,CAAC,GAAG,GAAG,CAAA;QACnB,CAAC;IACH,CAAC;IAAC,MAAM,CAAC,CAAC,yBAAyB,CAAC,CAAC;IACrC,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;;iEAGiE;AACjE,MAAM,eAAe,GAAG;IACtB,iBAAiB;IACjB,iBAAiB;IACjB,iBAAiB;IACjB,iBAAiB;IACjB,iBAAiB;IACjB,mBAAmB;IACnB,oBAAoB;CACZ,CAAA;AAEV,MAAM,iBAAiB,GAAG,oCAAoC,CAAA;AAE9D;;;;;iFAKiF;AACjF,MAAM,UAAU,kBAAkB;IAChC,IAAI,CAAC,UAAU,CAAC,iBAAiB,CAAC;QAAE,OAAO,EAAE,CAAA;IAC7C,MAAM,MAAM,GAA2B,EAAE,CAAA;IACzC,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,YAAY,CAAC,iBAAiB,EAAE,OAAO,CAAC,CAAA;QACpD,KAAK,MAAM,IAAI,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YACnC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAA;YAC3B,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,aAAa,CAAC;gBAAE,SAAQ;YAChD,mEAAmE;YACnE,MAAM,GAAG,GAAG,OAAO,CAAC,OAAO,CAAC,qBAAqB,EAAE,EAAE,CAAC,CAAA;YACtD,gEAAgE;YAChE,8DAA8D;YAC9D,8DAA8D;YAC9D,iEAAiE;YACjE,qBAAqB;YACrB,MAAM,MAAM,GAAG,GAAG,CAAC,KAAK,CAAC,cAAc,CAAC,IAAI,EAAE,CAAA;YAC9C,KAAK,MAAM,GAAG,IAAI,MAAM,EAAE,CAAC;gBACzB,MAAM,GAAG,GAAG,GAAG,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,GAAG,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAA;gBAC7E,MAAM,EAAE,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,CAAA;gBAC3B,IAAI,EAAE,IAAI,CAAC;oBAAE,SAAQ;gBACrB,MAAM,GAAG,GAAG,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAA;gBAC5B,MAAM,GAAG,GAAG,GAAG,CAAC,KAAK,CAAC,EAAE,GAAG,CAAC,CAAC,CAAA;gBAC7B,IAAI,GAAG;oBAAE,MAAM,CAAC,GAAG,CAAC,GAAG,GAAG,CAAA;YAC5B,CAAC;QACH,CAAC;IACH,CAAC;IAAC,MAAM,CAAC,CAAC,6CAA6C,CAAC,CAAC;IACzD,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;;;;;;;8EAQ8E;AAC9E,MAAM,UAAU,gBAAgB;IAC9B,MAAM,QAAQ,GAAG,WAAW,EAAE,CAAA;IAC9B,MAAM,QAAQ,GAAG,kBAAkB,EAAE,CAAA;IACrC,MAAM,MAAM,GAA2B,EAAE,GAAG,QAAQ,EAAE,CAAA;IACtD,KAAK,MAAM,CAAC,IAAI,eAAe,EAAE,CAAC;QAChC,IAAI,MAAM,CAAC,CAAC,CAAC,KAAK,SAAS;YAAE,SAAQ;QACrC,IAAI,OAAO,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,QAAQ,IAAI,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC;YAChE,MAAM,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,GAAG,CAAC,CAAC,CAAW,CAAA;YACpC,SAAQ;QACV,CAAC;QACD,IAAI,OAAO,QAAQ,CAAC,CAAC,CAAC,KAAK,QAAQ,IAAI,QAAQ,CAAC,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC;YAC1D,MAAM,CAAC,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAA;QACzB,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;;uBAGuB;AACvB,MAAM,UAAU,qBAAqB,CAAC,GAAW;IAC/C,MAAM,IAAI,GAAG,WAAW,EAAE,CAAA;IAC1B,IAAI,IAAI,CAAC,GAAG,CAAC,KAAK,SAAS;QAAE,OAAO,KAAK,CAAA;IACzC,IAAI,OAAO,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,KAAK,QAAQ,IAAI,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,KAAK,EAAE;QAAE,OAAO,IAAI,CAAA;IAChF,MAAM,IAAI,GAAG,kBAAkB,EAAE,CAAA;IACjC,OAAO,OAAO,IAAI,CAAC,GAAG,CAAC,KAAK,QAAQ,IAAI,IAAI,CAAC,GAAG,CAAC,KAAK,EAAE,CAAA;AAC1D,CAAC;AAED;;+BAE+B;AAC/B,MAAM,UAAU,aAAa,CAAC,OAAkD;IAC9E,qBAAqB,EAAE,CAAA;IACvB,MAAM,OAAO,GAAG,WAAW,EAAE,CAAA;IAC7B,KAAK,MAAM,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;QAC7C,IAAI,CAAC,KAAK,IAAI,IAAI,CAAC,KAAK,SAAS;YAAE,OAAO,OAAO,CAAC,CAAC,CAAC,CAAA;;YAC/C,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC,CAAA;IACrB,CAAC;IACD,SAAS,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;IACjD,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAClE,aAAa,CAAC,QAAQ,EAAE,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAA;IACtE,SAAS,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAA;AAC5B,CAAC;AAED,2CAA2C;AAC3C,MAAM,UAAU,WAAW,CAAC,GAAW;IACrC,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC;QAAE,OAAM;IACjC,qBAAqB,EAAE,CAAA;IACvB,MAAM,OAAO,GAAG,WAAW,EAAE,CAAA;IAC7B,IAAI,CAAC,CAAC,GAAG,IAAI,OAAO,CAAC;QAAE,OAAM;IAC7B,OAAO,OAAO,CAAC,GAAG,CAAC,CAAA;IACnB,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAClE,aAAa,CAAC,QAAQ,EAAE,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAA;IACtE,SAAS,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAA;AAC5B,CAAC;AAED,yDAAyD;AACzD,MAAM,UAAU,aAAa;IAC3B,qBAAqB,EAAE,CAAA;IACvB,IAAI,CAAC;QAAC,UAAU,CAAC,QAAQ,CAAC,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC;AAC3D,CAAC"}
+{"version":3,"file":"env-file.js","sourceRoot":"","sources":["../../src/cli-ui/env-file.ts"],"names":[],"mappings":"AAAA,sEAAsE;AACtE,EAAE;AACF,0EAA0E;AAC1E,oEAAoE;AACpE,+DAA+D;AAC/D,EAAE;AACF,8EAA8E;AAC9E,0EAA0E;AAE1E,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,SAAS,EAAE,SAAS,EAAE,UAAU,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,SAAS,CAAA;AACjH,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAChC,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,GAAG,EAAE,MAAM,WAAW,CAAA;AAC9C,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAA;AAEjD,MAAM,CAAC,MAAM,QAAQ,GAAG,IAAI,CAAC,SAAS,EAAE,KAAK,CAAC,CAAA;AAE9C;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,MAAM,UAAU,qBAAqB,CAAC,SAAiB,QAAQ;IAC7D,MAAM,SAAS,GACb,OAAO,CAAC,GAAG,CAAC,QAAQ,KAAK,MAAM;QAC/B,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,MAAM;QACpB,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAA;IAC9B,IAAI,CAAC,SAAS;QAAE,OAAM;IAEtB,uEAAuE;IACvE,uEAAuE;IACvE,wEAAwE;IACxE,yEAAyE;IACzE,IAAI,IAAY,CAAA;IAChB,IAAI,CAAC;QAAC,IAAI,GAAG,YAAY,CAAC,MAAM,EAAE,CAAC,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC;QAAC,IAAI,GAAG,MAAM,EAAE,CAAA;IAAC,CAAC;IAC/D,IAAI,GAAG,GAAG,OAAO,CAAC,MAAM,CAAC,CAAA;IACzB,IAAI,CAAC;QAAC,GAAG,GAAG,YAAY,CAAC,GAAG,CAAC,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC,CAAC,iCAAiC,CAAC,CAAC;IAE3E,IAAI,GAAG,KAAK,IAAI,IAAI,GAAG,CAAC,UAAU,CAAC,IAAI,GAAG,GAAG,CAAC;QAAE,OAAM;IAEtD,MAAM,IAAI,KAAK,CACb,iCAAiC,MAAM,iCAAiC;QACxE,6BAA6B,IAAI,kCAAkC;QACnE,uEAAuE;QACvE,oEAAoE;QACpE,uEAAuE;QACvE,8CAA8C,CAC/C,CAAA;AACH,CAAC;AAED;0EAC0E;AAC1E,MAAM,UAAU,WAAW;IACzB,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC;QAAE,OAAO,EAAE,CAAA;IACpC,MAAM,MAAM,GAA2B,EAAE,CAAA;IACzC,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,YAAY,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAC3C,KAAK,MAAM,IAAI,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YACnC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAA;YAC3B,IAAI,CAAC,OAAO,IAAI,OAAO,CAAC,UAAU,CAAC,GAAG,CAAC;gBAAE,SAAQ;YACjD,MAAM,EAAE,GAAG,OAAO,CAAC,OAAO,CAAC,GAAG,CAAC,CAAA;YAC/B,IAAI,EAAE,IAAI,CAAC;gBAAE,SAAQ;YACrB,MAAM,GAAG,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAA;YACvC,MAAM,GAAG,GAAG,OAAO,CAAC,KAAK,CAAC,EAAE,GAAG,CAAC,CAAC,CAAA;YACjC,MAAM,CAAC,GAAG,CAAC,GAAG,GAAG,CAAA;QACnB,CAAC;IACH,CAAC;IAAC,MAAM,CAAC,CAAC,yBAAyB,CAAC,CAAC;IACrC,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;;iEAGiE;AACjE,MAAM,eAAe,GAAG;IACtB,iBAAiB;IACjB,iBAAiB;IACjB,iBAAiB;IACjB,iBAAiB;IACjB,iBAAiB;IACjB,mBAAmB;IACnB,oBAAoB;CACZ,CAAA;AAEV,MAAM,iBAAiB,GAAG,oCAAoC,CAAA;AAE9D;;;;;iFAKiF;AACjF,MAAM,UAAU,kBAAkB;IAChC,IAAI,CAAC,UAAU,CAAC,iBAAiB,CAAC;QAAE,OAAO,EAAE,CAAA;IAC7C,MAAM,MAAM,GAA2B,EAAE,CAAA;IACzC,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,YAAY,CAAC,iBAAiB,EAAE,OAAO,CAAC,CAAA;QACpD,KAAK,MAAM,IAAI,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YACnC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAA;YAC3B,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,aAAa,CAAC;gBAAE,SAAQ;YAChD,mEAAmE;YACnE,MAAM,GAAG,GAAG,OAAO,CAAC,OAAO,CAAC,qBAAqB,EAAE,EAAE,CAAC,CAAA;YACtD,gEAAgE;YAChE,8DAA8D;YAC9D,8DAA8D;YAC9D,iEAAiE;YACjE,qBAAqB;YACrB,MAAM,MAAM,GAAG,GAAG,CAAC,KAAK,CAAC,cAAc,CAAC,IAAI,EAAE,CAAA;YAC9C,KAAK,MAAM,GAAG,IAAI,MAAM,EAAE,CAAC;gBACzB,MAAM,GAAG,GAAG,GAAG,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,GAAG,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAA;gBAC7E,MAAM,EAAE,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,CAAA;gBAC3B,IAAI,EAAE,IAAI,CAAC;oBAAE,SAAQ;gBACrB,MAAM,GAAG,GAAG,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAA;gBAC5B,MAAM,GAAG,GAAG,GAAG,CAAC,KAAK,CAAC,EAAE,GAAG,CAAC,CAAC,CAAA;gBAC7B,IAAI,GAAG;oBAAE,MAAM,CAAC,GAAG,CAAC,GAAG,GAAG,CAAA;YAC5B,CAAC;QACH,CAAC;IACH,CAAC;IAAC,MAAM,CAAC,CAAC,6CAA6C,CAAC,CAAC;IACzD,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;;;;;;;8EAQ8E;AAC9E,MAAM,UAAU,gBAAgB;IAC9B,MAAM,QAAQ,GAAG,WAAW,EAAE,CAAA;IAC9B,MAAM,QAAQ,GAAG,kBAAkB,EAAE,CAAA;IACrC,MAAM,MAAM,GAA2B,EAAE,GAAG,QAAQ,EAAE,CAAA;IACtD,KAAK,MAAM,CAAC,IAAI,eAAe,EAAE,CAAC;QAChC,IAAI,MAAM,CAAC,CAAC,CAAC,KAAK,SAAS;YAAE,SAAQ;QACrC,IAAI,OAAO,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,QAAQ,IAAI,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC;YAChE,MAAM,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,GAAG,CAAC,CAAC,CAAW,CAAA;YACpC,SAAQ;QACV,CAAC;QACD,IAAI,OAAO,QAAQ,CAAC,CAAC,CAAC,KAAK,QAAQ,IAAI,QAAQ,CAAC,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC;YAC1D,MAAM,CAAC,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAA;QACzB,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;;uBAGuB;AACvB,MAAM,UAAU,qBAAqB,CAAC,GAAW;IAC/C,MAAM,IAAI,GAAG,WAAW,EAAE,CAAA;IAC1B,IAAI,IAAI,CAAC,GAAG,CAAC,KAAK,SAAS;QAAE,OAAO,KAAK,CAAA;IACzC,IAAI,OAAO,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,KAAK,QAAQ,IAAI,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,KAAK,EAAE;QAAE,OAAO,IAAI,CAAA;IAChF,MAAM,IAAI,GAAG,kBAAkB,EAAE,CAAA;IACjC,OAAO,OAAO,IAAI,CAAC,GAAG,CAAC,KAAK,QAAQ,IAAI,IAAI,CAAC,GAAG,CAAC,KAAK,EAAE,CAAA;AAC1D,CAAC;AAED;;+BAE+B;AAC/B,MAAM,UAAU,aAAa,CAAC,OAAkD;IAC9E,qBAAqB,EAAE,CAAA;IACvB,MAAM,OAAO,GAAG,WAAW,EAAE,CAAA;IAC7B,KAAK,MAAM,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;QAC7C,IAAI,CAAC,KAAK,IAAI,IAAI,CAAC,KAAK,SAAS;YAAE,OAAO,OAAO,CAAC,CAAC,CAAC,CAAA;;YAC/C,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC,CAAA;IACrB,CAAC;IACD,SAAS,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;IACjD,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAClE,aAAa,CAAC,QAAQ,EAAE,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAA;IACtE,SAAS,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAA;AAC5B,CAAC;AAED,2CAA2C;AAC3C,MAAM,UAAU,WAAW,CAAC,GAAW;IACrC,qBAAqB,EAAE,CAAA;IACvB,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC;QAAE,OAAM;IACjC,MAAM,OAAO,GAAG,WAAW,EAAE,CAAA;IAC7B,IAAI,CAAC,CAAC,GAAG,IAAI,OAAO,CAAC;QAAE,OAAM;IAC7B,OAAO,OAAO,CAAC,GAAG,CAAC,CAAA;IACnB,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAClE,aAAa,CAAC,QAAQ,EAAE,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAA;IACtE,SAAS,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAA;AAC5B,CAAC;AAED,yDAAyD;AACzD,MAAM,UAAU,aAAa;IAC3B,qBAAqB,EAAE,CAAA;IACvB,IAAI,CAAC;QAAC,UAAU,CAAC,QAAQ,CAAC,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC;AAC3D,CAAC"}

package/dist/core/llm/agent-loop.d.ts

@@ -1,5 +1,24 @@
 import type { FinishReason, LlmMessage, LlmProvider, LlmUsage, ToolCallRequest, ToolChoice, ToolDef } from './provider-base.js';
 import type { ToolDispatcher } from './tool-dispatcher.js';
+/**
+ * Deterministic JSON.stringify with sorted object keys at every level.
+ * Used by the stuck-loop detector to derive a stable identity key for
+ * tool-call arguments: identical calls hash to identical strings
+ * regardless of key insertion order. Tolerates circular references and
+ * non-JSON-able values by collapsing them to a placeholder.
+ */
+export declare function stableStringify(value: unknown, seen?: WeakSet<object>): string;
+/**
+ * Stable error-type fingerprint of a dispatcher's result text. Returns
+ * the first non-empty line trimmed and truncated to 80 chars when
+ * isError; empty string otherwise. Used by the stuck-loop detector so
+ * "fs_read failed with ENOENT three times" → stuck, but "fs_read
+ * returned different bytes three times" → not stuck.
+ */
+export declare function extractErrorType(isError: boolean, text: string): string;
+/** True iff the last 3 entries are the same tool called with identical
+ *  args resulting in the identical outcome — the stuck-loop signal. */
+export declare function isStuckLoop(reports: Array<Pick<ToolCallReport, 'name' | 'argsKey' | 'errorType'>>): boolean;
 /**
  * Per-tool-call gate. Return 'allow' to dispatch, 'deny' to inject a
  * "user denied" tool result (so the model can react) without running
@@ -92,6 +111,17 @@ export interface AgentLoopInput {
         goalTitle?: string;
         goalBody?: string;
     };
+    /** v1.2.109 — names of tools the caller declares parallel-safe (no
+     *  shared state, no side effects, order-independent). When the model
+     *  returns multiple tool_calls in one iteration, declared-safe calls
+     *  run concurrently via Promise.allSettled; everything else (and any
+     *  tool not in this set) keeps the legacy serial behaviour, with a
+     *  barrier drain before each serial dispatch so serial calls observe
+     *  a settled world. Emission order to history + toolCalls is always
+     *  the original call order (tool_call_id ↔ tool_result alignment).
+     *  Leave undefined → loop is fully serial (no behaviour change vs
+     *  v1.2.108). */
+    parallelSafeTools?: ReadonlySet<string>;
     /** Lifecycle hooks for caller-side observability (heartbeats, progress
      *  push, custom logging). All hooks are best-effort: thrown errors are
      *  swallowed so a faulty observer cannot break the loop. */
@@ -119,6 +149,17 @@ export interface ToolCallReport {
      *  `/heartbeat run-now` style introspection; longer payloads aren't
      *  retained to keep memory + log size bounded. */
     preview: string;
+    /** v1.2.108 — stable JSON of the tool call's arguments (sorted keys),
+     *  used by the stuck-loop detector. Identical calls hash to identical
+     *  strings; model varying timestamps/UUIDs in the result no longer
+     *  hides a stuck loop where the model keeps re-calling with the same
+     *  args. See `stableStringify`. */
+    argsKey: string;
+    /** v1.2.108 — short fingerprint of the error (first non-empty line of
+     *  the result, ≤80 chars) when isError; empty string otherwise. Lets
+     *  the stuck-loop detector distinguish "same failure repeated" from
+     *  "same args returning different bytes". See `extractErrorType`. */
+    errorType: string;
 }
 export interface AgentLoopResult {
     /** Final assistant text the caller should show the user. May be the

package/dist/core/llm/agent-loop.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"agent-loop.d.ts","sourceRoot":"","sources":["../../../src/core/llm/agent-loop.ts"],"names":[],"mappings":"AAgDA,OAAO,KAAK,EAEV,YAAY,EACZ,UAAU,EACV,WAAW,EACX,QAAQ,EACR,eAAe,EACf,UAAU,EACV,OAAO,EACR,MAAM,oBAAoB,CAAA;AAC3B,OAAO,KAAK,EAAoB,cAAc,EAAE,MAAM,sBAAsB,CAAA;AAgB5E;;;;;;;;GAQG;AACH,MAAM,MAAM,YAAY,GAAG,CACzB,IAAI,EAAE,eAAe,EACrB,OAAO,EAAE,MAAM,KACZ,OAAO,CAAC,OAAO,GAAG,MAAM,CAAC,CAAA;AAE9B;;+BAE+B;AAC/B,eAAO,MAAM,eAAe,EAAE,YAAkC,CAAA;AAEhE;;;;;;;;;qDASqD;AACrD;;;4DAG4D;AAC5D,MAAM,MAAM,gBAAgB,GAAG,YAAY,GAAG,gBAAgB,GAAG,SAAS,GAAG,YAAY,GAAG,WAAW,CAAA;AAEvG,MAAM,WAAW,cAAc;IAC7B,QAAQ,EAAE,WAAW,CAAA;IACrB;;iEAE6D;IAC7D,YAAY,CAAC,EAAE,MAAM,CAAA;IACrB;gCAC4B;IAC5B,QAAQ,EAAE,UAAU,EAAE,CAAA;IACtB;yCACqC;IACrC,KAAK,CAAC,EAAE,OAAO,EAAE,CAAA;IACjB,sEAAsE;IACtE,UAAU,CAAC,EAAE,UAAU,CAAA;IAEvB;+DAC2D;IAC3D,aAAa,CAAC,EAAE,MAAM,CAAA;IACtB,+CAA+C;IAC/C,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,wEAAwE;IACxE,MAAM,CAAC,EAAE,WAAW,CAAA;IACpB,qEAAqE;IACrE,WAAW,CAAC,EAAE;QAAE,KAAK,CAAC,EAAE,MAAM,CAAC;QAAC,WAAW,CAAC,EAAE,MAAM,CAAC;QAAC,SAAS,CAAC,EAAE,MAAM,CAAA;KAAE,CAAA;IAE1E;4EACwE;IACxE,QAAQ,CAAC,EAAE,cAAc,CAAA;IACzB,kDAAkD;IAClD,OAAO,CAAC,EAAE,YAAY,CAAA;IAEtB;gFAC4E;IAC5E,KAAK,CAAC,EAAE;QACN,KAAK,EAAE,MAAM,CAAA;QACb,MAAM,CAAC,EAAE,MAAM,CAAA;QACf,MAAM,CAAC,EAAE,MAAM,CAAA;QACf,QAAQ,CAAC,EAAE,MAAM,CAAA;QACjB,OAAO,CAAC,EAAE,MAAM,CAAA;KACjB,CAAA;IAED;;;;8BAI0B;IAC1B,OAAO,CAAC,EAAE;QACR,QAAQ,EAAE,MAAM,CAAA;QAChB,SAAS,EAAE,MAAM,CAAA;QACjB,QAAQ,EAAE,MAAM,CAAA;KACjB,CAAA;IAED;;;;;;sBAMkB;IAClB,YAAY,CAAC,EAAE;QACb,MAAM,CAAC,EAAE,MAAM,CAAA;QACf,SAAS,CAAC,EAAE,MAAM,CAAA;QAClB,QAAQ,CAAC,EAAE,MAAM,CAAA;KAClB,CAAA;IAED;;gEAE4D;IAC5D,KAAK,CAAC,EAAE;QACN;mDAC2C;QAC3C,WAAW,CAAC,EAAE,CAAC,IAAI,EAAE,eAAe,KAAK,IAAI,CAAA;QAC7C,0DAA0D;QAC1D,SAAS,CAAC,EAAE,CAAC,IAAI,EAAE,eAAe,EAAE,OAAO,EAAE;YAAE,OAAO,EAAE,OAAO,CAAC;YAAC,UAAU,EAAE,MAAM,CAAA;SAAE,KAAK,IAAI,CAAA;KAC/F,CAAA;CACF;AAED,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAA;IACZ,iEAAiE;IACjE,QAAQ,EAAE,OAAO,GAAG,MAAM,CAAA;IAC1B,6DAA6D;IAC7D,OAAO,EAAE,OAAO,CAAA;IAChB,mEAAmE;IACnE,MAAM,EAAE,MAAM,CAAA;IACd,UAAU,EAAE,MAAM,CAAA;IAClB;;sDAEkD;IAClD,OAAO,EAAE,MAAM,CAAA;CAChB;AAED,MAAM,WAAW,eAAe;IAC9B;;mEAE+D;IAC/D,IAAI,EAAE,MAAM,CAAA;IACZ;yCACqC;IACrC,UAAU,EAAE,MAAM,CAAA;IAClB,mEAAmE;IACnE,SAAS,EAAE,cAAc,EAAE,CAAA;IAC3B,0EAA0E;IAC1E,KAAK,EAAE,QAAQ,CAAA;IACf,4BAA4B;IAC5B,YAAY,EAAE,gBAAgB,CAAA;IAC9B,6DAA6D;IAC7D,KAAK,CAAC,EAAE,MAAM,CAAA;CACf;AAED;;;;GAIG;AACH,wBAAsB,YAAY,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,eAAe,CAAC,CAsSlF"}
+{"version":3,"file":"agent-loop.d.ts","sourceRoot":"","sources":["../../../src/core/llm/agent-loop.ts"],"names":[],"mappings":"AAgDA,OAAO,KAAK,EAEV,YAAY,EACZ,UAAU,EACV,WAAW,EACX,QAAQ,EACR,eAAe,EACf,UAAU,EACV,OAAO,EACR,MAAM,oBAAoB,CAAA;AAC3B,OAAO,KAAK,EAAoB,cAAc,EAAE,MAAM,sBAAsB,CAAA;AAgB5E;;;;;;GAMG;AACH,wBAAgB,eAAe,CAAC,KAAK,EAAE,OAAO,EAAE,IAAI,GAAE,OAAO,CAAC,MAAM,CAAiB,GAAG,MAAM,CAa7F;AAED;;;;;;GAMG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAIvE;AAED;uEACuE;AACvE,wBAAgB,WAAW,CACzB,OAAO,EAAE,KAAK,CAAC,IAAI,CAAC,cAAc,EAAE,MAAM,GAAG,SAAS,GAAG,WAAW,CAAC,CAAC,GACrE,OAAO,CAQT;AAED;;;;;;;;GAQG;AACH,MAAM,MAAM,YAAY,GAAG,CACzB,IAAI,EAAE,eAAe,EACrB,OAAO,EAAE,MAAM,KACZ,OAAO,CAAC,OAAO,GAAG,MAAM,CAAC,CAAA;AAE9B;;+BAE+B;AAC/B,eAAO,MAAM,eAAe,EAAE,YAAkC,CAAA;AAEhE;;;;;;;;;qDASqD;AACrD;;;4DAG4D;AAC5D,MAAM,MAAM,gBAAgB,GAAG,YAAY,GAAG,gBAAgB,GAAG,SAAS,GAAG,YAAY,GAAG,WAAW,CAAA;AAEvG,MAAM,WAAW,cAAc;IAC7B,QAAQ,EAAE,WAAW,CAAA;IACrB;;iEAE6D;IAC7D,YAAY,CAAC,EAAE,MAAM,CAAA;IACrB;gCAC4B;IAC5B,QAAQ,EAAE,UAAU,EAAE,CAAA;IACtB;yCACqC;IACrC,KAAK,CAAC,EAAE,OAAO,EAAE,CAAA;IACjB,sEAAsE;IACtE,UAAU,CAAC,EAAE,UAAU,CAAA;IAEvB;+DAC2D;IAC3D,aAAa,CAAC,EAAE,MAAM,CAAA;IACtB,+CAA+C;IAC/C,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,wEAAwE;IACxE,MAAM,CAAC,EAAE,WAAW,CAAA;IACpB,qEAAqE;IACrE,WAAW,CAAC,EAAE;QAAE,KAAK,CAAC,EAAE,MAAM,CAAC;QAAC,WAAW,CAAC,EAAE,MAAM,CAAC;QAAC,SAAS,CAAC,EAAE,MAAM,CAAA;KAAE,CAAA;IAE1E;4EACwE;IACxE,QAAQ,CAAC,EAAE,cAAc,CAAA;IACzB,kDAAkD;IAClD,OAAO,CAAC,EAAE,YAAY,CAAA;IAEtB;gFAC4E;IAC5E,KAAK,CAAC,EAAE;QACN,KAAK,EAAE,MAAM,CAAA;QACb,MAAM,CAAC,EAAE,MAAM,CAAA;QACf,MAAM,CAAC,EAAE,MAAM,CAAA;QACf,QAAQ,CAAC,EAAE,MAAM,CAAA;QACjB,OAAO,CAAC,EAAE,MAAM,CAAA;KACjB,CAAA;IAED;;;;8BAI0B;IAC1B,OAAO,CAAC,EAAE;QACR,QAAQ,EAAE,MAAM,CAAA;QAChB,SAAS,EAAE,MAAM,CAAA;QACjB,QAAQ,EAAE,MAAM,CAAA;KACjB,CAAA;IAED;;;;;;sBAMkB;IAClB,YAAY,CAAC,EAAE;QACb,MAAM,CAAC,EAAE,MAAM,CAAA;QACf,SAAS,CAAC,EAAE,MAAM,CAAA;QAClB,QAAQ,CAAC,EAAE,MAAM,CAAA;KAClB,CAAA;IAED;;;;;;;;;qBASiB;IACjB,iBAAiB,CAAC,EAAE,WAAW,CAAC,MAAM,CAAC,CAAA;IAEvC;;gEAE4D;IAC5D,KAAK,CAAC,EAAE;QACN;mDAC2C;QAC3C,WAAW,CAAC,EAAE,CAAC,IAAI,EAAE,eAAe,KAAK,IAAI,CAAA;QAC7C,0DAA0D;QAC1D,SAAS,CAAC,EAAE,CAAC,IAAI,EAAE,eAAe,EAAE,OAAO,EAAE;YAAE,OAAO,EAAE,OAAO,CAAC;YAAC,UAAU,EAAE,MAAM,CAAA;SAAE,KAAK,IAAI,CAAA;KAC/F,CAAA;CACF;AAED,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAA;IACZ,iEAAiE;IACjE,QAAQ,EAAE,OAAO,GAAG,MAAM,CAAA;IAC1B,6DAA6D;IAC7D,OAAO,EAAE,OAAO,CAAA;IAChB,mEAAmE;IACnE,MAAM,EAAE,MAAM,CAAA;IACd,UAAU,EAAE,MAAM,CAAA;IAClB;;sDAEkD;IAClD,OAAO,EAAE,MAAM,CAAA;IACf;;;;uCAImC;IACnC,OAAO,EAAE,MAAM,CAAA;IACf;;;yEAGqE;IACrE,SAAS,EAAE,MAAM,CAAA;CAClB;AAED,MAAM,WAAW,eAAe;IAC9B;;mEAE+D;IAC/D,IAAI,EAAE,MAAM,CAAA;IACZ;yCACqC;IACrC,UAAU,EAAE,MAAM,CAAA;IAClB,mEAAmE;IACnE,SAAS,EAAE,cAAc,EAAE,CAAA;IAC3B,0EAA0E;IAC1E,KAAK,EAAE,QAAQ,CAAA;IACf,4BAA4B;IAC5B,YAAY,EAAE,gBAAgB,CAAA;IAC9B,6DAA6D;IAC7D,KAAK,CAAC,EAAE,MAAM,CAAA;CACf;AAED;;;;GAIG;AACH,wBAAsB,YAAY,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,eAAe,CAAC,CA0UlF"}

package/dist/core/llm/agent-loop.js

@@ -56,6 +56,57 @@ const DEFAULT_TIMEOUT_MS = 5 * 60 * 1000; // 5 min — generous; signal usually
 const CRITIC_MIN_TOOL_CALLS = 5;
 const CRITIC_EVERY_N = 5;
 const CRITIC_MAX_CALLS = 2;
+/**
+ * Deterministic JSON.stringify with sorted object keys at every level.
+ * Used by the stuck-loop detector to derive a stable identity key for
+ * tool-call arguments: identical calls hash to identical strings
+ * regardless of key insertion order. Tolerates circular references and
+ * non-JSON-able values by collapsing them to a placeholder.
+ */
+export function stableStringify(value, seen = new WeakSet()) {
+    if (value === null)
+        return 'null';
+    if (typeof value === 'undefined')
+        return 'null';
+    if (typeof value === 'function')
+        return '"[fn]"';
+    if (typeof value !== 'object')
+        return JSON.stringify(value);
+    if (seen.has(value))
+        return '"[circular]"';
+    seen.add(value);
+    if (Array.isArray(value)) {
+        return '[' + value.map((v) => stableStringify(v, seen)).join(',') + ']';
+    }
+    const obj = value;
+    const keys = Object.keys(obj).sort();
+    return '{' + keys.map((k) => JSON.stringify(k) + ':' + stableStringify(obj[k], seen)).join(',') + '}';
+}
+/**
+ * Stable error-type fingerprint of a dispatcher's result text. Returns
+ * the first non-empty line trimmed and truncated to 80 chars when
+ * isError; empty string otherwise. Used by the stuck-loop detector so
+ * "fs_read failed with ENOENT three times" → stuck, but "fs_read
+ * returned different bytes three times" → not stuck.
+ */
+export function extractErrorType(isError, text) {
+    if (!isError)
+        return '';
+    const firstNonEmpty = text.split('\n').find((line) => line.trim().length > 0) ?? '';
+    return firstNonEmpty.trim().slice(0, 80);
+}
+/** True iff the last 3 entries are the same tool called with identical
+ *  args resulting in the identical outcome — the stuck-loop signal. */
+export function isStuckLoop(reports) {
+    if (reports.length < 3)
+        return false;
+    const a = reports[reports.length - 1];
+    const b = reports[reports.length - 2];
+    const c = reports[reports.length - 3];
+    return a.name === b.name && b.name === c.name
+        && a.argsKey === b.argsKey && b.argsKey === c.argsKey
+        && a.errorType === b.errorType && b.errorType === c.errorType;
+}
 /** Default: skip approval entirely. The agent loop is safe-by-default
  *  for tests and the introspection use case; production AgentAdapters
  *  should pass a real gate. */
@@ -87,6 +138,11 @@ export async function runAgentLoop(input) {
     const usageAcc = {};
     // v1.2.98 — goal-critic per-turn fire counter (capped at CRITIC_MAX_CALLS).
     let criticCallsFired = 0;
+    // v1.2.109 — throttle for the post-batch critic check. With parallel
+    // batching, toolCalls.length now jumps in chunks instead of advancing
+    // by 1 per per-call modulo check. Track the last fire to keep at-most-
+    // every-CRITIC_EVERY_N cadence regardless of batch size.
+    let lastCriticAtToolCalls = 0;
     let iter = 0;
     for (; iter < maxIter; iter++) {
         if (signal.signal.aborted) {
@@ -154,29 +210,50 @@ export async function runAgentLoop(input) {
             toolCalls: result.toolCalls,
             reasoningContent: result.reasoningContent,
         });
-        // Execute each tool call (sequentially — preserves order in the
-        // conversation log and dodges races against the dispatcher's
-        // internal state e.g. MCP session). Concurrent execution lands
-        // when we have a real-world workload that justifies it.
-        for (const call of result.toolCalls) {
+        // v1.2.109 — execute tool calls with optional parallel batching.
+        // Tools whose names appear in input.parallelSafeTools dispatch via
+        // accumulating promises (Promise.allSettled); everything else (and
+        // any tool not declared safe) keeps the legacy serial path with a
+        // barrier drain before each serial dispatch so serial calls observe
+        // a settled world (MCP session state, file system, etc.).
+        // Emission order to history / toolCalls is always the original call
+        // order via slot arrays — some providers reject responses whose
+        // role:'tool' messages aren't laid out next to their assistant
+        // tool_calls in the expected order.
+        const parallelSafe = input.parallelSafeTools;
+        const historySlots = new Array(result.toolCalls.length).fill(null);
+        const reportSlots = new Array(result.toolCalls.length).fill(null);
+        const pendingTasks = [];
+        for (let idx = 0; idx < result.toolCalls.length; idx++) {
             if (signal.signal.aborted)
                 break;
+            const call = result.toolCalls[idx];
+            // Stable arg key precomputed once per call so all three slot
+            // writers (deny / no-dispatcher / dispatched) share the same
+            // identity for the stuck-loop detector.
+            const argsKey = stableStringify(call.arguments);
             const reportBase = {
                 name: call.name,
                 source: 'unknown',
                 preview: '',
+                argsKey,
+                errorType: '',
             };
+            // Approval is ALWAYS serial — it may block on user input, and we
+            // don't want to ask for N approvals simultaneously (the IM UI
+            // would get spammed).
             const decision = await safeApprove(approve, call, iter);
             if (decision === 'deny') {
                 const denyText = `tool call denied by user: ${call.name}`;
-                history.push({ role: 'tool', toolCallId: call.id, content: denyText });
-                toolCalls.push({
+                historySlots[idx] = { role: 'tool', toolCallId: call.id, content: denyText };
+                reportSlots[idx] = {
                     ...reportBase,
                     decision: 'deny',
                     isError: true,
                     durationMs: 0,
                     preview: denyText,
-                });
+                    errorType: extractErrorType(true, denyText),
+                };
                 continue;
             }
             if (!dispatch) {
@@ -184,152 +261,164 @@ export async function runAgentLoop(input) {
                 // theoretically a caller could pass tools+approve without a
                 // dispatcher. Don't crash; tell the model the tool failed.
                 const denyText = `tool call refused: no dispatcher registered`;
-                history.push({ role: 'tool', toolCallId: call.id, content: denyText });
-                toolCalls.push({
+                historySlots[idx] = { role: 'tool', toolCallId: call.id, content: denyText };
+                reportSlots[idx] = {
                     ...reportBase,
                     decision: 'allow',
                     isError: true,
                     durationMs: 0,
                     preview: denyText,
-                });
+                    errorType: extractErrorType(true, denyText),
+                };
                 continue;
             }
-            try {
-                input.hooks?.onToolStart?.(call);
-            }
-            catch { /* observer must not break loop */ }
-            const dt0 = Date.now();
-            let dispatched;
-            try {
-                dispatched = await dispatch(call, signal.signal);
-            }
-            catch (err) {
-                const msg = err instanceof Error ? err.message : String(err);
-                dispatched = { text: `tool threw: ${msg}`, isError: true, source: 'error' };
-                log.warn({
-                    event: 'agent-loop.dispatch_threw',
-                    tool: call.name, err: msg,
-                });
-            }
-            const durationMs = Date.now() - dt0;
-            try {
-                input.hooks?.onToolEnd?.(call, { isError: !!dispatched.isError, durationMs });
-            }
-            catch { /* observer must not break loop */ }
-            history.push({
-                role: 'tool',
-                toolCallId: call.id,
-                content: dispatched.text,
-            });
-            toolCalls.push({
-                name: call.name,
-                decision: 'allow',
-                isError: dispatched.isError,
-                source: dispatched.source ?? 'unknown',
-                durationMs,
-                preview: dispatched.text.slice(0, 200),
-            });
-            // v1.2.95 — stuck-loop detector. If the last three tool reports
-            // share name + error-flag + preview content, the model is calling
-            // the SAME tool with the SAME outcome — almost always a sign it's
-            // re-running a failing SQL / fetch / command without changing
-            // strategy. Break out early so we don't waste the full
-            // maxIterations budget on a stuck model, and so the caller can
-            // surface a much more focused "you're stuck on X" recap.
-            //
-            // Three consecutive matches is the smallest threshold that
-            // unambiguously says "loop" without being triggered by a
-            // legitimate "two reads on the same file" sequence. The preview
-            // is already truncated to 200 chars at push time, so this
-            // comparison is bounded.
-            if (toolCalls.length >= 3) {
-                const a = toolCalls[toolCalls.length - 1];
-                const b = toolCalls[toolCalls.length - 2];
-                const c = toolCalls[toolCalls.length - 3];
-                if (a.name === b.name && b.name === c.name
-                    && a.isError === b.isError && b.isError === c.isError
-                    && a.preview === b.preview && b.preview === c.preview) {
+            // Capture dispatch into an async closure so it can be awaited
+            // either immediately (serial path) or later via Promise.allSettled
+            // (parallel batch path). Captures idx by closure so the writer
+            // hits the correct slot regardless of completion order.
+            const dispatchCall = dispatch;
+            const runDispatch = async () => {
+                try {
+                    input.hooks?.onToolStart?.(call);
+                }
+                catch { /* observer must not break loop */ }
+                const t0 = Date.now();
+                let dispatched;
+                try {
+                    dispatched = await dispatchCall(call, signal.signal);
+                }
+                catch (err) {
+                    const msg = err instanceof Error ? err.message : String(err);
+                    dispatched = { text: `tool threw: ${msg}`, isError: true, source: 'error' };
                     log.warn({
-                        event: 'agent-loop.stuck_loop_detected',
-                        tool: a.name,
-                        isError: a.isError,
-                        iter,
-                        totalToolCalls: toolCalls.length,
-                        previewSnippet: (a.preview ?? '').slice(0, 80),
-                    }, `stuck loop: 3× ${a.name} with identical outcome — early stop at iter ${iter}`);
-                    signal.cleanup();
-                    return {
-                        text: '',
-                        iterations: iter + 1,
-                        toolCalls,
-                        usage: usageAcc,
-                        finishReason: 'stuck_loop',
-                    };
+                        event: 'agent-loop.dispatch_threw',
+                        tool: call.name, err: msg,
+                    });
                 }
+                const durationMs = Date.now() - t0;
+                try {
+                    input.hooks?.onToolEnd?.(call, { isError: !!dispatched.isError, durationMs });
+                }
+                catch { /* observer must not break loop */ }
+                historySlots[idx] = {
+                    role: 'tool',
+                    toolCallId: call.id,
+                    content: dispatched.text,
+                };
+                reportSlots[idx] = {
+                    name: call.name,
+                    decision: 'allow',
+                    isError: dispatched.isError,
+                    source: dispatched.source ?? 'unknown',
+                    durationMs,
+                    preview: dispatched.text.slice(0, 200),
+                    argsKey,
+                    errorType: extractErrorType(dispatched.isError, dispatched.text),
+                };
+            };
+            const isParallel = !!parallelSafe?.has(call.name);
+            if (isParallel) {
+                pendingTasks.push(runDispatch());
+            }
+            else {
+                // Serial barrier: drain accumulated parallel work first.
+                if (pendingTasks.length > 0) {
+                    await Promise.allSettled(pendingTasks.splice(0));
+                }
+                await runDispatch();
             }
-            // v1.2.98 / v1.2.99 — goal-critic check (semantic off-track).
-            // Fires when ALL of:
-            //   · caller supplied a criticAnchor
-            //   · critic mode is 'always' OR ('goal-only' AND anchor has a
-            //     long-task goal title). Per-turn prompts alone won't
-            //     trigger in goal-only mode — this is the v1.2.99 fix for
-            //     CR finding A2 (default ON in v1.2.98 surprised the cost
-            //     budget). Set IMHUB_NATIVE_CRITIC=always to re-enable for
-            //     every multi-step turn.
-            //   · ≥ CRITIC_MIN_TOOL_CALLS tool calls this turn
-            //   · haven't already fired CRITIC_MAX_CALLS times
-            //   · on a trigger mark (every CRITIC_EVERY_N tool calls)
-            // The critic itself decides whether to break. It returns
-            // onTrack=true on errors so provider hiccups never accidentally
-            // kill a working turn.
-            const criticMode = resolveCriticMode();
-            const criticGateOpen = criticMode === 'always'
-                || (criticMode === 'goal-only' && !!input.criticAnchor?.goalTitle?.trim());
-            if (input.criticAnchor
-                && criticGateOpen
-                && toolCalls.length >= CRITIC_MIN_TOOL_CALLS
-                && criticCallsFired < CRITIC_MAX_CALLS
-                && (toolCalls.length - CRITIC_MIN_TOOL_CALLS) % CRITIC_EVERY_N === 0) {
-                criticCallsFired += 1;
-                const recent = toolCalls.slice(-10);
-                const verdict = await runGoalCritic({
-                    prompt: input.criticAnchor.prompt,
-                    goalTitle: input.criticAnchor.goalTitle,
-                    goalBody: input.criticAnchor.goalBody,
-                    recentToolCalls: recent,
-                }, { signal: signal.signal });
-                log.info({
-                    event: 'agent-loop.goal_critic_verdict',
+        }
+        // End-of-iteration drain for any tail parallel tasks.
+        if (pendingTasks.length > 0) {
+            await Promise.allSettled(pendingTasks.splice(0));
+        }
+        // Commit slots in original call order so tool_call_id ↔ tool_result
+        // alignment is preserved regardless of dispatch completion order.
+        // Slots may be null if the loop aborted mid-iteration — skip those.
+        for (let idx = 0; idx < result.toolCalls.length; idx++) {
+            const h = historySlots[idx];
+            const r = reportSlots[idx];
+            if (h)
+                history.push(h);
+            if (r)
+                toolCalls.push(r);
+        }
+        // v1.2.108 — stuck-loop detector (now post-batch). Keyed on
+        // (name, argsKey, errorType) of the last 3 entries; with parallel
+        // batching this still fires correctly when 3 identical calls land
+        // in a row (whether in one batch or across batches).
+        if (isStuckLoop(toolCalls)) {
+            const a = toolCalls[toolCalls.length - 1];
+            log.warn({
+                event: 'agent-loop.stuck_loop_detected',
+                tool: a.name,
+                isError: a.isError,
+                argsKeySnippet: a.argsKey.slice(0, 200),
+                errorType: a.errorType,
+                iter,
+                totalToolCalls: toolCalls.length,
+            }, `stuck loop: 3× ${a.name} with identical args + outcome — early stop at iter ${iter}`);
+            signal.cleanup();
+            return {
+                text: '',
+                iterations: iter + 1,
+                toolCalls,
+                usage: usageAcc,
+                finishReason: 'stuck_loop',
+            };
+        }
+        // v1.2.98 / v1.2.99 / v1.2.109 — goal-critic check (semantic
+        // off-track), now post-batch with a since-last-fire throttle so
+        // cadence stays at-most-every-CRITIC_EVERY_N tool calls regardless
+        // of how many calls landed in this iteration's batch.
+        const criticMode = resolveCriticMode();
+        const criticGateOpen = criticMode === 'always'
+            || (criticMode === 'goal-only' && !!input.criticAnchor?.goalTitle?.trim());
+        if (input.criticAnchor
+            && criticGateOpen
+            && toolCalls.length >= CRITIC_MIN_TOOL_CALLS
+            && criticCallsFired < CRITIC_MAX_CALLS
+            && toolCalls.length - lastCriticAtToolCalls >= CRITIC_EVERY_N) {
+            criticCallsFired += 1;
+            lastCriticAtToolCalls = toolCalls.length;
+            const recent = toolCalls.slice(-10);
+            const verdict = await runGoalCritic({
+                prompt: input.criticAnchor.prompt,
+                goalTitle: input.criticAnchor.goalTitle,
+                goalBody: input.criticAnchor.goalBody,
+                recentToolCalls: recent,
+            }, { signal: signal.signal });
+            log.info({
+                event: 'agent-loop.goal_critic_verdict',
+                iter,
+                totalToolCalls: toolCalls.length,
+                onTrack: verdict.onTrack,
+                reason: verdict.reason,
+                modelUsed: verdict.modelUsed,
+            });
+            if (!verdict.onTrack) {
+                log.warn({
+                    event: 'agent-loop.off_track_detected',
                     iter,
-                    totalToolCalls: toolCalls.length,
-                    onTrack: verdict.onTrack,
                     reason: verdict.reason,
-                    modelUsed: verdict.modelUsed,
-                });
-                if (!verdict.onTrack) {
-                    log.warn({
-                        event: 'agent-loop.off_track_detected',
-                        iter,
-                        reason: verdict.reason,
-                        redirect: verdict.redirect,
-                    }, `goal-critic flagged off-track at iter ${iter}: ${verdict.reason}`);
-                    signal.cleanup();
-                    return {
-                        text: '',
-                        iterations: iter + 1,
-                        toolCalls,
-                        usage: usageAcc,
-                        finishReason: 'off_track',
-                        // Smuggle the critic's reason + redirect into `error` so
-                        // the native adapter can surface it verbatim in the recap
-                        // without re-importing goal-critic. Yes, abusing `error`
-                        // for non-error info is mildly ugly; alternative is adding
-                        // a new field to AgentLoopResult — overkill for one knob.
-                        error: verdict.redirect
-                            ? `${verdict.reason} || redirect: ${verdict.redirect}`
-                            : verdict.reason,
-                    };
-                }
+                    redirect: verdict.redirect,
+                }, `goal-critic flagged off-track at iter ${iter}: ${verdict.reason}`);
+                signal.cleanup();
+                return {
+                    text: '',
+                    iterations: iter + 1,
+                    toolCalls,
+                    usage: usageAcc,
+                    finishReason: 'off_track',
+                    // Smuggle the critic's reason + redirect into `error` so
+                    // the native adapter can surface it verbatim in the recap
+                    // without re-importing goal-critic. Yes, abusing `error`
+                    // for non-error info is mildly ugly; alternative is adding
+                    // a new field to AgentLoopResult — overkill for one knob.
+                    error: verdict.redirect
+                        ? `${verdict.reason} || redirect: ${verdict.redirect}`
+                        : verdict.reason,
+                };
             }
         }
         // Loop tail → next iteration calls provider with appended history.