agentwallet-mcp 1.14.3 → 1.14.4

package/README.md

@@ -374,7 +374,7 @@ A per-transaction cap bounds one signature. It cannot see that the five hundredt
 ```
 
 - **`budget`**: rolling dollar budgets (last 24 hours, last 30 days, lifetime) across every chain, counting x402 payments, stablecoin transfers and stablecoin approvals in registry stablecoins. A spend the budget cannot price (a token outside the registry) is refused while a budget is set, unless `"unpriced_spends": "allow"`.
-- **`native_budget`**: the same per chain, in the chain's own coin, counting native transfers and the native value sent into any contract (`wrap_eth`, `send_transaction` with value and calldata).
+- **`native_budget`**: the same per chain, in the chain's own coin, counting native transfers and the native value sent into any contract (`wrap_eth`, `send_transaction` with value and calldata), including value carried on a token transfer or approval call (a payable token contract keeps it, so it is a second spend, to that contract).
 - **Approvals are spends.** An ERC-20 allowance (`approve_token`, `approve_permit2`, `increaseAllowance`, `approveAndCall`, Permit2 `approve`, or the same calldata through `send_transaction`) is what the spender will take later, so it is priced, checked and reserved on the ledger when it is granted, the spender is checked against the recipient lists, and an unlimited approval (2^128 base units or more) is refused outright, since no budget can count it; grant an exact amount. A zero-value call to any other contract (a router, a bridge) reserves nothing: it can only pull what an approval already granted.
 - **`hosts`**, **`per_host`**: which endpoints `pay_x402` may pay, and how much per host per day, per month and per payment. A denied host is never contacted, not even for a quote.
 - **`recipients`**: where funds may go (transfers, x402 `payTo`, approval spenders, contracts that receive native value). The allow list covers those funds destinations; the contract a zero-value call is aimed at (a token being approved, a router) is checked against the deny list and the chain list.
@@ -384,7 +384,7 @@ A per-transaction cap bounds one signature. It cannot see that the five hundredt
 
 An unknown key stops the server, so a misspelled `budgets` cannot run unbounded. `spend_policy` reports the file, its sha256 (compare it with the file you reviewed), what each budget has spent and has left, and which tools the policy hid. `quote_x402` answers "would this be paid?" for a URL before anything is signed.
 
-**The ledger.** Budgets are enforced from an append-only, hash-chained JSONL file (`AGENTWALLET_LEDGER`, or the policy's `ledger`, or `agentwallet-ledger.jsonl` beside the policy when a budget needs one). A spend is reserved on it before the signature exists; a reservation whose signing fails is voided by a later line; the endpoint's or chain's answer is settled onto it afterwards. A spend that may already be on the wire (a broadcast that failed after the bytes left, a hosted request the API host took and then dropped) is kept counted as `unknown` rather than voided; a failure before anything left this process, including a policy or local-mode refusal, is always voided and never consumes budget. The chain is verified on every read, and a ledger that does not verify refuses every spend until the operator has looked at it. Several agentwallet-mcp processes may share one ledger (two MCP clients on one wallet): the file is re-read before every check, and the check-and-append step holds an exclusive lock file (`<ledger>.lock`) across processes, so the budget is one budget however many processes enforce it; a lock older than 30 seconds is treated as left by a dead process. A transaction signed without being broadcast (`sign_transaction`) is reserved and counted like a send, since whoever holds it can broadcast it. `list_payments` reads it, `export_receipts` turns it into CSV or JSONL, and `AGENTWALLET_WEBHOOK_URL` receives every line as it is written.
+**The ledger.** Budgets are enforced from an append-only, hash-chained JSONL file (`AGENTWALLET_LEDGER`, or the policy's `ledger`, or `agentwallet-ledger.jsonl` beside the policy when a budget needs one). A spend is reserved on it before the signature exists; a reservation whose signing fails is voided by a later line; the endpoint's or chain's answer is settled onto it afterwards. A spend that may already be on the wire (a broadcast that failed after the bytes left, a hosted request the API host took and then dropped) is kept counted as `unknown` rather than voided; a failure before anything left this process, including a policy or local-mode refusal, is always voided and never consumes budget. The chain is verified on every read, and a ledger that does not verify refuses every spend until the operator has looked at it. Several agentwallet-mcp processes may share one ledger (two MCP clients on one wallet): the file is re-read before every check, and the check-and-append step holds an exclusive lock file (`<ledger>.lock`) across processes, so the budget is one budget however many processes enforce it; a lock older than 30 seconds is treated as left by a dead process. A transaction signed without being broadcast (`sign_transaction`) is reserved and counted like a send, since whoever holds it can broadcast it. A call that moves two things (a token transfer that also carries native value) is two lines, reserved in turn and released together when either is refused. `list_payments` reads it, `export_receipts` turns it into CSV or JSONL, and `AGENTWALLET_WEBHOOK_URL` receives every line as it is written.
 
 ### Encrypted keyfiles
 

package/build/index.js

@@ -155,8 +155,12 @@ async function describeSend(body, chainId) {
             const token = layout.tokenIndex === undefined ? to : '0x' + wordAt(layout.tokenIndex).slice(24);
             const counterparty = '0x' + wordAt(layout.toIndex).slice(24);
             const amount = BigInt('0x' + wordAt(layout.amountIndex));
+            // The call's own value does not vanish because the calldata was
+            // recognised: the contract keeps whatever rides along. It is a second
+            // spend, native, to the contract (reported privately 2026-10-23, 1.14.4).
+            const carried = BigInt(value) > 0n ? { raw: value, to } : undefined;
             if (layout.kind === 'approval' && amount >= UNBOUNDED_APPROVAL) {
-                return { kind: 'approval', to: counterparty, asset: token, raw: amount.toString(), decimals: 0, unbounded: true };
+                return { kind: 'approval', to: counterparty, asset: token, raw: amount.toString(), decimals: 0, unbounded: true, ...(carried ? { carried } : {}) };
             }
             // Decimals matter here only to price a registry stablecoin (resolved
             // offline) and to print the ledger amount. A token the registry and the
@@ -171,7 +175,7 @@ async function describeSend(body, chainId) {
             catch {
                 decimals = 0;
             }
-            return { kind: layout.kind, to: counterparty, asset: token, raw: amount.toString(), decimals };
+            return { kind: layout.kind, to: counterparty, asset: token, raw: amount.toString(), decimals, ...(carried ? { carried } : {}) };
         }
     }
     if (data && data !== '0x')
@@ -197,28 +201,40 @@ function assertSendTarget(d, chainId) {
         if (!t.ok)
             throw new Error(t.reason + ' Nothing was signed.');
     }
+    if (d.carried) {
+        const t = targetAllowed(POL(), d.carried.to); // the contract that keeps the value
+        if (!t.ok)
+            throw new Error(t.reason + ' Nothing was signed.');
+    }
 }
 /**
- * The spend a decoded send makes against the policy, or null when nothing
- * leaves the wallet: a zero-value contract call can only pull what an
- * approval already granted, and approvals are counted when they are granted.
- * An approval's recipient is its spender, so recipients.allow and .deny bind
- * who may be allowed to pull; the native value of a contract call is a spend
- * to that contract. Throws for an unbounded approval under a policy.
+ * The spends a decoded send makes against the policy, in the order they are
+ * reserved; empty when nothing leaves the wallet: a zero-value contract call
+ * can only pull what an approval already granted, and approvals are counted
+ * when they are granted. An approval's recipient is its spender, so
+ * recipients.allow and .deny bind who may be allowed to pull; the native
+ * value of a contract call is a spend to that contract; the native value
+ * carried on a token or approval call is a second spend, native, to the
+ * called contract (1.14.4). Throws for an unbounded approval under a policy.
  */
-function spendInputOf(d, chainId, tool, wallet, ctx = {}) {
-    if (d.kind === 'contract' && BigInt(d.raw) === 0n)
-        return null;
+function spendInputsOf(d, chainId, tool, wallet, ctx = {}) {
     if (d.kind === 'approval' && d.unbounded && POLICY.path) {
         throw new Error(`policy: an unlimited approval of ${assetLabel(chainId, d.asset)} to ${d.to} cannot be counted against any budget, and under a spend policy every approval is a bounded spend. ` +
             `Approve an exact amount instead. Nothing was signed.`);
     }
-    return {
-        tool, mode: anyLocalMode() ? 'local' : 'hosted', wallet: anyLocalMode() ? 'local' : wallet, chainId,
-        asset: d.asset, assetLabel: assetLabel(chainId, d.asset), raw: d.raw, decimals: d.decimals,
-        usd: d.unbounded ? null : usdOf(chainId, d.asset, d.raw, d.decimals),
-        to: d.to, host: ctx.host ?? null, resource: ctx.resource ?? null, method: ctx.method ?? methodOf(d.kind),
-    };
+    const base = { tool, mode: anyLocalMode() ? 'local' : 'hosted', wallet: anyLocalMode() ? 'local' : wallet, chainId, host: ctx.host ?? null, resource: ctx.resource ?? null };
+    const out = [];
+    if (!(d.kind === 'contract' && BigInt(d.raw) === 0n)) {
+        out.push({
+            ...base, asset: d.asset, assetLabel: assetLabel(chainId, d.asset), raw: d.raw, decimals: d.decimals,
+            usd: d.unbounded ? null : usdOf(chainId, d.asset, d.raw, d.decimals),
+            to: d.to, method: ctx.method ?? methodOf(d.kind),
+        });
+    }
+    if (d.carried) {
+        out.push({ ...base, asset: '', assetLabel: assetLabel(chainId, ''), raw: d.carried.raw, decimals: nativeDecimals(chainId), usd: null, to: d.carried.to, method: 'native' });
+    }
+    return out;
 }
 // ─── Configuration ──────────────────────────────────────────────
 const API_BASE = (process.env.AGENTWALLET_API_URL || 'https://hifriendbot.com/wp-json/agentwallet/v1').trim().replace(/\/+$/, '');
@@ -444,7 +460,7 @@ async function api(path, method = 'GET', body, extraHeaders) {
         extraHeaders = { ...extraHeaders };
         delete extraHeaders['X-AGW-CTX'];
     }
-    let reserved = null;
+    const reserved = [];
     // /sign with a transaction body is the same spend as /send: the caller gets
     // a broadcast-ready raw transaction (2026-10-14 report, 1.14.2). A message
     // signature (body.message) moves nothing and is not gated.
@@ -454,18 +470,28 @@ async function api(path, method = 'GET', body, extraHeaders) {
         const chainId = localChainId(body, new URLSearchParams(path.split('?')[1] || ''));
         const d = await describeSend(body, chainId);
         assertSendTarget(d, chainId);
-        const input = spendInputOf(d, chainId, ctx.tool || (m[2] === 'sign' ? 'sign_transaction' : 'send'), m[1], ctx);
-        if (input)
-            reserved = reserveSpend(input);
+        const tool = ctx.tool || (m[2] === 'sign' ? 'sign_transaction' : 'send');
+        // One line per spend the call makes (a token call that also carries native
+        // value makes two); a refusal of a later one releases the earlier ones.
+        for (const input of spendInputsOf(d, chainId, tool, m[1], ctx)) {
+            try {
+                reserved.push(reserveSpend(input));
+            }
+            catch (e) {
+                for (const seq of reserved)
+                    LEDGER.void(seq, `released, the same call was refused: ${e.message}`);
+                throw e;
+            }
+        }
     }
     try {
         const out = await apiInner(path, method, body, extraHeaders);
-        if (reserved !== null) {
+        for (const seq of reserved) {
             const r = out;
             if (m[2] === 'sign')
-                LEDGER.settle(reserved, 'signed', 'raw transaction returned to the caller; not broadcast by this process, so it stays counted');
+                LEDGER.settle(seq, 'signed', 'raw transaction returned to the caller; not broadcast by this process, so it stays counted');
             else
-                LEDGER.settle(reserved, 'broadcast', typeof r?.tx_hash === 'string' ? r.tx_hash : typeof r?.signature === 'string' ? r.signature : null);
+                LEDGER.settle(seq, 'broadcast', typeof r?.tx_hash === 'string' ? r.tx_hash : typeof r?.signature === 'string' ? r.signature : null);
         }
         return out;
     }
@@ -476,12 +502,12 @@ async function api(path, method = 'GET', body, extraHeaders) {
         // message and the word "broadcasts" in the local-mode refusal of
         // transaction signing made every refused sign_transaction consume the
         // budget for good (reported privately 2026-10-16, 1.14.3).
-        if (reserved !== null) {
+        for (const seq of reserved) {
             const msg = e.message;
             if (isMaybeLive(e))
-                LEDGER.settle(reserved, 'unknown', msg);
+                LEDGER.settle(seq, 'unknown', msg);
             else
-                LEDGER.void(reserved, msg);
+                LEDGER.void(seq, msg);
         }
         throw e;
     }
@@ -1143,7 +1169,7 @@ function resolveChainId(network) {
 const AddressSchema = z.string().regex(/^(0x[a-fA-F0-9]{40}|[1-9A-HJ-NP-Za-km-z]{32,44})$/, 'Must be a 0x-prefixed 40-hex EVM address or a base58 Solana address');
 const server = new McpServer({
     name: 'agentwallet',
-    version: '1.14.3',
+    version: '1.14.4',
 }, {
     instructions: `AgentWallet gives AI agents their own blockchain wallets. ${anyLocalMode()
         ? 'This server runs in LOCAL (self-custody) mode: the operator supplied the key, signing happens in this process, and the only spend guards are the environment caps (AGENTWALLET_MAX_TX_NATIVE / _TOKEN / _SOL per transaction, AGENTWALLET_MAX_AUTOPAY and AGENTWALLET_AUTOPAY_ASSETS for x402, AGENTWALLET_X402_MAX_TIMEOUT for authorization windows). There is no server-side pause, daily limit or approval email; pause_wallet, unpause_wallet and delete_wallet are refused.'
@@ -2815,8 +2841,12 @@ tool('simulate_transaction', 'Dry-run an EVM transaction with nothing signed: th
     const policy = (() => {
         try {
             assertSendTarget(d, chain_id);
-            const input = spendInputOf(d, chain_id, 'simulate_transaction', String(wallet_id));
-            return input ? spendVerdict(input) : { ok: true };
+            for (const input of spendInputsOf(d, chain_id, 'simulate_transaction', String(wallet_id))) {
+                const v = spendVerdict(input);
+                if (!v.ok)
+                    return v;
+            }
+            return { ok: true };
         }
         catch (e) {
             return { ok: false, reason: e.message };
@@ -2826,6 +2856,7 @@ tool('simulate_transaction', 'Dry-run an EVM transaction with nothing signed: th
         kind: d.kind, to: d.to, asset: d.asset || 'native', asset_label: assetLabel(chain_id, d.asset),
         amount: d.unbounded ? 'unlimited' : formatUnits(d.raw, d.decimals),
         ...(d.kind === 'approval' ? { note: 'an approval: `to` is the spender being allowed to pull `amount` of `asset`' } : {}),
+        ...(d.carried ? { carried_native: { amount: formatUnits(d.carried.raw, nativeDecimals(chain_id)), to: d.carried.to, note: 'native value sent along with the calldata: a second spend, against native_budget, to the called contract' } } : {}),
     };
     if (isLocalMode()) {
         const sim = await localSimulate(chain_id, to, BigInt(valueRaw), data && data !== '0x' ? data : undefined);

package/package.json

@@ -5,7 +5,7 @@
     "type": "git",
     "url": "git+https://github.com/hifriendbot/agentwallet-mcp.git"
   },
-  "version": "1.14.3",
+  "version": "1.14.4",
   "description": "MCP server for AgentWallet: self-custody local signing on EVM and Solana, or hosted wallets. Sign, broadcast, pay x402 invoices, and accept x402 payments.",
   "type": "module",
   "main": "build/index.js",