@vultisig/cli 2.8.2 → 2.8.4

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @vultisig/cli
 
+## 2.8.4
+
+### Patch Changes
+
+- [#879](https://github.com/vultisig/vultisig-sdk/pull/879) [`f34ef93`](https://github.com/vultisig/vultisig-sdk/commit/f34ef932ba18c19accccc716098c3c16b83da625) Thanks [@neavra](https://github.com/neavra)! - Fix `agent ask` error reporting so a real first-turn backend/stream error is no
+  longer masked by the initialize-time `SESSION_NOT_FOUND` signal. The
+  stale-`--session` fallback signal is now kept separately and used only as a
+  lowest-priority fallback when the turn produced no error of its own, so a
+  genuine turn error wins while a clean turn after a stale-session fallback still
+  reports `SESSION_NOT_FOUND` (non-zero exit).
+
+- [#875](https://github.com/vultisig/vultisig-sdk/pull/875) [`a1d711b`](https://github.com/vultisig/vultisig-sdk/commit/a1d711beaf1694883688cd8e430c8b38162975b6) Thanks [@neavra](https://github.com/neavra)! - agent: surface a loop-depth overrun as a typed `LOOP_DEPTH_EXCEEDED` error instead of a success-shaped `done`.
+
+  When the agent message loop exceeds `MAX_MESSAGE_LOOP_DEPTH` (16), `processMessageLoop` previously cleared the queued tool results and called `ui.onDone()` — the same callback used on a clean finish — so a headless caller (`agent ask --json` / `--via-agent` pipe) could not distinguish a depth-capped truncation from a completed turn. It now emits `AgentErrorCode.LOOP_DEPTH_EXCEEDED` via `ui.onError` first (ask exits non-zero with an error envelope; pipe gets a typed `error` frame), then `onDone()` as the turn terminator.
+
+- Updated dependencies [[`26dd218`](https://github.com/vultisig/vultisig-sdk/commit/26dd218282b198fdea9c1118e7fe5bc800c071fb), [`69bb830`](https://github.com/vultisig/vultisig-sdk/commit/69bb8307de72883f0c7693871a6ca040b7a0756c)]:
+  - @vultisig/client-shared@0.2.17
+  - @vultisig/core-chain@2.17.10
+  - @vultisig/sdk@2.8.4
+
 ## 2.8.2
 
 ### Patch Changes

package/dist/index.js

@@ -8074,6 +8074,7 @@ var AgentErrorCode = /* @__PURE__ */ ((AgentErrorCode3) => {
   AgentErrorCode3["TRANSACTION_FAILED"] = "TRANSACTION_FAILED";
   AgentErrorCode3["SIGNING_FAILED"] = "SIGNING_FAILED";
   AgentErrorCode3["SESSION_NOT_INITIALIZED"] = "SESSION_NOT_INITIALIZED";
+  AgentErrorCode3["LOOP_DEPTH_EXCEEDED"] = "LOOP_DEPTH_EXCEEDED";
   AgentErrorCode3["SESSION_NOT_FOUND"] = "SESSION_NOT_FOUND";
   AgentErrorCode3["UNKNOWN_ERROR"] = "UNKNOWN_ERROR";
   return AgentErrorCode3;
@@ -8082,6 +8083,10 @@ var AGENT_ERROR_CODE_VALUES = new Set(Object.values(AgentErrorCode));
 function isAgentErrorCode(value) {
   return AGENT_ERROR_CODE_VALUES.has(value);
 }
+var TERMINAL_AGENT_ERROR_CODES = /* @__PURE__ */ new Set(["LOOP_DEPTH_EXCEEDED" /* LOOP_DEPTH_EXCEEDED */]);
+function isTerminalAgentErrorCode(code) {
+  return TERMINAL_AGENT_ERROR_CODES.has(code);
+}
 function mapVaultError(err) {
   if (err.code === VaultErrorCode2.InvalidConfig && /failed to unlock vault/i.test(err.message)) {
     return "AUTH_FAILED" /* AUTH_FAILED */;
@@ -8186,10 +8191,21 @@ var AskInterface = class {
   transactions = [];
   cards = [];
   error;
-  // Whether ask() has run at least once. initialize() drives getCallbacks()
-  // BEFORE the first ask() — a stale --session fallback fires
-  // onError(SESSION_NOT_FOUND) there. ask() must NOT clear that initialize-time
-  // error on its first turn, or the result envelope reports false success.
+  // Initialize-time error, kept SEPARATE from the turn error so it stays the
+  // LOWEST-priority signal. initialize() drives getCallbacks() BEFORE the first
+  // ask() — a stale --session fallback fires onError(SESSION_NOT_FOUND) there.
+  // A real first-turn error must override it, so we never pre-set `this.error`
+  // with the init signal; instead partialResult() falls back to `initError` only
+  // when the turn produced no error of its own. Cleared after the first turn so
+  // later turns don't carry the init signal.
+  initError;
+  // Tracks whether the currently-latched `error` is a terminal one (e.g. the
+  // depth cap). A terminal error may overwrite a prior non-terminal one; once a
+  // terminal error is recorded, later frames cannot replace it. See onError.
+  errorIsTerminal = false;
+  // Whether ask() has run at least once. Distinguishes init-time onError (sets
+  // initError) from turn onError (sets error), and gates clearing initError so
+  // the init signal only carries into the FIRST turn.
   hasAsked = false;
   constructor(session, verbose = false, autoApprove = false) {
     this.session = session;
@@ -8243,8 +8259,12 @@ var AskInterface = class {
         }
       },
       onError: (message, code) => {
-        if (!this.error) {
+        const isTerminal = isTerminalAgentErrorCode(code);
+        if (!this.hasAsked) {
+          this.initError = { message, code };
+        } else if (!this.error || isTerminal && !this.errorIsTerminal) {
           this.error = { message, code };
+          this.errorIsTerminal = isTerminal;
         }
         process.stderr.write(`[error] ${message} [${code}]
 `);
@@ -8275,8 +8295,10 @@ var AskInterface = class {
     this.toolCalls = [];
     this.transactions = [];
     this.cards = [];
+    this.error = void 0;
     if (this.hasAsked) {
-      this.error = void 0;
+      this.initError = void 0;
+      this.errorIsTerminal = false;
     }
     this.hasAsked = true;
     const callbacks = this.getCallbacks();
@@ -8298,7 +8320,9 @@ var AskInterface = class {
       toolCalls: this.toolCalls,
       transactions: this.transactions,
       cards: this.cards,
-      error: this.error
+      // A real turn error wins; fall back to the init-time signal (e.g. stale
+      // --session SESSION_NOT_FOUND) only when the turn produced no error.
+      error: this.error ?? this.initError
     };
   }
 };
@@ -12297,6 +12321,10 @@ var AgentSession = class {
 `
       );
       this.pendingToolResults = [];
+      ui.onError(
+        `agent message loop exceeded ${MAX_MESSAGE_LOOP_DEPTH} turns; conversation truncated`,
+        "LOOP_DEPTH_EXCEEDED" /* LOOP_DEPTH_EXCEEDED */
+      );
       ui.onDone();
       return;
     }
@@ -13431,7 +13459,7 @@ var cachedVersion = null;
 function getVersion() {
   if (cachedVersion) return cachedVersion;
   if (true) {
-    cachedVersion = "2.8.2";
+    cachedVersion = "2.8.4";
     return cachedVersion;
   }
   try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vultisig/cli",
-  "version": "2.8.2",
+  "version": "2.8.4",
   "description": "The self-custody MPC wallet CLI for AI coding agents (Claude Code, Cursor, OpenCode). Natural-language agent mode, 36+ chains, DKLS23 threshold signatures. Seedless.",
   "type": "module",
   "bin": {
@@ -73,10 +73,10 @@
     "@cosmjs/stargate": "^0.39.0",
     "@napi-rs/keyring": "^1.3.0",
     "@noble/hashes": "^2.2.0",
-    "@vultisig/client-shared": "^0.2.16",
-    "@vultisig/core-chain": "^2.17.8",
+    "@vultisig/client-shared": "^0.2.17",
+    "@vultisig/core-chain": "^2.17.10",
     "@vultisig/rujira": "^41.0.0",
-    "@vultisig/sdk": "^2.8.2",
+    "@vultisig/sdk": "^2.8.4",
     "chalk": "^5.6.2",
     "cli-table3": "^0.6.5",
     "commander": "^15.0.0",