Content
78%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, highly actionable body with an excellent progressive-disclosure split to references/rule.md. The main cost is redundancy: the mixed-content rationale is repeated across the intro, Quick Reference, and Explain sections, and the workflow lacks an explicit verify-then-retry loop for the batch search-and-replace step.
Suggestions
Consolidate the mixed-content explanation into one place — drop the duplicated warning language from the intro and Quick Reference, and keep only the domain-specific details (referrer ranking-signal loss, redirect cost) in Explain.
Add a feedback loop after the verify step: "If the grep still returns matches, fix them and re-run; then confirm the HTTP→HTTPS server redirect responds with 301" — this would lift workflow clarity to 5.
Move or compress the Code Review section's overlap with Check (both enumerate the same attribute scanning) so each section has a single distinct job.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The mixed-content consequence is explained three times — in the intro ("creates a mixed content situation that browsers warn users about or block entirely"), in the Quick Reference bullets, and again in the Explain section — concepts Claude already knows. The bulk (Check/Fix/Code Review) is lean, so this is removable padding rather than pervasive verbosity, matching the 3 anchor. | 3 / 5 |
Actionability | Fully executable guidance throughout: a copy-paste grep command (`grep -r 'href="http://' ./templates/`), concrete before/after transformations (`http://yourdomain.com/path` → `/path`), specific attribute targets (`<a href>`, `<img src>`, `<script src>`, `<link href>`), and framework-specific call sites (`fetch()`, `axios`, router navigation). | 5 / 5 |
Workflow Clarity | The Fix section is a clear 7-step sequence (audit → internal → external → resources → CMS replace → server redirect → verify) with a verification step and a mid-flow checkpoint (check the destination supports HTTPS before updating). It falls short of 5 because there is no feedback loop — nothing says what to do when the verify grep still finds matches. | 4 / 5 |
Progressive Disclosure | A ~40-line overview with clearly signaled, one-level-deep references: "For full implementation details, code examples, and framework-specific guidance, see `references/rule.md`" — verified to be a real 109-line file with no further nested references. The split (summary in SKILL.md, details in the reference) matches the 5 anchor exactly. | 5 / 5 |
Total | 17 / 20 Passed |