CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

nic-contribution-review

Triage gates for the GitHub Copilot Code Review bot on github.com only: provenance (fork or in-repo), intake checks for fork PRs, a one-logical-change scope check with split proposals and issue drafts, and re-review churn detection when fix commits introduce new defects. Use when acting as GitHub Copilot Code Review on a NIC pull request, before nic-code-review. Never load for local reviews (/pr-review prompt, IDE chat, Copilot CLI, Claude Code).

Quality

89%

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

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.

A high-quality instruction-only skill body: every rule is operational and non-obvious, gates have explicit pass conditions, and the output format is exactly specified down to comment prefixes and placement. Remaining gaps are minor — a few rationale sentences to trim, an implicit interaction between a failed Gate 1 and later gates, and data tables that could move to a reference file.

DimensionReasoningScore

Conciseness

The body is dense with repo-specific rules Claude cannot know (companion-file table, accepted-issue label criteria, churn strike classes) and explains no general concepts, but a few rationale sentences could be trimmed — e.g. 'Both front matter flags keep local agents from loading this skill; the bot reaches it through .github/copilot-instructions.md' and 'New Non-blocking findings on unchanged code are what keep reviews from ending'. This fits anchor 4 ('minor instances of over-explanation') rather than anchor 5 ('every token earns its place') and is far above anchor 3.

4 / 5

Actionability

Fully executable instruction guidance: exact comment prefixes ('Gate:', 'Scope:', 'Churn:') with exact placement (first changed file, first hunk of the part that should move, the regressing line), exact label and path names to check, a strike escalation ladder, and a copy-paste-ready issue-draft template. Per the scoring notes, absence of code in an instruction-only skill is not penalized; this matches the level-5 anchor.

5 / 5

Workflow Clarity

Clear sequence (Step 0 fetch table → gate-selection table → Gates 1–3 → output) with explicit pass/fail checkpoints per gate and an escalation ladder (strike 1 warning → strike 2 abandon). A minor gap keeps it at anchor 4 rather than 5: whether Gates 2–3 still run after Gate 1 fails ('then stop') is resolved only implicitly via the Principles line 'When a gate fails, report it and stop: no line-by-line review', not stated at the gate itself. It is well above anchor 3, whose checkpoints are missing or implicit.

4 / 5

Progressive Disclosure

No bundle files (references/, scripts/, assets/) exist, so this scores on the body alone: a single well-sectioned file with clearly signaled one-level references to the sibling skill ([nic-code-review](../nic-code-review/SKILL.md)) and CONTRIBUTING.md anchors. Structure is good with minor gaps — the large companion/scope tables are core operational content and justify inline placement, but the ~150-line body with three big data tables could plausibly split some material into a reference file. Anchor 4 ('good structure, most content appropriately placed, references mostly clear') fits better than anchor 5, which requires a deliberate overview-to-detail split.

4 / 5

Total

17

/

20

Passed

Description

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

An excellent description: it lists every gate concretely, gives an explicit positive trigger and an explicit negative-scope clause, and is tightly scoped to a single harness to avoid conflicts with local-review skills. The only gap is that 'NIC' is never expanded to its full name 'NGINX Ingress Controller', costing a little natural-term coverage.

DimensionReasoningScore

Specificity

The description enumerates every concrete gate: 'provenance (fork or in-repo)', 'intake checks for fork PRs', 'a one-logical-change scope check with split proposals and issue drafts', and 're-review churn detection when fix commits introduce new defects' — multiple specific concrete actions with comprehensive coverage, matching the level-5 anchor rather than level 4 (no coverage gap).

5 / 5

Completeness

It explicitly answers both questions: what ('Triage gates for the GitHub Copilot Code Review bot... intake checks... scope check... churn detection') and when ('Use when acting as GitHub Copilot Code Review on a NIC pull request, before nic-code-review' plus a never-load clause for local reviews). Both are concrete and explicit, matching the level-5 anchor; level 4's 'when could be more explicit' does not apply.

5 / 5

Trigger Term Quality

Strong natural triggers for the domain — 'GitHub Copilot Code Review', 'NIC pull request', 'fork PRs', 're-review', 'nic-code-review' — plus explicit negative triggers ('/pr-review prompt, IDE chat, Copilot CLI, Claude Code'). A few natural terms are missing, notably 'NIC' is never expanded to 'NGINX Ingress Controller', so it falls just short of the level-5 'comprehensive synonyms' anchor and clearly above level 3.

4 / 5

Distinctiveness Conflict Risk

A clear niche (one named bot, github.com only, sequenced before a named sibling skill) with distinct triggers and explicit exclusions for all local harnesses — minimal conflict risk, matching the level-5 anchor rather than level 4 ('minor overlap risk').

5 / 5

Total

19

/

20

Passed

Validation

93%

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

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 1 suspicious

Warning

Total

15

/

16

Passed

Repository

nginx/kubernetes-ingress

Path

.github/skills/nic-contribution-review/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.