Content
78%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A well-engineered skill body: command-structured, validation-checkpointed, with excellent progressive disclosure to real, appropriately-scoped reference files. The residual gaps are a repeated boundary statement, project-placeholder paths, and the absence of an explicit failure-recovery loop in the sync workflow.
Suggestions
State the Figma boundary (skill produces the artifact, plugin consumes it) once — in Skill Boundaries — and drop the repeats in the intro and setup step 2.
Add one error-recovery line to `/code-to-figma sync`: if the `jq` validation fails or `GIST_TOKEN` is unset, stop and report rather than pushing a partial or empty export.
Pair the `walk-<site>.mjs` placeholder with a single concrete example (e.g. `walk-site.mjs`) so the commands are copy-paste adaptable without opening the reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean — command tables, numbered steps, and non-obvious justifications ("pnpm writes a script header to stdout ... corrupts a `> file.json` redirect") rather than generic explanations. Minor trimming is possible: the Figma-boundary statement ("this skill does not edit Figma files / plugin is the consumer") appears three times — intro, setup step 2's "Keep the core boundary visible", and the Skill Boundaries section — keeping it just below the every-token-earns-its-place anchor. | 4 / 5 |
Actionability | Concrete executable commands throughout: `node scripts/tokens-to-figma/convert-to-dtcg.mjs`, `jq '.sections | length'`, `gh api gists/<id> --jq '.updated_at'`, `gh secret list --repo <org>/<repo>`, and a full six-step sync sequence. Falls short of fully copy-paste-ready because placeholders (`walk-<site>.mjs`, `<org>/<repo>`) are project-dependent and the actual file specifications for setup live in the reference files — flexibility that is explicitly justified, so this is "mostly executable ... minor gaps" rather than score 5. | 4 / 5 |
Workflow Clarity | Each command has a numbered sequence with explicit checkpoints: sync validates with `jq '.sections | length'` before pushing, update validates with `jq -e '.sections | type == "array"'`, setup lists "Required local checks before relying on CI", and there is a stop-and-ask fallback ("If no static HTML artifact can be produced, stop and tell the user rather than guessing a path"). Missing an explicit error-recovery loop for the sync path (what to do when the jq validation fails or `GIST_TOKEN` is absent), so it sits between the score-4 and score-5 anchors. | 4 / 5 |
Progressive Disclosure | SKILL.md is a clean overview (commands, boundaries, principles) with a dedicated "Reference Files — Load when" table plus point-of-use load instructions ("Load references/setup-scaffold.md for the file specifications"). All five referenced files exist, and cross-links between them are peer-level, keeping references one level deep from SKILL.md with no nested chains. Bulk detail (annotated walker templates, CI YAML, JSON contract) is appropriately split out, matching the clear-overview anchor. | 5 / 5 |
Total | 17 / 20 Passed |