Repository navigation
✨ [feat] host: unknown host halts, OpenCode detection, config host, hosts doc - #73
Merged
Merged
Conversation
…aude - super-board-host.sh prints claude only on CLAUDECODE; no sign exits 65 with a SUPER_BOARD_HOST hint on stderr - SUPER_BOARD_HOST accepts opencode; any other value still exits 64 - run-workflow.md: exit 65 or opencode halts before any claim with a host-selection line - tests/test-host.sh covers the new cases Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- super-board-host.sh: OPENCODE=1 prints opencode, checked before CLAUDECODE (leaks from a parent Claude session); AGENT=1 ignored - --config <file> reads the top-level host key when no override and no env sign; a bad value exits 64 - setup fix always writes a missing host from --host; check flags any value outside claude/codex/opencode - config-schema.json and onboard.md document host next to worker_backend - signal recorded on opencode v2.0.22 in the script header Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- docs/hosts.md: one row per host value (claude, codex, opencode) with tested version, run, sign-in and gaps - documents the detection order and the OpenCode v2.0.22 external_directory gap (attended-only) - README Install links the page - tests/test-hosts-doc.sh fails when a SUPER_BOARD_HOST value has no row Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
4 tasks
10 tasks
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.
Note
⏳ In review · round 1 · ✅ 10/10 AC · Closes #35 · Closes #39 · Closes #37 · head
621b38eProblem
scripts/super-board-host.shclaude, so OpenCode or a bare terminal silently started the Claude backendSUPER_BOARD_HOSTaccepted onlyclaudeorcodexsuper-board-host.sh, the board config,scripts/super-board-setup.pydocs/,README.mdSolution
claudeonly onCLAUDECODE; no sign prints nothing, explainsSUPER_BOARD_HOSTon stderr and exits 65.opencodeis a valid override.run-workflow.mdhalts before any claim on exit 65 oropencodeOPENCODE=1printsopencode(recorded on OpenCode v2.0.22), checked beforeCLAUDECODEbecause it leaks from a parent Claude session;AGENT=1is ignored.--config <file>reads a top-levelhostkey (same place ♻️ [refactor] install: shared .agents/ folder with per-host links #71's install.sh reads it). A bad config host exits 64fixalways writes a missinghostfrom--hostand never replaces a set one;checkflags values outside claude/codex/opencode (host_errors, red). Onboard writeshostfrom the host check, asking on exit 65. Documented inconfig-schema.jsondocs/hosts.mdwith a row per host (tested version, run, sign-in, gaps, incl. OpenCode's unenforcedexternal_directorydeny → attended-only), the detection order, and evidence. README Install links it.tests/test-hosts-doc.shfails when aSUPER_BOARD_HOSTvalue has no rowPrecedence:
SUPER_BOARD_HOST> Codex ids >OPENCODE>CLAUDECODE> confighost> exit 65, exactly as #39 orders it. A live env sign beats the saved config, so a Codex lane under a Claude-configured board still answerscodex.Acceptance criteria
CLAUDECODE=1→ claude; plusCODEX_THREAD_ID→ codex; empty env exits 65 with empty stdout↳
tests/test-host.shSUPER_BOARD_HOST=opencode→ opencode;geminiexits 64↳
tests/test-host.shopencodeas a halt with a host-selection request; no "otherwise Workflow tool" for an unknown host↳ "Unknown host halts" + step 4 Launch
↳
tests/test-host.sh--config {"host":"codex"}→ codex; nohostkey still exits 65↳
tests/test-host.shconfig-schema.jsondocumentshostnext toworker_backend; setup check flagsgeminiand accepts all three↳
tests/test-setup.shscenario 13host, never overwrites a set one↳
tests/test-setup.shscenarios 2 and 13docs/hosts.mdrow per host with version, invocation, sign-in, gaps; README Install links it↳
tests/test-hosts-doc.shbash tests/run-safety.shpasses on python 3.9↳ PASS 60 suites
bash tests/run-safety.shpasses on python 3.12↳ PASS 60 suites
Iteration history
bash tests/run-safety.sh→ PASS 60 suites (py3.9, py3.12) ·621b38eRisk
🟡 Medium · a Claude shell without
CLAUDECODEnow halts instead of running;SUPER_BOARD_HOSTor confighostis the escape hatch. Existing configs gethostadded on the next setupfix.🤖 Generated with Claude Code