@atlashub/smartstack-cli 4.35.0 → 4.36.0

package/package.json

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

package/templates/skills/apex/references/checks/seed-checks.sh

@@ -249,7 +249,7 @@ if [ -n "$PROVIDER" ]; then
     echo "CRITICAL: Translation seed data inserts without idempotency guard in $PROVIDER"
     echo "Fix: Before each NavigationTranslations.Add block, check existence:"
     echo "  if (!await context.NavigationTranslations.AnyAsync("
-    echo "      t => t.EntityId == {Module}NavigationSeedData.{Module}ModuleId"
+    echo "      t => t.EntityId == mod1Entity.Id  // actual DB ID, NOT seed-time GUID"
     echo "           && t.EntityType == NavigationEntityType.Module, ct))"
     echo "  { foreach (var t in ...) { context.NavigationTranslations.Add(...); } }"
     echo "The unique index IX_nav_Translations_EntityType_EntityId_LanguageCode will crash on duplicates."

package/templates/skills/apex/references/core-seed-data.md

@@ -109,6 +109,7 @@ public static class NavigationApplicationSeedData
         return new NavigationApplicationSeedEntry
         {
             Id = ApplicationId,
+            Zone = ApplicationZone.Business,
             Code = "{appCode}",
             Label = "{appLabel_en}",
             Description = "{appDesc_en}",
@@ -198,6 +199,7 @@ public static class NavigationApplicationSeedData
 public class NavigationApplicationSeedEntry
 {
     public Guid Id { get; init; }
+    public ApplicationZone Zone { get; init; }
     public string Code { get; init; } = null!;
     public string Label { get; init; } = null!;
     public string? Description { get; init; }
@@ -1007,11 +1009,12 @@ public class RolePermissionSeedEntry
 
 | Rule | Description |
 |------|-------------|
-| Factory methods | `NavigationModule.Create(...)`, `Role.Create(...)`, `Permission.CreateForModule(...)`, `RolePermission.Create(...)` — do not use `new Entity()` |
+| Factory methods | `NavigationApplication.Create(zone, ...)`, `Role.Create(name, shortName, category, ...)`, `Permission.CreateForModule(...)` — NEVER `new Entity()` |
 | Idempotence | Each Seed method checks existence before inserting |
 | Execution order | Navigation → Roles → Permissions → RolePermissions (roles MUST exist before mapping) |
 | SaveChanges per group | Navigation -> save -> Roles -> save -> Permissions -> save -> RolePermissions -> save |
-| FK resolution by Code | Parent entities (modules, roles) found by `Code`, not hardcoded GUID |
+| FK resolution by Code | Parent entities (applications, modules, roles) found by `Code` from DB, not seed-time GUID |
+| Translation EntityId | ALWAYS use actual DB ID (`app.Id`, `mod1Entity.Id`, `actualSection.Id`), NEVER seed-time GUID from SeedData classes |
 | DI registration | `services.AddScoped<IClientSeedDataProvider, {AppPascalName}SeedDataProvider>()` |
 
 ### Template
@@ -1054,7 +1057,7 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
         else
         {
             app = NavigationApplication.Create(
-                appEntry.Code, appEntry.Label,
+                appEntry.Zone, appEntry.Code, appEntry.Label,
                 appEntry.Description, appEntry.Icon, appEntry.IconType,
                 appEntry.Route, appEntry.DisplayOrder);
             context.NavigationApplications.Add(app);
@@ -1064,7 +1067,7 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
             foreach (var t in NavigationApplicationSeedData.GetTranslationEntries())
             {
                 context.NavigationTranslations.Add(
-                    NavigationTranslation.Create(t.EntityType, t.EntityId, t.LanguageCode, t.Label, t.Description));
+                    NavigationTranslation.Create(t.EntityType, app.Id, t.LanguageCode, t.Label, t.Description));
             }
             await ((DbContext)context).SaveChangesAsync(ct);
         }
@@ -1097,14 +1100,15 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
         // --- Module translations (idempotent — unique index IX_nav_Translations_EntityType_EntityId_LanguageCode) ---
         // Always check existence before inserting translations to avoid duplicate key errors
         // on re-runs, partial failures, or DB reset scenarios.
+        // CRITICAL: Use mod1Entity.Id (actual DB ID), NOT seed-time GUID from SeedData class.
         if (!await context.NavigationTranslations.AnyAsync(
-            t => t.EntityId == {Module1Pascal}NavigationSeedData.{Module1Pascal}ModuleId
+            t => t.EntityId == mod1Entity.Id
                  && t.EntityType == NavigationEntityType.Module, ct))
         {
             foreach (var t in {Module1Pascal}NavigationSeedData.GetTranslationEntries())
             {
                 context.NavigationTranslations.Add(
-                    NavigationTranslation.Create(t.EntityType, t.EntityId, t.LanguageCode, t.Label, t.Description));
+                    NavigationTranslation.Create(t.EntityType, mod1Entity.Id, t.LanguageCode, t.Label, t.Description));
             }
         }
         // Repeat for each module...
@@ -1129,16 +1133,21 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
         await ((DbContext)context).SaveChangesAsync(ct);
 
         // --- Section translations (idempotent — check before inserting) ---
+        // CRITICAL: secEntry.Id is a seed-time GUID ≠ actual DB ID.
+        // Resolve actual section from DB by Code+ModuleId to get the real ID.
         foreach (var secEntry in {Module1Pascal}NavigationSeedData.GetSectionEntries(mod1Entity.Id))
         {
+            var actualSection = await context.NavigationSections
+                .FirstAsync(s => s.Code == secEntry.Code && s.ModuleId == mod1Entity.Id, ct);
+
             if (!await context.NavigationTranslations.AnyAsync(
-                t => t.EntityId == secEntry.Id && t.EntityType == NavigationEntityType.Section, ct))
+                t => t.EntityId == actualSection.Id && t.EntityType == NavigationEntityType.Section, ct))
             {
                 foreach (var t in {Module1Pascal}NavigationSeedData.GetSectionTranslationEntries()
                     .Where(st => st.EntityId == secEntry.Id))
                 {
                     context.NavigationTranslations.Add(
-                        NavigationTranslation.Create(t.EntityType, t.EntityId, t.LanguageCode, t.Label, t.Description));
+                        NavigationTranslation.Create(t.EntityType, actualSection.Id, t.LanguageCode, t.Label, t.Description));
                 }
             }
         }
@@ -1174,10 +1183,13 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
 
     public async Task SeedRolesAsync(ICoreDbContext context, CancellationToken ct)
     {
-        // Check idempotence — verify by Code (roles may already exist from SmartStack core)
-        // Scope by ApplicationId — without this, roles from OTHER apps cause false positives
+        // Resolve application from DB by Code — NOT seed-time GUID
+        var app = await context.NavigationApplications
+            .FirstAsync(a => a.Code == "{appCode}", ct);
+
+        // Check idempotence — verify by Code, scoped by actual DB ApplicationId
         var existingRoleCodes = await context.Roles
-            .Where(r => r.ApplicationId == ApplicationRolesSeedData.ApplicationId
+            .Where(r => r.ApplicationId == app.Id
                 && (r.Code == "admin" || r.Code == "manager" || r.Code == "contributor" || r.Code == "viewer"))
             .Select(r => r.Code)
             .ToListAsync(ct);
@@ -1188,11 +1200,13 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
             if (existingRoleCodes.Contains(entry.Code)) continue; // Skip if already exists
 
             var role = Role.Create(
-                entry.Code,
-                entry.Name,
-                entry.Description,
-                entry.ApplicationId,
-                entry.IsSystem);
+                name: entry.Name,
+                shortName: entry.Code,
+                category: RoleCategory.Application,
+                description: entry.Description,
+                isSystem: entry.IsSystem,
+                applicationId: app.Id,
+                code: entry.Code);
             context.Roles.Add(role);
         }
         await ((DbContext)context).SaveChangesAsync(ct);
@@ -1231,13 +1245,17 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
             .AnyAsync(rp => rp.Permission!.Path.StartsWith("{appCode}."), ct);
         if (exists) return;
 
