CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

sweeper-fix

Implement a VS Code Sweeper fix spec — fetch the sweeper's agent-fixable review record from its public state repo, implement the narrow fix in the current vscode checkout, and — after showing the diff — open a draft PR. Use ONLY when the request explicitly asks for the sweeper — "sweeper-fix", "sweeper", "vscodesweeper", a sweeper record, or a sweeper fix spec. Do NOT use for a plain "fix this issue" request that doesn't mention the sweeper; fix those directly with your normal tools instead.

77

Quality

97%

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

92%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 tightly written operational skill: exact commands, an explicit gate-then-implement-then-ship workflow with validation and a human-approval checkpoint before any public action, and clean single-file structure. The only weakness is mild redundancy — the fork-remote `gh` warning and a few safety-rule sentences are stated twice.

Suggestions

State the fork-remote/default-repo `gh` warning once (section 2's gate, where the live fetch happens) instead of repeating it in both section 1 and section 2.

Trim restated safety rules — e.g. "No confirmation, no push — ever" duplicates the sentence before it, and "Keep edits minimal and reviewable" overlaps the "Stay narrow... No refactors, no drive-by cleanups" bullet.

Consider collapsing the 'Match the codebase' and 'Stay narrow' bullets in section 3 into one bullet, since both instruct minimal, style-consistent edits.

DimensionReasoningScore

Conciseness

The body is dense and operational with no padding about concepts Claude already knows, but it repeats itself in a few places: the fork-remote/`gh` default-repo warning appears in both section 1 ("never bare `gh issue view`, which a fork remote can redirect to the wrong repo's issue") and section 2 ("never rely on `gh`'s default-repo resolution, which a fork remote can redirect to the wrong repo's issue"), and "No confirmation, no push — ever" restates the preceding sentence. These are minor instances of over-explanation that could be trimmed, matching the score-4 anchor rather than the every-token-earns-its-place anchor at 5.

4 / 5

Actionability

Every step is copy-paste ready: the exact `gh api "repos/egamma/vscodesweeper-state/contents/records/..."` fetch command, `gh issue view <issue-number> --repo microsoft/vscode --json state,labels,updatedAt`, `gh search prs ... --json url,title`, the branch convention `<your-github-login>/fix-<issue-number>`, an explicit `git add`-by-path rule, and a full `gh pr create --repo microsoft/vscode --base main --draft ...` command with an ordered PR-body spec. Not below 5 because the only placeholders (<issue-number>, <body>, test command) are inherently per-issue dynamic, not pseudocode.

5 / 5

Workflow Clarity

The sequence is numbered 0–4 with explicit validation checkpoints at every risky boundary: clean-tree and auth preconditions, five live-state gates, a staleness check that asks the maintainer to confirm, a test that must fail before and pass after the fix ("run it both ways and say so"), a mandatory diff-approval pause before any push, and a re-run of gates 2–4 immediately before shipping because "a push is public". This matches the anchor for clear sequence, explicit validation, and error-recovery feedback loops, including for the destructive/public operations the rubric flags.

5 / 5

Progressive Disclosure

No bundle files exist (references/, scripts/, assets/ are all absent), and none are needed: the body is a single ~75-line linear procedure where every section (preconditions, fetch, gates, implement, safety, ship) is needed on each run, so nothing clearly belongs in a separate file. Sections are clearly headed and easy to navigate, and there are no nested references at all. Not below 5 because the only candidates for extraction (the no-record fallback, the PR-body spec) are short and sit naturally in their workflow steps.

5 / 5

Total

19

/

20

Passed

Description

100%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 exemplary description: it states the complete action pipeline in third-person imperative form, gives exhaustive natural trigger terms plus an explicit negative trigger, and cleanly separates this skill from ordinary fix requests. No dimension shows a weakness worth suggestions.

DimensionReasoningScore

Specificity

It lists multiple specific concrete actions with comprehensive coverage of the workflow: "fetch the sweeper's agent-fixable review record from its public state repo", "implement the narrow fix in the current vscode checkout", "after showing the diff — open a draft PR". Each action names the exact artifact and location, matching the comprehensive multi-action anchor rather than the minor-gaps anchor at 4.

5 / 5

Completeness

Both halves are explicit: the what is the full action pipeline quoted above, and the when is concrete with trigger phrases — "Use ONLY when the request explicitly asks for the sweeper", reinforced by a do-not-use clause ("Do NOT use for a plain 'fix this issue' request that doesn't mention the sweeper"). This clearly matches the anchor for explicitly answering both what and when with concrete trigger phrases, not the weaker-when anchor at 4.

5 / 5

Trigger Term Quality

It enumerates the natural trigger terms comprehensively with synonyms: "'sweeper-fix', 'sweeper', 'vscodesweeper', a sweeper record, or a sweeper fix spec" — covering the skill name, its shorthand, the repo/tool name, and the two artifact phrasings a maintainer would actually say. It also names the key anti-trigger ("a plain 'fix this issue' request"), which is exactly how a user would phrase the non-matching case.

5 / 5

Distinctiveness Conflict Risk

It occupies a clear niche (sweeper-review fix specs for microsoft/vscode) with distinct triggers, and the explicit negative boundary — "Do NOT use for a plain 'fix this issue' request... fix those directly with your normal tools instead" — removes the main overlap risk with generic issue-fixing skills. Minimal conflict risk, matching the clear-niche anchor rather than the minor-overlap anchor at 4.

5 / 5

Total

20

/

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

posit-dev/positron

Path

.github/skills/sweeper-fix/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.