@atlashub/smartstack-cli 4.5.0 → 4.7.0

package/templates/skills/cli-app-sync/SKILL.md

@@ -0,0 +1,272 @@
+---
+name: cli-app-sync
+description: Detect and fix template drift between SmartStack.app (source of truth) and CLI templates
+argument-hint: "[report|fix|check <file>]"
+model: sonnet
+allowed-tools: Read, Grep, Glob, Bash, Edit, Write
+---
+
+## Current state (auto-injected)
+- CLI version: !`node -p "require('./package.json').version" 2>/dev/null || echo "unknown"`
+- CLI branch: !`git rev-parse --abbrev-ref HEAD 2>/dev/null || echo "unknown"`
+- App branch: !`git -C "D:/01 - projets/SmartStack.app/02-Develop" rev-parse --abbrev-ref HEAD 2>/dev/null || echo "unknown"`
+- App last commit: !`git -C "D:/01 - projets/SmartStack.app/02-Develop" log --oneline -1 2>/dev/null || echo "unknown"`
+
+<objective>
+Detect drift between SmartStack.app (the platform / source of truth) and the CLI project templates used by `ss init`.
+
+SmartStack.app defines the reference architecture. The CLI templates generate **client projects** that consume the platform via NuGet/npm. Some differences are **intentional by design** (see exclusions below). This skill identifies **unintentional drift** that should be corrected.
+
+See [references/comparison-map.md](references/comparison-map.md) for the full file mapping and exclusion rules.
+</objective>
+
+<quick_start>
+```bash
+/cli-app-sync                # Full drift report (read-only)
+/cli-app-sync report         # Same as above
+/cli-app-sync fix            # Detect drift + apply fixes interactively
+/cli-app-sync check Program  # Check a single file mapping
+```
+</quick_start>
+
+<subcommands>
+
+| Command | Description |
+|---------|-------------|
+| `/cli-app-sync` | Full drift report across all comparison points |
+| `/cli-app-sync report` | Same as above (explicit) |
+| `/cli-app-sync fix` | Detect drift and propose fixes interactively |
+| `/cli-app-sync check <name>` | Check a single comparison point by name |
+
+</subcommands>
+
+<paths>
+
+**Source of truth (SmartStack.app):**
+```
+APP_ROOT = D:/01 - projets/SmartStack.app/02-Develop
+APP_API  = $APP_ROOT/src/SmartStack.Api
+APP_WEB  = $APP_ROOT/web/smartstack-web
+```
+
+**CLI templates (to validate):**
+```
+CLI_ROOT      = D:/01 - projets/SmartStack.cli/02-Develop
+CLI_TEMPLATES = $CLI_ROOT/templates/project
+CLI_INIT      = $CLI_ROOT/src/commands/init.ts
+```
+
+</paths>
+
+<workflow>
+
+## Step 1: Resolve paths and verify access
+
+```bash
+APP_ROOT="D:/01 - projets/SmartStack.app/02-Develop"
+CLI_ROOT="D:/01 - projets/SmartStack.cli/02-Develop"
+
+# Verify both projects are accessible
+[ -d "$APP_ROOT/src/SmartStack.Api" ] || { echo "ERROR: SmartStack.app not found at $APP_ROOT"; exit 1; }
+[ -d "$CLI_ROOT/templates/project" ] || { echo "ERROR: CLI templates not found at $CLI_ROOT"; exit 1; }
+```
+
+## Step 2: Run comparison for each mapping
+
+For each entry in the [comparison map](references/comparison-map.md):
+
+1. **Read the reference file** from SmartStack.app
+2. **Read the template file** from CLI (either `.template` file or inline content in `init.ts`)
+3. **Compare** the relevant patterns/sections (not byte-for-byte — templates have `{{ProjectName}}` placeholders)
+4. **Classify** the result: `OK`, `DRIFT`, or `INTENTIONAL`
+
+### Comparison strategy by type
+
+**Template files** (`templates/project/*.template`):
+- Read both files
+- Replace `{{ProjectName}}` with `SmartStack` in the template for comparison
+- Compare structure, imports, method calls, middleware order
+
+**Inline templates in `init.ts`**:
+- Extract the template string from `init.ts` (look for the variable assignment)
+- Compare key properties/patterns against the reference file
+- Focus on: dependency versions, compiler options, proxy config, CSS directives
+
+## Step 3: Generate report
+
+```
+================================================================================
+              CLI ↔ APP TEMPLATE SYNC REPORT
+================================================================================
+
+SmartStack.app: {branch} @ {commit}
+SmartStack.cli: v{version} @ {branch}
+Date: {date}
+
+COMPARISON POINTS: {total}
+--------------------------------------------------------------------------------
+
+  [OK]    Program.cs — Serilog, Kestrel, Swagger, AppInsights present
+  [DRIFT] package.json — @types/node ^24.0.0 vs ^26.0.0 in app
+  [DRIFT] vite.config.ts — bypass() present but missing ws reconnect
+  [OK]    tsconfig.json — ES2022, verbatimModuleSyntax, erasableSyntaxOnly
+  [OK]    index.css — @custom-variant dark present
+  [OK]    DependencyInjection.Application — MediatR warning present
+  [OK]    DependencyInjection.Infrastructure — SQL Server pattern matches
+  [SKIP]  App.tsx — Intentionally different (mergeRoutes pattern)
+  [SKIP]  ExtensionsDbContext — Intentionally simplified
+
+--------------------------------------------------------------------------------
+SUMMARY: {ok} OK | {drift} DRIFT | {skip} SKIPPED
+================================================================================
+```
+
+## Step 4: Handle drifts (fix mode)
+
+If running `/cli-app-sync fix` and drifts are detected:
+
+```yaml
+AskUserQuestion:
+  header: "Fix drifts"
+  question: "Drifts detected. How do you want to proceed?"
+  options:
+    - label: "Fix all (Recommended)"
+      description: "Apply all fixes automatically"
+    - label: "Fix interactively"
+      description: "Review and approve each fix individually"
+    - label: "Report only"
+      description: "Show details without fixing"
+  multiSelect: false
+```
+
+For each drift in fix mode:
+1. Show the diff (what changed in app vs what the template has)
+2. Propose the fix
+3. Apply using Edit tool (for `.template` files) or inline edit (for `init.ts`)
+
+## Step 5: Post-fix verification
+
+After applying fixes:
+```bash
+cd "$CLI_ROOT" && npm run build 2>&1
+```
+
+If build succeeds, display summary of changes.
+
+</workflow>
+
+<intentional_differences>
+
+These differences are **by design** and should always be marked `[SKIP]`:
+
+| File/Pattern | Reason |
+|---|---|
+| `App.tsx` — `mergeRoutes` + `SmartStackProvider` | Client projects use extension routing pattern, not platform routing |
+| `ExtensionsDbContext` | Intentionally simplified vs `CoreDbContext` — different architectural role |
+| `api.ts` | Re-exports SmartStack client — wrapper pattern is correct |
+| `DependencyInjection.Infrastructure.cs` | SQL Server connection pattern identical by design |
+| Skills/Agents/Hooks | Installed globally via `ss install`, not by `ss init` |
+| SmartStack.app-only skills | `bugfix-issue`, `feature-test`, `sonar`, `version-bump` are platform-specific |
+| `appsettings.json` | Client projects have a simplified config (no core service settings) |
+
+</intentional_differences>
+
+<comparison_categories>
+
+### Backend comparisons
+
+| ID | App reference | CLI template | Key patterns to check |
+|---|---|---|---|
+| `program-cs` | `src/SmartStack.Api/Program.cs` | `templates/project/Program.cs.template` | Serilog bootstrap, Kestrel HTTP/1.1, appsettings.Local.json, UseSmartStackSerilog, AppInsights, UseSmartStackSwagger, try/catch/finally |
+| `di-app` | *(no direct equivalent)* | `templates/project/DependencyInjection.Application.cs.template` | MediatR warning comment, no double-registration |
+| `di-infra` | `src/SmartStack.Infrastructure/DependencyInjection.cs` | `templates/project/DependencyInjection.Infrastructure.cs.template` | SQL Server connection pattern, AddDbContext |
+| `appsettings` | `src/SmartStack.Api/appsettings.json` | `templates/project/appsettings.json.template` | Serilog config section, ConnectionStrings pattern, Logging sinks |
+
+### Frontend comparisons
+
+| ID | App reference | CLI template (in `init.ts`) | Key patterns to check |
+|---|---|---|---|
+| `package-json` | `web/smartstack-web/package.json` | `init.ts` → `packageJson` variable | Dependency versions (`@types/node`, `typescript`, `eslint-plugin-react-refresh`), testing stack (`vitest`, `@testing-library/*`), scripts (`test`, `test:watch`, `test:coverage`) |
+| `vite-config` | `web/smartstack-web/vite.config.ts` | `init.ts` → `viteConfig` variable | Proxy bypass() function, plugin list, resolve aliases |
+| `tsconfig` | `web/smartstack-web/tsconfig.json` or `tsconfig.app.json` | `init.ts` → `tsConfig` variable | `target`, `lib`, `verbatimModuleSyntax`, `erasableSyntaxOnly`, no `isolatedModules` |
+| `index-css` | `web/smartstack-web/src/index.css` | `init.ts` → `indexCss` variable | `@import "tailwindcss"`, `@source` directive, `@custom-variant dark` |
+
+</comparison_categories>
+
+<extraction_patterns>
+
+### Extracting inline templates from `init.ts`
+
+Templates in `init.ts` are assigned as template literals or objects. Use these patterns to locate them:
+
+```
+package.json  → search for: "const packageJson = {"
+vite.config   → search for: "const viteConfig = \`"
+tsconfig      → search for: "const tsConfig = {"
+index.css     → search for: "const indexCss = \`"
+```
+
+### Comparing versions
+
+For `package.json` dependencies, compare:
+- Major version range: `^22.0.0` vs `^24.0.0` → DRIFT if major differs by > 1
+- Tilde vs caret: `~5.9.0` vs `^5.7.0` → DRIFT if strategy differs
+- Missing packages: present in app but absent in template → DRIFT
+- Extra packages in template are OK (client-specific needs)
+
+### Comparing TypeScript config
+
+Compare key compiler options:
+- `target` and `lib` → must match
+- `verbatimModuleSyntax` → must be present (replaces deprecated `isolatedModules`)
+- `erasableSyntaxOnly` → must be present
+- Other options → informational only
+
+</extraction_patterns>
+
+<output_format>
+
+### Per-comparison detail (verbose)
+
+```
+--- program-cs ---
+Reference: SmartStack.app/src/SmartStack.Api/Program.cs
+Template:  templates/project/Program.cs.template
+
+Checks:
+  [OK] Serilog bootstrap logger (Log.Logger = new LoggerConfiguration...)
+  [OK] try/catch/finally pattern
+  [OK] ConfigureKestrel HTTP/1.1
+  [OK] appsettings.Local.json loading
+  [OK] UseSmartStackSerilog()
+  [OK] Application Insights conditional
+  [OK] UseSmartStackSwagger()
+  [OK] Log.Information start/success messages
+
+Status: OK
+```
+
+### Drift detail
+
+```
+--- package-json ---
+Reference: SmartStack.app/web/smartstack-web/package.json
+Template:  init.ts → packageJson
+
+Drifts:
+  [DRIFT] @types/node: template=^24.0.0, app=^26.0.0
+  [DRIFT] Missing devDependency: @testing-library/dom
+
+Status: DRIFT (2 issues)
+```
+
+</output_format>
+
+<success_criteria>
+- All comparison points from the map are checked
+- Intentional differences are correctly identified as SKIP
+- Drift detection catches: missing patterns, outdated versions, removed options
+- Fix mode correctly updates template files and init.ts inline templates
+- Build passes after fixes are applied
+- Report is clear, actionable, and easy to scan
+</success_criteria>

