Skip to content

Comments

chore; update oft lazyer zero guide for prod #137

Open
okjintao wants to merge 6 commits intomainfrom
feature/layerzero-oft-dvn-configs
Open

chore; update oft lazyer zero guide for prod #137
okjintao wants to merge 6 commits intomainfrom
feature/layerzero-oft-dvn-configs

Conversation

@okjintao
Copy link
Contributor

@okjintao okjintao commented Oct 3, 2025

Guide Contribution PR

Description

Type of Change

  • ✨ New guide/example
  • 📝 Update to existing guide/example
  • 🐛 Bug fix in a guide/example
  • 🎨 Style/formatting update
  • ♻️ Refactor (no functional changes)
  • Other (please describe):

Guide Details

  • Guide Name:
  • Category: (e.g., Smart Contracts, Frontend, Infrastructure, Tooling)
  • Dependencies Added:
  • Network(s) Used: (e.g., Bepolia, Mainnet, Local)
  • Entry Point: (e.g., script, README section, etc.)

Checklist

  • The guide/example is clear and easy to follow
  • All steps are reproducible (instructions/scripts work as described)
  • All new/changed code is commented where necessary
  • Documentation is updated (README, inline docs, etc.)
  • No sensitive information or secrets are included
  • Example runs without errors or warnings
  • (If applicable) Screenshots or output are included for clarity

Additional Notes

- Fixed Base EID from 30110 (Arbitrum) to 30184 (Base)
- Configured DVNs for both Base and Berachain receive operations
- Configured executors on both chains with correct addresses
- Set up library configurations (send/receive) for both chains
- Added comprehensive configuration and verification scripts
- Updated all scripts to use correct Base EID 30184
- Added executor configuration scripts for both chains
- Added DVN configuration scripts with proper ULN config
- Added verification scripts to check complete setup
- Formatted all Solidity code with forge fmt

Key fixes:
- Corrected Base EID to 30184 throughout all scripts
- Configured peer connections for correct EID
- Set executor addresses: Base (0x2CCA08...) and Berachain (0x4208D6...)
- Configured ULN (DVN) settings matching LayerZero documentation
- All configurations verified on-chain
- Clean up all script logging to be more natural and human-like
- Remove verbose comments and formal logging patterns
- Add documentation about Bera DVN availability limitation
- Fix Base endpoint ID in MyOFT.s.sol (30110 -> 30184)
- Reorganize scripts: remove old config scripts, add new peer and config scripts
- Format all code with forge fmt
- Update receive configs to use 20 block confirmations (matching send configs)
- Previously receive configs used 15, send configs used 20
- All configs now consistently use 20 block confirmations for better finality
- Update README documentation to reflect the change
@camembera
Copy link
Contributor

@okjintao needs merge conflict repair
God, I hate package-lock.json

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.

2 participants