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 content is highly actionable, with executable SQL covering the common query patterns for both tables, and it is well organized for a single-file skill. Its main weaknesses are a duplicated stitching explanation that wastes tokens and a workflow that is implied by scattered sections rather than presented as an explicit sequence.
Suggestions
Remove the "Understanding stitching" section or reduce it to a one-line cross-reference, since it restates the "Profile change types" section almost verbatim.
Add a short numbered workflow at the top (1. Ask for the parent segment ID if not given, 2. Build the database name cdp_audience_<segment_id>_rt, 3. Apply the default TD_INTERVAL time range, 4. Run the queries) so the sequence is explicit.
Consider moving the full column-by-column schemas to a references/ file (e.g. SCHEMA.md) and keeping SKILL.md to an overview plus the query patterns.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient — the domain background, schemas, and change-type semantics are information Claude would not already know — but the "Understanding stitching" section repeats the "Profile change types" section nearly verbatim (profile_added, profile_updated_by_stitching, merged and evicted rows explained twice), which is unnecessary padding that could be tightened. | 3 / 5 |
Actionability | Provides fully executable, copy-paste-ready Trino/Presto queries covering the common cases: the eviction-count aggregation, the 8-hour log sample, the UNNEST/json_extract validation-failure query, and the cross-table td_rt_tracking_id trace. Only the kv_map snippet runs on literal sample data, which is a reasonable demonstration pattern, so this matches the top anchor rather than the 'minor gaps' anchor below it. | 5 / 5 |
Workflow Clarity | The workflow pieces exist (must ask for the parent segment ID, database name format cdp_audience_<id>_rt, default TD_INTERVAL '-1d/now' when no time range given) but the sequence is never enumerated — it is scattered across Requirements, Database, and Query tips sections with implicit checkpoints, matching the 'sequence present but checkpoints missing or implicit' anchor. All queries are read-only, so the destructive-operation cap does not apply. | 3 / 5 |
Progressive Disclosure | Good structure with well-organized sections (per-table schemas, change types, query tips, common queries) and no nested references; the minor gap is that ~200 lines of schema and change-type detail could arguably live in a separate reference file, keeping it just below the top anchor. | 4 / 5 |
Total | 15 / 20 Passed |