Skip to content

docs(pr-workflow): document Claude auto-fix PR capabilities - #149

Merged
d-morrison merged 3 commits into
mainfrom
docs/claude-auto-fix
Aug 30, 2026
Merged

docs(pr-workflow): document Claude auto-fix PR capabilities#149
d-morrison merged 3 commits into
mainfrom
docs/claude-auto-fix

Conversation

@d-morrison

Copy link
Copy Markdown
Collaborator

Closes #118

Summarizes /autofix-pr (Claude Code on the web watches the PR and pushes fixes for CI failures/review comments; gh pr view detection; prompt scoping) and the Marketplace action pr-autofix-with-claude-code from https://code.claude.com/docs/en/commands#autofix-pr and https://github.com/marketplace/actions/pr-autofix-with-claude-code — in chapters/pr-workflow-with-agents.qmd.

Summarizes /autofix-pr (Claude Code on the web watches the PR and pushes
fixes for CI failures/review comments, plus the Marketplace action) from
https://code.claude.com/docs/en/commands#autofix-pr and
https://github.com/marketplace/actions/pr-autofix-with-claude-code.

Closes #118
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

The v2.1.221 claim is not in either cited source and is contradicted by
the changelog (bug fix at v2.1.161). Remove it per review.

Addresses review finding on PR #149.
@github-actions

Copy link
Copy Markdown

Claude finished reviewView run


Review of PR #149 — docs(pr-workflow): document Claude auto-fix PR capabilities

What changed since the last review round (2ae78de3..771d2bba): the branch merged origin/main (bringing in commit 7416830, "docs(plugins): add Contextify searchable history #148", already on main and thus outside this PR's actual diff to main), and added commit 4c257c5 "fix(pr-workflow): remove unverified version claim for /autofix-pr". The merge-base against main is 7416830, so the PR's real diff to main is now just the single hunk in chapters/pr-workflow-with-agents.qmd (verified directly against .claude-review-pr.diff) — the useful-plugins.qmd/WORDLIST changes from the git diff --stat in the task instructions are pre-existing content on main pulled in via the merge, not new content introduced by this PR.

Finding 1 — resolved

The prior blocking finding (chapters/pr-workflow-with-agents.qmd:55, the fabricated "available from v2.1.221 or later" claim) has been fixed. Commit 4c257c5 removed it; the current text at chapters/pr-workflow-with-agents.qmd:50 reads:

"It requires the gh CLI and access to Claude Code on the web."

I re-fetched https://code.claude.com/docs/en/commands#autofix-pr directly to confirm: the docs state verbatim "Requires the gh CLI and access to Claude Code on the web" and mention no minimum version anywhere. The new section's prose (PR detection via gh pr view, default fix-everything behavior, scoping example /autofix-pr only fix lint and type errors) is a faithful, accurate paraphrase of the official docs — confirmed line-for-line against the fetched page.

Finding 2 — still open, non-blocking (carried over)

chapters/ai-tools/pr-activity-notifications.qmd:74-84 still separately describes the same /autofix-pr shortcut with no cross-reference between the two sections (checked: neither file contains sec-autofix-pr or @sec-autofix-pr outside the new heading's own label definition). This is the same minor redundancy flagged last round and wasn't addressed, but it remains non-blocking — a future factual correction (like the one that just happened) would need updating in two places, but nothing is currently inconsistent between them.

Things re-verified this round

  • Both cited URLs return HTTP 200: code.claude.com/docs/en/commands#autofix-pr and github.com/marketplace/actions/pr-autofix-with-claude-code.
  • autofix and gh are both already present in inst/WORDLIST; no new spell-check risk from this diff.
  • The label {#sec-autofix-pr} is unique in the repo — no collision, no broken cross-refs.
  • No _site/, _freeze/, or generated README.md changes; scope remains tightly limited to the one intended file.
  • Heading style, backticked command references, and prose formatting are consistent with sibling sections.

Verdict

Ready for merge — the only blocking issue from the prior round (the fabricated v2.1.221 version claim) has been removed and the replacement text is verified accurate against the official docs. The remaining item (cross-linking with pr-activity-notifications.qmd) is a minor, non-blocking suggestion carried over from the previous review, not a defect introduced by this diff.

Reviewed commit: 4c257c5

@github-actions

Copy link
Copy Markdown

💰 Cost: $0.3633 (review) — run

@d-morrison
d-morrison merged commit 1d6e59d into main Aug 30, 2026
13 checks passed
@d-morrison
d-morrison deleted the docs/claude-auto-fix branch August 30, 2026 06:12
@github-actions

Copy link
Copy Markdown
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://Morrison-Lab.github.io/wai/pr-preview/pr-149/

Built to branch gh-pages at 2026-08-30 06:26 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

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.

document claude's auto-fix PR capabilities

1 participant