Content
85%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 high-quality, highly actionable body with strong sequencing and validation checkpoints. Its main weakness is progressive disclosure: several references are broken or orphaned, leaving navigation unreliable.
Suggestions
Resolve the `rls-reference` citations: either create references/rls-reference.md and link it as a path, or rename the citations to an existing file, so every backtick reference points to a real bundle file.
Either add references/other-engines.md (referenced in the Limitations/References section) or remove that citation.
Surface the existing bundle files models.md and test-surfaces.md from the body (link them where their content is relevant) so no provided reference is orphaned.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with high-signal material (inventory tables, ten concrete test blocks, SQL, a full CI YAML) and avoids explaining generic concepts Claude already knows; minor redundancy between the per-test commentary and the Anti-patterns table and a slightly repetitive References section keep it just short of lean. | 4 / 5 |
Actionability | Fully executable guidance throughout — copy-paste Python tests for every named pattern, SQL 'Postgres-RLS-direct' blocks with concrete SET LOCAL ROLE usage, and a complete GitHub Actions workflow that creates a non-superuser role and runs pytest under it — covering the common cases specifically. | 5 / 5 |
Workflow Clarity | Clear sequenced workflow (Planning -> Step 1 runner/role -> Step 2 battery -> Step 3 CI -> Step 4 diagnose) with explicit validation checkpoints (the 'Expect: rolsuper=f, rolbypassrls=f' and '-- expect 0' assertions) and a Step 4 feedback loop (reproduce, check policy, write minimal regression test before fixing), which satisfies the validation requirement for this DB/CI-gate skill. | 5 / 5 |
Progressive Disclosure | Sectioning is reasonable and the three referenced references/*.md files (isolation-models, attack-patterns, framework-skeletons) exist, but the body repeatedly cites `rls-reference` as a bare non-resolvable name and cites references/other-engines.md which is absent from the bundle, while bundle files models.md and test-surfaces.md are never referenced — broken/orphaned references undermine clear navigation. | 3 / 5 |
Total | 17 / 20 Passed |