Content
47%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill functions well as a comprehensive routing hub with excellent workflow clarity and decision tables for matching user intent to sub-skills. However, it suffers significantly from verbosity—the 7-step context resolution procedure, metadata overlay rules, and legacy migration logic bloat the main file far beyond what's needed for routing. Moving these procedural details to a referenced sub-document would dramatically improve conciseness and progressive disclosure.
Suggestions
Extract Steps 1-7 ('Common Project Context Resolution') and the metadata write rules into a separate reference document (e.g., 'references/context-resolution.md') and replace with a 2-3 line summary and link in the main SKILL.md.
Remove or relocate the '.foundry Workspace Standard' section and 'Setup References' section to referenced documents—they are implementation details that don't belong in the top-level routing skill.
Add at least one concrete executable example (e.g., a sample azd command or MCP tool invocation) to the main skill body to improve actionability beyond pure routing tables.
Trim the legacy metadata migration logic (testSuites/testCases normalization, priority-to-tags mapping) into a dedicated migration reference document rather than embedding it in the main routing skill.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is extremely verbose at ~300+ lines. Much of the content—especially the multi-step context resolution (Steps 1-7), metadata overlay rules, legacy migration logic, and environment resolution cascades—is exhaustive procedural detail that could be in a referenced sub-document. The main SKILL.md reads like an internal specification rather than a lean routing document. | 1 / 3 |
Actionability | The skill provides concrete routing tables, specific azd variable names, and clear MCP tool references, which is useful. However, it contains no executable code examples or copy-paste-ready commands in the main body—most actionable content is deferred to sub-skills. The context resolution steps are detailed but procedural/descriptive rather than executable. | 2 / 3 |
Workflow Clarity | The multi-step context resolution workflow (Steps 1-7) is clearly sequenced with explicit decision points, fallback logic, and validation checkpoints (e.g., 'stop and ask which source is authoritative' when azd and metadata conflict). The agent development lifecycle table provides clear workflow chains with ordering. The infrastructure lifecycle includes disambiguation steps. | 3 / 3 |
Progressive Disclosure | The skill correctly uses a sub-skill table with clear routing and references to 17+ sub-documents, which is good progressive disclosure architecture. However, the main SKILL.md itself is a wall of text—Steps 1-7 of 'Common Project Context Resolution' and the metadata overlay rules are inline content that should be in a referenced document (e.g., 'references/context-resolution.md'), keeping the main skill as a lean routing hub. | 2 / 3 |
Total | 8 / 12 Passed |