Content
85%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.
The body is an actionable, well-sequenced pattern guide: every step has executable code, exact integration points, and explicit validation checkpoints, with troubleshooting tied to real error messages. Its weaknesses are repetition of a few details (frontmatter format, Cursor special case) and a monolithic structure — no reference files despite being well past the length where the per-platform catalog, worked example, and Common Issues could be split out.
Suggestions
Split the Step 2 per-platform append-block catalog, the full DevCode worked example, and the Common Issues section into files under references/ (e.g., references/platform-patterns.md, references/example-devcode.md, references/troubleshooting.md), leaving SKILL.md as a concise overview with clearly signaled one-level-deep links.
State the skill frontmatter format once and reference it from the 'Skill file has no frontmatter / malformed YAML' issue instead of restating it; do the same for the Cursor rule-injection special case, which appears in both Step 2 and Common Issues.
Trim line-number citations that will rot as the codebase changes (e.g., 'line 9', 'lines 19-22', 'line 40-47') in favor of function/anchor names (e.g., 'the writeSkills loop in src/writers/claude/index.ts'), which stay valid and cost fewer tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with information Claude cannot infer — exact helper compositions ('appendSyncBlock(appendLearningsBlock(appendPreCommitBlock(config.claudeMd)))'), file paths, and line-number pointers — so nearly every token earns its place, matching the 'efficient; minor instances of over-explanation that could be trimmed' anchor. It is not a 5 because of repetition: the skill frontmatter format is specified twice (Step 2 and the 'Skill file has no frontmatter' issue) and the Cursor rule-injection special case is explained in Step 2 and restated nearly verbatim in Common Issues. | 4 / 5 |
Actionability | The guidance is fully executable: complete TypeScript snippets (the DevCode example is copy-paste ready with interface, function, and test), exact commands ('npm test -- src/writers/__tests__/<platform>.test.ts'), and specific assertions (".toHaveBeenCalledWith(path, { recursive: true })", "expect content .toContain('caliber:managed:pre-commit')"). This matches the top anchor — specific examples cover the common cases; the 4 anchor's 'minor gaps' do not apply. | 5 / 5 |
Workflow Clarity | Steps 1–6 are clearly sequenced with an explicit '**Validation**' checkpoint after each, plus a pre-flight uniqueness check ('ls src/writers/ shows no <platform>/index.ts'), a test step, and a Common Issues section providing fix-and-retry feedback loops for concrete error messages ('ENOENT: no such file or directory', 'write<Platform>Config is not a function'). This matches the top anchor including error-recovery loops for the batch file-writing operations; the 4 anchor's 'minor validation gaps' are not present. | 5 / 5 |
Progressive Disclosure | There is good section structure (Critical, Instructions, Examples, Common Issues), but the skill is a ~260-line monolith with no bundle files at all (references/, scripts/, assets/ are absent). Content that plausibly belongs one level deep — the per-platform append-block catalog in Step 2, the full DevCode worked example, and the six Common Issues entries — is inlined, matching the 'some structure but could be better organized; content that should be separate is inline' anchor. It is above the 2 anchor because the inline content is well-sectioned and navigable, and below the 4 anchor because nothing is split out despite the length. | 3 / 5 |
Total | 17 / 20 Passed |