Description
When using experimental-chains in project.yaml, users must manually deploy a MockKeystoneForwarder contract and provide its address in the forwarder field. This adds friction to the local development workflow, especially when using tools like Anvil for local testing.
Current behavior
experimental-chains:
- chain-selector: 31337
rpc-url: http://127.0.0.1:8545
forwarder: "0x..." # User must deploy and provide this manually
If the forwarder field is missing or incorrect, the simulation silently fails — the forwarder can't deliver reports to the receiver contract, and there's no clear error message indicating the issue.
Proposed behavior
When cre workflow simulate runs with an experimental chain that has no forwarder specified:
- Compute a deterministic address for the MockKeystoneForwarder
- Check if code already exists at that address (eth_getCode) — if found, reuse it
- If no code found, auto-deploy it using the private key from CRE_ETH_PRIVATE_KEY
- Use the deployed/existing address for the simulation
- Log the address so the user is aware
This avoids redeploying on every simulation run while keeping the forwarder field optional, matching the seamless experience of official chains where the simulator handles this automatically.
Context
While setting up CRE workflow simulation against a local Anvil instance, we initially deployed the KeystoneForwarder contract instead of the MockKeystoneForwarder. The simulation appeared to succeed (returned a tx hash), but the forwarder's report delivery silently failed. There was no error indicating we needed the mock version. It took significant debugging — tracing transactions, checking ERC165 interface IDs, and inspecting ReportProcessed events — to discover the issue.
Auto-deploying the correct MockKeystoneForwarder during simulation would eliminate this confusion entirely and save developers from a non-obvious debugging path.
###Proposed implementation
The change would be in cmd/workflow/simulate/simulate.go around line 175, where it currently errors on missing forwarder:
return Inputs{}, fmt.Errorf("experimental chain %d missing forwarder", ec.ChainSelector)
Instead of erroring:
- Compute a deterministic address for the MockKeystoneForwarder (e.g., using CREATE2 or a fixed nonce)
- Call eth_getCode at that address — if bytecode exists, reuse it without redeploying
- If no code found, deploy using the user's private key from CRE_ETH_PRIVATE_KEY
- Log: "No forwarder specified for experimental chain %d — using MockKeystoneForwarder at %s"
The compiled artifact already exists in the repo at test/MockKeystoneForwarder.json. If forwarder is provided, behavior remains unchanged.
Happy to submit a PR if the team is open to this approach.
Description
When using
experimental-chainsinproject.yaml, users must manually deploy aMockKeystoneForwardercontract and provide its address in theforwarderfield. This adds friction to the local development workflow, especially when using tools like Anvil for local testing.Current behavior
If the forwarder field is missing or incorrect, the simulation silently fails — the forwarder can't deliver reports to the receiver contract, and there's no clear error message indicating the issue.
Proposed behavior
When cre workflow simulate runs with an experimental chain that has no forwarder specified:
This avoids redeploying on every simulation run while keeping the forwarder field optional, matching the seamless experience of official chains where the simulator handles this automatically.
Context
While setting up CRE workflow simulation against a local Anvil instance, we initially deployed the KeystoneForwarder contract instead of the MockKeystoneForwarder. The simulation appeared to succeed (returned a tx hash), but the forwarder's report delivery silently failed. There was no error indicating we needed the mock version. It took significant debugging — tracing transactions, checking ERC165 interface IDs, and inspecting ReportProcessed events — to discover the issue.
Auto-deploying the correct MockKeystoneForwarder during simulation would eliminate this confusion entirely and save developers from a non-obvious debugging path.
###Proposed implementation
The change would be in cmd/workflow/simulate/simulate.go around line 175, where it currently errors on missing forwarder:
Instead of erroring:
The compiled artifact already exists in the repo at test/MockKeystoneForwarder.json. If forwarder is provided, behavior remains unchanged.
Happy to submit a PR if the team is open to this approach.