nexus-agents 4.3.3 → 4.3.5

package/dist/{chunk-XABDFFQV.js → chunk-UI3WYJSZ.js}

@@ -2,7 +2,7 @@ import {
   createLogger,
   err,
   ok
-} from "./chunk-TMYLHKLJ.js";
+} from "./chunk-U4LM5Q2L.js";
 
 // src/scm/token-resolver.ts
 import { execFile } from "child_process";
@@ -87,4 +87,4 @@ export {
   hasToken,
   getTokenEnvVars
 };
-//# sourceMappingURL=chunk-XABDFFQV.js.map
+//# sourceMappingURL=chunk-UI3WYJSZ.js.map

package/dist/{chunk-Y3WYSLFZ.js → chunk-VWXQHGQ2.js}

@@ -8,7 +8,7 @@ import {
   getErrorMessage,
   ok,
   parseBoolEnv
-} from "./chunk-TMYLHKLJ.js";
+} from "./chunk-U4LM5Q2L.js";
 
 // src/cli/research-helpers-io.ts
 import * as fs2 from "fs/promises";
@@ -146,4 +146,4 @@ export {
   loadPapersRegistry,
   savePapersRegistry
 };
-//# sourceMappingURL=chunk-Y3WYSLFZ.js.map
+//# sourceMappingURL=chunk-VWXQHGQ2.js.map

package/dist/{chunk-DVKS77O5.js → chunk-XIIUIBSX.js}

@@ -76,6 +76,10 @@ function extractCount(value) {
   if (value !== null && typeof value === "object") {
     const v = value;
     if (v.ok === true && typeof v.value === "number") return v.value;
+    if (v.ok === false) {
+      const detail = v.error instanceof Error ? v.error.message : typeof v.error === "string" ? v.error : "unknown error";
+      throw new Error(`backend count() failed: ${detail}`);
+    }
   }
   return 0;
 }
@@ -89,4 +93,4 @@ export {
   StatsOnlyAdapter,
   ensureSharedMemoryRegistry
 };
-//# sourceMappingURL=chunk-DVKS77O5.js.map
+//# sourceMappingURL=chunk-XIIUIBSX.js.map

package/dist/{chunk-DVKS77O5.js.map → chunk-XIIUIBSX.js.map}

