@atlashub/smartstack-cli 4.80.0 → 5.0.0

package/templates/skills/business-analyse-develop/references/section-splitting.md

@@ -1,439 +0,0 @@
-# Section-Level Splitting — Large Module Handling
-
-> **Loaded by:** step-01 section 4c (detection) and compact-loop.md (execution)
-> **Purpose:** When a module has >4 domain entities AND >1 architecture section,
-> split `/apex -d` calls per SECTION instead of sending the whole module at once.
-> **Threshold:** `domainTaskCount > 4 AND sectionCount > 1`
-> **Backward compatible:** Modules with <=4 entities are UNAFFECTED.
-
----
-
-## Why Section-Level Splitting?
-
-When a module has 7+ entities (e.g., HR with Employee, Department, Position, LeaveRequest, LeaveType, TimeTracking, WorkLog), a single `/apex -d` saturates the context window. Result: migrations incomplete, pages missing, conventions lost. Success rate drops from ~95% to ~30%.
-
-**Solution:** Phase 0 builds ALL entities + ONE migration (foundation). Phase 1..N build services/controllers/pages PER SECTION. Each section has 1-3 entities — comfortably within apex's safe zone.
-
----
-
-## Architecture: Phases
-
-```
-Phase 0: FOUNDATION (1 apex call)
-  - ALL entity classes (Domain layer)
-  - ALL EF configurations (Infrastructure layer)
-  - ONE migration covering ALL entities
-  - Application + Module navigation seed data
-  - DbContext registration + DI skeleton
-  -> Eliminates migration conflicts between sections
-
-Phase 1..N: PER SECTION (1 apex call each)
-  - Services + DTOs + Validators for section entities
-  - Controllers for section entities
-  - Section-level seed data (NavigationSection, permissions, roles)
-  - Frontend pages (List, Detail, Create, Edit) for section entities
-  - i18n for section entities
-  - Tests for section entities
-
-Phase Final: CROSS-VALIDATION (inline in step-04)
-  - dotnet build (full project)
-  - npm run typecheck
-  - MCP validate_conventions
-```
-
-### Why ALL Entities in Phase 0?
-
-EF Core creates a single ModelSnapshot per DbContext. If Phase 1 creates a migration for Employee+Department and Phase 2 for LeaveRequest+LeaveType, the second migration fails when LeaveRequest has a FK to Employee (ModelSnapshot is inconsistent). **Phase 0 = all entities + 1 migration.** Section phases add services/controllers/pages on top of existing entities.
-
----
-
-## 1. Detection
-
-```javascript
-const prd = readJSON(currentPrdPath);
-const domainTasks = prd.tasks.filter(t => t.category === 'domain');
-const sections = prd.architecture?.sections ?? [];
-
-const shouldSplit = domainTasks.length > 4 && sections.length > 1;
-
-if (!shouldSplit) {
-  // Standard execution — no splitting needed
-  return;
-}
-
-console.log(`SECTION SPLIT: ${domainTasks.length} domain tasks, ${sections.length} sections → activating`);
-```
-
----
-
-## 2. Entity-to-Section Mapping
-
-Build from `prd.architecture.sections[].resources[].entity`:
-
-```javascript
-const entityToSection = {};
-for (const section of prd.architecture.sections) {
-  for (const resource of section.resources ?? []) {
-    if (resource.entity) {
-      entityToSection[resource.entity] = section.code;
-    }
-  }
-}
-
-// Orphan entities (not in any section) go into Phase 0
-const allEntities = prd.architecture.entities?.map(e => e.name) ?? [];
-const orphans = allEntities.filter(e => !entityToSection[e]);
-```
-
----
-
-## 3. FK Dependency Resolution
-
-Build section dependency graph from `prd.architecture.entities[].relationships[]`:
-
-```javascript
-const sectionDeps = {}; // sectionCode -> Set<sectionCode>
-for (const entity of prd.architecture.entities ?? []) {
-  const entitySection = entityToSection[entity.name];
-  if (!entitySection) continue;
-
-  for (const rel of entity.relationships ?? []) {
-    const targetSection = entityToSection[rel.target];
-    // If target is in a DIFFERENT section, this section depends on that one
-    if (targetSection && targetSection !== entitySection) {
-      sectionDeps[entitySection] = sectionDeps[entitySection] || new Set();
-      sectionDeps[entitySection].add(targetSection);
-    }
-  }
-}
-```
-
----
-
-## 4. Topological Sort of Sections
-
-Order sections by FK dependencies (sections with no deps first):
-
-```javascript
-function topoSort(sections, sectionDeps) {
-  const sorted = [];
-  const visited = new Set();
-  const visiting = new Set();
-
-  function visit(code) {
-    if (visited.has(code)) return;
-    if (visiting.has(code)) {
-      // Circular dependency — break cycle, include as-is
-      console.warn(`Circular section dependency detected involving: ${code}`);
-      return;
-    }
-    visiting.add(code);
-    for (const dep of sectionDeps[code] || []) {
-      visit(dep);
-    }
-    visiting.delete(code);
-    visited.add(code);
-    sorted.push(code);
-  }
-
-  for (const section of sections) {
-    visit(section.code);
-  }
-  return sorted;
-}
-
-const orderedSections = topoSort(prd.architecture.sections, sectionDeps);
-```
-
----
-
-## 5. Phase Construction
-
-### 5a. Phase 0 — Foundation PRD
-
-Copy the master PRD and keep ONLY foundation tasks:
-
-```javascript
-const phase0Prd = deepClone(prd);
-phase0Prd.tasks = prd.tasks.filter(t =>
-  t.category === 'domain' ||
-  t.category === 'infrastructure' ||
-  // Module-level seed data (NavigationApplicationSeedData, NavigationModuleSeedData)
-  (t.category === 'seedData' && !t.section)
-);
-
-// Remove section-specific content from architecture (keep all entities for EF)
-// Keep ALL entities, ALL relationships — Phase 0 needs the full domain model
-
-phase0Prd._phaseInfo = { phase: 0, type: 'foundation' };
-writeJSON(`.ralph/prd-${moduleCode}-phase0.json`, phase0Prd);
-```
-
-**Phase 0 includes:**
-- ALL `domain` tasks (entity classes, enums, value objects)
-- ALL `infrastructure` tasks (EF configs, DbContext, migration)
-- Module-level `seedData` tasks (NavigationApplication, NavigationModule seed data)
-
-**Phase 0 excludes:**
-- `application` tasks (services, DTOs, validators)
-- `api` tasks (controllers)
-- `frontend` tasks (pages, routes, i18n)
-- `test` tasks
-- Section-level `seedData` tasks (NavigationSection, permissions per section)
-
-### 5b. Section Phase PRDs (Phase 1..N)
-
-For each section in topological order:
-
-```javascript
-for (let i = 0; i < orderedSections.length; i++) {
-  const sectionCode = orderedSections[i];
-  const phaseNum = i + 1;
-
-  const sectionPrd = deepClone(prd);
-
-  // Filter tasks by section
-  sectionPrd.tasks = prd.tasks.filter(t =>
-    t.section === sectionCode && (
-      t.category === 'application' ||
-      t.category === 'api' ||
-      t.category === 'frontend' ||
-      t.category === 'i18n' ||
-      t.category === 'test' ||
-      t.category === 'validation' ||
-      // Section-level seed data
-      (t.category === 'seedData' && t.section === sectionCode)
-    )
-  );
-
-  // Filter architecture to section-relevant entities only
-  const sectionEntities = new Set();
-  const section = prd.architecture.sections.find(s => s.code === sectionCode);
-  for (const resource of section?.resources ?? []) {
-    if (resource.entity) sectionEntities.add(resource.entity);
-  }
-  sectionPrd.architecture.entities = prd.architecture.entities?.filter(
-    e => sectionEntities.has(e.name)
-  ) ?? [];
-  sectionPrd.architecture.sections = [section];
-
-  // Filter seedData to section-relevant parts
-  if (sectionPrd.seedData?.coreSeedData) {
-    const csd = sectionPrd.seedData.coreSeedData;
-    if (csd.navigationSections) {
-      csd.navigationSections = csd.navigationSections.filter(
-        s => s.code === sectionCode
-      );
-    }
-    if (csd.navigationResources) {
-      csd.navigationResources = csd.navigationResources.filter(
-        r => sectionEntities.has(r.entity)
-      );
-    }
-  }
-
-  // Resolve phase dependencies
-  const deps = [0]; // All sections depend on Phase 0
-  for (const depSection of sectionDeps[sectionCode] || []) {
-    const depPhaseNum = orderedSections.indexOf(depSection) + 1;
-    if (depPhaseNum > 0) deps.push(depPhaseNum);
-  }
-
-  sectionPrd._phaseInfo = { phase: phaseNum, type: 'section', sectionCode };
-  writeJSON(`.ralph/prd-${moduleCode}-section-${sectionCode}.json`, sectionPrd);
-}
-```
-
-### 5c. Orphan Entities
-
-Entities not mapped to any section are included in Phase 0 with their services/controllers:
-
-```javascript
-if (orphans.length > 0) {
-  // Add orphan application/api/frontend tasks to Phase 0 PRD
-  const orphanTasks = prd.tasks.filter(t =>
-    !t.section && (
-      t.category === 'application' ||
-      t.category === 'api' ||
-      t.category === 'frontend' ||
-      t.category === 'test'
-    )
-  );
-  phase0Prd.tasks.push(...orphanTasks);
-  writeJSON(`.ralph/prd-${moduleCode}-phase0.json`, phase0Prd);
-}
-```
-
----
-
-## 6. Build _sectionSplit State
-
-```javascript
-const phases = [
-  {
-    phase: 0,
-    type: 'foundation',
-    sectionCode: undefined,
-    entities: allEntities,
-    prdFile: `.ralph/prd-${moduleCode}-phase0.json`,
-    status: 'pending',
-    dependsOn: []
-  }
-];
-
-for (let i = 0; i < orderedSections.length; i++) {
-  const sectionCode = orderedSections[i];
-  const sectionEntities = [];
-  const section = prd.architecture.sections.find(s => s.code === sectionCode);
-  for (const resource of section?.resources ?? []) {
-    if (resource.entity) sectionEntities.push(resource.entity);
-  }
-
-  const deps = [0]; // Phase 0
-  for (const depSection of sectionDeps[sectionCode] || []) {
-    const depPhaseNum = orderedSections.indexOf(depSection) + 1;
-    if (depPhaseNum > 0) deps.push(depPhaseNum);
-  }
-
-  phases.push({
-    phase: i + 1,
-    type: 'section',
-    sectionCode,
-    entities: sectionEntities,
-    prdFile: `.ralph/prd-${moduleCode}-section-${sectionCode}.json`,
-    status: 'pending',
-    dependsOn: deps
-  });
-}
-
-prd._sectionSplit = {
-  enabled: true,
-  phases,
-  currentPhase: 0,
-  entityToSection
-};
-
-writeJSON(currentPrdPath, prd);
-```
-
----
-
-## 7. Execution (in compact-loop.md)
-
-```javascript
-// Find next pending phase with dependencies met
-const nextPhase = prd._sectionSplit.phases.find(p => p.status === 'pending');
-if (!nextPhase) {
-  // All phases done — fall through to standard completion check
-  return;
-}
-
-// Check phase dependencies
-const depsOk = nextPhase.dependsOn.every(depIdx =>
-  prd._sectionSplit.phases[depIdx].status === 'completed'
-);
-if (!depsOk) {
-  // Phase blocked — this should not happen with topological sort
-  console.warn(`Phase ${nextPhase.phase} blocked by incomplete dependencies`);
-  return;
-}
-
-// INVOKE /apex -d {nextPhase.prdFile}
-// apex sees a normal PRD (smaller scope) and executes normally
-
-// After apex returns:
-nextPhase.status = 'completed';
-prd._sectionSplit.currentPhase = nextPhase.phase;
-writeJSON(currentPrdPath, prd);
-```
-
----
-
-## 8. Result Merging
-
-After each phase apex call, merge task statuses back into the master PRD:
-
-```javascript
-const phasePrd = readJSON(nextPhase.prdFile);
-
-// Map phase task statuses back to master PRD
-for (const phaseTask of phasePrd.tasks) {
-  const masterTask = prd.tasks.find(t => t.id === phaseTask.id);
-  if (masterTask) {
-    masterTask.status = phaseTask.status;
-    masterTask.completed_at = phaseTask.completed_at;
-    masterTask.commit_hash = phaseTask.commit_hash;
-    masterTask.files_changed = phaseTask.files_changed;
-    masterTask.error = phaseTask.error;
-    masterTask.validation = phaseTask.validation;
-  }
-}
-
-writeJSON(currentPrdPath, prd);
-```
-
----
-
-## 9. Phase Failure Handling
-
-If a phase fails (apex returns with failed tasks):
-
-```javascript
-if (phasePrd.tasks.some(t => t.status === 'failed')) {
-  nextPhase.status = 'failed';
-
-  // Check retries
-  const retryCount = nextPhase._retryCount || 0;
-  if (retryCount < 2) {
-    nextPhase.status = 'pending';
-    nextPhase._retryCount = retryCount + 1;
-    console.log(`Phase ${nextPhase.phase} failed — retry ${retryCount + 1}/2`);
-
-    // Reset failed tasks in phase PRD
-    for (const task of phasePrd.tasks) {
-      if (task.status === 'failed') {
-        task.status = 'pending';
-        task.error = null;
-      }
-    }
-    writeJSON(nextPhase.prdFile, phasePrd);
-  } else {
-    console.error(`Phase ${nextPhase.phase} failed after 2 retries — marking failed`);
-  }
-}
-```
-
----
-
-## 10. Cleanup (in step-05-report.md)
-
-After all phases complete or on final report:
-
-```javascript
-if (prd._sectionSplit?.enabled) {
-  // Delete temporary phase PRD files
-  for (const phase of prd._sectionSplit.phases) {
-    if (fileExists(phase.prdFile) && phase.prdFile !== currentPrdPath) {
-      deleteFile(phase.prdFile);
-    }
-  }
-  // Remove split state from master PRD
-  delete prd._sectionSplit;
-  writeJSON(currentPrdPath, prd);
-}
-```
-
----
-
-## Summary Table
-
-| Phase | Content | Depends On | Entities |
-|-------|---------|------------|----------|
-| Phase 0 | ALL domain + infrastructure + migration + module seed data | — | ALL |
-| Phase 1 | Section A: services, controllers, seed, pages, tests | Phase 0 | Section A entities |
-| Phase 2 | Section B: idem | Phase 0 (+ Phase 1 if FK) | Section B entities |
-| Phase N | Section N: idem | Phase 0 (+ earlier phases if FK) | Section N entities |
-| Final | Cross-validation (dotnet build, typecheck, MCP) | All phases | — |
-
-**Threshold:** `> 4 domain tasks` AND `> 1 section`
-**Backward compatible:** `<= 4 domain tasks` OR `1 section` → no splitting, zero impact

