CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

aif-implement

Execute implementation tasks from the current plan. Works through tasks sequentially, marks completion, and preserves progress for continuation across sessions. Use when user says "implement", "start coding", "execute plan", or "continue implementation".

Quality

79%

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/aif-implement/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

67%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, highly actionable orchestration workflow with excellent reference hygiene — every cited bundle file exists, is one level deep, and carries an explicit read condition. Its weaknesses are redundancy (duplicated rule lists and restated description) and an overloaded entrypoint that inlines detail better suited to references, plus a few coherence gaps like the undefined "Step 0.2" and a Bash instruction absent from allowed-tools.

Suggestions

Consolidate the DO/DON'T and Critical Rules sections into a single rule list — seven items (in_progress marking, no tests, no reports, no added tasks, no skipped tasks, no false completion, one-at-a-time) appear in both, and drop the line-31 restatement of the frontmatter description.

Move the inline receipt-reconciliation rules (the 'Never offer an ordinary commit or stash...' block) fully into references/IMPLEMENT-COMMITS.md and keep only the routing instruction in the body, mirroring how other references are handled.

Fix coherence gaps: define or renumber 'Step 0.2' as an actual section (it is referenced but never appears as a heading), and reconcile the 'use the Bash tool' env-check instruction with the allowed-tools frontmatter, which does not include Bash.

DimensionReasoningScore

Conciseness

The body mostly assumes competence (no basic-concept explanations), but carries clear redundancy: line 31 restates the frontmatter description verbatim in function ("Execute tasks from the plan, track progress, and enable session continuation"), the DO/DON'T lists are duplicated almost item-for-item by the "Critical Rules" list (in_progress marking, no tests, no reports, one task at a time), and the receipt-reconciliation rules (lines 141-160) restate detail that the body itself says lives in references/IMPLEMENT-COMMITS.md. This fits 'mostly efficient but could be tightened' rather than anchor 4's 'minor instances'.

3 / 5

Actionability

Concrete, executable guidance throughout: exact shell commands ("printenv HANDOFF_MODE || true"), exact tool names and state transitions ("handoff_sync_status → 'implementing' with paused: true"), exact markers ("<!-- aif:task-kind:commit -->", "<!-- aif:plan-mode:ultra -->") and file patterns ("[0-9]{4}_<branch-slug>.md"). Below anchor 5 due to minor gaps: MCP call signatures are delegated to references rather than shown, and the env-check instructs "use the Bash tool" even though Bash is not in the skill's allowed-tools list.

4 / 5

Workflow Clarity

The workflow is clearly sequenced (Step 0 pre → Step 0 → Steps 1-2 → task execution → persistence → completion) with explicit validation gates and STOP conditions (requirement conflicts, malformed bundles, unresolved receipts, merge conflicts) plus error-recovery outcomes ("ERROR [requirement-conflict]", "blocked_external"). Below anchor 5 due to coherence gaps: "Step 0.2" is referenced ("see Step 0.2 (after reading the plan file)") but never defined as a section in the body, and the route branching (list/inline/recovery/plan-backed) uses dense forward references that are hard to follow.

4 / 5

Progressive Disclosure

All 12 references cited in the body exist in ./references/, are one level deep, and each is paired with an explicit load condition ("read only when the user needs command syntax examples"), which is strong signaling. Below anchor 5 because the body is a ~270-line dispatch document with substantial procedural detail inlined (receipt-reconciliation rules, the command-discovery contract, config resolution details) that would more appropriately live in the reference files, leaving the entrypoint a leaner overview.

4 / 5

Total

15

/

20

Passed

Description

83%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 strong description with clear third-person capability statements, explicit 'Use when' trigger phrases in the user's natural vocabulary, and a distinct plan-execution niche. The only weaknesses are minor: trigger coverage omits some common synonyms, and the capability list could enumerate a bit more of the workflow.

DimensionReasoningScore

Specificity

"Execute implementation tasks from the current plan. Works through tasks sequentially, marks completion, and preserves progress for continuation across sessions" lists several concrete third-person actions (sequential execution, completion marking, cross-session progress preservation). It falls short of anchor 5 because coverage has minor gaps (no mention of verification, commit handling, or status reporting), but exceeds anchor 3 with more than 1-2 actions.

4 / 5

Completeness

The description explicitly answers both what ("Works through tasks sequentially, marks completion, and preserves progress") and when with concrete quoted trigger phrases ("Use when user says 'implement', 'start coding', 'execute plan', or 'continue implementation'"). This matches the anchor-5 example structure; anchor 4's 'when could be more explicit' does not apply since the triggers are quoted verbatim.

5 / 5

Trigger Term Quality

"implement", "start coding", "execute plan", "continue implementation" are natural phrases users would say, giving good keyword coverage. Not anchor 5 because common synonyms/variants ("build", "resume", "run the tasks", "pick up where we left off") are missing; not anchor 3 because coverage goes beyond merely relevant keywords.

4 / 5

Distinctiveness Conflict Risk

"Execute implementation tasks from the current plan" carves a distinct plan-backed implementation niche, and triggers like "execute plan" and "continue implementation" are unlikely to fire for unrelated skills. Minor overlap risk remains with closely related plan/code skills in the same family (fix, verify) since "implement" and "start coding" are broad, keeping it below anchor 5.

4 / 5

Total

17

/

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

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

Total

15

/

16

Passed

Repository

lee-to/ai-factory

Path

skills/aif-implement/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.