Merged
Conversation
mpetrunic
previously approved these changes
Mar 27, 2026
There was a problem hiding this comment.
Pull request overview
Implements support for bridging between Ethereum and Gnosis Chain via the Gnosis Omnibridge, integrating it as a new provider into the on-chain Repayer/Rebalancer flow and adding Gnosis Chain configuration + fork tests (part of #171).
Changes:
- Add
GnosisOmnibridgeAdapterand wireProvider.GNOSIS_OMNIBRIDGEintoRepayerandRebalancer. - Extend domain/provider enums and mappings (Gnosis domain + Stargate endpoint id + chain id).
- Add unit tests, fork tests (Ethereum/Gnosis), config/script updates, and CI wiring for the new fork tests.
Reviewed changes
Copilot reviewed 31 out of 31 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| test/SprinterLiquidityMining.ts | Adds an allowance expectation for baseline state. |
| test/Repayer.ts | Adds GNOSIS_CHAIN domain/provider assertions and extensive Omnibridge tests. |
| test/Rebalancer.ts | Adds unsupported-provider coverage and Omnibridge rebalance tests. |
| test/LiquidityPoolAaveLongTerm.ts | Adjusts expectAlmostEqual tolerance. |
| test/LiquidityPoolAave.ts | Adds initial balance assertion; adjusts expectAlmostEqual tolerance. |
| test/LiquidityPool.ts | Adds a repay “not implemented” revert assertion. |
| specific-fork-test/gnosis/RepayerGnosisOmnibridge.ts | Adds Gnosis Chain fork test for Omnibridge path (incl. USDCe→USDCxDAI swap). |
| specific-fork-test/ethereum/RepayerGnosisOmnibridge.ts | Adds Ethereum fork test for Omnibridge Ethereum→Gnosis path. |
| specific-fork-test/ethereum/Repayer.ts | Updates fork repayer deployment args for new Gnosis adapter addresses. |
| scripts/upgradeRepayer.ts | Extends upgrade args/config defaults with Omnibridge-related addresses. |
| scripts/upgradeRebalancer.ts | Extends upgrade args/config defaults with Omnibridge-related addresses. |
| scripts/deployStandaloneRepayer.ts | Extends standalone repayer deploy args/config defaults with Omnibridge-related addresses. |
| scripts/deployRepayer.ts | Extends repayer deploy args/config defaults with Omnibridge-related addresses. |
| scripts/deploy.ts | Extends deploy flow (rebalancer/repayer) args/config defaults with Omnibridge-related addresses. |
| scripts/common.ts | Adds GNOSIS_CHAIN domain and GNOSIS_OMNIBRIDGE provider to solidity mappings. |
| package.json | Adds test:gnosis fork-test script. |
| network.config.ts | Adds GNOSIS_CHAIN network config + Omnibridge/AMB addresses and route config updates. |
| hardhat.config.ts | Adds GNOSIS_CHAIN network config and explorer metadata; adjusts fork chain hardfork history handling. |
| coverage-baseline.json | Updates coverage baseline thresholds. |
| contracts/utils/StargateAdapter.sol | Adds Stargate endpoint id mapping for GNOSIS_CHAIN. |
| contracts/utils/GnosisOmnibridgeAdapter.sol | Introduces Omnibridge adapter (initiate + finalize on Ethereum via AMB). |
| contracts/utils/AdapterHelper.sol | Adds GNOSIS_CHAIN chain id mapping. |
| contracts/testing/TestRepayer.sol | Updates test repayer constructor to pass new adapter args. |
| contracts/testing/TestRebalancer.sol | Updates test rebalancer constructor to pass new adapter args. |
| contracts/testing/TestGnosisOmnibridge.sol | Adds Omnibridge/AMB/transmuter test doubles. |
| contracts/interfaces/IRoute.sol | Extends enums with GNOSIS_CHAIN and GNOSIS_OMNIBRIDGE. |
| contracts/interfaces/IGnosisOmnibridge.sol | Adds interfaces for Omnibridge mediator, Ethereum AMB, and USDCe transmuter. |
| contracts/Repayer.sol | Wires in GNOSIS_OMNIBRIDGE initiate/process paths and emits token from finalize. |
| contracts/Rebalancer.sol | Wires in GNOSIS_OMNIBRIDGE initiate/process paths. |
| .github/workflows/test.yml | Adds CI step to run Gnosis Chain fork tests. |
| .env.example | Adds GNOSIS_CHAIN RPC and a default Gnosis USDCe owner address for fork tests. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mpetrunic
approved these changes
Mar 30, 2026
viatrix
approved these changes
Apr 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of #171