Conversation
…thub#4620) Assisted-by: Codex (model: GPT-6, autonomous)
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The focused change is consistent with OpenCode’s isolated paths and is covered by regression and registry contract tests.
Pull request overview
Declares OpenCode safe for concurrent installation with other isolated integrations.
Changes:
- Enables
multi_install_safefor OpenCode. - Adds a regression assertion.
- Documents OpenCode’s command directory.
File summaries
| File | Description |
|---|---|
src/specify_cli/integrations/opencode/__init__.py |
Enables safe multi-installation. |
tests/integrations/test_integration_opencode.py |
Guards the declaration. |
docs/reference/integrations.md |
Adds OpenCode to the safety table. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Closes #4620.
Declare
OpencodeIntegration.multi_install_safe = Trueso OpenCode can be installed alongside Codex and other declared-safe integrations without--force, and the missing declaration no longer causesunsafe-multi-installstatus findings. Add a focused regression assertion and list.opencode/commandsin the multi-install-safe reference table.The existing registry contracts now include OpenCode in static-path checks, all 351 integration pairs for root and command-directory isolation, and all 27 rotated installation orders for manifest isolation.
Documentation note for maintainers:
dshandvibealso declare multi-install safety but are absent from this table. Those pre-existing omissions are left for a separate documentation change.Testing
.venv/bin/python -m pytest tests/integrations/test_registry.py tests/integrations/test_integration_opencode.py -q— 852 passed on the latest upstream main baseline.Falsein memory..venv/bin/specify --help— passed.git diff upstream/main --check— passed.The registry contracts exercise sample-project initialization and installation without
--force. No slash-command content or execution behavior changed. The full repository test suite was not run.AI Disclosure
Implemented and tested on behalf of @philo-x by Codex (GPT-6), operating autonomously from the user's supplied diagnosis and three-file repair plan. Codex authored the changes, ran validation, and submitted this PR. The commit includes an
Assisted-by:trailer.