Skip to content

Conversation

@dylanlott
Copy link
Contributor

@dylanlott dylanlott commented Nov 5, 2025

feat: adds host simulation to block build

This PR adds host simulation to the block build loop.

Note: Requires the signet-sdk release of 0.13.0​ in order to pass CI.

  • bumps bin-base to 0.15.1
  • deprecates the BuilderHelper​ wrapper contract submission in favor of the raw Zenith​ contract
  • wires up the HostProvider​ into the Simulator for host transaction simulation

Copy link
Contributor Author

dylanlott commented Nov 5, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@dylanlott dylanlott force-pushed the dylan/host-simulation branch from 9b8137b to 40a2b56 Compare November 5, 2025 19:38
@dylanlott dylanlott force-pushed the dylan/remove-builder-helper branch from 59afcac to cf31bc9 Compare November 5, 2025 19:38
Copy link
Contributor

@anna-carroll anna-carroll left a comment

Choose a reason for hiding this comment

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

nits

@dylanlott dylanlott marked this pull request as ready for review November 7, 2025 18:12
@dylanlott dylanlott requested a review from Evalir as a code owner November 7, 2025 18:12
@dylanlott dylanlott requested a review from prestwich November 7, 2025 18:55
@dylanlott dylanlott changed the base branch from dylan/remove-builder-helper to graphite-base/167 November 7, 2025 18:56
- bumps bin-base to 0.15.1
- deprecates the BuilderHelper submission call in favor of the
Zenith Block and Block Header
- removes the BuilderHelper submit task
- wires up the HostProvider into the Simulator for host simulation
- cleans up unused code
@graphite-app
Copy link

graphite-app bot commented Nov 7, 2025

Merge activity

  • Nov 7, 6:57 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

Copy link
Contributor

@anna-carroll anna-carroll left a comment

Choose a reason for hiding this comment

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

need to leave keyboard for a sec but need to keep looking deeper at this code to determine what is supposed to be correct here. leaving these comments for now

(pretty much mirroring @Evalir 's callout above)

@dylanlott dylanlott requested a review from prestwich November 10, 2025 22:08
@prestwich
Copy link
Member

yeah its a bit unpleasant. could dry parts of it i think but other parts need to be separate

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