Skip to content

Commit 960f02f

Browse files
Add README and plugin manifest for code-review-calibration
CI requires every skill directory to ship a README.md and a .plugin/plugin.json with vendor symlinks so Codex and Claude Code can load it. Co-authored-by: openhands <openhands@all-hands.dev>
1 parent b9c5644 commit 960f02f

4 files changed

Lines changed: 64 additions & 0 deletions

File tree

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
.plugin
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
.plugin
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
{
2+
"name": "code-review-calibration",
3+
"version": "1.0.0",
4+
"description": "Investigate a repository's commit history, bug fixes, reverts, house conventions, testing strategy, CI coverage, and past PR reviews, then write a calibrated code-review skill whose failure patterns, conventions, and hotspots are all cited from that repository.",
5+
"author": {
6+
"name": "OpenHands",
7+
"email": "contact@all-hands.dev"
8+
},
9+
"homepage": "https://github.com/OpenHands/extensions",
10+
"repository": "https://github.com/OpenHands/extensions",
11+
"license": "MIT",
12+
"keywords": [
13+
"code-review",
14+
"calibration",
15+
"git-history",
16+
"failure-patterns",
17+
"conventions",
18+
"hotspots"
19+
]
20+
}
Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
# Code Review Calibration
2+
3+
Generate a code review skill that is calibrated to one specific repository, by investigating that repository's own history and conventions.
4+
5+
## What It Does
6+
7+
Most review guidance is generic, so the author has already heard it. This skill investigates a repo in two halves and writes what it learns to `.agents/skills/code-review.md`:
8+
9+
- **How the codebase breaks** - fix-density hotspots, reverts, fix-inducing commits (SZZ), and temporal coupling between files.
10+
- **How the codebase works** - house conventions and idioms, testing strategy and regression-test discipline, what CI already enforces, and ownership/blast-radius signals.
11+
12+
The result is a reviewer that can say "use `scopedQuery()`, the same omission caused `9f2a1bc`" instead of "consider filtering by tenant."
13+
14+
## When to Use
15+
16+
- "Write a code review skill for this repo"
17+
- "Recalibrate the reviewer to our codebase"
18+
- "Where do bugs cluster here?"
19+
- "What are our testing conventions?"
20+
- The current review skill is producing generic feedback
21+
- Onboarding to an unfamiliar codebase and needing to know its idioms and dangerous areas
22+
23+
## Requirements
24+
25+
- A git repository with real history. Very young or fully squash-merged repos yield weak patterns, and the skill is instructed to say so rather than pad the output.
26+
- Optional: GitHub CLI (`gh`), authenticated. Enables the PR review-comment phase, which shows what the team already knows is easy to get wrong.
27+
28+
## Output
29+
30+
Writes or updates `.agents/skills/code-review.md`. Generated sections are wrapped in `<!-- BEGIN GENERATED -->` / `<!-- END GENERATED -->` markers so later runs can refresh the evidence without discarding hand-written content.
31+
32+
Ground rules the generated file must satisfy:
33+
34+
- Every failure pattern cites two or more commit SHAs. Uncitable observations are nits and get dropped.
35+
- Six to twelve patterns. Beyond roughly fifteen the review loses focus.
36+
- Patterns that a design change made impossible move to a Retired section rather than being deleted.
37+
38+
## Related Skills
39+
40+
- `code-review` - perform a code review
41+
- `learn-from-code-review` - distill PR review feedback into guidelines
42+
- `agent-memory` - persist repository knowledge in AGENTS.md

0 commit comments

Comments
 (0)