Content
71%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 highly actionable with executable code and concrete gotchas, and its decision/workflow guidance is strong with write-validation present. Its weaknesses are repeated gotchas that bloat the token budget and progressive-disclosure references that point to non-existent files.
Suggestions
Ship the referenced bundle files (crud-operations.md, complex-queries.md, pagination.md, aggregation.md, realtime.md, geolocation.md, security-rules.md) or remove the dead 'Quick routing' links so navigation resolves.
Dedupe the repeated gotchas — state result._id and the CMS CUSTOM rule once (e.g. in Working rules) and reference them from elsewhere instead of restating them.
Consolidate write-validation into a single explicit checkpointed write workflow so validation steps are not scattered across Common mistakes and Working rules.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and free of generic concept explanations, but the same gotchas are repeated across sections (result._id appears in Common mistakes, Working rules #5, and Quick examples; the CMS CUSTOM rule appears in both Common mistakes and Working rules #3), so it could be tightened; not a 4 due to this redundancy. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready code for init, query, create, pagination, field selection, and error handling, plus concrete patterns (e.g. the CMS rule expression 'get(\'database.user_roles.\' + auth.uid).role == \'admin\' || doc.authorId == auth.uid') covering common cases. | 5 / 5 |
Workflow Clarity | Clear Activation Contract with sequenced 'Use this first when / Read before writing code if / Then also read' guidance, a Minimal checklist, and write-validation steps (check updated/deleted and code/message, get() round-trip after writes); not a 5 because validation is scattered across sections rather than presented as one clean linear sequence with explicit checkpoints. | 4 / 5 |
Progressive Disclosure | Good in-file structure with a dedicated 'Quick routing' section signaling one-level-deep references (crud-operations.md, complex-queries.md, etc.), but none of the referenced files (./crud-operations.md, ./security-rules.md, and the sibling ../auth-web-cloudbase/SKILL.md etc.) actually exist in the bundle, so the navigation is broken. | 3 / 5 |
Total | 15 / 20 Passed |