@@ -1 +1 @@
-{"version":3,"sources":["../src/mcp/tools/tool-memory-registry-adapters.ts"],"sourcesContent":["/**\n * Phase 5 of #2766 — thin {@link IMemoryBackend} adapters for tool-memory\n * backends so each one becomes discoverable through `getMemoryRegistry()`.\n *\n * Scope is intentionally narrow: each adapter exposes the underlying\n * backend's row count via `stats()` so `memory_stats` can read all four\n * domains from the unified registry. `read`/`write`/`query`/`delete` are\n * left as escape hatches — callers wanting CRUD continue to use the\n * underlying backend directly (HybridMemoryBackend, AgenticMemoryBackend,\n * etc.). Fully folding the storage into nexus-memory's SqliteBackend is\n * a follow-up Phase 5.1.\n *\n * @module mcp/tools/tool-memory-registry-adapters\n */\n\nimport { MemoryRegistry, hasMemoryRegistry, setMemoryRegistry } from 'nexus-memory';\nimport type { BackendStats, CliName, IMemoryBackend, QueryFilter, WriteMeta } from 'nexus-memory';\n\nimport { nexusDataPath } from '../../config/nexus-data-dir.js';\n\n/** Default cap when a consumer omits `limit` in `query()`. */\nconst DEFAULT_SEARCH_LIMIT = 10;\n\n/**\n * Pull the free-text search term out of a {@link QueryFilter}. Returns\n * `null` when the filter doesn't carry one, so callers can skip the search\n * dispatch entirely. The convention is `filter.where.text` — see\n * docs/architecture/memory-context-retrieval.md (Phase 1 of #2792).\n */\nfunction extractSearchText(filter?: QueryFilter<unknown>): string | null {\n  const where: unknown = filter?.where;\n  if (where === null || where === undefined || typeof where !== 'object') return null;\n  if (!('text' in where)) return null;\n  const candidate = where.text;\n  return typeof candidate === 'string' && candidate !== '' ? candidate : null;\n}\n\n/** Minimal \"count + close\" shape every tool-memory backend implements. */\nexport interface CountableBackend {\n  /** Returns total row count (the heart of `memory_stats`). */\n  count(): unknown;\n  /** Idempotent close. */\n  close?(): Promise<void> | void;\n  /**\n   * Optional native search surface used by {@link StatsOnlyAdapter.query}\n   * (Phase 1 of #2792). When attached from `tool-memory.ts`, each backend\n   * wires this to its idiomatic search call — `recallBySubject` for\n   * beliefs, `searchAgentic` for A-MEM, `retrieveByPriority` for adaptive,\n   * etc. Returns a flat array; the adapter is responsible for unwrapping\n   * Result types and projecting to a useful shape (typically the raw\n   * entry object, since consumers will project further).\n   */\n  search?(query: string, limit: number): Promise<readonly unknown[]>;\n}\n\n/**\n * Adapter that surfaces `count()` from an arbitrary tool-memory backend\n * as an `IMemoryBackend`. Methods other than `stats()` and `close()`\n * delegate to no-ops or rejections — call sites doing actual CRUD must\n * keep using the underlying backend directly until a deeper migration\n * folds the storage in.\n */\nexport class StatsOnlyAdapter implements IMemoryBackend<string, unknown> {\n  readonly domain: string;\n  private readonly backend: CountableBackend;\n\n  constructor(domain: string, backend: CountableBackend) {\n    this.domain = domain;\n    this.backend = backend;\n  }\n\n  read(_key: string): Promise<unknown> {\n    // Reads through the unified registry aren't wired yet. Callers wanting\n    // actual data should fetch via the underlying backend.\n    return Promise.resolve(undefined);\n  }\n\n  write(_key: string, _value: unknown, _meta?: WriteMeta): Promise<void> {\n    return Promise.reject(\n      new Error(\n        `nexus-memory: domain \"${this.domain}\" is attached as stats-only; ` +\n          `write through the underlying backend instead`\n      )\n    );\n  }\n\n  /**\n   * Phase 1 of #2792: real query fan-out via the backend's native search.\n   * Honors the free-text convention `filter.where.text`. Without a search\n   * callback on the underlying backend (or without a text term), returns\n   * `[]` — consumers that need full coverage must wire `search()` on every\n   * backend they attach. Search exceptions are swallowed; the registry's\n   * `memory_stats` consumer relies on `query()` never throwing.\n   */\n  async query(filter?: QueryFilter<unknown>): Promise<readonly unknown[]> {\n    if (this.backend.search === undefined) return [];\n    const text = extractSearchText(filter);\n    if (text === null) return [];\n    const limit = filter?.limit ?? DEFAULT_SEARCH_LIMIT;\n    try {\n      return await this.backend.search(text, limit);\n    } catch {\n      return [];\n    }\n  }\n\n  delete(_key: string): Promise<boolean> {\n    return Promise.resolve(false);\n  }\n\n  async stats(): Promise<BackendStats> {\n    const rawCount = await Promise.resolve(this.backend.count());\n    const count = extractCount(rawCount);\n    return {\n      domain: this.domain,\n      count,\n      oldestTimestamp: null,\n      newestTimestamp: null,\n    };\n  }\n\n  async close(): Promise<void> {\n    if (this.backend.close !== undefined) {\n      await Promise.resolve(this.backend.close());\n    }\n  }\n}\n\n/**\n * Normalize different backend `count()` return shapes:\n * - `number` (e.g., `MobiMem.profile.getEntryCount()`)\n * - `Result<number, MemoryError>` (HybridMemoryBackend)\n * - `Promise<number | Result<number, MemoryError>>` (after await)\n * Anything unrecognized falls back to 0 — the adapter must never throw on\n * a stats read because that would break `memory_stats` for the whole pipeline.\n */\nfunction extractCount(value: unknown): number {\n  if (typeof value === 'number') return value;\n  if (value !== null && typeof value === 'object') {\n    const v = value as { ok?: boolean; value?: unknown };\n    if (v.ok === true && typeof v.value === 'number') return v.value;\n  }\n  return 0;\n}\n\n/** Convenience tag the adapter passes to `WriteMeta.cli` for future writes. */\nexport type AdapterCli = CliName;\n\n// ============================================================================\n// Shared MemoryRegistry path injection (#3995)\n// ============================================================================\n\n/**\n * Ensure the process-wide shared {@link MemoryRegistry} is constructed with\n * the canonical nexus-agents data path (#3995).\n *\n * nexus-memory ships a dep-free `resolveDefaultDbPath()` fallback that\n * re-implements `~/.nexus-agents/memory/memory.db` inline so the package stays\n * free of inter-package imports (it is reused by `nexus-eval-*`). In\n * production nexus-agents we want the canonical {@link nexusDataPath} resolver\n * instead — it adds sandbox detection, the per-repo/cross-repo split, the\n * homedir-unwritable fallback, and `.gitignore` auto-wiring that the dep-free\n * fallback cannot know about. We keep nexus-memory dep-free by doing the\n * resolution HERE (on the nexus-agents side) and injecting the resolved path\n * as a plain string via `setMemoryRegistry`.\n *\n * No-op when a registry is already present (`hasMemoryRegistry()`): that means\n * either a test injected an in-memory registry, or a prior call already wired\n * the canonical path. Checking presence — rather than calling\n * `getMemoryRegistry()` — is what lets us win the race against nexus-memory's\n * lazy default-path initialization without ever triggering it. Call this\n * before the first `getMemoryRegistry()` in any production path that attaches\n * a backend.\n */\nexport function ensureSharedMemoryRegistry(): void {\n  if (hasMemoryRegistry()) return;\n  // `memory` is intentionally cross-repo (not in PER_REPO_SUBDIRS), so this\n  // resolves to `~/.nexus-agents/memory/memory.db` on a normal machine and to\n  // the sandbox / per-repo-fallback root under those modes.\n  const dbPath = nexusDataPath('memory', 'memory.db');\n  setMemoryRegistry(new MemoryRegistry({ dbPath }));\n}\n"],"mappings":";;;;;;;;;;AAqBA,IAAM,uBAAuB;AAQ7B,SAAS,kBAAkB,QAA8C;AACvE,QAAM,QAAiB,QAAQ;AAC/B,MAAI,UAAU,QAAQ,UAAU,UAAa,OAAO,UAAU,SAAU,QAAO;AAC/E,MAAI,EAAE,UAAU,OAAQ,QAAO;AAC/B,QAAM,YAAY,MAAM;AACxB,SAAO,OAAO,cAAc,YAAY,cAAc,KAAK,YAAY;AACzE;AA2BO,IAAM,mBAAN,MAAkE;AAAA,EAC9D;AAAA,EACQ;AAAA,EAEjB,YAAY,QAAgB,SAA2B;AACrD,SAAK,SAAS;AACd,SAAK,UAAU;AAAA,EACjB;AAAA,EAEA,KAAK,MAAgC;AAGnC,WAAO,QAAQ,QAAQ,MAAS;AAAA,EAClC;AAAA,EAEA,MAAM,MAAc,QAAiB,OAAkC;AACrE,WAAO,QAAQ;AAAA,MACb,IAAI;AAAA,QACF,yBAAyB,KAAK,MAAM;AAAA,MAEtC;AAAA,IACF;AAAA,EACF;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,EAUA,MAAM,MAAM,QAA4D;AACtE,QAAI,KAAK,QAAQ,WAAW,OAAW,QAAO,CAAC;AAC/C,UAAM,OAAO,kBAAkB,MAAM;AACrC,QAAI,SAAS,KAAM,QAAO,CAAC;AAC3B,UAAM,QAAQ,QAAQ,SAAS;AAC/B,QAAI;AACF,aAAO,MAAM,KAAK,QAAQ,OAAO,MAAM,KAAK;AAAA,IAC9C,QAAQ;AACN,aAAO,CAAC;AAAA,IACV;AAAA,EACF;AAAA,EAEA,OAAO,MAAgC;AACrC,WAAO,QAAQ,QAAQ,KAAK;AAAA,EAC9B;AAAA,EAEA,MAAM,QAA+B;AACnC,UAAM,WAAW,MAAM,QAAQ,QAAQ,KAAK,QAAQ,MAAM,CAAC;AAC3D,UAAM,QAAQ,aAAa,QAAQ;AACnC,WAAO;AAAA,MACL,QAAQ,KAAK;AAAA,MACb;AAAA,MACA,iBAAiB;AAAA,MACjB,iBAAiB;AAAA,IACnB;AAAA,EACF;AAAA,EAEA,MAAM,QAAuB;AAC3B,QAAI,KAAK,QAAQ,UAAU,QAAW;AACpC,YAAM,QAAQ,QAAQ,KAAK,QAAQ,MAAM,CAAC;AAAA,IAC5C;AAAA,EACF;AACF;AAUA,SAAS,aAAa,OAAwB;AAC5C,MAAI,OAAO,UAAU,SAAU,QAAO;AACtC,MAAI,UAAU,QAAQ,OAAO,UAAU,UAAU;AAC/C,UAAM,IAAI;AACV,QAAI,EAAE,OAAO,QAAQ,OAAO,EAAE,UAAU,SAAU,QAAO,EAAE;AAAA,EAC7D;AACA,SAAO;AACT;AA+BO,SAAS,6BAAmC;AACjD,MAAI,kBAAkB,EAAG;AAIzB,QAAM,SAAS,cAAc,UAAU,WAAW;AAClD,oBAAkB,IAAI,eAAe,EAAE,OAAO,CAAC,CAAC;AAClD;","names":[]}
+{"version":3,"sources":["../src/mcp/tools/tool-memory-registry-adapters.ts"],"sourcesContent":["/**\n * Phase 5 of #2766 — thin {@link IMemoryBackend} adapters for tool-memory\n * backends so each one becomes discoverable through `getMemoryRegistry()`.\n *\n * Scope is intentionally narrow: each adapter exposes the underlying\n * backend's row count via `stats()` so `memory_stats` can read all four\n * domains from the unified registry. `read`/`write`/`query`/`delete` are\n * left as escape hatches — callers wanting CRUD continue to use the\n * underlying backend directly (HybridMemoryBackend, AgenticMemoryBackend,\n * etc.). Fully folding the storage into nexus-memory's SqliteBackend is\n * a follow-up Phase 5.1.\n *\n * @module mcp/tools/tool-memory-registry-adapters\n */\n\nimport { MemoryRegistry, hasMemoryRegistry, setMemoryRegistry } from 'nexus-memory';\nimport type { BackendStats, CliName, IMemoryBackend, QueryFilter, WriteMeta } from 'nexus-memory';\n\nimport { nexusDataPath } from '../../config/nexus-data-dir.js';\n\n/** Default cap when a consumer omits `limit` in `query()`. */\nconst DEFAULT_SEARCH_LIMIT = 10;\n\n/**\n * Pull the free-text search term out of a {@link QueryFilter}. Returns\n * `null` when the filter doesn't carry one, so callers can skip the search\n * dispatch entirely. The convention is `filter.where.text` — see\n * docs/architecture/memory-context-retrieval.md (Phase 1 of #2792).\n */\nfunction extractSearchText(filter?: QueryFilter<unknown>): string | null {\n  const where: unknown = filter?.where;\n  if (where === null || where === undefined || typeof where !== 'object') return null;\n  if (!('text' in where)) return null;\n  const candidate = where.text;\n  return typeof candidate === 'string' && candidate !== '' ? candidate : null;\n}\n\n/** Minimal \"count + close\" shape every tool-memory backend implements. */\nexport interface CountableBackend {\n  /** Returns total row count (the heart of `memory_stats`). */\n  count(): unknown;\n  /** Idempotent close. */\n  close?(): Promise<void> | void;\n  /**\n   * Optional native search surface used by {@link StatsOnlyAdapter.query}\n   * (Phase 1 of #2792). When attached from `tool-memory.ts`, each backend\n   * wires this to its idiomatic search call — `recallBySubject` for\n   * beliefs, `searchAgentic` for A-MEM, `retrieveByPriority` for adaptive,\n   * etc. Returns a flat array; the adapter is responsible for unwrapping\n   * Result types and projecting to a useful shape (typically the raw\n   * entry object, since consumers will project further).\n   */\n  search?(query: string, limit: number): Promise<readonly unknown[]>;\n}\n\n/**\n * Adapter that surfaces `count()` from an arbitrary tool-memory backend\n * as an `IMemoryBackend`. Methods other than `stats()` and `close()`\n * delegate to no-ops or rejections — call sites doing actual CRUD must\n * keep using the underlying backend directly until a deeper migration\n * folds the storage in.\n */\nexport class StatsOnlyAdapter implements IMemoryBackend<string, unknown> {\n  readonly domain: string;\n  private readonly backend: CountableBackend;\n\n  constructor(domain: string, backend: CountableBackend) {\n    this.domain = domain;\n    this.backend = backend;\n  }\n\n  read(_key: string): Promise<unknown> {\n    // Reads through the unified registry aren't wired yet. Callers wanting\n    // actual data should fetch via the underlying backend.\n    return Promise.resolve(undefined);\n  }\n\n  write(_key: string, _value: unknown, _meta?: WriteMeta): Promise<void> {\n    return Promise.reject(\n      new Error(\n        `nexus-memory: domain \"${this.domain}\" is attached as stats-only; ` +\n          `write through the underlying backend instead`\n      )\n    );\n  }\n\n  /**\n   * Phase 1 of #2792: real query fan-out via the backend's native search.\n   * Honors the free-text convention `filter.where.text`. Without a search\n   * callback on the underlying backend (or without a text term), returns\n   * `[]` — consumers that need full coverage must wire `search()` on every\n   * backend they attach. Search exceptions are swallowed; the registry's\n   * `memory_stats` consumer relies on `query()` never throwing.\n   */\n  async query(filter?: QueryFilter<unknown>): Promise<readonly unknown[]> {\n    if (this.backend.search === undefined) return [];\n    const text = extractSearchText(filter);\n    if (text === null) return [];\n    const limit = filter?.limit ?? DEFAULT_SEARCH_LIMIT;\n    try {\n      return await this.backend.search(text, limit);\n    } catch {\n      return [];\n    }\n  }\n\n  delete(_key: string): Promise<boolean> {\n    return Promise.resolve(false);\n  }\n\n  async stats(): Promise<BackendStats> {\n    const rawCount = await Promise.resolve(this.backend.count());\n    const count = extractCount(rawCount);\n    return {\n      domain: this.domain,\n      count,\n      oldestTimestamp: null,\n      newestTimestamp: null,\n    };\n  }\n\n  async close(): Promise<void> {\n    if (this.backend.close !== undefined) {\n      await Promise.resolve(this.backend.close());\n    }\n  }\n}\n\n/**\n * Normalize different backend `count()` return shapes:\n * - `number` (e.g., `MobiMem.profile.getEntryCount()`)\n * - `Result<number, MemoryError>` (HybridMemoryBackend)\n * - `Promise<number | Result<number, MemoryError>>` (after await)\n * Anything unrecognized falls back to 0 — the adapter must never throw on\n * a stats read because that would break `memory_stats` for the whole pipeline.\n */\nfunction extractCount(value: unknown): number {\n  if (typeof value === 'number') return value;\n  if (value !== null && typeof value === 'object') {\n    const v = value as { ok?: boolean; value?: unknown; error?: unknown };\n    if (v.ok === true && typeof v.value === 'number') return v.value;\n    // #4827: a FAILED Result must NOT collapse to 0. Doing so made a corrupt\n    // or locked backend report `{count: 0, error: null}` — byte-identical to a\n    // healthy empty store — and left `collectRegistryStats`'s catch\n    // (memory-stats.ts:213) unreachable, so `RegistryDomainStats.error`\n    // documented a state nothing could produce. Throwing lets that catch,\n    // which is already written and correct, do its job.\n    if (v.ok === false) {\n      const detail =\n        v.error instanceof Error\n          ? v.error.message\n          : typeof v.error === 'string'\n            ? v.error\n            : 'unknown error';\n      throw new Error(`backend count() failed: ${detail}`);\n    }\n  }\n  // An unrecognised shape is genuinely unknown, not a reported failure — 0 is\n  // the long-standing behaviour and callers have no error to surface.\n  return 0;\n}\n\n/** Convenience tag the adapter passes to `WriteMeta.cli` for future writes. */\nexport type AdapterCli = CliName;\n\n// ============================================================================\n// Shared MemoryRegistry path injection (#3995)\n// ============================================================================\n\n/**\n * Ensure the process-wide shared {@link MemoryRegistry} is constructed with\n * the canonical nexus-agents data path (#3995).\n *\n * nexus-memory ships a dep-free `resolveDefaultDbPath()` fallback that\n * re-implements `~/.nexus-agents/memory/memory.db` inline so the package stays\n * free of inter-package imports (it is reused by `nexus-eval-*`). In\n * production nexus-agents we want the canonical {@link nexusDataPath} resolver\n * instead — it adds sandbox detection, the per-repo/cross-repo split, the\n * homedir-unwritable fallback, and `.gitignore` auto-wiring that the dep-free\n * fallback cannot know about. We keep nexus-memory dep-free by doing the\n * resolution HERE (on the nexus-agents side) and injecting the resolved path\n * as a plain string via `setMemoryRegistry`.\n *\n * No-op when a registry is already present (`hasMemoryRegistry()`): that means\n * either a test injected an in-memory registry, or a prior call already wired\n * the canonical path. Checking presence — rather than calling\n * `getMemoryRegistry()` — is what lets us win the race against nexus-memory's\n * lazy default-path initialization without ever triggering it. Call this\n * before the first `getMemoryRegistry()` in any production path that attaches\n * a backend.\n */\nexport function ensureSharedMemoryRegistry(): void {\n  if (hasMemoryRegistry()) return;\n  // `memory` is intentionally cross-repo (not in PER_REPO_SUBDIRS), so this\n  // resolves to `~/.nexus-agents/memory/memory.db` on a normal machine and to\n  // the sandbox / per-repo-fallback root under those modes.\n  const dbPath = nexusDataPath('memory', 'memory.db');\n  setMemoryRegistry(new MemoryRegistry({ dbPath }));\n}\n"],"mappings":";;;;;;;;;;AAqBA,IAAM,uBAAuB;AAQ7B,SAAS,kBAAkB,QAA8C;AACvE,QAAM,QAAiB,QAAQ;AAC/B,MAAI,UAAU,QAAQ,UAAU,UAAa,OAAO,UAAU,SAAU,QAAO;AAC/E,MAAI,EAAE,UAAU,OAAQ,QAAO;AAC/B,QAAM,YAAY,MAAM;AACxB,SAAO,OAAO,cAAc,YAAY,cAAc,KAAK,YAAY;AACzE;AA2BO,IAAM,mBAAN,MAAkE;AAAA,EAC9D;AAAA,EACQ;AAAA,EAEjB,YAAY,QAAgB,SAA2B;AACrD,SAAK,SAAS;AACd,SAAK,UAAU;AAAA,EACjB;AAAA,EAEA,KAAK,MAAgC;AAGnC,WAAO,QAAQ,QAAQ,MAAS;AAAA,EAClC;AAAA,EAEA,MAAM,MAAc,QAAiB,OAAkC;AACrE,WAAO,QAAQ;AAAA,MACb,IAAI;AAAA,QACF,yBAAyB,KAAK,MAAM;AAAA,MAEtC;AAAA,IACF;AAAA,EACF;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,EAUA,MAAM,MAAM,QAA4D;AACtE,QAAI,KAAK,QAAQ,WAAW,OAAW,QAAO,CAAC;AAC/C,UAAM,OAAO,kBAAkB,MAAM;AACrC,QAAI,SAAS,KAAM,QAAO,CAAC;AAC3B,UAAM,QAAQ,QAAQ,SAAS;AAC/B,QAAI;AACF,aAAO,MAAM,KAAK,QAAQ,OAAO,MAAM,KAAK;AAAA,IAC9C,QAAQ;AACN,aAAO,CAAC;AAAA,IACV;AAAA,EACF;AAAA,EAEA,OAAO,MAAgC;AACrC,WAAO,QAAQ,QAAQ,KAAK;AAAA,EAC9B;AAAA,EAEA,MAAM,QAA+B;AACnC,UAAM,WAAW,MAAM,QAAQ,QAAQ,KAAK,QAAQ,MAAM,CAAC;AAC3D,UAAM,QAAQ,aAAa,QAAQ;AACnC,WAAO;AAAA,MACL,QAAQ,KAAK;AAAA,MACb;AAAA,MACA,iBAAiB;AAAA,MACjB,iBAAiB;AAAA,IACnB;AAAA,EACF;AAAA,EAEA,MAAM,QAAuB;AAC3B,QAAI,KAAK,QAAQ,UAAU,QAAW;AACpC,YAAM,QAAQ,QAAQ,KAAK,QAAQ,MAAM,CAAC;AAAA,IAC5C;AAAA,EACF;AACF;AAUA,SAAS,aAAa,OAAwB;AAC5C,MAAI,OAAO,UAAU,SAAU,QAAO;AACtC,MAAI,UAAU,QAAQ,OAAO,UAAU,UAAU;AAC/C,UAAM,IAAI;AACV,QAAI,EAAE,OAAO,QAAQ,OAAO,EAAE,UAAU,SAAU,QAAO,EAAE;AAO3D,QAAI,EAAE,OAAO,OAAO;AAClB,YAAM,SACJ,EAAE,iBAAiB,QACf,EAAE,MAAM,UACR,OAAO,EAAE,UAAU,WACjB,EAAE,QACF;AACR,YAAM,IAAI,MAAM,2BAA2B,MAAM,EAAE;AAAA,IACrD;AAAA,EACF;AAGA,SAAO;AACT;AA+BO,SAAS,6BAAmC;AACjD,MAAI,kBAAkB,EAAG;AAIzB,QAAM,SAAS,cAAc,UAAU,WAAW;AAClD,oBAAkB,IAAI,eAAe,EAAE,OAAO,CAAC,CAAC;AAClD;","names":[]}

