Content
70%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.
Operationally superb: fully executable commands, exhaustive validation checkpoints, and error-recovery loops for every phase, including a carefully designed unattended-mode gate table. The weaknesses are the mirror image of that thoroughness — a ~25K-token body that inlines hundreds of lines of reference-grade material (much of it duplicated across phases) instead of pointing into the reference files it already maintains.
Suggestions
Move the Phase 6.7 import-retry/lock forensics, the Griffel shorthand-and-longhand tables, the secret-sanitizer limits, and the shell-quoting rules into the existing references (troubleshooting.md, rules.md) and leave a one-line rule plus pointer at each use site — this alone would roughly halve the body and lift both conciseness and progressive disclosure.
De-duplicate the Griffel check explanation between Phase 5b step 9 and Phase 6: state the rule once (or in rules.md) and reference it, rather than re-enumerating the ten shorthands and their longhands twice.
Rewrite the References section to list every file the body cites under one consistent base path (e.g. ${PLUGIN_ROOT}/references/...) so navigation is predictable, and fix the mixed relative paths (../../references/, bare 'edit-flow.md in this folder').
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~1500-line body is noticeably verbose: a ~70-line Phase 6.7 digression on import retry timing ("The sends are at most five and the time at most five minutes in all... 10, 20, 40 and 60 seconds between the sends") and import-job XML log forensics; the Griffel shorthand longhand enumeration explained at full length twice (Phase 5b step 9 and again in Phase 6); and a shell-quoting treatise ("in bash close, escape and reopen (`'Bob'\''s app'`)") covering quoting basics Claude already knows. Not 1 because almost none of it is generic padding or basic-concept tutorial — it is dense, non-obvious operational detail, just far beyond a main skill file's budget; not 3 because the over-explanation is systematic (hundreds of lines of reference-grade material inlined), not 'some sections that could be tightened'. | 2 / 5 |
Actionability | Fully executable throughout: exact copy-paste commands with every flag ("node "${PLUGIN_ROOT}/scripts/genpage-upload.js" --env '<org-url>' --app-id '<app-id>' --code-file ... --prompt-file ... --solution '<Solution unique name>' --add-to-sitemap"), JSON result schemas with exit-code semantics, and complete agent-dispatch prompt templates. Common cases (create, update, mock pages, connectors, new app) each have a filled-in example. Not 4 because there are no minor gaps — commands, expected outputs, and failure branches are all specified. | 5 / 5 |
Workflow Clarity | Phases 0–8 are clearly sequenced with explicit validation checkpoints at every risky step ("Continue only on exit 0", "Continue only on \"ok\":true\"", provenance verify before Phase 2, completeness and Griffel gates before deploy) and genuine feedback loops (rewrite once and re-run the gate, re-invoke the planner with answers, fix-and-redeploy in Phase 7). Destructive and unattended operations are explicitly gated ("Suppressing a prompt never authorizes destructive work... halt and say so"), so the destructive-cap does not apply. Not 4 because validation and error recovery are explicit everywhere, not just mostly present. | 5 / 5 |
Progressive Disclosure | On-demand loading is done well ("Only when the plan's ## Connector Bindings section contains an actual binding table... also read ${PLUGIN_ROOT}/references/connectors.md"; verify-flow.md loaded only when the user opts into verification), and references are one level deep. But large reference-grade content is inlined in the body (import-retry semantics, Griffel longhand tables, secret-sanitizer edge cases, compiler-iteration rules — the last even duplicating a rules.md section by name), path bases are inconsistent ("../../references/rules.md" vs "${PLUGIN_ROOT}/references/..." vs "edit-flow.md in this folder"), and the 'References' section lists only 3 of the ~15 files the body cites — none of which are present in this bundle to verify. Not 4 because a substantial fraction of the content that should be separate is inline; not 2 because references are clearly signaled with explicit load conditions rather than buried. | 3 / 5 |
Total | 15 / 20 Passed |