Skip to content

chore(mainnet): add incident multisig signer task#683

Open
jackchuma wants to merge 9 commits into
mainfrom
mainnet-incident-multisig-signers-2026-05-13
Open

chore(mainnet): add incident multisig signer task#683
jackchuma wants to merge 9 commits into
mainfrom
mainnet-incident-multisig-signers-2026-05-13

Conversation

@jackchuma
Copy link
Copy Markdown
Contributor

@jackchuma jackchuma commented May 13, 2026

Summary

  • add mainnet Incident Multisig safe-management task
  • set OWNER_SAFE to the mainnet incident multisig and SENDER to 0x1841CB3C2ce6870D0417844C817849da64E6e937
  • update OwnerDiff.json with the intended signer additions and removals
  • include generated base signer validation JSON with explicit Safe owner linked-list descriptions

Testing

  • jq empty mainnet/2026-05-13-incident-multisig-signers/OwnerDiff.json
  • jq empty mainnet/2026-05-13-incident-multisig-signers/validations/base-signer.json
  • forge fmt --check mainnet/2026-05-13-incident-multisig-signers/script/UpdateSigners.s.sol
  • make deps
  • forge build

Notes

  • Task status is READY TO SIGN.
  • The generated validation file uses the configured sender and includes the domain/message hashes for Incident Safe - Mainnet.

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented May 13, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/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

Set up a mainnet safe-management task for adding and removing multiple Incident Multisig signers with placeholder owner diffs.

Co-authored-by: Codex <noreply@openai.com>
@jackchuma jackchuma force-pushed the mainnet-incident-multisig-signers-2026-05-13 branch from 125b812 to eece937 Compare May 13, 2026 20:31
jackchuma and others added 8 commits May 13, 2026 16:40
Use the latest base/contracts commit and trim explicit Foundry remappings to the remaining base-contracts alias. Update imports for the latest base/contracts scripts directory layout.

Co-authored-by: Codex <noreply@openai.com>
Replace placeholder mainnet Incident Multisig owner add and remove entries with the requested signer addresses.

Co-authored-by: Codex <noreply@openai.com>
Set the mainnet Incident Multisig task sender to a current owner that is not removed by the task.

Co-authored-by: Codex <noreply@openai.com>
Set the task docs to ready-to-sign and include facilitator validation and execution guidance.

Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
@jackchuma jackchuma marked this pull request as ready for review May 13, 2026 23:04
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.

2 participants