Skip to content

chore: add solve-pr-comments agent skill#1444

Merged
hwbrzzl merged 4 commits into
masterfrom
bowen/add-solve-pr-comments-skill
Apr 15, 2026
Merged

chore: add solve-pr-comments agent skill#1444
hwbrzzl merged 4 commits into
masterfrom
bowen/add-solve-pr-comments-skill

Conversation

@hwbrzzl

@hwbrzzl hwbrzzl commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a new solve-pr-comments agent skill to .agents/skills/
  • Defines a decision tree for triaging PR comments into three actions: apply, decline, or answer
  • Establishes guardrails that prevent silently ignoring comments or applying changes believed to be wrong

Why

PR review comments are not always correct, and treating every comment as an order leads to worse code. Agents (and humans) need a clear framework for deciding when to change code, when to respectfully push back with a technical explanation, and when a comment is just a question that needs an answer — not a code change.

The skill introduces a decision tree that asks two questions in sequence: is the comment correct, and is it a question? This produces three distinct action paths, each with explicit rules for how to reply. The guardrails section forbids vague acknowledgements and silent resolution, ensuring every comment gets a substantive response.

@hwbrzzl
hwbrzzl requested a review from a team as a code owner April 15, 2026 07:53
Copilot AI review requested due to automatic review settings April 15, 2026 07:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 agent skill that provides a structured workflow for triaging and responding to PR review comments, ensuring comments are either applied, declined with justification, or answered directly.

Changes:

  • Introduces a new .agents/skills/solve-pr-comments/SKILL.md skill with a decision tree for classifying PR comments.
  • Defines concrete rules and guardrails to avoid silent resolutions, “pacifying” changes, and vague acknowledgements.
  • Documents a step-by-step workflow for fetching comments, batching fixes, replying, and (intended) thread resolution.

Comment thread .agents/skills/solve-pr-comments/SKILL.md Outdated
@codecov

codecov Bot commented Apr 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.95%. Comparing base (b24c0b1) to head (3359167).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1444   +/-   ##
=======================================
  Coverage   68.95%   68.95%           
=======================================
  Files         362      362           
  Lines       28072    28072           
=======================================
  Hits        19358    19358           
  Misses       7857     7857           
  Partials      857      857           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 agent skill document that standardizes how to triage and respond to PR review comments (apply / decline / answer), with an explicit workflow and guardrails to prevent silent or non-substantive resolutions.

Changes:

  • Introduces a decision tree for classifying PR comments into three action paths.
  • Adds a step-by-step workflow for fetching comments, batching fixes, replying, and resolving threads.
  • Establishes guardrails to ensure every comment receives a substantive response.

Comment thread .agents/skills/solve-pr-comments/SKILL.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 agent skill (solve-pr-comments) that defines a structured workflow for triaging PR review comments into: apply changes, decline with explanation, or answer questions—plus guardrails to ensure every comment gets a substantive response.

Changes:

  • Introduces a decision tree for classifying PR comments into three action paths (apply/decline/answer).
  • Documents an end-to-end workflow (fetch → triage → group edits → reply → resolve threads).
  • Adds guardrails to prevent silent resolution or “pacifying” changes.

Comment thread .agents/skills/solve-pr-comments/SKILL.md Outdated
@hwbrzzl
hwbrzzl merged commit 2beb8b3 into master Apr 15, 2026
19 checks passed
@hwbrzzl
hwbrzzl deleted the bowen/add-solve-pr-comments-skill branch April 15, 2026 09:37
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.

2 participants