CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

dsh-pre-push-checks

Use before pushing, force-pushing, marking ready for review, or claiming checks pass on a deepseek-harness branch, and immediately after gh stack sync publishes rewritten branches, to select the smallest tests and checks that cover the outgoing or just-published diff without reflexively running the full repository suite.

Quality

90%

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 dsh-pre-push-checks in river666/deepseek-harness-enterprise

SKILL.md
Quality
Evals
Security

Quality

Content

88%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 dense, high-signal skill body that assumes competence, gives real commands for every step, and sequences both ordinary and post-sync push workflows with explicit validation and repair loops. The only notable gaps are the absence of a fully concrete example invocation and the lack of any reference splitting despite a ~110-line body.

Suggestions

Include one filled-in example of the change-scope and vitest coverage invocations (real package and base ref from a recent change) so a common case is fully copy-paste ready.

Move the coverage-selection mechanics and the post-sync validation detail into a one-level-deep reference file, keeping SKILL.md as a leaner overview with clearly signaled links.

DimensionReasoningScore

Conciseness

The body is lean and assumes Claude's competence throughout: it never explains what rebase, Vitest, or CI are, and every sentence carries repo-specific, non-inferable guidance (e.g., "pre-commit fixes staged lint, checks staged whitespace, and guards vendored-source metadata"; "The configured per-file 100% thresholds still apply inside the selected source scope"). This matches the 5 anchor — every token earns its place — and shows none of the 4 anchor's trimmable over-explanation.

5 / 5

Actionability

Nearly all guidance is executable: `git status --short --branch`, `pnpm --silent run change-scope --base <verified-base-ref>`, the full `vitest run ... --coverage.include=...` invocation, `--force-with-lease=<branch>:<observed-oid>`, and `git rev-parse HEAD origin/$(git branch --show-current)` are real, copy-shape-ready commands. It sits at 4 rather than 5 because the vitest and change-scope examples use structural placeholders (`packages/<group>/<package>/...`) with no single fully concrete example invocation covering a common case, and the flexibility is only implicitly justified rather than demonstrated.

4 / 5

Workflow Clarity

Both multi-step workflows are explicitly sequenced with validation checkpoints and feedback loops: the push procedure (run checks once → commit and inspect fixer-touched files → push → "Verify the remote ref matches local HEAD" → "Report pending checks as pending") and the post-sync sequence (re-query heads → inspect each rewritten layer → run selected evidence → "Keep every PR unmerged... until all selected checks pass"). Failure handling includes a repair loop ("repair the failure, validate the repair, and publish the correction"), and the destructive force-push path is guarded by lease verification — matching the 5 anchor including its feedback-loop requirement for risky operations.

5 / 5

Progressive Disclosure

The body is well-structured with clear section headers (Inspect the outgoing change, Select relevant evidence, Full local rehearsal, Protect history-rewriting pushes, Handle failures, Push procedure) and every command inline is core to the flow. It does not reach 5 because the skill exceeds the ~50-line simple-skill case with no bundle files at all: detailed subsections like the coverage-selection mechanics and post-sync validation procedure could be split into one-level-deep references to keep SKILL.md a leaner overview.

4 / 5

Total

18

/

20

Passed

Description

87%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 with an explicit, multi-trigger when-clause and a precise, non-generic what-statement anchored to a named repository context. It is only marginally short of perfect on action coverage and natural-term synonyms.

Suggestions

Fold one more capability into the what-statement (e.g., failure/blocking handling or lease-protected force-push validation) to move from one composite action to multiple distinct actions.

Add a common user phrasing such as "run the right tests before a push" or "pre-push checks" as a trigger synonym alongside the existing conditions.

DimensionReasoningScore

Specificity

The description names a concrete core action — "to select the smallest tests and checks that cover the outgoing or just-published diff without reflexively running the full repository suite" — plus concrete operations like "force-pushing" and "after gh stack sync publishes rewritten branches". It falls just below the 5 anchor because it lists essentially one composite action (minimal evidence selection) with many trigger conditions rather than multiple distinct capability actions, and it omits secondary capabilities the body covers (failure handling, lease-protected rewrites).

4 / 5

Completeness

It explicitly answers both questions with concrete trigger phrases: what ("select the smallest tests and checks that cover the outgoing or just-published diff without reflexively running the full repository suite") and when ("Use before pushing, force-pushing, marking ready for review, or claiming checks pass... and immediately after gh stack sync publishes rewritten branches"). This matches the 5 anchor's pattern of an explicit 'Use when'-style clause plus a clear capability statement; the 4 anchor's "when could be more explicit" does not apply here.

5 / 5

Trigger Term Quality

Strong natural trigger terms: "before pushing", "force-pushing", "marking ready for review", "claiming checks pass", "gh stack sync" — phrases a user on this repository would plausibly say verbatim. It stops short of the 5 anchor's comprehensive synonym coverage: common variations like "run tests before push", "pre-push checks", or "verify the diff" are not present.

4 / 5

Distinctiveness Conflict Risk

The description is tightly scoped to "a deepseek-harness branch" and to repo-specific triggers like "gh stack sync publishes rewritten branches", giving it a clear niche with minimal overlap risk against generic testing or git skills. It is at least as distinct as the 5 anchor's PDF example; the 4 anchor's "minor overlap risk with closely related skills" would only fit if the domain qualifier were absent.

5 / 5

Total

18

/

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

river666/deepseek-harness-enterprise

Path

.claude/skills/dsh-pre-push-checks/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.