package/templates/skills/business-analyse-develop/references/task-transform-legacy.md

@@ -1,256 +0,0 @@
-# Task Transform: Legacy PrdJson → Ralph v2 (DEPRECATED)
-
-> Reference for step-01-task.md — fallback transformation for old FORMAT A PRDs (pre-v3.9.0).
-> New PRDs generated by `ss business-analyse-handoff` use unified v3 format (`$version: "3.0.0"`)
-> with pre-computed tasks. The v3 fast path in sections 0 and 1 bypasses this entirely.
-> This function will be removed in a future release.
-
-## FORMAT A Input Structure
-
-```
-{ version, source, project, requirements, businessRules, architecture, implementation, seedData }
-```
-
-## transformPrdJsonToRalphV2(prdJson, moduleCode)
-
-```javascript
-function transformPrdJsonToRalphV2(prdJson, moduleCode) {
-  const tasks = [];
-  let taskId = 1;
-
-  const layerOrder = [
-    { key: "domain", category: "domain" },
-    { key: "infrastructure", category: "infrastructure" },
-    { key: "application", category: "application" },
-    { key: "api", category: "api" },
-    { key: "frontend", category: "frontend" },
-    { key: "seedData", category: "infrastructure" },
-    { key: "tests", category: "test" }
-  ];
-
-  const filesToCreate = prdJson.implementation?.filesToCreate || prdJson.filesToCreate || {};
-  const lastIdByCategory = {};
-  const frontendFiles = [];
-  const i18nFiles = [];
-
-  // --- Generate tasks from filesToCreate ---
-  for (const layer of layerOrder) {
-    const files = filesToCreate[layer.key] || [];
-    if (layer.category === "frontend") { frontendFiles.push(...files); continue; }
-
-    for (const fileSpec of files) {
-      if (fileSpec.path?.includes('i18n/') || fileSpec.path?.includes('/locales/')) {
-        i18nFiles.push(fileSpec); continue;
-      }
-
-      const depIds = [];
-      if (lastIdByCategory[layer.category]) depIds.push(lastIdByCategory[layer.category]);
-      if (layer.category === "api" && lastIdByCategory["application"]) depIds.push(lastIdByCategory["application"]);
-      if (layer.category === "test" && lastIdByCategory["api"]) depIds.push(lastIdByCategory["api"]);
-
-      const allFRs = prdJson.requirements?.functionalRequirements || prdJson.functionalRequirements || [];
-      const linkedFRs = (fileSpec.linkedFRs || []).map(frId => allFRs.find(f => f.id === frId)?.statement || frId);
-      let criteria = linkedFRs.length > 0 ? `Implements: ${linkedFRs.join("; ")}` : `File ${fileSpec.path} created and compiles`;
-
-      const permMatrix = prdJson.architecture?.permissionMatrix || prdJson.permissionMatrix;
-      if (layer.category === "api" && permMatrix?.permissions?.length > 0) {
-        criteria += "; MANDATORY: [RequirePermission] on every endpoint (NOT [Authorize])";
-      }
-
-      tasks.push({
-        id: taskId, description: fileSpec.description ? `[${layer.category}] ${fileSpec.description} → ${fileSpec.path}` : `[${layer.category}] Create ${fileSpec.type}: ${fileSpec.path}`,
-        status: "pending", category: layer.category, dependencies: [...new Set(depIds)],
-        acceptance_criteria: criteria, started_at: null, completed_at: null, iteration: null,
-        commit_hash: null, files_changed: { created: [fileSpec.path], modified: [] },
-        validation: null, error: null, module: moduleCode
-      });
-      lastIdByCategory[layer.category] = taskId;
-      taskId++;
-    }
-  }
-
-  // --- 1b. Consolidated frontend task ---
-  if (frontendFiles.length > 0) {
-    const apiDepId = lastIdByCategory["api"] || lastIdByCategory["application"];
-    const app = prdJson.project?.application || moduleCode;
-    tasks.push({
-      id: taskId,
-      description: `[frontend] Generate COMPLETE frontend for module ${moduleCode} via MCP (${frontendFiles.length} files)`,
-      status: "pending", category: "frontend", dependencies: apiDepId ? [apiDepId] : [],
-      acceptance_criteria: `Pages in src/pages/${app}/${moduleCode}/; MCP scaffold_api_client + scaffold_routes; DataTable for lists; CSS variables ONLY; 4-language i18n; npm run typecheck passes`,
-      started_at: null, completed_at: null, iteration: null, commit_hash: null,
-      files_changed: { created: frontendFiles.map(f => f.path), modified: [] },
-      validation: null, error: null, module: moduleCode,
-      _frontendMeta: { wireframes: [...new Set(frontendFiles.flatMap(f => f.linkedWireframes || []))], filesToCreate: frontendFiles }
-    });
-    lastIdByCategory["frontend"] = taskId; taskId++;
-  }
-
-  // --- 1c. Consolidated i18n task ---
-  if (i18nFiles.length > 0) {
-    tasks.push({
-      id: taskId, description: `[i18n] Generate i18n for module ${moduleCode} (4 languages)`,
-      status: "pending", category: "i18n", dependencies: lastIdByCategory["frontend"] ? [lastIdByCategory["frontend"]] : [],
-      acceptance_criteria: "4 JSON files (fr, en, it, de) with identical keys",
-      started_at: null, completed_at: null, iteration: null, commit_hash: null,
-      files_changed: { created: i18nFiles.map(f => f.path), modified: [] },
-      validation: null, error: null, module: moduleCode
-    });
-    lastIdByCategory["i18n"] = taskId; taskId++;
-  }
-
-  // --- GUARDRAILS: Inject missing layers ---
-  const entities = prdJson.architecture?.entities || prdJson.entities || [];
-  const apiEndpoints = prdJson.architecture?.apiEndpoints || prdJson.apiEndpoints || [];
-  const sections = prdJson.architecture?.sections || prdJson.sections || [];
-  const wireframes = prdJson.specification?.uiWireframes || prdJson.uiWireframes || [];
-  const dashboards = prdJson.specification?.dashboards || prdJson.dashboards || [];
-
-  // 1d. INJECT missing infrastructure
-  if (!tasks.some(t => t.category === 'infrastructure') && entities.length > 0) {
-    tasks.push({ id: taskId, description: `[infrastructure] Create EF Core configs and seed data for ${moduleCode} (${entities.length} entities)`,
-      status: "pending", category: "infrastructure", dependencies: lastIdByCategory["domain"] ? [lastIdByCategory["domain"]] : [],
-      acceptance_criteria: `EF configs for ${entities.length} entities; DbSet in ExtensionsDbContext; DI registration`,
-      started_at: null, completed_at: null, iteration: null, commit_hash: null,
-      files_changed: { created: [], modified: [] }, validation: null, error: null, module: moduleCode });
-    lastIdByCategory["infrastructure"] = taskId; taskId++;
-  }
-
-  // 1e. INJECT missing API
-  if (!tasks.some(t => t.category === 'api') && apiEndpoints.length > 0) {
-    const permMatrix = prdJson.architecture?.permissionMatrix || prdJson.permissionMatrix;
-    tasks.push({ id: taskId, description: `[api] Create controllers for ${moduleCode} (${apiEndpoints.length} endpoints)`,
-      status: "pending", category: "api", dependencies: [lastIdByCategory["application"] || lastIdByCategory["infrastructure"]].filter(Boolean),
-      acceptance_criteria: `${apiEndpoints.length} endpoints; ${permMatrix?.permissions?.length > 0 ? "[RequirePermission] on every endpoint" : "Authorization"}; Swagger`,
-      started_at: null, completed_at: null, iteration: null, commit_hash: null,
-      files_changed: { created: [], modified: [] }, validation: null, error: null, module: moduleCode });
-    lastIdByCategory["api"] = taskId; taskId++;
-  }
-
-  // 1f. INJECT missing frontend
-  if (frontendFiles.length === 0 && (sections.length > 0 || wireframes.length > 0 || apiEndpoints.length > 0)) {
-    const app = prdJson.project?.application || moduleCode;
-    const derivedFiles = wireframes.map(wf => ({ path: `web/src/pages/${app}/${moduleCode}/${wf.screen || wf.id}.tsx`, type: 'Page' }));
-    if (apiEndpoints.length > 0) derivedFiles.push({ path: `web/src/services/api/${moduleCode}Api.ts`, type: 'ApiService' });
-
-    tasks.push({ id: taskId,
-      description: `[frontend] Generate COMPLETE frontend for ${moduleCode} via MCP (${derivedFiles.length} files)`,
-      status: "pending", category: "frontend", dependencies: [lastIdByCategory["api"] || lastIdByCategory["application"]].filter(Boolean),
-      acceptance_criteria: `Pages in src/pages/${app}/${moduleCode}/; DataTable for lists; CSS variables ONLY; 4-language i18n; npm run typecheck passes`,
-      started_at: null, completed_at: null, iteration: null, commit_hash: null,
-      files_changed: { created: derivedFiles.map(f => f.path), modified: [] },
-      validation: null, error: null, module: moduleCode,
-      _frontendMeta: { wireframes: wireframes.map(w => w.screen || w.id), filesToCreate: derivedFiles, source: "guardrail-derived" }
-    });
-    lastIdByCategory["frontend"] = taskId; taskId++;
-
-    // Also inject i18n
-    if (i18nFiles.length === 0) {
-      tasks.push({ id: taskId, description: `[i18n] Generate i18n for ${moduleCode} (4 languages)`,
-        status: "pending", category: "i18n", dependencies: [lastIdByCategory["frontend"]],
-        acceptance_criteria: "4 JSON files (fr, en, it, de) with identical keys",
-        started_at: null, completed_at: null, iteration: null, commit_hash: null,
-        files_changed: { created: [], modified: [] }, validation: null, error: null, module: moduleCode });
-      lastIdByCategory["i18n"] = taskId; taskId++;
-    }
-  }
-
-  // 1g. INJECT missing tests
-  if (!tasks.some(t => t.category === 'test') && entities.length > 0) {
-    const migDepId = lastIdByCategory["migration"] || lastIdByCategory["api"] || lastIdByCategory["application"];
-    tasks.push({ id: taskId, description: `[test] Create unit and integration tests for ${moduleCode}`,
-      status: "pending", category: "test", dependencies: migDepId ? [migDepId] : [],
-      acceptance_criteria: "Domain + service + controller tests; dotnet test passes; coverage >= 80%",
-      started_at: null, completed_at: null, iteration: null, commit_hash: null,
-      files_changed: { created: [], modified: [] }, validation: null, error: null, module: moduleCode });
-    lastIdByCategory["test"] = taskId; taskId++;
-  }
-
-  // 1h. GUARDRAIL: Core seed data (navigation, permissions, roles)
-  const coreSeedData = prdJson.seedData?.core || prdJson.seedDataCore || prdJson.specification?.seedDataCore;
-  const hasSeedDataInTasks = tasks.some(t => t.description?.includes("SeedData") || t.description?.includes("seed data"));
-  if (coreSeedData && !hasSeedDataInTasks) {
-    const infraDepId = lastIdByCategory["infrastructure"] || lastIdByCategory["domain"];
-    const navModules = coreSeedData.navigationModules || coreSeedData.navigation || [];
-    const permissions = coreSeedData.permissions || [];
-    const rolePerms = coreSeedData.rolePermissions || [];
-    tasks.push({
-      id: taskId, description: `[infrastructure] Create core seed data for ${moduleCode}: NavigationApplicationSeedData, ApplicationRolesSeedData, NavigationModuleSeedData, PermissionsSeedData, RolesSeedData`,
-      status: "pending", category: "infrastructure", dependencies: infraDepId ? [infraDepId] : [],
-      acceptance_criteria: `NavigationApplicationSeedData (app-level, once); ApplicationRolesSeedData (4 roles); NavigationModuleSeedData (${navModules.length} nav); PermissionsSeedData (${permissions.length} perms); RolesSeedData (${rolePerms.length} roles); SeedConstants; dotnet build passes`,
-      started_at: null, completed_at: null, iteration: null, commit_hash: null,
-      files_changed: { created: [], modified: [] }, validation: null, error: null, module: moduleCode,
-      _seedDataMeta: {
-        source: "guardrail-derived", coreSeedData,
-        navRoute: prdJson.project?.navRoute || `${prdJson.project?.application || 'app'}.${moduleCode}`,
-        appCode: prdJson.project?.application, appLabels: prdJson.project?.labels || null
-      }
-    });
-    lastIdByCategory["infrastructure"] = taskId; taskId++;
-  }
-
-  // 1i. GUARDRAIL: IClientSeedDataProvider
-  const hasProvider = tasks.some(t => t.description?.includes("IClientSeedDataProvider") || t.description?.includes("SeedDataProvider"));
-  if (!hasProvider && (filesToCreate["seedData"]?.length > 0 || coreSeedData)) {
-    tasks.push({
-      id: taskId, description: `[infrastructure] Create IClientSeedDataProvider for core seed data injection`,
-      status: "pending", category: "infrastructure",
-      dependencies: [lastIdByCategory["infrastructure"] || lastIdByCategory["domain"]].filter(Boolean),
-      acceptance_criteria: "SeedNavigationAsync + SeedRolesAsync + SeedPermissionsAsync + SeedRolePermissionsAsync; DI registered; idempotent",
-      started_at: null, completed_at: null, iteration: null, commit_hash: null,
-      files_changed: { created: [], modified: [] }, validation: null, error: null, module: moduleCode,
-      _providerMeta: {
-        navRoute: prdJson.project?.navRoute || `${prdJson.project?.application || 'app'}.${moduleCode}`,
-        appCode: prdJson.project?.application
-      }
-    });
-    lastIdByCategory["infrastructure"] = taskId; taskId++;
-  }
-
-  // 1j. GUARDRAIL: EF Core migration (CRITICAL — without this, no DB schema, no tests)
-  const hasEntities = entities.length > 0 || tasks.some(t => t.category === 'domain');
-  const hasMigration = tasks.some(t => t.description?.toLowerCase().includes('migration'));
-  if (hasEntities && !hasMigration) {
-    tasks.push({
-      id: taskId, description: `[infrastructure] Create EF Core migration for ${moduleCode}`,
-      status: "pending", category: "infrastructure",
-      dependencies: [lastIdByCategory["infrastructure"]].filter(Boolean),
-      acceptance_criteria: "MCP suggest_migration → name; dotnet ef migrations add; dotnet ef database update; dotnet build passes",
-      started_at: null, completed_at: null, iteration: null, commit_hash: null,
-      files_changed: { created: [], modified: [] }, validation: null, error: null, module: moduleCode,
-      _migrationMeta: { entities: entities.map(e => e.name || e.code), moduleCode }
-    });
-    lastIdByCategory["migration"] = taskId; taskId++;
-  }
-
-  // Final validation task
-  tasks.push({
-    id: taskId, description: `[validation] Build, test, and MCP validate module ${moduleCode}`,
-    status: "pending", category: "validation", dependencies: Object.values(lastIdByCategory),
-    acceptance_criteria: "dotnet build succeeds; dotnet test passes; MCP validate_conventions clean",
-    started_at: null, completed_at: null, iteration: null, commit_hash: null,
-    files_changed: { created: [], modified: [] }, validation: null, error: null, module: moduleCode
-  });
-
-  return {
-    $version: "2.0.0", feature: `${prdJson.project?.module || moduleCode} (${prdJson.project?.application || "app"})`,
-    status: "in_progress", created: new Date().toISOString(), updated_at: new Date().toISOString(),
-    metadata: { cli_version: "unknown", branch: getCurrentBranch(), project_path: process.cwd(), mcp_servers: { smartstack: true, context7: true }, moduleCode },
-    config: { max_iterations: {max_iterations}, completion_promise: "{completion_promise}", current_iteration: 1 },
-    source: { type: "ba-handoff", handoff_path: null, feature_json_path: prdJson.source?.featureJson || null, module: moduleCode },
-    tasks, history: []
-  };
-}
-```
-
-## Guardrails Summary
-
-| ID | Guardrail | Condition | Injection |
-|----|-----------|-----------|-----------|
-| 1d | Missing infrastructure | entities exist, no infra tasks | EF Core configs + seed data |
-| 1e | Missing API | apiEndpoints exist, no api tasks | Controllers |
-| 1f | Missing frontend | sections/wireframes exist, no frontend tasks | Pages + API service |
-| 1g | Missing tests | entities exist, no test tasks | Unit + integration tests |
-| 1h | Core seed data | coreSeedData exists, no seed tasks | Navigation + Permissions + Roles |
-| 1i | IClientSeedDataProvider | seedData files or coreSeedData, no provider task | Provider class |
-| 1j | EF Core migration | entities exist, no migration task | Migration (CRITICAL) |