CtrlK
BlogDocsLog inGet started
Tessl Logo

positron-pr-helper

Generates well-structured PR bodies with dynamically fetched e2e test tags

56

Quality

63%

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 ./.claude/skills/positron-pr-helper/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

77%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 strong, highly actionable skill body with an excellent workflow: concrete commands, complete executable examples, and genuine validation checkpoints (PETE verdict handling, '@:' tag self-scan). The main weakness is structure, not substance — the body inlines template/example material that duplicates the never-linked references/pr-templates.md and repeats the tag-safety rule multiple times.

DimensionReasoningScore

Conciseness

The body is mostly dense, useful guidance, but there is more than minor tightening to do: the '@:' tag-safety rule is stated three times (dedicated section, a Tips bullet, and inside Step 4), the opening narration ("This skill helps you create comprehensive PR bodies...", "I'll guide you through...") adds little, and the inline Templates/Examples sections (~120 lines) duplicate content that already exists in references/pr-templates.md. Anchor 3 (mostly efficient, could be tightened) fits better than 4 (only minor trims).

3 / 5

Actionability

Fully executable throughout: exact script path and usage with formats ("./scripts/fetch-test-tags.sh [format]" with markdown/json/list, matching the actual script), concrete commands ("gh issue view", "pbcopy", "gh pr edit"), and two complete copy-paste-ready example PR bodies covering the common bug-fix and feature cases. Matches anchor 5.

5 / 5

Workflow Clarity

A clear 5-step sequence (gather context → fetch tags → PETE evaluation → generate body → output) with explicit validation checkpoints and feedback loops: "If PETE reports Insufficient, I'll say so and suggest concrete additions ... before we finalize the PR body" and "Before finalizing any PR body, scan your own draft for '@:' outside the Validation Steps section and reword those away." The operation is not destructive/batch, so no cap applies; matches anchor 5 (explicit validation, error-recovery guidance).

5 / 5

Progressive Disclosure

The script reference (scripts/fetch-test-tags.sh) is real and accurately documented, but references/pr-templates.md (327 lines) exists in the bundle and is never linked from the body — instead the '## PR Body Templates' and '## Examples' sections inline ~120 lines of substantially the same template/example content. Anchor 3 (content that should be separate is inline; references present but not clearly signaled) is the best fit; anchor 4 would require the bulk of template material to live in the referenced file.

3 / 5

Total

16

/

20

Passed

Description

50%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 in third person and states a clear, concrete capability, but it reads as only a partial inventory of what the skill does and omits any "Use when" trigger guidance. It sits squarely at the midpoint of the rubric: correct ingredients, incomplete coverage.

Suggestions

Add a 'Use when...' clause, e.g. 'Use when creating or updating a PR in the posit-dev/positron repository, drafting a PR body, or when you need the current e2e test tags for Validation Steps.'

Spell out natural trigger variations users would say: 'pull request', 'PR description', 'release notes', and 'test-coverage check (PETE)'.

Mention the positron repository (or positron PRs) explicitly to distinguish this skill from generic PR-writing helpers and prevent cross-skill conflicts.

DimensionReasoningScore

Specificity

Names the domain ("PR bodies") and two concrete actions ("Generates well-structured PR bodies", "dynamically fetched e2e test tags"), but the action list is not comprehensive — updating existing PRs, the PETE test-coverage check, and output options described in the body are absent. Anchor 3 (1-2 concrete actions, not comprehensive) fits better than 4 (several specific actions with minor gaps).

3 / 5

Completeness

The "what" is clear (generates structured PR bodies with fetched e2e test tags), but there is no "Use when..." clause or equivalent explicit trigger guidance — the description never states when to invoke the skill. Per the rubric guideline, a missing 'Use when' clause caps completeness at 3; anchor 3 (clear what, when missing) is the best fit, not 4 (both present).

3 / 5

Trigger Term Quality

"PR bodies" and "e2e test tags" are natural terms users would say, but common variations are missing: "pull request" is never spelled out, and "release notes", "validation steps", and "test coverage check" are absent. This matches anchor 3 (relevant keywords, missing common variations/synonyms) rather than 4.

3 / 5

Distinctiveness Conflict Risk

"dynamically fetched e2e test tags" is a niche hook, but "Generates well-structured PR bodies" alone overlaps any generic PR-writing skill, and the description omits the repository name (posit-dev/positron), so it could trigger for other repos. Anchor 3 (somewhat specific, could overlap with similar skills) fits; anchor 4 would require a qualifier like the repo name.

3 / 5

Total

12

/

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

referenced_paths_exist

Referenced path issues: 1 missing

Warning

Total

15

/

16

Passed

Repository
posit-dev/positron
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.