CtrlK
BlogDocsLog inGet started
Tessl Logo

he-phase-heartbeat

Routes old phase-heartbeat requests into HE phase work: check the approved plan, reuse or create a 10-minute he-heartbeat only with authority, run phase gates, delegate the active slice to he-work, and stop before commit or tracker closure. Use when a user says phase heartbeat, keep this PR moving, or continue phases.

63

Quality

73%

Does it follow best practices?

Impact

No eval scenarios have been run

SecuritybySnyk

Passed

No known issues

Optimize this skill with Tessl

npx tessl skill review --optimize ./Plugins/harness-engineering/skills/he-phase-heartbeat/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

62%

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

This skill provides a well-structured routing/compatibility layer with clear sequencing, explicit stop conditions, and good safety boundaries. Its main weaknesses are moderate verbosity from overlapping sections (Constraints, Execution Boundaries, Failure Mode) and a lack of truly executable/concrete guidance—most steps reference abstract contracts or delegations rather than specific commands. The output template is a strong concrete element that anchors the otherwise abstract procedure.

Suggestions

Consolidate overlapping sections (Constraints, Execution Boundaries, Failure Mode) into a single 'Safety & Boundaries' section to reduce redundancy and improve token efficiency.

Make procedure steps more actionable by specifying exactly how to 'run the phase gate contract' or 'delegate to he-work'—e.g., include the actual invocation pattern or tool call format rather than abstract references.

Remove the Philosophy section or fold its single key insight ('heartbeat does not authorize implementation') into the procedure as a constraint, since Claude can infer the routing purpose from context.

DimensionReasoningScore

Conciseness

The skill is moderately efficient but includes some redundancy—execution boundaries largely restate constraints, and several sections (Failure Mode, Gotchas, Validation Gates) overlap in content. Some sections like 'Philosophy' explain routing concepts Claude could infer from the procedure itself.

2 / 3

Actionability

The procedure provides a clear sequence of steps and the output template gives a concrete YAML example, but there are no executable code/commands—steps like 'run the phase gate contract' and 'delegate to he-work' are abstract references to other processes rather than concrete, copy-paste-ready instructions.

2 / 3

Workflow Clarity

The procedure is clearly sequenced (steps 1-6) with explicit validation checkpoints (step 3 blocks on stale evidence, step 6 is a hard stop boundary), a fail-fast policy in Validation Gates, and a clear failure mode that returns blocked status and names recovery steps. The feedback loop of 'stop at first failed gate' is well-defined.

3 / 3

Progressive Disclosure

References to external contracts and files are listed at the bottom with clear paths, and the Stage Arc Boundary section references an external contract appropriately. However, without bundle files to verify these references exist, and with some inline content (like the full Stage Arc Boundary explanation) that could be deferred to the referenced contract, the disclosure structure is adequate but not optimal.

2 / 3

Total

9

/

12

Passed

Description

85%

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

This description is well-structured with a clear 'Use when' clause and specific actions listed. The main weakness is that the trigger terms are somewhat jargon-heavy and may not cover all natural ways a user might invoke this skill. The domain-specific terminology makes it highly distinctive but potentially limits discoverability for users unfamiliar with the exact terminology.

Suggestions

Consider adding more natural language variations to the trigger terms, such as 'next phase', 'advance PR phases', or 'run the next step' to improve discoverability.

DimensionReasoningScore

Specificity

Lists multiple specific concrete actions: routing phase-heartbeat requests, checking approved plans, creating 10-minute he-heartbeat with authority, running phase gates, delegating active slice to he-work, and stopping before commit or tracker closure.

3 / 3

Completeness

Clearly answers both 'what' (routes phase-heartbeat requests, checks plans, creates heartbeats, runs gates, delegates slices) and 'when' with an explicit 'Use when' clause listing trigger phrases.

3 / 3

Trigger Term Quality

Includes some natural trigger terms like 'phase heartbeat', 'keep this PR moving', and 'continue phases', but these are fairly niche and the description uses internal jargon (HE phase work, he-heartbeat, phase gates, he-work) that users may not naturally say.

2 / 3

Distinctiveness Conflict Risk

Highly specific niche around phase-heartbeat routing and HE phase work with distinct terminology and triggers; very unlikely to conflict with other skills due to the specialized domain vocabulary.

3 / 3

Total

11

/

12

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.

Validation11 / 11 Passed

Validation for skill structure

No warnings or errors.

Repository
jscraik/Agent-Skills
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.