Content
88%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-engineered operational skill: executable commands throughout, a safe workflow with dry-run and confirmation gates for every write path, and real one-level-deep reference files wired in at point-of-need. The main costs are sheer body length — detail for several subsystems is inlined that could be split into references — and a few prose sections that could be tightened.
Suggestions
Move the "Selected global settings keys" table and the Channels/Schedules operational detail into short reference files (linked like model-settings.md is), keeping SKILL.md as a routing overview; this would lift both conciseness and progressive disclosure.
Trim the billing-path discussion to a decision rule plus one example (the 'list candidates and ask before switching' rule is the actionable core; the rest restates it).
Compress "Guardrails are not security boundaries" into the two enforceable rules (never target another agent without explicit direction; recover out of band) since the surrounding text repeats them.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with product-specific operational detail (endpoint routing, base-URL resolution rules, permission-mode mappings) that Claude could not know, and it avoids generic concept explanations. A few sections could still be tightened — e.g. the multi-paragraph billing-route discussion and the "Guardrails are not security boundaries" prose carry some redundancy — placing it at 'efficient; minor instances that could be trimmed' rather than fully lean. | 4 / 5 |
Actionability | Nearly every section ends in copy-paste-ready commands with real flags: `npx tsx <SKILL_DIR>/scripts/update-agent-settings.ts --target conversation --model "openai/gpt-5.2" --dry-run`, `letta usage`, `letta secret set GITHUB_TOKEN --env GITHUB_TOKEN`, a complete JSON permission-rule example, and a model-settings JSON heredoc. Concrete command output labels (`offline_partial_patch`, `effective_merged_patch`) cover interpretation of the common cases. | 5 / 5 |
Workflow Clarity | The "Safe workflow" section gives an explicit six-step sequence (identify scope → inspect and save rollback patch → dry run → smallest change → verify effective state → report) with validation checkpoints built in, and it is reinforced in-section: dry-run modes for every patch script, `--show` for verification, confirmation gates for the two destructive operations (`--confirm-compaction-prompt`, `--confirm-system-replacement`), and labeled dry-run outputs forming a feedback loop. Batch/destructive operations all have validation, so the cap does not apply. | 5 / 5 |
Progressive Disclosure | The bundle structure is solid: all three referenced files exist (api-patch-examples.md, model-settings.md, compaction-prompt-patterns.md), are linked inline at point-of-need ("Read [references/model-settings.md] before changing reasoning"), are one level deep with no nested links, and are catalogued in a References section plus a helper-scripts table. However, the ~450-line body still inlines substantial per-subsystem detail (the global settings keys table, channels/secrets/schedules specifics) that could live in references, keeping it at 'most content appropriately placed; minor organization gaps'. | 4 / 5 |
Total | 18 / 20 Passed |