Content
75%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 thorough, highly actionable overview skill with concrete commands and code throughout and well-sequenced workflows that include a fail-fast validation checkpoint. Its main weakness is conciseness — repeated restatements of the env-pull behavior and dense inline config detail could be tightened or moved to referenced files.
Suggestions
Consolidate the repeated 'link and checkout pull env by default' explanation into a single statement; it currently appears in three sections (Branch-First Dev Flow, the code-block comments, and the Infrastructure as Code section).
Tighten the 'Opting out of local env vars' block (174-185), which lists four overlapping runtime-injection mechanisms; consider promoting the lower-level `fetchEnv`/`neon-env run` detail to a referenced file.
Add an explicit verify-before-proceeding step for destructive operations such as `neon set-context` and `neon config apply`/`neon deploy` to lift workflow_clarity toward a 5.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and free of basic-concept padding, but the fact that 'link' and 'checkout' pull env by default is restated across three sections (154, 159-160, 235) and the env-injection/opt-out block (174-185) is dense enough to tighten, fitting 'mostly efficient but could be tightened'. | 3 / 5 |
Actionability | Abundant copy-paste-ready commands (`neon link`, `neon checkout <branch>`, `neon config apply`, `npx -y neon init`) and complete TypeScript `defineConfig`/`parseEnv` examples cover the common cases, matching the fully-executable anchor. | 5 / 5 |
Workflow Clarity | Multi-step flows (Getting Started, Branch-First, config status/plan/apply) are clearly sequenced with a real fail-fast validation checkpoint on env-pull, but destructive operations like `set-context` ('replaces the file's contents entirely') and `neon config apply` lack an explicit validate-before-proceeding step, a minor validation gap. | 4 / 5 |
Progressive Disclosure | As an index/router skill it delegates one level deep to clearly-signaled sibling skills (`neon-postgres`, `neon-functions`, `neon-object-storage`, `neon-ai-gateway`) and external docs, with good section structure; the dense inline neon.ts/parseEnv detail (50+ lines each) could arguably live in referenced files, a minor organization gap. | 4 / 5 |
Total | 16 / 20 Passed |