CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

migration-migrate-module

Use when a complete module needs migration to the target architecture - runs 6 phases with approval gates.

Quality

76%

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 ./packs/astro/skills/migration-migrate-module/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

86%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 is a tight, well-sequenced phase checklist with specific paths, file conventions, and explicit validation and approval checkpoints after every phase. Its weaknesses are narrow: validation commands are named but not specified, there is no stated recovery loop when a phase's validation fails, and the undefined "adapter" in Phase 2 relies on the referenced architecture doc.

Suggestions

Specify the exact validation commands instead of "Validate build/check/tests", e.g. `npm run build && npm run check && npm test`, so each gate is executable verbatim.

Add an explicit error-recovery loop: "If validation fails, fix the issues, re-run validation, and only request approval once it passes."

Define or link what "adapter" means in Phase 2 (e.g., point to the specific section of docs/ARCHITECTURE.md covering the adapter contract).

DimensionReasoningScore

Conciseness

The ~17-line body is lean: each phase is one line with terse, load-bearing specifics ("no try/catch, fetch only", "Static -> .astro (zero JS)"), architecture details are delegated to `docs/ARCHITECTURE.md` rather than restated, and nothing explains concepts Claude already knows. This matches 'Lean and efficient; every token earns its place'; there is no over-explanation that would justify 4.

5 / 5

Actionability

Concrete guidance dominates: exact directories (`src/modules/[name]/`, `src/islands/`), exact file patterns (`.types.ts`, `.contracts.ts`), and enforceable rules (`client:*`, "zero JS", "fetch only") — fitting 'Mostly executable guidance; concrete code or commands with minor gaps'. It stops short of 5 because "Validate build/check/tests" gives no actual commands and Phase 2's "adapter" is undefined.

4 / 5

Workflow Clarity

Phases 0-6 are clearly numbered and sequenced, with explicit checkpoints ("Validate build/check/tests after each phase", "Ask for approval before each phase") that satisfy the validation requirement for batch/destructive work. It sits between 4 and 5: the explicit error-recovery feedback loop required by the 5 anchor (what to do when validation fails — fix, re-validate, re-approve) is absent.

4 / 5

Progressive Disclosure

The skill is under 50 lines with well-organized sections and no bundle files; the per the simple-skill guideline this can score 5 with just well-organized sections. Architecture detail is correctly pushed out to a clearly signaled one-level reference (`docs/ARCHITECTURE.md`) and nothing that belongs in a separate file is inlined.

5 / 5

Total

18

/

20

Passed

Description

57%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 has a clear, explicit 'Use when' trigger and a distinguishable niche, but it is thin on substance: it never names the target architecture (Astro) and describes the work only as "runs 6 phases with approval gates" rather than what those phases do. Trigger coverage is serviceable but misses the most natural terms a user would say.

Suggestions

Name the target architecture and the concrete phase actions, e.g. "Migrate a complete module to the Astro architecture — structure, types, services, routing, pages, and islands — in gated phases. Use when a module needs porting or rewriting to Astro."

Add natural trigger terms and synonyms users would actually say: "Astro", "port", "rewrite", "refactor", "SPA to file-based routing".

Fix the phase-count mismatch: the description says "6 phases" but the body lists Phase 0 through Phase 6, which is 7 phases.

DimensionReasoningScore

Specificity

"runs 6 phases with approval gates" names the domain ("module needs migration to the target architecture") but the only action described is the generic shape of the process — what the phases actually do (structure, types, services, routing, components) is never stated. This matches the anchor 'Names the domain but actions are minimal or generic' rather than 3, which requires 1-2 concrete capabilities; it is also above 1 since it is not pure abstraction.

2 / 5

Completeness

Both parts are present: the what ("runs 6 phases with approval gates" for module migration) and an explicit when ("Use when a complete module needs migration to the target architecture"). It fits anchor 4 — 'when' could be more explicit or specific (it never names the target architecture or concrete trigger phrases) — and does not reach 5, whose example lists multiple concrete trigger phrases and a fuller statement of capabilities.

4 / 5

Trigger Term Quality

Relevant keywords are present ("module", "migration", "complete module") and "Use when a complete module needs migration" is natural phrasing, but common variations are missing: the architecture name ("Astro", which the body targets), and synonyms like "port", "rewrite", or "refactor". This matches 'Some relevant keywords but missing common variations or synonyms'; it falls short of 4's 'good keyword coverage' because the single most distinctive term (Astro) is absent.

3 / 5

Distinctiveness Conflict Risk

"a complete module needs migration" carves out a distinct niche (whole-module migration, gated in phases) that would not collide with unrelated skills, fitting 'Mostly distinct; minor overlap risk'. It is not 5 because "target architecture" is unnamed, so requests to migrate or port something smaller or to a different target could plausibly pull this skill in; it is above 3, whose examples are far more generic.

4 / 5

Total

13

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

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

14

/

16

Passed

Repository

fernandezbaptiste/specialist-agent

Path

packs/astro/skills/migration-migrate-module/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.