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
contracts-bedrock: move immutables to storage for L1CrossDomainMessenger #8631
Conversation
Warning Rate Limit Exceeded@0xfuturistic has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 15 minutes and 35 seconds before requesting another review. How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. WalkthroughThe overall changes suggest a significant update to the Ethereum Optimism codebase, focusing on cross-domain messaging and synchronization logic. The updates include modifications to the Cross-Domain Messenger contracts across both L1 and L2, a change in gas benchmarks, refactoring of profiling and metrics services, and the removal of sync target logic. The versioning adjustments and the addition of new flags and configurations indicate a move towards a more flexible and potentially more performant system. Changes
Note: The file paths have been shortened with ellipsis (...) to group similar changes and save space. Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat/mcp-l1 #8631 +/- ##
===============================================
+ Coverage 25.69% 25.72% +0.03%
===============================================
Files 119 117 -2
Lines 4889 4844 -45
Branches 1063 1057 -6
===============================================
- Hits 1256 1246 -10
+ Misses 3527 3494 -33
+ Partials 106 104 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
|
You should run |
cfcd4bd
to
e6d8872
Compare
76d20a1
to
cab3bc6
Compare
Semgrep found 1
Iteration over a possibly empty map |
cab3bc6
to
e819800
Compare
packages/contracts-bedrock/snapshots/storageLayout/L1CrossDomainMessenger.json
Outdated
Show resolved
Hide resolved
packages/contracts-bedrock/snapshots/storageLayout/L2CrossDomainMessenger.json
Show resolved
Hide resolved
packages/contracts-bedrock/src/universal/CrossDomainMessenger.sol
Outdated
Show resolved
Hide resolved
a59caba
to
9d5f0d5
Compare
9d5f0d5
to
53ffba4
Compare
packages/contracts-bedrock/src/universal/CrossDomainMessenger.sol
Outdated
Show resolved
Hide resolved
packages/contracts-bedrock/src/universal/CrossDomainMessenger.sol
Outdated
Show resolved
Hide resolved
packages/contracts-bedrock/src/universal/CrossDomainMessenger.sol
Outdated
Show resolved
Hide resolved
…2CrossDomainMessenger
…2CrossdomainMessenger
…inst DeployConfig
d153a41
to
2e11da8
Compare
2e11da8
to
d0ca276
Compare
Good after https://github.com/ethereum-optimism/optimism/pull/8631/files#r1448073415 is addressed |
…grade script for L1CrossDomainMessenger
context: https://github.com/ethereum-optimism/client-pod/issues/265
security review (wip): https://www.notion.so/oplabs/DRAFT-Security-Review-Move-Immutables-to-Storage-in-L1-Contracts-56c07789fa454d8fb2df80f0b84a483b?pvs=4
legacy PR: #6574