package/dist/{chunk-LGTY3B3Q.js → chunk-XJORRLQT.js}

@@ -4,16 +4,16 @@ import {
   loadPapersRegistry,
   loadTechniquesRegistry,
   savePapersRegistry
-} from "./chunk-Y3WYSLFZ.js";
+} from "./chunk-VWXQHGQ2.js";
 import {
   resolveInsideRoot
 } from "./chunk-NUBSJGQZ.js";
 import {
   resolveToken
-} from "./chunk-XABDFFQV.js";
+} from "./chunk-UI3WYJSZ.js";
 import {
   getToolMemory
-} from "./chunk-2QYJ7IE2.js";
+} from "./chunk-UH4MK2N7.js";
 import {
   capitalizeWords,
   truncateSentence
@@ -38,7 +38,7 @@ import {
   getTimeProvider,
   ok,
   registerPersistentDistillerFactory
-} from "./chunk-TMYLHKLJ.js";
+} from "./chunk-U4LM5Q2L.js";
 import {
   ensureLearningDir,
   getRulesFile
@@ -4090,4 +4090,4 @@ export {
   DEFAULT_CONTEXT_BUDGET_TOKENS,
   getContextPromptPrefix
 };
-//# sourceMappingURL=chunk-LGTY3B3Q.js.map
+//# sourceMappingURL=chunk-XJORRLQT.js.map

package/dist/{chunk-3P6N3XZF.js → chunk-Y3UJ3VLA.js}

@@ -1,6 +1,6 @@
 import {
   getDefaultRegistry
-} from "./chunk-TMYLHKLJ.js";
+} from "./chunk-U4LM5Q2L.js";
 import {
   getNexusDataDir
 } from "./chunk-OQLFRJCW.js";
