Content
61%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, highly actionable body with a clear decision rule, ordered tool-selection fallback, and copy-paste CLI commands. The main deductions are repeated tool-path/Beta-feedback blocks and hard-coded expiration dates (conciseness), and the absence of an explicit post-operation validation step in the destructive reset-from-parent flow (workflow clarity).
Suggestions
Deduplicate the tool-path check: state the 'verify CLI with `neon --version`, fall back to MCP' rule once in the Selection order section and remove its repetition in both branch-type Steps lists and both Examples.
Replace the repeated literal `--expires-at 2026-12-15T18:02:16Z` with a placeholder like `--expires-at <iso-timestamp>` plus one relative example, and mention the Beta feedback links once.
Add an explicit verification step after destructive or state-changing operations, e.g. 'After reset, run `neon branches list` / `describe_branch` to confirm the branch now matches the parent before proceeding.'
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient, but there is noticeable duplication and padding: the tool-path check ("verify the CLI with `neon --version`, and fall back to MCP only if the CLI isn't usable") is repeated three times; the Beta feedback links (console.neon.tech modal + discord) appear twice; and the hard-coded timestamp `--expires-at 2026-12-15T18:02:16Z` is repeated in four commands — time-sensitive information placed inline rather than in a deprecated/context section. Explanatory asides like "Real production-like data can expose edge cases your seed or data migration scripts miss..." add tokens Claude does not need. This sits between the 2 and 4 anchors, and the duplication is more than 'minor instances', so 3. | 3 / 5 |
Actionability | Mostly executable, copy-paste-ready guidance: complete `neon branches create` / `neon branches reset` / `neon connection-string` / `neon set-context` commands with flags and placeholders, plus a full runnable neon.ts config example. Minor gaps keep it below 5: the MCP flow is vague ("Use branch tools (for example, `create_branch`)" with no parameter details), the REST API fallback is only a link, and the duplicated literal `--expires-at` date is a stale-prone example rather than a relative placeholder. | 4 / 5 |
Workflow Clarity | Sequencing is strong — a numbered decision rule, a 5-step tool-selection order with `neon --version` / `neon projects list` checkpoints, and per-branch-type step lists. However the destructive reset-from-parent flow has no explicit validation or verification step after the operation (no post-reset check equivalent to the MCP flow's `describe_branch` validation), and the guideline caps destructive-operation workflows without validation at 3. It is above the 2 anchor because pre-flight constraints (only child branches, blocked if the target has children, 24-hour restore window) and a rollback path (`--preserve-under-name`) are clearly documented. | 3 / 5 |
Progressive Disclosure | The body has clear, well-signaled sections and one-level-deep navigation: it defers the Neon overview to the parent `neon` skill ("see the `neon` skill for the full reference") and links external docs per topic rather than nesting references. There are no bundle files, so structure is judged on the body itself. It is not a 5 because sizable sections (the full neon.ts IaC example, CI/CD patterns, the Examples section) are inlined in a ~290-line SKILL.md where a references/ file split would fit the 'appropriately split' top anchor. | 4 / 5 |
Total | 14 / 20 Passed |