Skip to content

Add Pi review and enhancement runners - #249

Merged
atomantic merged 1 commit into
mainfrom
feat/pi-review-runner
Sep 6, 2026
Merged

Add Pi review and enhancement runners#249
atomantic merged 1 commit into
mainfrom
feat/pi-review-runner

Conversation

@atomantic

Copy link
Copy Markdown
Owner

Summary

Adds Pi as a model-taking local reviewer and draft enhancer, including model brackets, optional and round-cap suffixes, and --thinking effort pins. The Pi runner receives the complete review material without tools or resource discovery; unsupported isolation flags make the runner unavailable.

Updates command documentation and on-demand loop loading without registering Pi as a slashdo installation host.

Supports atomantic/PortOS#6350.

Test plan

  • 143 focused review-contract, transformer, and command-structure tests passed.
  • Full suite: 522 tests passed.
  • Claude review at low effort completed; validation-message findings fixed and focused tests rerun.

@atomantic
atomantic force-pushed the feat/pi-review-runner branch from fdd6fe6 to e15fa1f Compare September 6, 2026 00:46
@atomantic
atomantic merged commit 51cee9c into main Sep 6, 2026
4 checks passed
@atomantic
atomantic deleted the feat/pi-review-runner branch September 6, 2026 00:48
atomantic added a commit that referenced this pull request Sep 6, 2026
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.
@atomantic atomantic mentioned this pull request Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant