CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

green-prs

Inspect and prepare the selected PR queue against its exact current base, repair authorized defects with contributor credit, and report full versus partial validation. Use for maintainer pre-greening; merging requires separate owner authorization and the shared merge gates.

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

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

An exceptionally lean, high-signal instruction-only skill body: a clearly sequenced maintainer workflow with explicit validation gates, HOLD conditions, and evidence requirements, delegating deeper protocols to clearly signaled one-level-deep shared files. Its only weaknesses are the absence of any concrete commands or examples inline and its reliance on external shared files for several checkpoint definitions.

DimensionReasoningScore

Conciseness

Dense, imperative policy text with no concept explanations, no filler, and no padded sections; nearly every clause encodes a non-obvious project rule ("Base-red does not waive a required gate", "A locked fork needs the owner-reviewed credit-preserving fallback"). Every token earns its place by assuming Claude's competence.

5 / 5

Actionability

Concrete, executable process guidance throughout: one isolated worktree per PR, verify the checked-out SHA equals its observed live head before editing, run a RED reproduction then focused and affected suites, and a structured receipt with named fields (base SHA, head SHA, run IDs, artifact references). Not 5 because there are no copy-paste commands/examples and several directives defer entirely to shared files (merge-gates, teardown protocol) without restating key steps.

4 / 5

Workflow Clarity

A clear three-phase sequence (resolve scope and ownership, prepare and validate, evidence and handoff) with explicit pre-flight checkpoints ("Before pushing, verify the remote still equals the observed predecessor; a concurrent change means HOLD") and a full result classification taxonomy. As a batch/destructive-adjacent workflow it has validation, so the cap at 3 does not apply; it stays below 5 because error recovery beyond HOLD lacks explicit fix-and-retry loops and some checkpoint definitions live in external shared files.

4 / 5

Progressive Disclosure

Well-organized sections with the body kept as an overview; shared references are one level deep and clearly signaled with when-to-read conditions ("Read `../_shared/protected-surfaces.md` before any instruction-surface change or merge"). Not 5 because references use bare backtick paths outside the bundle rather than clearly navigable in-bundle files, and no bundle files (references/, scripts/, assets/) exist in this skill directory to verify against.

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 strong, niche-specific description with concrete multi-part capabilities and an explicit use-when clause plus authorization boundary. Its main weakness is trigger term coverage, which relies on internal jargon rather than the natural phrases a user would say.

Suggestions

Add natural trigger synonyms to the 'Use for' clause, e.g. "Use for maintainer pre-greening or when the owner asks to validate/get PRs ready to merge", so users phrasing it as 'pull requests' or 'ready to merge' match naturally.

Briefly name the reporting outputs in plain terms (e.g., "classify each PR as PASS, PARTIAL, or RED") so the what is concrete without leaning on body-only jargon.

DimensionReasoningScore

Specificity

"Inspect and prepare the selected PR queue against its exact current base, repair authorized defects with contributor credit, and report full versus partial validation" lists several distinct concrete actions. It falls short of 5 because the actions are process-level and leave coverage gaps (e.g., what validation entails), but exceeds 3's 1-2 concrete actions.

4 / 5

Completeness

Both what (inspect, prepare, repair, report validation level) and when ("Use for maintainer pre-greening") are explicitly stated, plus an authorization boundary. Not 5 because the when-clause could be more specific with concrete trigger phrases; not 3 because the when is explicit, not merely implied.

4 / 5

Trigger Term Quality

"Use for maintainer pre-greening" plus "PR queue" and "merge" provide some relevant keywords, but common natural variations users would say ("pull requests", "ready to merge", "green these PRs") are missing and the phrasing leans on internal jargon ("pre-greening"). Not 4 because keyword coverage is narrow rather than good-with-minor-gaps.

3 / 5

Distinctiveness Conflict Risk

A clear niche (maintainer pre-greening of a PR queue) with an explicit fence against the nearest competing skill: "merging requires separate owner authorization and the shared merge gates." Minimal risk of triggering for the wrong skill.

5 / 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

diegosouzapw/OmniRoute

Path

config/quality/skill-templates/green-prs/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.