fix(staking): min-stake preflight degrades if the chain can't report the minimum

The self-stake below-minimum check called getEpochInfo() unconditionally in the
validator-join/deposit + vesting create/deposit preflights. On a chain that
can't report epoch/minStake (a minimal or stub staking contract, or a transient
read failure) that read reverts and HARD-FAILS the write — e.g. the Tier-2
browser-signing lanes join against a StakingStub without epoch(), so
validator-join died with 'The contract function "epoch" reverted'.

Wrap the getEpochInfo() read: on failure, skip the (advisory) min check and
proceed, rather than blocking the operation. The mixing guard (owner /
isValidatorWallet) stays a hard, always-enforced check. Also pass --force on the
Tier-2 lanes' deliberately-sub-minimum test joins.

Verified: Tier-2 browser-signing suite 6/6 green; 786 unit tests pass.

Author: MuncleUscles

e2e/config-default.e2e.ts

@@ -46,7 +46,7 @@ test.describe.serial("S5 config default walletMode=browser", () => {
 
   test("bare validator-join signs via session (no --wallet flag)", async () => {
     const before = await readStubCallCount(anvil);
-    const res = await runCli(["staking", "validator-join", "--amount", "1"], scratch);
+    const res = await runCli(["staking", "validator-join", "--force", "--amount", "1"], scratch);
     expect(res.all).toContain("Validator created successfully!");
     expect(res.all.match(HASH_RE)?.[0]).toBeTruthy();
     expect(await readStubCallCount(anvil)).toBe(before + 1);
@@ -55,7 +55,7 @@ test.describe.serial("S5 config default walletMode=browser", () => {
   test("--wallet keystore overrides config, takes keystore path (no enqueue)", async () => {
     const before = await readStubCallCount(anvil);
     const res = await runCli(
-      ["staking", "validator-join", "--amount", "1", "--wallet", "keystore"],
+      ["staking", "validator-join", "--force", "--amount", "1", "--wallet", "keystore"],
       scratch,
     );
     // Keystore path selected: it fails on the missing account rather than

e2e/errors.e2e.ts

@@ -54,7 +54,7 @@ test.describe.serial("S6a user reject (4001)", () => {
 
   test("reject surfaces the message, exits non-zero, session survives", async () => {
     const res = await runCli(
-      ["staking", "validator-join", "--amount", "1", "--wallet", "browser"],
+      ["staking", "validator-join", "--force", "--amount", "1", "--wallet", "browser"],
       scratch,
     );
     expect(res.all).toContain("Transaction rejected in wallet");
@@ -118,7 +118,7 @@ test.describe.serial("S6b tab closed", () => {
     expect(stale, "heartbeat should go stale after tab close").toBe(true);
 
     const res = await runCli(
-      ["staking", "validator-join", "--amount", "1", "--wallet", "browser"],
+      ["staking", "validator-join", "--force", "--amount", "1", "--wallet", "browser"],
       scratch,
       {timeoutMs: 20_000},
     );

e2e/lane-a-staking.e2e.ts

@@ -48,7 +48,7 @@ test.describe.serial("Lane A staking (validator-join)", () => {
   test("S2: validator-join --wallet browser signs and mines", async () => {
     const before = await readStubCallCount(anvil);
     const res = await runCli(
-      ["staking", "validator-join", "--amount", "1", "--wallet", "browser"],
+      ["staking", "validator-join", "--force", "--amount", "1", "--wallet", "browser"],
       scratch,
     );
 
@@ -65,11 +65,11 @@ test.describe.serial("Lane A staking (validator-join)", () => {
     const before = await readStubCallCount(anvil);
 
     const first = await runCli(
-      ["staking", "validator-join", "--amount", "1", "--wallet", "browser"],
+      ["staking", "validator-join", "--force", "--amount", "1", "--wallet", "browser"],
       scratch,
     );
     const second = await runCli(
-      ["staking", "validator-join", "--amount", "2", "--wallet", "browser"],
+      ["staking", "validator-join", "--force", "--amount", "2", "--wallet", "browser"],
       scratch,
     );
 

src/commands/staking/validatorDeposit.ts

@@ -29,11 +29,10 @@ export class ValidatorDepositAction extends StakingAction {
     amount: bigint,
     force?: boolean,
   ): Promise<void> {
-    const [info, epochInfo] = await Promise.all([
-      client.getValidatorInfo(validatorWallet),
-      client.getEpochInfo(),
-    ]);
+    const info = await client.getValidatorInfo(validatorWallet);
 
+    // Mixing guard — a liquid deposit into a vesting-owned wallet reverts
+    // on-chain; fail fast with guidance. This is a hard guard, always enforced.
     if (info.owner.toLowerCase() !== signerAddress.toLowerCase()) {
       throw new Error(
         "This validator wallet is owned by a vesting contract (vesting-funded self-stake). " +
@@ -42,6 +41,14 @@ export class ValidatorDepositAction extends StakingAction {
       );
     }
 
+    // The self-stake minimum is advisory. If the chain can't report it, skip
+    // the check rather than blocking the deposit.
+    let epochInfo;
+    try {
+      epochInfo = await client.getEpochInfo();
+    } catch {
+      return;
+    }
     const pendingSelfStakeRaw = info.pendingDeposits.reduce((sum, d) => sum + d.stakeRaw, 0n);
     const resultingSelfStakeRaw = info.vStakeRaw + pendingSelfStakeRaw + amount;
     this.assertOrWarnSelfStakeMinimum({

src/commands/staking/validatorJoin.ts

@@ -23,11 +23,19 @@ export class ValidatorJoinAction extends StakingAction {
     amount: bigint,
     force?: boolean,
   ): Promise<void> {
-    const epochInfo = await client.getEpochInfo();
     this.logInfo(
       "Creating a liquid (wallet-funded) validator. Self-stake source is fixed at creation — " +
         "you won't be able to add vesting tokens later.",
     );
+    // The self-stake minimum is advisory. If the chain can't report it
+    // (a minimal/stub staking contract without epoch/minStake reads, or a
+    // transient read failure), skip the check rather than blocking the join.
+    let epochInfo;
+    try {
+      epochInfo = await client.getEpochInfo();
+    } catch {
+      return;
+    }
     this.assertOrWarnSelfStakeMinimum({
       currentEpoch: epochInfo.currentEpoch,
       minStakeRaw: epochInfo.validatorMinStakeRaw,

src/commands/vesting/validatorCreate.ts

@@ -20,11 +20,17 @@ export class VestingValidatorCreateAction extends VestingAction {
    * minimum, and surface the source note.
    */
   private async preflight(client: VestingClient, amount: bigint, force?: boolean): Promise<void> {
-    const epochInfo = await client.getEpochInfo();
     this.logInfo(
       "Creating a vesting-funded validator. Self-stake source is fixed — you won't be able to add " +
         "liquid self-stake later.",
     );
+    // Advisory min check — skip if the chain can't report the minimum.
+    let epochInfo;
+    try {
+      epochInfo = await client.getEpochInfo();
+    } catch {
+      return;
+    }
     this.assertOrWarnSelfStakeMinimum({
       currentEpoch: epochInfo.currentEpoch,
       minStakeRaw: epochInfo.validatorMinStakeRaw,

src/commands/vesting/validatorDeposit.ts

@@ -40,10 +40,17 @@ export class VestingValidatorDepositAction extends VestingAction {
       );
     }
 
-    const [info, epochInfo] = await Promise.all([
-      client.getValidatorInfo(wallet),
-      client.getEpochInfo(),
-    ]);
+    // Advisory min check (the mixing guard above is already enforced) — skip
+    // if the chain can't report validator/epoch state.
+    let info, epochInfo;
+    try {
+      [info, epochInfo] = await Promise.all([
+        client.getValidatorInfo(wallet),
+        client.getEpochInfo(),
+      ]);
+    } catch {
+      return;
+    }
     const pendingSelfStakeRaw = info.pendingDeposits.reduce((sum, d) => sum + d.stakeRaw, 0n);
     const resultingSelfStakeRaw = info.vStakeRaw + pendingSelfStakeRaw + amount;
     this.assertOrWarnSelfStakeMinimum({