@@ -128,4 +128,4 @@ export {
   loadUsageEvents,
   rollupByModel
 };
-//# sourceMappingURL=chunk-3P6N3XZF.js.map
+//# sourceMappingURL=chunk-Y3UJ3VLA.js.map

package/dist/{chunk-73U6W5ZU.js → chunk-YIV4GQHK.js}

@@ -1,6 +1,6 @@
 import {
   getErrorMessage
-} from "./chunk-TMYLHKLJ.js";
+} from "./chunk-U4LM5Q2L.js";
 
 // src/cli/setup-config.ts
 import { copyFileSync, existsSync, mkdirSync, writeFileSync } from "fs";
@@ -77,4 +77,4 @@ function ensureBackup(outputPath) {
 export {
   runConfigInitSync
 };
-//# sourceMappingURL=chunk-73U6W5ZU.js.map
+//# sourceMappingURL=chunk-YIV4GQHK.js.map

package/dist/{chunk-TXAWKJWL.js → chunk-ZLGQY55I.js}

@@ -1,6 +1,6 @@
 import {
   readOpenAICompatEnv
-} from "./chunk-ROAVF3S5.js";
+} from "./chunk-SSPEB62Y.js";
 import {
   allOf
 } from "./chunk-OQB3XC5O.js";
@@ -13,10 +13,10 @@ import {
 import {
   createAllAdapters,
   probeCli
-} from "./chunk-MPCBYY4J.js";
+} from "./chunk-RUN6KRCP.js";
 import {
   DEFAULTS
-} from "./chunk-DL5RN53R.js";
+} from "./chunk-65DZ7WHY.js";
 import {
   getNexusTmpDir
 } from "./chunk-SNJNRDKZ.js";
