@atlashub/smartstack-cli 3.22.0 → 3.24.0

package/templates/skills/business-analyse/steps/step-05a-handoff.md

@@ -287,13 +287,24 @@ For each module, generate these SEPARATE frontend file entries (not one monolith
 
 ```json
 "frontend": [
-  { "path": "src/pages/{Mod}/{ListPage}Page.tsx", "type": "Page", "linkedWireframes": ["{module}-list"], "module": "{moduleCode}" },
-  { "path": "src/pages/{Mod}/{DetailPage}Page.tsx", "type": "Page", "linkedWireframes": ["{module}-detail"], "module": "{moduleCode}" },
+  { "path": "src/pages/{Context}/{App}/{Mod}/{ListPage}Page.tsx", "type": "Page", "linkedWireframes": ["{module}-list"], "module": "{moduleCode}", "skill": "/ui-components" },
+  { "path": "src/pages/{Context}/{App}/{Mod}/{Entity}CreatePage.tsx", "type": "Page", "linkedWireframes": ["{module}-create"], "module": "{moduleCode}", "skill": "/ui-components", "fkFields": ["EmployeeId→Employee", "DepartmentId→Department"] },
+  { "path": "src/pages/{Context}/{App}/{Mod}/{Entity}EditPage.tsx", "type": "Page", "linkedWireframes": ["{module}-edit"], "module": "{moduleCode}", "skill": "/ui-components", "fkFields": ["EmployeeId→Employee", "DepartmentId→Department"] },
+  { "path": "src/pages/{Context}/{App}/{Mod}/{DetailPage}Page.tsx", "type": "Page", "linkedWireframes": ["{module}-detail"], "module": "{moduleCode}", "skill": "/ui-components" },
   { "path": "src/hooks/use{Module}.ts", "type": "Hook", "module": "{moduleCode}" },
-  { "path": "src/i18n/{module}.json", "type": "I18n", "module": "{moduleCode}" }
+  { "path": "src/i18n/locales/fr/{moduleLower}.json", "type": "I18n", "language": "fr", "module": "{moduleCode}" },
+  { "path": "src/i18n/locales/en/{moduleLower}.json", "type": "I18n", "language": "en", "module": "{moduleCode}" },
+  { "path": "src/i18n/locales/it/{moduleLower}.json", "type": "I18n", "language": "it", "module": "{moduleCode}" },
+  { "path": "src/i18n/locales/de/{moduleLower}.json", "type": "I18n", "language": "de", "module": "{moduleCode}" }
 ]
 ```
 
+**FK fields in forms:** Create/Edit page entries MUST include `"fkFields"` array listing FK properties and their target entities (format: `"PropertyId→TargetEntity"`). This tells ralph-loop to use `EntityLookup` component instead of plain text inputs. Derive FK fields from `analysis.entities[].relationships[]` where `type = "N:1"` or from properties ending in `Id` with a navigation property. See `smartstack-frontend.md` section 6 for the EntityLookup pattern.
+
+**Page generation:** ALL pages MUST use `/ui-components` skill (indicated by `"skill"` field). NEVER write raw TSX without the skill patterns.
+
+**I18n generation:** 4 JSON files per module with identical key structures. `{moduleLower}` = lowercase module code. Keys: actions, labels, columns, form, errors, validation, messages, empty. See `smartstack-frontend.md` template.
+
 Additional per-section pages (dashboard, import, etc.) are separate entries.
 
 **Route wiring task (MANDATORY — separate entry):**
@@ -344,7 +355,7 @@ Add to progress.txt after all module tasks.
 >     "application": [{"path": "src/Application/Services/{Ctx}/{App}/{Mod}/{Svc}Service.cs", "type": "Service|Dto|Validator", "linkedFRs": [], "linkedUCs": [], "module": "..."}],
 >     "infrastructure": [{"path": "src/Infrastructure/Persistence/Configurations/{Ctx}/{App}/{Mod}/{Entity}Configuration.cs", "type": "EFConfiguration", "linkedFRs": [], "module": "..."}],
 >     "api": [{"path": "src/API/Controllers/{CtxShort}/{App}/{Entity}Controller.cs", "type": "ApiController", "linkedUCs": [], "linkedFRs": [], "module": "..."}],
->     "frontend": [{"path": "src/pages/{Mod}/{Page}Page.tsx", "type": "Page|Component|Hook|DashboardPage", "linkedUCs": [], "linkedWireframes": [], "module": "..."}],
+>     "frontend": [{"path": "src/pages/{Ctx}/{App}/{Mod}/{Page}Page.tsx", "type": "Page|Component|Hook|DashboardPage|I18n", "linkedUCs": [], "linkedWireframes": [], "module": "...", "skill": "/ui-components"}],
 >     "seedData": [{"path": "src/Infrastructure/Persistence/Seeding/Data/{Mod}/...SeedData.cs", "type": "SeedData", "category": "core|business", "source": "...", "module": "..."}],
 >     "tests": [{"path": "src/Tests/Unit/Domain/{Ctx}/{App}/{Mod}/{Entity}Tests.cs", "type": "UnitTests|IntegrationTests|SecurityTests", "linkedFRs": [], "linkedUCs": [], "module": "..."}]
 >   },

