Skip to content

[DBA-270] Add check-pr-comments skill for Claude#61

Merged
gasparian merged 4 commits intomainfrom
simon.karan/DBA-270-github-pr-comments-skill
Mar 20, 2026
Merged

[DBA-270] Add check-pr-comments skill for Claude#61
gasparian merged 4 commits intomainfrom
simon.karan/DBA-270-github-pr-comments-skill

Conversation

@SimonKaran13
Copy link
Member

Summary

  • add a check-pr-comments Claude skill for handling GitHub PR review threads with gh
  • document the required workflow, including explicit approval before edits or GitHub replies
  • add eval coverage for PR discovery, unresolved thread triage, and explanation-only handling

Testing

  • not run (documentation and eval definitions only)

@SimonKaran13 SimonKaran13 requested review from a team and Copilot March 19, 2026 13:51
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new Claude skill, check-pr-comments, to standardize how an agent uses the GitHub CLI (gh) to discover and work through unresolved PR review threads, including an explicit “triage table then wait for approval” gate before edits or GitHub replies.

Changes:

  • Introduces check-pr-comments skill documentation with a step-by-step workflow for PR discovery, thread triage, implementation/validation, and in-thread replies/resolution.
  • Adds eval definitions covering PR discovery (by number or current branch), unresolved-thread triage, and “reply-only” outcomes.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.claude/skills/check-pr-comments/SKILL.md Documents the gh-based workflow for fetching review threads, triaging, waiting for explicit approval, then implementing/replying/resolving.
.claude/skills/check-pr-comments/evals/evals.json Adds eval cases/assertions to enforce PR discovery, unresolved-thread focus, triage-table output, and approval gating.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

SimonKaran13 and others added 3 commits March 20, 2026 10:01
- Use explicit <OWNER>/<REPO> placeholders with a gh repo view step
- Add pageInfo pagination support to the GraphQL query

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 20, 2026 09:02
@SimonKaran13 SimonKaran13 force-pushed the simon.karan/DBA-270-github-pr-comments-skill branch from 284b1bd to cf13e08 Compare March 20, 2026 09:02
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new Claude skill (check-pr-comments) to standardize how PR review threads are discovered, triaged, and (only after explicit approval) acted on using the GitHub CLI (gh).

Changes:

  • Added check-pr-comments skill documentation describing an approval-gated workflow for triaging and addressing unresolved PR review threads via gh + GraphQL.
  • Added eval definitions covering PR discovery, unresolved thread triage, and “reply-only” handling.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.claude/skills/check-pr-comments/SKILL.md New skill doc outlining the end-to-end gh workflow (auth, PR discovery, GraphQL thread fetch, triage table, approval gate, reply/resolve).
.claude/skills/check-pr-comments/evals/evals.json New eval cases validating the intended workflow and guardrails (especially “pause for explicit approval”).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gasparian gasparian self-requested a review March 20, 2026 10:37
Copy link
Contributor

@gasparian gasparian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • tested on #67
  • tweaked a bit graphql command to fix shell escaping

@gasparian gasparian merged commit 23a54e6 into main Mar 20, 2026
8 checks passed
@gasparian gasparian deleted the simon.karan/DBA-270-github-pr-comments-skill branch March 20, 2026 12:12
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.

4 participants