@atlashub/smartstack-cli 3.8.0 → 3.10.0

package/templates/skills/business-analyse/SKILL.md

@@ -35,12 +35,16 @@ The skill auto-detects which use case applies by scanning existing features in `
 
 <parameters>
 
-No flags. The entire input is the `{feature_description}`.
+| Flag | Description |
+|------|-------------|
+| (aucun) | Mode standard — analyse complete ou mise a jour |
+| `-review` | Mode review — lit `ba-review.json`, cree une nouvelle version avec les corrections appliquees |
 
-Step-00 handles all detection automatically:
+Step-00 handles detection automatically:
 - **New vs Update**: scans `docs/business/` for existing applications
 - **Single vs Multi-module**: determined during step-02 decomposition
 - **Language**: detected from config or asked once
+- **Review mode**: if `-review` flag, routes directly to step-06
 
 </parameters>
 
@@ -71,6 +75,52 @@ docs/business/
 **No intermediate markdown files - all state in feature.json**
 </output_structure>
 
+<navigation_hierarchy>
+**SmartStack uses a 5-level navigation structure mapped to database tables:**
+
+| Level | Entity | Table DB | Created in Step | Description |
+|-------|--------|----------|-----------------|-------------|
+| 1 | Context | `core.nav_Contexts` | N/A | Fixed ("business", "platform", "system") |
+| 2 | Application | `core.nav_Applications` | 00-01 | Top-level application (e.g., "RessourcesHumaines") |
+| 3 | Module | `core.nav_Modules` | 02 | Functional module (e.g., "Employees", "TimeManagement") |
+| 4 | **Section** | **`core.nav_Sections`** | **03a-03c** | **Page within module (e.g., "list", "detail", "create", "dashboard")** |
+| 5 | **Resource** | **`core.nav_Resources`** | **03b-03c** | **React component (SmartTable, SmartForm, Chart, KpiCard, etc.)** |
+
+**Hierarchical relationship:**
+- Context → Application → Module → **Section** → **Resource**
+- Each Section MUST have ≥1 Resource
+- Each Section corresponds to 1 React page route
+- Each Resource corresponds to 1 SmartStack UI component
+
+**SeedData generation (step-03c):**
+
+The `specification.seedDataCore` contains **7 mandatory arrays** for database seeding:
+
+| Array | Table | Source | Description |
+|-------|-------|--------|-------------|
+| `navigationModules` | `core.nav_Modules` | Manual (step-03c) | Module entry (Level 3) |
+| **`navigationSections`** | **`core.nav_Sections`** | **Derived from `specification.sections[]`** | **Section entries (Level 4)** |
+| **`navigationResources`** | **`core.nav_Resources`** | **Derived from `specification.sections[].resources[]`** | **Resource entries (Level 5)** |
+| `navigationTranslations` | `core.nav_Translations` | Manual (i18n) | Multi-language labels |
+| `permissions` | `core.Permissions` | Manual (step-03c) | Permission definitions |
+| `rolePermissions` | `core.RolePermissions` | Manual (step-03c) | Role-permission mappings |
+| `permissionConstants` | `PermissionConstants.cs` | Manual (step-03c) | C# permission constants |
+
+**Transform algorithm (step-03c section 8f-bis):**
+- `navigationSections` = transform `specification.sections[]` → flatten to { code, label, icon, route, parentCode, permission, sort }
+- `navigationResources` = transform `specification.sections[].resources[]` → flatten to { code, type, entity, parentCode, permission }
+
+**Example:**
+- Module: `Employees` (Level 3)
+  - Section: `list` (Level 4) → contains resources:
+    - Resource: `employees-grid` (SmartTable) (Level 5)
+    - Resource: `status-filter` (FilterBar) (Level 5)
+  - Section: `detail` (Level 4) → contains resources:
+    - Resource: `employee-card` (DetailCard) (Level 5)
+    - Resource: `contract-history` (SmartTable) (Level 5)
+
+</navigation_hierarchy>
+
 <resume_workflow>
 **Update mode (auto-detected):**
 
@@ -100,14 +150,21 @@ When step-00 detects that the description matches an existing application:
 - **Step 03c:** Per-module compile: actors, UCs, FRs, permissions, navigation, seed data, i18n
 - **Step 03d:** Per-module validate: completeness checks, write feature.json, incremental HTML, loop
   - Loop: 03a → 03b → 03c → 03d → 03a (next module) until all specified (specified)
-- **Step 04:** Cross-module consolidation (consolidated)
+- **Step 04a:** Collect: module summaries, cross-module interactions (FK, events, shared data)
+- **Step 04b:** Analyze: permission coherence, semantic validation, E2E flows, global risks
+- **Step 04c:** Decide: final approval, write consolidation section, proceed to handoff (consolidated)
 - **Step 05a:** Handoff: file mapping (7 categories), BR-to-code mapping, API summary, write to feature.json
 - **Step 05b:** Deploy: prd.json, progress.txt, manifest, ba-interactive.html pre-populated (handed-off)
+- **Step 05c:** Ralph Readiness Check: validation gate before /ralph-loop (optional but recommended)
 
 **Update workflow (same phases, delta focus):**
 - **Step 00:** Detection, locate existing feature, create version N+1
 - **Step 01:** Cadrage delta: what changes, impact on existing
 - **Step 02-05:** Same flow as new, with existing context loaded
+
+**Review workflow (`-review` flag):**
+- **Step 00:** Detection, scan for latest application, locate `ba-review.json`
+- **Step 06:** Apply corrections, create new version, reverse-map data, regenerate all artifacts
 </workflow>
 
 <state_variables>
@@ -144,11 +201,15 @@ When step-00 detects that the description matches an existing application:
 | 02   | `steps/step-02-decomposition.md`  | Opus   | Module decomposition, dependency graph, client checkpoint |
 | 03a  | `steps/step-03a-data.md`          | Opus   | Per-module: sections, entities, BRs, questionnaires |
 | 03b  | `steps/step-03b-ui.md`            | Opus   | Per-module: state machines, wireframes, layouts, dashboards |
-| 03c  | `steps/step-03c-compile.md`       | Opus   | Per-module: actors, UCs, FRs, permissions, nav, seed data, i18n |
+| 03c  | `steps/step-03c-compile.md`       | Opus   | Per-module: actors, UCs, FRs, permissions, navigation, seedDataCore (7 arrays), i18n |
 | 03d  | `steps/step-03d-validate.md`      | Sonnet | Per-module: validation, write, incremental HTML, loop decision |
-| 04   | `steps/step-04-consolidation.md`  | Opus   | Cross-module validation, E2E flows, permissions coherence |
+| 04a  | `steps/step-04a-collect.md`       | Opus   | Collect module summaries & cross-module interactions |
+| 04b  | `steps/step-04b-analyze.md`       | Opus   | Analyze permission coherence, semantic validation, E2E flows |
+| 04c  | `steps/step-04c-decide.md`        | Opus   | Final approval, write consolidation, proceed to handoff |
 | 05a  | `steps/step-05a-handoff.md`       | Sonnet | Handoff: file mapping (7 categories), BR-to-code mapping, API summary, write handoff |
-| 05b  | `steps/step-05b-deploy.md`        | Sonnet | Deploy: prd.json, progress.txt, manifest, ba-interactive.html, auto-launch ralph-loop |
+| 05b  | `steps/step-05b-deploy.md`        | Sonnet | Deploy: prd.json, progress.txt, manifest, ba-interactive.html |
+| 05c  | `steps/step-05c-ralph-readiness.md` | Sonnet | Ralph readiness validation gate (optional but recommended) |
+| 06   | `steps/step-06-review.md`         | Opus   | Apply review corrections, create new version, regenerate all artifacts |
 
 </step_files>
 

package/templates/skills/business-analyse/html/ba-interactive.html

@@ -1380,6 +1380,18 @@ body {
     font-size: 0.65rem;
 }
 
+/* Review save button */
+.btn-review {
+    background: var(--success);
+    color: #fff;
+    border-color: var(--success);
+    font-weight: 600;
+}
+.btn-review:hover {
+    background: #16a34a;
+    border-color: #16a34a;
+}
+
 @media (max-width: 768px) {
     .body.review-open {
         grid-template-columns: 1fr;
@@ -1408,6 +1420,7 @@ body {
             <div class="header-spacer"></div>
             <div class="header-actions">
                 <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>
                 <button class="btn btn-sm review-toggle-btn" id="reviewToggleBtn" onclick="toggleReviewPanel()" title="Ouvrir/fermer le panneau de review">
                     Review
@@ -2009,6 +2022,7 @@ body {
    ============================================ */
 const APP_KEY = 'ba-{{APPLICATION_ID}}';
 let data = {{FEATURE_DATA}};
+const ORIGINAL_DATA = JSON.parse(JSON.stringify(data));
 const EMBEDDED_ARTIFACTS = {{EMBEDDED_ARTIFACTS}};
 
 // Initialize optional data structures
@@ -2605,6 +2619,8 @@ function renderAllModuleSpecs() {
     });
     // Restore currently visible section after re-render
     restoreCurrentSection();
+    // Re-initialize inline comment buttons on module spec cards
+    if (typeof initInlineComments === 'function') initInlineComments();
 }
 
 function renderModuleSpecSection(mod) {
@@ -3007,6 +3023,17 @@ function renderModuleMockups(code) {
 }
 
 function getPermRoles() {
+    // Extract roles from actual permission data (handles English role names from feature.json)
+    const rolesFromPerms = [];
+    const seen = new Set();
+    data.modules.forEach(m => {
+        (data.moduleSpecs[m.code]?.permissions || []).forEach(p => {
+            const role = p.split('|')[0];
+            if (role && !seen.has(role)) { seen.add(role); rolesFromPerms.push(role); }
+        });
+    });
+    if (rolesFromPerms.length > 0) return [...rolesFromPerms, ...(data.customRoles || [])];
+    // Fallback: use stakeholder names if no permission data exists
     const baseRoles = data.cadrage.stakeholders.length > 0
         ? data.cadrage.stakeholders.map(s => s.role)
         : ['Administrateur', 'Responsable', 'Contributeur', 'Lecteur'];
@@ -3021,9 +3048,7 @@ function getPermActions() {
 function renderPermissionGrid(code) {
     const roles = getPermRoles();
     const actions = getPermActions();
-    const baseRolesCount = data.cadrage.stakeholders.length > 0
-        ? data.cadrage.stakeholders.length
-        : 4;
+    const baseRolesCount = roles.length - (data.customRoles || []).length;
     const baseActionsCount = 6;
 
     const perms = data.moduleSpecs[code]?.permissions || [];
@@ -3040,7 +3065,8 @@ function renderPermissionGrid(code) {
                         <td style="font-weight:500;color:var(--text-bright);">${role}${ri >= baseRolesCount ? ' <span onclick="removeCustomRole('+`'${code}','${role}'`+')" style="cursor:pointer;color:var(--danger);font-size:0.7rem;" title="Supprimer ce role">&#10005;</span>' : ''}</td>
                         ${actions.map(action => {
                             const key = role + '|' + action;
-                            const checked = perms.includes(key);
+                            const wildcardKey = role + '|*';
+                            const checked = perms.includes(key) || perms.includes(wildcardKey);
                             return `<td style="text-align:center;"><input type="checkbox" ${checked ? 'checked' : ''} onchange="togglePermission('${code}','${key}',this.checked)" style="cursor:pointer;width:16px;height:16px;"></td>`;
                         }).join('')}
                     </tr>
@@ -3171,10 +3197,15 @@ function renderConsolPermissions() {
     roles.forEach(role => {
         html += `<tr><td style="font-weight:500;color:var(--text-bright);">${role}</td>`;
         data.modules.forEach(m => {
-            const perms = (data.moduleSpecs[m.code]?.permissions || []).filter(p => p.startsWith(role + '|'));
+            const allPerms = data.moduleSpecs[m.code]?.permissions || [];
+            const hasWildcard = allPerms.includes(role + '|*');
+            const perms = hasWildcard
+                ? getPermActions()
+                : allPerms.filter(p => p.startsWith(role + '|'));
             const count = perms.length;
             const color = count > 4 ? 'var(--success)' : count > 2 ? 'var(--warning)' : count > 0 ? 'var(--text-muted)' : 'var(--border)';
-            html += `<td style="text-align:center;color:${color};font-weight:600;">${count > 0 ? count + ' droits' : 'Aucun'}</td>`;
+            const label = hasWildcard ? 'Tous' : (count > 0 ? count + ' droits' : 'Aucun');
+            html += `<td style="text-align:center;color:${color};font-weight:600;">${label}</td>`;
         });
         html += '</tr>';
     });
@@ -3446,6 +3477,109 @@ function exportJSON() {
     showNotification('Export JSON telecharge');
 }
 
+/* ============================================
+   SAVE REVIEW JSON (corrections for new version)
+   ============================================ */
+function detectChanges(original, current) {
+    const changes = { cadrage: false, modulesAdded: [], modulesRemoved: [], modulesModified: [], commentsCount: 0 };
+
+    // Cadrage changes
+    if (JSON.stringify(original.cadrage) !== JSON.stringify(current.cadrage)) {
+        changes.cadrage = true;
+    }
+
+    // Module changes
+    const origCodes = (original.modules || []).map(m => m.code);
+    const currCodes = (current.modules || []).map(m => m.code);
+    changes.modulesAdded = currCodes.filter(c => !origCodes.includes(c));
+    changes.modulesRemoved = origCodes.filter(c => !currCodes.includes(c));
+    currCodes.filter(c => origCodes.includes(c)).forEach(code => {
+        const origSpec = JSON.stringify(original.moduleSpecs?.[code] || {});
+        const currSpec = JSON.stringify(current.moduleSpecs?.[code] || {});
+        if (origSpec !== currSpec) changes.modulesModified.push(code);
+    });
+
+    // Comments count
+    changes.commentsCount = (current.comments || []).length;
+
+    return changes;
+}
+
+function saveReviewJSON() {
+    // Collect all editable fields (same as exportJSON)
+    document.querySelectorAll('.editable[data-field]').forEach(el => {
+        setNestedValue(data, 'cadrage.' + el.dataset.field, el.textContent.trim());
+    });
+    document.querySelectorAll('.editable[data-module-field]').forEach(el => {
+        const code = el.dataset.moduleCode;
+        const field = el.dataset.moduleField;
+        if (data.moduleSpecs[code]) {
+            data.moduleSpecs[code][field] = el.textContent.trim();
+        }
+    });
+
+    const changes = detectChanges(ORIGINAL_DATA, data);
+    const hasChanges = changes.cadrage || changes.modulesAdded.length > 0
+        || changes.modulesRemoved.length > 0 || changes.modulesModified.length > 0
+        || changes.commentsCount > 0;
+
+    // Build review export with metadata envelope
+    const reviewData = {
+        _reviewMeta: {
+            sourceVersion: data.metadata.version || '1.0',
+            sourceApplicationId: data.metadata.applicationId || '',
+            exportedAt: new Date().toISOString(),
+            hasChanges: hasChanges,
+            changeSummary: {
+                cadrageModified: changes.cadrage,
+                modulesAdded: changes.modulesAdded,
+                modulesRemoved: changes.modulesRemoved,
+                modulesModified: changes.modulesModified,
+                commentsCount: changes.commentsCount
+            }
+        },
+        metadata: data.metadata,
+        cadrage: data.cadrage,
+        modules: data.modules,
+        dependencies: data.dependencies,
+        moduleSpecifications: {},
+        consolidation: data.consolidation,
+        wireframeComments: data.wireframeComments,
+        specComments: data.specComments,
+        customRoles: data.customRoles,
+        customActions: data.customActions,
+        comments: data.comments
+    };
+
+    // Structure module specs (same logic as exportJSON)
+    data.modules.forEach(m => {
+        const spec = data.moduleSpecs[m.code] || {};
+        reviewData.moduleSpecifications[m.code] = {
+            module: m,
+            useCases: (spec.useCases || []).map((uc, i) => ({
+                id: 'UC-' + String(i + 1).padStart(3, '0'),
+                ...uc
+            })),
+            businessRules: (spec.businessRules || []).map((br, i) => ({
+                id: 'BR-' + br.category.toUpperCase().substring(0, 4) + '-' + String(i + 1).padStart(3, '0'),
+                ...br
+            })),
+            entities: spec.entities || [],
+            permissions: spec.permissions || [],
+            notes: spec.notes || ''
+        };
+    });
+
+    const blob = new Blob([JSON.stringify(reviewData, null, 2)], { type: 'application/json' });
+    const url = URL.createObjectURL(blob);
+    const a = document.createElement('a');
+    a.href = url;
+    a.download = 'ba-review.json';
+    a.click();
+    URL.revokeObjectURL(url);
+    showNotification('ba-review.json telecharge — sauvegardez-le a cote du HTML');
+}
+
 
 /* --- 10-comments.js --- */
 /* ============================================
@@ -3463,14 +3597,20 @@ function exportJSON() {
  */
 
 function initInlineComments() {
-    // Add comment buttons under each card and uc-item
+    // Add comment buttons under each card and uc-item (direct children of sections)
     document.querySelectorAll('.section').forEach(section => {
         const sectionId = section.id;
-        const cards = section.querySelectorAll(':scope > .card, :scope > .uc-item');
-        cards.forEach((card, index) => {
-            if (card.querySelector('.comment-btn-container')) return; // already initialized
-            const container = createCommentUI(sectionId, index);
-            card.appendChild(container);
+        section.querySelectorAll(':scope > .card, :scope > .uc-item').forEach((card, index) => {
+            if (card.querySelector('.comment-btn-container')) return;
+            card.appendChild(createCommentUI(sectionId, index));
+        });
+    });
+    // Second pass: nested list containers in module specs (ucList-*, brList-*, entList-*)
+    document.querySelectorAll('[id^="ucList-"], [id^="brList-"], [id^="entList-"]').forEach(list => {
+        const listId = list.id;
+        list.querySelectorAll(':scope > .uc-item, :scope > .entity-block, :scope > div').forEach((item, index) => {
+            if (item.querySelector('.comment-btn-container')) return;
+            item.appendChild(createCommentUI(listId, index));
         });
     });
 }
@@ -3738,12 +3878,34 @@ function getSectionLabel(sectionId) {
         const mod = data.modules.find(m => m.code === code);
         return mod ? mod.name : code;
     }
+    // Handle list-based sectionIds (ucList-*, brList-*, entList-*)
+    const listMatch = sectionId.match(/^(uc|br|ent)List-(.+)$/);
+    if (listMatch) {
+        const tabLabels = { uc: 'Cas d\'utilisation', br: 'Regles metier', ent: 'Donnees' };
+        const mod = data.modules.find(m => m.code === listMatch[2]);
+        const modName = mod ? mod.name : listMatch[2];
+        return modName + ' > ' + (tabLabels[listMatch[1]] || listMatch[1]);
+    }
     return sectionId;
 }
 
 function navigateToComment(sectionId, cardIndex) {
-    showSection(sectionId);
-    // Scroll to the card and open its comment thread
+    // Handle list-based sectionIds (ucList-*, brList-*, entList-*) → navigate to module + tab
+    const listMatch = sectionId.match(/^(uc|br|ent)List-(.+)$/);
+    if (listMatch) {
+        const [, tabType, moduleCode] = listMatch;
+        showSection('module-spec-' + moduleCode);
+        setTimeout(() => {
+            switchTab(moduleCode, tabType);
+            scrollToCommentThread(sectionId, cardIndex);
+        }, 150);
+    } else {
+        showSection(sectionId);
+        scrollToCommentThread(sectionId, cardIndex);
+    }
+}
+
+function scrollToCommentThread(sectionId, cardIndex) {
     setTimeout(() => {
         const thread = document.getElementById('comment-thread-' + sectionId + '-' + cardIndex);
         if (thread && !thread.classList.contains('visible')) {

package/templates/skills/business-analyse/html/src/scripts/01-data-init.js

@@ -3,6 +3,7 @@
    ============================================ */
 const APP_KEY = 'ba-{{APPLICATION_ID}}';
 let data = {{FEATURE_DATA}};
+const ORIGINAL_DATA = JSON.parse(JSON.stringify(data));
 const EMBEDDED_ARTIFACTS = {{EMBEDDED_ARTIFACTS}};
 
 // Initialize optional data structures

package/templates/skills/business-analyse/html/src/scripts/05-render-specs.js

@@ -17,6 +17,8 @@ function renderAllModuleSpecs() {
     });
     // Restore currently visible section after re-render
     restoreCurrentSection();
+    // Re-initialize inline comment buttons on module spec cards
+    if (typeof initInlineComments === 'function') initInlineComments();
 }
 
 function renderModuleSpecSection(mod) {
@@ -419,6 +421,17 @@ function renderModuleMockups(code) {
 }
 
 function getPermRoles() {
+    // Extract roles from actual permission data (handles English role names from feature.json)
+    const rolesFromPerms = [];
+    const seen = new Set();
+    data.modules.forEach(m => {
+        (data.moduleSpecs[m.code]?.permissions || []).forEach(p => {
+            const role = p.split('|')[0];
+            if (role && !seen.has(role)) { seen.add(role); rolesFromPerms.push(role); }
+        });
+    });
+    if (rolesFromPerms.length > 0) return [...rolesFromPerms, ...(data.customRoles || [])];
+    // Fallback: use stakeholder names if no permission data exists
     const baseRoles = data.cadrage.stakeholders.length > 0
         ? data.cadrage.stakeholders.map(s => s.role)
         : ['Administrateur', 'Responsable', 'Contributeur', 'Lecteur'];
@@ -433,9 +446,7 @@ function getPermActions() {
 function renderPermissionGrid(code) {
     const roles = getPermRoles();
     const actions = getPermActions();
-    const baseRolesCount = data.cadrage.stakeholders.length > 0
-        ? data.cadrage.stakeholders.length
-        : 4;
+    const baseRolesCount = roles.length - (data.customRoles || []).length;
     const baseActionsCount = 6;
 
     const perms = data.moduleSpecs[code]?.permissions || [];
@@ -452,7 +463,8 @@ function renderPermissionGrid(code) {
                         <td style="font-weight:500;color:var(--text-bright);">${role}${ri >= baseRolesCount ? ' <span onclick="removeCustomRole('+`'${code}','${role}'`+')" style="cursor:pointer;color:var(--danger);font-size:0.7rem;" title="Supprimer ce role">&#10005;</span>' : ''}</td>
                         ${actions.map(action => {
                             const key = role + '|' + action;
-                            const checked = perms.includes(key);
+                            const wildcardKey = role + '|*';
+                            const checked = perms.includes(key) || perms.includes(wildcardKey);
                             return `<td style="text-align:center;"><input type="checkbox" ${checked ? 'checked' : ''} onchange="togglePermission('${code}','${key}',this.checked)" style="cursor:pointer;width:16px;height:16px;"></td>`;
                         }).join('')}
                     </tr>

package/templates/skills/business-analyse/html/src/scripts/06-render-consolidation.js

@@ -38,10 +38,15 @@ function renderConsolPermissions() {
     roles.forEach(role => {
         html += `<tr><td style="font-weight:500;color:var(--text-bright);">${role}</td>`;
         data.modules.forEach(m => {
-            const perms = (data.moduleSpecs[m.code]?.permissions || []).filter(p => p.startsWith(role + '|'));
+            const allPerms = data.moduleSpecs[m.code]?.permissions || [];
+            const hasWildcard = allPerms.includes(role + '|*');
+            const perms = hasWildcard
+                ? getPermActions()
+                : allPerms.filter(p => p.startsWith(role + '|'));
             const count = perms.length;
             const color = count > 4 ? 'var(--success)' : count > 2 ? 'var(--warning)' : count > 0 ? 'var(--text-muted)' : 'var(--border)';
-            html += `<td style="text-align:center;color:${color};font-weight:600;">${count > 0 ? count + ' droits' : 'Aucun'}</td>`;
+            const label = hasWildcard ? 'Tous' : (count > 0 ? count + ' droits' : 'Aucun');
+            html += `<td style="text-align:center;color:${color};font-weight:600;">${label}</td>`;
         });
         html += '</tr>';
     });

package/templates/skills/business-analyse/html/src/scripts/09-export.js

@@ -63,3 +63,106 @@ function exportJSON() {
     URL.revokeObjectURL(url);
     showNotification('Export JSON telecharge');
 }
+
+/* ============================================
+   SAVE REVIEW JSON (corrections for new version)
+   ============================================ */
+function detectChanges(original, current) {
+    const changes = { cadrage: false, modulesAdded: [], modulesRemoved: [], modulesModified: [], commentsCount: 0 };
+
+    // Cadrage changes
+    if (JSON.stringify(original.cadrage) !== JSON.stringify(current.cadrage)) {
+        changes.cadrage = true;
+    }
+
+    // Module changes
+    const origCodes = (original.modules || []).map(m => m.code);
+    const currCodes = (current.modules || []).map(m => m.code);
+    changes.modulesAdded = currCodes.filter(c => !origCodes.includes(c));
+    changes.modulesRemoved = origCodes.filter(c => !currCodes.includes(c));
+    currCodes.filter(c => origCodes.includes(c)).forEach(code => {
+        const origSpec = JSON.stringify(original.moduleSpecs?.[code] || {});
+        const currSpec = JSON.stringify(current.moduleSpecs?.[code] || {});
+        if (origSpec !== currSpec) changes.modulesModified.push(code);
+    });
+
+    // Comments count
+    changes.commentsCount = (current.comments || []).length;
+
+    return changes;
+}
+
+function saveReviewJSON() {
+    // Collect all editable fields (same as exportJSON)
+    document.querySelectorAll('.editable[data-field]').forEach(el => {
+        setNestedValue(data, 'cadrage.' + el.dataset.field, el.textContent.trim());
+    });
+    document.querySelectorAll('.editable[data-module-field]').forEach(el => {
+        const code = el.dataset.moduleCode;
+        const field = el.dataset.moduleField;
+        if (data.moduleSpecs[code]) {
+            data.moduleSpecs[code][field] = el.textContent.trim();
+        }
+    });
+
+    const changes = detectChanges(ORIGINAL_DATA, data);
+    const hasChanges = changes.cadrage || changes.modulesAdded.length > 0
+        || changes.modulesRemoved.length > 0 || changes.modulesModified.length > 0
+        || changes.commentsCount > 0;
+
+    // Build review export with metadata envelope
+    const reviewData = {
+        _reviewMeta: {
+            sourceVersion: data.metadata.version || '1.0',
+            sourceApplicationId: data.metadata.applicationId || '',
+            exportedAt: new Date().toISOString(),
+            hasChanges: hasChanges,
+            changeSummary: {
+                cadrageModified: changes.cadrage,
+                modulesAdded: changes.modulesAdded,
+                modulesRemoved: changes.modulesRemoved,
+                modulesModified: changes.modulesModified,
+                commentsCount: changes.commentsCount
+            }
+        },
+        metadata: data.metadata,
+        cadrage: data.cadrage,
+        modules: data.modules,
+        dependencies: data.dependencies,
+        moduleSpecifications: {},
+        consolidation: data.consolidation,
+        wireframeComments: data.wireframeComments,
+        specComments: data.specComments,
+        customRoles: data.customRoles,
+        customActions: data.customActions,
+        comments: data.comments
+    };
+
+    // Structure module specs (same logic as exportJSON)
+    data.modules.forEach(m => {
+        const spec = data.moduleSpecs[m.code] || {};
+        reviewData.moduleSpecifications[m.code] = {
+            module: m,
+            useCases: (spec.useCases || []).map((uc, i) => ({
+                id: 'UC-' + String(i + 1).padStart(3, '0'),
+                ...uc
+            })),
+            businessRules: (spec.businessRules || []).map((br, i) => ({
+                id: 'BR-' + br.category.toUpperCase().substring(0, 4) + '-' + String(i + 1).padStart(3, '0'),
+                ...br
+            })),
+            entities: spec.entities || [],
+            permissions: spec.permissions || [],
+            notes: spec.notes || ''
+        };
+    });
+
+    const blob = new Blob([JSON.stringify(reviewData, null, 2)], { type: 'application/json' });
+    const url = URL.createObjectURL(blob);
+    const a = document.createElement('a');
+    a.href = url;
+    a.download = 'ba-review.json';
+    a.click();
+    URL.revokeObjectURL(url);
+    showNotification('ba-review.json telecharge — sauvegardez-le a cote du HTML');
+}

package/templates/skills/business-analyse/html/src/scripts/10-comments.js

@@ -13,14 +13,20 @@
  */
 
 function initInlineComments() {
-    // Add comment buttons under each card and uc-item
+    // Add comment buttons under each card and uc-item (direct children of sections)
     document.querySelectorAll('.section').forEach(section => {
         const sectionId = section.id;
-        const cards = section.querySelectorAll(':scope > .card, :scope > .uc-item');
-        cards.forEach((card, index) => {
-            if (card.querySelector('.comment-btn-container')) return; // already initialized
-            const container = createCommentUI(sectionId, index);
-            card.appendChild(container);
+        section.querySelectorAll(':scope > .card, :scope > .uc-item').forEach((card, index) => {
+            if (card.querySelector('.comment-btn-container')) return;
+            card.appendChild(createCommentUI(sectionId, index));
+        });
+    });
+    // Second pass: nested list containers in module specs (ucList-*, brList-*, entList-*)
+    document.querySelectorAll('[id^="ucList-"], [id^="brList-"], [id^="entList-"]').forEach(list => {
+        const listId = list.id;
+        list.querySelectorAll(':scope > .uc-item, :scope > .entity-block, :scope > div').forEach((item, index) => {
+            if (item.querySelector('.comment-btn-container')) return;
+            item.appendChild(createCommentUI(listId, index));
         });
     });
 }

package/templates/skills/business-analyse/html/src/scripts/11-review-panel.js

@@ -120,12 +120,34 @@ function getSectionLabel(sectionId) {
         const mod = data.modules.find(m => m.code === code);
         return mod ? mod.name : code;
     }
+    // Handle list-based sectionIds (ucList-*, brList-*, entList-*)
+    const listMatch = sectionId.match(/^(uc|br|ent)List-(.+)$/);
+    if (listMatch) {
+        const tabLabels = { uc: 'Cas d\'utilisation', br: 'Regles metier', ent: 'Donnees' };
+        const mod = data.modules.find(m => m.code === listMatch[2]);
+        const modName = mod ? mod.name : listMatch[2];
+        return modName + ' > ' + (tabLabels[listMatch[1]] || listMatch[1]);
+    }
     return sectionId;
 }
 
 function navigateToComment(sectionId, cardIndex) {
-    showSection(sectionId);
-    // Scroll to the card and open its comment thread
+    // Handle list-based sectionIds (ucList-*, brList-*, entList-*) → navigate to module + tab
+    const listMatch = sectionId.match(/^(uc|br|ent)List-(.+)$/);
+    if (listMatch) {
+        const [, tabType, moduleCode] = listMatch;
+        showSection('module-spec-' + moduleCode);
+        setTimeout(() => {
+            switchTab(moduleCode, tabType);
+            scrollToCommentThread(sectionId, cardIndex);
+        }, 150);
+    } else {
+        showSection(sectionId);
+        scrollToCommentThread(sectionId, cardIndex);
+    }
+}
+
+function scrollToCommentThread(sectionId, cardIndex) {
     setTimeout(() => {
         const thread = document.getElementById('comment-thread-' + sectionId + '-' + cardIndex);
         if (thread && !thread.classList.contains('visible')) {

package/templates/skills/business-analyse/html/src/styles/08-review-panel.css

@@ -215,6 +215,18 @@
     font-size: 0.65rem;
 }
 
+/* Review save button */
+.btn-review {
+    background: var(--success);
+    color: #fff;
+    border-color: var(--success);
+    font-weight: 600;
+}
+.btn-review:hover {
+    background: #16a34a;
+    border-color: #16a34a;
+}
+
 @media (max-width: 768px) {
     .body.review-open {
         grid-template-columns: 1fr;