package/templates/skills/business-analyse/templates/tpl-frd.md

@@ -100,8 +100,8 @@
 
 | Element | Type | Behavior | Permission |
 |---------|------|----------|------------|
-| [+ New] | Button | Opens create modal | `{module}.create` |
-| [E] | Icon | Opens edit modal | `{module}.update` |
+| [+ New] | Button | Navigates to `/create` page | `{module}.create` |
+| [E] | Icon | Navigates to `/:id/edit` page | `{module}.update` |
 | [D] | Icon | Confirmation + delete | `{module}.delete` |
 
 ## 5. Validation and Messages

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

@@ -272,8 +272,8 @@
 
 | Element | Type | Behavior | Permission |
 |---------|------|----------|------------|
-| [+ New] | Button | Opens create modal | `{module}.create` |
-| [E] | Icon | Opens edit modal | `{module}.update` |
+| [+ New] | Button | Navigates to `/create` page | `{module}.create` |
+| [E] | Icon | Navigates to `/:id/edit` page | `{module}.update` |
 | [D] | Icon | Confirmation + delete | `{module}.delete` |
 
 ## 5. Validation and Messages

package/templates/skills/ralph-loop/references/category-rules.md

@@ -281,6 +281,8 @@ fi
 - Swagger XML documentation
 - Consistent route patterns: `api/{context}/{app}/{module}`
 - Return DTOs, never domain entities
+- **ALL GetAll endpoints MUST accept `?search=` query parameter** (enables EntityLookup on frontend)
+- **GetAll returns paginated results:** `PaginatedResult<T>` with items, totalCount, page, pageSize
 
 **Controller coverage (BLOCKING):**
 - EVERY entity in the module MUST have a controller with CRUD endpoints (GET list, GET by id, POST, PUT, DELETE)
@@ -307,18 +309,28 @@ fi
 3. **Wire routes to App.tsx (detect pattern: `contextRoutes` array OR JSX `<Route>`):**
    - **Pattern A** (`contextRoutes: ContextRouteExtensions` in App.tsx): add to `contextRoutes.{context}[]` with RELATIVE paths → auto-injected into both standard + tenant trees
    - **Pattern B** (JSX `<Route>` in App.tsx): insert `<Route>` entries inside correct Layout wrapper (BOTH standard AND tenant-prefixed `/t/:slug/...` blocks)
-4. Create pages using SmartStack components
+4. Create pages using **`/ui-components` skill** (MANDATORY for entity lists, grids, tables, dashboards, charts). The skill ensures CSS variables, EntityCard, SmartTable, loading/error/empty states, and responsive grid patterns.
 5. Create preferences hook: `use{Module}Preferences.ts`
-6. Generate i18n (4 languages: fr, en, it, de)
+6. Generate i18n (4 languages: fr, en, it, de) — see I18n section below for detailed rules
 7. `npm run typecheck` MUST pass (BLOCKING)
 
 **Components:**
 - Lists: `SmartTable` + `SmartFilter` (NOT HTML `<table>`)
 - Grids: `EntityCard` (NOT custom `<div>` cards)
 - Detail: `EntityDetailCard`, `StatusBadge`, tab layout
