@atlashub/smartstack-cli 4.21.0 → 4.23.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlashub/smartstack-cli",
-  "version": "4.21.0",
+  "version": "4.23.0",
   "description": "SmartStack Claude Code automation toolkit - GitFlow, EF Core migrations, prompts and more",
   "author": {
     "name": "SmartStack",

package/templates/skills/ba-design-ui/SKILL.md

@@ -0,0 +1,91 @@
+---
+name: ba-design-ui
+description: "Design user interfaces for each module: screens, wireframes, and navigation tree from Phase 1 analysis data."
+model: opus
+user_invocable: true
+---
+
+# /ba-design-ui
+
+Design user interfaces for each module based on the business analysis data (entities, use cases, permissions) produced by `/business-analyse`.
+
+## Triggers
+
+- Explicit invocation: `/ba-design-ui <feature-id>`
+- After `/business-analyse` step-04 (consolidation complete)
+- After `/ba-review` when corrections target UI/screens
+
+## Parameters
+
+| Parameter | Required | Description |
+|-----------|----------|-------------|
+| `<feature-id>` | Yes | Feature identifier (e.g., FEAT-001) |
+| `--module` | No | Restrict to a single module (e.g., HumanResources) |
+
+## Prerequisites
+
+- `/business-analyse` must be completed (step-04 consolidated)
+- JSON files must exist: `index.json`, `entities.json`, `usecases.json`, `permissions.json` per module
+- Feature status must be `consolidated` or `reviewed`
+
+## Workflow
+
+3 progressive steps:
+
+| Step | File | Role |
+|------|------|------|
+| 01 | `steps/step-01-screens.md` | Define sections/resources with detailed specs per module |
+| 02 | `steps/step-02-wireframes.md` | Generate wireframes (ASCII/HTML) for each screen |
+| 03 | `steps/step-03-navigation.md` | Define the complete navigation tree (seedDataCore) |
+
+## Input
+
+For each module, load via ba-reader:
+- `index.json` (module metadata, anticipatedSections)
+- `entities.json` (attributes with types, relationships)
+- `usecases.json` (actor actions, scenarios)
+- `permissions.json` (role matrix, permission paths)
+
+## Output
+
+Per module:
+- `screens.json` — Detailed interface specifications (SmartTable columns, SmartForm fields, SmartDashboard KPIs)
+- Wireframes embedded in screens.json (mockupFormat, elements[], layout, componentMapping)
+
+Project-level:
+- `navigation.json` — Complete navigation tree (routes, menus, breadcrumbs)
+
+## Absolute Rules
+
+- **NEVER** invent entities or permissions not defined in Phase 1 — UI specs MUST reference existing analysis data
+- **Every screen** MUST have a `permission` field referencing a valid permission path from permissions.json
+- **Every SmartTable column** MUST reference a valid entity attribute or relationship
+- **Every SmartForm field** MUST reference a valid entity attribute
+- **screens.json** MUST have at least one section per module
+
+## Post-Design Workflow
+
+```
+/ba-design-ui completes
+    |
+    v
+/ba-generate-html (generates interactive HTML with mockups, ERD, permission matrix)
+    |
+    v
+Client reviews HTML
+    |
+    v
+/ba-review (apply corrections, route back to /business-analyse or /ba-design-ui)
+```
+
+## References
+
+| File | Content |
+|------|---------|
+| `/business-analyse` skill | Source analysis data (entities, UCs, permissions) |
+| `/ba-generate-html` skill | HTML generation consuming screens.json |
+| `ba-generate-html/references/data-mapping.md` | JSON to FEATURE_DATA field mapping |
+
+## Entry Point
+
+**FIRST ACTION:** Load `steps/step-01-screens.md`

package/templates/skills/ba-design-ui/steps/step-01-screens.md

@@ -0,0 +1,177 @@
+---
+name: step-01-screens
+description: Define sections and resources with detailed UI specs per module
+model: opus
+next_step: steps/step-02-wireframes.md
+---
+
+# Step 01 — Screens (Define Sections & Resources)
+
+## Objective
+
+For each module, define all sections and resources with detailed interface specifications: SmartTable columns, SmartForm fields with types, SmartDashboard KPIs, SmartKanban columns.
+
+## Prerequisites
+
+- `/business-analyse` completed (entities, usecases, permissions exist)
+- Module index.json accessible via ba-reader
+
+## Execution
+
+### 1. Load Analysis Data
+
+For each module (in dependency order from index.json):
+
+```
+Load via ba-reader:
+  - entities.json → entity attributes with types, relationships
+  - usecases.json → actor actions, CRUD operations
+  - permissions.json → role matrix, permission paths
+  - index.json → anticipatedSections from step-02
+```
+
+### 2. Define Sections & Resources
+
+For each module, transform anticipatedSections into detailed screen specs.
+
+**SmartTable resource:**
+```json
+{
+  "code": "{entity-plural}-grid",
+  "type": "SmartTable",
+  "label": "Grille des {entity-plural}",
+  "columns": [
+    { "field": "code", "label": "Code", "type": "text", "sortable": true, "width": "100px" },
+    { "field": "fullName", "label": "Nom", "type": "text", "sortable": true },
+    { "field": "department.name", "label": "Departement", "type": "lookup", "filterable": true },
+    { "field": "status", "label": "Statut", "type": "badge", "filterable": true, "options": ["Active", "Inactive"] }
+  ],
+  "actions": ["create", "edit", "delete", "export"],
+  "filters": [
+    { "field": "department", "type": "select", "label": "Departement" },
+    { "field": "status", "type": "select", "label": "Statut" }
+  ],
+  "permission": "{AppCode}.{ModuleCode}.Read",
+  "rowActions": ["edit", "view-detail"],
+  "defaultSort": { "field": "code", "direction": "asc" },
+  "pageSize": 25
+}
+```
+
+**SmartForm resource:**
+```json
+{
+  "code": "{entity}-form",
+  "type": "SmartForm",
+  "label": "Fiche {entity}",
+  "tabs": [
+    {
+      "label": "Informations",
+      "fields": [
+        { "field": "firstName", "label": "Prenom", "type": "text", "required": true, "col": 6 },
+        { "field": "lastName", "label": "Nom", "type": "text", "required": true, "col": 6 },
+        { "field": "email", "label": "Email", "type": "email", "required": true, "validation": "unique" },
+        { "field": "department", "label": "Departement", "type": "lookup", "entity": "Department", "required": true },
+        { "field": "status", "label": "Statut", "type": "select", "options": ["Active", "Inactive"], "readonly": "onCreate" }
+      ]
+    },
+    {
+      "label": "Contrats",
+      "type": "subtable",
+      "entity": "Contract",
+      "columns": ["contractType", "startDate", "endDate", "status"]
+    }
+  ],
+  "permission": "{AppCode}.{ModuleCode}.Update"
+}
+```
+
+**SmartDashboard resource:**
+```json
+{
+  "code": "{module}-dashboard",
+  "type": "SmartDashboard",
+  "label": "Tableau de bord {module}",
+  "kpis": [
+    { "label": "Total actifs", "value": "count(status=Active)", "icon": "users", "color": "primary" },
+    { "label": "En conge", "value": "count(status=OnLeave)", "icon": "calendar", "color": "warning" }
+  ],
+  "charts": [
+    { "type": "pie", "label": "Repartition par departement", "dataSource": "employees", "groupBy": "department" },
+    { "type": "bar", "label": "Evolution des effectifs", "dataSource": "employees", "groupBy": "hireDate.year" }
+  ],
+  "permission": "{AppCode}.{ModuleCode}.Read"
+}
+```
+
+**SmartKanban resource:**
+```json
+{
+  "code": "{entity}-kanban",
+  "type": "SmartKanban",
+  "label": "Kanban {entity}",
+  "statusField": "status",
+  "columns": [
+    { "value": "Draft", "label": "Brouillon", "color": "gray" },
+    { "value": "Submitted", "label": "Soumis", "color": "blue" },
+    { "value": "Approved", "label": "Approuve", "color": "green" }
+  ],
+  "cardFields": ["code", "assignee", "dueDate"],
+  "permission": "{AppCode}.{ModuleCode}.Read"
+}
+```
+
+### 3. Column/Field Type Mapping
+
+Map entity attribute types to UI component types:
+
+| Entity type | SmartTable type | SmartForm type |
+|-------------|----------------|----------------|
+| `string` | `text` | `text` |
+| `int`, `decimal` | `number` | `number` |
+| `date` | `date` | `datepicker` |
+| `datetime` | `datetime` | `datetimepicker` |
+| `bool` | `boolean` | `checkbox` |
+| `enum` | `badge` | `select` |
+| `guid` (FK) | `lookup` | `lookup` |
+| `text` (long) | `text` (truncated) | `textarea` |
+
+### 4. Write screens.json
+
+For each module, write via ba-writer:
+
+```json
+{
+  "sections": [
+    {
+      "sectionCode": "list",
+      "sectionLabel": "Liste des employes",
+      "resources": [ /* SmartTable, filters */ ]
+    },
+    {
+      "sectionCode": "form",
+      "sectionLabel": "Fiche employe",
+      "resources": [ /* SmartForm with tabs */ ]
+    },
+    {
+      "sectionCode": "dashboard",
+      "sectionLabel": "Tableau de bord",
+      "resources": [ /* SmartDashboard */ ]
+    }
+  ]
+}
+```
+
+Update `index.json` with screens.json hash and status.
+
+## Validation
+
+- [ ] Every resource has a valid `permission` referencing permissions.json
+- [ ] Every SmartTable column references a valid entity attribute
+- [ ] Every SmartForm field references a valid entity attribute
+- [ ] Every section has at least one resource
+- [ ] Every module has at least one section
+
+## Transition
+
+-> Step 02 (wireframes) — Generate visual mockups for each screen

package/templates/skills/ba-design-ui/steps/step-02-wireframes.md

@@ -0,0 +1,140 @@
+---
+name: step-02-wireframes
+description: Generate wireframes (ASCII/HTML mockups) for each screen defined in step-01
+model: opus
+next_step: steps/step-03-navigation.md
+---
+
+# Step 02 — Wireframes (Generate Mockups)
+
+## Objective
+
+Generate visual wireframes for each screen defined in screens.json. Each wireframe provides a concrete representation of the UI layout, component placement, and user interactions.
+
+## Prerequisites
+
+- Step 01 (screens) completed
+- screens.json exists for each module with detailed section/resource specs
+
+## Execution
+
+### 1. Load Screen Specs
+
+For each module, read screens.json via ba-reader.
+
+### 2. Generate Wireframes per Resource
+
+For each resource in each section, generate a wireframe object:
+
+**SmartTable wireframe:**
+```json
+{
+  "screen": "{sectionCode}-{resourceCode}",
+  "section": "{sectionCode}",
+  "mockupFormat": "ascii",
+  "mockup": "+----------------------------------------------------------+\n| [Departement v] [Statut v]        [+ Nouveau employe]    |\n+------+----------------+---------------+--------+---------+\n| Code | Nom complet    | Departement   | Statut | Actions |\n+------+----------------+---------------+--------+---------+\n| E001 | Dupont Marie   | RH            | Actif  | [E] [S] |\n| E002 | Martin Pierre  | IT            | Conge  | [E] [S] |\n+------+----------------+---------------+--------+---------+\n| << 1 2 3 ... 10 >>                           25 par page |\n+----------------------------------------------------------+",
+  "elements": ["FilterBar", "DataGrid", "Pagination", "ActionButtons"],
+  "actions": ["filter", "sort", "create", "edit", "delete", "export"],
+  "componentMapping": [
+    { "wireframeElement": "FilterBar", "reactComponent": "SmartFilter" },
+    { "wireframeElement": "DataGrid", "reactComponent": "SmartTable" },
+    { "wireframeElement": "Pagination", "reactComponent": "SmartPagination" },
+    { "wireframeElement": "ActionButtons", "reactComponent": "SmartToolbar" }
+  ],
+  "layout": {
+    "type": "vertical",
+    "regions": ["filters", "grid", "pagination"]
+  },
+  "permissionsRequired": ["{AppCode}.{ModuleCode}.Read"]
+}
+```
+
+**SmartForm wireframe:**
+```json
+{
+  "screen": "{sectionCode}-{resourceCode}",
+  "section": "{sectionCode}",
+  "mockupFormat": "ascii",
+  "mockup": "+----------------------------------------------------------+\n| Fiche Employe                              [Sauvegarder] |\n+----------------------------------------------------------+\n| [Informations] [Contrats] [Documents]                    |\n+----------------------------------------------------------+\n| Prenom:      [_______________]  Nom: [_______________]   |\n| Email:       [_______________]  Tel: [_______________]   |\n| Departement: [Selectionnez v ]  Poste: [_____________]   |\n| Date embauche: [__ /__ /____]   Statut: [Actif      v]  |\n+----------------------------------------------------------+",
+  "elements": ["TabBar", "FormFields", "SaveButton"],
+  "actions": ["save", "cancel", "tab-switch"],
+  "componentMapping": [
+    { "wireframeElement": "TabBar", "reactComponent": "SmartTabs" },
+    { "wireframeElement": "FormFields", "reactComponent": "SmartForm" },
+    { "wireframeElement": "SaveButton", "reactComponent": "SmartButton" }
+  ],
+  "layout": {
+    "type": "vertical",
+    "regions": ["header", "tabs", "form-body", "actions"]
+  },
+  "permissionsRequired": ["{AppCode}.{ModuleCode}.Update"]
+}
+```
+
+**SmartDashboard wireframe:**
+```json
+{
+  "screen": "{sectionCode}-{resourceCode}",
+  "section": "{sectionCode}",
+  "mockupFormat": "ascii",
+  "mockup": "+----------------------------------------------------------+\n| Tableau de bord RH                                       |\n+-------------+-------------+-------------+----------------+\n| Total actifs| En conge    | Nouveaux    | Taux rotation  |\n|    156      |     12      |      8      |    3.2%        |\n+-------------+-------------+-------------+----------------+\n| [Repartition par dept (pie)]  [Evolution effectifs (bar)]|\n+----------------------------------------------------------+",
+  "elements": ["KPICards", "PieChart", "BarChart"],
+  "actions": ["filter-period", "export-report"],
+  "componentMapping": [
+    { "wireframeElement": "KPICards", "reactComponent": "SmartKPI" },
+    { "wireframeElement": "PieChart", "reactComponent": "SmartChart" },
+    { "wireframeElement": "BarChart", "reactComponent": "SmartChart" }
+  ],
+  "layout": {
+    "type": "grid",
+    "regions": ["kpi-row", "charts-row"]
+  },
+  "permissionsRequired": ["{AppCode}.{ModuleCode}.Read"]
+}
+```
+
+### 3. Enrich screens.json with Wireframes
+
+For each module, update screens.json to embed wireframes at the resource level:
+
+```json
+{
+  "sections": [
+    {
+      "sectionCode": "list",
+      "sectionLabel": "Liste des employes",
+      "resources": [
+        {
+          "code": "employees-grid",
+          "type": "SmartTable",
+          "columns": [ /* ... */ ],
+          "wireframe": {
+            "screen": "list-employees-grid",
+            "mockupFormat": "ascii",
+            "mockup": "...",
+            "elements": [...],
+            "componentMapping": [...],
+            "layout": {...}
+          }
+        }
+      ]
+    }
+  ]
+}
+```
+
+### 4. Write Updated screens.json
+
+Write via ba-writer for each module. Update index.json hashes.
+
+## Validation
+
+- [ ] Every resource has a wireframe object
+- [ ] Every wireframe has a non-empty `mockup` field
+- [ ] Every wireframe has `elements[]` and `componentMapping[]`
+- [ ] Wireframe `permissionsRequired` references valid permission paths
+- [ ] All wireframes have `mockupFormat` set ("ascii" or "html")
+
+## Transition
+
+-> Step 03 (navigation) — Define the complete navigation tree

package/templates/skills/ba-design-ui/steps/step-03-navigation.md

@@ -0,0 +1,134 @@
+---
+name: step-03-navigation
+description: Define the complete navigation tree (routes, menus, breadcrumbs)
+model: opus
+---
+
+# Step 03 — Navigation (Navigation Tree)
+
+## Objective
+
+Define the complete navigation tree for the application: routes, menus, breadcrumbs, and seedDataCore entries. This ensures consistent navigation across all modules.
+
+## Prerequisites
+
+- Step 01 (screens) and Step 02 (wireframes) completed
+- screens.json exists for each module with sections and resources
+
+## Execution
+
+### 1. Load All Screens
+
+Read screens.json for every module via ba-reader. Collect all sections and resources.
+
+### 2. Build Navigation Tree
+
+Build the navigation hierarchy following SmartStack conventions:
+
+```
+Application
+  |-- Module 1
+  |   |-- Section 1 (e.g., "Liste")
+  |   |   |-- Resource 1 (e.g., "employees-grid")
+  |   |-- Section 2 (e.g., "Fiche")
+  |   |   |-- Resource 1 (e.g., "employee-form")
+  |   |-- Section 3 (e.g., "Dashboard")
+  |       |-- Resource 1 (e.g., "hr-dashboard")
+  |-- Module 2
+      |-- ...
+```
+
+### 3. Generate navigation.json
+
+```json
+{
+  "application": {
+    "code": "{AppCode}",
+    "name": "{AppName}",
+    "icon": "app-icon"
+  },
+  "modules": [
+    {
+      "code": "{ModuleCode}",
+      "name": "{ModuleName}",
+      "icon": "module-icon",
+      "route": "/{app-code}/{module-code}",
+      "permission": "{AppCode}.{ModuleCode}.Read",
+      "sections": [
+        {
+          "code": "{sectionCode}",
+          "label": "{sectionLabel}",
+          "route": "/{app-code}/{module-code}/{section-code}",
+          "icon": "section-icon",
+          "resources": [
+            {
+              "code": "{resourceCode}",
+              "type": "{SmartTable|SmartForm|SmartDashboard|SmartKanban}",
+              "route": "/{app-code}/{module-code}/{section-code}/{resource-code}",
+              "permission": "{AppCode}.{ModuleCode}.{Action}"
+            }
+          ]
+        }
+      ]
+    }
+  ],
+  "menuItems": [
+    {
+      "label": "{ModuleName}",
+      "icon": "module-icon",
+      "route": "/{app-code}/{module-code}",
+      "permission": "{AppCode}.{ModuleCode}.Read",
+      "children": [
+        {
+          "label": "{sectionLabel}",
+          "route": "/{app-code}/{module-code}/{section-code}",
+          "permission": "{AppCode}.{ModuleCode}.Read"
+        }
+      ]
+    }
+  ],
+  "breadcrumbs": {
+    "/{app-code}/{module-code}/{section-code}": [
+      { "label": "{AppName}", "route": "/{app-code}" },
+      { "label": "{ModuleName}", "route": "/{app-code}/{module-code}" },
+      { "label": "{sectionLabel}", "route": "" }
+    ]
+  }
+}
+```
+
+### 4. Write navigation.json
+
+Write via ba-writer at project level (alongside master index.json). Update index.json with navigation.json reference.
+
+## Validation
+
+- [ ] Every module in index.json has a corresponding navigation entry
+- [ ] Every section in screens.json has a corresponding menu item
+- [ ] Every route is unique
+- [ ] Every menu item has a valid permission reference
+- [ ] Breadcrumbs cover all routes
+
+## Completion Summary
+
+```
+================================================================
+  UI DESIGN COMPLETE
+================================================================
+
+| Module          | Sections | Resources | Wireframes |
+|-----------------|----------|-----------|------------|
+| {moduleCode}    | {count}  | {count}   | {count}    |
+| ...             | ...      | ...       | ...        |
+
+| Artifact        | Path                              |
+|-----------------|-----------------------------------|
+| screens.json    | {module-path}/screens.json        |
+| navigation.json | {project-path}/navigation.json    |
+
+NEXT STEPS:
+1. Run /ba-generate-html to produce the interactive HTML with mockups
+2. Client reviews the HTML (mockups, ERD, permission matrix)
+3. Run /ba-review to apply corrections
+================================================================
+```

package/templates/skills/ba-generate-html/SKILL.md

@@ -25,7 +25,6 @@ Generate the interactive HTML document of the business analysis from the JSON an
 ## Prerequisites
 
 - JSON analysis files must exist (at minimum `index.json` + `cadrage.json`)
-- PRD files must exist (`.ralph/prd-{moduleCode}.json`) — run `/derive-prd` first
 - Feature status must be `consolidated`
 
 ## Workflow

package/templates/skills/ba-generate-html/references/data-build.md

@@ -77,8 +77,8 @@ const FEATURE_DATA = {
         name: ent.name,
         description: ent.description || "",
         attributes: (ent.attributes || []).length > 0
-          ? ent.attributes.map(a => ({ name: a.name, description: a.description || "" }))
-          : (ent.fields || []).map(f => ({ name: f.name, description: f.description || f.type || "" })),
+          ? ent.attributes.map(a => ({ name: a.name, type: a.type || "string", required: a.required || false, description: a.description || "" }))
+          : (ent.fields || []).map(f => ({ name: f.name, type: f.type || "string", required: f.required || false, description: f.description || "" })),
         relationships: (ent.relationships || []).length > 0
           ? ent.relationships.map(r =>
               typeof r === 'string' ? r : `${r.target} (${r.type}) - ${r.description || ""}`)

package/templates/skills/ba-generate-html/references/data-mapping.md

@@ -126,6 +126,8 @@ moduleSpecs[moduleCode] = {
     description: ent.description || "",
     attributes: (ent.attributes || []).map(a => ({
       name: a.name,
+      type: a.type || "string",                            // string, int, decimal, date, datetime, bool, enum, guid, text
+      required: a.required || false,
       description: a.description || ""
     })),
     relationships: (ent.relationships || []).map(r =>

package/templates/skills/ba-generate-html/steps/step-01-collect.md

@@ -36,20 +36,7 @@ ELSE:
 Store: workflow.mode
 ```
 
-### 3. Verify PRD Exists
-
-```
-FOR each module in modules:
-  prdPath = ".ralph/prd-{moduleCode}.json"
-  IF NOT exists(prdPath):
-    BLOCKING ERROR: "PRD not found: {prdPath} — Run /derive-prd first"
-    STOP
-  IF fileSize(prdPath) < 100:
-    BLOCKING ERROR: "PRD file too small: {prdPath} — Run /derive-prd to regenerate"
-    STOP
-```
-
-### 4. Verify Consolidation Status
+### 3. Verify Consolidation Status
 
 ```
 IF workflow.mode === "project":
@@ -64,7 +51,7 @@ ELSE:
     BLOCKING ERROR: "Feature status: {feature.status}, expected consolidated"
 ```
 
-### 5. Read All JSON Files
+### 4. Read All JSON Files
 
 ```
 collected_data = {
@@ -86,13 +73,9 @@ FOR each module in master.index.modules[]:
   }
 ```
 
-### 6. Display Collection Summary
+### 5. Display Collection Summary
 
 ```
-PRD verified: {module_count} modules
-  .ralph/prd-{module1}.json ({size} bytes)
-  .ralph/prd-{module2}.json ({size} bytes)
-
 JSON collected:
   Master: index.json + cadrage.json
   Modules: {module_count} module index files
@@ -105,7 +88,6 @@ JSON collected:
 
 - BLOCK if index.json absent
 - BLOCK if 0 modules found
-- BLOCK if any PRD file missing
 - BLOCK if feature not consolidated
 
 ## NEXT STEP

package/templates/skills/ba-generate-html/steps/step-02-build-data.md

@@ -119,8 +119,8 @@ moduleSpecs[moduleCode] = {
     name: ent.name,
     description: ent.description || "",
     attributes: (ent.attributes || []).length > 0
-      ? ent.attributes.map(a => ({ name: a.name, description: a.description || "" }))
-      : (ent.fields || []).map(f => ({ name: f.name, description: f.description || f.type || "" })),
+      ? ent.attributes.map(a => ({ name: a.name, type: a.type || "string", required: a.required || false, description: a.description || "" }))
+      : (ent.fields || []).map(f => ({ name: f.name, type: f.type || "string", required: f.required || false, description: f.description || "" })),
     relationships: (ent.relationships || []).map(r =>
       typeof r === 'string' ? r : r.target + " (" + r.type + ") - " + (r.description || ""))
   })),

package/templates/skills/ba-review/SKILL.md

@@ -32,7 +32,22 @@ Apply corrections exported from the interactive HTML review (`ba-review.json`) t
 |------|------|------|
 | 01 | `steps/step-01-apply.md` | Read ba-review.json, create new version, apply corrections to JSON files |
 
-After step-01 completes, delegate to `/ba-generate-html` to regenerate the interactive HTML document.
+After step-01 completes:
+1. Determine correction scope (analysis data vs UI design vs both)
+2. Route to the appropriate phase for re-execution
+3. Regenerate HTML via `/ba-generate-html`
+
+## Correction Routing
+
+After applying corrections, ask the client which phase to re-execute:
+
+| Correction type | Route to | Reason |
+|-----------------|----------|--------|
+| Entities, rules, UCs, permissions changed | `/business-analyse` step-03 (specify) | Re-run analysis for affected modules |
+| Screens, wireframes, navigation changed | `/ba-design-ui` | Re-run UI design for affected modules |
+| Both analysis and UI changed | `/business-analyse` step-03, then `/ba-design-ui` | Full re-specification |
+| Only cadrage/scope corrections | `/ba-generate-html` | Just regenerate HTML |
+| Client approves (no corrections) | `/derive-prd` | Proceed to PRD generation |
 
 ## References
 

package/templates/skills/ba-review/steps/step-01-apply.md

@@ -180,7 +180,31 @@ review_section = {
 Create review.json with review_section and update hash in index.json
 ```
 
-## Step 8: Regenerate Interactive HTML
+## Step 8: Determine Correction Routing
+
+Analyze the corrections applied to determine which phase needs re-execution:
+
+```
+analysis_changes = false
+ui_changes = false
+
+FOR EACH module in corrected_modules:
+  IF entities, rules, usecases, or permissions were modified:
+    analysis_changes = true
+  IF screens, wireframes, or navigation were modified:
+    ui_changes = true
+
+IF analysis_changes AND ui_changes:
+  route = "both"
+ELSE IF analysis_changes:
+  route = "analysis"
+ELSE IF ui_changes:
+  route = "ui"
+ELSE:
+  route = "html-only"
+```
+
+## Step 9: Regenerate Interactive HTML
 
 Delegate HTML generation to `/ba-generate-html` skill:
 
@@ -190,7 +214,7 @@ Delegate HTML generation to `/ba-generate-html` skill:
 3. Output: {new_version_dir}/ba-interactive.html
 ```
 
-## Step 9: Update Manifest
+## Step 10: Update Manifest
 
 ```
 1. Read docs/index.json
@@ -200,12 +224,12 @@ Delegate HTML generation to `/ba-generate-html` skill:
 5. Write docs/index.json
 ```
 
-## Step 10: Display Completion Summary
+## Step 11: Display Completion Summary
 
 ```
-═══════════════════════════════════════════════════════════════
+=====================================================================
   REVIEW CORRECTIONS APPLIED SUCCESSFULLY
-═══════════════════════════════════════════════════════════════
+=====================================================================
 
 | Field                | Value                                |
 |----------------------|--------------------------------------|
@@ -231,11 +255,37 @@ Artifacts regenerated:
 | Thematic files       | {new_version_dir}/*.json             |
 | Manifest             | docs/index.json                      |
 
-NEXT STEPS:
-1. Open the new ba-interactive.html to verify corrections
-2. Run /ralph-loop to start development from the updated analysis
-   (Use /ralph-loop -r ONLY to resume an interrupted loop)
-═══════════════════════════════════════════════════════════════
+NEXT STEPS (based on correction routing: {route}):
+```
+
+Display next steps based on `route`:
+
+```
+IF route == "analysis":
+  "1. Run /business-analyse step-03 for modules: {corrected_module_list}"
+  "2. Then run /ba-design-ui to update UI specs"
+  "3. Then run /ba-generate-html to regenerate documentation"
+
+IF route == "ui":
+  "1. Run /ba-design-ui for modules: {corrected_module_list}"
+  "2. Then run /ba-generate-html to regenerate documentation"
+
+IF route == "both":
+  "1. Run /business-analyse step-03 for modules: {analysis_corrected_modules}"
+  "2. Then run /ba-design-ui for modules: {ui_corrected_modules}"
+  "3. Then run /ba-generate-html to regenerate documentation"
+
+IF route == "html-only":
+  "HTML has been regenerated. Review the updated document."
+  "If satisfied, run /derive-prd to generate PRD files."
+
+IF no corrections (client approves):
+  "No corrections needed. Run /derive-prd to generate PRD files."
+  "Then run /ralph-loop to start development."
+```
+
+```
+=====================================================================
 ```
 
 ## Error Handling

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

@@ -23,23 +23,27 @@ user_invocable: true
 
 | Step | File | Model | Role |
 |------|------|-------|------|
-| 03 | `step-03-specify.md` | Opus | Deep dive per module: entities, BRs, UCs, permissions, screens. Sequential internal loop |
+| 03 | `step-03-specify.md` | Opus | Deep dive per module: entities, BRs, UCs, permissions. Sequential internal loop |
 | 04 | `step-04-consolidate.md` | Opus | Cross-module validation: data model, permission coherence, E2E flows |
 
 ### Post-BA Workflow
 ```
-Phase 1 (steps 00-02) → Phase 2 (steps 03-04) → /ba-generate-html → Client reviews HTML
-                                                                            ↓
-                                                               Client comments/enriches
-                                                                            ↓
-                                                               Export ba-review.json
-                                                                            ↓
-                                                               /ba-review → /ba-generate-html
-                                                                            ↓
-                                                               /derive-prd → /ralph-loop
+Phase 1: /business-analyse (steps 00-04)
+    |
+    v
+Phase 2: /ba-design-ui (screens, wireframes, navigation)
+    |
+    v
+Phase 3: /ba-generate-html (interactive HTML with mockups, ERD, permissions)
+    |
+    v
+Phase 4: /ba-review (client corrections → back to Phase 1 or 2)
+    |
+    v (client approves)
+Phase 5: /derive-prd → Phase 6: /ralph-loop
 ```
 
-> After step-04, invoke `/ba-generate-html` to generate the HTML, then `/ba-review` for the client correction cycle.
+> After step-04, invoke `/ba-design-ui` to design interfaces, then `/ba-generate-html` to generate the HTML, then `/ba-review` for the client correction cycle.
 > **PRD generation:** Handoff data and PRD files are generated by the `/derive-prd` skill.
 
 ## JSON Architecture — Granular files + Index
@@ -54,7 +58,7 @@ docs/{app}/{module}/business-analyse/v1.0/
   rules.json              ← Business rules (BRs)
   usecases.json           ← Use cases (UCs)
   permissions.json        ← Permissions, roles, access matrix
-  screens.json            ← Interface specs (sections, resources, mockups)
+  screens.json            ← Interface specs (produced by /ba-design-ui)
   validation.json         ← Validation results, checks
   handoff.json            ← File mapping, APIs, handoff data
 ```
@@ -135,15 +139,15 @@ STEP 02 (structure)
 STEP 03 (specify) — SEQUENTIAL MODULE LOOP
   → For each module (in dependency order):
     → Load: _shared.md, step-03-specify.md, module index.json
-    → Execute: entities, BRs, UCs, permissions, screens
-    → Output: entities.json, rules.json, usecases.json, permissions.json, screens.json
+    → Execute: entities, BRs, UCs, permissions
+    → Output: entities.json, rules.json, usecases.json, permissions.json
   → No team mode — sequential ensures 100% context per module
 
 STEP 04 (consolidate) — FINAL BA STEP
   → Load: _shared.md, step-04-consolidate.md, all index.json
   → Execute: Data model validation, permission coherence, E2E flows
   → Output: validation.json, consolidated data
-  → Next: invoke /ba-generate-html, then /ba-review for client cycle
+  → Next: invoke /ba-design-ui (Phase 2), then /ba-generate-html, then /ba-review for client cycle
 ```
 
 ## References

package/templates/skills/business-analyse/steps/step-03-specify.md

@@ -1,6 +1,6 @@
 ---
 name: step-03-specify
-description: Sequential deep dive per module - entities, BRs, UCs, permissions, screens
+description: Sequential deep dive per module - entities, BRs, UCs, permissions
 model: opus
 next_step: steps/step-04-consolidate.md
 ---
@@ -9,7 +9,7 @@ next_step: steps/step-04-consolidate.md
 
 ## Objective
 
-For each module (in dependency order), produce complete specifications: entities, business rules, use cases, permissions, and interface specs (screens).
+For each module (in dependency order), produce complete specifications: entities, business rules, use cases, and permissions. Interface specs (screens, wireframes, navigation) are handled by Phase 2 (`/ba-design-ui`).
 
 ## Prerequisites
 
@@ -123,40 +123,11 @@ Define the permission matrix:
 }
 ```
 
-### F. Interface Specs (Screens)
+### F. Interface Specs — Delegated to /ba-design-ui
 
-For each section/resource identified in step 02, specify the full interface:
-
-```json
-{
-  "sectionCode": "list",
-  "sectionLabel": "Liste des employes",
-  "resources": [
-    {
-      "code": "employees-grid",
-      "type": "SmartTable",
-      "label": "Grille des employes",
-      "columns": [
-        { "field": "code", "label": "Code", "type": "text", "sortable": true },
-        { "field": "fullName", "label": "Nom complet", "type": "text", "sortable": true },
-        { "field": "department.name", "label": "Departement", "type": "lookup", "filterable": true },
-        { "field": "status", "label": "Statut", "type": "badge", "filterable": true }
-      ],
-      "actions": ["create", "edit", "delete", "export"],
-      "filters": ["department", "status"],
-      "permission": "HumanResources.Employees.Read"
-    }
-  ]
-}
-```
-
-Resource types:
-- **SmartTable**: columns[], actions[], filters[], permission
-- **SmartForm**: tabs[{ label, fields[{ field, label, type, required, readonly, options }] }], actions[], permission
-- **SmartCard**: columns/fields[], permission
-- **SmartKanban**: options/columns[], permission
-- **SmartDashboard**: kpis[{ label, value }], permission
-- **SmartFilter**: options[], permission
+> **Screen specifications are NOT produced in this step.**
+> Interface design (SmartTable columns, SmartForm fields, wireframes, navigation) is handled by Phase 2: `/ba-design-ui`.
+> This separation allows iterating on UI design without replaying the full analysis cycle.
 
 ### G. Write & Advance
 
@@ -167,7 +138,6 @@ After completing all sub-steps for a module:
    - `rules.json`
    - `usecases.json`
    - `permissions.json`
-   - `screens.json`
    - Update `index.json` (hashes, summary counts)
 
 2. Update module status to `specified`
@@ -178,7 +148,8 @@ After completing all sub-steps for a module:
 
 After all modules are specified:
 - Verify all modules have status `specified`
-- Display summary: total entities, rules, UCs, permissions, screens across all modules
+- Display summary: total entities, rules, UCs, permissions across all modules
+- Remind user: "Run /ba-design-ui to design interfaces for these modules"
 - Transition to step 04
 
 ## POST-CHECK
@@ -188,7 +159,7 @@ Before advancing to step 04, verify:
 - [ ] Every module has at least 1 use case
 - [ ] Every module has at least 1 business rule
 - [ ] Every module has permissions defined
-- [ ] Every module has at least 1 screen defined
+- [ ] Entity attributes have `type` defined (string, int, date, enum, guid, etc.)
 - [ ] All entity relationships reference existing entities
 - [ ] All UC business rule references exist
 - [ ] All permission paths follow the convention