CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-state

Use when reviewing a branch diff for implicit state-machine defects — a status, phase, flag or lifecycle timestamp written without the prior state in its WHERE, a state and its paired columns written apart, terminal or first-time values overwritten, a multi-statement transition that does not converge, an insert racing a unique index, a value set only the HTTP boundary enforces, a refusal in the wrong order — by listing every writer of each state across the repository, and reporting findings with G-C/H/M/L IDs in the four-field format.

Quality

89%

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

88%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.

An unusually disciplined procedural skill: every step is executable, the workflow has genuine validation checkpoints with recovery paths, and all detail is project-specific rather than padding. The only costs are some duplication of the report-shape rules and a long inlined check catalog that a references file could absorb.

DimensionReasoningScore

Conciseness

The body is dense and almost entirely project-specific (Drizzle idioms, D1 driver limits, `bun:sqlite` yielding, file:line citations) with no explanation of concepts Claude already knows, so it is well above the mostly-efficient anchor. It falls short of lean-and-efficient because the report-shape rules are stated twice (Step 0 and "Report shape") and some prose is ornamental ("The defect it exists for lives between two writers of one column"), which could be trimmed.

4 / 5

Actionability

Fully executable throughout: copy-paste bash pipelines (the `git diff | perl` state-grep, `git grep -n -w`, the `cat > STATE-REVIEW.md` heredoc skeleton, the `grep -c` shape checks), exact table and finding-format templates with placeholders, and concrete file:line examples for test arrangements (`beforeStatement`, `Effect.all` with `concurrency: "unbounded"`). This matches the anchor for copy-paste ready commands covering the common cases, not the anchor below it with minor gaps.

5 / 5

Workflow Clarity

The sequence is explicit (Step 0 skeleton before any reading, then find states, then list writers, then work nine checks), with a nine-item checklist, mid-run validation (the heading-count grep as soon as the first finding lands, with a recovery instruction when it prints the wrong number), incremental recording so a usage-limited run resumes from the file, and final `grep -c` gates plus a completion line. This matches the anchor for a clear sequence with explicit validation, error-recovery feedback loops and a checklist; the operation is read-only, so the destructive-operation cap does not apply.

5 / 5

Progressive Disclosure

There are no bundle files (no references/, scripts/ or assets/), and the one external pointer — `wiki/shared/state-transitions.md` for the Drizzle idiom examples — is clearly signaled, one level deep, and explicitly marked optional ("The checks below are complete without it"). Structure is strong throughout, but the ~260-line body inlines the full check catalog and report conventions that could partly live in a reference file, which keeps it at good-structure rather than the top anchor.

4 / 5

Total

18

/

20

Passed

Description

88%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.

A dense, highly specific description that names a comprehensive defect taxonomy and a concrete method, with an explicit trigger clause. Its main weaknesses are the absence of a few natural trigger synonyms (e.g. "race condition", "state transition") and a generic leading trigger ("reviewing a branch diff") that overlaps sibling review skills.

DimensionReasoningScore

Specificity

The description enumerates a comprehensive taxonomy of concrete defect classes ("a status, phase, flag or lifecycle timestamp written without the prior state in its WHERE", "an insert racing a unique index", "a refusal in the wrong order") plus the concrete method ("listing every writer of each state across the repository, and reporting findings with G-C/H/M/L IDs in the four-field format"). This matches the anchor for multiple specific concrete actions with comprehensive coverage, not the anchor below it, which requires minor gaps in coverage.

5 / 5

Completeness

It explicitly answers both questions: the opening "Use when reviewing a branch diff for implicit state-machine defects" gives concrete trigger phrases for when, and the closing "by listing every writer of each state across the repository, and reporting findings with G-C/H/M/L IDs in the four-field format" gives a concrete what. This matches the anchor that clearly and explicitly answers both what and when with concrete trigger phrases, not the anchor below it where the when could be more specific.

5 / 5

Trigger Term Quality

Natural terms are well covered — "reviewing a branch diff", "status", "phase", "flag", "lifecycle timestamp", "racing" — but common variations a user would actually say, such as "race condition", "concurrency" or "state transition", are absent. This fits the anchor for good keyword coverage with a few natural terms missing, falling short of the comprehensive synonym coverage of the top anchor.

4 / 5

Distinctiveness Conflict Risk

The niche is clear and the defect classes are highly distinctive, but the leading trigger "reviewing a branch diff" is shared with sibling review lenses (the body references a `review-security` skill and an automated scope classifier), creating minor overlap risk with closely related skills. This fits the anchor for mostly distinct with minor overlap risk rather than the top anchor's minimal conflict risk.

4 / 5

Total

18

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 1 missing

Warning

referenced_paths_exist

Referenced path issues: 1 missing

Warning

Total

14

/

16

Passed

Repository

englishstreetventures/osn

Path

.claude/skills/review-state/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.