CtrlK
BlogDocsLog inGet started
Tessl Logo

skill-staged-review

Use when a PR or feature needs both specification and code-quality review

56

Quality

62%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./skills/skill-staged-review/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

63%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

The body delivers a well-sequenced two-stage workflow with executable bash, explicit gates, fallbacks, and output templates — highly actionable and clearly ordered. Its weaknesses are padding (mandatory-contract preamble, host notes, Bottom Line section) and a fully monolithic layout with no bundle files despite referencing one. Trimming the non-essential prose and moving long scripts to script files would improve both efficiency and structure.

Suggestions

Trim non-essential sections (the 'Execution Contract' preamble, host-adapter note, and 'The Bottom Line') or compress them to one or two lines each.

Define ${COMBINED_REPORT} in the posting script (e.g., from the unified report template output) and add an explicit wait/collect step for the backgrounded external provider reviews.

Extract the stub-detection bash block and the PR-posting script into a scripts/ bundle and reference them by path, replacing the inline 'codex-host-adapter.md' reference with a file that actually exists in the bundle.

DimensionReasoningScore

Conciseness

The core is dense, executable bash and status tables, but several sections add non-essential tokens: the 'Execution Contract (MANDATORY)' preamble, the host-adapter note, 'The Bottom Line' section repeating the intro, and the multi-LLM rationale prose ('A Claude-only review pipeline misses what external models catch...'). This matches 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than 4, where such padding would be only minor.

3 / 5

Actionability

Concrete, mostly executable bash for loading the intent contract, stub detection, provider dispatch, and PR posting, plus markdown output templates for each stage. Falls short of anchor 5 on small gaps: ${COMBINED_REPORT} is used but never defined in the posting script, and the 'Git not available -> use file listing' fallback gives no command.

4 / 5

Workflow Clarity

Clear staged sequence with an explicit Stage 1 gate table (PASS/FAIL -> proceed, ask user, or stop), per-criterion and per-boundary status marking, and an error-handling table covering five failure modes. Not 5: external reviews are spawned with '&' but no wait/collect mechanism is shown, and the stub-detection results are reported without a required follow-up action per issue.

4 / 5

Progressive Disclosure

The body has good section headers (two stages, numbered steps, gate/error tables), but it is a fully monolithic ~330-line file with no bundle at all: content that should be in separate files is inlined (the ~55-line stub-detection script, the PR-posting section), and the referenced 'skills/blocks/codex-host-adapter.md' does not exist in this bundle. This matches anchor 3 — structure present but content that should be separate is inline; not 2 because headers and navigation are clear, not 4 because there are no well-signaled reference files at all.

3 / 5

Total

14

/

20

Passed

Description

61%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

The description has a clear, natural trigger clause with good domain keywords, but it omits any statement of what the skill does, forcing the reader to infer capabilities from the when-clause. It is concise and non-generic, sitting just above the midpoint of quality. Adding a concrete 'what' statement and a few more synonyms would lift it substantially.

Suggestions

Add an explicit 'what' statement, e.g., "Runs a two-stage review pipeline that validates spec compliance first, then code quality" before the Use-when clause.

Include additional natural trigger synonyms such as 'pull request', 'code review', 'spec compliance', and 'before merge/ship' to broaden trigger coverage.

State the gating behavior (Stage 1 must pass before Stage 2) as a distinguishing capability to reduce overlap with generic PR code-review skills.

DimensionReasoningScore

Specificity

The description names the domain ("a PR or feature") and two concrete review activities ("specification and code-quality review"), but never states what the skill actually does (e.g., runs a two-stage review pipeline). This matches anchor 3 — domain plus 1-2 concrete actions without comprehensive coverage — rather than 4, which requires several specific listed actions.

3 / 5

Completeness

The 'when' is explicit and specific ("Use when a PR or feature needs both specification and code-quality review"), but the 'what' is absent as a standalone statement — only weakly implied by the activities inside the trigger clause. Not score 2 because the when-clause is far more specific than the vague 'Use when working with documents' example; not 4-5 because no clear 'what' is stated.

3 / 5

Trigger Term Quality

"PR", "feature", "specification", and "code-quality review" are natural terms a user would say when needing this skill. It falls short of anchor 5 because common variations are missing, such as "pull request", "code review", "spec compliance", or "merge readiness".

4 / 5

Distinctiveness Conflict Risk

The requirement for "both specification and code-quality review" carves a distinct niche versus single-purpose code-review skills, but there is minor overlap risk with closely related generic PR-review skills that also mention specification and quality. Not 5, since the description lacks distinctive trigger phrases that fully separate it from those skills.

4 / 5

Total

14

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
nyldn/claude-octopus
Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.