Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: Trusted builder MVP #39

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

bharath-123
Copy link
Contributor

@bharath-123 bharath-123 commented Aug 2, 2024

Trusted Builder MVP

This PR contains the changes made in Astria-geth for the trusted builder. A trusted builder is an entity responsible for sending bundles of transactions to sequencer via composer. These transactions are package in a special sequence action called BuilderBundlePacket (Name subject to change) . These txs are placed at the top-of-block of the block being built for the Flame node.

More details on the design can be viewed at: https://www.notion.so/astria-org/Trusted-Block-Builder-MVP-8eba7cd7cbd34e4e90a4adf726879028?pvs=4

The following are the changes made:

  1. Reference the TrustedBuilder related proto changes by updating the go.mod. Changes are listed: https://buf.build/astria/composer-apis/docs/1356/merge:astria.composer.v1alpha1
  2. Logic to check if a BuilderBundlePacket was sent and to unbundle it.
  3. Currently, when a tx reverts an error is not thrown. Since we want to remove reverted txs, we can throw an error for reverting txs in commitTransactions method and add the tx to the AstriaExcludedBlock
  4. Support simulateOnly mode where we do not insert the built block to the chain
  5. Return the txs that have been included in the block.
  6. Add tests

@bharath-123 bharath-123 force-pushed the ENG-626/execution-api-compaitable branch from 2743416 to aa48e8e Compare August 2, 2024 18:13
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.

1 participant