Repository navigation
feat(installer): OpenCode discovery plugin and cache-aware discovery - #53
Merged
Merged
Conversation
5 tasks
johnnyhuy
force-pushed
the
agent/issue-17-opencode-discovery
branch
from
November 9, 2026 19:49
6616a9f to
5777e85
Compare
Contributor
Author
|
Rebased onto current main (3a461a7) after #48, #50 and #52 landed behind this branch. Head is now 5777e85. The only conflict was the test:unit line in package.json - resolved by keeping both sides, so the suite now runs the cloud probe tests and the discovery cache test. Local verification on the rebased head, from the worktree sandbox:
Nothing user-facing changed in the rebase itself. The slice remains app-side only, refs #17, CLI parity stays a separate PR. |
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.
Adds the OpenCode discovery plugin and teaches discoverInstalled to read its cache before probing.
Refs #17 (app side only - CLI parity stays a separate PR, the release flows are independent, so the issue stays open)
What changed
Verification
returned: {"claude-code":{"path":"/tmp/hoist-proof/homebrew/bin/claude","version":"2.1.204 (Claude Code)"},"codex":{"path":"/tmp/hoist-proof/npm-global/bin/codex","version":"codex-cli 0.46.0"}}
Homebrew-path and npm-global installs both found, absent opencode correctly omitted.
Reviewer notes