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 well-structured, actionable skill with an explicit validate-and-retry loop and thorough troubleshooting, appropriately split across two genuine reference files. Its main defects are a small amount of trimmable explanation and a dangling reference to a scripts/ directory that is absent from the bundle, which leaves the credential pre-flight branch of Section 3 unexecutable as written.
Suggestions
Ship the scripts/ directory (linear.sh, slack.sh, notion.sh) referenced in Section 3, or restate where those provider scripts should come from — the instruction currently points at files that do not exist in the bundle.
Trim passages that restate what Claude already knows (e.g., the JSON validity rules in Section 5) and the closing scope-justification paragraph in Section 3.
Show the actual `tessl skill new` invocation with its flags (or direct the reader to check its --help) instead of naming the command without arguments.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient, Tessl-specific instruction with little generic padding, but a few passages could be trimmed: "Keep JSON valid: quote every key and string, no trailing commas, no comments" restates JSON basics, and Section 3's closing scope paragraph ("Checking every scheduled skill would require a platform launch-preflight hook and is outside this workflow") is justification Claude does not need to act. Not 3 because the bulk of the content (field rules, cap behavior, apply-time vs validate-time checks) is non-obvious and earns its place. | 4 / 5 |
Actionability | Concrete, executable commands throughout (`tessl schedule --help`, `tessl env view --json --workspace <workspace> <environment>`, `tessl schedule validate`, `tessl schedule list --json --limit 100`) plus a complete JSON example and exact ref formats. Not 5 because Section 3 instructs copying provider scripts "verbatim and unmodified from this skill's `scripts/` directory" while no scripts/ directory exists in the bundle, and `tessl skill new` is named without showing its arguments. Not 3 because the guidance is otherwise copy-paste ready with specific error-recovery steps. | 4 / 5 |
Workflow Clarity | A clearly sequenced seven-phase workflow (Preflight → Gather → Author → Choose key → Write → Validate → Explain next) with an explicit validation checkpoint and feedback loop: "Fix anything it reports and run it again until it passes", followed by a troubleshooting list for schedules that do not appear after push. Preflight checks (file existence, key collisions, current CLI surface) catch errors early. | 5 / 5 |
Progressive Disclosure | The body is a decision workflow with the full field list and shorthand tables correctly split into two real, clearly signaled, one-level-deep references (references/schedules-block.md, references/cron-cadence.md), both of which exist in the bundle. Not 5 because the body also references "this skill's `scripts/` directory" (linear.sh, slack.sh, notion.sh) as the source for credential pre-flight scripts, and that directory does not exist in the bundle — a referenced path that cannot be resolved. | 4 / 5 |
Total | 17 / 20 Passed |