safe-global / safe-client-gateway / 18500910781

90%

main: 90%

Build Type

Pull #2746

github

Committed by katspaugh

Commit Message

refactor: remove SafeToL2Migration support, keep only SafeMigration

Simplifies the codebase by removing SafeToL2Migration detection logic
and focusing only on SafeMigration contract support, which is what's
needed for the production issue on opBNB/Binance chains.

## Changes

- Remove SafeToL2Migration ABI import and function decoding
- Remove getSafeToL2MigrationDeployments wrapper function
- Remove getVersionFromMastercopy usage (was only for SafeToL2Migration)
- Rename detectDelegateMigration -> detectSafeMigration
- Simplify logic to only check SafeMigration contracts
- Remove SafeToL2Migration tests (3 tests)
- Update test suite name to reflect SafeMigration-only support

## Benefits

- ~40 lines of code removed
- Simpler, more maintainable implementation
- Focuses on actual production use case (SafeMigration on opBNB)
- No ABI decoding complexity needed

All tests passing: 65 tests (was 68, removed 3 SafeToL2Migration tests)

Pull Request Pull Request #2746: fix: add SafeMigration contract support for version detection

Run Details

3859 of 4761 branches covered (81.05%)

Branch coverage included in aggregate %.

13 of 14 new or added lines in 2 files covered. (92.86%)

12 existing lines in 1 file now uncovered.

13014 of 13991 relevant lines covered (93.02%)

539.4 hits per line