@@ -36,7 +36,7 @@ import {
   ok,
   symbols,
   writeLine
-} from "./chunk-TMYLHKLJ.js";
+} from "./chunk-U4LM5Q2L.js";
 import {
   getLearningDir,
   getOutcomesFile,
@@ -52,7 +52,7 @@ import {
 } from "./chunk-OQLFRJCW.js";
 
 // src/version.ts
-var VERSION = true ? "4.3.3" : "dev";
+var VERSION = true ? "4.3.5" : "dev";
 
 // src/config/schemas-core.ts
 import { z } from "zod";
@@ -2339,7 +2339,7 @@ async function runDoctorFix(result) {
   writeLine2("\u2500".repeat(40));
   let fixCount = 0;
   if (!result.dataDirectory.rootExists || result.dataDirectory.subdirectories.some((d) => !d.exists || !d.writable)) {
-    const { runSetup } = await import("./setup-command-R6LRVQ7P.js");
+    const { runSetup } = await import("./setup-command-DLIUAXMJ.js");
     const setupResult = runSetup({
       skipMcp: true,
       skipRules: true,
@@ -2353,7 +2353,7 @@ async function runDoctorFix(result) {
     }
   }
   if (!result.configFile.found) {
-    const { runConfigInitSync } = await import("./setup-config-P722TC57.js");
+    const { runConfigInitSync } = await import("./setup-config-Q2WGGIQT.js");
     const configResult = runConfigInitSync(process.cwd(), false, false);
     if (configResult.success && configResult.created) {
       writeLine2(`\u2713 Generated config: ${configResult.path}`);
@@ -2452,4 +2452,4 @@ export {
   startStdioServer,
   closeServer
 };
-//# sourceMappingURL=chunk-TXAWKJWL.js.map
+//# sourceMappingURL=chunk-ZLGQY55I.js.map

package/dist/{chunk-A7DEF2I6.js → chunk-ZQWCL7YB.js}

@@ -1,9 +1,9 @@
 import {
   loadPapersRegistry
-} from "./chunk-Y3WYSLFZ.js";
+} from "./chunk-VWXQHGQ2.js";
 import {
   getErrorMessage
-} from "./chunk-TMYLHKLJ.js";
+} from "./chunk-U4LM5Q2L.js";
 
 // src/cli/research-helpers-synthesize.ts
 import { z } from "zod";
@@ -876,4 +876,4 @@ export {
   AttributedInsightSchema,
   synthesizeResearch
 };
-//# sourceMappingURL=chunk-A7DEF2I6.js.map
+//# sourceMappingURL=chunk-ZQWCL7YB.js.map

package/dist/{cli-circuit-breaker-DG2L4C3C.js → cli-circuit-breaker-3CH36TPE.js}

@@ -3,8 +3,8 @@ import {
   createCliCircuitBreakerIntegration,
   getCliCircuitBreakerSnapshot,
   getDefaultCliCircuitBreakerRegistry
-} from "./chunk-4JKO5B7A.js";
-import "./chunk-TMYLHKLJ.js";
+} from "./chunk-4OSPP2KN.js";
+import "./chunk-U4LM5Q2L.js";
 import "./chunk-2HUC6NZN.js";
 import "./chunk-OQLFRJCW.js";
 import "./chunk-ZPWHCABL.js";
@@ -14,4 +14,4 @@ export {
   getCliCircuitBreakerSnapshot,
   getDefaultCliCircuitBreakerRegistry
 };
-//# sourceMappingURL=cli-circuit-breaker-DG2L4C3C.js.map
+//# sourceMappingURL=cli-circuit-breaker-3CH36TPE.js.map

package/dist/cli.js

@@ -13,13 +13,13 @@ import {
   generateSummary,
   parseFindings,
   sumFindings
-} from "./chunk-SF6KSPBW.js";
-import "./chunk-JV5HRP3S.js";
+} from "./chunk-NKTANKWH.js";
+import "./chunk-TKFWYFBD.js";
 import {
   setupCommandAsync,
   verifyCommand
-} from "./chunk-BG7IE3V5.js";
-import "./chunk-73U6W5ZU.js";
+} from "./chunk-HI63BMZ6.js";
+import "./chunk-YIV4GQHK.js";
 import {
   AuthHandler,
   DEFAULT_EXPERTS,
@@ -139,8 +139,8 @@ import {
   validateCommand,
   validateWorkflow,
   wrapInMarkdownFence
-} from "./chunk-GWL7WO64.js";
-import "./chunk-FHXDRCIW.js";
+} from "./chunk-ES3CPAON.js";
+import "./chunk-SHG3MYMD.js";
 import "./chunk-HFOQKCD2.js";
 import "./chunk-RSMG2DPB.js";
 import {
@@ -166,14 +166,14 @@ import {
   loadConfig,
   runDoctor,
   validateNexusEnv
-} from "./chunk-TXAWKJWL.js";
+} from "./chunk-ZLGQY55I.js";
 import {
   buildOpenAICompatAdapters,
   readOpenAICompatEnv
-} from "./chunk-ROAVF3S5.js";
+} from "./chunk-SSPEB62Y.js";
 import {
   shutdownExpertBridge
-} from "./chunk-OYNZ6FCZ.js";
+} from "./chunk-QTP4AIVR.js";
 import {
   ConsensusVoteInputSchema,
   DEFAULT_VOTE_TIMEOUT_MS,
@@ -183,29 +183,29 @@ import {
   executeVoting,
   mapOutcomeToDecision,
   registerConsensusVoteTool
-} from "./chunk-CXA4H3PZ.js";
+} from "./chunk-CFEZ2DG6.js";
 import {
   loadUsageEvents,
   rollupByModel
-} from "./chunk-3P6N3XZF.js";
+} from "./chunk-Y3UJ3VLA.js";
 import {
   allOf,
   anyOf
 } from "./chunk-OQB3XC5O.js";
 import {
   synthesizeResearch
-} from "./chunk-A7DEF2I6.js";
+} from "./chunk-ZQWCL7YB.js";
 import {
   classifyTrust,
   createFullGitHubProvider,
   evaluatePolicy,
   parsePRUrl
-} from "./chunk-H64DUJSZ.js";
-import "./chunk-55XLIDEN.js";
+} from "./chunk-AVB5GOSW.js";
+import "./chunk-LPUJG7C6.js";
 import {
   ReputationCache,
   sanitizeInput
-} from "./chunk-AQAGVYWS.js";
+} from "./chunk-EJLOD53Z.js";
 import {
   ImprovementReviewInputSchema,
   calculateFitnessScore,
@@ -219,15 +219,15 @@ import {
   registerImprovementReviewTool,
   runImprovementReview,
   scanForSecrets
-} from "./chunk-MPEQHP4D.js";
+} from "./chunk-IXMTCWMN.js";
 import {
   createDefaultPolicyFirewall
-} from "./chunk-WX5TASVN.js";
-import "./chunk-5GUP54YH.js";
+} from "./chunk-K2ZYZZNZ.js";
+import "./chunk-DFCJUKGD.js";
 import {
   createUnifiedRegistry,
   getGlobalRegistry
-} from "./chunk-B6JICUQ6.js";
+} from "./chunk-E6ET2WJI.js";
 import {
   CodebaseIndexSchema,
   PaperStatusSchema,
@@ -253,27 +253,27 @@ import {
   loadSourcesRegistry,
   rankDiscoveredItems,
   saveSourcesRegistry
-} from "./chunk-LGTY3B3Q.js";
+} from "./chunk-XJORRLQT.js";
 import {
   loadPapersRegistry,
   loadTechniquesRegistry,
   savePapersRegistry
-} from "./chunk-Y3WYSLFZ.js";
+} from "./chunk-VWXQHGQ2.js";
 import "./chunk-NUBSJGQZ.js";
 import {
   resolveToken
-} from "./chunk-XABDFFQV.js";
+} from "./chunk-UI3WYJSZ.js";
 import {
   MemoryError,
   shutdownToolMemory
-} from "./chunk-2QYJ7IE2.js";
+} from "./chunk-UH4MK2N7.js";
 import {
   capitalize,
   capitalizeKebab
 } from "./chunk-6T3EPABN.js";