-- Forms: `SmartForm` with FluentValidation-backed fields
+- Forms: `SmartForm` with FluentValidation-backed fields — **MUST be full pages, NEVER modals**
+- FK fields: `EntityLookup` for searchable entity selection (NEVER plain text for Guid FK)
 - Dashboard: `StatCard`, Recharts components
 
+**Form pages (CRITICAL — ZERO modals/popups/drawers):**
+- Create form: `EntityCreatePage.tsx` with route `/{module}/create`
+- Edit form: `EntityEditPage.tsx` with route `/{module}/:id/edit`
+- ALL forms are full pages with their own URL — NEVER Modal/Dialog/Drawer/Popup
+- Back button with `navigate(-1)` on every form page
+- Use React.lazy() + `<Suspense fallback={<PageLoader />}>` for form page imports
+- **Form tests (MANDATORY):** Co-located `EntityCreatePage.test.tsx` and `EntityEditPage.test.tsx`
+  → Cover: rendering, validation, submit, pre-fill (edit), navigation, error handling
+
 **Layout wrapper mapping:**
 
 | Context | Layout | Route path |
@@ -355,17 +367,43 @@ Adding routes to clientRoutes[] instead of contextRoutes.{context}[] → MUST us
 '00000000-0000-0000-0000-000000000000'       → use dynamic ID from auth context or route params
 const api = axios.create(...)                → use @/services/api/apiClient (single instance)
 useXxx with raw axios inside hooks           → hooks MUST use the shared apiClient from @/services/api
