CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

post-merge-scan

Scan recent merges to main for follow-up cleanup: TODOs, deprecations, broken doc links, stale flags. Use in post-merge cleanup loops.

Quality

80%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

Fix and improve this skill with Tessl

tessl review fix ./starters/post-merge-cleanup/.claude/skills/post-merge-scan/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

80%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 lean, well-structured instruction skill with a strong output contract and a meaningful prompt-injection defense. The body is token-efficient and actionable; its weakness is the implicit workflow — no explicit scan sequence or verification checkpoints, and a couple of undefined references ('state', the routing choice between minimal-fix and skip).

Suggestions

Add an explicit ordered scan sequence with the concrete command to enumerate merges (e.g. `gh pr list --merged --base main --since <date>`) so the entry step is unambiguous.

Add validation checkpoints: verify a TODO/FIXME is newly introduced in the merge (compare against the pre-merge tree) and confirm doc links actually fail to resolve before reporting them as broken.

Define the 'state' file (where it lives, what it contains) and complete the routing rules so it is clear when to choose minimal-fix vs. skip.

DimensionReasoningScore

Conciseness

The ~38-line body is lean with zero concept explanation: a copy-paste output template, a five-item detection list, five terse routing rules, and a substantive untrusted-input policy. Every token earns its place; not 4 because there are no over-explanations to trim (the safety block is load-bearing for an autonomous loop that ingests PR/issue text).

5 / 5

Actionability

Concrete, executable guidance: an exact output template (file:line, risk/effort enums, four loop-action choices), named detection targets, and routing rules ("Large refactors → ticket", "Medium+ risk → escalate-human", 7-day window). Not 5 due to minor gaps: the "state" file is referenced but never defined, no command for enumerating recent merges, and no rule for when to choose minimal-fix vs. skip.

4 / 5

Workflow Clarity

The sequence (enumerate merges → scan each for listed items → emit per-merge report → route action) is coherent but only implicit via section organization, and checkpoints are missing or implicit — e.g. "introduced in merge" states a filter without any method to verify it, and there is no verification step before reporting (confirming a doc link is actually broken). Not 4 because most checkpoints are absent rather than minor gaps; the read-only nature of the scan keeps it above level 2.

3 / 5

Progressive Disclosure

Under 50 lines with no external content needed; well-organized sections (output format, detection list, rules, safety policy) with the one external pointer being a clearly signaled background link. Per the simple-skill scoring note, this earns 5 with just well-organized sections; no bundle files exist, so no navigation burden applies.

5 / 5

Total

17

/

20

Passed

Description

75%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 concise, specific description in third-person imperative voice that clearly states both what the skill does and when to use it. It sits solidly at the good level on every dimension, with the main headroom being more concrete user-facing trigger phrases and synonyms (e.g., PRs/pull requests) in the when-clause.

Suggestions

Expand the when-clause with user-facing trigger phrases, e.g. 'Use when running post-merge cleanup loops or when the user mentions merged PRs, leftover TODOs, or stale flags ready for removal'.

Add common synonyms such as 'PRs' / 'pull requests' / 'merged PRs' to the trigger terms so natural user phrasings match.

Consider mentioning the remaining cleanup categories from the body (unused imports, dead code) for fuller what-coverage.

DimensionReasoningScore

Specificity

Names the domain ("recent merges to main", "follow-up cleanup") and lists several concrete scan targets ("TODOs, deprecations, broken doc links, stale flags"). Fits anchor 4 — several specific actions with minor gaps (body also covers unused imports/dead code and the report format, which the description omits); not 5 because coverage is not comprehensive, and clearly above anchor 3's 1-2 actions.

4 / 5

Completeness

Both parts present: a clear "what" ("Scan recent merges to main for follow-up cleanup: TODOs, deprecations, broken doc links, stale flags") and an explicit "when" ("Use in post-merge cleanup loops"). Not 5 because the "when" is loop-oriented rather than concrete user trigger phrases (no "when the user mentions merged PRs / TODOs / flags ready for removal"); comfortably above anchor 3 since the Use clause is present and explicit.

4 / 5

Trigger Term Quality

Good natural keyword coverage — "merges to main", "post-merge cleanup", "TODOs", "deprecations", "stale flags", "broken doc links", "follow-up" — phrases a user would plausibly say. Not 5: misses common synonyms and variations such as "PRs", "pull requests", "merged PRs"; not 3 because coverage goes well beyond a single generic keyword.

4 / 5

Distinctiveness Conflict Risk

Clear niche (post-merge scanning of main for cleanup follow-ups) with mostly distinct triggers. Minor overlap risk: generic requests like "clean up TODOs" or code-review/cleanup skills could also match; not 5, but well above anchor 3's vague overlap with similar skills.

4 / 5

Total

16

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

referenced_paths_exist

Referenced path issues: 1 missing

Warning

Total

14

/

16

Passed

Repository

cobusgreyling/loop-engineering

Path

starters/post-merge-cleanup/.claude/skills/post-merge-scan/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.