@atlashub/smartstack-cli 3.20.0 → 3.22.0

package/templates/skills/business-analyse/html/src/styles/03-navigation.css

@@ -1,62 +1,120 @@
 /* ============================================
-   SIDEBAR - Navigation 5 niveaux
+   SIDEBAR - Hierarchical Tree Navigation
    ============================================ */
-.nav-group { padding: 1rem 0; }
+
+/* Sidebar header */
+.sidebar-header {
+    padding: 0.75rem 1rem;
+    border-bottom: 1px solid var(--border);
+    background: var(--bg-hover);
+}
+.sidebar-app-name {
+    font-size: 0.95rem;
+    font-weight: 600;
+    color: var(--primary-light);
+    display: block;
+    white-space: nowrap;
+    overflow: hidden;
+    text-overflow: ellipsis;
+}
+
+/* Nav groups (top-level: Cadrage, Modules, Consolidation, Synthese) */
+.nav-group { padding: 0.4rem 0; }
 .nav-group + .nav-group { border-top: 1px solid var(--border); }
 
 .nav-group-title {
-    font-size: 0.65rem;
+    font-size: 0.7rem;
     text-transform: uppercase;
     letter-spacing: 0.1em;
     color: var(--text-muted);
-    padding: 0 1rem;
-    margin-bottom: 0.5rem;
+    padding: 0.4rem 1rem;
+    margin-bottom: 0.15rem;
     font-weight: 600;
+    cursor: pointer;
+    display: flex;
+    align-items: center;
+    gap: 0.3rem;
+    user-select: none;
+    transition: color var(--transition-fast);
+}
+.nav-group-title:hover { color: var(--text-bright); }
+
+/* Chevron icon for expand/collapse */
+.nav-chevron {
+    font-size: 0.6rem;
+    display: inline-block;
+    transition: transform var(--transition-fast);
+    color: var(--text-muted);
+    width: 12px;
+    text-align: center;
+    flex-shrink: 0;
 }
+.nav-chevron.expanded { transform: rotate(90deg); }
 
+/* Nav items (leaf nodes) */
 .nav-item {
     display: flex;
     align-items: center;
-    gap: 0.5rem;
-    padding: 0.45rem 1rem;
+    gap: 0.4rem;
+    padding: 0.35rem 1rem;
     color: var(--text);
     text-decoration: none;
-    font-size: 0.85rem;
+    font-size: 0.82rem;
     cursor: pointer;
     transition: all var(--transition-fast);
     border-left: 3px solid transparent;
 }
 .nav-item:hover { background: var(--bg-hover); color: var(--text-bright); }
-.nav-item.active { background: rgba(99,102,241,0.1); border-left-color: var(--primary); color: var(--primary-light); font-weight: 500; }
+.nav-item.active {
+    background: rgba(99,102,241,0.1);
+    border-left-color: var(--primary);
+    color: var(--primary-light);
+    font-weight: 500;
+}
 
-.nav-item .nav-icon { font-size: 1rem; width: 20px; text-align: center; }
+.nav-item .nav-icon {
+    font-size: 0.45rem;
+    width: 12px;
+    text-align: center;
+    color: var(--border-light);
+    flex-shrink: 0;
+}
 .nav-item .nav-badge {
     margin-left: auto;
-    font-size: 0.65rem;
+    font-size: 0.6rem;
     background: var(--bg-hover);
-    padding: 0.1rem 0.4rem;
+    padding: 0.1rem 0.35rem;
     border-radius: 10px;
     color: var(--text-muted);
+    min-width: 16px;
+    text-align: center;
 }
 
-.nav-children { margin-left: 1.2rem; }
-.nav-children .nav-item { font-size: 0.8rem; padding: 0.3rem 1rem; }
+/* Nested children (indented) */
+.nav-children { margin-left: 0; }
+.nav-children .nav-item { padding-left: 1.6rem; font-size: 0.8rem; }
+.nav-children .nav-children .nav-item { padding-left: 2.4rem; font-size: 0.78rem; }
+.nav-children .nav-children .nav-children .nav-item { padding-left: 3rem; font-size: 0.75rem; }
 
