Skip to content

feat: simulate sponsored txs before broadcast#150

Merged
emmajam merged 5 commits into
tempoxyz:mainfrom
stevencartavia:sponsor-presimulate-cosign
Jun 18, 2026
Merged

feat: simulate sponsored txs before broadcast#150
emmajam merged 5 commits into
tempoxyz:mainfrom
stevencartavia:sponsor-presimulate-cosign

Conversation

@stevencartavia

Copy link
Copy Markdown
Contributor

Adds pre-broadcast simulation for locally co-signed sponsored Tempo charges. Before broadcasting, the server simulates the co-signed tx via tempo_simulateV1 and refuses to broadcast (fail-closed, including on RPC error) if it would revert, so the sponsor never pays gas for a failing transaction.

Part of OSS-314

@grandizzy grandizzy 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.

Please check blocker below.

Comment thread src/mpp/methods/tempo/intents.py Outdated

@figtracer figtracer 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.

left one inline comment.

Comment thread tests/test_tempo_integration.py
Comment thread src/mpp/methods/tempo/intents.py Outdated

@mablr mablr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SG, but the CI failure may require a fix on mpp-tools side ?

@emmajam emmajam merged commit 514ab1c into tempoxyz:main Jun 18, 2026
13 checks passed
@github-actions github-actions Bot mentioned this pull request Jun 18, 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.

5 participants