Feature: oft adapter step3 - deploy script and test by blueogin · Pull Request #11 · GoodDollar/GoodBridge

blueogin · 2026-01-30T12:51:02Z

Description

This PR implements the deployment scripts and testing infrastructure for the GoodDollar OFT (Omnichain Fungible Token) adapter system. The implementation enables cross-chain bridging of GoodDollar tokens between XDC and CELO networks using LayerZero's OFT protocol.

About #7

How Has This Been Tested?

https://layerzeroscan.com/tx/0x3575146c0e395d46b4a3e09ec3ae79e1005ea6d4461765d6dc19d6aebed512bb
https://layerzeroscan.com/tx/0xa267c36a25337ea8d45d83de83a3e83a291ba4d2eaf0fd0393d365faf3c078e8

Checklist:

…acts, including enabling hardhat-deploy and adding new network configurations for CELO and XDC

… management structures, and add deployment and configuration scripts for cross-chain functionality

…C network, enhance GoodDollarOFTAdapter with upgrade authorization event, and modify deployment scripts for improved functionality

…ant fee deduction details

sourcery-ai

Sorry @blueogin, your pull request is larger than the review limit of 150000 diff characters

sirpy · 2026-02-02T07:39:27Z

packages/bridge-contracts/contracts/oft/GoodDollarOFTAdapter.sol

