safe-global / safe-client-gateway / 18192665576

89%

main: 89%

Build Type

Pull #2730

github

Committed by katspaugh

Commit Message

feat: Allow migration from unsupported to supported Safe mastercopies

Enables Safes with unsupported base contracts to migrate to official mastercopies via both relay transactions and normal transaction proposals.

## Changes

### Relay Transactions
- Modified `LimitAddressesMapper.isOfficialMastercopy()` to detect migration transactions
- Added `LimitAddressesMapper.isMigrationToOfficialMastercopy()` to validate changeMasterCopy calls to official singletons
- Updated `SafeDecoder` to support deprecated `changeMasterCopy` function (v1.2.0 ABI)
- Added `changeMasterCopyEncoder` test helper

### Transaction Proposals (safeTxHash calculation)
- Added `getVersionFromMastercopy()` to detect Safe version from mastercopy address
- Added `detectVersionFromMigrationTransaction()` to extract target version from migration transactions
- Modified `getSafeTxHash()` to handle null versions by detecting migration transactions
- When Safe.version is null, attempts to infer version from changeMasterCopy target

## How It Works

For Safes with unsupported mastercopies (version = null):
1. System detects if transaction is self-call to changeMasterCopy
2. Validates target mastercopy is an official L1 or L2 singleton
3. Determines the target version from mastercopy address
4. Uses detected version for safeTxHash calculation or relay validation

## Test Coverage

- Added 5 tests for null version migration scenarios in safe.spec.ts
- Added 3 tests for relay migration scenarios in limit-addresses.mapper.spec.ts
- All 987 tests passing

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Pull Request Pull Request #2730: feat: Allow migration from unsupported to supported Safe mastercopies

Run Details

3814 of 4694 branches covered (81.25%)

Branch coverage included in aggregate %.

53 of 57 new or added lines in 4 files covered. (92.98%)

2 existing lines in 2 files now uncovered.

12737 of 13859 relevant lines covered (91.9%)

528.86 hits per line