CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

he-plan

Create bounded Harness Engineering execution plans from approved specs or issue slices. Use when work needs ordered implementation units, explicit scope boundaries, rollback posture, traceability, and validation gates before code changes.

Quality

68%

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 ./Plugins/harness-engineering/skills/he-plan/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

56%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 has a genuine strength in workflow discipline — an ordered procedure, concrete validation scripts, and explicit fail-closed gates — but it is overloaded with inline contract detail, relies on undefined jargon and Type 1/Type 2 shorthand, and defers critical mechanics to reference files that are missing from the bundle, so a reader cannot actually navigate to the details it points at.

Suggestions

Move the professional-review prompt-matching paragraphs, the Output Format heading inventory, and the validation lint details into a dedicated reference file (e.g. references/professional-confidence-review.md) and keep one-line pointers in SKILL.md, cutting the body substantially.

Add a minimal worked example of one plan unit (a PU-* block with source ID, acceptance ID, validation command, and rollback note) so the Type 1/Type 2 and 'smallest proof-producing unit' directives are executable rather than abstract.

Fix the reference layout: either ship the referenced contracts (plan-artifact-contract.md, post-plan-handoff.md, stage-arc-boundary-contract.md) inside the bundle with consistent relative paths, or consolidate their operative rules inline and drop the dead pointers.

DimensionReasoningScore

Conciseness

The body does not explain concepts Claude already knows — there is no padding or tutorial prose — but at ~215 lines of maximally compressed jargon it could be tightened considerably: the long exact-prompt-matching paragraphs for professional confidence review mode (lines 21-40), the 25+ field names listed in Outputs, and the exhaustive heading/value inventories in Output Format all restate contract detail inline. This is 'mostly efficient but could be tightened' rather than noticeably padded (2) or lean with every token earning its place (4-5).

3 / 5

Actionability

Concrete executable commands are present (e.g. 'python3 Plugins/harness-engineering/scripts/check_bluf_structure.py <plan-path> --json' and the traceability lints), and the 12-step procedure is directive, but most steps defer their actual content to references that are not in this bundle ('Classify mode and depth using references/', 'Use the execution-first plan template in .../plan-artifact-contract.md'), and key constructs like 'Type 1 / Type 2 decisions' are used without definition or example. There is no sample plan artifact or worked example, matching 'some concrete guidance but incomplete; missing key details'.

3 / 5

Workflow Clarity

The Procedure is a clearly numbered 12-step sequence, the Validation section runs concrete scripts with explicit fail-fast 'pass/fail/blocked' recording and blocked-handoff conditions, and the Failure Mode section defines blocked_reason classes with one recovery step — real feedback loops and checkpoints. It falls short of 5 because several steps are so jargon-dense their execution is ambiguous (e.g. 'resolve stage context', 'apply the strict boundary contract') and the mechanics of steps 2 and 4 are entirely deferred to unavailable references — 'clear sequence with most checkpoints present; minor validation gaps'.

4 / 5

Progressive Disclosure

A final References section with a 'Load only triggered references' policy provides reasonable signaling, but large blocks that belong in separate files are inlined in SKILL.md (the exact-prompt trigger matching, the full output-heading inventory, the lint details), and the actual bundle does not contain the referenced contracts: paths like '../../references/skills/he-plan/plan-artifact-contract.md' and 'post-plan-handoff.md' do not resolve in this bundle, whose references/ holds only eval fixtures (contract.yaml, evals.yaml, task-profile.json). This matches 'some structure; references present but not clearly signaled; content that should be separate is inline'.

3 / 5

Total

13

/

20

Passed

Description

67%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 solid description with an explicit what-and-when structure and a clearly bounded niche. Its main weakness is trigger phrasing built from internal harness jargon rather than the words a user would naturally say when asking for planning help.

Suggestions

Add natural user-facing synonyms to the when-clause, e.g. 'Use when the user asks to create an implementation or execution plan, sequence or break down an approved spec, or plan delivery before coding.'

Replace at least one jargon-heavy trigger term ('ordered implementation units', 'rollback posture') with plain-language equivalents ('ordered work steps', 'rollback plan') so the description matches how users actually phrase planning requests.

DimensionReasoningScore

Specificity

The description states a concrete deliverable — 'Create bounded Harness Engineering execution plans from approved specs or issue slices' — and enumerates concrete plan components ('ordered implementation units, explicit scope boundaries, rollback posture, traceability, and validation gates'). It is not 5 because it describes essentially one composite action (plan creation) rather than multiple distinct actions like extract/fill/merge, and not 3 because the deliverable's components are explicitly and comprehensively named.

4 / 5

Completeness

Both what ('Create bounded Harness Engineering execution plans from approved specs or issue slices') and when ('Use when work needs ordered implementation units, explicit scope boundaries, rollback posture, traceability, and validation gates before code changes') are explicitly present. Not 5 because the when-clause describes abstract work characteristics rather than concrete user trigger phrases or utterances, leaving it one notch below the fully user-anchored anchor; clearly above anchor 3 since the when is explicit, not merely implied.

4 / 5

Trigger Term Quality

Relevant keywords exist ('execution plans', 'specs', 'issue slices', 'rollback', 'scope', 'validation', 'code changes'), but the phrasing leans on internal system jargon ('Harness Engineering', 'ordered implementation units', 'rollback posture', 'validation gates') that a user would not naturally say. Common natural variations ('implementation plan', 'break down the spec', 'sequence tasks', 'delivery plan', 'milestones') are missing, matching the anchor for some relevant keywords but missing common variations or synonyms.

3 / 5

Distinctiveness Conflict Risk

It carves a clear niche (pre-implementation planning from approved sources) with distinctive vocabulary ('Harness Engineering execution plans', 'issue slices', 'rollback posture'). Minor overlap risk remains with closely related sibling skills in the same family (he-spec, he-strategy, he-linear-plan are referenced in the body), since a generic 'plan this' request could plausibly route to a sibling — matching 'mostly distinct; minor overlap risk with closely related skills' rather than the minimal-conflict anchor.

4 / 5

Total

15

/

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

metadata_version

'metadata.version' is missing

Warning

Total

15

/

16

Passed

Repository

jscraik/Agent-Skills

Path

Plugins/harness-engineering/skills/he-plan/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.