Skip to content

chore(sepolia): add incident multisig signer replacement task#771

Merged
mw2000 merged 3 commits into
mainfrom
chore/sepolia-incident-multisig-signers-2026-07-24
Jul 24, 2026
Merged

chore(sepolia): add incident multisig signer replacement task#771
mw2000 merged 3 commits into
mainfrom
chore/sepolia-incident-multisig-signers-2026-07-24

Conversation

@leopoldjoy

@leopoldjoy leopoldjoy commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a Sepolia task to replace 0x72069542Ca378553A3D479c82Adc31f21CA6dE4a with 0x2c1475476B586d66a85bC65A5aB396BBbAa4f3aD.
  • Apply the same owner diff to the Incident Multisig and mock Security Council Safe.
  • Preserve each Safe threshold and owner count, with signer and facilitator documentation for both roles.
  • Include generated validation files and successful execution artifacts for both Safes.

Validation

  • make validate-config
  • mise exec -- forge fmt --check
  • mise exec -- forge build --force
  • Full fork signing simulations for both Sepolia Safes, including prechecks and postchecks
  • Both validation files pass the signer tool TaskConfigSchema
  • Independent security-focused review with no findings
  • Both transaction receipts have status 1 and emit the expected owner addition and removal events
  • Post-execution RPC checks confirm 12 owners and unchanged thresholds of 3 and 1

Execution

Notes

  • Task-origin validation is explicitly disabled for both Sepolia validation files, matching the prior task.

Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
@cb-heimdall

cb-heimdall commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 2
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 2
2
1 if commit is unverified 0
Sum 2

Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
@cb-heimdall
cb-heimdall dismissed roger-bai-coinbase’s stale review July 24, 2026 22:31

Approved review 4776476365 from roger-bai-coinbase is now dismissed due to new commit. Re-request for approval.

@mw2000
mw2000 merged commit b832f75 into main Jul 24, 2026
11 checks passed
@mw2000
mw2000 deleted the chore/sepolia-incident-multisig-signers-2026-07-24 branch July 24, 2026 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants