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 highly actionable, well-sequenced operational skill: complete YAML patterns, a phase-by-phase wizard with explicit confirm-before-write checkpoints, error-retry loops, and troubleshooting keyed to specific error codes, with real one-level-deep local references. Its main weaknesses are redundancy (the CD-on-CI-only rule and the do-not-execute rule are each restated 3-5 times) and reliance on cross-skill reference paths that are not part of this bundle.
Suggestions
Consolidate the CD-on-CI-only guidance, currently repeated across the Interaction model (items 11-12), the phase table row 3b, the 'CD edge case (mandatory workflow)' section, Performance Notes, and Troubleshooting, into a single authoritative Phase 3b section and reference it from the other locations.
Slim the wizard phase table to the breadcrumb plus phase names and defer the per-phase actions to references/interactive-wizard-flow.md, which already documents every phase in full.
Copy the essential patterns from the sibling-skill references (cd-containerized-step-group.md, sbom-orchestration-step.md, entity-sbom.md) into this skill's references/ directory - or inline the key YAML fragments - so the skill remains navigable when the /create-sbom and /create-policy skills are not installed.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient - terse tables, imperative bullets ("Fetch before configure", "Never guess image tags"), no explanations of concepts Claude already knows - but it could be tightened: the CD-on-CI-only rule is restated in the Interaction model ("Never block CD on 'no Deployment stage'"), the phase table row 3b, a dedicated "CD edge case (mandatory workflow)" section, Performance Notes, and Troubleshooting, and "do not call harness_execute" / "Do not execute pipelines" appears three times. This matches the anchor 'Mostly efficient but includes some unnecessary explanation or could be tightened' rather than 2, since the padding is repetition of actionable rules, not concept filler. | 3 / 5 |
Actionability | Fully executable guidance: a copy-paste-ready SscaEnforcement YAML block with placeholders, variants for policy-only and no-verification, the concrete harness_update call with "body: { yamlPipeline: \"<updated pipeline YAML>\" }", a complete summary template, and error-code-keyed fixes ("DUPLICATE_IDENTIFIER - rename enforce_sbom", "Invalid policy.policySets - use policy set identifiers, not display names"). CD YAML is fully covered in the local reference (sbom-enforcement-step.md has 'Full example - CI' and 'CD Deployment (containerized step group)' sections), so specific examples cover the common cases, matching the score-5 anchor. | 5 / 5 |
Workflow Clarity | The wizard is explicitly sequenced (Phases 0-10 plus 3b with a breadcrumb) with explicit validation checkpoints - "Fetch before configure" (harness_get before questions), "Confirm before write" (Phase 10 confirm before harness_update), "Do not modify unrelated steps, variables, or failure strategies" - and a real error-recovery feedback loop: "On validation errors, read the API message, fix fields (often verifyAttestation shape or policy.policySets), retry", plus a thorough symptom-to-fix Troubleshooting section. This matches the anchor 'Clear sequence with explicit validation steps; feedback loops for error recovery'. | 5 / 5 |
Progressive Disclosure | Good structure: the two local references are real, clearly signaled, and one level deep ("Wizard: references/interactive-wizard-flow.md", "Full UI mapping: references/sbom-enforcement-step.md"), and the body appropriately keeps the overview inline. It falls short of 5 because several referenced paths point outside the bundle - skills/platform/create-sbom/references/cd-containerized-step-group.md, skills/platform/create-sbom/references/sbom-orchestration-step.md, and skills/observability-governance/create-policy/references/entity-sbom.md do not exist in this skill's bundle - so navigation breaks if the sibling skills are not installed, and the body's wizard phase table duplicates content in interactive-wizard-flow.md. | 4 / 5 |
Total | 17 / 20 Passed |