Content
71%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 highly actionable, well-structured reference: nearly every line is an executable Harbor command, the setup and troubleshooting workflows carry explicit verification steps, and agent-specific hazards (the log-tailing trap) are prominently flagged. Its weaknesses are duplication — config tables and several workflows appear twice — and the absence of any progressive disclosure, with reference-grade per-service material inlined in a single very long file instead of being split into one-level-deep reference files.
Suggestions
Deduplicate the duplicated material: the 'Environment Variable Quick Reference' repeats the per-service config tables verbatim, and the web-search, code-execution, and tunnel procedures appear in both 'Agent Decision Trees' and 'Common Workflows' — keep one canonical location and cross-reference it.
Move the per-service reference material (CLI listings, config tables, and env var tables for each of Ollama, llama.cpp, vLLM, Open WebUI, SearXNG, Open Terminal) into one-level-deep reference files (e.g. references/ollama.md), leaving SKILL.md as a concise overview with the decision trees, setup workflow, and troubleshooting playbooks.
Add a validation gate before the destructive sandbox reset ('rm -rf services/openterminal/data'): confirm the openterminal container is down, back up anything needed from the workspace, and verify the service is healthy after the reset.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — nearly everything is Harbor-specific commands rather than concepts Claude already knows — but there is real duplication that could be tightened: the per-service config tables (Ollama, llama.cpp, WebUI, SearXNG, Open Terminal) are repeated verbatim in the 'Environment Variable Quick Reference', and the web-search, code-execution, and tunnel workflows appear in both 'Agent Decision Trees' and 'Common Workflows'. This is more than the minor over-explanation of anchor 4. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout, covering the common cases: 'harbor pull qwen3:4b && harbor up', 'harbor vllm args --max-model-len 4096 --enforce-eager', 'docker logs harbor.<service>', working curl examples for the router API, and complete numbered troubleshooting commands. No pseudocode or vague direction. | 5 / 5 |
Workflow Clarity | Clear sequences with most checkpoints present: the 5-step setup workflow includes explicit verification ('harbor ps → confirm services healthy', 'harbor doctor', 'Wait for "Application startup complete"'), and the GPU/OOM troubleshooting playbooks are numbered with verification commands. Not 5 because the sandbox reset ('rm -rf services/openterminal/data') is a destructive step with only log/health checks before it and no real validation gate. | 4 / 5 |
Progressive Disclosure | Section structure is good — clear headers, per-service sections, quick-reference tables — but there are no reference files at all: reference-grade material (per-service CLI listings, config tables, and environment variable tables for 8+ services) is inlined in a single ~1300-line SKILL.md. This fits anchor 3 (content that should be separate is inline, though well organized) rather than anchor 2's unstructured blob. | 3 / 5 |
Total | 15 / 20 Passed |