-import "./chunk-3K3VGOY4.js";
+import "./chunk-PG563EVI.js";
 import "./chunk-YPNQPT2F.js";
-import "./chunk-DVKS77O5.js";
+import "./chunk-XIIUIBSX.js";
 import "./chunk-KSQSWEFH.js";
 import {
   NEXUS_SUBPROCESS_DEPTH_ENV,
@@ -282,19 +282,19 @@ import {
   isRecord,
   probeAllClis,
   readSubprocessDepth
-} from "./chunk-MPCBYY4J.js";
-import "./chunk-MLR4PF64.js";
+} from "./chunk-RUN6KRCP.js";
+import "./chunk-QGXA25PQ.js";
 import {
   DEFAULTS
-} from "./chunk-DL5RN53R.js";
-import "./chunk-X57EI3U6.js";
+} from "./chunk-65DZ7WHY.js";
+import "./chunk-OPH5S776.js";
 import {
   nexusMkdtempSync
 } from "./chunk-SNJNRDKZ.js";
-import "./chunk-TKA6653I.js";
+import "./chunk-SPZOZGBS.js";
 import "./chunk-QZUG667L.js";
-import "./chunk-CYCXJNPA.js";
-import "./chunk-4JKO5B7A.js";
+import "./chunk-PZSPL5KN.js";
+import "./chunk-4OSPP2KN.js";
 import {
   API_TIMEOUTS,
   AgentCapability,
@@ -368,7 +368,7 @@ import {
   validateTimeout,
   writeEmptyLine,
   writeLine
-} from "./chunk-TMYLHKLJ.js";
+} from "./chunk-U4LM5Q2L.js";
 import "./chunk-2HUC6NZN.js";
 import {
   applyPortableMode,
@@ -4126,7 +4126,7 @@ ${file.patch}
         violations: policyResult.violations
       });
     }