+        // Resolve application from DB by Code — NOT seed-time GUID
+        var app = await context.NavigationApplications
+            .FirstAsync(a => a.Code == "{appCode}", ct);
+
         // Resolve roles by Code from DB — do not use hardcoded GUIDs.
         // Application-scoped roles (admin, manager, contributor, viewer) are created by
         // SeedRolesAsync() above. System roles use their own IDs that do not match
         // DeterministicGuid("role:admin").
         // Load THIS application's roles + system roles only
         var roles = await context.Roles
-            .Where(r => r.ApplicationId == ApplicationRolesSeedData.ApplicationId || r.IsSystem)
+            .Where(r => r.ApplicationId == app.Id || r.IsSystem)
             .ToListAsync(ct);
 
         // Resolve permissions
@@ -1333,11 +1351,11 @@ Before marking the task as completed, verify ALL:
 
 **Application-Level (FIRST — before modules):**
 - [ ] `NavigationApplicationSeedData.cs` created (once per application, at `Infrastructure/Persistence/Seeding/Data/`)
-- [ ] Application GUID is deterministic (SHA256 of `"navigation-application-{appCode}"`)
-- [ ] GetApplicationEntry() takes no parameters (no contextId)
-- [ ] Application translations created (4 languages: fr, en, it, de, EntityType = Application)
+- [ ] Application GUID is random (`Guid.NewGuid()`) — FK resolution is by Code lookup, not fixed ID
+- [ ] GetApplicationEntry() takes no parameters (no contextId), includes `Zone = ApplicationZone.Business`
+- [ ] Application translations created (4 languages: fr, en, it, de, EntityType = Application), using `app.Id` (actual DB ID) for EntityId
 - [ ] `IClientSeedDataProvider.SeedNavigationAsync()` uses `NavigationApplicationSeedData` (NO hardcoded `{appLabel_en}` / `{appIcon}` placeholders)
-- [ ] `ApplicationRolesSeedData.ApplicationId` references `NavigationApplicationSeedData.ApplicationId` (NO `{ApplicationGuid}` placeholder)
+- [ ] `ApplicationRolesSeedData.ApplicationId` references `NavigationApplicationSeedData.ApplicationId` (DTO only — provider code resolves from DB by Code)
 
 **Module-Level:**
 - [ ] Random GUIDs via `Guid.NewGuid()` (no deterministic/sequential/fixed values)

package/templates/skills/application/references/application-roles-template.md

