Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughThe PR deletes two changeset files and updates version numbers and changelogs across three packages to document new crossChainBridging support with the 'erc7786native' option and associated breaking changes for the 'custom' option. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~5 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
packages/core/solidity/CHANGELOG.md (1)
4-9: Add a short migration checklist for the breaking change.The entry flags breaking behavior, but it would be safer to include explicit migration steps (required access-control setting and new constructor/initializer parameter order) to prevent integration mistakes.
✍️ Suggested changelog refinement
- Solidity `erc20`, `stablecoin`, `realWorldAsset`: Support 'erc7786native' option for `crossChainBridging`. ([`#747`](https://github.com/OpenZeppelin/contracts-wizard/pull/747)) - Uses OpenZeppelin Contracts 5.6.0 - **Breaking changes**: Solidity `erc20`, `stablecoin`, `realWorldAsset`: 'custom' option for `crossChainBridging` now requires access control, and adds a function to allow updating the token bridge address after deployment. Constructor/initializer parameter order changed due to access control requirement. + - **Migration notes**: + - Enable an access-control mode when using `'custom'` `crossChainBridging`. + - Update constructor/initializer argument ordering to match the new access-control flow. + - If applicable, wire the new token bridge updater function into operational runbooks.Based on learnings: In OpenZeppelin contracts-wizard, breaking changes that have concrete migration paths can be handled as minor version bumps.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@packages/core/solidity/CHANGELOG.md` around lines 4 - 9, The changelog entry for 0.10.7 notes breaking changes but lacks explicit migration steps; update the Solidity changelog entry for the affected contracts (erc20, stablecoin, realWorldAsset) to include a short migration checklist describing: (1) that the 'custom' crossChainBridging option now requires access control and which access role or modifier to configure post-deployment, (2) that a new setter function was added to update the token bridge address and how to call it, and (3) the new constructor/initializer parameter order and the exact parameter mapping to previous versions so integrators can reorder arguments correctly; reference the symbols crossChainBridging, erc7786native, and the new bridge-update function name in the checklist for clarity.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@packages/mcp/CHANGELOG.md`:
- Line 9: The release note includes a duplicated commit reference for hash
32ff02c3f7fff6687bcaec52d15ce28763f96840 in the dependency line (the bracketed
list after "Updated dependencies"); edit that line in CHANGELOG.md to remove the
repeated reference so the commit hash appears only once, preserving the
surrounding markdown and link format.
---
Nitpick comments:
In `@packages/core/solidity/CHANGELOG.md`:
- Around line 4-9: The changelog entry for 0.10.7 notes breaking changes but
lacks explicit migration steps; update the Solidity changelog entry for the
affected contracts (erc20, stablecoin, realWorldAsset) to include a short
migration checklist describing: (1) that the 'custom' crossChainBridging option
now requires access control and which access role or modifier to configure
post-deployment, (2) that a new setter function was added to update the token
bridge address and how to call it, and (3) the new constructor/initializer
parameter order and the exact parameter mapping to previous versions so
integrators can reorder arguments correctly; reference the symbols
crossChainBridging, erc7786native, and the new bridge-update function name in
the checklist for clarity.
ℹ️ Review info
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
.changeset/ninety-ears-slide.md.changeset/sad-cases-yawn.mdpackages/common/CHANGELOG.mdpackages/common/package.jsonpackages/core/solidity/CHANGELOG.mdpackages/core/solidity/package.jsonpackages/mcp/CHANGELOG.mdpackages/mcp/package.json
💤 Files with no reviewable changes (2)
- .changeset/sad-cases-yawn.md
- .changeset/ninety-ears-slide.md
packages/mcp/CHANGELOG.md
Outdated
| - Solidity `erc20`, `stablecoin`, `realWorldAsset`: Support 'erc7786native' option for `crossChainBridging`. ([#747](https://github.com/OpenZeppelin/contracts-wizard/pull/747)) | ||
| - Uses OpenZeppelin Contracts 5.6.0 | ||
| - **Breaking changes**: Solidity `erc20`, `stablecoin`, `realWorldAsset`: 'custom' option for `crossChainBridging` now requires access control, and adds a function to allow updating the token bridge address after deployment. Constructor/initializer parameter order changed due to access control requirement. | ||
| - Updated dependencies [[`32ff02c`](https://github.com/OpenZeppelin/contracts-wizard/commit/32ff02c3f7fff6687bcaec52d15ce28763f96840), [`32ff02c`](https://github.com/OpenZeppelin/contracts-wizard/commit/32ff02c3f7fff6687bcaec52d15ce28763f96840)]: |
There was a problem hiding this comment.
Deduplicate the repeated commit reference in dependency notes.
The same commit hash appears twice in the bracketed reference list, which is noisy in release docs.
🧹 Suggested cleanup
-- Updated dependencies [[`32ff02c`](https://github.com/OpenZeppelin/contracts-wizard/commit/32ff02c3f7fff6687bcaec52d15ce28763f96840), [`32ff02c`](https://github.com/OpenZeppelin/contracts-wizard/commit/32ff02c3f7fff6687bcaec52d15ce28763f96840)]:
+- Updated dependencies [[`32ff02c`](https://github.com/OpenZeppelin/contracts-wizard/commit/32ff02c3f7fff6687bcaec52d15ce28763f96840)]:📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - Updated dependencies [[`32ff02c`](https://github.com/OpenZeppelin/contracts-wizard/commit/32ff02c3f7fff6687bcaec52d15ce28763f96840), [`32ff02c`](https://github.com/OpenZeppelin/contracts-wizard/commit/32ff02c3f7fff6687bcaec52d15ce28763f96840)]: | |
| - Updated dependencies [[`32ff02c`](https://github.com/OpenZeppelin/contracts-wizard/commit/32ff02c3f7fff6687bcaec52d15ce28763f96840)]: |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@packages/mcp/CHANGELOG.md` at line 9, The release note includes a duplicated
commit reference for hash 32ff02c3f7fff6687bcaec52d15ce28763f96840 in the
dependency line (the bracketed list after "Updated dependencies"); edit that
line in CHANGELOG.md to remove the repeated reference so the commit hash appears
only once, preserving the surrounding markdown and link format.
1e1a9ac to
f2e8252
Compare
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and publish to npm yourself or setup this action to publish automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.
Releases
@openzeppelin/wizard-common@0.4.4
Changelog
0.4.4 (2026-02-26)
erc20,stablecoin,realWorldAsset: Support 'erc7786native' option forcrossChainBridging. (#747)0.4.3 (2026-02-17)
Cairo: added AI prompt descriptions for: (#774)
Add support for Confidential Contracts Wizard (#652)
0.4.2 (2026-01-29)
0.4.1 (2026-01-14)
0.4.0 (2026-01-10)
Add support for
with_componentsmacro. (#703)Add AccessControlDefaultAdminRules. (#698)
Breaking changes:
0.3.3 (2025-11-28)
0.3.2 (2025-11-26)
0.3.1 (2025-11-12)
WebAuthnto the list of signers available. (#718)custodianoption tofreezable. Replace ERC20Custodian with ERC20Freezable. (#719)0.2.0 (2025-11-03)
limitationsoption torestrictions. Replace ERC20Allowlist and ERC20Blocklist with ERC20Restricted. (#715)@openzeppelin/contractsand@openzeppelin/contracts-upgradeabledependencies to 5.5.0 (#681)ERC7702option is renamed asEIP7702. Imported contractSignerERC7702is renamed asSignerEIP7702.InitializableandUUPSUpgradeableare imported from@openzeppelin/contractsinstead of@openzeppelin/contracts-upgradeable.0.1.2 (2025-10-29)
0.1.1 (2025-09-16)
0.1.0 (2025-08-15)
0.0.3 (2025-08-12)
0.0.2 (2025-07-03)
0.0.1 (2025-06-20)
@openzeppelin/wizard@0.10.7
Changelog
0.10.7 (2026-02-26)
erc20,stablecoin,realWorldAsset: Support 'erc7786native' option forcrossChainBridging. (#747)erc20,stablecoin,realWorldAsset: 'custom' option forcrossChainBridgingnow requires access control, and adds a function to allow updating the token bridge address after deployment. Constructor/initializer parameter order changed due to access control requirement.0.10.6 (2026-02-17)
0.10.5 (2026-01-27)
0.10.4 (2025-11-28)
0.10.3 (2025-11-26)
0.10.2 (2025-11-12)
WebAuthnto the list of signers available. (#718)custodianoption tofreezable. Replace ERC20Custodian with ERC20Freezable. (#719)0.10.0 (2025-11-03)
Update
@openzeppelin/contractsand@openzeppelin/contracts-upgradeabledependencies to 5.5.0 (#681)ERC7702option is renamed asEIP7702. Imported contractSignerERC7702is renamed asSignerEIP7702.InitializableandUUPSUpgradeableare imported from@openzeppelin/contractsinstead of@openzeppelin/contracts-upgradeable.Breaking changes: Solidity Stablecoin and RWA: Change
limitationsoption torestrictions. Replace ERC20Allowlist and ERC20Blocklist with ERC20Restricted. (#715)0.9.0 (2025-10-29)
tokenBridgewhen cross-chain bridging is set to'custom'and upgradeability is enabled._nextTokenIdwhen mintable, auto increment IDs, and upgradeability are enabled.0.8.1 (2025-10-14)
0.8.0 (2025-09-16)
SignerECDSA,SignerP256,SignerRSA,SignerERC7702,SignerERC7913,MultiSignerERC7913andMultiSignerERC7913Weighted(#609)AccountERC7579,AccountERC7579Hooked,SignerECDSA,SignerP256,SignerRSA,SignerERC7702,SignerERC7913andMultiSignerERC7913(#609)Account,AccountERC7579,AccountERC7579Hooked,ERC7812,ERC7739Utils,ERC7913Utils,AbstractSigner,SignerECDSA,SignerP256,SignerRSA,SignerERC7702,SignerERC7913,MultiSignerERC7913, andMultiSignerERC7913Weightedfrom OpenZeppelin Contracts 5.4.0 instead of Community Contracts (#609)0.7.1 (2025-08-15)
0.7.0 (2025-08-12)
0.6.0 (2025-06-20)
Add support for Wizard MCP server. (#569)
Governor: Remove usage ofaccessoption. This option now has no effect.Accounts: Add_disableInitializers()to account implementations (#568)0.5.6 (2025-05-21)
MultisigERC7913: AddonlyEntryPointOrSelfmodifier to public configuration functions. (#554)onlyGovernanceto restrict upgrades for Governor with UUPS (#544)_authorizeUpgradefunction is restricted byonlyGovernanceinstead ofonlyOwner0.5.5 (2025-05-13)
accountcontract types for ERC-4337. (#486, #523, #527)@openzeppelin/community-contracts. (#537)0.5.4 (2025-04-01)
callbackin ERC20 features. (#500)0.5.3 (2025-03-13)
Add ERC20 Cross-Chain Bridging, SuperchainERC20. (#436)
Note: Cross-Chain Bridging is experimental and may be subject to change.
Potentially breaking changes:
recipientwhen usingpremint.0.5.2 (2025-02-21)
0.5.1 (2025-02-05)
recipientwhen usingpremintinerc20,stablecoin, andrealWorldAsset. (#435)0.5.0 (2025-01-23)
Update to use TypeScript v5. (#231)
Remove unused dependencies. (#430)
Breaking changes:
0.4.6 (2024-11-20)
0.4.5 (2024-11-18)
stablecoinandrealWorldAssetcontract types. (#404)Note:
stablecoinandrealWorldAssetare experimental and may be subject to change.0.4.4 (2024-10-23)
Potentially breaking changes
0.4.3 (2024-04-08)
0.4.2 (2024-02-22)
0.4.1 (2023-10-18)
managedaccess control option for use with AccessManager. (#298)0.4.0 (2023-10-05)
Breaking changes
0.3.0 (2023-05-25)
governor. (#258)0.2.3 (2023-03-23)
0.2.2 (2023-03-17)
0.2.1 (2023-03-17)
@openzeppelin/contracts. (#233)0.2.0 (2022-11-08)
governor. (fdcf912)0.1.1 (2022-06-30)
0.1.0 (2022-06-15)
@openzeppelin/contracts-mcp@0.5.5
Changelog
0.5.5 (2026-02-26)
erc20,stablecoin,realWorldAsset: Support 'erc7786native' option forcrossChainBridging. (#747)erc20,stablecoin,realWorldAsset: 'custom' option forcrossChainBridgingnow requires access control, and adds a function to allow updating the token bridge address after deployment. Constructor/initializer parameter order changed due to access control requirement.f2e8252,32ff02c,32ff02c]:0.5.4 (2026-02-18)
0.5.3 (2026-02-17)
79b4aeb,17d9de8,17d9de8]:0.5.2 (2026-01-29)
86fe280]:0.5.1 (2026-01-14)
aa0f0d2]:0.5.0 (2026-01-10)
Add support for
with_componentsmacro. (#703)Add AccessControlDefaultAdminRules. (#698)
Breaking changes:
Updated dependencies [
53c042a]:0.4.6 (2025-12-12)
ce60539]:0.4.5 (2025-12-08)
0.4.4 (2025-11-28)
985d5a4]:0.4.3 (2025-11-26)
3bb0213,57dabc8]:0.4.2 (2025-11-12)
WebAuthnto the list of signers available. (#718)be91f8f]:custodianoption tofreezable. Replace ERC20Custodian with ERC20Freezable. (#719)6b8f8f5,c14be02,bec86b5,ce8fbba,5c1fa0f]:0.4.0 (2025-11-03)
Update
@openzeppelin/contractsand@openzeppelin/contracts-upgradeabledependencies to 5.5.0 (#681)ERC7702option is renamed asEIP7702. Imported contractSignerERC7702is renamed asSignerEIP7702.InitializableandUUPSUpgradeableare imported from@openzeppelin/contractsinstead of@openzeppelin/contracts-upgradeable.Breaking changes: Solidity Stablecoin and RWA: Change
limitationsoption torestrictions. Replace ERC20Allowlist and ERC20Blocklist with ERC20Restricted. (#715)Updated dependencies [
880b9b8,b49e056]:0.3.0 (2025-10-29)
tokenBridgewhen cross-chain bridging is set to'custom'and upgradeability is enabled._nextTokenIdwhen mintable, auto increment IDs, and upgradeability are enabled.38da80c,0f0509d]:0.2.0 (2025-09-16)
SignerECDSA,SignerP256,SignerRSA,SignerERC7702,SignerERC7913,MultiSignerERC7913andMultiSignerERC7913Weighted(#609)AccountERC7579,AccountERC7579Hooked,SignerECDSA,SignerP256,SignerRSA,SignerERC7702,SignerERC7913andMultiSignerERC7913(#609)Account,AccountERC7579,AccountERC7579Hooked,ERC7812,ERC7739Utils,ERC7913Utils,AbstractSigner,SignerECDSA,SignerP256,SignerRSA,SignerERC7702,SignerERC7913,MultiSignerERC7913, andMultiSignerERC7913Weightedfrom OpenZeppelin Contracts 5.4.0 instead of Community Contracts (#609)41d5c74,b9d58dc,41d5c74,41d5c74,41d5c74,029790c]:0.1.1 (2025-09-02)
0.1.0 (2025-08-15)
2bb2a16,c65acb7]:0.0.7 (2025-08-12)
f39adfd]:0.0.6 (2025-07-29)
0.0.5 (2025-07-25)
@openzeppelin/wizard-mcpto@openzeppelin/contracts-mcp(#607)0.0.4 (2025-07-22)
9e61c0f]:0.0.3 (2025-07-03)
4b86b07,4b86b07]:0.0.2 (2025-06-27)
8997a89]:0.0.1 (2025-06-20)