Content
88%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 tightly engineered operational skill: exact tool invocations, verbatim plan/verify templates, a confirmation-gated six-phase workflow with collision checks, live-state failure recovery, and read-back verification. The weaknesses are minor: duplicated collision/routing text costs tokens, and every detailed reference (tool-map, write-safety, targeting-recipes, concepts, schema-validation-loop) points outside the skill directory and none ship in the bundle.
Suggestions
Consolidate the name-collision protocol — currently written out in both Phase 3 and the Troubleshooting 409 row — into one location, and cross-reference it from the other; the same applies to sibling-skill routing, which appears in both the intro and Phase 6.
Ship the referenced files (tool-map.md, write-safety.md, concepts.md, targeting-recipes.md, schema-validation-loop.md) inside this skill's bundle, or inline the critical safe-default definition body — all current links use ../../../ or ../ paths that resolve outside the skill directory and none exist in the provided bundle.
Split Phase 3's overloaded bullets (e.g., the environment-initialization rule mixing default, delegated, and STOP conditions in one sentence; the owners-confirmation rule) into labeled sub-bullets so each constraint scans independently.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational with zero explanation of concepts Claude already knows (no "what is a feature flag" padding), and detail is pushed to references. Not 5: tokens don't all earn their place — the name-collision protocol is stated twice (Phase 3 "If the flag exists during a delegated create, return the collision..." and the 409 Troubleshooting row), sibling routing is repeated (intro "Related: update-flag-targeting (ramp), manage-experiments (A/B test)" and again in Phase 6), and Phase 3/Phase 5 pack multiple rules into single overloaded sentences. Not 3: this is minor duplication, not "some unnecessary explanation". | 4 / 5 |
Actionability | Quotes: exact MCP tool names with parameter shapes and CLI commands per operation ("harness_create · fme_feature_flag_definition · params: { feature_flag_name, environment_id } · body: { treatments, defaultTreatment, defaultRule, comment? }"), a verbatim plan template, a literal audit comment ("create-feature-flag: initial definition with safe default — <ticket or purpose>"), and a verification restatement template. Guidance is fully executable, including the CLI tags/owners syntax caveat. Not 4: no meaningful gaps — the only deferred spec (the safe-default definition body) is pointed to by an exact anchor link, which is deliberate disclosure, not missing detail. | 5 / 5 |
Workflow Clarity | Quotes: six clearly sequenced phases (Scope → Explore → Propose → Plan and confirm → Execute → Verify) with explicit checkpoints: "STOP and wait for explicit confirmation", a pre-create collision check ("A 404 still requires correct scope/identifier/access before creation"), a mid-failure feedback loop ("report exactly which environments succeeded and which failed... Resume from that live state"), 400 handling routed to a schema-validation loop, and a verify phase that reads back live flag/definition state. Validation steps and error-recovery loops are explicit for this batch-write operation. Not 4: no validation gaps — collision check, confirmation gate, live-state resumption, and post-write verification are all present. | 5 / 5 |
Progressive Disclosure | Quotes: references are clearly signaled with purpose and anchor targets ("Restate: Working in org=..., project=..." → scope-establishment.md; "Never control (reserved; see [concepts.md])"; "See [targeting-recipes.md](...#i-initialize-a-definition-where-none-exists) for the definition body"; pagination and common-errors anchors in tool-map.md) — all one level deep, no chained references, and the SKILL.md body stays a lean overview. Not 5: scored against the actual bundle structure, no references/ directory exists and every link resolves outside the skill's own directory ("../../../references/fme/tool-map.md", "../update-flag-targeting/references/targeting-recipes.md"), so the referenced detail is not shipped with or verifiable in this bundle. Not 3: references are clearly signaled and content is appropriately split, not buried or inlined. | 4 / 5 |
Total | 18 / 20 Passed |