Content
75%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.
Highly actionable, ADO-dense content with explicit validation feedback loops and a well-verified one-level-deep reference structure. The main cost is token efficiency: Best Practices and Anti-Patterns sections say the same things twice, validation instructions repeat four times, and seven inline YAML examples duplicate the bundled asset files.
Suggestions
Merge the 'Best Practices to Enforce' numbered list into the 'Anti-Patterns' section (or vice versa) as a single BAD/GOOD catalogue — they currently cover the same eight rules (version pinning, secrets, ubuntu-latest, :latest deploys, displayName, deployment jobs, test publishing, validation) in two different formats, roughly halving that portion of the body.
State the validation workflow once (Overview) and reference it elsewhere; the generate → validate → fix → re-validate loop currently appears in the Overview, Mandatory Standards #7 context, the 'NEVER deliver a complete pipeline without validating it' anti-pattern, and the Troubleshooting table.
Trim the inline per-capability YAML examples to short distinguishing snippets and point to the corresponding assets/examples/*.yml file (which already exists for each capability), keeping the body as an overview rather than a duplicate of the example bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most content is ADO-specific knowledge Claude cannot be assumed to know (task version pins, 'GoTool@0 (only major version available)', Cache@2 key formats, deployment-job requirements), so tokens mostly earn their place. However, the 'Best Practices to Enforce' list and the 'Anti-Patterns' section cover the same ground twice (version pinning, secrets, ubuntu-latest, :latest deploys, displayName, deployment jobs, test publishing, validation), the Docker tagging rule appears in both section 3 and Anti-Patterns, and validation instructions appear in four places (Overview, Mandatory Standards, Anti-Patterns, Troubleshooting). Not a 4 because consolidating these duplicated sections would meaningfully shrink the ~420-line body; not a 2 because there is little explanation of concepts Claude already knows. | 3 / 5 |
Actionability | Fully executable guidance throughout: seven complete copy-paste-ready YAML examples with pinned task versions ('NodeTool@0', 'Docker@2', 'KubernetesManifest@0'), exact naming conventions ('PascalCase', "Sentence case ('Build application')"), specific Go flags ('CGO_ENABLED=0', '-race -coverprofile'), concrete MCP tool chain ('mcp__context7__resolve-library-id → mcp__context7__get-library-docs'), a ready fallback search query, and an error-to-resolution troubleshooting table. Common cases are covered by the examples. Not a 4 because no significant execution gap was found. | 5 / 5 |
Workflow Clarity | The core loop is explicit with a feedback checkpoint: generate → 'always validate it using the devops-skills:azure-pipelines-validator skill, fix any reported issues, and re-validate before presenting', plus a documentation-lookup escalation path (local refs → Context7 → WebSearch → Microsoft Learn → ask user) and a validator-error resolution table. Not a 5 because the end-to-end sequence (Understanding → Reading → Lookup → Generate → Validate → Present) lives only in references/typical-workflow.md and the process guidance is distributed across Overview, section 7, 'Workflow Process', and 'Troubleshooting' rather than stated once inline; not a 3 because validation checkpoints and recovery steps are explicit, not implicit. | 4 / 5 |
Progressive Disclosure | Good structure verified against the actual bundle: every referenced path exists (references/yaml-schema.md, tasks-reference.md, best-practices.md, templates-guide.md, typical-workflow.md; all 10 assets/examples files including templates/build-template.yml and deploy-template.yml), references are one level deep, and each capability section signals exactly which files to 'Read', reinforced by a When-to-Use reference table. Not a 5 because the body inlines seven full YAML examples that substantially duplicate the assets/examples/*.yml files, and the 10-entry Anti-Patterns catalogue overlaps references/best-practices.md — content that could largely live in the referenced files. | 4 / 5 |
Total | 16 / 20 Passed |