Content
68%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 concise, largely actionable skill with a clear example workflow and a properly-sized structure whose referenced script is real. The main weakness is the absence of any validation step around the batch push operation (confirm-to-sender is prescribed without verifying the push), plus an unexplained `hermes-sync` command that does not match the bundled `hermes-sync.py` path.
Suggestions
Add a verification checkpoint after push — e.g., check the script's exit status or list the pushed files (or a --dry-run flag) — before confirming to the sender, so the batch write has a feedback loop.
Reconcile the `hermes-sync` invocation with the bundled `scripts/hermes-sync.py`: either reference its actual path or state that it is installed on PATH.
Trim the redundancy between the 'When to use' bullets and the example workflow, which repeats the same hermes-sync and push-shared.sh commands a second time.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and command-driven with no padding that explains concepts Claude already knows. Minor over-explanation remains — the 'When to use' bullets and example workflow restate commands already shown above — keeping it just below the every-token-earns-its-place anchor 5. | 4 / 5 |
Actionability | Guidance is mostly executable: `hermes-sync`, `cat shared/tasks/{task-id}/spec.md`, and `bash ./skills/file-sync/scripts/push-shared.sh tasks/{task-id}/ --exclude "spec.md"` with a concrete worked example (st-01) and a real script in the bundle. The minor gap is inconsistent invocation — push-shared.sh gets a full relative path while `hermes-sync` appears as a bare command despite the bundle containing `scripts/hermes-sync.py`, so the reader cannot tell how to run it from the skill. | 4 / 5 |
Workflow Clarity | The example workflow gives a clear sequence (sync → read spec → work → push → confirm), but pushing a directory to MinIO is a batch write with no verification checkpoint — the skill says 'Always confirm to the sender after push completes' without ever checking that the push succeeded or what was uploaded. Per the rubric's cap, a batch operation without validation cannot score above 3. | 3 / 5 |
Progressive Disclosure | Content is well organized into focused sections with the only external reference (`scripts/push-shared.sh`) being a real one-level-deep bundle file, appropriate for a skill of this size. It falls short of 5 because `scripts/hermes-sync.py` exists in the bundle yet is never linked from the body — the `hermes-sync` command's provenance is undocumented. | 4 / 5 |
Total | 15 / 20 Passed |