Skip to content

Comments

feat: update operator fee contract bytecodes#613

Merged
sebastianst merged 8 commits intoethereum-optimism:mainfrom
succinctlabs:update-operator-fee-contract-bytecodes
Mar 12, 2025
Merged

feat: update operator fee contract bytecodes#613
sebastianst merged 8 commits intoethereum-optimism:mainfrom
succinctlabs:update-operator-fee-contract-bytecodes

Conversation

@leruaa
Copy link
Contributor

@leruaa leruaa commented Mar 10, 2025

Description

  • Update operator fee L2 contract bytecodes.
  • Specify the behavior on receipts

Closes #610

Tests

Additional context

Metadata

@tynes
Copy link
Contributor

tynes commented Mar 10, 2025

Which commit was this generated from?

@tynes
Copy link
Contributor

tynes commented Mar 10, 2025

We need to replace the following text:

git checkout TODO
make build-contracts
jq -r ".bytecode.object" packages/contracts-bedrock/forge-artifacts/L1Block.sol/L1Block.json

@tynes
Copy link
Contributor

tynes commented Mar 10, 2025

I think it would be ideal to use a tagged release, potentially https://github.com/ethereum-optimism/optimism/tree/op-contracts/v3.0.0-rc.1 - does this include the feature complete code?

@refcell
Copy link
Contributor

refcell commented Mar 10, 2025

I think it would be ideal to use a tagged release, potentially https://github.com/ethereum-optimism/optimism/tree/op-contracts/v3.0.0-rc.1 - does this include the feature complete code?

Nice! This does include ethereum-optimism/optimism#12166 and it looks like any follow up work that touches contracts is included (the OPCM update in ethereum-optimism/optimism#14662).

So I believe this should work.

@leruaa leruaa force-pushed the update-operator-fee-contract-bytecodes branch from eb896f3 to 4d16388 Compare March 11, 2025 19:42
Copy link
Contributor

@refcell refcell left a comment

Choose a reason for hiding this comment

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

Update operator fee vault code hash and add verify instructions below

Copy link
Contributor

@refcell refcell left a comment

Choose a reason for hiding this comment

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

Very nice!

Copy link
Member

@sebastianst sebastianst left a comment

Choose a reason for hiding this comment

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

approving per @refcell's verification, thanks!

@sebastianst sebastianst merged commit e704e6c into ethereum-optimism:main Mar 12, 2025
1 check passed
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.

Isthmus: specs updates

5 participants