Skip to content

ensure doc-pr review posts as PR comment#456

Merged
jth-nw merged 2 commits intomainfrom
dev
Mar 10, 2026
Merged

ensure doc-pr review posts as PR comment#456
jth-nw merged 2 commits intomainfrom
dev

Conversation

@jth-nw
Copy link
Collaborator

@jth-nw jth-nw commented Mar 10, 2026

No description provided.

jth-nw and others added 2 commits March 10, 2026 12:10
Root cause: Claude completed all 3 review stages but never called
gh pr comment — it returned the results as text output instead.
Also, skill argument parsing was broken ($1 got the PR number
instead of the file list).

Changes:
- Pass REPO, PR_NUMBER, and FILES as env vars (DOC_PR_*) so the
  skill reads them reliably instead of depending on positional args
- Add explicit "you MUST post" instructions in both the workflow
  prompt and the skill output section
- Restructure skill output as two clear steps: write file, then post
- Add fetch-depth: 1 to checkout for faster clones

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
fix: ensure doc-pr review posts as PR comment
@jth-nw jth-nw merged commit b71722b into main Mar 10, 2026
5 of 7 checks passed
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