+<input type="text" value={...employeeId}    → FK Guid fields MUST use EntityLookup (searchable select)
+placeholder="Enter ID" / "Enter GUID"       → EntityLookup provides search-based entity selection
+<Modal>/<Dialog>/<Drawer>/<Popup> for forms  → forms are FULL PAGES with own URL routes (/create, /:id/edit)
+useState(showCreateModal/editDialog)         → navigate to form page, NEVER toggle modal visibility
 ```
 
 ---
 
-## I18n
+## I18n (MANDATORY for frontend)
+
+> **CRITICAL:** Every frontend module MUST have 4 translation files. Missing i18n = broken UI (untranslated labels, empty buttons).
+
+**File path:** `src/i18n/locales/{lang}/{moduleLower}.json` (4 files: fr, en, it, de)
+
+**JSON key structure (identical across all 4 languages):**
+```json
+{
+  "actions": { "create": "...", "edit": "...", "delete": "...", "save": "...", "cancel": "...", "search": "...", "filter": "...", "export": "...", "refresh": "..." },
+  "labels": { "title": "...", "description": "...", "status": "...", "createdAt": "...", "updatedAt": "..." },
+  "columns": { "name": "...", "status": "...", "date": "...", "actions": "..." },
+  "form": { "name": "...", "description": "...", "submit": "...", "required": "..." },
+  "errors": { "loadFailed": "...", "saveFailed": "...", "deleteFailed": "...", "notFound": "..." },
+  "validation": { "required": "...", "minLength": "...", "maxLength": "...", "invalid": "..." },
+  "messages": { "created": "...", "updated": "...", "deleted": "...", "confirmDelete": "..." },
+  "empty": { "title": "...", "description": "...", "action": "..." }
+}
+```
+
+**Usage pattern in TSX:** `t('{moduleLower}:actions.create', 'Create')` — ALWAYS namespace prefix + fallback value.
 
 **Rules:**
-- 4 JSON files: fr, en, it, de
-- Identical key structures across all languages
+- 4 JSON files: fr, en, it, de — ALL mandatory (not just fr/en)
+- Identical key structures across all languages — same keys, translated values
 - All UI labels, validation messages, button text, empty states
-- Depends on frontend task completion
+- Depends on frontend page completion (pages define which keys are needed)
+- Add entity-specific keys under `labels`, `columns`, `form` for each entity field
+- Reference `smartstack-frontend.md` for the complete template
 
 ---
 

package/templates/skills/ralph-loop/references/compact-loop.md

@@ -99,8 +99,8 @@ Batch: {batch.length} [{firstCategory}] → {batch.map(t => `[${t.id}] ${t.descr
 | `infrastructure` with migration | `Migrations/` folder has .cs files | No migration files |
 | `infrastructure` with seed data | Seed data .cs files exist | 0 files created |
 | `application` | Service + DTO + Validator files exist | 0 files created |
-| `api` | Controller .cs files exist | 0 files created |
-| `frontend` | .tsx page files exist | 0 files created |
+| `api` | Controller .cs files exist + GetAll supports `?search=` | 0 files created OR GetAll without search param |
+| `frontend` | .tsx page files exist + FK fields use EntityLookup (not plain text) | 0 files created OR FK Guid field as `<input type="text">` |
 | `test` | Test project dir exists AND contains test .cs files | No test project or 0 test files |
 | `validation` | `dotnet test` exit 0 AND `dotnet build` exit 0 | Either command fails |
 

package/templates/skills/ralph-loop/references/core-seed-data.md

@@ -1253,3 +1253,13 @@ public class {Module}DevDataSeeder : IDevDataSeeder
 - Using `Guid.NewGuid()` for TenantId
 - Omitting idempotency check (`AnyAsync`)
 - Hardcoding TenantId inline (use `SeedConstants.DefaultTenantId`)
+
+### CROSS-SCHEMA FK WARNING
+
+> `SeedConstants.DefaultTenantId` MUST reference a tenant that EXISTS in `core.tenant_Tenants`.
+> The SmartStack platform seeds a default tenant during `InitializeSmartStackAsync()`.
+> If you use a custom GUID, ensure it is created BEFORE `DevDataSeeder` runs (Order >= 200).
+>
+> **Pipeline validation:**
+> - ralph-loop POST-CHECK warns if GUID not found in project config
+> - validate-feature step-05 verifies FK exists in real database via SQL query

package/templates/skills/ralph-loop/steps/step-02-execute.md

@@ -63,7 +63,7 @@ Key category triggers:
 |----------|---------------|
 | `infrastructure` with `_migrationMeta` | Migration sequence: MCP suggest → add → update → build |
 | `infrastructure` with seed data keywords | **MANDATORY**: Read `references/core-seed-data.md` |
-| `frontend` | MCP-first: scaffold_api_client → scaffold_routes → pages |
+| `frontend` | MCP-first: scaffold_api_client → scaffold_routes → `/ui-components` skill (pages) → form pages (create/edit = full pages, ZERO modals) → form tests → i18n (4 langues) |
 | `test` | Generate via MCP → run → fix loop → 100% pass required |
 | `validation` | Clean build → full test suite → MCP validate |
 
@@ -164,6 +164,43 @@ fi
 - If `auth_Permissions` is empty → all authorization checks reject → 403 on every endpoint
 - These are **foundational** — without them, ALL subsequent features (frontend, API, tests) are useless
 
+**POST-CHECK: DefaultTenantId cross-schema FK validation (WARNING)**
+
+After generating SeedConstants.cs and DevDataSeeder, verify that `DefaultTenantId` is not a phantom GUID that doesn't exist in `core.tenant_Tenants`:
+
+```bash
+# Find SeedConstants.cs
+SEED_CONSTANTS=$(find . -path "*/SeedConstants.cs" 2>/dev/null | head -1)
+if [ -n "$SEED_CONSTANTS" ]; then
+  # Extract the DefaultTenantId GUID value
+  TENANT_GUID=$(grep -oP 'DefaultTenantId\s*=\s*Guid\.Parse\("([^"]+)"\)' "$SEED_CONSTANTS" | grep -oP '"[^"]+"' | tr -d '"')
+
+  if [ -z "$TENANT_GUID" ]; then
+    echo "⚠️ WARNING: SeedConstants.cs found but DefaultTenantId not defined"
+    echo "DevDataSeeder requires SeedConstants.DefaultTenantId for business seed data"
+    echo "Fix: Add 'public static readonly Guid DefaultTenantId = Guid.Parse(\"...\");' to SeedConstants.cs"
+  else
+    # Verify the GUID is referenced somewhere in platform config (not an invented value)
+    GUID_FOUND=$(grep -r "$TENANT_GUID" appsettings*.json src/ 2>/dev/null | grep -v SeedConstants)
+
+    if [ -z "$GUID_FOUND" ]; then
+      echo "⚠️ WARNING: SeedConstants.DefaultTenantId ($TENANT_GUID) not found in platform config"
+      echo "This GUID must exist in core.tenant_Tenants at runtime"
+      echo "Verify: The SmartStack platform seeds this tenant during InitializeSmartStackAsync()"
+      echo "If using a custom TenantId, ensure it's created before DevDataSeeder runs (Order >= 200)"
+      echo "validate-feature step-05 will verify this FK exists in real database"
+    else
+      echo "✓ SeedConstants.DefaultTenantId ($TENANT_GUID) found in platform config"
+    fi
+  fi
+fi
+```
+
+**Why this matters:**
+- If `DefaultTenantId` references a GUID not in `core.tenant_Tenants`, DevDataSeeder inserts orphan rows
+- Business entities with invalid `TenantId` FK cause runtime 500 errors
+- The tenant must be seeded by `InitializeSmartStackAsync()` BEFORE DevDataSeeder (Order >= 200) runs
+
 ### 6. Validate with MCP
 
 ```
@@ -206,6 +243,78 @@ npm run lint         # ESLint — MUST exit 0
 ```
 If FAIL → read errors → fix TSX/TS files → re-run → loop until pass.
 
+**Frontend POST-CHECKs (BLOCKING for frontend category):**
+```bash
+# POST-CHECK: Forms must be full pages — ZERO modals/popups/drawers
+PAGE_FILES=$(find src/pages/ -name "*.tsx" 2>/dev/null)
+if [ -n "$PAGE_FILES" ]; then
+  MODAL_IMPORTS=$(grep -Pn "import.*(?:Modal|Dialog|Drawer|Popup|Sheet)" $PAGE_FILES 2>/dev/null)
+  if [ -n "$MODAL_IMPORTS" ]; then
+    echo "BLOCKING: Form pages must NOT use Modal/Dialog/Drawer/Popup components"
+    echo "Create/Edit forms MUST be full pages with own URL routes (/create, /:id/edit)"
+    echo "$MODAL_IMPORTS"
+    exit 1
+  fi
+fi
+
+# POST-CHECK: Create/Edit pages must exist for each module with a list page
+LIST_PAGES=$(find src/pages/ -name "*ListPage.tsx" -o -name "*sPage.tsx" 2>/dev/null | grep -v test)
+if [ -n "$LIST_PAGES" ]; then
+  for LP in $LIST_PAGES; do
+    DIR=$(dirname "$LP"); MOD=$(basename "$DIR")
+    if [ -z "$(find "$DIR" -name "*CreatePage.tsx" 2>/dev/null)" ]; then
+      echo "WARNING: Module $MOD has list page but no CreatePage"
+    fi
+    if [ -z "$(find "$DIR" -name "*EditPage.tsx" 2>/dev/null)" ]; then
+      echo "WARNING: Module $MOD has list page but no EditPage"
+    fi
+  done
+fi
+
+# POST-CHECK: Form pages must have companion test files
+FORM_PAGES=$(find src/pages/ -name "*CreatePage.tsx" -o -name "*EditPage.tsx" 2>/dev/null | grep -v test)
+if [ -n "$FORM_PAGES" ]; then
+  for FP in $FORM_PAGES; do
+    TEST="${FP%.tsx}.test.tsx"
+    if [ ! -f "$TEST" ]; then
+      echo "BLOCKING: Form page missing test file: $FP → expected $TEST"
+      exit 1
+    fi
+  done
+fi
+
+# POST-CHECK: FK fields must NOT be plain text inputs — use EntityLookup
+FORM_PAGES=$(find src/pages/ -name "*CreatePage.tsx" -o -name "*EditPage.tsx" 2>/dev/null | grep -v test | grep -v node_modules)
+if [ -n "$FORM_PAGES" ]; then
+  FK_TEXT_INPUTS=$(grep -Pn 'type=["\x27]text["\x27].*[a-z]+Id|value=\{[^}]*\.[a-z]+Id\}.*type=["\x27]text["\x27]' $FORM_PAGES 2>/dev/null)
+  if [ -n "$FK_TEXT_INPUTS" ]; then
+    echo "BLOCKING: FK Guid fields rendered as plain text inputs — MUST use EntityLookup"
+    echo "See smartstack-frontend.md section 6 for the EntityLookup pattern"
+    echo "$FK_TEXT_INPUTS"
+    exit 1
+  fi
+  FK_PLACEHOLDER=$(grep -Pn 'placeholder=["\x27].*[Ee]nter.*[Ii][Dd]|placeholder=["\x27].*[Gg][Uu][Ii][Dd]' $FORM_PAGES 2>/dev/null)
+  if [ -n "$FK_PLACEHOLDER" ]; then
+    echo "BLOCKING: Form placeholder asks user to enter ID/GUID — use EntityLookup instead"
+    echo "$FK_PLACEHOLDER"
+    exit 1
+  fi
+fi
+
+# POST-CHECK: Backend GetAll endpoints must support ?search= for EntityLookup
+CTRL_FILES=$(find src/ -path "*/Controllers/*" -name "*Controller.cs" 2>/dev/null)
+if [ -n "$CTRL_FILES" ]; then
+  for f in $CTRL_FILES; do
+    if grep -q "\[HttpGet\]" "$f" && grep -q "GetAll" "$f"; then
+      if ! grep -q "search" "$f"; then
+        echo "WARNING: Controller missing search parameter on GetAll: $f"
+        echo "GetAll MUST accept ?search= to enable EntityLookup on frontend"
+      fi
+    fi
+  done
+fi
+```
+
 **Error resolution cycle (ALL categories):**
 1. Read the FULL error output (not just first line)
 2. Identify root cause: file path + line number

package/templates/skills/validate-feature/steps/step-05-db-validation.md

@@ -132,6 +132,65 @@ fi
 
 ---
 
+## 5b. Verify Dev Seeding Configuration
+
+Before checking seed data, verify that dev seeding is actually enabled:
+
+```bash
+# Check 1: EnableDevSeeding in appsettings
+APPSETTINGS_DEV=$(ls appsettings.Development.json 2>/dev/null || ls src/*Api*/appsettings.Development.json 2>/dev/null)
+APPSETTINGS_BASE=$(ls appsettings.json 2>/dev/null || ls src/*Api*/appsettings.json 2>/dev/null)
+
+SEEDING_EXPLICIT_TRUE=false
+SEEDING_EXPLICIT_FALSE=false
+
+# Check Development appsettings first (overrides base)
+if [ -n "$APPSETTINGS_DEV" ]; then
+  if grep -q '"EnableDevSeeding".*true' "$APPSETTINGS_DEV" 2>/dev/null; then
+    SEEDING_EXPLICIT_TRUE=true
+  fi
+fi
+
+# Check base appsettings
+if [ -n "$APPSETTINGS_BASE" ] && [ "$SEEDING_EXPLICIT_TRUE" = "false" ]; then
+  if grep -q '"EnableDevSeeding".*false' "$APPSETTINGS_BASE" 2>/dev/null; then
+    SEEDING_EXPLICIT_FALSE=true
+  fi
+fi
+
+# Check 2: Program.cs override pattern
+PROGRAM_CS=$(find . -name "Program.cs" -path "*/Api/*" 2>/dev/null | head -1)
+PROGRAM_OVERRIDE=false
+if [ -n "$PROGRAM_CS" ]; then
+  if grep -q 'EnableDevSeeding.*IsDevelopment\|IsDevelopment.*EnableDevSeeding' "$PROGRAM_CS" 2>/dev/null; then
+    PROGRAM_OVERRIDE=true
+  fi
+fi
+
+# Evaluate
+if [ "$SEEDING_EXPLICIT_FALSE" = "true" ] && [ "$PROGRAM_OVERRIDE" = "false" ]; then
+  echo "⚠️ WARNING: EnableDevSeeding is FALSE in appsettings.json and NO Program.cs override detected"
+  echo "DevDataSeeder will NOT execute — test data will be empty"
+  echo "Fix option 1: Add to appsettings.Development.json: \"EnableDevSeeding\": true"
+  echo "Fix option 2: Add to Program.cs: options.EnableDevSeeding = builder.Environment.IsDevelopment();"
+  SEEDING_ENABLED=false
+elif [ "$PROGRAM_OVERRIDE" = "true" ]; then
+  echo "✓ EnableDevSeeding: Program.cs overrides to IsDevelopment() — active in Development"
+  SEEDING_ENABLED=true
+else
+  echo "✓ EnableDevSeeding: explicitly enabled"
+  SEEDING_ENABLED=true
+fi
+```
+
+| Result | Meaning | Action |
+|--------|---------|--------|
+| Program.cs override detected | PASS | Dev seeding active in Development environment |
+| Explicitly enabled | PASS | Dev seeding active |
+| FALSE + no override | **WARNING** | DevDataSeeder won't run — seed data checks in section 6 will likely fail |
+
+---
+
 ## 6. Verify Seed Data Execution
 
 ```bash
@@ -182,6 +241,28 @@ if [ "$STARTED" = "true" ]; then
   fi
 fi
 
+# --- DefaultTenantId cross-schema FK validation ---
+# Verify that SeedConstants.DefaultTenantId exists in core.tenant_Tenants
+SEED_CONSTANTS=$(find . -path "*/SeedConstants.cs" 2>/dev/null | head -1)
+if [ -n "$SEED_CONSTANTS" ]; then
+  TENANT_GUID=$(grep -oP 'DefaultTenantId\s*=\s*Guid\.Parse\("([^"]+)"\)' "$SEED_CONSTANTS" | grep -oP '"[^"]+"' | tr -d '"')
+
+  if [ -n "$TENANT_GUID" ] && [ -n "$DB_NAME" ]; then
+    TENANT_EXISTS=$(sqlcmd -S "(localdb)\MSSQLLocalDB" -d "$DB_NAME" -Q "
+      SET NOCOUNT ON;
+      SELECT COUNT(*) FROM core.tenant_Tenants WHERE Id = '$TENANT_GUID'
+    " -h -1 2>/dev/null | tr -d ' ')
+
+    if [ "$TENANT_EXISTS" = "0" ]; then
+      echo "❌ FAIL: SeedConstants.DefaultTenantId ($TENANT_GUID) does NOT exist in core.tenant_Tenants"
+      echo "DevDataSeeder inserts rows with an invalid FK — causes runtime 500 errors"
+      echo "Fix: Use a TenantId that is seeded by InitializeSmartStackAsync()"
+    else
+      echo "✓ DefaultTenantId ($TENANT_GUID) exists in core.tenant_Tenants"
+    fi
+  fi
+fi
+
 # Cleanup
 kill $SEED_PID 2>/dev/null
 wait $SEED_PID 2>/dev/null
@@ -224,6 +305,8 @@ fi
 | Pending model changes          | PASS/FAIL | dotnet ef migrations has-pending           |
 | Migrations apply cleanly       | PASS/FAIL | dotnet ef database update on temp DB       |
 | Integration tests (SQL Server) | PASS/FAIL | dotnet test --filter Integration           |
+| EnableDevSeeding active        | PASS/WARN | Program.cs override or appsettings check   |
+| DefaultTenantId FK valid       | PASS/FAIL | SeedConstants GUID exists in tenant_Tenants|
 | Seed data accessible           | PASS/WARN | API endpoints return seeded data           |
 | Temp DB cleanup                | PASS/WARN | Database dropped                           |
 
@@ -231,7 +314,7 @@ DB Validation Result: {PASS / FAIL / SKIPPED}
 ```
 
 **Interpretation:**
-- **PASS**: All 6 gaps covered — migrations, SQL Server, LINQ→SQL, isolation, seed data verified
+- **PASS**: All 8 checks covered — migrations, SQL Server, LINQ→SQL, isolation, seeding config, tenant FK, seed data verified
 - **FAIL**: One or more checks failed — fix before committing
 - **SKIPPED**: LocalDB not available — tests ran on SQLite only (partial coverage)
 
@@ -248,3 +331,5 @@ DB Validation Result: {PASS / FAIL / SKIPPED}
 | `Cannot insert the value NULL into column` | Missing NOT NULL | Add default value or make column nullable |
 | `Login failed for user` | LocalDB auth | Run `sqllocaldb start MSSQLLocalDB` |
 | `A network-related or instance-specific error` | LocalDB not running | Run `sqllocaldb start MSSQLLocalDB` |
+| `DefaultTenantId does NOT exist in core.tenant_Tenants` | Phantom FK | Use a TenantId seeded by `InitializeSmartStackAsync()` |
+| `EnableDevSeeding is FALSE` + empty tables | Seeding disabled | Add `options.EnableDevSeeding = builder.Environment.IsDevelopment()` in Program.cs |