-/* ============================================
-   PHASE PROGRESS
-   ============================================ */
-.phase-progress {
-    display: flex; align-items: center; gap: 0.3rem;
-    padding: 0.75rem 1rem; border-bottom: 1px solid var(--border);
-}
-.phase-dot {
-    width: 24px; height: 24px; border-radius: 50%;
-    display: flex; align-items: center; justify-content: center;
-    font-size: 0.6rem; font-weight: 700; color: var(--text-muted);
-    background: var(--bg-hover); border: 2px solid var(--border);
-    transition: all var(--transition-fast);
+/* Module header in nav (collapsible) */
+.nav-module-header {
+    font-weight: 500;
+    color: var(--text-bright);
+    font-size: 0.85rem;
+}
+.nav-module-header:hover { color: var(--primary-light); }
+
+/* Section/Resource items in nav */
+.nav-section-item { margin-top: 0.1rem; }
+.nav-section-link { font-style: italic; }
+.nav-icon-section { font-size: 0.5rem; color: var(--accent); }
+.nav-icon-resource { font-size: 0.7rem; color: var(--border-light); }
+.nav-resource-link {
+    cursor: default;
+    font-size: 0.72rem;
+    color: var(--text-muted);
+    padding-top: 0.2rem;
+    padding-bottom: 0.2rem;
 }
-.phase-dot.completed { background: var(--success); border-color: var(--success); color: #fff; }
-.phase-dot.current { background: var(--primary); border-color: var(--primary); color: #fff; }
-.phase-line { flex: 1; height: 2px; background: var(--border); }
-.phase-line.completed { background: var(--success); }
+.nav-resource-link:hover { background: transparent; color: var(--text-muted); }
+.nav-resources { margin-left: 0.5rem; }

package/templates/skills/business-analyse/html/src/styles/05-modules.css

@@ -388,3 +388,67 @@
     padding-left: 0.75rem; border-left: 2px solid var(--accent);
     margin-bottom: 0.2rem;
 }
+
+/* ============================================
+   STRUCTURE TAB (Sections/Resources)
+   ============================================ */
+.struct-section {
+    background: var(--bg-card);
+    border: 1px solid var(--border);
+    border-radius: 10px;
+    margin-bottom: 1rem;
+    overflow: hidden;
+}
+.struct-section-header {
+    display: flex;
+    align-items: center;
+    gap: 0.75rem;
+    padding: 0.75rem 1rem;
+    background: var(--bg-hover);
+    border-bottom: 1px solid var(--border);
+}
+.struct-section-code {
+    font-weight: 600;
+    color: var(--text-bright);
+    font-size: 0.95rem;
+}
+.struct-section-desc {
+    flex: 1;
+    font-size: 0.8rem;
+    color: var(--text-muted);
+}
+.struct-section-badge {
+    font-size: 0.65rem;
+    color: var(--text-muted);
+    background: rgba(99,102,241,0.1);
+    padding: 0.1rem 0.5rem;
+    border-radius: 4px;
+    white-space: nowrap;
+}
+.struct-resources {
+    padding: 0.5rem 0;
+}
+.struct-resource {
+    display: flex;
+    align-items: baseline;
+    gap: 0.5rem;
+    padding: 0.35rem 1rem 0.35rem 1.5rem;
+    font-size: 0.85rem;
+    transition: background var(--transition-fast);
+}
+.struct-resource:hover {
+    background: var(--bg-hover);
+}
+.struct-resource-icon {
+    color: var(--primary-light);
+    font-size: 0.7rem;
+    flex-shrink: 0;
+}
+.struct-resource-name {
+    font-weight: 500;
+    color: var(--text-bright);
+}
+.struct-resource-desc {
+    font-size: 0.8rem;
+    color: var(--text-muted);
+}

package/templates/skills/business-analyse/html/src/styles/06-wireframes.css

@@ -61,6 +61,48 @@
 .svg-wireframe svg {
     max-width: 100%;
     height: auto;
+    display: block;
+}
+
+/* ============================================
+   WIREFRAME VIEW TOGGLE (SVG / ASCII)
+   ============================================ */
+.wireframe-toggle {
+    display: flex;
+    gap: 2px;
+    background: var(--bg-dark);
+    border-radius: 6px;
+    padding: 2px;
+    border: 1px solid var(--border);
+    margin-left: auto;
+}
+
+.wireframe-toggle-btn {
+    padding: 0.25rem 0.6rem;
+    font-size: 0.7rem;
+    font-weight: 500;
+    letter-spacing: 0.02em;
+    border: none;
+    border-radius: 4px;
+    cursor: pointer;
+    background: transparent;
+    color: var(--text-muted);
+    transition: all 0.15s ease;
+    font-family: inherit;
+}
+
+.wireframe-toggle-btn:hover {
+    color: var(--text-bright);
+    background: var(--bg-hover);
+}
+
+.wireframe-toggle-btn.active {
+    background: var(--primary);
+    color: #ffffff;
+}
+
+.wireframe-view:not(.active) {
+    display: none;
 }
 
 .wireframe-description {

package/templates/skills/business-analyse/html/src/template.html

@@ -20,6 +20,7 @@
             <span class="header-app-name" id="appName">{{APPLICATION_NAME}}</span>
             <div class="header-spacer"></div>
             <div class="header-actions">
+                <button class="btn btn-sm" onclick="resetToEmbedded()" title="Reinitialiser depuis les donnees d'origine (supprime les modifications locales)">Reset</button>
                 <button class="btn btn-sm" onclick="saveToLocalStorage()" title="Sauvegarder les modifications dans le navigateur">Sauvegarder</button>
                 <button class="btn btn-sm btn-review" onclick="saveReviewJSON()" title="Sauvegarder les corrections pour creer une nouvelle version">Sauvegarder corrections</button>
                 <button class="btn btn-sm btn-primary" onclick="exportJSON()" title="Exporter les donnees au format JSON pour l'extraction">Exporter JSON</button>
@@ -32,85 +33,16 @@
 
         <div class="body" id="appBody">
             <!-- ============================================
-                 SIDEBAR - Navigation 5 niveaux
+                 SIDEBAR - Navigation hierarchique
                  ============================================ -->
             <aside class="sidebar">
-                <!-- Phase Progress -->
-                <div class="phase-progress">
-                    <div class="phase-dot current" id="phase-1" title="Cadrage">1</div>
-                    <div class="phase-line" id="pline-1"></div>
-                    <div class="phase-dot" id="phase-2" title="Decomposition">2</div>
-                    <div class="phase-line" id="pline-2"></div>
-                    <div class="phase-dot" id="phase-3" title="Specification">3</div>
-                    <div class="phase-line" id="pline-3"></div>
-                    <div class="phase-dot" id="phase-4" title="Consolidation">4</div>
-                    <div class="phase-line" id="pline-4"></div>
-                    <div class="phase-dot" id="phase-5" title="Synthese">5</div>
+                <!-- Application Name -->
+                <div class="sidebar-header">
+                    <span class="sidebar-app-name" id="sidebarAppName">{{APPLICATION_NAME}}</span>
                 </div>
-
-                <!-- Phase 1 : Cadrage -->
-                <div class="nav-group">
-                    <div class="nav-group-title">1. Cadrage</div>
-                    <a class="nav-item active" onclick="showSection('cadrage-context')" data-section="cadrage-context">
-                        <span class="nav-icon">&#9679;</span> Contexte
-                    </a>
-                    <a class="nav-item" onclick="showSection('cadrage-stakeholders')" data-section="cadrage-stakeholders">
-                        <span class="nav-icon">&#9679;</span> Parties prenantes
-                        <span class="nav-badge" id="stakeholderCount">0</span>
-                    </a>
-                    <a class="nav-item" onclick="showSection('cadrage-scope')" data-section="cadrage-scope">
-                        <span class="nav-icon">&#9679;</span> Perimetre fonctionnel
-                    </a>
-                    <a class="nav-item" onclick="showSection('cadrage-risks')" data-section="cadrage-risks">
-                        <span class="nav-icon">&#9679;</span> Risques et hypotheses
-                    </a>
-                    <a class="nav-item" onclick="showSection('cadrage-success')" data-section="cadrage-success">
-                        <span class="nav-icon">&#9679;</span> Criteres de reussite
-                    </a>
-                </div>
-
-                <!-- Phase 2 : Decomposition -->
-                <div class="nav-group">
-                    <div class="nav-group-title">2. Decomposition</div>
-                    <a class="nav-item" onclick="showSection('decomp-modules')" data-section="decomp-modules">
-                        <span class="nav-icon">&#9679;</span> Domaines fonctionnels
-                        <span class="nav-badge" id="moduleCount">0</span>
-                    </a>
-                    <a class="nav-item" onclick="showSection('decomp-dependencies')" data-section="decomp-dependencies">
-                        <span class="nav-icon">&#9679;</span> Dependances
-                    </a>
-                </div>
-
-                <!-- Phase 3 : Specification par module -->
-                <div class="nav-group" id="modulesNav">
-                    <div class="nav-group-title">3. Specification</div>
-                    <!-- Populated dynamically per module -->
-                </div>
-
-                <!-- Phase 4 : Consolidation -->
-                <div class="nav-group">
-                    <div class="nav-group-title">4. Consolidation</div>
-                    <a class="nav-item" onclick="showSection('consol-datamodel')" data-section="consol-datamodel">
-                        <span class="nav-icon">&#9679;</span> Modele de donnees
-                        <span class="nav-badge" id="entityCount">0</span>
-                    </a>
-                    <a class="nav-item" onclick="showSection('consol-interactions')" data-section="consol-interactions">
-                        <span class="nav-icon">&#9679;</span> Interactions
-                    </a>
-                    <a class="nav-item" onclick="showSection('consol-permissions')" data-section="consol-permissions">
-                        <span class="nav-icon">&#9679;</span> Coherence des acces
-                    </a>
-                    <a class="nav-item" onclick="showSection('consol-flows')" data-section="consol-flows">
-                        <span class="nav-icon">&#9679;</span> Parcours bout en bout
-                    </a>
-                </div>
-
-                <!-- Phase 5 : Synthese -->
-                <div class="nav-group">
-                    <div class="nav-group-title">5. Synthese</div>
-                    <a class="nav-item" onclick="showSection('handoff-summary')" data-section="handoff-summary">
-                        <span class="nav-icon">&#9679;</span> Vue d'ensemble
-                    </a>
+                <!-- Dynamic Tree Navigation -->
+                <div id="sidebarNav">
+                    <!-- Populated by buildNavTree() -->
                 </div>
             </aside>
 

package/templates/skills/business-analyse/references/acceptance-criteria.md

@@ -0,0 +1,169 @@
+# Module Acceptance Criteria (Shared Reference)
+
+> **Loaded by:** step-03d-validate (section 11-POST-CHECK-BASH), step-05a-handoff (pre-handoff gate), ralph-loop step-01-task (input validation)
+> **Purpose:** Define measurable, bash-verifiable acceptance criteria that a module MUST pass before being marked "specified".
+> **Key principle:** These checks read the REAL feature.json file on disk — NOT in-memory data the model "thinks" it wrote.
+
+---
+
+## Why Bash-Verifiable?
+
+All previous checks were **pseudocode interpreted by the model**. The model can "decide" a check passes even when it fails (hallucination of validation). Bash checks are **objective** — they read the actual file and count real elements.
+
+---
+
+## Acceptance Criteria Table
+
+| # | Criterion | Minimum | Field Path | Blocking | Category |
+|---|-----------|---------|------------|----------|----------|
+| AC-01 | Entities present | 1 | `analysis.entities[]` | YES | Data Model |
+| AC-02 | Entity attributes have `type` field | ALL | `analysis.entities[].attributes[].type` | YES | Data Model |
+| AC-03 | Use cases present | 2 | `specification.useCases[]` | YES | Requirements |
+| AC-04 | Functional requirements present | 4 | `specification.functionalRequirements[]` | YES | Requirements |
+| AC-05 | Wireframes present | 1 | `specification.uiWireframes[] \|\| specification.wireframes[]` | YES | UI |
+| AC-06 | Wireframes >= sections | count match | wireframes.length >= sections.length | YES | UI |
+| AC-07 | Wireframe content non-empty | ALL | `wireframe.mockup \|\| wireframe.ascii \|\| wireframe.content` | YES | UI |
+| AC-08 | Sections present | 1 | `specification.sections[]` | YES | UI |
+| AC-09 | SeedDataCore 7 arrays non-empty | 7/7 | `specification.seedDataCore.*` | YES | Seed Data |
+| AC-10 | Gherkin scenarios is array | `Array.isArray` | `specification.gherkinScenarios` | YES | Format |
+| AC-11 | API endpoints present | 1 | `specification.apiEndpoints[]` | YES | API |
+| AC-12 | Messages present | 4 | `specification.messages[]` | YES | Messages |
+| AC-13 | Validations present | 1 | `specification.validations[]` | YES | Validation |
+| AC-14 | Wireframe field naming | canonical | `screen` not `title`, `mockup` not `ascii` | WARNING | Convention |
+| AC-15 | Validation rules format | ALL array | `specification.validations[].rules` is `Array` | YES | Format |
+| AC-16 | Messages have `message` field | ALL present | `specification.messages[].message` truthy | YES | Format |
+| AC-17 | Gherkin content structure | ALL valid | Each element has `feature` (string) + `scenarios` (array) | YES | Format |
+
+---
+
+## Bash Verification Script (node -e)
+
+> **Usage:** Called from step-03d section 11-POST-CHECK-BASH after writing module feature.json.
+> Also callable standalone for debugging: `node -e "..." path/to/feature.json`
+
+```bash
+MODULE_JSON="{module_feature_json_path}"
+node -e "
+const fs = require('fs');
+const data = JSON.parse(fs.readFileSync(process.argv[1], 'utf-8'));
+const spec = data.specification || {};
+const analysis = data.analysis || {};
+const wf = spec.uiWireframes || spec.wireframes || [];
+const sections = spec.sections || [];
+const sdc = spec.seedDataCore || {};
+
+const checks = [
+  ['AC-01: entities >= 1',          (analysis.entities||[]).length,               1],
+  ['AC-03: useCases >= 2',          (spec.useCases||[]).length,                   2],
+  ['AC-04: FRs >= 4',               (spec.functionalRequirements||[]).length,     4],
+  ['AC-05: wireframes >= 1',        wf.length,                                    1],
+  ['AC-06: wireframes >= sections', wf.length,                                    sections.length],
+  ['AC-08: sections >= 1',          sections.length,                               1],
+  ['AC-09: seedDataCore 7 arrays',  Object.keys(sdc).filter(k => Array.isArray(sdc[k]) && sdc[k].length > 0).length, 7],
+  ['AC-10: gherkin is array',       Array.isArray(spec.gherkinScenarios) ? 1 : 0,  1],
+  ['AC-11: apiEndpoints >= 1',      (spec.apiEndpoints||[]).length,                1],
+  ['AC-12: messages >= 4',          (spec.messages||[]).length,                    4],
+  ['AC-13: validations >= 1',       (spec.validations||[]).length,                 1]
+];
+
+const fails = [];
+checks.forEach(c => {
+  if (c[1] < c[2]) {
+    fails.push(c);
+    console.error('FAIL: ' + c[0] + ' = ' + c[1] + ' (min: ' + c[2] + ')');
+  }
+});
+
+// AC-02: Entity attribute types
+const badAttrs = (analysis.entities||[]).flatMap(e =>
+  (e.attributes||[]).filter(a => !a.type).map(a => e.name + '.' + a.name)
+);
+if (badAttrs.length > 0) {
+  fails.push(['AC-02: attr.type missing', badAttrs.length, 0]);
+  console.error('FAIL: AC-02: ' + badAttrs.length + ' attributes without type: ' + badAttrs.slice(0, 5).join(', ') + (badAttrs.length > 5 ? '...' : ''));
+}
+
+// AC-07: Wireframe content non-empty
+const emptyWf = wf.filter(w => !w.mockup && !w.ascii && !w.content);
+if (emptyWf.length > 0) {
+  fails.push(['AC-07: wireframe content empty', emptyWf.length, 0]);
+  console.error('FAIL: AC-07: ' + emptyWf.length + ' wireframes have EMPTY content');
+}
+
+// AC-14: Wireframe field naming (WARNING only)
+const badFields = wf.filter(w => w.title || w.ascii || (w.name && !w.screen));
+if (badFields.length > 0) {
+  console.warn('WARNING: AC-14: ' + badFields.length + ' wireframes use non-canonical field names (title/ascii/name)');
+}
+
+// AC-15: Validation rules format (rules must be array, not string)
+const badRules = (spec.validations||[]).filter(v => v.rules && !Array.isArray(v.rules));
+if (badRules.length > 0) {
+  fails.push(['AC-15: validations[].rules not array', badRules.length, 0]);
+  console.error('FAIL: AC-15: ' + badRules.length + ' validations have rules as string instead of array');
+}
+
+// AC-16: Messages must have 'message' field
+const noMsg = (spec.messages||[]).filter(m => !m.message);
+if (noMsg.length > 0) {
+  fails.push(['AC-16: messages[].message missing', noMsg.length, 0]);
+  console.error('FAIL: AC-16: ' + noMsg.length + ' messages missing "message" field');
+}
+
+// AC-17: Gherkin content structure (each element needs feature + scenarios[])
+if (Array.isArray(spec.gherkinScenarios)) {
+  const badGherkin = spec.gherkinScenarios.filter(g => !g.feature || !Array.isArray(g.scenarios));
+  if (badGherkin.length > 0) {
+    fails.push(['AC-17: gherkin content invalid', badGherkin.length, 0]);
+    console.error('FAIL: AC-17: ' + badGherkin.length + ' gherkin entries missing feature or scenarios[]');
+  }
+}
+
+// Summary
+if (fails.length > 0) {
+  console.error('\\nBLOCKING: ' + fails.length + ' acceptance criteria FAILED');
+  process.exit(1);
+}
+console.log('PASS: All acceptance criteria met (AC-01 to AC-17)');
+" "$MODULE_JSON"
+```
+
+---
+
+## Integration Points
+
+### 1. step-03d-validate.md (section 11-POST-CHECK-BASH)
+
+After writing module feature.json, run the bash script above.
+IF FAIL → re-execute section 11 write with corrected data → re-run until PASS.
+
+### 2. step-05a-handoff.md (pre-handoff gate)
+
+Before writing handoff for each module (section 7a step 3), run the bash script on the module feature.json.
+IF FAIL → STOP handoff for this module, report which criteria failed.
+
+### 3. ralph-loop step-01-task.md (input validation)
+
+At PRD load time, run the bash script on each module feature.json.
+IF FAIL → BLOCKING: "Module {name} does not meet acceptance criteria — run /business-analyse to fix".
+This prevents ralph-loop from generating code from incomplete specifications.
+
+---
+
+## Recovery Actions per Criterion
+
+| Criterion | Recovery |
+|-----------|----------|
+| AC-01/02 | Re-run step-03a2-analysis (entity definition) |
+| AC-03/04 | Re-run step-03c-compile sections 8b/8c |
+| AC-05/06/07 | Re-run step-03b-ui (wireframe generation) — step-03b now writes intermediately |
+| AC-08 | Re-run step-03b section 3a-bis (section definition) |
+| AC-09 | Re-run step-03c section 8f-bis (seedDataCore transform) |
+| AC-10 | Auto-fix: wrap in array `[gherkinScenarios]` |
+| AC-11 | Re-run step-03c section 8k |
+| AC-12 | Re-run step-03c section 8i |
+| AC-13 | Re-run step-03c section 8h |
+| AC-14 | Auto-fix: rename title→screen, ascii→mockup (done by step-03c/03d auto-fix) |
+| AC-15 | Auto-fix: wrap string in array `[rules]` (done by step-03c ABSOLUTE FORMAT CHECKS) |
+| AC-16 | Auto-fix: copy `description` → `message` (done by step-03c ABSOLUTE FORMAT CHECKS) |
+| AC-17 | Re-run step-03c section 8g — ensure each gherkin entry has `feature` string + `scenarios` array |

package/templates/skills/business-analyse/references/deploy-data-build.md

@@ -26,14 +26,15 @@ const FEATURE_DATA = {
     scope: {
       // CONVERT feature.json keys to HTML keys:
       // mustHave → vital, shouldHave → important, couldHave → optional, outOfScope → excluded
+      // CRITICAL: Preserve per-item `module` assignment from coverageMatrix (NOT modules[0])
       vital: (master.cadrage.globalScope?.mustHave || master.cadrage.coverageMatrix?.filter(i => i.category === "mustHave") || [])
-        .map(item => typeof item === 'string' ? { name: item, description: "" } : { name: item.item || item, description: item.notes || "" }),
+        .map(item => typeof item === 'string' ? { name: item, description: "", module: null } : { name: item.item || item, description: item.notes || "", module: item.module || null }),
       important: (master.cadrage.globalScope?.shouldHave || master.cadrage.coverageMatrix?.filter(i => i.category === "shouldHave") || [])
-        .map(item => typeof item === 'string' ? { name: item, description: "" } : { name: item.item || item, description: item.notes || "" }),
+        .map(item => typeof item === 'string' ? { name: item, description: "", module: null } : { name: item.item || item, description: item.notes || "", module: item.module || null }),
       optional: (master.cadrage.globalScope?.couldHave || master.cadrage.coverageMatrix?.filter(i => i.category === "couldHave") || [])
-        .map(item => typeof item === 'string' ? { name: item, description: "" } : { name: item.item || item, description: item.notes || "" }),
+        .map(item => typeof item === 'string' ? { name: item, description: "", module: null } : { name: item.item || item, description: item.notes || "", module: item.module || null }),
       excluded: (master.cadrage.globalScope?.outOfScope || master.cadrage.coverageMatrix?.filter(i => i.category === "outOfScope") || [])
-        .map(item => typeof item === 'string' ? { name: item, description: "" } : { name: item.item || item, description: item.notes || "" })
+        .map(item => typeof item === 'string' ? { name: item, description: "", module: null } : { name: item.item || item, description: item.notes || "", module: item.module || null })
     },
     risks: (master.cadrage.risks || []).map(r => ({
       description: r.description,
@@ -123,12 +124,14 @@ const FEATURE_DATA = {
 const EMBEDDED_ARTIFACTS = {
   wireframes: {
     // PER-MODULE keyed object (NOT a flat array)
-    // FOR EACH module: extract from specification.uiWireframes[]
-    [moduleCode]: moduleFeature.specification.uiWireframes.map(wf => ({
-      screen: wf.screen || wf.name || wf.id || "",  // SAFETY NET: fallback name/id → screen
+    // FOR EACH module: extract from specification.uiWireframes[] OR specification.wireframes[] (SAFETY NET)
+    // IMPORTANT: The agent may write wireframes under either key name — always check BOTH
+    [moduleCode]: (moduleFeature.specification.uiWireframes || moduleFeature.specification.wireframes || []).map(wf => ({
+      screen: wf.screen || wf.name || wf.title || wf.id || "",  // SAFETY NET: fallback name/title/id → screen
       section: wf.section || "",                      // e.g. "list"
       format: wf.mockupFormat || "ascii",             // RENAME: mockupFormat → format
-      content: wf.mockup,                             // RENAME: mockup → content (ASCII art string)
+      content: wf.mockup || wf.ascii || wf.content || "",  // SAFETY NET: mockup/ascii/content → content
+      svgContent: null,                              // Populated by SVG generation step (see wireframe-svg-style-guide.md)
       description: wf.description || "",
       elements: wf.elements || [],                    // [{ id, type, label }] or ["DataGrid", ...]
       actions: wf.actions || [],                      // [{ trigger, action }] or ["filter", ...]
@@ -161,10 +164,11 @@ const EMBEDDED_ARTIFACTS = {
 
 ### Artifact Gathering
 
-1. For EACH module: read `specification.uiWireframes[]`, **rename fields** (`mockupFormat`→`format`, `mockup`→`content`), store under `wireframes[moduleCode]`
+1. For EACH module: read `specification.uiWireframes[]` OR `specification.wireframes[]` (check BOTH keys — agent may use either), **rename fields** (`mockupFormat`→`format`, `mockup`/`ascii`/`content`→`content`, `screen`/`name`/`title`→`screen`), store under `wireframes[moduleCode]`
 2. Read master's `consolidation.e2eFlows[]` and build e2eFlows array with diagram generation
 3. Read master's `dependencyGraph` and build nodes/edges
 4. Serialize as JSON with 2-space indentation
+5. **SVG Wireframe Enrichment** (after wireframe extraction): For EACH wireframe with `content` (ASCII) and `svgContent === null`, generate SVG using parallel Task(sonnet) agents. See [wireframe-svg-style-guide.md](wireframe-svg-style-guide.md) for the complete prompt template and orchestration process. If generation fails for any wireframe → leave `svgContent` as null (graceful fallback to ASCII).
 
 ## Placeholder Replacement
 

package/templates/skills/business-analyse/references/handoff-file-templates.md

@@ -82,10 +82,11 @@ From `specification.uiWireframes[]`, `specification.dashboards[]` and `analysis.
 
 ## 4.6 SeedData Files
 
-**OBLIGATORY: 5 CORE + business per module:**
+**OBLIGATORY: 6 CORE (1 app-level + 5 per module) + business per module:**
 
 ```json
 "seedData": [
+  { "path": "src/Infrastructure/Persistence/Seeding/Data/NavigationApplicationSeedData.cs", "type": "SeedData", "category": "core", "source": "specification.seedDataCore.navigationApplications", "module": "shared", "description": "Application-level navigation seed data. Created ONCE per application (FIRST). Provides ApplicationId for modules and roles." },
   { "path": "src/Infrastructure/Persistence/Seeding/Data/{ModuleName}/NavigationModuleSeedData.cs", "type": "SeedData", "category": "core", "source": "specification.seedDataCore.navigationModules", "module": "{moduleCode}" },
   { "path": "src/Infrastructure/Persistence/Seeding/Data/{ModuleName}/PermissionsSeedData.cs", "type": "SeedData", "category": "core", "source": "specification.seedDataCore.permissions", "module": "{moduleCode}" },
   { "path": "src/Infrastructure/Persistence/Seeding/Data/{ModuleName}/RolesSeedData.cs", "type": "SeedData", "category": "core", "source": "specification.seedDataCore.roles", "module": "{moduleCode}" },