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

81%

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

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 strong validation gates and feedback loops throughout — the workflow engineering is excellent. Its main weakness is token efficiency: repeated rationale passages and a redundant "Important" section could be trimmed or offloaded to a reference file.

Suggestions

Trim the motivational rationale (the Step 4 "Why this step exists" blockquote and the Step 5 failure-mode paragraph) to one line each — the operational rules already carry the intent.

Deduplicate the "Important" section, which restates rules already pinned in Steps 2, 4, and 8; keep only the rules not stated elsewhere.

Consider moving the detailed Step 4 sub-agent dispatch spec to a reference file (e.g. references/review-gate.md) to shrink SKILL.md toward an overview-plus-commands document.

DimensionReasoningScore

Conciseness

The body assumes Claude's competence (no explanations of git/PR basics, commands given bare) but carries several multi-sentence rationale passages — the "Why this step exists" blockquote, the deferral failure-mode paragraph, and an "Important" section that largely restates rules already pinned in the steps. Mostly efficient but noticeably over-explained in these spots, matching anchor 3 rather than the minor-trim anchor 4.

3 / 5

Actionability

Commands are copy-paste ready throughout: `devflow check`, `make flows-check`, the merge-base staging/diff recipe, the pinned commit-subject regex, exact PR section headings, the deterministic greps with fallbacks, and the exact summary block. Specific failure handling (eval mode, checks-unavailable fallback, empty-diff "fail loud") covers the common cases, matching the fully-executable anchor.

5 / 5

Workflow Clarity

Nine clearly sequenced steps with explicit validation checkpoints (hard-stop branch guard, checks-fail stop, empty-diff fail-loud, severity gating via AskUserQuestion, commit-message approval) and feedback loops (fix-and-revalidate, add a locking test per fixed alignment gap), plus a final checklist-style block. This matches the anchor for clear sequence with explicit validation, error recovery, and checklists.

5 / 5

Progressive Disclosure

No bundle files exist, so everything lives in a single ~200-line SKILL.md; sections and numbered steps are well organized and easy to navigate, which earns good-structure anchor 4. It falls short of anchor 5 because rationale prose and the detailed Step 4 gate spec stay inline where a one-level-deep reference file could carry them, but the structure is stronger than the disorganized anchor 3.

4 / 5

Total

17

/

20

Passed

Description

71%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, action-dense description with a well-defined niche, weakened by the absence of any explicit "Use when..." trigger clause, which caps its completeness and leaves invocation conditions only implied. Adding an explicit trigger sentence would lift it into the top tier.

Suggestions

Append an explicit trigger clause, e.g. "Use when a feature's implementation is complete and it's time to verify, review, and open the PR — not for mid-implementation review."

Include natural user phrasings like "wrap up the feature", "done implementing", or "open the pull request" alongside "Finish a feature" to broaden trigger coverage.

Trim the parenthetical "(re-grounded on the original design doc + ticket ACs, not the plan)" to "against the original design doc and ticket ACs" to reduce jargon while keeping distinctiveness.

DimensionReasoningScore

Specificity

The description lists six concrete actions — "run verification", "adversarial review gate", "deferral-closure gate", "create the PR", "retain learnings", "hand off cleanup" — giving comprehensive coverage of the skill's capabilities with no generic filler. It matches the anchor for multiple specific concrete actions; a 4 would require minor coverage gaps that are not present.

5 / 5

Completeness

The "what" is clear and detailed, but there is no "Use when..." clause or equivalent explicit trigger guidance — the "when" is only weakly implied by the opening "Finish a feature", which caps completeness at 3 per the rubric guideline. A 4 would require an explicit, if imperfect, statement of when to invoke the skill.

3 / 5

Trigger Term Quality

It includes natural phrases users would say such as "Finish a feature", "verification", "review", "create the PR", and "cleanup". Coverage is good but missing common variations like "wrap up the feature", "done implementing", or "open a pull request", which keeps it below the comprehensive-synonyms anchor.

4 / 5

Distinctiveness Conflict Risk

Distinctive phrases like "re-grounded on the original design doc + ticket ACs, not the plan" and "deferral-closure gate" carve a clear niche, but generic components ("run verification", "create the PR") overlap with plain review/PR-creation skills. Mostly distinct with minor overlap risk, matching anchor 4 rather than the fully-distinct anchor 5.

4 / 5

Total

16

/

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

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.