code-quality · zernie — Claude Code skill for TS/JS quality
A skill for refactoring TypeScript and JavaScript: tightens types, removes impossible states, and separates logic from IO.
npx skills add zernie/vigiles --skill code-quality -a claude-codeWhat code-quality does
The skill fires when you ask to refactor, clean up, harden or review the design of code or a public API. It does not hunt bugs — that job belongs to /code-review. Five techniques sit inside, each with its own reference file: the principle, a before/after from the repo, and the smells to grep for.
The techniques cover parsing at the boundary instead of re-validating primitives, types that make illegal states unrepresentable, exhaustiveness guards on switch and if chains over unions, pure functions with injected seams for spawn, fetch and fs, and the shape of public exports. Read the reference that matches the code in front of you, not all five at once.
The workflow is strict. Name the decision out loud first, then change the type and let tsc point at every call site, fix them, run the tests. One technique per change, tests stay green. An abstraction for a difference that does not exist yet is the bug this skill exists to prevent, so rule-of-three and YAGNI cut against you here.
What it helps with
- stop re-validating the same primitive at many call sites
- make an impossible field combination fail to compile
- add an exhaustiveness guard to a
switchover a union - pull pure logic away from
fetchandfsso it can be tested - narrow a public export to a clear shape
Example prompts for the agent
- refactor the
ordersmodule and apply parse-don't-validate at the boundary - drop the boolean flags on
Joband make illegal states unrepresentable - go through the package's public exports and tighten their shape
How to install code-quality
1Skills CLI (Vercel)
npx skills add zernie/vigiles --skill code-quality -a claude-code2Manually: put the files in place
~/.claude/skills/code-quality/SKILL.mdCopy the whole skill folder; for one project use .claude/skills/code-quality/
1Skills CLI (Vercel)
npx skills add zernie/vigiles --skill code-quality -a codex2Manually: put the files in place
.agents/skills/code-quality/SKILL.mdFor all projects use ~/.agents/skills/
1Skills CLI (Vercel)
npx skills add zernie/vigiles --skill code-quality -a cursor2Manually: put the files in place
.cursor/skills/code-quality/SKILL.mdCursor also reads .agents/skills/ and .claude/skills/
1GitHub CLI
gh skill install zernie/vigiles code-quality2Manually: put the files in place
.github/skills/code-quality/SKILL.md1Skills CLI (Vercel)
npx skills add zernie/vigiles --skill code-quality -a gemini-cli2Platform CLI
gemini skills install https://github.com/zernie/vigiles --path .claude/skills/code-qualityManually: put the files in place
.devin/skills/code-quality/SKILL.mdThe legacy .windsurf/skills/ path still works
Skills CLI (Vercel)
npx skills add zernie/vigiles --skill code-quality -a opencodeClaude.ai
Zip the folder and upload it in Settings → Capabilities → Skills
Step by step, with commands for each platform: How to install skills in Claude Code: five ways
Questions about code-quality
How do I install code-quality in Claude?
Run npx skills add zernie/vigiles --skill code-quality -a claude-code. The skill lands at ~/.claude/skills/code-quality/SKILL.md.
What does code-quality do?
It applies a set of type-driven techniques to TypeScript and JavaScript: parsing at boundaries, unrepresentable illegal states, exhaustive union matching, pure functions with injected seams, and public-API hygiene.
How is code-quality different from code-review?
One works on shape and design, the other hunts bugs. The skill states plainly that bug hunting is not its job.
Source text
SKILL.md · MIT
--- name: code-quality description: Improve the quality and type-safety of TypeScript/JavaScript being written or refactored — parse-don't-validate, make-illegal-states-irrepresentable, exhaustive matching, pure-function/IO separation, and public-API hygiene. Use when asked to refactor, clean up, improve, harden, or review the design/quality of code or an API. NOT for hunting bugs (that's /code-review) and NOT for vigiles's own spec/lint rules. --- Apply a small set of **type-driven** quality techniques when writing or refactoring code. Each technique has its own reference — the principle, a real before/after from this repo, and the smells to grep for. **Read the reference(s) that match the code in front of you; don't apply all five blindly.** ## When this fires A request to _refactor / clean up / improve / harden / tighten_ code or an API — or when you're about to write a new module/type/public export and want it right the first time. Not a bug hunt (use `/code-review`). ## The techniques — read the matching reference | Smell in front of you | Technique | Reference | | --------------------------------------------------------------------------------------------------------------------------- | --------------------------------------- | -------------------------------------------------------------------------------------------------------- | | a magic `0` / `""` / `-1`; the same primitive re-validated at many sites; `"x" in obj` repeated | **Parse, don't validate** | [`references/parse-dont-validate.md`](references/parse-dont-validate.md) | | a field that's always `""`; boolean-flag soup; a sentinel standing in for "absent"; an impossible combo that still compiles | **Make illegal states irrepresentable** | [`references/make-illegal-states-irrepresentable.md`](references/make-illegal-states-irrepresentable.md) | | a `switch`/`if`-chain on a union with no exhaustiveness guard | **Exhaustive matching** | [`references/exhaustive-matching.md`](references/exhaustive-matching.md) | | logic tangled with IO (`spawn`/`fetch`/`fs`); a function that's hard to test | **Pure functions + injected seams** | [`references/pure-functions.md`](references/pure-functions.md) | | a public export with a fuzzy shape; a `string` that means an enum; a leaked internal helper | **Public-API quality** | [`references/public-api-quality.md`](references/public-api-quality.md) | ## How to apply 1. **Name the decision out loud** before you change the shape — what illegal state it forbids, where the parse boundary sits (the `surface-architecture-decisions` reflex). A quality change the reviewer can't see is churn. 2. **One technique at a time, tests stay green.** Change the type, let `tsc` show you every call site, fix them, run the tests. Never batch five refactors into one unreviewable diff. 3. **Don't over-abstract** (rule-of-three / YAGNI). A tagged union for 3 real variants is good; an abstraction for a difference that doesn't exist yet is the bug this skill is supposed to prevent. 4. **Prefer the strongest option the DECIDABILITY allows** — a type that can't express the bad state beats a runtime check that hopes to catch it; but a runtime guard at a boundary beats nothing when the input is genuinely dynamic. ## Guardrails - **Quality only, not correctness.** This skill improves shape/design; it does not hunt for bugs — that's `/code-review`. - **Every change removes a real failure mode.** If you can't name the bug the new shape prevents, don't make the change. - **Keep the public surface small.** A new `export` is a contract (see the public-API reference); don't widen it to make a refactor convenient.
© zernie · MIT · zernie/vigiles
Similar skills
All skills for Code review and refactoring →code-quality by other authors comes first.