Content
96%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 lean, highly actionable skill body: exact URL templates, a copy-paste-ready command with complete argument handling, explicit stop/ask validation, and browser-failure recovery. The only structural weakness is that its core logic and shared instructions live in files outside the provided bundle, making the delegation unverifiable from this directory.
Suggestions
Include scripts/open-wrap-url.js in the skill's scripts/ directory (or verify its path resolves from the plugin root) so the delegated core logic is present and reviewable alongside SKILL.md.
Clarify the location and role of ../../shared/shared-instructions.md — a path escaping the skill directory is easy to break when the skill is moved or published.
Consider noting in the Notes section what the script does on failure beyond opening the URL (e.g., exit codes or messages), so fallback behavior is fully specified without reading the script.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and operational throughout — URL templates, an argument spec, one command, and fallback rules — with no explanation of concepts Claude already knows. Re-reading anchor 4 ('minor instances of over-explanation'), only the trivially redundant step summary on the 'Workflow' line could be trimmed, which does not pull it below 5. | 5 / 5 |
Actionability | Guidance is fully executable: the copy-paste-ready command 'node scripts/open-wrap-url.js --app-id "<app-id>" --env-id "<environment-id>" [--env "Prod|Test"]' plus exact URL templates cover the common Prod and Test cases. There are no missing key details that would warrant a 4. | 5 / 5 |
Workflow Clarity | The simple two-step workflow has explicit checkpoints and feedback loops: 'If either value is missing in $ARGUMENTS, ask once for the missing value(s). If still missing, STOP' and 'If the browser auto-open fails, still print the URL and tell the user to open it manually'. The operation is read-only, so the destructive/batch validation cap does not apply. | 5 / 5 |
Progressive Disclosure | Sections are well organized and references are one level deep and clearly signaled ('This skill delegates URL construction... to scripts/open-wrap-url.js', the shared-instructions.md link). However, the delegated script containing the core logic, '../../shared/shared-instructions.md', and the plugin check script all lie outside the provided bundle (no references/, scripts/, or assets/ directories exist), so the structure cannot fully verify as the clean self-contained 5. | 4 / 5 |
Total | 19 / 20 Passed |