Content
57%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 dense, information-rich reference that is strong on concrete Terra-specific details (data schemas, overwrite semantics, provider limits) and largely executable code. Its weaknesses are systematic redundancy between docstrings and sample JSON, a monolithic single-file layout with no external references, and batch workflows (backfill, bulk fetch, db writes) that lack validation and error-handling checkpoints.
Suggestions
Move the six per-type sample JSON payloads, provider historical-limits table, and SQL schema into references/ files (e.g. references/samples.md, references/providers.md) and link them from a lean SKILL.md overview, reducing inlined bulk and enabling progressive disclosure.
Consolidate the seven near-identical client.<type>.get sections into one parameterized call pattern with a short per-type field table, dropping the duplicated docstring-plus-sample-JSON redundancy.
Add validation checkpoints to the batch workflows: check response status/pagination in get_all_user_data, confirm webhook delivery expectations in backfill_user_data for the >28-day path, and verify upsert success in handle_data_update.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly high-value, non-obvious material (Terra schemas, provider history limits, overwrite semantics), but each operation repeats a docstring field list followed by a sample JSON showing the same fields, the Data Types Overview table restates what each section repeats, and the 7 near-identical client.<type>.get blocks could be consolidated. This fits 'mostly efficient but includes some unnecessary explanation or could be tightened' — not 2 (nothing explains concepts Claude already knows; there is no padding like the bad examples), and not 4 (the redundancy is systematic, not minor). | 3 / 5 |
Actionability | Concrete executable code for every operation plus sample JSON responses, e.g. "client.activity.get(user_id=..., start_date=..., end_date=..., to_webhook=...)" and full Quick Start. Minor gaps keep it from 5: the Quick Start uses os.environ without importing os, handle_data_update references an undefined db object, and get_all_user_data is marked async while making synchronous calls. It is clearly above 3 (code is real, not pseudocode, and covers the common cases). | 4 / 5 |
Workflow Clarity | There is a coherent sequence (Quick Start → per-type retrieval → bulk/backfill → update strategy → storage schema), but the batch operations lack validation checkpoints: get_all_user_data and backfill_user_data never check responses for errors, pagination, or webhook confirmation, and handle_data_update performs db writes with no verification. Per the guideline capping batch operations without validation, this sits at 3 ('sequence present but checkpoints missing or implicit') rather than 4. | 3 / 5 |
Progressive Disclosure | The single SKILL.md (~550 lines, 12KB) inlines six per-type sample JSON payloads, provider limit tables, SQL schemas, and write examples — content that clearly belongs in reference files — with no references/, scripts/, or assets/ directories at all. Section headers do give it real structure ('Some structure... content that should be separate is inline'), so it scores 3 rather than 2, but it is far from the well-signaled one-level-deep reference structure of a 5. | 3 / 5 |
Total | 13 / 20 Passed |