Repository navigation
fix: update Codewhale installer pin and synchronize verified bootstraps - #9
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The focused changes have consistent integrity pins and argument validation, with supporting regression coverage and no unresolved findings.
Review effort: Balanced
Findings: None
What changed in this PR
Updates the Codewhale installer’s provider pin and keeps both standalone launchers synchronized with the verified helper.
Changes:
- Pins the provider to
adad64b7ac77ec997b5be4662c6b6ccf36998357. - Rejects forwarded
--profileoverrides while preserving native-pand-cflags. - Updates launcher revision/checksum pins, regression tests, and installation guidance.
| File | Description |
|---|---|
| test-codewhale.mjs | Tests profile rejection, native flags, and the updated helper revision. |
| codewhale.sh | Updates the immutable helper revision and required checksum. |
| codewhale.ps1 | Applies matching helper revision and checksum pins. |
| codewhale.mjs | Updates the provider pin and blocks profile overrides. |
| CODEWHALE.md | Documents the new pin, argument restrictions, and integrity checks. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Changes
Validation
Passed Bash syntax, Node Codewhale/OpenCode/entrypoint suites, Python menu/core/AI-tool contract suites, ShellCheck and the PowerShell Codewhale tests in the upgrade validation run. PR CI additionally exercises the cross-platform installers.
The temporary upgrade writer has been removed. No existing user config, production OAuth login, main-branch merge, publication or deployment was performed.