Content
81%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 highly actionable, well-structured reference for the metadata-filter API with genuinely useful gotcha coverage. Its main weakness is redundancy: the quick-reference table, the usage section, and the Guidelines section each restate material already presented, inflating token cost without adding information.
Suggestions
Drop the per-operator subsections and keep only the Quick Reference table (or vice versa) — the operator syntax is currently shown twice, and the edge-case notes can be folded into whichever presentation remains.
Merge "The Tool" and "Using search_notes with Metadata" into a single section; both explain the same call pattern and the tags/status merging rules.
Trim the Guidelines section to only the decision guidance (metadata vs. text search, when to combine) and delete bullets that restate syntax rules from the Filter Syntax section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The Quick Reference table restates every per-operator section above it, "Using search_notes with Metadata" repeats the intro from "The Tool", and "Guidelines" re-explains AND-logic, dot notation, and query-omission already covered earlier. It is efficient reference material with no concept-padding, but three redundant presentations push it below the 4 anchor ("minor instances that could be trimmed"). | 3 / 5 |
Actionability | All examples are executable Python against the real `search_notes` API, covering filter-only queries, text+metadata combinations, convenience shortcuts, tag shorthand, and a realistic worked example with sample frontmatter. Copy-paste ready guidance covering the common cases matches the top anchor. | 5 / 5 |
Workflow Clarity | This is a single-action read-only skill (pass filters to `search_notes`) and the action is unambiguous, with explicit error-avoidance checkpoints: the `$`-prefix silent-failure warning, rejected null-in-operator semantics, and no-hit behavior for non-Markdown files. The simple-skill exception applies; no destructive/batch cap is relevant. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned with clear headers, a summary table, and a coherent flow from syntax to usage to worked example, and there are no bundle files so inline content is appropriate. It falls short of the 5 anchor because the full operator reference and extended example sections could plausibly be split into a separate reference file rather than kept entirely in SKILL.md. | 4 / 5 |
Total | 17 / 20 Passed |