fix(ci): use fresh wallet for anvil fork tests #255
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The anvil fork tests reused the same wallet ($PK) that was drained by prior devnet tests (liquidity, swaps, sponsors, batches). By the time the fork tests ran, the account had insufficient fee token balance, causing flaky 'Insufficient funds' / 'Out of gas' errors. Generate a new wallet and fund it via tempo_fundAddress before forking, so the fork snapshot always has a fully-funded account. Amp-Thread-ID: https://ampcode.com/threads/T-019c40f9-c790-771a-a1d5-5524db9edb2d Co-authored-by: Amp <amp@ampcode.com>
gakonst
requested review from
0xrusowsky,
DaniPopes,
grandizzy,
mattsse,
onbjerg and
zerosnacks
as code owners
The sleep 5 was not reliable — funding txs could still be pending when anvil forked. Poll the fee token balance instead to confirm the funding is actually mined before taking the fork snapshot. Amp-Thread-ID: https://ampcode.com/threads/T-019c40f9-c790-771a-a1d5-5524db9edb2d Co-authored-by: Amp <amp@ampcode.com>
Extract fund_and_wait() helper that calls tempo_fundAddress then polls the fee token balanceOf until non-zero. Replaces unreliable sleep 3/5 after funding the main wallet, access key, sponsor, and fork wallet. All four funding sites had the same race: the funding txs could still be pending when the next step ran, causing 'insufficient funds' errors. Amp-Thread-ID: https://ampcode.com/threads/T-019c40f9-c790-771a-a1d5-5524db9edb2d Co-authored-by: Amp <amp@ampcode.com>
The fresh fork wallet had fee token balance but never had setUserToken() called, so the fee controller didn't know which token to charge for gas. The first broadcast tx (setUserToken itself) failed because gas allowance was 0 — a chicken-and-egg problem. Call setUserToken on the devnet before starting the fork so the snapshot includes the fee token configuration. Amp-Thread-ID: https://ampcode.com/threads/T-019c40f9-c790-771a-a1d5-5524db9edb2d Co-authored-by: Amp <amp@ampcode.com>
The fork wallet has no native ETH balance — only fee tokens from tempo_fundAddress. When FEE_TOKEN_ARG is empty (default PathUSD), forge script doesn't include feeToken in eth_estimateGas requests, causing anvil to check ETH balance instead of fee token balance. Gas allowance computes to 0 and the tx is rejected. Always pass --tempo.fee-token explicitly for fork forge script commands so anvil knows to check fee token balance. Amp-Thread-ID: https://ampcode.com/threads/T-019c40f9-c790-771a-a1d5-5524db9edb2d Co-authored-by: Amp <amp@ampcode.com>
zerosnacks
approved these changes
Feb 9, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fix flaky
ANVIL FORK: FORGE SCRIPT BROADCASTfailures caused by fee token exhaustion.Motivation
The anvil fork tests reused
$PK— the same wallet drained by 27+ prior devnet transactions (liquidity additions, DEX swaps, sponsor tests, batch tests). By the time the fork snapshot was taken, the account had insufficient PathUSD to pay gas, causing intermittentOut of gas: gas required exceeds allowance: 0andInsufficient funds for gas * price + valueerrors.Failing workflow: tempo-tests-wmj9p
Thread: https://tempoxyz.slack.com/archives/C0A6PD3BQ3F/p1770616527072009
Changes
FORK_PK/FORK_ADDR) and fund it viatempo_fundAddressbefore starting the forked anvil$PKreferences in the anvil fork section with$FORK_PKTesting
N/A — CI script change, will be validated by the next tempo-tests workflow run.