ce-resolve-pr-feedback

Resolve PR Review Feedback

Evaluate and fix PR review feedback, then reply and resolve threads. The orchestrator judges every item centrally (the legitimacy gate), then dispatches generic subagents seeded with a skill-local fixer prompt only for items it has approved for a fix.

Escalations never block. needs-human is the escalation channel: leave the thread open with a natural reply and report the structured decision_context. Never pause mid-run to ask. That is what lets an autonomous caller — ce-babysit-pr running unattended, for example — loop this skill. Items that need a human decision come back as needs-human results for the caller to surface, rather than stalling the run; that includes a fix that would change behavior the author chose deliberately (see the rubric).

mode:pipeline (set by an orchestrator like ce-babysit-pr or lfg): the run is unattended, so never call the blocking-question tool for any reason, and read references/pipeline-mode.md before acting. It owns the two things ordinary mode leaves open. First, the open thread is the escalation ledger, so never write a PR-body residual section of your own. Second, the caller may pass a trajectory (unresolved_trend, new_threads_this_tick, invariant_rounds); when it shows that the feedback is not converging, or invariant_rounds[].rounds >= 2 for a key this pass would continue (the next fix would be that key's third round) and that key's escalation is unanswered, answer with one approach-level needs-human rather than fixing nit after nit — an answered escalation authorizes the next action instead. On a fix outcome, return a stable invariant_key per fixed root; do not run pr-snapshot.

Authority in pipeline mode. Being invoked by an orchestrator is not itself authorization. You act under the inherited scope it holds from the user: actions = fix / commit / push / reply / resolve on the PR head, plus ticking a ## Unapplied review findings bullet a committed fix closed (below); exclusions = merge, rebase, force-push, approve CI. You may narrow this (decline a fix, defer a needs-human) but never broaden it — if resolving a thread would require an excluded action, defer it as needs-human rather than perform it.

Default to fixing. Don't churn on what isn't real. Most review feedback -- nitpicks included -- is correct and worth fixing; work the list and fix. Validation is a tripwire, not a gate: you read the code to make the fix anyway, so divert only on a concrete signal. Judge every item on its merits regardless of source (human or bot) or form. references/evaluation-rubric.md carries the four diverts and the evidence each one owes; read it before judging any item.

The PR body's ## Unapplied review findings checklist. A shipping workflow may have left this section: review findings it declined to apply unattended, one - [ ] bullet each, for the reviewer to decide. When a fix you commit closes one of those bullets (same file and concern), tick it to - [x] in the body so the inventory at the top of the PR stays true. Tick only; never add to, reorder, or create that section — it is the author's record, not the escalation ledger.