@@ -147,20 +147,26 @@ Add a new method `SeedRolesAsync()` to the provider:
 ```csharp
 public async Task SeedRolesAsync(ICoreDbContext context, CancellationToken ct)
 {
-    // Check idempotence
+    // Resolve application from DB by Code — NOT seed-time GUID
+    var app = await context.NavigationApplications
+        .FirstAsync(a => a.Code == "{app_code}", ct);
+
+    // Check idempotence — scoped by actual DB ApplicationId
     var exists = await context.Roles
-        .AnyAsync(r => r.ApplicationId == ApplicationRolesSeedData.ApplicationId, ct);
+        .AnyAsync(r => r.ApplicationId == app.Id, ct);
     if (exists) return;
 
     // Create application-scoped roles using factory method
     foreach (var entry in ApplicationRolesSeedData.GetRoleEntries())
     {
         var role = Role.Create(
-            entry.Code,
-            entry.Name,
-            entry.Description,
-            entry.ApplicationId,
-            entry.IsSystem);
+            name: entry.Name,
+            shortName: entry.Code,
+            category: RoleCategory.Application,
+            description: entry.Description,
+            isSystem: entry.IsSystem,
+            applicationId: app.Id,
+            code: entry.Code);
 
         context.Roles.Add(role);
     }
@@ -205,7 +211,7 @@ Before marking the task as completed, verify:
 ## Notes
 
 - **Application ID source:** Read from the navigation application created in `SeedNavigationAsync()` or from `{AppPascal}NavigationSeedData.cs`
-- **Role factory method:** Use `Role.Create(code, name, description, applicationId, isSystem)` from SmartStack.Domain
+- **Role factory method:** Use `Role.Create(name, shortName, category, description, isSystem, applicationId: ..., code: ...)` from SmartStack.Domain
 - **Code uniqueness:** Role codes must be unique within the application scope
 - **System roles:** These are NOT system roles (IsSystem = false) - they are application-scoped roles
 - **Tenant isolation:** Application-scoped roles are automatically tenant-isolated via the Core authorization system

package/templates/skills/application/references/provider-template.md

@@ -43,7 +43,7 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
         else
         {
             app = NavigationApplication.Create(
-                appEntry.Code, appEntry.Label,
+                ApplicationZone.Business, appEntry.Code, appEntry.Label,
                 appEntry.Description, appEntry.Icon, appEntry.IconType,
                 appEntry.Route, appEntry.DisplayOrder);
             context.NavigationApplications.Add(app);
@@ -53,7 +53,7 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
             foreach (var t in NavigationApplicationSeedData.GetTranslationEntries())
             {
                 context.NavigationTranslations.Add(
-                    NavigationTranslation.Create(t.EntityType, t.EntityId, t.LanguageCode, t.Label, t.Description));
+                    NavigationTranslation.Create(t.EntityType, app.Id, t.LanguageCode, t.Label, t.Description));
             }
             await ((DbContext)context).SaveChangesAsync(ct);
         }
@@ -72,18 +72,24 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
         // --- Module translations (IDEMPOTENT — check before inserting) ---
         // CRITICAL: nav_Translations has unique index IX_nav_Translations_EntityType_EntityId_LanguageCode
         // Always guard with AnyAsync to prevent duplicate key errors on re-runs.
+        // CRITICAL: Use modEntity.Id (actual DB ID), NOT seed-time GUID from SeedData class.
         // if (!await context.NavigationTranslations.AnyAsync(
-        //     t => t.EntityId == {Module}NavigationSeedData.{Module}ModuleId
+        //     t => t.EntityId == mod1Entity.Id
         //          && t.EntityType == NavigationEntityType.Module, ct))
-        // { foreach (var t in {Module}NavigationSeedData.GetTranslationEntries()) { ... } }
+        // { foreach (var t in {Module}NavigationSeedData.GetTranslationEntries())
+        //   { context.NavigationTranslations.Add(
+        //       NavigationTranslation.Create(t.EntityType, mod1Entity.Id, t.LanguageCode, t.Label, t.Description)); } }
         await ((DbContext)context).SaveChangesAsync(ct);
 
         // --- Sections (idempotent per-section) ---
         // Check each section before inserting: AnyAsync(s => s.Code == secEntry.Code && s.ModuleId == ...)
 
         // --- Section translations (IDEMPOTENT — same guard pattern as module translations) ---
+        // CRITICAL: secEntry.Id is a seed-time GUID ≠ actual DB ID.
+        // Resolve actual section from DB: var actualSection = await context.NavigationSections
+        //     .FirstAsync(s => s.Code == secEntry.Code && s.ModuleId == modEntity.Id, ct);
         // if (!await context.NavigationTranslations.AnyAsync(
-        //     t => t.EntityId == secEntry.Id && t.EntityType == NavigationEntityType.Section, ct))
+        //     t => t.EntityId == actualSection.Id && t.EntityType == NavigationEntityType.Section, ct))
 
         // --- Resources (idempotent — use ACTUAL section IDs from DB) ---
         // CRITICAL: NavigationSection.Create() generates its own random ID.
@@ -95,10 +101,13 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
 
     public async Task SeedRolesAsync(ICoreDbContext context, CancellationToken ct)
     {
-        // Check idempotence
-        var applicationId = ApplicationRolesSeedData.ApplicationId;
+        // Resolve application from DB by Code — NOT seed-time GUID
+        var app = await context.NavigationApplications
+            .FirstAsync(a => a.Code == "{app_code}", ct);
+
+        // Check idempotence — scoped by actual DB ApplicationId
         var exists = await context.Roles
-            .AnyAsync(r => r.ApplicationId == applicationId, ct);
+            .AnyAsync(r => r.ApplicationId == app.Id, ct);
         if (exists) return;
 
         // Create application-scoped roles (Admin, Manager, Contributor, Viewer)
@@ -106,11 +115,13 @@ public class {AppPascalName}SeedDataProvider : IClientSeedDataProvider
         foreach (var entry in ApplicationRolesSeedData.GetRoleEntries())
         {
             var role = Role.Create(
-                entry.Code,
-                entry.Name,
-                entry.Description,
-                entry.ApplicationId,
-                entry.IsSystem);
+                name: entry.Name,
+                shortName: entry.Code,
+                category: RoleCategory.Application,
+                description: entry.Description,
+                isSystem: entry.IsSystem,
+                applicationId: app.Id,
+                code: entry.Code);
             context.Roles.Add(role);
         }
         await ((DbContext)context).SaveChangesAsync(ct);
@@ -167,11 +178,12 @@ services.AddScoped<IClientSeedDataProvider, {AppPascalName}SeedDataProvider>();
 
 ## Critical Rules
 
-1. **Factory methods mandatory**: `NavigationModule.Create(...)`, `Role.Create(...)`, `Permission.CreateForModule(...)`, `RolePermission.Create(...)` — NEVER `new Entity()`
+1. **Factory methods mandatory**: `NavigationApplication.Create(zone, ...)`, `NavigationModule.Create(...)`, `Role.Create(name, shortName, category, ...)`, `Permission.CreateForModule(...)`, `RolePermission.Create(...)` — NEVER `new Entity()`
 2. **Idempotence**: Each Seed method checks existence before inserting. **Translations require explicit `AnyAsync` guard** — unique index `IX_nav_Translations_EntityType_EntityId_LanguageCode` causes crashes on duplicate inserts
-3. **Translation guard pattern**: `if (!await context.NavigationTranslations.AnyAsync(t => t.EntityId == {id} && t.EntityType == ..., ct))` — MANDATORY before every translation batch insert
-4. **SaveChangesAsync per group**: Navigation → save → Roles → save → Permissions → save → RolePermissions → save
-4. **Execution order**: `SeedRolesAsync()` MUST be called BEFORE `SeedRolePermissionsAsync()` (roles must exist before mapping)
-5. **Random GUIDs**: ALWAYS use `Guid.NewGuid()` — resolve FKs by Code lookup, not fixed IDs
-6. **Resolve FK by Code**: Parent modules and roles are found by `Code`, not hardcoded GUID
-7. **Order property**: Use `100` as default. If multiple providers exist, they run in Order sequence
+3. **Translation EntityId**: ALWAYS use actual DB ID (e.g. `app.Id`, `mod1Entity.Id`, `actualSection.Id`), NEVER seed-time GUID from SeedData classes
+4. **Translation guard pattern**: `if (!await context.NavigationTranslations.AnyAsync(t => t.EntityId == {actualDbId} && t.EntityType == ..., ct))` — MANDATORY before every translation batch insert
+5. **SaveChangesAsync per group**: Navigation → save → Roles → save → Permissions → save → RolePermissions → save
+6. **Execution order**: `SeedRolesAsync()` MUST be called BEFORE `SeedRolePermissionsAsync()` (roles must exist before mapping)
+7. **Random GUIDs**: ALWAYS use `Guid.NewGuid()` — resolve FKs by Code lookup, not fixed IDs
+8. **Resolve FK by Code**: Parent entities (applications, modules, roles) are found by `Code` from DB, not by seed-time GUID
+9. **Order property**: Use `100` as default. If multiple providers exist, they run in Order sequence

package/templates/skills/application/templates-frontend.md

@@ -79,6 +79,274 @@ export function $MODULE_PASCALPage() {
 
 ---
 
+## TEMPLATE: DETAIL PAGE
+
+```tsx
+// pages/$APPLICATION/$MODULE/$MODULE_PASCALDetailPage.tsx
+
+import { useState, useEffect, useCallback } from 'react';
+import { useParams, useNavigate } from 'react-router-dom';
+import { useTranslation } from 'react-i18next';
+import { ArrowLeft, Pencil, Trash2, Loader2 } from 'lucide-react';
+import { Breadcrumb } from '@/components/ui/Breadcrumb';
+import { $moduleApi, type $ENTITY_PASCALDto } from '@/services/api/$moduleApi';
+
+export function $MODULE_PASCALDetailPage() {
+  // ⚠️ CRITICAL: DynamicRouter generates /:id — use destructuring rename
+  const { id: $entityId } = useParams<{ id: string }>();
+  const navigate = useNavigate();
+  const { t } = useTranslation(['$module', 'common']);
+
+  const [$entity, set$ENTITY_PASCAL] = useState<$ENTITY_PASCALDto | null>(null);
+  const [loading, setLoading] = useState(true);
+  const [error, setError] = useState<string | null>(null);
+
+  const loadData = useCallback(async () => {
+    if (!$entityId) return;
+    try {
+      setLoading(true);
+      setError(null);
+      const data = await $moduleApi.getById($entityId);
+      set$ENTITY_PASCAL(data);
+    } catch (err) {
+      setError(t('common:errors.loadFailed'));
+      console.error('Failed to load $entity:', err);
+    } finally {
+      setLoading(false);
+    }
+  }, [$entityId, t]);
+
+  useEffect(() => {
+    loadData();
+  }, [loadData]);
+
+  const handleDelete = async () => {
+    if (!$entityId || !confirm(t('common:confirmDelete'))) return;
+    try {
+      await $moduleApi.delete($entityId);
+      navigate('..');
+    } catch (err) {
+      console.error('Failed to delete:', err);
+    }
+  };
+
+  if (loading) {
+    return (
+      <div className="flex items-center justify-center py-12">
+        <Loader2 className="w-8 h-8 animate-spin text-[var(--color-accent-500)]" />
+      </div>
+    );
+  }
+
+  if (error || !$entity) {
+    return (
+      <div className="p-6">
+        <p className="text-[var(--error-text)]">{error || t('common:errors.notFound')}</p>
+      </div>
+    );
+  }
+
+  return (
+    <div className="space-y-6">
+      <Breadcrumb
+        items={[
+          { label: t('$module:title'), href: '/$APPLICATION_KEBAB/$MODULE_KEBAB' },
+          { label: $entity.name },
+        ]}
+      />
+
+      <div className="flex items-center justify-between">
+        <div className="flex items-center gap-3">
+          <button onClick={() => navigate('..')} className="p-2 hover:bg-[var(--bg-hover)]" style={{ borderRadius: 'var(--radius-button)' }}>
+            <ArrowLeft className="w-5 h-5" />
+          </button>
+          <h1 className="text-2xl font-bold text-[var(--text-primary)]">{$entity.name}</h1>
+        </div>
+        <div className="flex items-center gap-2">
+          <button
+            onClick={() => navigate('edit')}
+            className="flex items-center gap-2 px-4 py-2 bg-[var(--color-accent-500)] hover:bg-[var(--color-accent-600)] text-white font-medium transition-colors"
+            style={{ borderRadius: 'var(--radius-button)' }}
+          >
+            <Pencil className="w-4 h-4" />
+            {t('common:actions.edit')}
+          </button>
+          <button
+            onClick={handleDelete}
+            className="flex items-center gap-2 px-4 py-2 bg-[var(--error-bg)] hover:bg-[var(--error-border)] text-[var(--error-text)] font-medium transition-colors"
+            style={{ borderRadius: 'var(--radius-button)' }}
+          >
+            <Trash2 className="w-4 h-4" />
+            {t('common:actions.delete')}
+          </button>
+        </div>
+      </div>
+
+      {/* Detail content — adapt to your entity */}
+      <div className="bg-[var(--bg-card)] border border-[var(--item-color-border)] p-6" style={{ borderRadius: 'var(--radius-card)' }}>
+        <dl className="grid grid-cols-1 md:grid-cols-2 gap-4">
+          <div>
+            <dt className="text-sm font-medium text-[var(--text-secondary)]">{t('$module:columns.name')}</dt>
+            <dd className="mt-1 text-[var(--text-primary)]">{$entity.name}</dd>
+          </div>
+          <div>
+            <dt className="text-sm font-medium text-[var(--text-secondary)]">{t('$module:columns.description')}</dt>
+            <dd className="mt-1 text-[var(--text-primary)]">{$entity.description || '-'}</dd>
+          </div>
+        </dl>
+      </div>
+    </div>
+  );
+}
+```
+
+---
+
+## TEMPLATE: EDIT PAGE
+
+```tsx
+// pages/$APPLICATION/$MODULE/$MODULE_PASCALEditPage.tsx
+
+import { useState, useEffect, useCallback } from 'react';
+import { useParams, useNavigate } from 'react-router-dom';
+import { useTranslation } from 'react-i18next';
+import { ArrowLeft, Save, Loader2 } from 'lucide-react';
+import { Breadcrumb } from '@/components/ui/Breadcrumb';
+import { $moduleApi, type $ENTITY_PASCALDto, type Update$ENTITY_PASCALRequest } from '@/services/api/$moduleApi';
+
+export function $MODULE_PASCALEditPage() {
+  // ⚠️ CRITICAL: DynamicRouter generates /:id/edit — use destructuring rename
+  const { id: $entityId } = useParams<{ id: string }>();
+  const navigate = useNavigate();
+  const { t } = useTranslation(['$module', 'common']);
+
+  const [data, setData] = useState<$ENTITY_PASCALDto | null>(null);
+  const [form, setForm] = useState<Update$ENTITY_PASCALRequest>({ name: '', description: '' });
+  const [loading, setLoading] = useState(true);
+  const [saving, setSaving] = useState(false);
+  const [error, setError] = useState<string | null>(null);
+
+  const loadData = useCallback(async () => {
+    if (!$entityId) return;
+    try {
+      setLoading(true);
+      const result = await $moduleApi.getById($entityId);
+      setData(result);
+      setForm({ name: result.name, description: result.description || '' });
+    } catch (err) {
+      setError(t('common:errors.loadFailed'));
+      console.error('Failed to load $entity:', err);
+    } finally {
+      setLoading(false);
+    }
+  }, [$entityId, t]);
+
+  useEffect(() => {
+    loadData();
+  }, [loadData]);
+
+  const handleSave = async () => {
+    if (!$entityId) return;
+    try {
+      setSaving(true);
+      setError(null);
+      await $moduleApi.update($entityId, form);
+      navigate('..');
+    } catch (err) {
+      setError(t('common:errors.saveFailed'));
+      console.error('Failed to save:', err);
+    } finally {
+      setSaving(false);
+    }
+  };
+
+  if (loading) {
+    return (
+      <div className="flex items-center justify-center py-12">
+        <Loader2 className="w-8 h-8 animate-spin text-[var(--color-accent-500)]" />
+      </div>
+    );
+  }
+
+  return (
+    <div className="space-y-6">
+      <Breadcrumb
+        items={[
+          { label: t('$module:title'), href: '/$APPLICATION_KEBAB/$MODULE_KEBAB' },
+          { label: data?.name || '...', href: '..' },
+          { label: t('common:actions.edit') },
+        ]}
+      />
+
+      <div className="flex items-center gap-3">
+        <button onClick={() => navigate('..')} className="p-2 hover:bg-[var(--bg-hover)]" style={{ borderRadius: 'var(--radius-button)' }}>
+          <ArrowLeft className="w-5 h-5" />
+        </button>
+        <h1 className="text-2xl font-bold text-[var(--text-primary)]">
+          {t('common:actions.edit')} — {data?.name}
+        </h1>
+      </div>
+
+      {error && (
+        <div className="p-4 bg-[var(--error-bg)] border border-[var(--error-border)]" style={{ borderRadius: 'var(--radius-card)' }}>
+          <span className="text-[var(--error-text)]">{error}</span>
+        </div>
+      )}
+
+      <div className="bg-[var(--bg-card)] border border-[var(--item-color-border)] p-6" style={{ borderRadius: 'var(--radius-card)' }}>
+        <div className="space-y-4 max-w-lg">
+          <div>
+            <label className="block text-sm font-medium text-[var(--text-secondary)] mb-1">
+              {t('$module:columns.name')}
+            </label>
+            <input
+              type="text"
+              value={form.name}
+              onChange={(e) => setForm(prev => ({ ...prev, name: e.target.value }))}
+              className="w-full px-3 py-2 bg-[var(--bg-primary)] border border-[var(--border-color)] text-sm focus:outline-none focus:border-[var(--color-accent-500)]"
+              style={{ borderRadius: 'var(--radius-input)' }}
+            />
+          </div>
+          <div>
+            <label className="block text-sm font-medium text-[var(--text-secondary)] mb-1">
+              {t('$module:columns.description')}
+            </label>
+            <textarea
+              value={form.description || ''}
+              onChange={(e) => setForm(prev => ({ ...prev, description: e.target.value }))}
+              rows={3}
+              className="w-full px-3 py-2 bg-[var(--bg-primary)] border border-[var(--border-color)] text-sm focus:outline-none focus:border-[var(--color-accent-500)]"
+              style={{ borderRadius: 'var(--radius-input)' }}
+            />
+          </div>
+        </div>
+
+        <div className="flex justify-end gap-3 mt-6 pt-4 border-t border-[var(--border-color)]">
+          <button
+            onClick={() => navigate('..')}
+            className="px-4 py-2 bg-[var(--bg-secondary)] hover:bg-[var(--bg-tertiary)] text-[var(--text-primary)] font-medium transition-colors"
+            style={{ borderRadius: 'var(--radius-button)' }}
+          >
+            {t('common:actions.cancel')}
+          </button>
+          <button
+            onClick={handleSave}
+            disabled={saving}
+            className="flex items-center gap-2 px-4 py-2 bg-[var(--color-accent-500)] hover:bg-[var(--color-accent-600)] text-white font-medium transition-colors disabled:opacity-50"
+            style={{ borderRadius: 'var(--radius-button)' }}
+          >
+            {saving ? <Loader2 className="w-4 h-4 animate-spin" /> : <Save className="w-4 h-4" />}
+            {t('common:actions.save')}
+          </button>
+        </div>
+      </div>
+    </div>
+  );
+}
+```
+
+---
+
 ## TEMPLATE: LIST VIEW (Reusable component)
 
 ```tsx
