If you've been building Claude Code skills for a while, you've probably noticed a pattern: every skill author makes the same mistakes the first few times. Vague descriptions that fail to trigger. Workflows that don't say what to do when files are missing. allowed-tools
that ask for Bash
with no glob restriction. No eval scenarios, so you have no idea if the skill actually works.
I built skill-insp to catch those mistakes automatically. It's a skill that inspects other skills, scores them across 8 dimensions, and tells you what to fix.
Source:
[github.com/conanttu/skills]
You point skill-insp at a folder containing a SKILL.md
and it gives you:
β¨ skill-insp β¨
Overall: my-skill scores 66/100 β risk low, readiness usable-with-improvements.
Key strengths
- Minimal, appropriate permissions (Read and Write only)
- Clear 3-step workflow
- Clean YAML frontmatter with version tracking
Scorecard
| Dimension | Score |
|------------------------|-------|
| Structure | 8/10 |
| Triggering | 9/15 |
| Usability | 9/15 |
| Completeness | 7/15 |
| Progressive Disclosure | 7/10 |
| Testability | 4/10 |
| Maintainability | 8/10 |
| Safety & Trust | 14/15 |
| Total | 66/100 |
Recommendations
Medium Add error-handling for missing or unreadable files.
Medium Create evals/ with at least one input and expected output.
Low Expand README with usage examples or remove the placeholder.
HTML report: /abs/path/to/cache/my-skill/latest.html
β¨ skill-insp β¨
The 100 points are weighted toward what actually breaks skills in practice:
| Dimension | Max | Why it matters |
|---|---|---|
| Structure | 10 | Frontmatter parses, folder layout makes sense |
| Triggering | 15 | Description gets the skill invoked in the right contexts |
| Usability | 15 | Workflow steps are concrete and runnable |
| Completeness | 15 | Edge cases, inputs/outputs, failure handling |
| Progressive Disclosure | 10 | SKILL.md stays lean; details live in references |
| Testability | 10 | Evals or success criteria exist |
| Maintainability | 10 | No duplication, no stale placeholders |
| Safety & Trust | 15 | Permissions scoped, no hidden network, no destructive ops |
Safety & Trust is the dimension where pattern-matching tools fall apart, so skill-insp does semantic analysis here. It distinguishes between documentation and executable code: a SKILL.md that says "check for rm -rf
usage" in a safety checklist is not a destructive operation. A scripts/cleanup.sh
that actually runs rm -rf "$TEMP_DIR"
is, and gets flagged with the file:line reference.
The skill follows the "model is the analyzer" pattern. There's no Python or Node script that parses YAML and counts characters. Instead:
skill-insp/
βββ SKILL.md # Workflow + the 4 modes (default, detailed, apply, revert) + Run Evals
βββ README.md
βββ references/
β βββ rubric.md # Scoring dimensions and what good looks like
β βββ output-format.md # JSON schema for analysis.json
βββ scripts/
β βββ render-html.js # analysis.json β latest.html
β βββ run-evals.js # fixture setup + sub-agent prompt generation
βββ assets/
β βββ report_template.html # Self-contained HTML template
βββ evals/
βββ evals.json # 8 functional eval scenarios
Two deterministic scripts handle the parts that should be deterministic:
render-html.js
run-evals.js
cache/_fixtures/<id>/
, copies a snapshot of skill-insp itself into _skill_home/
so sub-agents can resolve <this-skill>
references, and prints a self-contained sub-agent prompt.Everything else β reading files, parsing YAML, evaluating the rubric, distinguishing documentation from code β is done by the model. This sounds slower than a parser, but it's actually the only way to do it correctly. A regex doesn't know that rm -rf
inside a markdown code fence labeled "examples to flag" is not the same as rm -rf
inside an executable script.
Earlier versions of skill-insp had a 300-line SKILL.md with the entire rubric inline. That hit context budget hard and made the skill harder to edit. The current layout pushes details to references:
analysis.json
.The result: SKILL.md is small enough to read in one sitting, and the model only loads the rubric when it's about to score.
This is the part that took the most iteration. The idea is simple: skill-insp ships with 8 eval scenarios in evals/evals.json
, each describing a user prompt, fixture files to create, and expectations to verify. To run them:
node scripts/run-evals.js <skill-path> list
node scripts/run-evals.js <skill-path> setup <id>
setup
creates a fixture directory, writes the fixture files, copies skill-insp's own resources into _skill_home/
, and prints a JSON payload that includes a sub_agent_prompt
. The parent agent reads that JSON, spawns a sub-agent with the prompt, and after the sub-agent finishes, checks each expectation:
find
over the fixture directory.In the first version, fixtures were created under os.tmpdir()
. This worked when I ran it manually, but sub-agents spawned by the harness were sandboxed to the project root β they got "permission denied" on every Read
and Bash
call against /var/folders/.../T/eval-skill-insp-*
. Three out of eight evals failed for sandbox reasons that had nothing to do with the skill's logic.
The fix was a one-line change: move fixtures into cache/_fixtures/<id>/
inside the project. Now sub-agents inherit the project's filesystem permissions, and the cache directory is .gitignore
d so it doesn't pollute commits. After the change, all 8 evals run cleanly.
Lesson worth remembering: if you're going to spawn sub-agents, keep their working directory inside the parent's sandbox. Temp directories outside the project tree look like a clean choice but break under tighter permission policies.
When you say "apply recommendations", skill-insp:
<cache_dir>/last-apply/
.manifest.json
.The manifest looks like this:
{
"applied_at": "2026-05-25T23:16:00Z",
"recommendations_applied": [
{ "priority": "high", "dimension": "triggering", "text": "Replace vague description..." },
{ "priority": "high", "dimension": "usability", "text": "Add a ## Workflow section..." },
{ "priority": "high", "dimension": "completeness", "text": "Add allowed-tools list..." }
],
"files": [
{
"relative_path": "SKILL.md",
"before_sha256": "e698920f94613f8fc335cd0e941938e0990bedd72cea66e52a6b956d4ff47845",
"after_sha256": "10c1fafffbfd2b0089a85e72aafc43432374ef627cb0b41602f8396083fa2800"
}
]
}
Revert is the inverse: read the manifest, verify the current file hash matches the recorded after_sha256
(so we don't blow away edits made after the apply), then restore from backup. If the hash doesn't match, skill-insp reports the conflict instead of overwriting. It never falls back to git reset --hard
or git checkout --
β those are blast-radius operations that don't belong in a recovery path.
Because skill-insp is itself a skill, it can score itself:
You: θ―δΌ° .claude/skills/skill-insp
Claude: β¨ skill-insp β¨
Overall: skill-insp scores 94/100 β risk low, readiness ready.
...
The first time I did this, the report flagged things I'd already half-noticed but not bothered to fix: the cache slug derivation rule was dense without an example, the description didn't mention "fix" or "improve" as triggers, and there was no Node.js version floor documented. All of these became Low/Medium recommendations, which I then applied β and the score went up.
This is the most useful feedback loop I've found for skill authoring: write the skill, run skill-insp against it, apply the high-priority recommendations, repeat. The eval suite then verifies the workflow still works end-to-end.
skill-insp's description explicitly says "Not for general code review." It's not a linter for arbitrary Python or TypeScript. It's specifically tuned to the structure and conventions of Claude Code skills:
SKILL.md
with YAML frontmatter.If you point it at a normal source tree, it'll refuse to score because there's no SKILL.md
β that's the intended behavior, not a bug.
Clone the repo and drop the folder into your Claude Code skills directory:
git clone https://github.com/conanttu/skills.git
ln -s "$(pwd)/skills/skill-insp" ~/.claude/skills/skill-insp
Then in any Claude Code session:
inspect the skill at ./my-skill
Or in Chinese:
evaluate ./my-skill
The trigger phrases are listed in the description so the skill is invoked automatically. After the inspection, follow the numbered prompts:
detailed mode
to expand evidenceapply recommendations
to auto-fix high-priority findingsrun evals
to verify the skill with eval scenariosrevert
to undo the last applyThe current version is 1.0.0. A few things I'd like to add:
apply
actually changed.evals.json
and analysis.json
.If you build skills regularly, give it a try and let me know what falls over. The eval scenarios in evals/evals.json
are a good place to start if you want to extend it β adding a new scenario is just adding a JSON entry with a prompt, fixture files, and expectations.