+    function _authorizeUpgrade(address newImplementation) internal override onlyOwner {
+        // Authorization is handled by onlyOwner modifier
+        // Additional checks can be added here if needed
+        emit AuthorizedUpgrade(newImplementation);


no required

Okay, It was required for UUPSUpgradeable contract.
I will use transparent model and remove that function

i ment that the emit event is not requred. we should use uups

packages/bridge-contracts/release/deployment.json

sirpy · 2026-02-02T07:51:35Z

packages/bridge-contracts/scripts/oft/configure-oft-xdc-celo.sh

+# 6. Test bridge functionality (optional, last step)
+#
+# Usage:
+#   ./scripts/multichain-deploy/oft/configure-oft-xdc-celo.sh


In general scripts should be hardhat scripts in typescript. not shell.

the script should configure a single network ie configure-oft.ts --network development-xdc

the script should be a hardhat script so --network is available

all config values should be read from a json file ie oft.config.json (import config from './oft.config.json')
where each network/env has its entry in the config file.

This file is rarely used in a production environment and is intended mainly as a reference.
Also, including all configuring logic into 1 script is not ideal I think
I’ll add an OFT_CONFIGURING_GUIDE file under scripts/oft to explain the configuration steps.

we use hadhat-deploy scripts , see messagepassingbridge

sirpy · 2026-02-02T07:52:15Z

packages/bridge-contracts/scripts/oft/configure-oft-xdc-celo.sh

+echo ""
+
+# Step 7: Test bridge (optional, last step)
+if [ "$SKIP_BRIDGE_TEST" != "true" ]; then


should be separated into a different script. ( a hardhat script)

packages/bridge-contracts/scripts/oft/oft-deploy.ts

packages/bridge-contracts/.env.sample

packages/bridge-contracts/hardhat.config.ts

packages/bridge-contracts/contracts/oft/GoodDollarOFTAdapter.sol

… unused dependencies in bridge-contracts

openzeppelin-code · 2026-02-02T12:54:33Z

Feature: oft adapter step3 - deploy script and test

Generated at commit: 52284e0ccec3a15e0ddda9010d2e88c618bd0b2d

🚨 Report Summary

	Severity Level	Results
Contracts	Critical High Medium Low Note Total	0 1 0 9 36 46
Dependencies	Critical High Medium Low Note Total	0 0 0 0 0 0

For more details view the full report in OpenZeppelin Code Inspector

…nd to transparent in GoodDollarOFTAdapter and deployment script

…pts to reference new deployment file

…nd update set-minter-burner-limits script to utilize new config

…nd CELO networks

…t for cleaner implementation

…and integrating IMessagePassingBridge for limit management

sirpy · 2026-02-04T10:12:15Z

packages/bridge-contracts/scripts/oft/oft-deploy.ts

    MinterBurner = await upgrades.deployProxy(
      MinterBurnerFactory,
      [nameServiceAddress],
-      { kind: "uups", initializer: "initialize" }


i dont understand. why this was changed. we use uups

sirpy · 2026-02-04T10:14:53Z

packages/bridge-contracts/scripts/oft/oft-deploy.ts

 import fse from "fs-extra";
 import Contracts from "@gooddollar/goodprotocol/releases/deployment.json";
-import release from "../../release/deployment.json";
+import release from "../../release/deployment-oft.json";


why are we not using hardhat-deploy for deployment scripts

sirpy · 2026-02-04T10:16:23Z

packages/bridge-contracts/scripts/oft/OFT_CONFIGURING_GUIDE.md

+
+**Note**: Limit values can be specified in decimal format (e.g., "5000" for 5,000 G$). The scripts automatically convert them to wei (18 decimals).
+
+## Manual Configuration: Step-by-Step


we should have a hardhat deploy script just like messagepassingbridge has

sirpy · 2026-02-04T10:17:07Z

packages/bridge-contracts/contracts/oft/GoodDollarOFTAdapter.sol

+    function _authorizeUpgrade(address newImplementation) internal override onlyOwner {
+        // Authorization is handled by onlyOwner modifier
+        // Additional checks can be added here if needed
+        emit AuthorizedUpgrade(newImplementation);


i ment that the emit event is not requred. we should use uups

sirpy · 2026-02-04T10:18:51Z

packages/bridge-contracts/contracts/oft/GoodDollarOFTAdapter.sol

-import { UUPSUpgradeable } from "@openzeppelin/contracts-upgradeable/proxy/utils/UUPSUpgradeable.sol";
-import { OwnableUpgradeable } from "@openzeppelin/contracts-upgradeable/access/OwnableUpgradeable.sol";
 import { IMintableBurnable } from "@layerzerolabs/oft-evm/contracts/interfaces/IMintableBurnable.sol";
+import { BridgeHelperLibrary } from "../messagePassingBridge/BridgeHelperLibrary.sol";


i'm totally confused by this. in the other PRs you dont use bridgehelperlibrary.
you need to be consistent

IMessagePassingBridge and BridgeHelperLibrary imports are not necessary - will remove it

…code clarity and reduce dependencies

…te request approval mechanism to use bytes32 IDs, and implement request ID generation for improved limit management

… MinterBurner contracts

…oved clarity, implement request approval checks, and enforce limits on sending and receiving functions

…8.9 with optimizer settings and modify LayerZero config to reference the correct deployment file

…zation for enhanced fee management

…ove legacy deployment script; add set-oft-operator script for DAO governance integration

…directory structure

…yment scripts for improved clarity and functionality

…est handling in GoodDollarOFTAdapter, enhancing error management and contract clarity

…on and update tests for new constructor argument

…ollarOFTAdapter with implementation address retrieval and verification functionality

…oved error handling in receive requests

…DollarOFTAdapter, and adjust bridging amount and transaction limits in configuration

blueogin added 6 commits January 26, 2026 14:51

chore: update hardhat configuration and dependencies for bridge contr…

7d00674

…acts, including enabling hardhat-deploy and adding new network configurations for CELO and XDC

feat: enhance GoodDollarOFTAdapter with new bridge limits and account…

6a11b79

… management structures, and add deployment and configuration scripts for cross-chain functionality

Merge branch 'feat/oft-adapter-step2' into feat/oft-adatper-step3

b54c09b

feat: add .env.sample for configuration, update Hardhat config for XD…

56be154

…C network, enhance GoodDollarOFTAdapter with upgrade authorization event, and modify deployment scripts for improved functionality

Merge branch 'feat/oft-adapter-step2' into feat/oft-adatper-step3

3b1ea49

fix: clarify documentation in GoodDollarOFTAdapter by removing redund…

b45c2ae

…ant fee deduction details

sourcery-ai bot reviewed Jan 30, 2026

View reviewed changes

chore: remove outdated verification script for GoodDollar OFT contracts

89fae4a

sirpy changed the base branch from master to feat/oft-adapter-step2 February 2, 2026 07:37

sirpy requested changes Feb 2, 2026

View reviewed changes

sirpy reviewed Feb 2, 2026

View reviewed changes

packages/bridge-contracts/contracts/oft/GoodDollarOFTAdapter.sol Show resolved Hide resolved

refactor: update ISuperGoodDollar interface implementation and remove…

76e9709

… unused dependencies in bridge-contracts

blueogin added 9 commits February 2, 2026 08:48

Merge branch 'feat/oft-adapter-step2' into feat/oft-adatper-step3

d77238d

refactor: remove upgrade authorization logic and change deployment ki…

fb5d269

…nd to transparent in GoodDollarOFTAdapter and deployment script

feat: introduce deployment-oft.json for OFT contracts and update scri…

1d08bce

…pts to reference new deployment file

feat: add oft.config.json for network-specific limit configurations a…

124a1a1

…nd update set-minter-burner-limits script to utilize new config

docs: add OFT configuration guide for cross-chain setup between XDC a…

d1dc1d4

…nd CELO networks

refactor: remove unused initialization code from OFT deployment scrip…

9d11522

…t for cleaner implementation

chore: enable contract size checking on compile in Hardhat configuration

c5b45df

refactor: streamline GoodDollarOFTAdapter by removing unused structs …

e907906

…and integrating IMessagePassingBridge for limit management

Merge branch 'feat/oft-adapter-step2' into feat/oft-adatper-step3

5c4c510

sirpy requested changes Feb 4, 2026

View reviewed changes

blueogin added 7 commits February 4, 2026 09:36

refactor: remove unused imports from GoodDollarOFTAdapter to enhance …

30178c0

…code clarity and reduce dependencies

refactor: enhance GoodDollarOFTAdapter with UUPS upgradeability, upda…

71da58c

…te request approval mechanism to use bytes32 IDs, and implement request ID generation for improved limit management

refactor: update OFT deployment script to use UUPS upgradeability for…

e67caf1

… MinterBurner contracts

refactor: reorganize GoodDollarOFTAdapter contract structure for impr…

52284e0

…oved clarity, implement request approval checks, and enforce limits on sending and receiving functions

refactor: update Hardhat configuration to include Solidity version 0.…

0f8d105

…8.9 with optimizer settings and modify LayerZero config to reference the correct deployment file

refactor: add feeRecipient parameter to GoodDollarOFTAdapter initiali…

a4b2c68

…zation for enhanced fee management

feat: introduce deployOFT script for GoodDollar OFT contracts and rem…

b678f7b

…ove legacy deployment script; add set-oft-operator script for DAO governance integration

blueogin added 9 commits February 10, 2026 09:03

fix: update contract path in grant-minter-role script to reflect new …

dcfc3fb

…directory structure

feat: add deployment configuration for OFT contracts and update deplo…

866e1d1

…yment scripts for improved clarity and functionality

refactor: replace request approval mechanism with failed receive requ…

6ccca46

…est handling in GoodDollarOFTAdapter, enhancing error management and contract clarity

Merge branch 'feat/oft-adapter-step2' into feat/oft-adatper-step3

73f995b

feat: add feeRecipient parameter to GoodDollarOFTAdapter initializati…

5f6ed35

…on and update tests for new constructor argument

Merge branch 'feat/oft-adapter-step2' into feat/oft-adatper-step3

2ac107d

feat: enhance deployment scripts for GoodDollarMinterBurner and GoodD…

c859ba3

…ollarOFTAdapter with implementation address retrieval and verification functionality

feat: add ReceiveRequestFailed event to GoodDollarOFTAdapter for impr…

a1dc6aa

…oved error handling in receive requests

feat: update deployment addresses for GoodDollarMinterBurner and Good…

ffbc78c

…DollarOFTAdapter, and adjust bridging amount and transaction limits in configuration


		Note: Limit values can be specified in decimal format (e.g., "5000" for 5,000 G$). The scripts automatically convert them to wei (18 decimals).

		## Manual Configuration: Step-by-Step

Conversation

blueogin commented Jan 30, 2026

Description

How Has This Been Tested?

Checklist:

Uh oh!

sourcery-ai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

blueogin Feb 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

blueogin Feb 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

openzeppelin-code bot commented Feb 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Feature: oft adapter step3 - deploy script and test

🚨 Report Summary

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

blueogin Feb 2, 2026 •

edited

Loading

blueogin Feb 2, 2026 •

edited

Loading

openzeppelin-code bot commented Feb 2, 2026 •

edited

Loading