Skip to content

fix: post Vale issues as inline PR review comments#468

Merged
jth-nw merged 1 commit intodevfrom
fix/doc-pr-inline-vale
Mar 11, 2026
Merged

fix: post Vale issues as inline PR review comments#468
jth-nw merged 1 commit intodevfrom
fix/doc-pr-inline-vale

Conversation

@jth-nw
Copy link
Collaborator

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

Vale issues now appear as inline comments on the exact lines in the diff, posted via the GitHub PR review API. Claude's task is reduced to just the editorial review + posting the summary comment.

Changes:

  • Vale step parses line output and builds JSON for inline comments
  • Posts a PR review with inline comments via gh api pulls/reviews
  • Dismisses previous bot reviews on re-runs to avoid piling up
  • Claude prompt simplified to editorial review only
  • Vale count passed to Claude for the summary line

Vale issues now appear as inline comments on the exact lines in the
diff, posted via the GitHub PR review API. Claude's task is reduced
to just the editorial review + posting the summary comment.

Changes:
- Vale step parses line output and builds JSON for inline comments
- Posts a PR review with inline comments via gh api pulls/reviews
- Dismisses previous bot reviews on re-runs to avoid piling up
- Claude prompt simplified to editorial review only
- Vale count passed to Claude for the summary line

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jth-nw jth-nw merged commit 12d4964 into dev Mar 11, 2026
2 of 5 checks passed
@jth-nw jth-nw deleted the fix/doc-pr-inline-vale branch March 16, 2026 22:26
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