chore: bump gnus-ai submodule — Safe upgrade pipeline fixes + bridgeOut Sepolia deploy#3
chore: bump gnus-ai submodule — Safe upgrade pipeline fixes + bridgeOut Sepolia deploy#3Super-Genius wants to merge 13 commits into
Conversation
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- Update gnus-ai submodule to include ROADMAP.md phases 8-12 - Add Update-Smart-Contracts-Architecture.md (architecture analysis) - Update settings.local.json permissions for gsd-inbox/explore/discuss
gnus-ai: 761c373..886bdf5 (feature/bridge-out-initiated) feat(08.2): check:safe-executed + verify/confirm fixes docs(08.2): verification report (status pass) docs(08.2): ship bridge-out workstream nested diamonds/GeniusDiamond develop: 6800205..7c4bb20 (3 commits) fix(deploy): reconcile sepolia deployment to live v2.5 + exec tx hash
…ut Sepolia deploy (PR #63)
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 765532c646
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -1 +1 @@ | |||
| Subproject commit 524aff08d5a3229f4114edf6b0d52e6645c8ad62 | |||
| Subproject commit 17654d4c448b1512e9b9e4d4f93f2a1eb9b2d019 | |||
There was a problem hiding this comment.
The new gnus-ai submodule pin adds scripts/safe/safeUpgrade.ts, but step3Confirm invokes confirmDeployment.ts with only --artifact and --deployment, even though confirmDeployment.ts requires --rpc-url unless RPC_URL is set. In the supported case where an operator runs safe:upgrade with --rpc-url instead of an env var, proposing and waiting can succeed, then confirmation exits with “RPC URL required,” leaving the upgrade half-orchestrated and the deployed-data file unconfirmed.
Useful? React with 👍 / 👎.
| @@ -1 +1 @@ | |||
| Subproject commit 524aff08d5a3229f4114edf6b0d52e6645c8ad62 | |||
| Subproject commit 17654d4c448b1512e9b9e4d4f93f2a1eb9b2d019 | |||
There was a problem hiding this comment.
The pinned gnus-ai revision adds an orchestrator that accepts --network, but both wait and confirm build the deployed-data path as geniusdiamond-${opts.network}-11155111.json. For any non-Sepolia network, this either fails to find the correct file or mutates a stale/Sepolia-named deployment record after a real Safe execution, so the orchestration can corrupt or miss the deployment state. Use the proposal artifact/RPC/config chain id instead of hard-coding 11155111.
Useful? React with 👍 / 👎.
1 participant
Bumps gnus-ai submodule to include:
yarn safe:upgradeorchestrator command