Skip to content

docs: add PR review skill for Review Agent - #1996

Merged
aws-cdk-automation merged 1 commit into
mainfrom
9pace/pr-review-skill
Sep 24, 2026
Merged

aws-cdk-automation merged 1 commit into
mainfrom
9pace/pr-review-skill

Conversation

@9pace

@9pace 9pace commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Adds skills/pr-review, a general PR review skill that review agents load.

  • SKILL.md: scope, review process, harm-based severity (BLOCKING / RECOMMENDED / OPTIONAL), what not to flag, and the structured finding format.
  • references/review-rules.md: 17 rules covering compatibility, architecture, error handling and UX, testing, code quality, verification, and PR scope. Where possible, each rule cites the relevant section of AGENTS.md or another doc in this repo.

The skill favors precision: it only flags concrete defects that the PR itself introduces, and a review with zero findings is valid. It adds documentation only, with no code or build changes.

Checklist

  • This change contains a major version upgrade for a dependency and I confirm all breaking changes are addressed
    • Release notes for the new version:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@9pace 9pace added the pr/exempt-integ-test Skips the integ test steps if set. label Sep 24, 2026
@9pace
9pace deployed to automation September 24, 2026 13:25 — with GitHub Actions Active
@9pace
9pace deployed to automation September 24, 2026 13:25 — with GitHub Actions Active
@github-actions github-actions Bot added the p2 label Sep 24, 2026
@9pace
9pace deployed to no-approval September 24, 2026 13:26 — with GitHub Actions Active
@aws-cdk-automation
aws-cdk-automation requested a review from a team September 24, 2026 13:26
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace
9pace deployed to run-tests September 24, 2026 13:28 — with GitHub Actions Active
@9pace 9pace changed the title chore: add PR review skill for Review Agent docs: add PR review skill for Review Agent Sep 24, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.64%. Comparing base (c5d3958) to head (f14d1ab).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1996      +/-   ##
==========================================
- Coverage   91.67%   91.64%   -0.04%     
==========================================
  Files          82       82              
  Lines       13039    13039              
  Branches     1826     1824       -2     
==========================================
- Hits        11954    11950       -4     
- Misses       1047     1051       +4     
  Partials       38       38              
Flag Coverage Δ
suite.unit 91.64% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aws-cdk-automation
aws-cdk-automation added this pull request to the merge queue Sep 24, 2026
Merged via the queue into main with commit 00fb6f6 Sep 24, 2026
66 checks passed
@aws-cdk-automation
aws-cdk-automation deleted the 9pace/pr-review-skill branch September 24, 2026 13:52

This branch was successfully deployed

3 active deployments
run-tests — f14d1abb Deployed Sep 24, 2026 by 9pace via integ_init-templates (init-java, 24.19) #6993
no-approval — f14d1abb Deployed Sep 24, 2026 by 9pace via prepare #6993
automation — f14d1abb Deployed Sep 24, 2026 by 9pace via Triage Pull Requests #2136
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p2 pr/exempt-integ-test Skips the integ test steps if set.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants