CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

spec-loop-prepare-execution-approval

Mandatory only on the task-file path of `spec-loop-plan-task`. Use when an active task file already exists and the next user-facing action would otherwise present task-file work for evaluation, feedback, review, or execution approval.

Quality

56%

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

Fix and improve this skill with Tessl

tessl review fix ./skills/spec-loop-prepare-execution-approval/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.

A lean, well-sequenced orchestration skill with explicit fix-and-re-run gating, clear sibling-skill handoffs, and good guard conditions. Its main weaknesses are repetition (near-duplicate readiness-check paragraphs and repeated pointers to the same guidance file) and the fact that its authoritative guidance file is not present in the bundle, leaving the operational core unreachable from the skill alone.

Suggestions

Merge the two near-identical readiness-check paragraphs ('directly fixable task defects' / 'issues that can be resolved from existing evidence') and consolidate the three consecutive directives pointing at execution-approval-guidance.md into a single authoritative-source statement.

Ship execution-approval-guidance.md in the skill's bundle (or inline its readiness-check list) so the delegated authoritative content is actually reachable, and verify the sibling skill paths (../spec-loop-plan-task/SKILL.md, ../spec-loop-compact-task-file/SKILL.md, ../spec-loop-clarify-task/SKILL.md, ../spec-loop-implementation-flow/SKILL.md) resolve.

Name the readiness checks and allowed user-facing outcomes in one or two lines each in SKILL.md so the fix-and-re-run loop remains actionable even before opening the guidance file.

DimensionReasoningScore

Conciseness

The body teaches no concepts Claude already knows and contains no padded prose, but it repeats itself: [execution-approval-guidance.md] is directed at four times ('Then read...', 'Use ... as the authoritative source...', 'Follow ... for the allowed user-facing outcomes...', 'handled specially by...'), and 'If readiness checks find directly fixable task defects, update the active task in place and re-run the readiness checks before producing user-facing output' is nearly duplicated by 'If readiness checks find issues that can be resolved from existing evidence, update the active task in place and re-run the readiness checks...'. This fits anchor 3 (mostly efficient but could be tightened) rather than 4, because the duplicate paragraph and consecutive same-target pointers are real trimmable content, and not 2, because there is no over-explanation.

3 / 5

Actionability

Routing directives are concrete and executable — 'Update the same active task file for user-facing review readiness. Do not create a second task artifact', 'Invoke [spec-loop-clarify-task/SKILL.md] in the same turn, then resume this skill after clarification' — with explicit branch conditions. However, the operational core ('readiness checks, diagram-first preparation, duplication removal, user-facing review gating, and approval-seeking output') is entirely delegated to execution-approval-guidance.md, which is not present in this bundle, so the skill alone cannot execute its purpose. Fits anchor 3 (some concrete guidance but incomplete, key details missing); not 4 because the gap is central rather than minor, not 2 because the routing-level instructions are specific commands, not high-level hints.

3 / 5

Workflow Clarity

The process is coherently sequenced: read prerequisites ('Before work: read [spec-loop-plan-task/SKILL.md] ... Then read [execution-approval-guidance.md]'), update the active task file, run readiness checks, then fix-and-re-run loops ('update the active task in place and re-run the readiness checks before producing user-facing output'), compaction and clarification detours with explicit resume points, and post-approval routing ('implementation work is governed by [spec-loop-implementation-flow/SKILL.md]; investigation work stays under the spec-loop-plan-task task-file path'). Explicit feedback loops and a guard condition ('If no active task file exists ... do not use this skill') are present. Fits anchor 4 (clear sequence, most checkpoints present); not 5 because the sequence is implied by paragraph order rather than enumerated and the checkpoints' substance lives in the absent guidance file; not 3 because fix→re-run gating is explicit, not implicit.

4 / 5

Progressive Disclosure

The router pattern is correct — a short SKILL.md delegating detail one level deep, with each reference clearly signaled and its purpose stated ('as the authoritative source for readiness checks, diagram-first preparation, duplication removal, user-facing review gating, and approval-seeking output'). But the bundle contains only SKILL.md (no references/, scripts/, or assets/ directories), so the primary referenced file execution-approval-guidance.md is absent and the skill's first instruction ('Then read [execution-approval-guidance.md]') dead-ends. Fits anchor 3 (structure present but undermined) rather than 4, because a missing primary reference is more than a minor organization gap, and not 2, because nothing is improperly inlined and references are prominent rather than buried.

3 / 5

Total

13

/

20

Passed

Description

40%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 precisely scoped about when and where the skill applies — an explicit 'Use when...' clause conditioned on an active task file and an imminent approval-seeking action — but it never states what the skill actually does, forcing readers to infer its purpose from the name. Trigger keywords are appropriate to the niche but jargon-heavy and lacking natural user-side variations.

Suggestions

Lead with a concrete 'what' statement (e.g., 'Prepares the active task file for user-facing review and gates execution until the user approves it') before the 'Use when...' clause so the description answers both what and when.

Add natural user-side trigger phrases (e.g., 'approve execution', 'sign off on the task', 'task ready for review') alongside the existing system terms to broaden trigger-term coverage.

Reduce jargon such as 'task-file path' in favor of phrasing a user would naturally say, keeping the spec-loop scoping via the explicit skill-name reference.

DimensionReasoningScore

Specificity

The description names its domain — 'the task-file path of `spec-loop-plan-task`' and 'execution approval' — but states no concrete action the skill performs; 'Mandatory only on...' and 'Use when...' are pure scoping/trigger conditions with no capability verbs. Matches anchor 2 (names the domain but actions are minimal or generic); not 3 because no 1-2 concrete skill actions are listed, and not 1 because the domain and situation are specific rather than abstract filler.

2 / 5

Completeness

The 'when' is explicitly and concretely answered ('Use when an active task file already exists and the next user-facing action would otherwise present task-file work for evaluation, feedback, review, or execution approval'), but the 'what' is entirely absent — the description never states what the skill does (prepare, gate, format, review-check). This matches anchor 2's 'only when is present without what'; not 3 because that anchor requires a clear 'what', and not 1 because the 'when' clause is detailed and concrete rather than extremely vague.

2 / 5

Trigger Term Quality

Keywords like 'active task file', 'execution approval', and 'evaluation, feedback, review' are the natural vocabulary inside this spec-loop workflow, but the phrasing leans on system jargon ('task-file path', '`spec-loop-plan-task`') and offers no common variations or user-side phrasings such as 'sign off' or 'ready for review'. Fits anchor 3 (some relevant keywords but missing common variations or synonyms); not 4 because synonym coverage is absent, and not 2 because several genuinely relevant trigger terms are present rather than one or two generic ones.

3 / 5

Distinctiveness Conflict Risk

Hard scoping — 'Mandatory only on the task-file path of `spec-loop-plan-task`' and the condition 'an active task file already exists' — carves a clear niche and prevents triggering for chat-only planning or unrelated work. Minor overlap risk remains from the broad words 'evaluation, feedback, review' shared with sibling spec-loop skills and general review skills. Fits anchor 4 (mostly distinct, minor overlap risk); not 5 because those common trigger words keep some ambiguity, and not 3 because the conditional scoping is much tighter than 'somewhat specific'.

4 / 5

Total

11

/

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

relative_links

Relative link issues: 4 missing, 4 suspicious

Warning

Total

15

/

16

Passed

Repository

dpolivaev/spec-loop

Path

skills/spec-loop-prepare-execution-approval/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.