safe-global / safe-client-gateway / 18313008342

89%

push

github

feat: Allow migration from unsupported to supported Safe mastercopies (#2730)

* feat: Allow transaction proposals for Safe mastercopy migrations

Enables transaction proposals (safeTxHash calculation) for Safes with unsupported mastercopies when they are migrating to official L2 singletons.

## Problem

Safes with unsupported base contracts have `version: null` in the Transaction Service API response. This causes `getSafeTxHash()` to fail with "Safe version is required", preventing any transaction proposals including migration transactions.

## Solution

### Version Detection from Migration Transactions

When `Safe.version` is `null`, the system now attempts to detect the target version from migration transactions:

1. **Direct changeMasterCopy (v1.0.0-1.2.0)**
   - Self-call to `changeMasterCopy(newMastercopy)`
   - Extracts target mastercopy and looks up its version

2. **Delegate call to SafeToL2Migration (v1.3.0+)**
   - Delegate call to official `SafeToL2Migration` contract
   - Decodes `migrateToL2(l2Singleton)` function
   - Validates migration contract is official
   - Extracts L2 singleton and looks up its version

The detected version is then used for safeTxHash calculation via EIP-712 typed data hashing.

## Changes

### Core Logic
- **src/domain/common/utils/safe.ts**
  - `getSafeTxHash()` - handles null version by detecting migration
  - `detectVersionFromMigrationTransaction()` - orchestrates detection
  - `detectDirectMigration()` - handles changeMasterCopy pattern
  - `detectDelegateMigration()` - handles migrateToL2 pattern

- **src/domain/common/utils/deployments.ts**
  - Added `getSafeToL2MigrationDeployments()` helper
  - Added `getVersionFromMastercopy()` - reverse lookup version from address

### Supporting Code
- **src/domain/contracts/decoders/safe-to-l2-migration-decoder.helper.ts**
  - New decoder for SafeToL2Migration contract
  - Decodes `migrateToL2(l2Singleton)` function

### Tests
- **src/domain/common/utils/__tests__/s... (continued)

3852 of 4745 branches covered (81.18%)

Branch coverage included in aggregate %.

38 of 50 new or added lines in 3 files covered. (76.0%)

12835 of 13970 relevant lines covered (91.88%)

521.04 hits per line