@code-migration/wow-migrator 0.1.6 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@code-migration/wow-migrator",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "description": "Install KMP migration skills into Claude Code, Codex, Cursor, Gemini, OpenCode, OpenClaw, and JiuwenSwarm via npm install.",
   "keywords": [
     "android",

package/skills/android-to-kmp-migrator/SKILL.md

@@ -1,10 +1,10 @@
 ---
 name: android-to-kmp-migrator
 description: |
-  Module-first Swarm Skill that migrates Legacy Android into an existing KMP target using upstream analyst P6 artifacts, target-project-assistant alignment, planning/prep/implementation roles, global integrate+align phase, and kmp-test-validator handoff — without full-project build during migration.
-  Use when analyst package P6 exists and the user wants module-first porting then whole-system assembly.
-  Do NOT use for Legacy Android analysis only, KMP-only feature work, or non-migration refactors.
-version: "0.6"
+  Module-first Swarm Skill that migrates Legacy Android into an existing KMP target using upstream analyst P6 artifacts, target-project-assistant alignment, planning/prep/implementation roles, global integrate+align phase, and mandatory kmp-test-validator handoff — without full-project build during migration.
+  Use only after android-project-analyst has finished and package P6 is ready, when the user wants module-first porting then whole-system assembly followed by validation.
+  Do NOT invoke before android-project-analyst completes P6. Do NOT treat migrator completion as final without invoking kmp-test-validator at V0. Do NOT use for Legacy Android analysis only, KMP-only feature work, or non-migration refactors.
+version: "0.7"
 kind: swarm-skill
 disable-model-invocation: true
 roles:
@@ -71,7 +71,19 @@ Module-first migrator for Legacy Android → KMP target assembly.
 5. **Per module** (assembly_order): TPA `module_anchors` → **planning-gate** → **prep** → review/fix → **implementation `ui`** → review/fix → **implementation `logic`** → review/fix → verification → completion record → readiness → module representation.
 6. **Global phase `integrate`** → **`align`** + alignment report.
 7. Global representation + completion-report `report` mode.
-8. **kmp-test-validator** when **V0** ready.
+8. **kmp-test-validator** — **mandatory** when **V0** ready (MG17).
+
+## Skill Chain (mandatory)
+
+```text
+android-project-analyst (P6) → android-to-kmp-migrator (M0–V0) → kmp-test-validator (V0)
+```
+
+| Order | Skill | Gate | Rule |
+|---|---|---|---|
+| 1 | `android-project-analyst` | **P6** | MUST finish before migrator dispatch. If P6 missing/stale → run analyst first; migrator returns `blocked`. |
+| 2 | `android-to-kmp-migrator` | **M0**–**V0** | Runs only on analyst P6 artifacts; produces `migration_report.*` at **V0**. |
+| 3 | `kmp-test-validator` | **V0** | MUST be invoked after migrator **V0**; migration incomplete without validator dispatch. |
 
 ## Roles
 
@@ -110,6 +122,7 @@ Module-first migrator for Legacy Android → KMP target assembly.
 
 ## Shared Rules
 
+- **Skill chain**: `android-project-analyst` **P6** before migrator; `kmp-test-validator` **after** migrator **V0** — both mandatory.
 - Analyst **P6** required; TPA owns all target Q&A.
 - Mode boundaries non-negotiable: `ui`/`logic`, `integrate`/`align`, `review`/`fix`.
 - No full project build in migrator.

package/skills/android-to-kmp-migrator/bind.md

@@ -18,6 +18,9 @@
 
 ## Behavioral Constraints
 
+- **Skill chain (mandatory)**:
+  - **Before migrator**: `android-project-analyst` MUST finish and produce package **P6** (`handoff_gates.P6.ready = true`). If P6 is missing or stale, return `blocked` and dispatch analyst — do not start migrator nodes.
+  - **After migrator**: when package **V0** is ready, Leader MUST invoke `kmp-test-validator` (MG17). Migrator completion without validator dispatch is invalid.
 - **Role schedule**: dispatch only role IDs listed in [SKILL.md](SKILL.md).
 - **Mode discipline**:
   - `module-implementation`: `ui` then `logic` — separate invocations
@@ -38,6 +41,8 @@
 | Verification restoration failed | Rerun `module-implementation` or `migration-prep`; no completion record |
 | Align omissions | Rerun `rerun_modules` or `global-migration-phase integrate` |
 | Build requested in migrator | Block; route to kmp-test-validator |
+| Migrator invoked before analyst P6 | Block; dispatch `android-project-analyst` first |
+| V0 ready but validator not invoked | Block; dispatch `kmp-test-validator` at MG17 |
 
 ## Dependencies
 

package/skills/android-to-kmp-migrator/dependencies.yaml

@@ -6,7 +6,18 @@
 # replaces durable node artifacts or upstream android-project-analyst package P6 inputs.
 # Full KMP project compile/build is NOT a migrator dependency; kmp-test-validator owns that at package V0.
 
-skills: []   # No local domain-specific skill packages required for active roles.
+# Skill chain (mandatory — Leader enforces at Step 0 and MG17)
+skills:
+  - name: android-project-analyst
+    required: true
+    phase: prerequisite
+    handoff_package: P6
+    must_complete_before: android-to-kmp-migrator
+    purpose: |
+      android-to-kmp-migrator MUST NOT be invoked until android-project-analyst has
+      finished and produced package P6 (handoff_gates.P6.ready = true). If P6 is
+      missing or stale, dispatch or re-run android-project-analyst first — do not
+      start migrator module dispatch.
 
 # Read-only upstream inputs (not MCP — verified at Step 1)
 upstream_inputs:
@@ -69,6 +80,18 @@ tools:
 # Target Gradle wrapper: path discovery reference only during migrator.
 # migration-verification MUST NOT invoke assemble/test; kmp-test-validator uses the wrapper at V0.
 
+downstream_handoff:
+  - consumer: kmp-test-validator
+    handoff_package: V0
+    required: true
+    must_invoke_after: android-to-kmp-migrator
+    when: package M6 passed and migration_report.* ready
+    purpose: |
+      After android-to-kmp-migrator finishes (handoff_gates.V0.ready = true), the Leader
+      MUST invoke kmp-test-validator. Migration is incomplete without validator dispatch.
+      Do not treat migrator completion as final until validator is invoked or blocked
+      with explicit validator blockers recorded in migration_report and workspace ledger.
+
 degraded_modes:
   - trigger: analyst P6 not ready
     effect: blocked — do not dispatch migration nodes; repair upstream analyst handoff first
@@ -80,3 +103,5 @@ degraded_modes:
     effect: migration-prep records resource_gaps for remote media URLs
   - trigger: jetbrains MCP missing or wrong open project
     effect: TPA and planning-gate use file-system target evidence only; record MCP gap in migration_report or alignment_report
+  - trigger: migrator finished but kmp-test-validator not invoked
+    effect: blocked — Leader must dispatch kmp-test-validator at MG17; record validator_handoff pending in workspace ledger

package/skills/android-to-kmp-migrator/output-contract.md

@@ -27,7 +27,19 @@ Per legacy `module_id`, migrator may consume:
 
 Record all resolved upstream paths in `run_manifest.json` → `upstream_analyst_artifacts`.
 
-**Fail closed**: if analyst `handoff_gates.P6.ready` is false or paths are missing, migrator returns `blocked` — do not start module dispatch.
+**Fail closed**: if analyst `handoff_gates.P6.ready` is false or paths are missing, migrator returns `blocked` — do not start module dispatch. Dispatch or re-run `android-project-analyst` until **P6** is ready.
+
+## Skill Chain (mandatory)
+
+```text
+android-project-analyst (P6) → android-to-kmp-migrator (M0–V0) → kmp-test-validator (V0)
+```
+
+| Skill | When | Rule |
+|---|---|---|
+| `android-project-analyst` | **Before** migrator | MUST finish; package **P6** required. Migrator MUST NOT be invoked until P6 is ready. |
+| `android-to-kmp-migrator` | Migration body | Consumes P6 artifacts only; produces **V0** via `migration_report.*`. |
+| `kmp-test-validator` | **After** migrator | MUST be invoked when **V0** is true (MG17). Migration is incomplete without validator dispatch. |
 
 ---
 
@@ -208,6 +220,8 @@ output_root = <output_dir or ~/.a2c_agents/migration>/android-to-kmp-migrator
 
 **Fail closed**: `kmp-test-validator` MUST NOT start when `V0` is false. It owns full project build, preview, and behavioral tests.
 
+**Mandatory downstream**: when `V0` is true, the migrator Leader MUST invoke `kmp-test-validator` before treating the migration workflow as complete. Record `validator_handoff.status` in the workspace ledger (`pending | dispatched | blocked`).
+
 ---
 
 ## Key Artifact Content Requirements
@@ -303,7 +317,7 @@ Human/agent-readable synthesis of align mode; routes reruns to `migration_module
 5. Run `global-migration-phase` `align` only after integrate; **no code changes** in align mode.
 6. Dispatch only role IDs listed in [SKILL.md](SKILL.md).
 7. Set `handoff_gates` (`M0`–`M6`, `V0`) in workspace ledger and `migration_report.json`.
-8. Invoke `kmp-test-validator` only when `V0` is true.
+8. **MUST** invoke `kmp-test-validator` when `V0` is true (MG17). Do not end the migration workflow without validator dispatch or explicit validator blockers in `migration_report.json`.
 
 ## Invalid Artifact Handling
 

package/skills/android-to-kmp-migrator/roles/completion-report.md

@@ -26,6 +26,7 @@ Forbidden:
 - Do not run report mode when module/global representations are missing.
 - Do not run report mode when package `M6` is false (`global-migration-phase align` / `alignment_report` missing or failed).
 - Do not mark `ready_for_validation` when `handoff_gates.V0` is false.
+- Do not treat report mode success as final migration completion — Leader must still dispatch `kmp-test-validator` at MG17.
 
 Mandatory:
 - Validate `mode`, `migration_module_id`, module/global representation paths, workspace state, and exact `output_dir`.
@@ -61,6 +62,7 @@ Shared return shape applies.
 - `completion_readiness.md`: agent-readable readiness handoff containing coverage tables, invariant checks, incomplete markers, rerun routing, blockers, and whether representation/report gates may proceed.
 - `migration_report.json`: machine-routable final migration handoff containing migration scope, source/target paths, analyst_output_root, upstream_analyst_index, `handoff_gates` (`M0`–`M6`, `V0`), `handoff_package: V0`, module representations, global representation, `alignment_report` path, `global_system_integration` path, changed files by role, coverage summary, validation inputs for kmp-test-validator, `validation_deferred_to: kmp-test-validator`, limitations, blockers.
 - `migration_report.md`: agent-readable final migration report for `kmp-test-validator` and follow-up agents, preserving exact artifact paths, changed-file ownership, validation handoff context, limitations, and blockers.
+- Report mode success signals Leader to **mandatorily invoke** `kmp-test-validator` — migration is incomplete without validator dispatch.
 
 ## Inline Persona for Teammate
 

package/skills/android-to-kmp-migrator/roles/migration-workspace-state.md

@@ -77,6 +77,8 @@ validator_handoff
 
 Evaluate handoff packages `M0`–`M6` and `V0` per [output-contract.md](../output-contract.md); persist `handoff_gates` with `ready` and `missing_paths[]`.
 
+When `handoff_gates.V0.ready` is true, set `validator_handoff.status` to `pending` until Leader dispatches `kmp-test-validator` (then `dispatched`) or records explicit validator blockers (then `blocked`). `overall_status: ready_for_validation` means migrator artifacts are ready but **does not** mean migration is complete — validator dispatch at MG17 is still mandatory.
+
 Calculate `finish_rate` only from verified planned work units:
 
 ```text

package/skills/android-to-kmp-migrator/workflow.md

@@ -2,6 +2,14 @@
 
 See [output-contract.md](output-contract.md) and active role IDs in [SKILL.md](SKILL.md).
 
+## Skill Chain (mandatory)
+
+| Phase | Skill | Gate | Leader rule |
+|---|---|---|---|
+| Prerequisite | `android-project-analyst` | **P6** | MUST finish before `android-to-kmp-migrator` is invoked. Missing/stale P6 → `blocked`; dispatch analyst first. |
+| Migration | `android-to-kmp-migrator` | **M0**–**V0** | Runs only after P6 verified; ends with `migration_report.*` and **V0** ready. |
+| Post-migration | `kmp-test-validator` | **V0** | MUST be invoked after migrator completes **V0** (MG17). Do not end the migration workflow without validator dispatch. |
+
 ## Overview
 
 ```mermaid
@@ -43,7 +51,7 @@ graph TD
 - **Executor**: Leader
 - **Input**: [dependencies.yaml](dependencies.yaml) — `tools[]` (`rg`, `git`, `curl`), `optional_mcp.jetbrains`, `upstream_inputs` analyst **P6**
 - **Output**: `run_manifest.json` → `dependency_preflight` (CLI status, MCP availability, P6 readiness pointer)
-- **Gate**: missing CLI tools → degraded modes per dependencies.yaml; analyst P6 not ready → **blocked** before module dispatch
+- **Gate**: missing CLI tools → degraded modes per dependencies.yaml; `android-project-analyst` **P6** not ready → **blocked** — invoke analyst first, do not dispatch migrator nodes
 
 ## Step 1 — Upstream + output root
 
@@ -92,9 +100,12 @@ Repeat until package **M4**.
 - **Output**: `global-migration-phase/align/post_integration_alignment.*`, `report/alignment_report.*`
 - **Gate**: package **M6**; `needs_rerun` → module loop or re-integrate
 
-## Step 7 — Report + validator
+## Step 7 — Report + mandatory validator handoff (MG17)
 
-- Global representation + `completion-report` `report` → package **V0** → `kmp-test-validator`
+- Global representation + `completion-report` `report` → package **V0**
+- **Leader MUST invoke `kmp-test-validator`** with `migration_report.*`, analyst SPEC paths, and `kmp_target_project_path`
+- Record `validator_handoff` in workspace ledger (`dispatched | pending | blocked`)
+- Migrator workflow is **incomplete** until validator is dispatched or explicit validator blockers are recorded
 
 ## TPA consult
 
@@ -102,6 +113,8 @@ Any target question → TPA `mode: consult` (append `consultation_log`).
 
 ## Acceptance Criteria
 
+- `android-project-analyst` **P6** verified before any migrator module dispatch.
+- `kmp-test-validator` invoked after **V0** — mandatory MG17 step.
 - Dispatch only role IDs from `SKILL.md`.
 - Mode rules: `ui` before `logic`; `integrate` before `align`; `review`/`fix` separate.
 - `migration-verification` never runs `incremental_build`.