feat: make PR review automation agent-agnostic - #804
Conversation
…andoff feat: persist and route PR review feedback
|
Assessed in an open-PR sweep: the reviewer-spawn architecture here (bespoke per-agent reviewerCodex*/reviewerClaude* keys) is the pattern the codebase has since moved away from — the agentManager registry + agentSpawnHelper design (PR #1022 lineage) covers agent-agnostic spawning generically, and merging this would revert main's applyWorktreeSessionsAddedPayload refactor in client/app.js (structural conflicts there). HOWEVER: the review-routing/paste-back feature (reviewSourceSessionId linking, latestReviewSummary cache, open/paste-review buttons) is genuinely novel — nothing on main or #1022 duplicates it. Recommendation: CLOSE this branch, and reimplement the paste-back feature fresh on top of the agentSpawnHelper architecture. |
|
Closing per the sweep assessment — the reviewer-spawn architecture is superseded by the agentSpawnHelper/agentManager registry design. NOTE for future work: the review-routing/paste-back feature in this PR (reviewSourceSessionId linking, latestReviewSummary cache, open/paste-review-to-source-terminal buttons) is genuinely novel and unduplicated — worth re-implementing fresh on the current architecture. |
Summary
Files
Notes
-claudeand-codexsessions and labels messages by actual agent.reviewerAgent,reviewerMode,reviewerProviderreviewerSkipPermissions(Claude)reviewerCodexModel,reviewerCodexReasoning,reviewerCodexVerbosity,reviewerCodexFlags(Codex)