Skip to content

fix(bot): use NO_FROM instead of AztecAddress.ZERO in deploy simulate#21402

Merged
Maddiaa0 merged 1 commit intomerge-train/fairiesfrom
claudebox/fix-merge-train-fairies
Mar 19, 2026
Merged

fix(bot): use NO_FROM instead of AztecAddress.ZERO in deploy simulate#21402
Maddiaa0 merged 1 commit intomerge-train/fairiesfrom
claudebox/fix-merge-train-fairies

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Mar 12, 2026

Summary

The feat!: no_from PR (#21716) changed how AztecAddress.ZERO is handled in the embedded wallet's simulate path. BotFactory.setupAccountWithPrivateKey was passing AztecAddress.ZERO as the from address in the simulate call, which now causes the wallet to attempt looking up account data for the zero address — resulting in "Account 0x000...000 does not exist on this wallet".

Fixed by using NO_FROM (already imported and used in the adjacent send call) which is the correct API for opting out of account contract mediation during simulation.

Failing tests fixed

  • e2e_bot › bridge resume › reuses prior bridge claims
  • e2e_bot › bridge resume › does not reuse prior bridge claims if recipient address changes
  • e2e_bot › setup via bridging funds cross-chain › creates bot after inbox drift

Test plan

  • All 10 e2e_bot tests pass locally (previously 3 were failing)

ClaudeBox log: https://claudebox.work/s/11a8e25f0aefa248?run=1

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 12, 2026
@AztecBot AztecBot force-pushed the claudebox/fix-merge-train-fairies branch from 4c1a54b to e3e22ab Compare March 12, 2026 05:14
@AztecBot AztecBot changed the title fix: prevent block hash not found error when proven tip lags behind finalized fix(e2e): prevent L1 finalization from racing rollback in offchain payment test Mar 12, 2026
… call

The no_from PR (#21716) changed how AztecAddress.ZERO is handled in the
embedded wallet simulate path. The simulate call in setupAccountWithPrivateKey
was using AztecAddress.ZERO as the from address, which now causes the wallet
to attempt looking up account data for the zero address (which doesn't exist).
Changed to use NO_FROM which is the correct API for opting out of account
contract mediation during simulation.
@AztecBot AztecBot force-pushed the claudebox/fix-merge-train-fairies branch from e3e22ab to ba664f1 Compare March 19, 2026 00:10
@AztecBot AztecBot changed the title fix(e2e): prevent L1 finalization from racing rollback in offchain payment test fix(bot): use NO_FROM instead of AztecAddress.ZERO in deploy simulate Mar 19, 2026
@Maddiaa0 Maddiaa0 marked this pull request as ready for review March 19, 2026 00:39
@Maddiaa0 Maddiaa0 enabled auto-merge (squash) March 19, 2026 00:39
@Maddiaa0 Maddiaa0 merged commit d1392ef into merge-train/fairies Mar 19, 2026
18 checks passed
@Maddiaa0 Maddiaa0 deleted the claudebox/fix-merge-train-fairies branch March 19, 2026 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants