CtrlK
BlogDocsLog inGet started
Tessl Logo

aif-verify

Verify completed implementation against the plan. Checks that all tasks were implemented, nothing was forgotten, code compiles, tests pass, and quality standards are met. Use after "/aif-implement" completes, or when user says "verify", "check work", "did we miss anything".

68

Quality

85%

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

SKILL.md
Quality
Evals
Security

Quality

Content

81%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.

An exceptionally actionable, well-sequenced verification workflow with real validation checkpoints and feedback loops, backed by genuine one-level-deep reference files. Its main cost is length and redundancy — several rule blocks are stated twice and the body carries detail that the reference contract already owns.

Suggestions

Deduplicate the plan-discovery rules stated in both Step 0.0 (workflow.plan_id_format) and Step 0.2, keeping one canonical location and referencing it from the other.

Fix the duplicate step numbering — '0.3 Read Plan & Tasks' and '0.3 Gather Changed Files' — and merge or delete the redundant 'Compatibility wording for the same rule' paragraph that restates the research-drift rule immediately above it.

Move the per-gate pass/warn/fail criteria repeated in Step 3.5 and the SHA256 verification procedure into the existing references (or a new one-level-deep reference file), leaving the body with the decision rules and a pointer.

DimensionReasoningScore

Conciseness

The body is dense and operational (commands, config keys, paths) rather than explaining things Claude already knows, but it is noticeably redundant: plan-discovery rules appear in both Step 0.0 and Step 0.2, ui_language handling is stated twice, two different steps are both numbered 0.3, and the drift-compatibility wording paragraph ("Compatibility wording for the same rule...") restates the rule above it. This fits anchor 3's "mostly efficient but could be tightened" rather than anchor 4, since whole paragraphs, not just minor phrasings, could be cut.

3 / 5

Actionability

Guidance is fully executable throughout: exact build/test/lint command tables keyed to detection files ("tsconfig.json → npx tsc --noEmit"), concrete grep patterns ("[T][O][D][O]|[F][I][X][M][E]|HACK..."), a copy-paste report template, a complete aif-gate-result JSON example, and ready-made AskUserQuestion scripts. This matches anchor 5's copy-paste-ready coverage of the common cases.

5 / 5

Workflow Clarity

Steps 0 through 5 are clearly sequenced with explicit validation checkpoints (build/test/lint checks, per-gate pass/warn/fail criteria, plan-integrity blocking failures) and feedback loops ("After fixing, re-run the relevant verification checks to confirm"), plus a task-status checklist in the report. The duplicate 0.3 numbering is a cosmetic defect, not a missing checkpoint, so this stays at anchor 5 rather than 4.

5 / 5

Progressive Disclosure

Both referenced bundle files (references/CONTEXT-GATES-AND-OWNERSHIP.md, references/GATE-RESULT-CONTRACT.md) exist and are clearly signaled in Step 0.1 with an explicit precedence rule ("If this contract conflicts with older examples in this file, follow the contract"), and Step 3.5 defers to the contract. It falls short of anchor 5 because the ~615-line body still inlines substantial material the references own (per-gate pass/warn/fail criteria repeated in 3.5, detailed plan-discovery and SHA256 hashing procedures) that could itself be split out one level deep.

4 / 5

Total

17

/

20

Passed

Description

88%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 that clearly states what the skill does and when to use it, with mostly natural trigger phrases. Its only weaknesses are a slightly generic "quality standards" clause and trigger terms that could collide with sibling review/QA skills.

DimensionReasoningScore

Specificity

The description lists multiple concrete actions — "Checks that all tasks were implemented, nothing was forgotten, code compiles, tests pass, and quality standards are met" — giving comprehensive coverage of the verification workflow. Only "quality standards are met" is generic; per anti-drift review this sits at the anchor-5 example's level rather than anchor 4, since nearly every capability is named specifically.

5 / 5

Completeness

It explicitly answers both questions: the what ("Verify completed implementation against the plan. Checks that all tasks were implemented...") and the when with concrete trigger phrases ("Use after '/aif-implement' completes, or when user says 'verify', 'check work', 'did we miss anything'"). This matches the anchor-5 example structure exactly.

5 / 5

Trigger Term Quality

"verify", "check work", and "did we miss anything" are natural phrases a user would actually say, plus the workflow anchor "/aif-implement completes". It falls below anchor 5 because common variations like "review the work", "is it done", or "did we finish everything" are missing, but it is clearly above anchor 3's partial coverage.

4 / 5

Distinctiveness Conflict Risk

The plan-verification niche and the /aif-implement tie-in make it mostly distinct from unrelated skills, matching anchor 4's "minor overlap risk with closely related skills". It is not anchor 5 because broad triggers like "check work" and "did we miss anything" could plausibly fire for general code-review or QA skills in the same suite.

4 / 5

Total

18

/

20

Passed

Validation

81%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation13 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

skill_md_line_count

SKILL.md is long (618 lines); consider splitting into references/ and linking

Warning

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

frontmatter_unknown_keys

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

Warning

Total

13

/

16

Passed

Repository
lee-to/ai-factory
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.