@haven_ai/mcp 0.2.1-alpha.0 → 0.3.0-alpha.0

package/README.md

@@ -44,9 +44,12 @@ Create a private JSON file from the values in the Haven agent handoff:
 `delegate_key` is required. Without it the MCP server cannot sign locally.
 `account_address` is the name the connector writes since #2908; a file that
 still says `safe_address` (or `safeAddress`) is read the same way, permanently.
-The same holds for the environment: `HAVEN_ACCOUNT_ADDRESS` is read first, and
-the older `HAVEN_WALLET_ADDRESS` / `HAVEN_SAFE_ADDRESS` are accepted for one
-release (#2914 removes them).
+The environment is **not** the same: only `HAVEN_ACCOUNT_ADDRESS` is read.
+`HAVEN_WALLET_ADDRESS` and `HAVEN_SAFE_ADDRESS` were accepted for one release
+and #2914 removed them, so a machine still configured through either resolves
+no account address at all. The credential-FILE fallback above is permanent —
+a file on disk never rewrites itself — and the environment is not, which is
+the whole difference between the two paragraphs.
 
 The Haven connector may also write split credentials:
 

package/dist/cli.cjs

@@ -56,7 +56,6 @@ async function loadCredentialsFromFile(path) {
     delegateKey,
     agentId: stringField(raw.agent_id ?? raw.agentId),
     accountAddress,
-    safeAddress: accountAddress,
     delegateAddress: stringField(raw.delegate_address ?? raw.delegateAddress),
     chainId: numberField(raw.chain_id ?? raw.chainId),
     network: stringField(raw.network),
@@ -86,13 +85,11 @@ async function loadCredentialsFromSplitFiles(identityPath, signerPath) {
       identity.agent_id ?? identity.agentId,
       signer.agent_id ?? signer.agentId
     ),
-    ...accountAddressTwins(
-      matchingStringField(
-        "account_address",
-        readAccountAddressField(identity),
-        readAccountAddressField(signer),
-        { caseInsensitive: true }
-      )
+    accountAddress: matchingStringField(
+      "account_address",
+      readAccountAddressField(identity),
+      readAccountAddressField(signer),
+      { caseInsensitive: true }
     ),
     delegateAddress: matchingStringField(
       "delegate_address",
@@ -147,7 +144,7 @@ function loadCredentialsFromEnv() {
     apiKey,
     delegateKey,
     agentId: stringField(process.env.HAVEN_AGENT_ID),
-    ...accountAddressTwins(readAccountAddressEnv(process.env)),
+    accountAddress: readAccountAddressEnv(process.env),
     chainId: numberField(process.env.HAVEN_CHAIN_ID),
     network: stringField(process.env.HAVEN_NETWORK),
     apiUrl: stringField(process.env.HAVEN_API_URL)
@@ -157,10 +154,22 @@ function readAccountAddressField(raw) {
   return stringField(raw.account_address ?? raw.safe_address ?? raw.safeAddress);
 }
 function readAccountAddressEnv(env) {
-  return stringField(env.HAVEN_ACCOUNT_ADDRESS ?? env.HAVEN_WALLET_ADDRESS ?? env.HAVEN_SAFE_ADDRESS);
-}
-function accountAddressTwins(address) {
-  return { accountAddress: address, safeAddress: address };
+  const current = stringField(env.HAVEN_ACCOUNT_ADDRESS);
+  for (const name of ["HAVEN_WALLET_ADDRESS", "HAVEN_SAFE_ADDRESS"]) {
+    const retired = stringField(env[name]);
+    if (!retired) continue;
+    if (!current) {
+      throw new Error(
+        `${name} is retired (#2906) \u2014 Haven accounts are addressed as accounts, not Safes. Set HAVEN_ACCOUNT_ADDRESS to the same value. Refusing rather than ignoring it, because an unset account address silently skips the sweep-destination check.`
+      );
+    }
+    if (retired.toLowerCase() !== current.toLowerCase()) {
+      throw new Error(
+        `${name} and HAVEN_ACCOUNT_ADDRESS were both set to different addresses. Remove the retired name, or make them match \u2014 picking one silently would hide the mismatch.`
+      );
+    }
+  }
+  return current;
 }
 function stringField(value) {
   return typeof value === "string" && value.trim() ? value.trim() : void 0;
@@ -723,7 +732,7 @@ function computeConsentHash(input) {
     input.apiKeyPrefix,
     input.apiUrl ?? "",
     input.agentId ?? "",
-    (input.safeAddress ?? "").toLowerCase(),
+    (input.accountAddress ?? "").toLowerCase(),
     (input.delegateAddress ?? "").toLowerCase(),
     input.chainId ?? ""
   ].join("|");
@@ -742,7 +751,7 @@ function renderConsentBlock(input, hash) {
   ];
   if (input.apiUrl) lines.push(`Haven API: ${input.apiUrl}`);
   if (input.agentId) lines.push(`Agent ID:  ${input.agentId}`);
-  if (input.safeAddress) lines.push(`Haven wallet (Safe): ${input.safeAddress}`);
+  if (input.accountAddress) lines.push(`Haven wallet (Safe): ${input.accountAddress}`);
   if (input.delegateAddress) lines.push(`Delegate (local signer): ${input.delegateAddress}`);
   if (typeof input.chainId === "number") lines.push(`Chain ID:  ${input.chainId}`);
   lines.push("");
@@ -849,14 +858,14 @@ async function writeAckFile(path$1, hash) {
 }
 async function consentInputFromClient(haven, seed, toolNames) {
   let allowanceSummary = seed.allowanceSummary ?? [];
-  let safeAddress = seed.safeAddress;
+  let accountAddress = seed.accountAddress;
   let delegateAddress = seed.delegateAddress;
   let chainId = seed.chainId;
   try {
     const summary = await haven.getAllowances();
     const list = isAllowanceSummary(summary) ? summary.allowances : Array.isArray(summary) ? summary : [];
     if (isAllowanceSummary(summary)) {
-      safeAddress = summary.accountAddress ?? summary.safeAddress ?? safeAddress;
+      accountAddress = summary.accountAddress ?? accountAddress;
       delegateAddress = summary.delegateAddress;
       chainId = typeof summary.chainId === "number" ? summary.chainId : chainId;
     }
@@ -872,7 +881,7 @@ async function consentInputFromClient(haven, seed, toolNames) {
     apiKeyPrefix: derivePrefix(seed.apiKey),
     apiUrl: seed.apiUrl,
     agentId: seed.agentId,
-    safeAddress,
+    accountAddress,
     delegateAddress,
     chainId,
     toolNames,
@@ -906,7 +915,7 @@ async function resolveHavenClient(options = {}) {
   return { client, credentials };
 }
 var MCP_NAME = "@haven_ai/mcp";
-var MCP_VERSION = "0.2.1-alpha.0";
+var MCP_VERSION = "0.3.0-alpha.0";
 var MCP_INSTRUCTIONS = [
   "Haven local MCP server: signs in-process with the delegate key it holds on",
   "this machine \u2014 the key never leaves this process. Call haven_get_agent",
@@ -979,7 +988,7 @@ async function runConsentGate(haven, credentials, options) {
       apiKey: credentials.apiKey,
       apiUrl: credentials.apiUrl,
       agentId: credentials.agentId,
-      safeAddress: credentials.accountAddress ?? credentials.safeAddress,
+      accountAddress: credentials.accountAddress,
       delegateAddress: credentials.delegateAddress,
       chainId: credentials.chainId,
       allowanceSummary: credentials.allowanceSummary