Content
38%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.
The body is cleanly sectioned and security-conscious, but it reads as a generic process contract rather than 1Password expertise: no op CLI commands, syntax, or reference formats appear anywhere, and its progressive-disclosure pointers reference paths that do not exist in the bundle while ignoring the files that do. The result is a skill that describes how to behave but not how to actually accomplish 1Password tasks.
Suggestions
Add a concrete command quick-start for the core workflows: op signin / service-account setup, op item get, the op://<vault>/<item>/<field> reference format, and op run / op inject usage examples — the skill currently contains no executable op CLI syntax.
Fix the progressive-disclosure pointers to match the actual bundle: link references/contract.yaml, references/evals.yaml, and references/task-profile.json, and remove or correct the nonexistent 'Infrastructure/references/...' and 'deferred-skill-context/security-ops-1password/' paths.
Define the workflow's validation gates concretely (e.g., 'op whoami' to verify session, 'op run -- printenv <VAR>' to verify injection) and merge the duplicated redaction rules from 'Security Constraints' and 'Execution Boundaries' into one section, dropping the verbatim description repeat at the top of the body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Individual lines are terse and it never explains concepts Claude already knows, but the body carries padding: the frontmatter description is repeated verbatim at the top of the body ('Plan, diagnose, and validate 1Password CLI workflows...'), redaction rules appear in both 'Security Constraints' and 'Execution Boundaries', and 'Philosophy'/'Avoid'/'Gotchas'/'Anti-Patterns' are generic process boilerplate applicable to any skill. This fits anchor 3 ('mostly efficient but includes some unnecessary explanation or could be tightened') rather than 2, since no section is an actual explanation of known material — the issue is duplicated and generic content that could be consolidated. | 3 / 5 |
Actionability | For a CLI skill, the body contains zero executable commands or syntax — no 'op signin', no 'op run --', no 'op://vault/item/field' reference format, no 'op inject' usage — only abstract directives like 'Run the narrowest real validator or command path available' and 'Inspect 2-3 focused surfaces'. This matches anchor 2 ('minimal concrete guidance; high-level hints but missing the specific steps to execute'): there are a few concrete behavioral rules (redact by default, don't write credentials to disk, stop at the first failed gate), so it is not anchor 1's 'entirely vague', but nothing is executable. | 2 / 5 |
Workflow Clarity | The Workflow section lists a clear 5-step sequence and the 'Failure Mode' and 'Validation' sections do provide feedback-loop language ('Stop at the first failed gate', 'Rerun the relevant validation after fixes before claiming completion'), which prevents a score of 2. But the checkpoints are implicit in practice — 'gate', 'validator', and the '2-3 focused surfaces' are never concretely defined for 1Password, so the reader cannot tell what to actually check — fitting anchor 3 ('steps listed but validation gaps... checkpoints missing or implicit') rather than 4's 'most checkpoints present'. | 3 / 5 |
Progressive Disclosure | The body points to 'Infrastructure/references/software-literature-expert-lens-pack.md', 'Infrastructure/references/software-literature-skill-expertise-map.md', and 'Infrastructure/references/deferred-skill-context/security-ops-1password/' — none of which exist in this bundle — while the actual bundle files (references/contract.yaml, references/evals.yaml, references/task-profile.json) are never mentioned. Per the guideline to score against the actual bundle structure, these are broken/dangling references that make navigation impossible, matching anchor 2 ('references are buried' / minimal structure). It is not anchor 1 because the body itself is well-sectioned with headers rather than a monolithic wall of text. | 2 / 5 |
Total | 10 / 20 Passed |