Content
73%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-structured engineering skill with genuinely non-obvious project policy, exact commands and MCP tool calls, and an unusually strong validation culture (two-layer self-verify gate with explicit error paths). Its weaknesses are duplication — the 404/History-mode rule and two workflow sections are each stated twice, roughly doubling token cost — and a truncated final code snippet plus references that don't resolve in this partial bundle.
Suggestions
Merge 'How to use this skill' and 'Core workflow' (they cover the same four points: choose path, follow conventions, browser-validate, CloudBase-as-branch) and deduplicate the 404/History-mode rule, which is written out twice at full length — state it once in 'Static hosting defaults' and link from 'Common mistakes'.
Complete the truncated CloudBase quick-start snippet — `const auth = app.auth` is cut off mid-statement; finish it with the real call (e.g. `app.auth()` with persistence config) or drop the line.
Ship the referenced bundle files (`frameworks.md`, `browser-testing.md`, and the cross-cutting protocol files) alongside SKILL.md, or inline the minimal needed content, since in this environment every 'Then also read' pointer dead-ends.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with genuinely non-obvious, project-specific policy (the History-mode 404 gotcha, manageApps vs manageHosting deployment-path distinction, EnvId alias rule), but it is noticeably duplicated: the History-mode 404 warning is stated twice at full length ("Common mistakes" and "Static hosting defaults"), the "Do NOT use for" list appears in both the Activation Contract and "When to use this skill", and "How to use this skill" and "Core workflow" cover the same four points (choose path, follow conventions, browser-validate, CloudBase-as-branch) twice. It is not 2 because there is no conceptual padding — nothing explains what React or a frontend is — and every section carries project-specific rules Claude could not infer. It is not 4 because the duplication and the somewhat over-elaborated TypeScript rules are real token waste that could be cut roughly in third without losing information. | 3 / 5 |
Actionability | Mostly executable: exact commands (`tsc --noEmit`, `npm install @cloudbase/js-sdk`, `npm run build`), exact files to inspect (`src/lib/backend.*`, `src/lib/auth.*`, route guards), a CDN URL, and copy-paste MCP tool calls (manageHosting setWebsiteDocument / queryHosting websiteConfig) cover the common deploy and validation cases. It is not 5 because the one JS example ends with a truncated line — `const auth = app.auth` (no call, no completion) — and because the primary guidance pointers (`frameworks.md`, `browser-testing.md`, sibling skill paths) resolve to files that are not present in this bundle, so an agent following them hits dead ends (the skill does at least instruct asking the user to install missing siblings). It is above 3 because the inline guidance is concrete and executable, not pseudocode or vague direction. | 4 / 5 |
Workflow Clarity | The "Self-verify before claiming done" section is a full validation checklist with explicit sequencing: static/build layer first (`tsc --noEmit`, eslint, build, unit tests), then runtime/browser layer (agent-browser: route loads, fixed interaction behaves, no new console errors, no adjacent regressions), with an explicit gate ("Only after both layers pass may you say the task is done") and an error-recovery path ("If either layer cannot be executed locally... say so explicitly and list exactly which step is still unverified"). The debugging path also encodes a loop (reproduce → narrow → patch). It is not 4 because the anchor-4 weakness ('minor validation gaps') does not apply — validation is present at every risky stage including deployment (referenced deployment-gate protocol) and results recording (route, action, expected, actual). | 5 / 5 |
Progressive Disclosure | The body acts as an overview with clearly signaled, one-level-deep references using a consistent arrow convention ("General React / Vue / Vite guidance -> `frameworks.md`", "Browser flow checks -> `browser-testing.md`", sibling SKILL.md and protocol paths), and it gracefully handles missing siblings ("If a referenced sibling skill file is missing from this environment, ask the user to install the full CloudBase plugin"). It is not 5 because, against the actual bundle structure, the referenced files (`frameworks.md`, `browser-testing.md`, the sibling skills and protocols) do not exist in this environment, so navigation dead-ends, and some content that would more naturally live in a reference file (the full TypeScript constitution, the detailed static-hosting deployment rules) is inlined in SKILL.md. It is not 3 because references are clearly signaled and never nested more than one level — the 200-line inline blocks that plague the anchor-3 example are absent, and the SKILL.md remains a navigable overview. | 4 / 5 |
Total | 16 / 20 Passed |