Content
67%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-engineered, highly actionable multi-mode skill with excellent trigger routing, executable commands, and a disciplined reference structure. Main weaknesses are redundancy between the scope-resolution and routing sections and one orphaned bundle file.
Suggestions
Merge the "Resolve the analysis target" section's routing rules into the Entry Router table (or vice versa) and deduplicate the repeated do-not-browse-other-products prohibitions to cut a significant token block.
Tighten the Preflight edition/digest policy prose and move the detailed rules into references/analysis_checklist.md, keeping only the command block inline.
Either link references/original-app-demo.md from the relevant workflow section (e.g., fact gathering or preflight for app bundles) or remove it, since it is currently unreachable from SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | There is no padding of concepts Claude already knows — the length is dense domain policy — but "Resolve the analysis target" substantially overlaps the Entry Router, prohibitions repeat ("Do not browse other product directories" / "browse $COMPETITORS_BASE to choose an unrelated product"), and Preflight prose like "A digest identifies bytes, not the requested edition, latest release, or completeness of the collection" could be tightened. Mostly efficient but several sections need consolidation, so it sits at the could-be-tightened anchor rather than the minor-trims one. | 3 / 5 |
Actionability | Copy-paste-ready blocks cover the main paths: preflight git commands, clone with SSH/HTTPS fallback, `gh search repos` with exact flags and JSON fields, `nl -ba` citation practice, and script invocations with env vars and subcommands. Minor gaps keep it below 5: paths are placeholder-templated ("{product-slug}/{owner-repo}"), the HTTPS retry condition "only when that keeps the work moving" is vague, and Profile/Landscape steps defer to reference files. | 4 / 5 |
Workflow Clarity | Sequence is clear — resolve target → route via the intent table → preflight with verification commands (test -d, remote -v, pinned commit) → discovery → ordered fact-gathering → report template → Output Quality Bar checklist — with a recovery loop for SSH failure. Below 5 because several checkpoints (mechanism evidence checks, final checklist details) are delegated to reference files rather than validated inline, and the two routing sections create mild sequencing ambiguity about which governs. | 4 / 5 |
Progressive Disclosure | References are well-signaled, one level deep, purpose-stated at each link (e.g., "Before making mechanism-level claims ... read references/mechanism_evidence.md"), and every linked path and deep anchor resolves to a real file. Not 5 because references/original-app-demo.md exists in the bundle but is never referenced or navigable from SKILL.md. | 4 / 5 |
Total | 15 / 20 Passed |