-    const { formatReviewComment: formatReviewComment2 } = await import("./pr-reviewer-helpers-KZ4MNSUT.js");
+    const { formatReviewComment: formatReviewComment2 } = await import("./pr-reviewer-helpers-UOP4JVHI.js");
     const body = formatReviewComment2(result);
     const postResult = await provider.createReview(pr.prNumber, body, result.decision);
     if (!postResult.ok) {
@@ -13204,9 +13204,10 @@ async function recordToolTask(storage, input) {
   const taskDescription = `${input.tool_name}: ${summarizeToolInput(input.tool_input)}`;
   const taskResult = await storage.addTask(activeSession.id, taskDescription);
   if (!taskResult.ok) return;
+  const outcome = classifyToolResponse(input.tool_response);
   await storage.updateTask(taskResult.value.id, {
-    result: summarizeToolResponse(input.tool_response),
-    status: TaskStatus.COMPLETED,
+    result: outcome.summary,
+    status: outcome.status,
     durationMs: metrics.durationMs,
     tokensUsed: metrics.tokensUsed
   });
@@ -13236,17 +13237,23 @@ function summarizeToolInput(toolInput) {
   const keys = Object.keys(toolInput);
   return keys.length > 0 ? keys.join(", ") : "(no input)";
 }
-function summarizeToolResponse(toolResponse) {
+function classifyToolResponse(toolResponse) {
   const error = toolResponse.error ?? toolResponse.stderr;
   if (error !== void 0 && error !== "") {
-    return `Error: ${safeString(error).substring(0, 100)}`;
+    return {
+      status: TaskStatus.FAILED,
+      summary: `Error: ${safeString(error).substring(0, 100)}`
+    };
   }
   const stdout = toolResponse.stdout ?? toolResponse.output ?? toolResponse.content;
   if (stdout !== void 0) {
     const out = safeString(stdout);
-    return out.length > 100 ? `${out.substring(0, 100)}...` : out;
+    return {
+      status: TaskStatus.COMPLETED,
+      summary: out.length > 100 ? `${out.substring(0, 100)}...` : out
+    };
   }
-  return "completed";
+  return { status: TaskStatus.COMPLETED, summary: "completed" };
 }
 function extractFilePath(toolInput) {
   const filePath = toolInput.file_path ?? toolInput.path ?? toolInput.notebook_path;
@@ -17532,7 +17539,7 @@ function startImprovementReviewScheduler(options) {
   let running = false;
   const runOnce = async () => {
     try {
-      const { runImprovementReview: runImprovementReview2, ImprovementReviewInputSchema: ImprovementReviewInputSchema2 } = await import("./improvement-review-52J36YLJ.js");
+      const { runImprovementReview: runImprovementReview2, ImprovementReviewInputSchema: ImprovementReviewInputSchema2 } = await import("./improvement-review-I4WFQK25.js");
       const input = ImprovementReviewInputSchema2.parse({ fileIssues });
       const result = await runImprovementReview2(input, { logger: logger19 });
       logger19.info("Scheduled improvement_review complete", {
@@ -19589,7 +19596,7 @@ function adaptConfigToLibrary(config) {
 }
 function createBeliefPromoter() {
   return async (event) => {
-    const { getToolMemory } = await import("./tool-memory-AL5PXKB4.js");
+    const { getToolMemory } = await import("./tool-memory-MHP37ASX.js");
     const tm = getToolMemory();
     await tm.recordBelief(
       `skill:${event.name}`,
@@ -20776,7 +20783,7 @@ async function handleResearchCommand(args) {
   }
 }
 async function handleRegistryCommand(args) {
-  const { registryCommand, isValidRegistrySubcommand, formatRegistryUsage } = await import("./registry-command-AHGSBCNL.js");
+  const { registryCommand, isValidRegistrySubcommand, formatRegistryUsage } = await import("./registry-command-EPG2X2ZI.js");
   const subcommand = args.subcommand;
   if (!isValidRegistrySubcommand(subcommand)) {
     process.stdout.write(`${formatRegistryUsage()}
@@ -20824,12 +20831,12 @@ async function handleVerifyCommand(args) {
 async function handleDoctorCommand(args) {
   const exitCode = await doctorCommand({ fix: args.options.fix });
   if (args.options.deep) {
-    const { runDeepDiagnostics: runDeepDiagnostics2, formatDeepDiagnostics: formatDeepDiagnostics2 } = await import("./doctor-deep-FZGNQXDZ.js");
+    const { runDeepDiagnostics: runDeepDiagnostics2, formatDeepDiagnostics: formatDeepDiagnostics2 } = await import("./doctor-deep-UHFL3AE4.js");
     const diag = runDeepDiagnostics2();
     process.stdout.write(formatDeepDiagnostics2(diag) + "\n");
   }
   if (args.options.live) {
-    const { runLiveReadiness, formatLiveReadiness } = await import("./doctor-live-KGQWMYJY.js");
+    const { runLiveReadiness, formatLiveReadiness } = await import("./doctor-live-2CGAFGFG.js");
     const report = await runLiveReadiness();
     process.stdout.write(formatLiveReadiness(report) + "\n");
     if (report.some((r) => r.levels.serves.status === "failed")) {
@@ -20882,7 +20889,7 @@ async function runInitPortableFlow(args) {
   return cliExit(result.success ? EXIT_CODES.SUCCESS : EXIT_CODES.SERVER_START_FAILED);
 }
 async function runInitOpencodeFlow(args) {
-  const { runInitOpencode } = await import("./init-opencode-JAJXMGOI.js");
+  const { runInitOpencode } = await import("./init-opencode-HTSSQOLE.js");
   const opencodePath = args.options.opencode;
   if (opencodePath === void 0 || opencodePath === "") {
     process.stderr.write("Error: --opencode requires a path argument.\n");
@@ -20908,7 +20915,7 @@ async function runInitOpencodeFlow(args) {
   return cliExit(EXIT_CODES.SUCCESS);
 }
 async function renderOpencodeValidate(opencodePath) {
-  const { runOpencodeValidate } = await import("./init-opencode-JAJXMGOI.js");
+  const { runOpencodeValidate } = await import("./init-opencode-HTSSQOLE.js");
   const result = await runOpencodeValidate(opencodePath);
   if (!result.ok) {
     process.stderr.write(`init --opencode --validate: ${result.reason ?? "failed"}
@@ -20947,7 +20954,7 @@ async function handleSetupCommandAsync(args) {
   return cliExitFromStatus(exitCode);
 }
 async function runCustomApiSetup(args) {
-  const { configureCustomApi } = await import("./setup-custom-api-P7T5YKIK.js");
+  const { configureCustomApi } = await import("./setup-custom-api-GXF5MFBG.js");
   const baseUrl = args.options.customApi;
   if (baseUrl === void 0) return EXIT_CODES.SERVER_START_FAILED;
   const input = {