Skip to content

Change a running auto session's reviewer settings with pair reviewer - #28

Merged
leonlee merged 2 commits into
mainfrom
feature/reviewer-set
Nov 6, 2026
Merged

leonlee merged 2 commits into
mainfrom
feature/reviewer-set

Conversation

@leonlee

@leonlee leonlee commented Nov 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • pair reviewer --reviewer-model MODEL|default --reviewer-effort LEVEL|default changes the reviewer of a running pair start session over its control socket, without a restart. It applies to new threads; a thread keeps the settings of its first round.
  • pair reviewer --thread current|ID changes only the pending thread whose last review round failed (e.g. a mistyped model). Its next round starts a new reviewer conversation, which reads the thread so far; the author's pair wait retries it. Refused while a round runs.
  • The author and the reviewer can't run it (PAIR_AUTO_ROLE); this is workflow discipline, not a security boundary.
  • pair status shows the settings for new threads and for the current thread. The restart command pair stop prints uses the current settings, and a failed round's error names pair reviewer --thread current instead of a restart.

Design discussed with Codex in pair threads 2026-11-06T02-32-35-906Z-042f55 and 02-34-36-579Z-dad4bf; implementation approved in 02-59-16-585Z-721431.

Test plan

  • npm test: 115 pass, including gated tests for new-thread defaults, thread snapshots, failed-thread correction with a fresh reviewer conversation, and the refusals
  • CI on Node 22 and 24

🤖 Generated with Claude Code

leonlee and others added 2 commits November 6, 2026 11:59
pair reviewer --reviewer-model/--reviewer-effort (a value or default) changes
the reviewer for new threads over the control socket, without a restart. A
thread keeps the settings of its first round. --thread current|ID changes only
the pending thread whose last round failed, e.g. over a mistyped model, and its
next round starts a new reviewer conversation. The author and reviewer can't
run it, and pair stop's restart command uses the current settings.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Y3kg5hjT2zSZM9qUEEinM
@leonlee
leonlee merged commit e2c4923 into main Nov 6, 2026
2 checks passed
@leonlee
leonlee deleted the feature/reviewer-set branch November 6, 2026 03:08
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