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.
A well-structured, action-oriented skill body with concrete executable checks and clear report formats. Its main weaknesses are a correctness bug in the build-status check, a grep that lists all tests rather than failed ones, and some redundant sections.
Suggestions
Fix the build-status check: `if [ $? -eq 0 ]` evaluates the `[ -f "$LOG" ]` test, not the build result, so build status always reports PASS when a log exists — capture the build exit code in the log or echo a status line into it.
In the test-failure branch, filter to failed tests (e.g. parse entries where numFailedTests-related fields or failed statuses appear) instead of `grep '"fullName"'`, which lists every test.
Trim the redundant Quick reference table and Usage section, and verify the `../../commands/*.md` suite links resolve from the skill's installed location.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and code-first with no explanations of concepts Claude already knows, but the Quick reference table and the Usage section largely duplicate the detailed check sections and report format, fitting the anchor for efficient content with minor instances that could be trimmed. | 4 / 5 |
Actionability | Mostly executable, copy-paste-ready bash for detection, log parsing, and re-running checks, but the build-status branch tests `[ $? -eq 0 ]` on the preceding `[ -f "$LOG" ]` result rather than the build outcome, and the test-failure branch greps all '"fullName"' entries instead of only failed tests — concrete code with notable gaps rather than fully correct. | 4 / 5 |
Workflow Clarity | The detect → read logs → parse → report sequence is clear and each step has executable code, a worktree guard, and failure branches with actionable next steps, but there are minor validation gaps such as no handling of stale or partially written logs from still-running tasks. | 4 / 5 |
Progressive Disclosure | Content is well-organized into clearly signaled sections with nothing inlined that belongs in a separate file, but the suite cross-links ([../../commands/git-worktree.md] etc.) point outside the bundle and do not resolve, leaving minor navigation gaps. | 4 / 5 |
Total | 16 / 20 Passed |