Content
65%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, information-dense reference: executable Terra calls, realistic payloads with exact field names, and useful operational guidance (overwrite semantics, provider history limits, async webhook behavior). Its weaknesses are structural — it is a self-contained monolith with sample responses and DB schema inlined, and its batch/backfill workflows lack validation checkpoints. The Quick Start also embeds what appear to be real dev credentials rather than placeholders.
Suggestions
Add validation checkpoints to the batch workflows: after a >28-day backfill (which returns {"status": "processing"}), confirm webhook delivery and verify expected record counts before declaring the backfill complete, rather than ending the flow with no verification.
Move the per-type sample JSON responses and the SQL DDL into a references/ file (e.g., references/response-schemas.md and references/db-schema.md), keeping SKILL.md as a concise overview with clearly signaled one-level-deep links.
Trim the seven repeated "def get_*" wrapper blocks (keep the client call and field summary) and replace the hard-coded dev_id/api_key in Quick Start with placeholders pointing to the terra-auth skill.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense Terra-specific material with no padding explaining concepts Claude already knows, but the seven repeated "def get_*" wrapper blocks each duplicate the client call shown inside them — trimmable boilerplate that keeps this below anchor 5. | 4 / 5 |
Actionability | The Quick Start and every "client.activity.get(user_id=..., start_date=..., end_date=...)" call are copy-paste ready with realistic sample responses showing exact field names. Minor gaps keep it below anchor 5: "handle_data_update" depends on an undefined "db" object (pseudocode), and "get_all_user_data" is declared "async" but makes synchronous calls. | 4 / 5 |
Workflow Clarity | The backfill branch "if days_back > 28 ... to_webhook=True" is a clear decision point and the overwrite vs. insert-or-ignore update strategy is stated, but batch operations (historical backfill, bulk upserts) include no validation or verification steps — no webhook-delivery confirmation, no check that expected data arrived — which caps this dimension at 3 per the batch-operations guideline. | 3 / 5 |
Progressive Disclosure | Section headers are well organized, but roughly 300 lines of inline sample JSON responses plus the SQL DDL are reference material that belongs in separate bundle files; no references/ directory exists and SKILL.md carries everything, matching anchor 3's 'content that should be separate is inline'. | 3 / 5 |
Total | 14 / 20 Passed |