CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

mcp-dependency-review

Statically review MCP configuration for mutable package references before approval or CI, without executing discovered MCP servers.

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

The canonical home for this skill is mcp-dependency-review in affaan-m/ECC

SKILL.md
Quality
Evals
Security

Quality

Content

81%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 body is an unusually actionable, well-sequenced static-review procedure with exhaustive decision tables, worked parsing examples for every supported runner, and explicit guardrails at each step. Its main costs are token redundancy from repeating the same safety boundaries in three to four places, and a monolithic inline structure where runner-parsing details could live in a one-level-deep reference file.

Suggestions

Consolidate the repeated boundary statements (no-execution rule, SAFE caveat, mutable-≠-breach, workspace scope) into a single section and cross-reference it, trimming roughly 15-20% of the body's tokens.

Move the runner-specific parsing rules (npx/docker/uvx/pipx flag handling in step 2) into a references/ file, keeping SKILL.md as a compact overview that signals the reference clearly.

DimensionReasoningScore

Conciseness

The classification tables and runner-parsing rules are dense and earn their tokens, but the same boundary statements repeat three to four times (the no-execution rule at lines 11/27/95, the SAFE caveat at lines 61/118, mutable-≠-breach at lines 31/117/150, and the scope boundary at lines 23/81/158-162). This is 'some unnecessary explanation or could be tightened' (anchor 3) rather than the 'minor instances' required for anchor 4.

3 / 5

Actionability

Fully actionable for an instruction-only skill: a concrete config path list, exhaustive selector-classification tables, worked parsing examples for each runner ("npx --package ecc-universal ecc", "docker run --rm -i example/server:1.2.3 serve", "uvx --from example-tool==1.2.3 example-command"), a fixed six-column report format, and a complete input-to-output example. These are copy-paste-ready decision rules covering the common cases, matching anchor 5.

5 / 5

Workflow Clarity

Five clearly sequenced steps with per-step guardrails, an explicit ambiguity fallback ("If the syntax is ambiguous, return REVIEW rather than guessing"), unknown-version handling ("If the reviewed version is unknown, say so"), a never-omit-completeness rule, and a closing boundary checklist for the report. This is a read-only review with no destructive or batch operations, so the validation cap does not apply, and the edge-case feedback paths satisfy anchor 5 better than anchor 4's 'minor validation gaps'.

5 / 5

Progressive Disclosure

Well-organized sections with clear headers, easy navigation, and no buried or nested references (the skill has no bundle files, and the only pointers are clearly signaled cross-skill names and one optional external link). Not 5 because everything is inline in a ~170-line file: the runner-specific parsing details and the extended Docker/Python tables sit at a detail level that could be split into a one-level-deep reference file, leaving SKILL.md as an overview.

4 / 5

Total

17

/

20

Passed

Description

73%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 concise, in third-person-adjacent imperative voice, and clearly communicates both what the skill does and when to use it, with a strong distinct niche. Its main weaknesses are limited action coverage (one 'review' verb rather than the full classify/report capability set) and a narrow when-clause that misses natural trigger terms like .mcp.json, npx, or @latest.

Suggestions

Add one or two more concrete actions to the description, e.g. "classifies each package selector as SAFE/MEDIUM/HIGH/REVIEW and produces a bounded report".

Broaden the when-clause with additional concrete triggers, e.g. "Use when reviewing .mcp.json or when a config references @latest, bare package names, or version ranges".

DimensionReasoningScore

Specificity

"Statically review MCP configuration for mutable package references" names the domain and one concrete action with method/timing modifiers, but does not enumerate the fuller capability set (classification levels, bounded report, pin recommendations). This matches anchor 3 (domain plus 1-2 concrete actions, not comprehensive); anchor 4 requires several distinct specific actions, which the single 'review' verb does not provide.

3 / 5

Completeness

Explicit what ("Statically review MCP configuration for mutable package references") plus an explicit when-equivalent ("before approval or CI"), clearing the cap of 3 for missing trigger guidance. Not 5 because the when clause covers only approval/CI timing and omits other concrete triggers, e.g. a config using @latest or reviewing .mcp.json changes.

4 / 5

Trigger Term Quality

Good natural keyword coverage: "MCP configuration", "package references", "approval", "CI", "MCP servers" — comparable to the anchor-4 example. Not 5 because common user phrasings and extensions are missing (".mcp.json", "npx", "@latest", "pin", "dependencies"); above 3 because coverage is good rather than merely 'some relevant keywords'.

4 / 5

Distinctiveness Conflict Risk

Clear niche (static MCP-config dependency-drift review) with distinct triggers; "MCP configuration for mutable package references" is unlikely to fire for generic code-review or security skills. Fits anchor 5 (clear niche, minimal conflict) better than anchor 4, whose 'minor overlap with closely related skills' is not present in the description text itself.

5 / 5

Total

16

/

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

frontmatter_unknown_keys

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

Warning

Total

15

/

16

Passed

Repository

affaan-m/ECC

Path

.cursor/skills/mcp-dependency-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.