CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

contributor-screen

Screen a pull request from an outside contributor for hidden, obfuscated, or supply-chain-risky changes before any of its code runs.

69

Quality

87%

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

93%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 exemplary instruction-only skill body: lean, dense with concrete detection indicators, explicit about scope exclusions, severity, and output format, with sound prompt-injection defenses for reviewing untrusted input. The only real improvement is making the implicit scan→triage→report sequence explicit with a verification step before high-severity findings.

DimensionReasoningScore

Conciseness

The body is lean and assumes Claude's competence: it names 'base64, hex, char codes', '~/.npmrc', 'pnpm overrides', 'onlyBuiltDependencies', 'GITHUB_*' without explaining what any of them are, and every section (untrusted-input framing, threat categories, exclusions, severity, output format) earns its place. Nothing here is padding or a concept Claude already needs explained, so it matches the anchor-5 example's economy rather than the 'could be tightened' anchor 3 or 'minor trimming' anchor 4.

5 / 5

Actionability

For an instruction-only skill the guidance is fully actionable: exact APIs ('eval, new Function, vm, dynamic require/import of computed names'), exact paths ('~/.npmrc', '~/.config', '/proc'), exact hooks ('postinstall', 'prepare'), exact CI variables ('CI', 'GITHUB_*'), a defined three-level severity scale, and a concrete output spec ('file and line, what is hidden or risky, and what a maintainer should check'). Per the code_vs_instruction scoring note, absence of code is not penalized when guidance is this specific, and the indicator list covers the common cases comprehensively — the anchor-5 condition.

5 / 5

Workflow Clarity

The skill is single-purpose with an unambiguous framing question ('Answer one question: could this change hide behavior a reviewer reading the diff would not see...') and a coherent implicit flow (scan categories → exclude non-qualifying issues → assign severity → report), which fits anchor 4's 'clear sequence with minor gaps'. It is not 5 because the sequence is implicit (organized by topic, not steps), there is no verification checkpoint (e.g., 'confirm the encoded blob is actually decoded before rating it high'), and the under-50-line simple-skill exception does not apply to this 63-line body; it is not 3 because the destructive/batch validation cap is inapplicable to this read-only task and the reporting contract is explicit.

4 / 5

Progressive Disclosure

There are no bundle files (references/, scripts/, assets/ are absent) and the body references none, so there is nothing misfiled or nested: the ~63 lines of screening criteria are appropriately self-contained in SKILL.md with clear sections (Untrusted input, look-for categories, scope exclusions, severity, output) and easy navigation. Splitting this into reference files would add indirection without benefit, so it matches anchor 5's 'content appropriately split; easy navigation' rather than anchor 4, which requires organization gaps that are not present.

5 / 5

Total

19

/

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 tight, concrete description that clearly states what the skill does and for which situation (outside-contributor PRs, before code runs) with good natural trigger terms. Its main gaps are the absence of an explicit 'Use when...' trigger clause and of common synonyms (PR, external contributor, untrusted code) that would push completeness and trigger quality to 5.

Suggestions

Add an explicit trigger clause, e.g., 'Use when reviewing a PR from an external/third-party contributor or when asked to check a diff for hidden or malicious behavior.'

Include natural synonyms users would say — 'PR', 'external contributor', 'untrusted code', 'malicious dependency' — to broaden trigger-term coverage.

Optionally name the deliverable (e.g., 'report findings by severity with file and line') so the output is part of the stated capability.

DimensionReasoningScore

Specificity

The single action 'Screen a pull request' is made concrete by three specific screening targets ('hidden, obfuscated, or supply-chain-risky changes') plus an explicit condition ('before any of its code runs'), placing it between the 1-2-action anchor (3) and the several-actions anchor (5). It is more concrete than anchor 3's example but names fewer distinct actions than anchor 4's, so 4 fits best; 5 would require a fuller list of capabilities (e.g., reporting format or severity triage).

4 / 5

Completeness

The 'what' is explicit ('screen a PR for hidden, obfuscated, or supply-chain-risky changes') and the 'when' is explicitly stated as a condition — 'from an outside contributor' and 'before any of its code runs' — so the missing-'Use when' cap at 3 does not bind. It is not 5 because there is no explicit trigger phrase of the form 'Use when reviewing a PR from an external contributor'; the when is embedded in the object phrase and could be more explicit.

4 / 5

Trigger Term Quality

Natural phrases users would say are present: 'pull request', 'outside contributor', 'supply-chain', 'obfuscated' — good coverage matching anchor 4 ('good keyword coverage; a few natural terms missing'). It is not 5 because common synonyms like 'PR', 'external/third-party contributor', 'untrusted code', or 'malicious dependency' are absent, and not 3 because the terms present go beyond a bare domain mention.

4 / 5

Distinctiveness Conflict Risk

The niche (pre-execution screening of untrusted contributor diffs for obfuscation and supply-chain risk) is clearly distinct with concrete triggers, matching anchor 4 ('mostly distinct; minor overlap risk'). It is not 5 because a user asking generically to 'review this PR for security' could plausibly trigger a general security-review skill instead — the skill body itself acknowledges this overlap ('the regular security review covers those later').

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

different-ai/openwork

Path

.warden/skills/contributor-screen/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.