Skip to content

feat(simulator): opt-in state-override stream for pAMM-aware settlement simulation#4606

Open
louisponet wants to merge 1 commit into
cowprotocol:mainfrom
louisponet:titan-pamm-state-override-stream
Open

feat(simulator): opt-in state-override stream for pAMM-aware settlement simulation#4606
louisponet wants to merge 1 commit into
cowprotocol:mainfrom
louisponet:titan-pamm-state-override-stream

Conversation

@louisponet

@louisponet louisponet commented Jul 6, 2026

Copy link
Copy Markdown

Problem

Titan's propAMMs (pAMMs) hold their live prices in maker quote streams. The CoW driver simulates every solver solution before scoring (gas estimation via eth_estimateGas against a plain node), and the trade verifier simulates proposed quotes the same way — both without any state overrides.

Titan publishes a public pAMM state stream: eth_call-style state-override sets over websocket (wss://{eu,ap,us}.rpc.titanbuilder.xyz/ws/pamm_quote_stream, no auth, ~100–400 ms cadence), plus an HTTP snapshot method titan_getPammStateOverrides. Without applying the overrides, a node sim only sees the pAMM's last-committed-block storage.

Ref: https://docs.titanbuilder.xyz/propamms/takers#pamm-state-stream

Approach

A generic, fully opt-in "simulation state-override stream" feature. The wire format is exactly Ethereum's standard State Override Set, so nothing Titan-branded lives in code — the Titan endpoint appears only in example.toml comments and this PR description. When the config section is absent: no task is spawned, and the RPC override parameters are omitted entirely — byte-identical behavior to today.

The stream module (crates/simulator/src/state_override_stream.rs) spawns a background tokio task that connects to the websocket, parses per-venue frames, and flattens them into one StateOverride (latest frame wins per account, matching the builder's pamm_quote_stream accumulator design). It publishes via tokio::sync::watch; SimulationOverrides::current() gates on staleness (local receive time, not the producer wall-clock frame timestamp) and block lag, returning None so callers omit the RPC params entirely on a stale/unconfigured stream. Reconnect uses exponential backoff (250 ms → 15 s cap). Prometheus metrics cover frames, parse failures, reconnects, account count, and simulations_with_overrides{applied,stale}.

Next-block pinning

current() also returns BlockOverrides { number: head+1, time: head_timestamp + chain_block_time }. The stream frame's top-level timestamp is the producer wall clock (nanoseconds), not block time, so the next-block timestamp is derived from the current head block's timestamp plus Chain::block_time_in_ms() instead. This pins the simulation to the next block — the block context the streamed venue state is intended for.

Backends

  • Ethereum: estimate_gas gains Option<StateOverride> + Option<BlockOverrides> via overrides_opt / with_block_overrides_opt (None omits the RPC params). Requires a node supporting eth_estimateGas state/block overrides (geth ≥1.13, reth, nethermind, recent erigon, anvil).
  • Tenderly: simulate gains Option<StateOverride>; nonce is stripped inline (Tenderly's StateObject::try_from rejects it, pAMM frames always carry it, and contract nonces only matter for CREATE). Full-state overrides are left for try_from to reject.
  • access_list(): unchanged — the access-list probe is a 1-wei ETH transfer that never touches pAMM state, and create_access_list has no override support.

Trade verifier

SettlementSimulator gains an optional SimulationOverrides handle. finish_simulation_builder appends streamed overrides as AccountOverrideRequest::Custom so build_final_state_overrides' existing merge + conflict handling arbitrates against the verifier's own overrides (solver code, trader balance/approval slots). Block overrides are forwarded into EthCallInputs and applied in simulate() (eth_call) and simulation_report() (debug_traceCall). The shared SettlementSimulator covers both the trade verifier and the orderbook order-creation simulator.

Config

# [simulator.state-override-stream]
# ws-url = "wss://eu.rpc.titanbuilder.xyz/ws/pamm_quote_stream"
# max-age = "3s"        # ignore the snapshot if the last frame is older than this
# max-block-lag = 1     # ignore the snapshot if its block number lags head by more than this

The same optional section is added to the price-estimation config for the trade-verifier path.

Tests

  • 9 hermetic unit tests: frame parsing (verbatim Titan sample + real frames captured from wss://eu.rpc.titanbuilder.xyz/ws/pamm_quote_stream), per-account flattening across venues, latest-wins conflict handling, all three staleness gates, and an in-process WebSocket server reconnect/accumulation test.
  • 2 #[ignore]'d anvil integration tests: deploy a contract reverting unless storage slot 0 is nonzero, assert estimate_gas reverts without the override and succeeds with it, and that block overrides are accepted.

…nt simulation

Add a generic, fully opt-in "simulation state-override stream" to the CoW
driver so pAMM and other live-quote venues are simulated against their current
in-memory state rather than stale previous-block state. The wire format is the
standard Ethereum State Override Set (nothing Titan-branded in code); the Titan
endpoint appears only in example.toml comments. When the config section is
absent, no task is spawned and all RPC override parameters are omitted —
byte-identical to today's behavior.

Stream module (crates/simulator/src/state_override_stream.rs):
- Background tokio task connects to a websocket, parses per-venue frames
  (camelCase, forward-compatible flatten of unknown metadata keys), and
  flattens them directly into one StateOverride (latest frame wins per
  account, matching the builder's pamm_quote_stream accumulator design).
- Publishes via tokio::sync::watch; SimulationOverrides::current() gates on
  staleness (local receive time, not the producer wall-clock frame timestamp)
  and block lag, returning None so callers omit the RPC params entirely.
- Reconnect with exponential backoff (250ms -> 15s cap). Prometheus metrics
  for frames, parse failures, reconnects, venue/account count, and
  simulations_with_overrides{applied,stale}.

Next-block pinning:
- current() also returns BlockOverrides { number: head+1, time:
  head_timestamp + chain block_time }. The frame timestamp is the producer
  wall clock, not block time, so the next-block timestamp is derived from the
  head block's timestamp plus Chain::block_time_in_ms().

Backends:
- simulator/ethereum: estimate_gas gains Option<StateOverride> +
  Option<BlockOverrides> via overrides_opt/with_block_overrides_opt (None
  omits the RPC params). Requires a node supporting eth_estimateGas
  state/block overrides (geth >=1.13, reth, nethermind, recent erigon, anvil).
- simulator/tenderly: simulate gains Option<StateOverride>; nonce is stripped
  inline (Tenderly's StateObject::try_from rejects it, pAMM frames always
  carry it, and contract nonces only matter for CREATE). Full-state overrides
  are left for try_from to reject.
- access_list() unchanged (1-wei ETH probe never touches pAMM state;
  create_access_list has no override support).

Simulator integration (crates/simulator/src/lib.rs):
- Simulator gains a simulation_overrides field + set_simulation_overrides
  setter; gas() fetches current(head_number, head_timestamp) and forwards
  state overrides to Tenderly (which already simulates the next block) and
  both state + block overrides to the Ethereum backend.

Trade verifier (crates/price-estimation):
- configs/price_estimation.rs: optional state_override_stream field.
- simulator/simulation_builder.rs: SettlementSimulator gains an optional
  SimulationOverrides handle.
- simulator/encoding.rs: finish_simulation_builder appends streamed overrides
  as AccountOverrideRequest::Custom so build_final_state_overrides' merge
  arbitrates against the verifier's own; block overrides forwarded into
  EthCallInputs and applied in simulate() (eth_call) and simulation_report()
  (debug_traceCall). Block overrides apply only for Block::Latest.
- factory.rs: spawn one stream per service at factory time; the shared
  SettlementSimulator covers both the trade verifier and the orderbook
  order-creation simulator.

Wiring + config:
- driver/run.rs: spawn the stream when [simulator.state-override-stream] is
  present, passing the chain block time.
- configs/simulator.rs: StateOverrideStream struct (ws-url, max-age 3s,
  max-block-lag 1) with deserialize tests.
- driver/example.toml: commented example config.

Tests:
- 9 hermetic unit tests: frame parsing (verbatim Titan sample + real captured
  frames from wss://eu.rpc.titanbuilder.xyz), per-account flattening across
  venues, latest-wins conflict handling, all three staleness gates, and an
  in-process WebSocket server reconnect/accumulation test.
- 2 #[ignore]'d anvil integration tests: deploy a contract reverting unless
  storage slot 0 is nonzero, assert estimate_gas reverts without the override
  and succeeds with it, and that block overrides are accepted.
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@louisponet

louisponet commented Jul 6, 2026

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@louisponet louisponet marked this pull request as ready for review July 6, 2026 19:22
@louisponet louisponet requested a review from a team as a code owner July 6, 2026 19:22
@louisponet

Copy link
Copy Markdown
Author

recheck

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

@jmg-duarte jmg-duarte left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't finish reviewing yet but this is the lowest hanging fruit

tracing = { workspace = true }
url = { workspace = true }
tokio = { workspace = true, features = ["sync", "time", "rt", "macros"] }
tokio-tungstenite = { version = "0.28", features = ["rustls-tls-webpki-roots"] }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
tokio-tungstenite = { version = "0.28", features = ["rustls-tls-webpki-roots"] }
tokio-tungstenite = { workspace = true, features = ["rustls-tls-webpki-roots"] }

and add

tokio-tungstenite = { version = "0.28", default-features = false }

in the top level Cargo.toml

Comment on lines +61 to +66
fn default_override_max_age() -> Duration {
Duration::from_secs(3)
}

fn default_override_max_block_lag() -> u64 {
1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit

Suggested change
fn default_override_max_age() -> Duration {
Duration::from_secs(3)
}
fn default_override_max_block_lag() -> u64 {
1
const fn default_override_max_age() -> Duration {
Duration::from_secs(3)
}
const fn default_override_max_block_lag() -> u64 {
1

@MartinquaXD MartinquaXD left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The overall idea of the code (manage updates in background task and expose latest state via watcher) makes a lot of sense to me.
Just seems like the current PR could be simplified further.

Comment on lines +60 to +61
current_block_number: u64,
current_block_timestamp: u64,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of forcing the caller to tell us the current block the SimulationOverrides could also store its own CurrentBlockWatcher.

And more importantly since updates get streamed on a sub-block interval does it even make sense to think in terms of blocks here?

Comment on lines +126 to +130
impl SimulationsWithOverridesExt for IntCounterVec {
fn inc_by_result(&self, result: &str) {
self.with_label_values(&[result]).inc();
}
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This implementation is a footgun because it allows you to call this on metrics with an unexpected set of labels.

venues: frame_venues,
} = frame;

for (key, value) in frame_venues {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of parsing the data here why not configure serde to already deserialize the JSON string into those types?

}

Metrics::get().frames_received.inc();
match serde_json::from_str::<Frame>(message.to_text().unwrap_or("")) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
match serde_json::from_str::<Frame>(message.to_text().unwrap_or("")) {
match serde_json::from_slice::<Frame>(&message.into_data()) {

}

#[derive(Clone)]
pub struct SimulationOverrides {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's have SimulationOverrides(Arc<Inner>) where Inner contains the data the struct holds now.

Comment on lines +113 to +114
#[ignore = "requires anvil on PATH (or ANVIL_COMMAND set)"]
async fn estimate_gas_applies_streamed_state_override() {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you move this into the e2e crate there are already a bunch of helper function for setting stuff up.
Also if you call the test local_node_<TESTNAME> and #[ignore] it, CI will automatically run the test with a docker image that has anvil installed.

github-actions Bot added a commit that referenced this pull request Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants