@mmerterden/multi-agent-pipeline 8.6.2 → 10.2.0

package/pipeline/skills/figma-common/performance-swiftui/SKILL.md

@@ -8,7 +8,7 @@ status: wip-v5
 sourced-from: upstream/performance-swiftui
 ---
 
-# /performance-swiftui — Performance-Optimized Figma-to-SwiftUI Workflow
+# /performance-swiftui  -  Performance-Optimized Figma-to-SwiftUI Workflow
 
 > Based on `/figma-to-swiftui`. All phase instruction files, scripts, and reference docs
 > are shared from `.instructions/figma/figma-to-swiftui/`. Only the orchestrator routing,
@@ -31,7 +31,7 @@ sourced-from: upstream/performance-swiftui
 
 ### Lean Mode (`--lean`)
 
-When `--lean` is appended to a Figma URL, the workflow runs **Phases 0–4 + Phase 6 only**.
+When `--lean` is appended to a Figma URL, the workflow runs **Phases 0-4 + Phase 6 only**.
 Phase 5 (testing) and Phase 7 (wiki) are **skipped**.
 
 Lean mode is persisted in the `_perf_tracker.json` tracker:
@@ -62,7 +62,7 @@ Convert `-` to `:` in node-id (e.g., `9419-30700` → `9419:30700`).
 
 ### Tracker File
 
-All performance metrics are stored in `{artifactPath}/_perf_tracker.json`. This file is the **single source of truth** — never rely on in-memory variables for timing or reliability data.
+All performance metrics are stored in `{artifactPath}/_perf_tracker.json`. This file is the **single source of truth**  -  never rely on in-memory variables for timing or reliability data.
 
 **At workflow start** (Phase 0 or first resumed phase), create/update the tracker file:
 
@@ -114,7 +114,7 @@ with open(tracker_path, "w") as f:
     json.dump(tracker, f, indent=2)
 ```
 
-**CRITICAL:** The tracker file write happens IMMEDIATELY after each phase dispatch returns — before proceeding to the next phase. This is the mechanism that survives context loss.
+**CRITICAL:** The tracker file write happens IMMEDIATELY after each phase dispatch returns  -  before proceeding to the next phase. This is the mechanism that survives context loss.
 
 ---
 
@@ -141,7 +141,7 @@ These 6 rules govern orchestrator behavior. Violations have caused repeated fail
 
 In lean mode, Phase 5 (testing) and Phase 7 (wiki) are **SKIPPED**. Phase 6 runs sequentially (no parallel partner).
 
-This skill dispatches Phase 2 and Phase 5 sub-phases directly — no orchestrator subagent.
+This skill dispatches Phase 2 and Phase 5 sub-phases directly  -  no orchestrator subagent.
 
 When `phase-7` is invoked:
 1. Dispatch `phase-7a` (generate two wiki pages: main + iOS, plus assets into wiki submodule).
@@ -228,11 +228,11 @@ You are executing {Phase Name} of the Figma-to-SwiftUI workflow.
 - Artifact Path: {artifactPath}
 - Component Location: {componentLocation}
 - GitHub Issue: {issueNumber} (omit if not available)
-- Wiki Path: {wikiPath} (Phase 7 only — `{repos.wiki.path}`)
-- Specs: {specsPath} (if `{artifactPath}/pass0/specs.md` exists — include in Phase 3, 4 dispatch prompts as additional implementation guidance)
+- Wiki Path: {wikiPath} (Phase 7 only  -  `{repos.wiki.path}`)
+- Specs: {specsPath} (if `{artifactPath}/pass0/specs.md` exists  -  include in Phase 3, 4 dispatch prompts as additional implementation guidance)
 
 ## Tool Restrictions
-{from dispatch table — e.g., "Do NOT use the Edit tool."}
+{from dispatch table  -  e.g., "Do NOT use the Edit tool."}
 
 ## Your Instructions
 Read and STRICTLY FOLLOW your phase instructions gate-by-gate from this file:
@@ -245,18 +245,18 @@ Read the return protocol from:
 
 ## Specs (if present)
 - {artifactPath}/pass0/specs.md
-Read and incorporate specs alongside phase instructions. Both are inputs you must follow — phase instructions define methodology and coding standards, specs define acceptance criteria and design intent. Neither overrides the other.
+Read and incorporate specs alongside phase instructions. Both are inputs you must follow  -  phase instructions define methodology and coding standards, specs define acceptance criteria and design intent. Neither overrides the other.
 
 ## Reference Files (read during execution if needed)
 - .instructions/figma/figma-to-swiftui/{reference path}
 
 ## Previous Phase Artifacts (read in this order)
-{use the Read-Order Table below — list file PATHS only, do not embed content}
+{use the Read-Order Table below  -  list file PATHS only, do not embed content}
 ```
 
 ### 2. Dispatch
 
-**Phase 1 (script):** Run directly via Bash — no subagent needed:
+**Phase 1 (script):** Run directly via Bash  -  no subagent needed:
 
 ```bash
 python3 .instructions/figma/figma-to-swiftui/scripts/phase1-gather.py \
@@ -271,7 +271,7 @@ The script tries remote MCP first, falls back to local MCP automatically.
 
 **If `status` = `COMPLETE_WITH_DEFERRED`:**
 Design context was deferred (too many sublayers). All other artifacts are present.
-Do NOT ask the user — dispatch a **Phase 1 subagent** for deferred resolution:
+Do NOT ask the user  -  dispatch a **Phase 1 subagent** for deferred resolution:
 
 ```
 Agent(
@@ -301,7 +301,7 @@ Agent(
 
 ### 3. Parse the return
 
-> **All counters are persisted to `{artifactPath}/_perf_tracker.json` — NOT held in memory.**
+> **All counters are persisted to `{artifactPath}/_perf_tracker.json`  -  NOT held in memory.**
 > Read the tracker file before updating, write it back immediately after. This survives context loss.
 
 **Before dispatching each phase**, record the start time in the tracker:
@@ -319,16 +319,16 @@ with open(tracker_path, "w") as f:
 **STATUS: COMPLETE**
 → Record gates and artifacts
 → **Update tracker file on disk**: set phase `end`, `duration_seconds`, `status: "COMPLETE"`
-→ **Append phase timing to IMPLEMENTATION_HISTORY.md** (incremental write — see step 4)
+→ **Append phase timing to IMPLEMENTATION_HISTORY.md** (incremental write  -  see step 4)
 → Proceed to next phase
 
 **STATUS: HALT**
 → **Update tracker file on disk**: increment `halt_count`, append to `halt_log`
 → Show user: GATE, REASON, USER_ACTION_REQUIRED via AskUserQuestion
 → Options: "Fix and retry", "Skip this phase", "Abort workflow"
-→ Retry: update tracker — increment `retry_count`, set `resolution: "retry"`, re-dispatch same phase
-→ Skip: update tracker — set `resolution: "skipped"`, `status: "SKIPPED"`, proceed
-→ Abort: update tracker — set `resolution: "aborted"`, `status: "ABORTED"`, stop workflow
+→ Retry: update tracker  -  increment `retry_count`, set `resolution: "retry"`, re-dispatch same phase
+→ Skip: update tracker  -  set `resolution: "skipped"`, `status: "SKIPPED"`, proceed
+→ Abort: update tracker  -  set `resolution: "aborted"`, `status: "ABORTED"`, stop workflow
 
 **STATUS: NEEDS_INPUT**
 → Show user: QUESTION, OPTIONS, SUGGESTED via AskUserQuestion
@@ -339,7 +339,7 @@ with open(tracker_path, "w") as f:
     ANSWER: {user's choice}
   ```
 
-### 4. Update history (incremental — after EVERY phase)
+### 4. Update history (incremental  -  after EVERY phase)
 
 **After each phase completes, IMMEDIATELY write two things:**
 
@@ -371,15 +371,15 @@ Also update IMPLEMENTATION_HISTORY.md with the standard phase status, gates, and
 
 ---
 
-## Phase 2 Routing (inline — do NOT read phase-2-orchestrator.md)
+## Phase 2 Routing (inline  -  do NOT read phase-2-orchestrator.md)
 
 **Do NOT read `phase-2-orchestrator.md`. All routing logic is here. Subagents read their own docs.**
 
-> **UNCHANGED from original** — Phase 2 routing preserved exactly.
+> **UNCHANGED from original**  -  Phase 2 routing preserved exactly.
 
 1. **PRE-CHECK**: Read `IMPLEMENTATION_HISTORY.md` (already in context). Verify `pass1/_index.md` exists.
-2. **NORMALIZE**: Check `pass1/design_context_response.json` — if `sublayers: true` and no `response` field, concatenate `responses` values into `response` field and write back.
-3. **REGISTRY_LOOKUP**: `cd {repos.common.path} && .build/debug/FigmaRegistryUtility utility-lookup --node-id '{nodeId}'` — pass result as `registryData` context to subagents.
+2. **NORMALIZE**: Check `pass1/design_context_response.json`  -  if `sublayers: true` and no `response` field, concatenate `responses` values into `response` field and write back.
+3. **REGISTRY_LOOKUP**: `cd {repos.common.path} && .build/debug/FigmaRegistryUtility utility-lookup --node-id '{nodeId}'`  -  pass result as `registryData` context to subagents.
 4. **Wave 1 (parallel)**: Dispatch 2A + 2B + 2C as subagents (each reads its own instruction doc by path).
 5. **Wave 2**: Dispatch 2D as subagent (depends on 2A output).
 6. **FINALIZE (script)**:
@@ -393,11 +393,11 @@ Also update IMPLEMENTATION_HISTORY.md with the standard phase status, gates, and
 
 ---
 
-## Phase 3 Routing (inline — do NOT read phase-3-orchestrator.md)
+## Phase 3 Routing (inline  -  do NOT read phase-3-orchestrator.md)
 
 **Do NOT read `phase-3-orchestrator.md`. Subagents read their own docs. Scripts handle deterministic phases.**
 
-> **UNCHANGED from original** — Phase 3 routing preserved exactly.
+> **UNCHANGED from original**  -  Phase 3 routing preserved exactly.
 
 1. **PRE-CHECK**: Verify `pass2/_index.md` exists.
 2. **NORMALIZE**: Same sublayer check as Phase 2 (skip if already normalized).
@@ -405,14 +405,14 @@ Also update IMPLEMENTATION_HISTORY.md with the standard phase status, gates, and
    - **Scripts**: `python3 .instructions/figma/figma-to-swiftui/scripts/phase3-scripts.py wave1 --artifact-path '{artifactPath}'` (runs 3B+3E+3F, ~5s)
    - **Subagents**: 3A (`phase-3a-location.md`, sonnet, 20 turns), 3C (`phase-3c-nested.md`, sonnet, 20 turns), 3H (`phase-3h-variant-config.md`, opus, 25 turns)
 4. **MISSING TOKEN RESOLUTION**: After Wave 1, check `pass3/02_token_mapping.md` for missing tokens (lines with "⚠️ not found"). If any exist, dispatch a subagent:
-   - **Subagent** (`general-purpose`, opus, 25 turns) — reads `reference/missing-tokens.md` for procedure, resolves each missing token by:
+   - **Subagent** (`general-purpose`, opus, 25 turns)  -  reads `reference/missing-tokens.md` for procedure, resolves each missing token by:
      - **Typography**: Adding alias entries to `typography-registry.json` (match font size + weight to existing Swift tokens)
      - **Colors**: Adding to `Shared/UIAssets/Colors/*.json` (source), running `swift package plugin --allow-writing-to-package-directory generate-ui-assets`, adding to `ui-token-index.json`
      - **Variables**: Using `FigmaRegistryUtility ui-token-lookup` / `ui-token-index` to verify
    - Pass artifact path and token mapping path as context. Subagent updates token mapping after resolution.
    - If 0 missing tokens, skip this step entirely.
 5. **Wave 2 (parallel)**: Launch simultaneously:
-   - **Subagent**: 3D (`phase-3d-patterns.md`, opus, 30 turns) — reads 3H output (`pass3/04c_variant_config_analysis.json`)
+   - **Subagent**: 3D (`phase-3d-patterns.md`, opus, 30 turns)  -  reads 3H output (`pass3/04c_variant_config_analysis.json`)
 6. **Wave 3**: `python3 .instructions/figma/figma-to-swiftui/scripts/phase3-scripts.py 3g --artifact-path '{artifactPath}'` (depends on 3D JSON output)
 7. **COLLECT**: Verify 12 artifacts exist in `pass3/` (10 original + 2 from 3H).
 8. **SPEC (script)**: `python3 .instructions/figma/figma-to-swiftui/scripts/phase3-scripts.py spec --artifact-path '{artifactPath}'`
@@ -421,24 +421,24 @@ Also update IMPLEMENTATION_HISTORY.md with the standard phase status, gates, and
 
 ---
 
-## Phase 4 Routing (inline — do NOT read phase-4-orchestrator.md)
+## Phase 4 Routing (inline  -  do NOT read phase-4-orchestrator.md)
 
 **Do NOT read `phase-4-orchestrator.md`. Subagents read their own docs. Orchestrator stays lean.**
 
 1. **PRE-CHECK**: Verify `pass3/_index.md` exists.
-2. **LOAD_SPEC**: Read `pass3/07_implementation_spec.md` and `pass3/05_complexity.md` — extract `componentLocation`, `complexity` (SIMPLE/COMPLEX).
+2. **LOAD_SPEC**: Read `pass3/07_implementation_spec.md` and `pass3/05_complexity.md`  -  extract `componentLocation`, `complexity` (SIMPLE/COMPLEX).
 3. **PREPARE_DIRECTORY**: `mkdir -p {componentLocation}`. Get `gitUser` via `git config user.name`. Get `date` in DD.MM.YYYY format.
 4. **Wave 1 (parallel)**: Dispatch subagents:
-   - **4A** (`phase-4a-configuration.md`, opus, 30 turns) — **COMPLEX only** (skip if SIMPLE). Writes `{ComponentName}Configuration.swift`.
-   - **4B** (`phase-4b-view.md`, opus, 40 turns) — **ALWAYS**. Writes `{ComponentName}.swift`.
-   - **4C** (`phase-4c-documentation.md`, sonnet, 20 turns) — **ALWAYS**. Writes `FIGMA.md`.
-   - Pass to each: componentName, componentLocation, artifactPath, nodeId, fileKey, complexity, gitUser, date, issueNumber (from IMPLEMENTATION_HISTORY.md `GitHub Issue` field — extract the number after `#`).
+   - **4A** (`phase-4a-configuration.md`, opus, 30 turns)  -  **COMPLEX only** (skip if SIMPLE). Writes `{ComponentName}Configuration.swift`.
+   - **4B** (`phase-4b-view.md`, opus, 40 turns)  -  **ALWAYS**. Writes `{ComponentName}.swift`.
+   - **4C** (`phase-4c-documentation.md`, sonnet, 20 turns)  -  **ALWAYS**. Writes `FIGMA.md`.
+   - Pass to each: componentName, componentLocation, artifactPath, nodeId, fileKey, complexity, gitUser, date, issueNumber (from IMPLEMENTATION_HISTORY.md `GitHub Issue` field  -  extract the number after `#`).
 5. **Wave 2 (parallel)**: Dispatch subagents that depend on 4A+4B:
-   - **4D** (`phase-4d-preview.md`, sonnet, 20 turns) — appends Preview to `{ComponentName}.swift`.
-   - **4E** (`phase-4e-modifiers.md`, sonnet, 20 turns) — **COMPLEX only** (skip if SIMPLE). Writes `{ComponentName}+Modifiers.swift`.
+   - **4D** (`phase-4d-preview.md`, sonnet, 20 turns)  -  appends Preview to `{ComponentName}.swift`.
+   - **4E** (`phase-4e-modifiers.md`, sonnet, 20 turns)  -  **COMPLEX only** (skip if SIMPLE). Writes `{ComponentName}+Modifiers.swift`.
 6. **COLLECT**: Verify `{ComponentName}.swift`, `{ComponentName}Configuration.swift` (if COMPLEX), `{ComponentName}+Modifiers.swift` (if COMPLEX), `FIGMA.md` exist.
-7. **VALIDATE**: `bash .instructions/figma/figma-to-swiftui/scripts/validate-phase4.sh "{componentLocation}" "{ComponentName}" "{artifactPath}/pass3/00_property_coverage.json"` — fix violations if any.
-8. **BUILD**: `xcodebuild build -project "$(git rev-parse --show-toplevel)/{build.xcodeproj}" -scheme {build.scheme} -destination 'generic/platform=iOS' CODE_SIGNING_ALLOWED=NO -quiet` — fix compilation errors if any.
+7. **VALIDATE**: `bash .instructions/figma/figma-to-swiftui/scripts/validate-phase4.sh "{componentLocation}" "{ComponentName}" "{artifactPath}/pass3/00_property_coverage.json"`  -  fix violations if any.
+8. **BUILD**: `xcodebuild build -project "$(git rev-parse --show-toplevel)/{build.xcodeproj}" -scheme {build.scheme} -destination 'generic/platform=iOS' CODE_SIGNING_ALLOWED=NO -quiet`  -  fix compilation errors if any.
 9. **REGISTRY FINALIZE**: After build passes, mark `status.ios.implemented` green in the component registry with a content-hashed version bump:
    ```bash
    python3 .instructions/figma/figma-to-swiftui/scripts/phase4-finalize.py \
@@ -447,32 +447,32 @@ Also update IMPLEMENTATION_HISTORY.md with the standard phase status, gates, and
        --source-file "{componentLocation}/{ComponentName}.swift" \
        --platform ios
    ```
-   Parse stdout JSON. On success (`"status": "ok"`), the registry entry now has `implemented.value: true`, `implemented.version: 1` (or bumped), and a content hash. On failure, log the error but do NOT halt — the finalize can be re-run manually or via `/figma-iteration-commit`.
+   Parse stdout JSON. On success (`"status": "ok"`), the registry entry now has `implemented.value: true`, `implemented.version: 1` (or bumped), and a content hash. On failure, log the error but do NOT halt  -  the finalize can be re-run manually or via `/figma-iteration-commit`.
 10. **UPDATE HISTORY**: Append Phase 4 summary.
 
 ---
 
-## Phase 5 Routing (inline — do NOT read phase-5-orchestrator.md)
+## Phase 5 Routing (inline  -  do NOT read phase-5-orchestrator.md)
 
 > **LEAN MODE: If `lean_mode` is `true` in `_perf_tracker.json`, SKIP this entire section.**
-> Record Phase 5 as `"status": "SKIPPED"` in the tracker and append `| Phase 5 (Testing) | — | ⏭️ lean | ... |` to IMPLEMENTATION_HISTORY.md.
+> Record Phase 5 as `"status": "SKIPPED"` in the tracker and append `| Phase 5 (Testing) |  -  | ⏭️ lean | ... |` to IMPLEMENTATION_HISTORY.md.
 > Proceed directly to Phase 6 (sequentially, not parallel with Phase 7).
 
 **Do NOT read `phase-5-orchestrator.md`. Subagents read their own docs.**
 
-> **PERF CHANGE: Combined test execution** — runs ViewInspector + Snapshot tests in a single xcodebuild invocation instead of two separate build+test cycles.
-> **PERF CHANGE: Simulator pre-warm** — boots simulator in background during test file generation to eliminate cold-start latency.
+> **PERF CHANGE: Combined test execution**  -  runs ViewInspector + Snapshot tests in a single xcodebuild invocation instead of two separate build+test cycles.
+> **PERF CHANGE: Simulator pre-warm**  -  boots simulator in background during test file generation to eliminate cold-start latency.
 
 1. **PRE-CHECK**: Verify Phase 4 complete (component Swift file exists at `{componentLocation}`).
 2. **SIMULATOR PRE-WARM** (parallel with Wave 1): Boot the simulator in the background before test files are written:
    ```bash
    xcrun simctl boot "iPhone 17 Pro" 2>/dev/null &
    ```
-   This runs asynchronously — no need to wait. By the time test execution starts (step 4b), the simulator is ready.
+   This runs asynchronously  -  no need to wait. By the time test execution starts (step 4b), the simulator is ready.
 3. **Wave 1 (parallel)**: Dispatch subagents:
-   - **5A** (`phase-5a-viewinspector.md`, sonnet, 40 turns) — ViewInspector structure + accessibility tests.
-   - **5B** (`phase-5b-snapshot.md`, sonnet, 40 turns) — Snapshot tests for all Figma variants.
-4. **Wave 2 (conditional)**: **5C** (`phase-5c-unit.md`, sonnet, 30 turns) — Unit tests. Only if component has ViewModel/model logic. N/A for pure View + Configuration.
+   - **5A** (`phase-5a-viewinspector.md`, sonnet, 40 turns)  -  ViewInspector structure + accessibility tests.
+   - **5B** (`phase-5b-snapshot.md`, sonnet, 40 turns)  -  Snapshot tests for all Figma variants.
+4. **Wave 2 (conditional)**: **5C** (`phase-5c-unit.md`, sonnet, 30 turns)  -  Unit tests. Only if component has ViewModel/model logic. N/A for pure View + Configuration.
 5. **TEST EXECUTION (combined)**: Run all tests via a SINGLE xcodebuild invocation:
    a. **Switch to local common dependency**: In `{repos.components.path}/Package.swift`, comment out the `.package(url: "...{project.slug}-common...", ...)` line and add `.package(path: "../{project.slug}-common"),` below it. This is needed because the SPM package resolves dependencies from remote git, but Phase 2 adds identifiers/keys to the local editable copy.
    b. **Run ALL tests in a single invocation**:
@@ -499,10 +499,10 @@ Also update IMPLEMENTATION_HISTORY.md with the standard phase status, gates, and
 
 > **LEAN MODE:** If `lean_mode` is `true` in `_perf_tracker.json`:
 > - Run **Phase 6 only** (sequentially, after Phase 4).
-> - **SKIP Phase 7 entirely.** Record as `"status": "SKIPPED"` in the tracker and append `| Phase 7 (Wiki) | — | ⏭️ lean | ... |` to IMPLEMENTATION_HISTORY.md.
+> - **SKIP Phase 7 entirely.** Record as `"status": "SKIPPED"` in the tracker and append `| Phase 7 (Wiki) |  -  | ⏭️ lean | ... |` to IMPLEMENTATION_HISTORY.md.
 > - Do NOT touch the wiki submodule.
 
-> **PERF CHANGE:** Phase 6 (Code Connect) and Phase 7 (Wiki Markdown generation) have NO dependency on each other. Phase 7 reads component Swift files + FIGMA.md + pass3 artifacts — all completed in Phase 4. Phase 6 publishes to Figma. Running them in parallel saves ~5-10 minutes.
+> **PERF CHANGE:** Phase 6 (Code Connect) and Phase 7 (Wiki Markdown generation) have NO dependency on each other. Phase 7 reads component Swift files + FIGMA.md + pass3 artifacts  -  all completed in Phase 4. Phase 6 publishes to Figma. Running them in parallel saves ~5-10 minutes.
 
 After Phase 5 completes (or Phase 4 in lean mode):
 
@@ -524,7 +524,7 @@ After Phase 5 completes (or Phase 4 in lean mode):
      subagent_type: "general-purpose",
      model: sonnet,
      max_turns: 30,
-     prompt: {standard Phase 7 dispatch prompt — include wikiPath in context}
+     prompt: {standard Phase 7 dispatch prompt  -  include wikiPath in context}
    )
    ```
 
@@ -549,24 +549,24 @@ After Phase 5 completes (or Phase 4 in lean mode):
 
 When building the `Previous Phase Artifacts` section of the dispatch prompt, list artifacts in this order. Compact artifacts go FIRST so the subagent establishes context before large files consume the context window.
 
-Every phase must be independently operable from a clean session — all artifacts remain accessible on disk. This table is a **read-ordering preference**, not a prohibition.
+Every phase must be independently operable from a clean session  -  all artifacts remain accessible on disk. This table is a **read-ordering preference**, not a prohibition.
 
 | Phase | Read FIRST (compact) | Read SECOND (if detail needed) |
 |-------|---------------------|-------------------------------|
 | 2A-2D | `pass1/component_summary.json` | `pass1/design_context_response.json`, `pass1/component_properties.json` |
-| 3A | `pass1/component_summary.json`, `IMPLEMENTATION_HISTORY.md` | — |
+| 3A | `pass1/component_summary.json`, `IMPLEMENTATION_HISTORY.md` |  -  |
 | 3B | `pass1/component_summary.json`, `pass1/variable_defs_response.json` | `pass1/design_context_response.json` (for binding context) |
-| 3C | `pass1/component_summary.json`, `pass1/code_connect_map_response.json` | — |
+| 3C | `pass1/component_summary.json`, `pass1/code_connect_map_response.json` |  -  |
 | 3D | `pass1/component_summary.json`, `pass1/component_properties.json` | `pass1/design_context_response.json`, `pass1/metadata_response.xml` |
-| 3E | `pass1/component_summary.json`, `pass1/design_context_assets.txt` | — |
+| 3E | `pass1/component_summary.json`, `pass1/design_context_assets.txt` |  -  |
 | 3F | `pass1/component_summary.json`, `pass2/utility_*.md` | `pass1/design_context_response.json` |
-| 3G | `pass1/component_summary.json`, `pass3/04b_component_architecture.md` | — |
+| 3G | `pass1/component_summary.json`, `pass3/04b_component_architecture.md` |  -  |
 | 4A | `pass3/04b_component_architecture.md`, `pass3/05_complexity.md`, `pass3/00_property_coverage.json` | `pass1/component_summary.json` |
 | 4B | `pass1/component_summary.json`, `pass3/00_property_coverage.json`, `pass3/07_implementation_spec.md`, `pass3/02_token_mapping.md`, `pass3/04b_component_architecture.md`, `pass3/04_patterns.md`, `pass3/03_nested_components.md`, `pass3/08_utility_integration.md`, `pass1/screenshot.png` | `pass1/design_context_response.json` (for complex layout detail) |
 | 4C | `pass3/07_implementation_spec.md`, `pass3/04b_component_architecture.md`, `pass3/00_property_coverage.json` | `pass1/component_summary.json` |
 | 4D | `{componentLocation}/{ComponentName}.swift`, `{componentLocation}/{ComponentName}Configuration.swift` (if exists) | `pass3/04_patterns.md` |
-| 4E | `{componentLocation}/{ComponentName}Configuration.swift`, `{componentLocation}/{ComponentName}.swift` | — |
-| 6 | `pass1/component_properties.json`, `{sourceLocation}/FIGMA.md`, `{sourceLocation}/{componentName}.swift` | — |
+| 4E | `{componentLocation}/{ComponentName}Configuration.swift`, `{componentLocation}/{ComponentName}.swift` |  -  |
+| 6 | `pass1/component_properties.json`, `{sourceLocation}/FIGMA.md`, `{sourceLocation}/{componentName}.swift` |  -  |
 
 ---
 
@@ -591,7 +591,7 @@ When `$ARGUMENTS` is `status`:
 2. For each found IMPLEMENTATION_HISTORY.md, look for a sibling `_perf_tracker.json` in the same directory.
 3. If `_perf_tracker.json` exists:
    a. Read it and display the full timing/reliability summary (same format as Final Summary).
-   b. If IMPLEMENTATION_HISTORY.md is missing the `## Workflow Summary` section but the tracker shows all phases completed, **generate and append the summary now** — this recovers from context loss.
+   b. If IMPLEMENTATION_HISTORY.md is missing the `## Workflow Summary` section but the tracker shows all phases completed, **generate and append the summary now**  -  this recovers from context loss.
 4. If `_perf_tracker.json` does not exist, fall back to displaying: component name, current phase, last gate, last updated (from IMPLEMENTATION_HISTORY.md only).
 
 ---
@@ -618,8 +618,8 @@ Derive confluencePath from artifactPath: replace `Figma/Components` with `Conflu
 
 When the last phase completes (or workflow is aborted/skipped):
 
-1. **Read `{artifactPath}/_perf_tracker.json`** — this is the single source of truth for all timing and reliability data.
-2. **Read IMPLEMENTATION_HISTORY.md** `## Phase Timings` table — as a cross-check.
+1. **Read `{artifactPath}/_perf_tracker.json`**  -  this is the single source of truth for all timing and reliability data.
+2. **Read IMPLEMENTATION_HISTORY.md** `## Phase Timings` table  -  as a cross-check.
 3. **Compute totals** from the tracker file (not from memory).
 4. **Output the summary** to the user AND **append it to IMPLEMENTATION_HISTORY.md**.
 
@@ -633,9 +633,9 @@ When the last phase completes (or workflow is aborted/skipped):
 ## Workflow Summary
 
 - Component: {componentName}
-- Lean Mode: {yes/no — from tracker `lean_mode` field}
-- Phases Executed: {list of completed phases — from tracker}
-- Phases Skipped: {list, if any — from tracker. In lean mode: Phase 5, Phase 7}
+- Lean Mode: {yes/no  -  from tracker `lean_mode` field}
+- Phases Executed: {list of completed phases  -  from tracker}
+- Phases Skipped: {list, if any  -  from tracker. In lean mode: Phase 5, Phase 7}
 - Total Elapsed Time: {HH:MM:SS} (from tracker workflow_start to last phase end)
 - Artifacts: {artifactPath}
 - Component Location: {componentLocation}
@@ -655,12 +655,12 @@ When the last phase completes (or workflow is aborted/skipped):
 
 ### Reliability
 - First-pass success: {phases_without_halt}/{total_phases} phases ({percentage}%)
-- HALTs: {halt_count from tracker} — {halt_log entries}
+- HALTs: {halt_count from tracker}  -  {halt_log entries}
 - Retries: {retry_count from tracker}
 
 ### Bottlenecks
-⚠️ Phase {N} took {actual} (expected ~{expected}) — {reason if known from halt_log, else "investigate"}
-{If no bottlenecks: "None — all phases within expected duration."}
+⚠️ Phase {N} took {actual} (expected ~{expected})  -  {reason if known from halt_log, else "investigate"}
+{If no bottlenecks: "None  -  all phases within expected duration."}
 ```
 
 **Expected Phase Durations** (used for bottleneck detection):
@@ -700,7 +700,7 @@ The `status` command reads `_perf_tracker.json`, computes the summary, and appen
 | **Combined test execution** | Single `xcodebuild test` with multiple `--only-testing:` instead of 2 separate build+test cycles |
 | **Simulator pre-warm** | Boots iOS Simulator in background during test file generation, eliminating cold-start latency |
 | **Phase 6 ∥ 7 parallelism** | Code Connect publish + Wiki Markdown generation run simultaneously |
-| **Phase 4 registry finalize** | Calls `phase4-finalize.py` after BUILD to mark `implemented: true` — was missing in original routing, caused gray implementation flags on issue board |
-| **Disk-persisted tracking** | All timing, HALT, and retry data written to `_perf_tracker.json` after every phase — survives context loss |
-| **Incremental history writes** | Phase timing rows appended to IMPLEMENTATION_HISTORY.md after each phase — no data lost if conversation cuts |
-| **Bottleneck detection** | Flags phases exceeding 2× expected duration — reconstructable from `_perf_tracker.json` at any time |
+| **Phase 4 registry finalize** | Calls `phase4-finalize.py` after BUILD to mark `implemented: true`  -  was missing in original routing, caused gray implementation flags on issue board |
+| **Disk-persisted tracking** | All timing, HALT, and retry data written to `_perf_tracker.json` after every phase  -  survives context loss |
+| **Incremental history writes** | Phase timing rows appended to IMPLEMENTATION_HISTORY.md after each phase  -  no data lost if conversation cuts |
+| **Bottleneck detection** | Flags phases exceeding 2× expected duration  -  reconstructable from `_perf_tracker.json` at any time |

package/pipeline/skills/figma-common/performance-tour/SKILL.md

@@ -1,4 +1,9 @@
-# /performance-tour — Batch Component Production Pipeline
+---
+name: performance-tour
+description: Produces multiple Figma components in a single session. Validates dependencies, determines implementation order, produces each component sequentially with clean subagent contexts, then hands off to review and batch.
+---
+
+# /performance-tour  -  Batch Component Production Pipeline
 
 > Produces multiple Figma components in a single session. Validates dependencies,
 > determines implementation order, produces each component sequentially with clean
@@ -8,9 +13,9 @@
 
 | Parameter | Required | Default | Description |
 |-----------|----------|---------|-------------|
-| `<issue-numbers>` | Yes | — | Comma-separated GitHub issue numbers from uicomponents repo |
-| `--resume` | No | — | Resume an interrupted tour from `_tour_tracker.json` |
-| `--lean` | No | — | Lean mode: skip Phase 5 (testing) and Phase 7 (wiki) for all components. Forwarded to `performance-start --lean` → `performance-swiftui --lean`. |
+| `<issue-numbers>` | Yes |  -  | Comma-separated GitHub issue numbers from uicomponents repo |
+| `--resume` | No |  -  | Resume an interrupted tour from `_tour_tracker.json` |
+| `--lean` | No |  -  | Lean mode: skip Phase 5 (testing) and Phase 7 (wiki) for all components. Forwarded to `performance-start --lean` → `performance-swiftui --lean`. |
 
 ```
 /performance-tour 1000, 1001, 1002
@@ -78,12 +83,12 @@ context loss and conversation timeouts. Written after every state change.
 }
 ```
 
-**CRITICAL:** Write the tracker file after every state change — component status update,
+**CRITICAL:** Write the tracker file after every state change  -  component status update,
 step transition, HALT event, review completion. This is the mechanism that survives context loss.
 
 ---
 
-## STEP 0 — Parse, Validate & Sort
+## STEP 0  -  Parse, Validate & Sort
 
 ### 0a. Parse Issues
 
@@ -95,11 +100,11 @@ gh issue view <N> --repo ${GH_REPO} --json number,title,body,labels,state
 ```
 
 Extract from each issue body:
-- **Component Name** — from `### Component Name (PascalCase)` section
-- **Figma URL** — from `### Figma URL` section
-- **Node ID** — from `### Node ID` section
-- **Registry Name** — from `### Registry Name` section
-- **Jira Key** — from `### Jira Issue` section
+- **Component Name**  -  from `### Component Name (PascalCase)` section
+- **Figma URL**  -  from `### Figma URL` section
+- **Node ID**  -  from `### Node ID` section
+- **Registry Name**  -  from `### Registry Name` section
+- **Jira Key**  -  from `### Jira Issue` section
 
 Validate:
 - Issue is open → if closed, remove from tour with warning
@@ -109,8 +114,8 @@ Validate:
 ### 0b. Validate Each Component
 
 Run `/figma-validate` logic for each issue (inline, not as skill invocation):
-- Criterion 7 (Already Implemented) — if already implemented, remove from tour
-- Criterion 4 (Dependency Readiness) — collect nested dependencies
+- Criterion 7 (Already Implemented)  -  if already implemented, remove from tour
+- Criterion 4 (Dependency Readiness)  -  collect nested dependencies
 
 ### 0c. Build Dependency Graph
 
@@ -170,13 +175,13 @@ Print tour plan:
 
 ```
 ╔══════════════════════════════════════════════════════════════╗
-║  PERFORMANCE TOUR — PLAN                                     ║
+║  PERFORMANCE TOUR  -  PLAN                                     ║
 ╠══════════════════════════════════════════════════════════════╣
 ║                                                              ║
 ║  Execution Order:                                            ║
-║  1. #1000 — ComponentA (independent)                        ║
-║  2. #1002 — ComponentC (independent)                        ║
-║  3. #1001 — ComponentB (depends on: ComponentA)             ║
+║  1. #1000  -  ComponentA (independent)                        ║
+║  2. #1002  -  ComponentC (independent)                        ║
+║  3. #1001  -  ComponentB (depends on: ComponentA)             ║
 ║                                                              ║
 ║  Dependency Graph:                                           ║
 ║  ComponentA ← ComponentB                                     ║
@@ -200,7 +205,7 @@ These 6 rules govern orchestrator behavior across the tour. Rules 2 (every dispa
 
 ---
 
-## STEP 1 — Sequential Produce + Local Commit
+## STEP 1  -  Sequential Produce + Local Commit
 
 For each component in `execution_order`:
 
@@ -208,22 +213,22 @@ For each component in `execution_order`:
 
 Invoke `/performance-start <issue-number> --tour` (add `--lean` if `lean_mode` is `true` in tracker):
 - Parses issue metadata (STEP 0)
-- **SKIPS** fetch + pull (STEP 1) — already done in STEP 0e
-- **SKIPS** branch creation (STEP 2) — staying on iteration/develop
+- **SKIPS** fetch + pull (STEP 1)  -  already done in STEP 0e
+- **SKIPS** branch creation (STEP 2)  -  staying on iteration/develop
 - Runs registry lookup (STEP 3)
 - Saves specs artifact (STEP 3.5)
 - Assigns to me + sets In Progress (STEP 4, 5)
 - Prints summary (STEP 6)
-- Auto-launches `/performance-swiftui <figma-url>` (STEP 6) — with `--lean` if lean mode
+- Auto-launches `/performance-swiftui <figma-url>` (STEP 6)  -  with `--lean` if lean mode
 
 ### 1b. Wait for production to complete
 
-`performance-swiftui` runs Phase 0–7 as a subagent chain (or Phase 0–4 + 6 in lean mode). When it completes:
+`performance-swiftui` runs Phase 0-7 as a subagent chain (or Phase 0-4 + 6 in lean mode). When it completes:
 
 **If COMPLETE:**
 
 ```bash
-# Local commit — all submodules
+# Local commit  -  all submodules
 cd ${COMMON}
 git add -A
 git commit -m "feat(common): [${JIRA_KEY}] add ${COMPONENT_NAME} tokens and keys
@@ -238,7 +243,7 @@ git commit -m "feat(uicomponents): [${JIRA_KEY}] add ${COMPONENT_NAME} component
 Refs: ${JIRA_KEY}
 Tour: performance-tour $(date +%Y%m%d)"
 
-# Wiki commit — SKIP if lean_mode is true
+# Wiki commit  -  SKIP if lean_mode is true
 if [ "$LEAN_MODE" != "true" ]; then
 cd ${WIKI}
 git add -A
@@ -262,7 +267,7 @@ ARTIFACT_PATH="${component.artifact_path}"
 
 # 2. Remove isolated files (component-specific directories)
 rm -rf "${UICOMPONENTS}/${COMPONENT_LOCATION}"
-# Remove test directories if created (skip check in lean mode — tests weren't generated)
+# Remove test directories if created (skip check in lean mode  -  tests weren't generated)
 if [ "$LEAN_MODE" != "true" ]; then
 find "${UICOMPONENTS}/Tests" -type d -name "${COMPONENT_NAME}*" -exec rm -rf {} +
 # Remove snapshot baselines if created
@@ -298,7 +303,7 @@ Add to `halted_components` list.
 Print:
 
 ```
-⚠️ ComponentB HALTED at Phase 4 — build failure
+⚠️ ComponentB HALTED at Phase 4  -  build failure
    Files cleaned up. Proceeding to next component.
    ComponentB can be fixed during review session.
 ```
@@ -309,9 +314,9 @@ After each component (produced or halted), print progress:
 
 ```
 Tour Progress: 2/3 produced, 1/3 halted
-  ✅ #1000 — ComponentA (produced)
-  ❌ #1001 — ComponentB (halted: Phase 4 build failure)
-  🔄 #1002 — ComponentC (in progress...)
+  ✅ #1000  -  ComponentA (produced)
+  ❌ #1001  -  ComponentB (halted: Phase 4 build failure)
+  🔄 #1002  -  ComponentC (in progress...)
 ```
 
 ### 1d. Loop
@@ -329,9 +334,9 @@ Print:
 ║  Produced: 2/3                                               ║
 ║  Halted:   1/3                                               ║
 ║                                                              ║
-║  ✅ #1000 — ComponentA                                       ║
-║  ✅ #1002 — ComponentC                                       ║
-║  ❌ #1001 — ComponentB (Phase 4 build failure)               ║
+║  ✅ #1000  -  ComponentA                                       ║
+║  ✅ #1002  -  ComponentC                                       ║
+║  ❌ #1001  -  ComponentB (Phase 4 build failure)               ║
 ║                                                              ║
 ║  Next: /performance-review-next                              ║
 ║  Or:   /performance-tour --resume                            ║
@@ -340,18 +345,18 @@ Print:
 
 ---
 
-## STEP 2 — Review Session
+## STEP 2  -  Review Session
 
-Invoke `/performance-review-next` — this is a separate skill that reads the tracker
+Invoke `/performance-review-next`  -  this is a separate skill that reads the tracker
 and manages the interactive review loop.
 
 See `.instructions/figma/performance-review-next/SKILL.md` for details.
 
 ---
 
-## STEP 3 — Batch Commit
+## STEP 3  -  Batch Commit
 
-After all components are reviewed, invoke `/performance-iteration-commit-all` — this is
+After all components are reviewed, invoke `/performance-iteration-commit-all`  -  this is
 a separate skill that validates, pushes, and handles issue housekeeping.
 
 See `.instructions/figma/performance-iteration-commit-all/SKILL.md` for details.
@@ -373,7 +378,7 @@ When invoked with `--resume`:
 ╔══════════════════════════════════════════════════════════════╗
 ║  RESUMING TOUR                                               ║
 ╠══════════════════════════════════════════════════════════════╣
-║  Last state: STEP 1 — producing ComponentC (#1002)          ║
+║  Last state: STEP 1  -  producing ComponentC (#1002)          ║
 ║  Produced:   1 (ComponentA)                                  ║
 ║  Halted:     1 (ComponentB)                                  ║
 ║  Remaining:  1 (ComponentC)                                  ║

package/pipeline/skills/figma-ios/REVIEW_CHECKLIST.md

@@ -1,23 +1,23 @@
 # Figma-iOS Skill Review Checklist
 
-> **Status: WIP — v5.0.0 pass 1 (markdown only).** Every skill in this directory was imported from an upstream source via `pipeline/scripts/import-figma-skills.sh`. Placeholders have been applied, personal-data smoke is green, but **no skill is production-ready until it passes this checklist**. Common skills were moved to `pipeline/skills/figma-common/` in WS-7b — only platform-specific skills (orchestrator + implement + test + code-connect + wiki) remain here.
+> **Status: WIP  -  v5.0.0 pass 1 (markdown only).** Every skill in this directory was imported from an upstream source via `pipeline/scripts/import-figma-skills.sh`. Placeholders have been applied, personal-data smoke is green, but **no skill is production-ready until it passes this checklist**. Common skills were moved to `pipeline/skills/figma-common/` in WS-7b  -  only platform-specific skills (orchestrator + implement + test + code-connect + wiki) remain here.
 
 ## Scope of this pass
 
 - ✅ Markdown files transformed: `SKILL.md`, `phases/*.md`, `reference/*.md`, top-level support files (e.g. `halt-return-protocol.md`).
-- ❌ Python/shell scripts **not** imported in this pass — hardcoded paths beyond the placeholder map require manual review. Scheduled for WS-2b.
+- ❌ Python/shell scripts **not** imported in this pass  -  hardcoded paths beyond the placeholder map require manual review. Scheduled for WS-2b.
 
 ## Per-skill review gates
 
 For each skill listed below, a reviewer must confirm:
 
-1. **Frontmatter accuracy** — `description`, `argument-hint`, `allowed-tools`, `user-invocable` reflect the generic behaviour. Remove `status: wip-v5` when passed.
-2. **Placeholder coverage** — run `grep -E '\{(github|jira|repos|build|confluence|project|user)\.' SKILL.md` — every placeholder must resolve from `figma-project-config.schema.json` v2.0.0. Any unresolved placeholder either extends the schema or becomes a literal default.
-3. **No upstream bleed** — `grep -riE 'turkish|dijital|thy\.com|PVT_[A-Za-z0-9]+' skill/` returns nothing. (Guarded by `smoke-personal-data.sh` globally.)
-4. **Cross-references** — internal links point at the new names (no `figma-to-swift-ui-*`). Script references either resolve under `scripts/` once WS-2b lands, or are explicitly commented as TODO.
-5. **Progress-contract adoption** — add the `<!-- progress-contract: applied -->` marker once the skill emits `→ <verb> <object>` lines per `refs/progress-contract.md`.
-6. **Android parity note** — for component-implement/component-test, note which bits are iOS-only so WS-7 can write the Android peer.
-7. **Smoke green** — `smoke-figma-skill-import.sh`, `smoke-personal-data.sh`, `smoke-commands-skills-parity.sh`, `smoke-progress-contract.sh`.
+1. **Frontmatter accuracy**  -  `description`, `argument-hint`, `allowed-tools`, `user-invocable` reflect the generic behaviour. Remove `status: wip-v5` when passed.
+2. **Placeholder coverage**  -  run `grep -E '\{(github|jira|repos|build|confluence|project|user)\.' SKILL.md`  -  every placeholder must resolve from `figma-project-config.schema.json` v2.0.0. Any unresolved placeholder either extends the schema or becomes a literal default.
+3. **No upstream bleed**  -  `grep -riE 'turkish|dijital|thy\.com|PVT_[A-Za-z0-9]+' skill/` returns nothing. (Guarded by `smoke-personal-data.sh` globally.)
+4. **Cross-references**  -  internal links point at the new names (no `figma-to-swift-ui-*`). Script references either resolve under `scripts/` once WS-2b lands, or are explicitly commented as TODO.
+5. **Progress-contract adoption**  -  add the `<!-- progress-contract: applied -->` marker once the skill emits `→ <verb> <object>` lines per `refs/progress-contract.md`.
+6. **Android parity note**  -  for component-implement/component-test, note which bits are iOS-only so WS-7 can write the Android peer.
+7. **Smoke green**  -  `smoke-figma-skill-import.sh`, `smoke-personal-data.sh`, `smoke-commands-skills-parity.sh`, `smoke-progress-contract.sh`.
 
 ## Skills to review (32)
 
@@ -58,10 +58,10 @@ For each skill listed below, a reviewer must confirm:
 
 ## Follow-up workstreams that consume this output
 
-- **WS-2b** — Import Python/shell scripts with transform + manual review.
-- **WS-4** — Reduce Phase 3 dispatch to single `figma-to-component` call.
-- **WS-5** — Wiki mode adapter; update `figma-component-wiki`.
-- **WS-6** — Issue → Jira → Wiki triad; extend `figma-issue` + `figma-component-wiki`.
-- **WS-7** — Android peer skills (`figma-component-implement-android`, `figma-component-test-android`).
-- **WS-8** — Cross-CLI parity matrix expansion.
-- **WS-11** — Add `<!-- progress-contract: applied -->` marker + `→ <verb>` lines to each skill.
+- **WS-2b**  -  Import Python/shell scripts with transform + manual review.
+- **WS-4**  -  Reduce Phase 3 dispatch to single `figma-to-component` call.
+- **WS-5**  -  Wiki mode adapter; update `figma-component-wiki`.
+- **WS-6**  -  Issue → Jira → Wiki triad; extend `figma-issue` + `figma-component-wiki`.
+- **WS-7**  -  Android peer skills (`figma-component-implement-android`, `figma-component-test-android`).
+- **WS-8**  -  Cross-CLI parity matrix expansion.
+- **WS-11**  -  Add `<!-- progress-contract: applied -->` marker + `→ <verb>` lines to each skill.