CtrlK
BlogDocsLog inGet started
Tessl Logo

finishing-a-development-branch

Use this when you have completed some feature implementation and have written passing tests, and you are ready to create a PR.

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 ./.agency/plugins/nori/skills/finishing-a-development-branch/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

77%

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

The body is highly actionable and well-sequenced with strong validation feedback loops, the rubric's most-valued qualities. It loses points mainly on conciseness (duplicated detection blocks and repeated boilerplate) and on progressive disclosure (a header-less monolithic list over 50 lines).

Suggestions

Merge the formatter (step 3) and linter (step 4) detection snippets into a single per-language table or block to remove the near-duplicate Node/Rust/Python/Go listings.

De-duplicate the "fresh pair of eyes / you do NOT have to follow suggestions" note shared by steps 6 and 7 — state it once and reference it.

Add section headers (e.g. ## Verify, ## Clean up, ## Review, ## Ship) over the numbered steps to improve navigation across the 110-line body.

DimensionReasoningScore

Conciseness

Mostly efficient command-driven steps, but the formatter (step 3) and linter (step 4) detection blocks are near-duplicates across Node/Rust/Python/Go, and the "fresh pair of eyes" note is repeated verbatim in steps 6 and 7 — both could be tightened.

2 / 3

Actionability

Provides fully executable commands ("npm test / cargo test / pytest", "cargo clippy --fix --allow-dirty --allow-staged", "ruff check --fix .", "gh pr create …") including a complete PR body template — copy-paste ready.

3 / 3

Workflow Clarity

A clearly numbered 12-step sequence with explicit validation checkpoints and feedback loops: tests-fail halt ("Cannot proceed until tests pass"), CI failure recovery with "go back to step 10", and a main-branch push guard.

3 / 3

Progressive Disclosure

At ~110 lines it exceeds the simple-skill threshold and is a monolithic numbered list with no section headers and no external references; the step numbering gives some structure but navigation and content splitting could be improved.

2 / 3

Total

10

/

12

Passed

Description

47%

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, distinct trigger and natural phrasing, but it over-relies on preconditions and second-person voice while leaving the skill's actual actions implied. Rewriting in third person and listing the concrete finishing steps would raise specificity and completeness.

Suggestions

Rewrite in third person and lead with concrete actions, e.g. "Verifies tests, runs formatters/linters and type checks, performs a self-review, then pushes and opens a PR. Use when a feature is implemented with passing tests and is ready to ship."

Add common trigger-term variations such as "pull request", "open a PR", and "finish a branch" so the skill surfaces for more user phrasings.

Make the "what" explicit instead of implied — state that the skill orchestrates test verification, formatting, linting, type checking, self-review, push, PR creation, and CI follow-up.

DimensionReasoningScore

Specificity

Base 2: it names the domain and one concrete action ("create a PR") but is dominated by preconditions ("completed some feature implementation", "written passing tests") rather than listing the skill's actions; reduced to 1 because the description uses second person ("Use this when you have…") which the rubric penalizes by -1 on specificity.

1 / 3

Completeness

The "when" is explicit ("Use this when you have completed… and you are ready to create a PR"), but the "what" is only implied — it never states the skill runs tests, formatters, linters, type checks, self-review, and CI verification.

2 / 3

Trigger Term Quality

"feature implementation", "passing tests", and "create a PR" are natural trigger terms, but coverage is incomplete — missing common variations like "pull request", "finish/merge branch", or "open a PR".

2 / 3

Distinctiveness Conflict Risk

The trigger (completed feature + passing tests + ready to create a PR) carves a clear niche unlikely to fire for unrelated skills.

3 / 3

Total

8

/

12

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.

Validation16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
microsoft/FluidFramework
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.