Content
90%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 strong, dense skill body: fully executable commands, precise paths, and repo-specific knowledge that assumes Claude's competence with zero filler. The endpoint workflow is well sequenced but leaves validation implicit (tests are documented but not wired into the workflow), and the auth-model section is the only candidate for extraction to a reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every section carries non-inferable, repo-specific knowledge (the repo layout tree, 'Route auth is declared per-route via AuthType in pkg/api/http/controller/group.py', exact uv/pnpm/pytest commands) with no padding or explanation of concepts Claude already knows. Assumes competence throughout, e.g. 'NOT Next.js — dev script is vite' delivers the one non-obvious fact in a parenthetical. | 5 / 5 |
Actionability | Commands are copy-paste ready and specific: 'uv run python -m langbot.pkg.persistence.alembic_runner autogenerate "description"', 'uv run pytest tests/unit_tests/api -q', 'cd web; pnpm install; pnpm dev'. The endpoint workflow gives exact directories ('pkg/api/http/controller/groups/', 'pkg/api/mcp/server.py') and enumerated AuthType values; only the conditional OpenAPI step is slightly vague, which is explicitly hedged. | 5 / 5 |
Workflow Clarity | The 'Adding an API endpoint' section is a clear numbered sequence with real checkpoints ('API and MCP surface must stay aligned', 'check the separate langbot-cli client for compatibility'). However, no explicit validation step is wired into the workflow — test commands exist in a separate 'Tests' section but the workflow never says to run them after an endpoint change, a minor checkpoint gap. | 4 / 5 |
Progressive Disclosure | Good structure for a single-file skill: well-organized sections (Stack, Dev environment, Repo layout, auth model, endpoint workflow, Tests, See also) with clear one-level pointers to sibling skills ('langbot-mcp-ops', 'langbot-plugin-dev'). No bundle files exist, and the ~30-line auth-model detail is the only content that arguably belongs in a separate reference file were the skill to grow. | 4 / 5 |
Total | 18 / 20 Passed |