Content
63%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 highly actionable, validation-rich skill whose commands are copy-paste ready and whose troubleshooting covers real failure modes. Its weaknesses are token efficiency and structure: large amounts of reference-grade detail are inlined in SKILL.md instead of being split into bundled reference files, and a few referenced artifacts are missing from the bundle.
Suggestions
Move the deep operational detail — the NANOCLAW_GATEWAY_UNCREDENTIALED_READS grant-caching/race semantics and the approval-front architecture description — into a bundled reference file (alongside references/control-plane.md), keeping a one-to-two-line summary and a clear pointer in SKILL.md.
Split the Codex and OpenCode authentication sections into a reference file (or a dedicated auth.md), keeping only the provider-auth entry-point commands and the key warnings inline.
Fix the bundle's dangling references: ship the missing REMOVE.md and versions.json referenced by the body, and move the "Provider credentials on shared model hosts" subsection out from under "Remove" into its own clearly labeled upgrade/refresh section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-inferable operational detail (timeouts, fail-closed rules, env var defaults), so it avoids explaining things Claude already knows, but sections like the ten-line NANOCLAW_GATEWAY_UNCREDENTIALED_READS grant-cache/race paragraph and the twice-described front architecture (in the install section and again in Validate) are padded beyond overview level. It fits anchor 3 ("could be tightened") better than anchor 4's "minor instances". | 3 / 5 |
Actionability | Guidance is overwhelmingly executable: exact nc:copy path maps, pinned dependency versions, copy-paste tsx commands (setup --with-control, control.ts grant, --allow-host, provider-auth), and env-var defaults with remediation commands. It falls short of anchor 5 only because the Remove workflow directs the reader to a REMOVE.md that is not present in the bundle. | 4 / 5 |
Workflow Clarity | The install→register→dependencies→setup→validate sequence is clear, with an explicit Validate section (build plus a full named test suite) and a Troubleshooting section with diagnosis and recovery steps ("On failure, fix the reported access or service issue and rerun setup"). It misses anchor 5 because the refresh/upgrade procedure is buried in the "Provider credentials on shared model hosts" subsection under Remove, and sequencing across the multiple workflows is only implicit. | 4 / 5 |
Progressive Disclosure | Section headers are good and the one bundled reference is clearly signaled ("Read [references/control-plane.md]... for the exact supported UI, source links, policy example, and verification steps"), but ~19KB of reference-grade detail — the OpenCode authentication minutiae and the credential-rule caching semantics — is inlined rather than split out, and three referenced paths (REMOVE.md, versions.json, docs/gateway-seam.md) are absent from the bundle. This fits anchor 3 (content that should be separate is inline) better than anchor 4's "minor organization gaps". | 3 / 5 |
Total | 14 / 20 Passed |