CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

merge-gate

合入 main,消费风险匹配的验证与适用的 review 证据。Use when: 准备合入 PR。 Not for: 未完成作者自检、处理 review findings 或 Feature 终态验收。 Output: 产品快车道的作者自验或风险要求的独立 review,加受影响检查支撑的合入证据与结果。

Quality

72%

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 ./cat-cafe-skills/merge-gate/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

52%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 content is highly actionable and validation-rich within its project domain, with concrete commands, fail-closed gates, and explicit feedback loops throughout. However, it is a monolithic 73KB document that buries its executable workflow under incident history and duplicated policy prose, with essentially no use of bundle reference files for the large specialized sections.

Suggestions

Move stable, rarely-needed policy detail into reference files (e.g. references/browser-verification-policy.md, references/review-provenance.md, references/case-law.md) and keep SKILL.md to the lane decision, the numbered Step 0.5–8.5 procedure, and the E1–E5 checklist.

Consolidate rules stated in three or four places (核心知识, 合入方式 comments, Common Mistakes, QA) into single canonical sections; the duplication (e.g. merge exit-code semantics, @handle rules) is the main source of token bloat.

Move dated incident narratives and specific SHAs/PR numbers into a 'historical incidents' or 'old patterns' reference section so time-sensitive details stop padding the live workflow sections.

DimensionReasoningScore

Conciseness

The ~780-line body is noticeably verbose: dated incident narratives ("F168 PR #2214 实测 21 轮", "operator 2026-07-15 投诉②修复"), rationale essays, and time-stamped SHAs/dates ("main `1419b60a74`") sit in live sections rather than an old-patterns section, and key rules are restated across 核心知识, 合入方式, Common Mistakes, and QA. This fits anchor 2 ('several unnecessary explanations or padded sections') rather than anchor 3 ('mostly efficient with some tightening possible') because the padding is extensive and structural; it avoids anchor 1 only because the material is novel project policy rather than concepts Claude already knows.

2 / 5

Actionability

The body is dominated by concrete, executable guidance: complete bash blocks (root artifact guard with regex, PR-body @handle checks, hotfix detection with jq parsing, `classify-merge-outcome.mjs` three-state case handling, worktree finish/retain), the E1–E5 table pairing each check with a verification command and failure action, and an exact MCP parameter table for Step 7.6. It stops short of anchor 5 because of placeholder-laden snippets ({PR_NUMBER}, {branch}) and prose-only steps like the 6.5 guardian gate that defer to 'live schema' rather than concrete invocations.

4 / 5

Workflow Clarity

The sequence is explicitly numbered (Step 0.5 → 6.8 → 6.9 → 7 → 7.5a–c → 7.6 → 8 → 8.5) with exceptional validation checkpoints, fail-closed gates, feedback loops (re-run check-feature-truth after doc sync, CI repair state machine, three-state merge exit classification), and checklists. Held at 4 rather than 5 because the linear path is fragmented across interleaved policy essays and duplicated in Quick Reference / Common Mistakes, so a reader must reassemble the order from multiple sections.

4 / 5

Progressive Disclosure

Scored against the actual bundle: no references/, scripts/, or assets/ directories exist, so ~73KB of niche policy (browser verification policy, targeted 终态语义, 判例 #4152/#4163, 封板协议, Evidence Manifest field tables, CI Repair Loop) is inlined in SKILL.md — content that clearly belongs in separate reference files, matching anchor 2. It is above anchor 1 because section headers are clear and external references (../.cat-cafe-shared-refs/pr-template.md, pr-signals.md, sop-definitions/development.yaml) are well signaled and one level deep.

2 / 5

Total

12

/

20

Passed

Description

83%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 is strong: it explicitly states what the skill does, when to use it, what it is not for, and what output to expect, with concrete trigger phrasing and explicit boundaries against sibling skills. Its only weaknesses are heavy project-internal jargon and a couple of missing natural trigger synonyms.

DimensionReasoningScore

Specificity

The description lists several concrete actions — "合入 main,消费风险匹配的验证与适用的 review 证据", author self-verification ("产品快车道的作者自验"), risk-required independent review, and "受影响检查支撑的合入证据" — with only minor gaps in coverage. It falls short of anchor 5 because the actions are wrapped in project-internal jargon (e.g. "产品快车道") that is not self-explanatory, but it clearly exceeds anchor 3's '1-2 concrete actions'.

4 / 5

Completeness

It explicitly answers both what ("合入 main,消费风险匹配的验证与适用的 review 证据") and when ("Use when: 准备合入 PR") with a concrete trigger phrase, and goes further with explicit "Not for" exclusions and an Output clause. This matches the anchor-5 example structure of what + explicit when with concrete trigger phrases.

5 / 5

Trigger Term Quality

The description itself contains natural phrases users would say ("合入 main", "准备合入 PR", "review"), and the frontmatter triggers list adds "merge", "开 PR", "gh pr create", "cloud review" for good bilingual coverage. It is not a 5 because a few natural variations (English "merge PR", "code review", "squash merge") are absent from the description itself.

4 / 5

Distinctiveness Conflict Risk

The niche is clear (post-review merge flow into main) and the "Not for: 未完成作者自检、处理 review findings 或 Feature 终态验收" clause explicitly excludes adjacent skills. Not a 5 because triggers like "cloud review" and "开 PR" overlap with the review-request/receive skills in the same family, creating minor conflict risk.

4 / 5

Total

17

/

20

Passed

Validation

81%

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

Validation — 13 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

skill_md_line_count

SKILL.md is long (779 lines); consider splitting into references/ and linking

Warning

frontmatter_unknown_keys

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

Warning

referenced_paths_exist

Referenced path issues: 14 missing, 2 deeper-than-1-level

Warning

Total

13

/

16

Passed

Repository

zts212653/clowder-ai

Path

cat-cafe-skills/merge-gate/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.