@owney/sdk 0.7.3 → 0.7.4

package/dist/index.cjs

@@ -1318,20 +1318,94 @@ var ZyfaiAgent = class {
       );
     }
   }
+  /**
+   * True when `smartWallet` is the backend-managed pool wallet Zyfai assigned
+   * to this EOA — the "new wallet" kind that is predeployed on the chains it
+   * was onboarded to. Legacy wallets (created before @zyfai/sdk 0.2.44) have no
+   * pool record.
+   *
+   * A failed lookup returns false rather than throwing: misclassifying is
+   * recoverable (ensureWalletDeployed tries the other deploy call), but
+   * failing the deposit outright is not.
+   */
+  async isPredeployedPoolWallet(ownerAddress, smartWallet) {
+    try {
+      const byEoa = await this.sdk.getSmartWalletByEOA(ownerAddress);
+      return typeof byEoa?.smartWallet === "string" && byEoa.smartWallet.toLowerCase() === smartWallet.toLowerCase();
+    } catch (error) {
+      console.warn(
+        "[zyfai] getSmartWalletByEOA failed; assuming a legacy wallet:",
+        error instanceof Error ? error.message : String(error)
+      );
+      return false;
+    }
+  }
+  /**
+   * Guarantee the user's Safe has code on `chainId` before anything transfers
+   * funds into it, and return its address.
+   *
+   * Zyfai predeploys wallets created since @zyfai/sdk 0.2.44, but ONLY on the
+   * chains they were onboarded to. Legacy wallets, and any chain a pool wallet
+   * has not been onboarded to, are *counterfactual*: `getSmartWalletAddress`
+   * returns a real deterministic address that holds no code yet. Depositing
+   * there strands the funds until the Safe is deployed, and Owney's own sponsor
+   * relayer refuses to broadcast to an address with no bytecode
+   * (`to-address.guard-helper.ts`), so the deposit fails as an opaque 500.
+   *
+   * ROUT-320 removed the old deploySafe step believing every wallet was
+   * predeployed from 0.2.44 on. Zyf later clarified that only NEW wallets are;
+   * old ones still need the legacy flow. Hence the gate is on the per-chain
+   * `isDeployed` flag rather than on the wallet's age — that covers both a
+   * legacy wallet and a pool wallet on a fresh chain.
+   */
+  async ensureWalletDeployed(ownerAddress, chainId) {
+    const wallet = await this.sdk.getSmartWalletAddress(ownerAddress, chainId);
+    if (wallet.isDeployed) return wallet.address;
+    const poolFirst = await this.isPredeployedPoolWallet(
+      ownerAddress,
+      wallet.address
+    );
+    const deployAsPool = () => this.sdk.deployOnChains([chainId]);
+    const deployAsLegacy = () => (
+      // `createSessionKey: false` — session keys are backend-owned since the
+      // predeployed-pool rollout; asking for one here costs a signature prompt
+      // and is rejected anyway.
+      this.sdk.deploySafe(ownerAddress, chainId, "aggressive", false)
+    );
+    const [first, second] = poolFirst ? [deployAsPool, deployAsLegacy] : [deployAsLegacy, deployAsPool];
+    try {
+      await first();
+    } catch (error) {
+      console.warn(
+        `[zyfai] ${poolFirst ? "deployOnChains" : "deploySafe"} failed on chain ${chainId}; trying the other deploy path:`,
+        error instanceof Error ? error.message : String(error)
+      );
+      await second();
+    }
+    const redeployed = await this.sdk.getSmartWalletAddress(
+      ownerAddress,
+      chainId
+    );
+    if (!redeployed.isDeployed) {
+      throw new OwneyError(
+        "WALLET_NOT_DEPLOYED",
+        `Smart wallet ${redeployed.address} is still not deployed on chain ${chainId}. Depositing would send funds to an address with no code.`,
+        { chainId, smartWallet: redeployed.address },
+        this.id
+      );
+    }
+    return redeployed.address;
+  }
   async deposit(state, chainId, amount, asset, depositCallback) {
     try {
       const validChainId = isValidChainId(chainId);
       await this.ensureConnected(state, validChainId);
       if (depositCallback) {
-        const wallet = await this.sdk.getSmartWalletAddress(
+        const smartWallet = await this.ensureWalletDeployed(
           this.getAddress(),
           validChainId
         );
-        const txHash = await depositCallback(
-          wallet.address,
-          validChainId,
-          amount
-        );
+        const txHash = await depositCallback(smartWallet, validChainId, amount);
         const tokenAddress = asset === "WETH" ? WETH_ADDRESS_BY_CHAIN[validChainId] : void 0;
         try {
           await (tokenAddress ? this.sdk.logDeposit(validChainId, txHash, amount, tokenAddress) : this.sdk.logDeposit(validChainId, txHash, amount));
@@ -1341,8 +1415,9 @@ var ZyfaiAgent = class {
             logError
           );
         }
-        return { txHash, smartWallet: wallet.address, amount };
+        return { txHash, smartWallet, amount };
       }
+      await this.ensureWalletDeployed(this.getAddress(), validChainId);
       const raw = await this.sdk.depositFunds(
         this.getAddress(),
         validChainId,

package/dist/index.d.cts

@@ -609,7 +609,7 @@ declare class OwneySDK {
     getAllocationApy({ agentId, }?: AllocationApyOptions): Promise<OwneyAllocationApy>;
 }
 
-type OwneyErrorCode = "NOT_CONNECTED" | "NO_ACTIVE_CHAIN" | "WALLET_NO_ACCOUNTS" | "WALLET_ADDRESS_REQUIRED" | "AGENT_NOT_FOUND" | "AGENT_CHAIN_INCOMPATIBLE" | "AGENT_EMPTY_LIST" | "AGENT_DISABLED" | "CHAIN_UNSUPPORTED" | "CHAIN_NO_COMPATIBLE_AGENTS" | "ASSET_UNSUPPORTED" | "ASSET_NO_COMPATIBLE_AGENTS" | "DEPOSIT_AMOUNT_BELOW_MINIMUM" | "DEPOSIT_CALLBACK_REQUIRED" | "DEPOSIT_CALLBACK_INVALID" | "DEPOSIT_NO_PERMITTED_TOKENS" | "DEPOSIT_INSUFFICIENT_BALANCE" | "WITHDRAW_NO_PERMITTED_TOKENS" | "WITHDRAW_INSUFFICIENT_BALANCE" | "WITHDRAW_ALL_FAILED" | "WITHDRAW_PARTIAL_FAILURE" | "WITHDRAW_FAILED" | "API_ROUTING_ERROR" | "API_ROUTING_FAILED" | "API_SAIL_ERROR" | "API_SAIL_TIMEOUT" | "API_NO_AGENTS" | "SPONSOR_REQUEST_FAILED" | "PERMIT2_APPROVAL_REQUIRED" | "BALANCE_ALL_FAILED" | "ALLOCATION_ALL_FAILED" | "VALIDATION_INVALID_DAYS";
+type OwneyErrorCode = "NOT_CONNECTED" | "NO_ACTIVE_CHAIN" | "WALLET_NO_ACCOUNTS" | "WALLET_ADDRESS_REQUIRED" | "WALLET_NOT_DEPLOYED" | "AGENT_NOT_FOUND" | "AGENT_CHAIN_INCOMPATIBLE" | "AGENT_EMPTY_LIST" | "AGENT_DISABLED" | "CHAIN_UNSUPPORTED" | "CHAIN_NO_COMPATIBLE_AGENTS" | "ASSET_UNSUPPORTED" | "ASSET_NO_COMPATIBLE_AGENTS" | "DEPOSIT_AMOUNT_BELOW_MINIMUM" | "DEPOSIT_CALLBACK_REQUIRED" | "DEPOSIT_CALLBACK_INVALID" | "DEPOSIT_NO_PERMITTED_TOKENS" | "DEPOSIT_INSUFFICIENT_BALANCE" | "WITHDRAW_NO_PERMITTED_TOKENS" | "WITHDRAW_INSUFFICIENT_BALANCE" | "WITHDRAW_ALL_FAILED" | "WITHDRAW_PARTIAL_FAILURE" | "WITHDRAW_FAILED" | "API_ROUTING_ERROR" | "API_ROUTING_FAILED" | "API_SAIL_ERROR" | "API_SAIL_TIMEOUT" | "API_NO_AGENTS" | "SPONSOR_REQUEST_FAILED" | "PERMIT2_APPROVAL_REQUIRED" | "BALANCE_ALL_FAILED" | "ALLOCATION_ALL_FAILED" | "VALIDATION_INVALID_DAYS";
 declare class OwneyError extends Error {
     readonly code: OwneyErrorCode;
     readonly details?: Record<string, unknown>;

package/dist/index.d.ts

@@ -609,7 +609,7 @@ declare class OwneySDK {
     getAllocationApy({ agentId, }?: AllocationApyOptions): Promise<OwneyAllocationApy>;
 }
 
-type OwneyErrorCode = "NOT_CONNECTED" | "NO_ACTIVE_CHAIN" | "WALLET_NO_ACCOUNTS" | "WALLET_ADDRESS_REQUIRED" | "AGENT_NOT_FOUND" | "AGENT_CHAIN_INCOMPATIBLE" | "AGENT_EMPTY_LIST" | "AGENT_DISABLED" | "CHAIN_UNSUPPORTED" | "CHAIN_NO_COMPATIBLE_AGENTS" | "ASSET_UNSUPPORTED" | "ASSET_NO_COMPATIBLE_AGENTS" | "DEPOSIT_AMOUNT_BELOW_MINIMUM" | "DEPOSIT_CALLBACK_REQUIRED" | "DEPOSIT_CALLBACK_INVALID" | "DEPOSIT_NO_PERMITTED_TOKENS" | "DEPOSIT_INSUFFICIENT_BALANCE" | "WITHDRAW_NO_PERMITTED_TOKENS" | "WITHDRAW_INSUFFICIENT_BALANCE" | "WITHDRAW_ALL_FAILED" | "WITHDRAW_PARTIAL_FAILURE" | "WITHDRAW_FAILED" | "API_ROUTING_ERROR" | "API_ROUTING_FAILED" | "API_SAIL_ERROR" | "API_SAIL_TIMEOUT" | "API_NO_AGENTS" | "SPONSOR_REQUEST_FAILED" | "PERMIT2_APPROVAL_REQUIRED" | "BALANCE_ALL_FAILED" | "ALLOCATION_ALL_FAILED" | "VALIDATION_INVALID_DAYS";
+type OwneyErrorCode = "NOT_CONNECTED" | "NO_ACTIVE_CHAIN" | "WALLET_NO_ACCOUNTS" | "WALLET_ADDRESS_REQUIRED" | "WALLET_NOT_DEPLOYED" | "AGENT_NOT_FOUND" | "AGENT_CHAIN_INCOMPATIBLE" | "AGENT_EMPTY_LIST" | "AGENT_DISABLED" | "CHAIN_UNSUPPORTED" | "CHAIN_NO_COMPATIBLE_AGENTS" | "ASSET_UNSUPPORTED" | "ASSET_NO_COMPATIBLE_AGENTS" | "DEPOSIT_AMOUNT_BELOW_MINIMUM" | "DEPOSIT_CALLBACK_REQUIRED" | "DEPOSIT_CALLBACK_INVALID" | "DEPOSIT_NO_PERMITTED_TOKENS" | "DEPOSIT_INSUFFICIENT_BALANCE" | "WITHDRAW_NO_PERMITTED_TOKENS" | "WITHDRAW_INSUFFICIENT_BALANCE" | "WITHDRAW_ALL_FAILED" | "WITHDRAW_PARTIAL_FAILURE" | "WITHDRAW_FAILED" | "API_ROUTING_ERROR" | "API_ROUTING_FAILED" | "API_SAIL_ERROR" | "API_SAIL_TIMEOUT" | "API_NO_AGENTS" | "SPONSOR_REQUEST_FAILED" | "PERMIT2_APPROVAL_REQUIRED" | "BALANCE_ALL_FAILED" | "ALLOCATION_ALL_FAILED" | "VALIDATION_INVALID_DAYS";
 declare class OwneyError extends Error {
     readonly code: OwneyErrorCode;
     readonly details?: Record<string, unknown>;

package/dist/index.js

@@ -1285,20 +1285,94 @@ var ZyfaiAgent = class {
       );
     }
   }
+  /**
+   * True when `smartWallet` is the backend-managed pool wallet Zyfai assigned
+   * to this EOA — the "new wallet" kind that is predeployed on the chains it
+   * was onboarded to. Legacy wallets (created before @zyfai/sdk 0.2.44) have no
+   * pool record.
+   *
+   * A failed lookup returns false rather than throwing: misclassifying is
+   * recoverable (ensureWalletDeployed tries the other deploy call), but
+   * failing the deposit outright is not.
+   */
+  async isPredeployedPoolWallet(ownerAddress, smartWallet) {
+    try {
+      const byEoa = await this.sdk.getSmartWalletByEOA(ownerAddress);
+      return typeof byEoa?.smartWallet === "string" && byEoa.smartWallet.toLowerCase() === smartWallet.toLowerCase();
+    } catch (error) {
+      console.warn(
+        "[zyfai] getSmartWalletByEOA failed; assuming a legacy wallet:",
+        error instanceof Error ? error.message : String(error)
+      );
+      return false;
+    }
+  }
+  /**
+   * Guarantee the user's Safe has code on `chainId` before anything transfers
+   * funds into it, and return its address.
+   *
+   * Zyfai predeploys wallets created since @zyfai/sdk 0.2.44, but ONLY on the
+   * chains they were onboarded to. Legacy wallets, and any chain a pool wallet
+   * has not been onboarded to, are *counterfactual*: `getSmartWalletAddress`
+   * returns a real deterministic address that holds no code yet. Depositing
+   * there strands the funds until the Safe is deployed, and Owney's own sponsor
+   * relayer refuses to broadcast to an address with no bytecode
+   * (`to-address.guard-helper.ts`), so the deposit fails as an opaque 500.
+   *
+   * ROUT-320 removed the old deploySafe step believing every wallet was
+   * predeployed from 0.2.44 on. Zyf later clarified that only NEW wallets are;
+   * old ones still need the legacy flow. Hence the gate is on the per-chain
+   * `isDeployed` flag rather than on the wallet's age — that covers both a
+   * legacy wallet and a pool wallet on a fresh chain.
+   */
+  async ensureWalletDeployed(ownerAddress, chainId) {
+    const wallet = await this.sdk.getSmartWalletAddress(ownerAddress, chainId);
+    if (wallet.isDeployed) return wallet.address;
+    const poolFirst = await this.isPredeployedPoolWallet(
+      ownerAddress,
+      wallet.address
+    );
+    const deployAsPool = () => this.sdk.deployOnChains([chainId]);
+    const deployAsLegacy = () => (
+      // `createSessionKey: false` — session keys are backend-owned since the
+      // predeployed-pool rollout; asking for one here costs a signature prompt
+      // and is rejected anyway.
+      this.sdk.deploySafe(ownerAddress, chainId, "aggressive", false)
+    );
+    const [first, second] = poolFirst ? [deployAsPool, deployAsLegacy] : [deployAsLegacy, deployAsPool];
+    try {
+      await first();
+    } catch (error) {
+      console.warn(
+        `[zyfai] ${poolFirst ? "deployOnChains" : "deploySafe"} failed on chain ${chainId}; trying the other deploy path:`,
+        error instanceof Error ? error.message : String(error)
+      );
+      await second();
+    }
+    const redeployed = await this.sdk.getSmartWalletAddress(
+      ownerAddress,
+      chainId
+    );
+    if (!redeployed.isDeployed) {
+      throw new OwneyError(
+        "WALLET_NOT_DEPLOYED",
+        `Smart wallet ${redeployed.address} is still not deployed on chain ${chainId}. Depositing would send funds to an address with no code.`,
+        { chainId, smartWallet: redeployed.address },
+        this.id
+      );
+    }
+    return redeployed.address;
+  }
   async deposit(state, chainId, amount, asset, depositCallback) {
     try {
       const validChainId = isValidChainId(chainId);
       await this.ensureConnected(state, validChainId);
       if (depositCallback) {
-        const wallet = await this.sdk.getSmartWalletAddress(
+        const smartWallet = await this.ensureWalletDeployed(
           this.getAddress(),
           validChainId
         );
-        const txHash = await depositCallback(
-          wallet.address,
-          validChainId,
-          amount
-        );
+        const txHash = await depositCallback(smartWallet, validChainId, amount);
         const tokenAddress = asset === "WETH" ? WETH_ADDRESS_BY_CHAIN[validChainId] : void 0;
         try {
           await (tokenAddress ? this.sdk.logDeposit(validChainId, txHash, amount, tokenAddress) : this.sdk.logDeposit(validChainId, txHash, amount));
@@ -1308,8 +1382,9 @@ var ZyfaiAgent = class {
             logError
           );
         }
-        return { txHash, smartWallet: wallet.address, amount };
+        return { txHash, smartWallet, amount };
       }
+      await this.ensureWalletDeployed(this.getAddress(), validChainId);
       const raw = await this.sdk.depositFunds(
         this.getAddress(),
         validChainId,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@owney/sdk",
-  "version": "0.7.3",
+  "version": "0.7.4",
   "type": "module",
   "main": "dist/index.cjs",
   "module": "dist/index.js",