Content
61%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-structured procedural skill: clear step sequencing with a concrete policy-creation example and a good request-to-action Examples section. It is weakened by missing MCP tool calls for the SBOM-generation and storage/dashboard workflows, absent validation checkpoints after impactful configuration operations, and some general-knowledge padding in Performance Notes and Troubleshooting.
Suggestions
Add concrete MCP tool calls with parameters for Step 3 (SBOM generation configuration) and Step 5 (storage and dashboards), mirroring the harness_create pattern used in Step 4, so all three workflows are executable rather than descriptive.
Insert validation checkpoints after impactful operations: test a newly created policy against a sample artifact before enabling deployment blocking, and confirm SBOM generation succeeds on one build before requiring it on every build.
Trim general-knowledge explanations (Cosign keyless vs. KMS tradeoffs, Sigstore/Fulcio reachability, SLSA hermetic-build requirements) from Performance Notes and Troubleshooting, keeping only Harness-specific behavior such as the exemption workflow and SBOM scope settings.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The bulk is tight, Harness-specific procedural content ("SBOM scope: direct dependencies, transitive, or full", "Flag dependencies with known CVEs above CVSS threshold (default 7.0)"), but Performance Notes and parts of Troubleshooting explain concepts Claude already knows: "Cosign keyless signing (Sigstore) is simpler to set up than KMS-backed keys but requires internet access", "SLSA Level 3 requires hermetic builds", "verify the OIDC provider (Sigstore/Fulcio) is reachable". This is minor trimmable over-explanation fitting anchor 4; not 3 because there is little filler and the workflow steps assume Claude's competence, and not 5 because the general-knowledge padding is real. | 4 / 5 |
Actionability | Steps 1 and 4 are fully executable ("Call MCP tool: harness_list" with parameters; "Call MCP tool: harness_create" with a complete Rego policy), and requirement lists name concrete options ("Cosign keyless (Sigstore), Cosign with KMS, AWS KMS, GCP KMS"). However, Step 3 ("Configure SBOM generation in the CI pipeline") and Step 5 ("Enable the SSCA portal dashboard") describe what to configure without any MCP tool call or parameters, leaving key executable details missing for two of the three main workflows. That fits anchor 3; not 4 because the gaps are not minor, and not 2 because the guidance that is present is concrete and copy-adaptable. | 3 / 5 |
Workflow Clarity | Steps 1-5 are clearly sequenced with a conditional branch in Step 2 (SBOM generation vs. policy enforcement), but validation checkpoints are absent or implicit: no step verifies the created policy before it blocks deployments (e.g., testing it against a sample artifact) or confirms a successful SBOM generation run before rollout. Error recovery lives in a separate Troubleshooting section rather than as feedback loops in the workflow. This matches anchor 3 (sequence present, checkpoints missing); not 4 because only Step 1's scope confirmation acts as a checkpoint. | 3 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), so all content is inline in a ~125-line body organized under clear headers (## Instructions with numbered steps, ## Examples, ## Performance Notes, ## Troubleshooting), which is appropriate placement for its size. This fits anchor 4; not 5 because at this length the Rego policy library (five policy types listed, only two exemplified) and troubleshooting recipes would naturally live in one-level-deep reference files, and not 3 because the structure is clean and nothing glaringly belongs elsewhere. | 4 / 5 |
Total | 14 / 20 Passed |