@@ -557,6 +825,28 @@ INSERT INTO core.nav_Sections (...)
 VALUES (..., ComponentKey = '$APPLICATION_KEBAB.$MODULE_KEBAB', ...);
 ```
 
+### ⚠️ IMPLICIT ROUTE PARAM CONVENTION
+
+DynamicRouter generates implicit sub-routes for `.detail`, `.create`, and `.edit` page keys:
+
+| Suffix | URL pattern | Route param |
+|--------|-------------|-------------|
+| `.detail` | `/:id` | `id` |
+| `.create` | `/create` | — |
+| `.edit` | `/:id/edit` | `id` |
+
+**The route param is ALWAYS `id`** — never `ticketId`, `userId`, or any entity-specific name.
+
+In detail/edit pages, use destructuring rename to keep a readable local variable:
+
+```tsx
+// ✅ CORRECT — matches DynamicRouter's /:id pattern
+const { id: $entityId } = useParams<{ id: string }>();
+
+// ❌ WRONG — will be undefined because the route param is :id, not :$entityId
+const { $entityId } = useParams<{ $entityId: string }>();
+```
+
 ### How DynamicRouter resolves routes
 
 1. Fetches menu from `GET /api/navigation/menu`
@@ -618,11 +908,13 @@ These patterns are **strictly prohibited** in generated frontend code:
 | Check | Status |
 |-------|--------|
 | ☐ Main page created (`$MODULE_PASCALPage.tsx`) | |
+| ☐ Detail page created (`$MODULE_PASCALDetailPage.tsx`) with `useParams<{ id: string }>` | |
+| ☐ Edit page created (`$MODULE_PASCALEditPage.tsx`) with `useParams<{ id: string }>` | |
 | ☐ ListView component created (`$MODULE_PASCALListView.tsx`) | |
 | ☐ Preferences hook created (`use$MODULE_PASCALPreferences.ts`) | |
 | ☐ API service created (uses `apiClient`, NOT raw axios) | |
-| ☐ Routes added inside Layout wrapper in App.tsx | |
-| ☐ Route path follows `/{application_kebab}/{module_kebab}` (kebab-case) | |
+| ☐ PageRegistry keys registered (`.detail`, `.create`, `.edit`) in `componentRegistry.generated.ts` | |
+| ☐ Route param uses `{ id: entityId }` destructuring (NOT `{ entityId }`) | |
 
 ### Theme Compliance
 | Check | Status |