CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

finish-feature

Finish a feature — run verification, an adversarial review gate (re-grounded on the original design doc + ticket ACs, not the plan) and a deferral-closure gate, then create the PR, retain learnings, and hand off cleanup to the terminal.

Quality

79%

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 ./devflow-plugin/skills/finish-feature/SKILL.md

The canonical home for this skill is finish-feature in AndreJorgeLopes/devflow

SKILL.md
Quality
Evals
Security

Quality

Content

81%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.

A highly actionable, well-sequenced completion pipeline with pinned deterministic formats, hard validation gates, and explicit feedback loops — exemplary workflow clarity and actionability. The main cost is token overhead from motivational rationale blocks and redundant restatements, plus a long single-file layout that could offload some detail to a reference file.

DimensionReasoningScore

Conciseness

Mostly efficient operational content (lean bash blocks, pinned formats), but includes unnecessary explanation that could be tightened: the ~150-word "Why this step exists" rationale blockquote in Step 4, the philosophical closing of Step 5 ("it is how a correctly-identified-but-blocked deliverable ... ends up missing from the MR with nobody realising"), and "Important" bullets that restate in-step guidance ("Never merge to main", "Always retain learnings"). Not 2 because there is no concept-teaching padding — Claude-known material is not explained — and the commands themselves are lean.

3 / 5

Actionability

Fully executable, copy-paste-ready guidance throughout: `devflow check`, `make flows-check`, `git diff --cached "$BASE"`, the pinned commit regex `^(feat|fix|refactor|chore|test|docs)(\([^)]+\))?: .{1,72}$`, exact PR section headings, exact summary-block template, and deterministic ticket/commit/file-count bash one-liners. Subagent dispatch (verifier/debugger roles) is concretely specified.

5 / 5

Workflow Clarity

A clearly sequenced 9-step pipeline with explicit validation checkpoints and feedback loops: hard stop if checks fail ("Do NOT continue past this step with failing checks"), the "Fail loud" check that an empty diff-stat is not a pass, severity-gated findings requiring fix or explicit user acceptance, and the deferral-closure checklist with done/accepted/pulled-in outcomes. Error-recovery paths (fix the range, extend tests for recovered requirements) are spelled out.

5 / 5

Progressive Disclosure

No bundle files exist (no references/, scripts/, or assets/), and the single SKILL.md is well organized with clear headers (Preamble, Determinism, Steps, Important) and clearly signaled external pointers (~/.claude/lib/determinism/CONTRACT.md, /devflow:review by URL, Hindsight retain). Not 5 because at ~195 lines the long Step 4 review-gate instructions and the summary-block template are candidates for a one-level-deep reference file; not 3 because structure is genuinely good and nothing is buried.

4 / 5

Total

17

/

20

Passed

Description

66%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.

A specific, distinctive description that enumerates the concrete pipeline stages, but it lacks any explicit 'Use when...' trigger guidance, leaving the invocation condition only weakly implied. It also misses natural synonyms for feature completion. Adding a trigger clause and a few natural variations would raise completeness and trigger-term quality.

Suggestions

Add an explicit trigger clause, e.g. "Use when a feature branch is complete and ready to be verified, reviewed, and turned into a PR" — this directly addresses the completeness cap of 3.

Include natural user phrasings/synonyms such as "wrap up", "finalize", or "open a pull request" alongside "Finish a feature" to broaden trigger-term coverage.

Mention the commit step (e.g. "stage and commit with a conventional-commit message") so the enumerated actions fully cover the finish pipeline.

DimensionReasoningScore

Specificity

Quotes several concrete actions — "run verification", "an adversarial review gate (re-grounded on the original design doc + ticket ACs, not the plan)", "a deferral-closure gate", "create the PR, retain learnings, and hand off cleanup to the terminal" — matching the 'several specific actions' anchor. Not 5 because coverage has minor gaps (staging/committing and the final summary are unmentioned); not 3 because it goes well beyond 1-2 actions.

4 / 5

Completeness

The 'what' is explicit and detailed (verification, review gate, deferral-closure gate, PR creation, learnings, cleanup handoff), but there is no 'Use when...' clause or equivalent explicit trigger guidance — "Finish a feature" only weakly implies the when. Per the judging guideline, a missing explicit 'Use when' clause caps completeness at 3; it is not 2 because the 'what' is clear and specific.

3 / 5

Trigger Term Quality

Contains natural user phrases like "Finish a feature", "create the PR", and "verification", giving good keyword coverage. Not 5 because common synonyms and variations ("wrap up", "finalize", "open a pull request", "ship it") are absent; not 3 because multiple natural terms are present, not just one generic keyword.

4 / 5

Distinctiveness Conflict Risk

The composite pipeline (adversarial review gate re-grounded on design doc + ticket ACs, deferral-closure gate, cleanup handoff to the terminal) carves a distinct niche, but "review" and "create the PR" could trigger overlap with generic code-review or PR-creation skills. Not 5 due to that minor overlap risk with closely related skills; not 3 because the gate terminology is quite specific.

4 / 5

Total

15

/

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

AndreJorgeLopes/devflow

Path

devflow-plugin/skills/finish-feature/SKILL.md

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.