Content
65%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.
The body is lean, well-sectioned, and provides concrete commands with realistic examples, but it references script files absent from the bundle and offers no verification step around the destructive replace-text operation. Including the scripts and adding a verify step would raise both actionability and workflow clarity.
Suggestions
Ship the referenced `scripts/auth.py` and `scripts/docs.py` in the skill bundle (or remove the commands) — every documented operation currently fails because the scripts do not exist.
Add a validation step around destructive edits, e.g. "Run `get-text` first to confirm the target text exists, and again after `replace-text` to verify the change."
Trim the Token Management section to one or two lines (where tokens are stored and that they auto-refresh); the per-OS keyring enumeration is unnecessary detail.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and command-driven ("All operations via `scripts/docs.py`. Auto-authenticates on first use if not logged in."), but the per-OS keyring enumeration ("macOS: Keychain", "Windows Credential Locker", "Linux: Secret Service API") and the "Service name" line are details Claude does not need. Minor trimmable content keeps it below the lean 5 anchor. | 4 / 5 |
Actionability | Commands are concrete with realistic arguments (real document ID, full URL, quoted content strings), but every command depends on `scripts/auth.py` and `scripts/docs.py`, which are not present in the skill bundle, so the guidance is not fully executable as shipped. Not 3 because the command forms, arguments, and examples are otherwise precise and copy-paste shaped. | 4 / 5 |
Workflow Clarity | Single-command operations are unambiguous and setup/auth is clearly sequenced, but `replace-text` destructively modifies user documents and no validation step is given (e.g., `get-text` to verify before/after). Per the guidelines, destructive operations without validation cap workflow clarity at 3. Not 2 because the command set and first-time setup flow are clearly laid out. | 3 / 5 |
Progressive Disclosure | Sections are well-organized and the skill is appropriately self-contained for its size, but the only external references (`scripts/auth.py`, `scripts/docs.py`) point to files that do not exist in the bundle (no scripts/ directory), breaking navigation to the referenced material. This fits the 3 anchor (structure present, references not resolvable) rather than 4. | 3 / 5 |
Total | 14 / 20 Passed |