tp-pr-fix

tp-pr-fix — Single-Round Worker

A round-shaped worker over a single <design> PR. It receives reviewer comments, classifies them (structural / minor / unclear), filters to collaborator-authored structural issues, generates ONE fix commit, pushes, and applies the tp:do-not-merge-yet label. The result is a fix-envelope.v1.json dict the caller can persist or feed into the loop driver.

Two callers:

1. tp-pr-iterate loop driver — polls the PR, hands a fresh classified[] to tp-pr-fix each iteration until termination.
2. Standalone, human-invoked — /tp-pr-fix my-design --pr-url <url> --iteration N. You ran a review, want to apply one targeted fix round, and stop. The worker makes no assumption about being inside a loop — it produces exactly one commit per invocation (or none, with verdict="no-applicable-fixes") and exits.

Arguments

• {design} — kebab-case design name, validated per skills/_shared/validate-name.md. Resolves the active branch (tp/{design}) and the design directory under three-pillars-docs/tp-designs/.
• --pr-url <url> — full https://github.com/<owner>/<repo>/pull/<n> URL. Required. The shipped run-state state.v1.json schema does not carry a pr_url field; the loop driver passes the URL into fix_round.run_round from its own arguments, and standalone callers must supply it on the CLI.
• --iteration N — the iteration index recorded in the envelope and the commit prefix. Defaults to 1 when invoked standalone; the loop driver passes its current counter.

Prerequisites

• gh CLI installed and authenticated. The worker uses it for:
  • gh pr view <url> --json labels — label idempotency check.
  • gh pr view <url> --json headRefName — resolve the PR head ref the commit must land on (F1; see ## Run sequence step 6).
  • gh api repos/{owner}/{repo}/issues/{n}/labels and gh label create <label> — label application via REST (delegated to label_manager.ensure_pr_label; the REST endpoint is used instead of gh pr edit --add-label, which fails on a classic-Projects repo with a GraphQL projectCards deprecation error).
  • gh api repos/{owner}/{repo}/collaborators/{user} — identity gate (with a trusted-reviewer-bot short-circuit; see step 6).
  • gh pr diff <url> — diff context for the classifier.
• A clean enough working tree on the PR head ref to commit and push. The head is resolved from the PR, not assumed to be tp/{design} — an orchestrator PR's head is the worker's candidate branch (candidate/{slug}/single), not the design branch (F1).
• The repo's .three-pillars/config.json is populated. The only config surface the worker honors today is pdw.comment_url_allowlist — extra netlocs (beyond github.com and the repo origin host) that reviewer comments may reference. The tp:do-not-merge-yet label name and the claude-sonnet-4-6 model are hardcoded; there is no pr_fix config block.