package/templates/skills/cli-app-sync/references/comparison-map.md

@@ -0,0 +1,228 @@
+# Comparison Map: SmartStack.app ↔ CLI Templates
+
+> This file defines the **exact mapping** between SmartStack.app (source of truth)
+> and CLI templates. Used by `/cli-app-sync` to detect drift.
+
+## Paths
+
+| Alias | Path |
+|---|---|
+| `APP_API` | `D:/01 - projets/SmartStack.app/02-Develop/src/SmartStack.Api` |
+| `APP_WEB` | `D:/01 - projets/SmartStack.app/02-Develop/web/smartstack-web` |
+| `CLI_TPL` | `D:/01 - projets/SmartStack.cli/02-Develop/templates/project` |
+| `CLI_INIT` | `D:/01 - projets/SmartStack.cli/02-Develop/src/commands/init.ts` |
+
+---
+
+## Backend Comparisons
+
+### `program-cs` — Program.cs startup pipeline
+
+| | Path |
+|---|---|
+| **Reference** | `APP_API/Program.cs` |
+| **Template** | `CLI_TPL/Program.cs.template` |
+
+**Patterns to verify (ordered by startup sequence):**
+
+| # | Pattern | Grep/search | Critical |
+|---|---|---|---|
+| 1 | Serilog usings | `using Serilog;` + `using Serilog.Events;` | YES |
+| 2 | Bootstrap logger | `Log.Logger = new LoggerConfiguration()` | YES |
+| 3 | try/catch/finally | `try {` ... `catch (Exception` ... `finally` | YES |
+| 4 | Kestrel HTTP/1.1 | `ConfigureKestrel` + `HttpProtocols.Http1` | YES |
+| 5 | appsettings.Local.json | `AddJsonFile("appsettings.Local.json"` | YES |
+| 6 | Serilog from config | `UseSmartStackSerilog()` | YES |
+| 7 | Application Insights | `AddApplicationInsightsTelemetry` | YES |
+| 8 | AddSmartStack | `AddSmartStack(builder.Configuration` | YES |
+| 9 | InitializeSmartStackAsync | `InitializeSmartStackAsync()` | YES |
+| 10 | Swagger | `UseSmartStackSwagger()` | YES |
+| 11 | Middleware pipeline | `UseSmartStack()` then `MapSmartStack()` | YES |
+| 12 | Success log | `Log.Information(` ... `started successfully` | NO |
+| 13 | Fatal log | `Log.Fatal(ex,` | YES |
+| 14 | CloseAndFlush | `Log.CloseAndFlush()` | YES |
+
+**Template-specific additions (not in app):**
+- `using Microsoft.EntityFrameworkCore;` (for ExtensionsDbContext migration)
+- `Add{{ProjectName}}Infrastructure` / `Add{{ProjectName}}Application` (client DI)
+- ExtensionsDbContext migration block (after InitializeSmartStackAsync)
+
+---
+
+### `di-app` — DependencyInjection.Application.cs
+
+| | Path |
+|---|---|
+| **Reference** | *(no app equivalent — client-only pattern)* |
+| **Template** | `CLI_TPL/DependencyInjection.Application.cs.template` |
+
+**Patterns to verify:**
+
+| # | Pattern | Search | Critical |
+|---|---|---|---|
+| 1 | MediatR warning | `MediatR is already registered` or `do NOT register it again` | YES |
+| 2 | No AddMediatR example | must NOT contain `services.AddMediatR(cfg =>` | YES |
+
+---
+
+### `di-infra` — DependencyInjection.Infrastructure.cs
+
+| | Path |
+|---|---|
+| **Reference** | `APP_API/../SmartStack.Infrastructure/DependencyInjection.cs` |
+| **Template** | `CLI_TPL/DependencyInjection.Infrastructure.cs.template` |
+
+**Patterns to verify:**
+
+| # | Pattern | Search | Critical |
+|---|---|---|---|
+| 1 | AddDbContext pattern | `AddDbContext<ExtensionsDbContext>` | YES |
+| 2 | SQL Server | `UseSqlServer` | YES |
+| 3 | Connection string | `GetConnectionString` | YES |
+
+---
+
+### `appsettings` — appsettings.json
+
+| | Path |
+|---|---|
+| **Reference** | `APP_API/appsettings.json` |
+| **Template** | `CLI_TPL/appsettings.json.template` |
+
+**Patterns to verify:**
+
+| # | Pattern | Search | Critical |
+|---|---|---|---|
+| 1 | Serilog config section | `"Serilog"` | YES |
+| 2 | ConnectionStrings | `"ConnectionStrings"` | YES |
+| 3 | Logging sinks | `"Logging"` section | NO |
+
+---
+
+## Frontend Comparisons
+
+### `package-json` — package.json dependencies and scripts
+
+| | Path |
+|---|---|
+| **Reference** | `APP_WEB/package.json` |
+| **Template** | `CLI_INIT` → variable `packageJson` (search: `const packageJson = {`) |
+
+**Dependency version checks:**
+
+| Package | Category | Compare rule |
+|---|---|---|
+| `@types/node` | devDependencies | Major range must match app (e.g., `^24` vs `^26`) |
+| `typescript` | devDependencies | Range strategy and major must match (`~5.9` vs `~5.9`) |
+| `eslint-plugin-react-refresh` | devDependencies | Major range must match |
+| `react` | dependencies | Major range must match |
+| `react-dom` | dependencies | Major range must match |
+| `vite` | devDependencies | Major range must match |
+| `tailwindcss` | dependencies | Major range must match |
+
+**Testing stack (must be present in template):**
+
+| Package | Required |
+|---|---|
+| `vitest` | YES |
+| `@vitest/coverage-v8` | YES |
+| `@testing-library/react` | YES |
+| `@testing-library/jest-dom` | YES |
+| `@testing-library/user-event` | YES |
+| `jsdom` | YES |
+
+**Scripts (must be present in template):**
+
+| Script | Command |
+|---|---|
+| `test` | `vitest run` |
+| `test:watch` | `vitest` |
+| `test:coverage` | `vitest run --coverage` |
+
+---
+
+### `vite-config` — vite.config.ts proxy and plugins
+
+| | Path |
+|---|---|
+| **Reference** | `APP_WEB/vite.config.ts` |
+| **Template** | `CLI_INIT` → variable `viteConfig` (search: `` const viteConfig = ` ``) |
+
+**Patterns to verify:**
+
+| # | Pattern | Search | Critical |
+|---|---|---|---|
+| 1 | React plugin | `react()` | YES |
+| 2 | Tailwind plugin | `tailwindcss()` | YES |
+| 3 | API proxy | `'/api':` | YES |
+| 4 | Proxy bypass | `bypass(req` or `bypass:` | YES |
+| 5 | HTML accept check | `accept?.includes('text/html')` | YES |
+| 6 | SignalR proxy | `'/hubs':` | YES |
+| 7 | WebSocket flag | `ws: true` | YES |
+| 8 | Path alias | `'@': '/src'` or `'@/': './src'` | YES |
+
+---
+
+### `tsconfig` — tsconfig.json compiler options
+
+| | Path |
+|---|---|
+| **Reference** | `APP_WEB/tsconfig.json` or `APP_WEB/tsconfig.app.json` |
+| **Template** | `CLI_INIT` → variable `tsConfig` (search: `const tsConfig = {`) |
+
+**Options to verify:**
+
+| Option | Expected value | Critical |
+|---|---|---|
+| `target` | `ES2022` | YES |
+| `lib` | includes `ES2022` | YES |
+| `verbatimModuleSyntax` | `true` | YES |
+| `erasableSyntaxOnly` | `true` | YES |
+| `isolatedModules` | must NOT be present (deprecated) | YES |
+| `moduleResolution` | `bundler` | YES |
+| `jsx` | `react-jsx` | YES |
+| `strict` | `true` | YES |
+
+---
+
+### `index-css` — index.css Tailwind directives
+
+| | Path |
+|---|---|
+| **Reference** | `APP_WEB/src/index.css` |
+| **Template** | `CLI_INIT` → variable `indexCss` (search: `` const indexCss = ` ``) |
+
+**Patterns to verify:**
+
+| # | Pattern | Search | Critical |
+|---|---|---|---|
+| 1 | Tailwind import | `@import "tailwindcss"` | YES |
+| 2 | Source directive | `@source "../node_modules/@atlashub/smartstack` | YES |
+| 3 | Dark mode variant | `@custom-variant dark` | YES |
+
+---
+
+## Intentional Exclusions (SKIP)
+
+These comparisons should **always** return `[SKIP]` — differences are by design.
+
+| Item | Reason |
+|---|---|
+| `App.tsx` | Client uses `mergeRoutes` + `SmartStackProvider`; app uses direct routing |
+| `api.ts` | Client re-exports SmartStack client — wrapper pattern is correct |
+| `ExtensionsDbContext` | Intentionally simplified vs `CoreDbContext` |
+| `GlobalUsings.cs` | App has platform-wide usings; client has minimal set |
+| Skills/Agents/Hooks | Installed globally via `ss install`, not by `ss init` |
+| Platform-specific skills | `bugfix-issue`, `feature-test`, `sonar`, `version-bump` |
+| CI/CD files | `azure-pipelines.yml` is platform-specific |
+| `CLAUDE.md` | Each project has its own instructions |
+
+---
+
+## Maintenance
+
+When adding a new comparison point:
+1. Add an entry to the appropriate section above
+2. Define the reference path and template location
+3. List the key patterns to check with criticality
+4. Update the SKILL.md `<comparison_categories>` table

package/templates/skills/gitflow/SKILL.md

@@ -59,6 +59,7 @@ Execute GitFlow workflows with automatic versioning, worktree management, and EF
 | `-s` | `--status` | Status mode: show state only, no actions |
 | `-v` | `--verbose` | Verbose output |
 | `--dry-run` | | Preview actions without executing |
+| `--smartstack` | | Init + SmartStack project scaffolding (backend, frontend, config) |
 
 **⚠️ CRITICAL: Without `-a`, each command executes ONLY its own step then STOPS.**
 
@@ -125,6 +126,9 @@ Execute GitFlow workflows with automatic versioning, worktree management, and EF
 # UTILITY COMMANDS
 # ══════════════════════════════════════════════════════════════
 
+/gitflow init              # Git structure only (bare + worktrees)
+/gitflow init --smartstack # Git structure + SmartStack project init
+
 /gitflow status           # Show current state
 /gitflow -s               # Shorthand for status
 /gitflow abort            # Rollback current operation
@@ -186,6 +190,7 @@ start ──► commit* ──► sync ──► pr ──► merge ──► fi
 | Input Pattern | Step File | Task Agent (`subagent_type`) |
 |---------------|-----------|----------------------------|
 | `init` | `steps/step-init.md` | **ALWAYS INLINE** (see below) |
+| `init --smartstack` | `steps/step-init.md` | **ALWAYS INLINE** (Git + SmartStack init) |
 | `start`, `-f`, `-r`, `-h` | `steps/step-start.md` | `gitflow-start` |
 | `commit [msg]` | `steps/step-commit.md` | `gitflow-commit` |
 | `sync` | `steps/step-sync.md` | — |

package/templates/skills/gitflow/steps/step-init.md

@@ -121,6 +121,21 @@ Store paths as `D:/path/to/folder` format.
 Spawn `Task(gitflow-init-validate)` (haiku) to verify structure.
 Checks: .bare/HEAD, worktree branches, directories, config file.
 
+### 10b. SmartStack Project Initialization (conditional)
+
+**Skip if `--smartstack` flag was NOT provided.**
+
+After git structure is ready, initialize the SmartStack project in the develop worktree:
+
+1. Navigate to the develop worktree: `{DEVELOP_WORKTREE_PATH}`
+2. Run `smartstack init {PROJECT_NAME} --here`
+   - `--here` flag initializes in the current directory (no subdirectory creation)
+   - This creates: .NET solution, frontend project, .smartstack/config.json
+3. Run `smartstack install --local`
+   - Deploys skills, agents, hooks to `.claude/` in the project
+4. If init fails, display error but do NOT fail the entire gitflow init
+   - The git structure is still valid and usable
+
 ### 11. Summary
 
 ```
@@ -145,6 +160,10 @@ Checks: .bare/HEAD, worktree branches, directories, config file.
 ╠══════════════════════════════════════════════════════════════════╣
 ║  Validation: {ERRORS == 0 ? "ALL CHECKS PASSED" : "errors"}      ║
 ╠══════════════════════════════════════════════════════════════════╣
+{if SMARTSTACK_INIT}
+║  SmartStack: Initialized (backend + frontend)  ✅                ║
+╠══════════════════════════════════════════════════════════════════╣
+{endif}
 {if SIBLING_PROJECTS not empty}
 ║  Workspace: {WORKSPACE_DIR}                                       ║
 ║  GitFlow projects: {#SIBLING_PROJECTS + 1} (including this one)   ║

package/templates/skills/ralph-loop/references/task-transform-legacy.md

@@ -77,7 +77,7 @@ function transformPrdJsonToRalphV2(prdJson, moduleCode) {
       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; SmartTable for lists; CSS variables ONLY; 4-language i18n; npm run typecheck passes`,
+      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,
@@ -136,7 +136,7 @@ function transformPrdJsonToRalphV2(prdJson, moduleCode) {
     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}/; SmartTable for lists; CSS variables ONLY; 4-language i18n; npm run typecheck passes`,
+      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,