Conversation
…243) rewrite_for_opencode only rewrote lib-path and config-path references, so a wrapper command's ~/.claude/commands/do/<name>.md execution reference (e.g. do-prd.md -> do/goals.md) stayed dangling under a curl-only OpenCode install, pointing at a path OpenCode never installs anything into. Extend the sed rewrite to resolve those references to the flat installed filename (~/.config/opencode/commands/do-<name>.md), mirroring resolveCommandExecRef in src/transformer.js.
Matches the +-quantifier CMD_EXEC_RE uses in src/transformer.js and the JS test mirror; the prior * allowed an empty basename to match.
fix: rewrite command execution references for OpenCode curl installs (#243)
fix: remove blanket permission grants from code reviewers
feat: support opencode zen reviewer in --review-with
Add Pi review and enhancement runners
The release diff added pi as a local-agent reviewer (PR #249) but never updated lib/better-options.md's supported-agent list or lib/better-review-loop.md's dispatch gate, so --review-with pi on /do:better, /do:better-swift, or /do:pr-better was documented as accepted in better-swift.md's own prose but silently rejected/undispatched by the shared lib files. README.md never mentioned pi at all. Also fixes a stale effort-carrier table missing pi's --thinking row and a dangling reference to a "Flag rationale" section removed by the permissions fix.
The Editing-mode section's example of --reviewer-applies working
('asking agy to both find and patch its own concerns') is no longer
achievable: pre-flight step 9 forces REVIEWER_APPLIES=false for any
reviewer without a verified write-isolated profile, and agy has none
today, so it always falls back to review-only. Point the example at
codex (the one reviewer with a verified profile) and cross-reference
step 9's fallback rule so the two sections read consistently.
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
1 participant
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.
Release v3.38.0
Released: 2026-09-06
Highlights
--review-withnow acceptspias a local reviewer and draft enhancer, with model brackets,~opt/~max/~effortsuffixes, and--thinkingeffort pins, so a review loop can add Pi's perspective alongside the existing agents.--review-withalso gainsopencode(aliaseszen,opencode-zen), running headlessly in pure isolation mode with a sensible default model and friendly model-name normalization, plus~effort=<level>mapped to OpenCode's native--variantflag.do-prd.md→do/goals.md) no longer dangle on a curl-only OpenCode install — they now resolve to the flat installed filename the same waysrc/transformer.jsdoes.Added
pireviewer/enhancer support across the review and enhancement loops (Add Pi review and enhancement runners #249).opencode/zen/opencode-zenreviewer support in--review-with, with model normalization and--varianteffort mapping (feat: support opencode zen reviewer in --review-with #248).Fixed
install.sh's OpenCode rewrite now resolves wrapper commands' execution references instead of leaving them dangling (fix: rewrite command execution references for OpenCode curl installs (#243) #246, closes curl installer's OpenCode path leaves dangling ~/.claude/commands/do/*.md execution references #243).src/transformer.jsand its test mirror.--review-with pinow works on/do:better,/do:better-swift, and/do:pr-better— the shared reviewer-grammar and dispatch libraries were missing thepislug even though it was documented as accepted.Full Changelog
Full Diff: v3.37.1...v3.38.0