Content
68%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 well-organized references, but it carries notable redundancy (Quick Reference vs Decision Tree, Activation Context vs frontmatter) and its QA workflow lacks concrete validation checkpoints for batch data operations.
Suggestions
Remove the Quick Reference code block (it restates the Decision Tree) or merge it into the tree to cut the main redundancy.
Delete or drastically shrink the Activation Context section — those triggers already live in the frontmatter description and add ~40 lines of duplication.
Add a concrete validation step to the QA workflow, e.g. 'After execution: run `uv run scripts/quick-query.py <out> --describe` and confirm row counts/schema match expectations' so verification is executable rather than implied.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with clean code blocks and no generic concept padding, but the Quick Reference block duplicates the Decision Tree, the Activation Context section (~40 lines) restates triggers already in the frontmatter, and ABSOLUTE RULES overlaps Core Principles — more than minor trimming needed. | 3 / 5 |
Actionability | Copy-paste-ready executable examples for DuckDB direct file queries, Polars lazy evaluation, zero-copy Arrow handoff, and the hybrid pattern, plus exact `uv run --with ...` package commands and a documented CLI — covering the common cases fully. | 5 / 5 |
Workflow Clarity | The Quality Assurance Process (Before/During/After Execution) gives a rough sequence, but validation is implicit ('Confirm output matches expectations', 'have fallback strategy ready') with no concrete verify command; per the batch-operation cap, vague validation caps this at 3. | 3 / 5 |
Progressive Disclosure | Clear overview with well-signaled one-level-deep references (uv-setup.md, performance-benchmarks.md, integration-patterns.md, execution-templates.md, common-scenarios.md) that are real files, but some content that could live in references (the inline Activation Context triggers, the redundant Quick Reference) is kept inline. | 4 / 5 |
Total | 15 / 20 Passed |