Content
81%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 operational skill: fully sequenced workflow with explicit drift validation, graceful degradation, and a concrete output contract, plus executable commands and realistic heuristics. The remaining gaps are small — a duplicated shell-guard boilerplate, a missing WhoAmI command, an only-approximated team-scope filter, and heuristics tables that could be split into a reference file.
Suggestions
State the `cd -- '<working_dir>'` guard once as a precondition applying to every shell call, instead of repeating the four-line block in Steps 1–3 (conciseness).
Add the exact `WhoAmI` invocation via `dataverse-request.js` and a concrete query for the team-owned scope, so all five scope rows have executable commands rather than prose approximations (actionability).
Move the byte-per-row heuristics table and scope-filter mapping into a references/ file (e.g. estimate-heuristics.md) linked from Steps 3–4, keeping SKILL.md as overview plus workflow (progressive_disclosure).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense and domain-specific throughout — the scope-filter table, byte heuristics, and report template all earn their tokens, and nothing explains concepts Claude already knows. It is not 5 because the `cd -- '<working_dir>' || { echo "BLOCKED..." }` guard block is repeated verbatim in three code snippets and "Read-only diagnostic ... No mutations" restates the description; not 3 because there is no real padding. | 4 / 5 |
Actionability | Mostly executable: exact node CLI invocations with flags (`--tenant-id`, `--no-cache --require-tenant`), exact OData query strings for three of the five scopes, a numeric byte-per-row table, transfer formulas, and a complete report template. It is not 5 because the `WhoAmI` identity call is described in prose without its command line, and the team-owned scope filter is only approximated ("use team count from `teamroles_association`") with no concrete query; not 3 because the given commands are copy-paste ready, not pseudocode. | 4 / 5 |
Workflow Clarity | The 5-step sequence is stated up front ("1. Verify project + locate profile → 2. Run verify → 3. Per-table row counts → 4. Cache-size estimate → 5. Report") and elaborated with explicit validation (drift check with "surface the drift list verbatim" and a reconcile recommendation), error-recovery paths (BLOCKED/NEEDS_CONTEXT exits, per-scope fallback "instead of blocking the whole preview"), and a terminal status-code contract. Not 4 because checkpoints and feedback loops are explicit rather than implicit. | 5 / 5 |
Progressive Disclosure | Good structure for a self-contained skill: clear section headers, an upfront workflow summary, and well-signaled one-level-deep links to plugin-shared files (shared-instructions.md, app-working-directory.md); no bundle files exist to organize. It is not 5 because ~40 lines of estimate heuristics and the scope-filter mapping are inlined in a ~160-line file where a references/ file could carry them, and the sub-50-line exception does not apply; not 3 because nothing is buried and the inline content is core to execution. | 4 / 5 |
Total | 17 / 20 Passed |