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.
The content is concrete and actionable with mostly efficient code, but its destructive and batch operations lack explicit validation checkpoints and it is a monolithic file with no progressive disclosure or shipped supporting bundle. Tightening compliance-flow verification and splitting reference material into files would most improve it.
Suggestions
Add explicit verification to destructive/batch workflows: for GDPR deletion, confirm the lead no longer appears via search-by-contact after deletion and surface failures; for bulk lead/block-list operations, require a confirm or dry-run step before applying.
Split the API endpoint table and error catalog into a reference file (e.g. references/api.md) referenced one level deep, so the SKILL.md body stays a lean overview.
Resolve the undefined client dependency: either ship the ./src/instantly/client bundle (with client.leads.list/delete) or standardize all examples on the client.request pattern so code is copy-paste self-contained.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient code with lean comments and assumes Claude's competence, but includes some narration (console.log compliance logging) and commented example cruft in seedBlockList that could be trimmed. | 4 / 5 |
Actionability | Provides concrete, executable TypeScript functions with real endpoints and field names covering common cases, but references an undefined InstantlyClient/client.leads API and an absent ./src/instantly/client bundle, so it is not fully self-contained. | 4 / 5 |
Workflow Clarity | A clear five-step sequence with some import-time validation exists, but destructive and batch operations (GDPR deletion across campaigns, bulk lead delete, bulk block-list create) lack explicit verification checkpoints and confirmation, capping workflow clarity at 3 per the destructive-ops rule. | 3 / 5 |
Progressive Disclosure | Section structure is present and coherent, but everything (full API endpoint table, error catalog, all code) is inlined in a 336-line file with no separate reference files, and an inline ./src/instantly/client reference points to a non-existent bundle. | 3 / 5 |
Total | 14 / 20 Passed |