CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

diff-bug-review

Flag concrete bugs introduced by a merged diff. Used only by the post-merge sweep (warden-sweep.yml); never gates clearance.

Quality

90%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

SKILL.md
Quality
Evals
Security

Quality

Content

100%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

The body is an exemplary lean review checklist: concrete bug patterns, explicit inclusion and exclusion criteria, a verify-before-report checkpoint, and a fully specified finding format with severity definitions. It assumes Claude's competence throughout, wastes no tokens, and its single-file structure is the right shape for a skill of this scope.

DimensionReasoningScore

Conciseness

The body is lean with zero filler: no concepts Claude already knows are explained, and lines like "Zero findings is the correct and common result. Do not manufacture findings" each earn their place by steering behavior. Matches anchor 5 ('lean and efficient; assumes Claude's competence; every token earns its place'); there are no over-explanations to trim as anchor 4 requires.

5 / 5

Actionability

For an instruction-only skill the guidance is fully actionable: a concrete bug-pattern checklist with technical specifics (missing await, stale closures, "queries missing an organization scope", "Zod schemas vs. their users"), a real repo path (apps/app/src/app/lib/den.ts), explicit inclusion/exclusion criteria, and a precise reporting format with defined severity levels. Matches anchor 5's 'specific examples cover the common cases'; per the rubric's code_vs_instruction note, absence of code is not penalized when guidance is this concrete.

5 / 5

Workflow Clarity

This is a single-purpose review skill with an unambiguous action, and it includes an explicit verification checkpoint — "You read enough of the surrounding code... to rule out the obvious reason it is fine. Check and address contrary evidence before reporting" — plus the ALL-hold reporting gate. Per the simple-skill note, workflow clarity scores 5 when the single action is unambiguous; the task is read-only review, so the destructive/batch cap does not apply.

5 / 5

Progressive Disclosure

There are no bundle files and none are needed: the body is ~46 non-blank lines of self-contained guidance organized into clearly delimited sections (trust model, reporting criteria, patterns to seek, exclusions, output format). Nothing belongs in a separate file, there are no nested or buried references, and navigation is easy — matching anchor 5's 'content appropriately split' for a skill of this size.

5 / 5

Total

20

/

20

Passed

Description

75%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

The description states one clear, concrete action and an unusually explicit when-clause that names the owning workflow and its non-gating role, giving it minimal conflict risk with other review skills. Its weaknesses are limited natural-keyword coverage (no synonyms like "regression" or "broken") and a single stated action rather than a fuller capability list. Third-person voice is used correctly and there is no fluff.

Suggestions

Add one or two more concrete capabilities to the description (e.g., 'assign high/medium/low severity and name the smallest fix per finding') to move beyond the single 'flag bugs' action.

Include natural trigger variations users or workflows might use — e.g., 'regressions', 'broken by a merge', 'post-merge review' — to broaden keyword coverage beyond 'bugs' and 'merged diff'.

DimensionReasoningScore

Specificity

"Flag concrete bugs introduced by a merged diff" names the domain (merged diffs) and one concrete, non-generic action (flag bugs), but stops there rather than listing several specific capabilities (e.g., severity rating, root-cause attribution, fix suggestion). Matches anchor 3 ('names domain and 1-2 concrete actions, but not comprehensive'); not 2 because the action is specific rather than generic, not 4 because only one action is listed.

3 / 5

Completeness

Both parts are answered explicitly: the what ("Flag concrete bugs introduced by a merged diff") and a highly concrete when ("Used only by the post-merge sweep (warden-sweep.yml); never gates clearance") that names the exact triggering workflow. This satisfies anchor 5 ('clearly and explicitly answers both what AND when with concrete trigger phrases'); anchor 4's caveat ('when could be more explicit or specific') does not apply since the when names a specific workflow file and usage constraint.

5 / 5

Trigger Term Quality

Relevant keywords are present ("bugs", "merged diff", "post-merge", "sweep", "clearance"), but common variations and synonyms a user might naturally say — "regression", "broken", "defect", "review the merge" — are absent. Fits anchor 3 ('some relevant keywords but missing common variations or synonyms'); coverage is thinner than anchor 4's 'good keyword coverage'.

3 / 5

Distinctiveness Conflict Risk

The description carves out a clear niche (bugs introduced by a *merged* diff) and the "Used only by the post-merge sweep... never gates clearance" clause explicitly disambiguates it from sibling review skills (interactive code review, security review, clearance gating). Matches anchor 5 ('clear niche with distinct triggers; minimal conflict risk').

5 / 5

Total

16

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

different-ai/openwork

Path

.warden/skills/diff-bug-review/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.