@atlashub/smartstack-cli 1.11.0 → 1.12.0

package/templates/commands/efcore/migration.md

@@ -4,329 +4,111 @@ agent: efcore-migration
 model: sonnet
 ---
 
-# EF Core Migration - 1 Migration per Feature
+# EF Core Migration
 
-Creates or recreates the unique migration for the current branch.
-
-**GOLDEN RULE: 1 single migration per feature/hotfix. If it already exists, delete and recreate it.**
-
-> **CLAUDE INSTRUCTION:** The `AskUserQuestion({...})` blocks are instructions to use the `AskUserQuestion` tool **interactively**. You MUST execute the tool with these parameters to get the user's response BEFORE continuing.
+> **Ref:** [_shared.md](_shared.md)
+> **Règle:** 1 migration par feature. Si existe déjà, supprimer et recréer.
 
 ---
 
-## STEP 0: Cross-Branch Validation (v1.2)
-
-**NEW:** Before creating a migration, check for conflicts with other branches.
+## STEP 0: Validation Cross-Branch
 
 ```bash
-# Check if cross-branch validation is enabled
-CROSS_BRANCH_ENABLED=$(git config --get efcore.crossBranch.enabled 2>/dev/null || echo "true")
-BLOCK_ON_CONFLICT=$(git config --get efcore.crossBranch.blockOnConflict 2>/dev/null || echo "true")
-
 if [ "$CROSS_BRANCH_ENABLED" = "true" ]; then
-  echo "Cross-branch validation..."
-
-  # Scan other branches via worktrees
-  WORKTREE_BASE=$(git config --get gitflow.worktrees.basePath 2>/dev/null || echo "../worktrees")
-
-  if [ -d "$WORKTREE_BASE/develop" ]; then
-    # Compare ModelSnapshot with develop
-    LOCAL_SNAPSHOT=$(find . -name "*ModelSnapshot.cs" -not -path "*/node_modules/*" | head -1)
-    DEVELOP_SNAPSHOT=$(find "$WORKTREE_BASE/develop" -name "*ModelSnapshot.cs" | head -1)
-
-    if [ -n "$LOCAL_SNAPSHOT" ] && [ -n "$DEVELOP_SNAPSHOT" ]; then
-      if ! diff -q "$LOCAL_SNAPSHOT" "$DEVELOP_SNAPSHOT" > /dev/null 2>&1; then
-        # Differences detected - analyze risk level
-        DIFF_LINES=$(diff "$DEVELOP_SNAPSHOT" "$LOCAL_SNAPSHOT" 2>/dev/null | wc -l)
-
-        if [ "$DIFF_LINES" -gt 100 ]; then
-          echo ""
-          echo "WARNING: CROSS-BRANCH CONFLICT DETECTED"
-          echo ""
-          echo "Your ModelSnapshot differs significantly from develop."
-          echo "Differences: $DIFF_LINES lines"
-          echo ""
-          echo "RESOLUTIONS:"
-          echo "  1. /efcore:rebase-snapshot    (recommended)"
-          echo "  2. /efcore:conflicts          (see details)"
-          echo "  3. /efcore:migration --force  (not recommended)"
-          echo ""
-
-          if [ "$BLOCK_ON_CONFLICT" = "true" ]; then
-            echo "BLOCKED: Use --force to ignore"
-            exit 1
-          fi
-        fi
-      fi
-    fi
+  # Comparer ModelSnapshot avec develop
+  DIFF_LINES=$(diff "$LOCAL_SNAPSHOT" "$DEVELOP_SNAPSHOT" 2>/dev/null | wc -l)
+  if [ "$DIFF_LINES" -gt 100 ]; then
+    echo "⚠️ CONFLIT CROSS-BRANCH DÉTECTÉ ($DIFF_LINES lignes)"
+    echo "→ /efcore:rebase-snapshot (recommandé)"
+    [ "$BLOCK_ON_CONFLICT" = "true" ] && exit 1
   fi
-
-  echo "Cross-branch validation: OK"
 fi
 ```
 
-**Options to ignore:**
-- `--force` : Ignore cross-branch validation
-- `--no-cross-check` : Disable validation for this execution
-- `--rebase-first` : Run rebase-snapshot automatically if conflict
+Options: `--force`, `--no-cross-check`, `--rebase-first`
 
 ---
 
-## STEP 1: Analyze Git Context
+## STEP 1: Contexte Git
 
 ```bash
-# Current branch
 CURRENT_BRANCH=$(git branch --show-current)
-
-# Extract type and name
-if [[ "$CURRENT_BRANCH" == feature/* ]]; then
-  BRANCH_TYPE="Feature"
-  BRANCH_NAME=$(echo "$CURRENT_BRANCH" | sed 's/feature\///' | sed 's/-/_/g' | sed 's/\b\w/\u&/g')
-elif [[ "$CURRENT_BRANCH" == hotfix/* ]]; then
-  BRANCH_TYPE="Hotfix"
-  BRANCH_NAME=$(echo "$CURRENT_BRANCH" | sed 's/hotfix\///' | sed 's/-/_/g' | sed 's/\b\w/\u&/g')
-elif [[ "$CURRENT_BRANCH" == release/* ]]; then
-  BRANCH_TYPE="Release"
-  BRANCH_NAME=$(echo "$CURRENT_BRANCH" | sed 's/release\///' | sed 's/v//')
-else
-  BRANCH_TYPE="Dev"
-  BRANCH_NAME="Manual"
-fi
-
-# Current version
-VERSION=$(grep -oP '"version":\s*"\K[^"]+' package.json 2>/dev/null || grep -oP '(?<=<Version>).*(?=</Version>)' *.csproj 2>/dev/null | head -1)
-VERSION_CLEAN=$(echo "$VERSION" | sed 's/\./_/g')
-```
-
----
-
-## STEP 2: Detect EF Core Project et Environnement
-
-```bash
-# Find the project with EF Core
-CSPROJ=$(find . -name "*.csproj" -exec grep -l "Microsoft.EntityFrameworkCore" {} \; | head -1)
-PROJECT_DIR=$(dirname "$CSPROJ")
-PROJECT_NAME=$(basename "$CSPROJ" .csproj)
-
-# Migrations folder
-MIGRATIONS_DIR="$PROJECT_DIR/Migrations"
-
-# Detect API project for environment display
-API_DIR=$(find . -type d -name "*.Api" | head -1)
-[ -z "$API_DIR" ] && API_DIR="src/SmartStack.Api"
-
-# Detect appsettings to show which DB will be affected on deploy
-SELECTED_ENV="appsettings.Local.json"
-if [ -f "$API_DIR/$SELECTED_ENV" ]; then
-  CONNECTION_STRING=$(powershell -Command "
-    \$json = Get-Content '$API_DIR/$SELECTED_ENV' -ErrorAction SilentlyContinue | ConvertFrom-Json
-    \$json.ConnectionStrings.DefaultConnection
-  " 2>/dev/null)
-  DATABASE_NAME=$(echo "$CONNECTION_STRING" | grep -oP 'Database=\K[^;]+' 2>/dev/null)
-fi
-```
-
-**Afficher info environnement (informatif):**
-
-```
-NOTE: La migration sera appliquee sur: {DATABASE_NAME:-"(detecte au deploy)"}
-      Config: {SELECTED_ENV}
+# Extraire type (feature/hotfix/release) et nom
+detect_efcore_project
 ```
 
 ---
 
-## STEP 3: Search for Existing Migration for This Branch
+## STEP 2: Recherche Migration Existante
 
 ```bash
-# Search pattern based on branch name
 SEARCH_PATTERN="${BRANCH_TYPE}_.*_${BRANCH_NAME}"
-
-# Find matching migration files
-EXISTING_MIGRATIONS=$(find "$MIGRATIONS_DIR" -name "*.cs" | grep -E "$SEARCH_PATTERN" | grep -v "Designer" | grep -v "Snapshot")
-MIGRATION_COUNT=$(echo "$EXISTING_MIGRATIONS" | grep -c "." || echo "0")
-```
-
-**Display context:**
-
-```
-================================================================================
-                    EF CORE MIGRATION - CONTEXT
-================================================================================
-
-BRANCH
-  Current:     {CURRENT_BRANCH}
-  Type:        {BRANCH_TYPE}
-  Name:        {BRANCH_NAME}
-
-VERSION:       {VERSION}
-
-EXISTING MIGRATIONS FOR THIS BRANCH:
-  {MIGRATION_COUNT} migration(s) found
-  {File list if > 0}
-
-================================================================================
+EXISTING_MIGRATIONS=$(find "$MIGRATIONS_DIR" -name "*.cs" | grep -E "$SEARCH_PATTERN")
 ```
 
 ---
 
-## STEP 4: Decision - Create or Recreate
-
-### If existing migration (MIGRATION_COUNT > 0):
+## STEP 3: Décision
 
 ```javascript
+// Si migration existe
 AskUserQuestion({
   questions: [{
-    question: "A migration already exists for this branch. What to do?",
+    question: "Migration existante. Que faire?",
     header: "Migration",
     options: [
-      { label: "Recreate", description: "Delete and recreate the migration (Recommended)" },
-      { label: "Keep", description: "Keep existing, add a new one (Not recommended)" },
-      { label: "Cancel", description: "Do nothing" }
+      { label: "Recréer", description: "Supprimer et recréer (Recommandé)" },
+      { label: "Garder", description: "Conserver, ajouter nouvelle (Non recommandé)" },
+      { label: "Annuler", description: "Ne rien faire" }
     ],
     multiSelect: false
   }]
 })
 ```
 
-**If Recreate:**
-
-```bash
-# 1. List migrations to delete
-echo "Migrations to delete:"
-for file in $EXISTING_MIGRATIONS; do
-  echo "  - $file"
-  # Find associated files (Designer, etc.)
-  BASE_NAME=$(basename "$file" .cs)
-  rm -f "$MIGRATIONS_DIR/${BASE_NAME}.cs"
-  rm -f "$MIGRATIONS_DIR/${BASE_NAME}.Designer.cs"
-done
-
-# 2. Rollback in DB if applied
-LAST_GOOD_MIGRATION=$(dotnet ef migrations list 2>/dev/null | grep -v "(Pending)" | tail -2 | head -1)
-if [ -n "$LAST_GOOD_MIGRATION" ]; then
-  dotnet ef database update "$LAST_GOOD_MIGRATION" --force
-fi
-```
-
 ---
 
-## STEP 5: Request Description
+## STEP 4: Nom Migration
+
+Pattern: `{context}_v{version}_{sequence}_{Description}`
 
 ```javascript
 AskUserQuestion({
   questions: [{
-    question: "Short migration description (e.g., AddUserRoles, FixEmailIndex)",
+    question: "Description courte (ex: AddUserRoles)",
     header: "Description",
     options: [
-      { label: "Add", description: "Adding tables/columns" },
-      { label: "Update", description: "Structure modification" },
-      { label: "Fix", description: "Schema correction" },
-      { label: "Remove", description: "Removing elements" }
+      { label: "Add", description: "Ajout tables/colonnes" },
+      { label: "Update", description: "Modification structure" },
+      { label: "Fix", description: "Correction schéma" },
+      { label: "Remove", description: "Suppression éléments" }
     ],
     multiSelect: false
   }]
 })
-
-// Then ask for specific name in free text
-// E.g., "AddUserRoles", "FixEmailNullable", "RemoveObsoleteTable"
 ```
 
 ---
 
-## STEP 6: Generate Migration Name
+## STEP 5: Création
 
 ```bash
-# Pattern: {context}_v{version}_{sequence}_{Description}
-# Example: core_v1.0.0_001_AddUserRoles
-# This format is aligned with SmartStack MCP conventions
-
-# Determine context based on branch type
-if [[ "$BRANCH_TYPE" == "Feature" ]] || [[ "$BRANCH_TYPE" == "Hotfix" ]]; then
-  CONTEXT="core"  # Default context for feature/hotfix
-else
-  CONTEXT="core"
-fi
-
-# Get next sequence number
-EXISTING_COUNT=$(find "$MIGRATIONS_DIR" -name "${CONTEXT}_v${VERSION}*.cs" | grep -v "Designer" | grep -v "Snapshot" | wc -l)
-SEQUENCE=$(printf "%03d" $((EXISTING_COUNT + 1)))
-
-MIGRATION_NAME="${CONTEXT}_v${VERSION}_${SEQUENCE}_${DESCRIPTION}"
-
-# Clean the name (no spaces, no special characters except underscore and dot)
-MIGRATION_NAME=$(echo "$MIGRATION_NAME" | sed 's/[^a-zA-Z0-9_.]//g')
-
-echo "Migration name: $MIGRATION_NAME"
-```
-
-**Generated name examples (aligned with MCP):**
-
-| Context | Version | Sequence | Description | Final Name |
-|---------|---------|----------|-------------|------------|
-| core | 1.2.0 | 001 | AddUserRoles | core_v1.2.0_001_AddUserRoles |
-| core | 1.2.0 | 002 | AddPermissions | core_v1.2.0_002_AddPermissions |
-| extensions | 1.3.0 | 001 | AddCustomFields | extensions_v1.3.0_001_AddCustomFields |
-
----
-
-## STEP 7: Create Migration
-
-```bash
-cd "$PROJECT_DIR"
-
-# Create migration with generated name
 dotnet ef migrations add "$MIGRATION_NAME" --verbose
-
-# Verify creation
-if [ $? -eq 0 ]; then
-  echo "OK Migration created successfully"
-
-  # List created files
-  NEW_FILES=$(find "$MIGRATIONS_DIR" -name "*${MIGRATION_NAME}*" -type f)
-  echo ""
-  echo "Created files:"
-  for f in $NEW_FILES; do
-    echo "  - $f"
-  done
-else
-  echo "ERROR Failed to create migration"
-  exit 1
-fi
 ```
 
 ---
 
-## STEP 8: Content Validation
-
-```bash
-# Display migration preview
-MIGRATION_FILE=$(find "$MIGRATIONS_DIR" -name "*${MIGRATION_NAME}.cs" | grep -v "Designer" | head -1)
-
-echo ""
-echo "================================================================================
-                    MIGRATION PREVIEW
-================================================================================"
-echo ""
-
-# Display Up() and Down() methods
-grep -A 20 "protected override void Up" "$MIGRATION_FILE"
-echo ""
-echo "..."
-echo ""
-grep -A 10 "protected override void Down" "$MIGRATION_FILE"
-```
-
-**Verify operations:**
+## STEP 6: Validation
 
 ```javascript
 AskUserQuestion({
   questions: [{
-    question: "Does the migration look correct?",
+    question: "Migration correcte?",
     header: "Validation",
     options: [
-      { label: "Yes, apply", description: "Deploy to local DB" },
-      { label: "Yes, not now", description: "Keep without applying" },
-      { label: "No, delete", description: "Cancel and start over" }
+      { label: "Oui, appliquer", description: "Deploy sur DB locale" },
+      { label: "Oui, pas maintenant", description: "Garder sans appliquer" },
+      { label: "Non, supprimer", description: "Annuler et recommencer" }
     ],
     multiSelect: false
   }]
@@ -335,118 +117,15 @@ AskUserQuestion({
 
 ---
 
-## STEP 9: Resume
-
-**Afficher un resume apres la creation de la migration:**
+## Résumé
 
 ```
-===============================================================================
-                         MIGRATION CRÉÉE
-===============================================================================
-
-DÉTAILS MIGRATION
--------------------------------------------------------------------------------
-  • Nom:        {MIGRATION_NAME}
-  • Branche:    {CURRENT_BRANCH}
-  • Version:    {VERSION}
-  • Context:    {CONTEXT}
-
-FICHIERS GÉNÉRÉS
--------------------------------------------------------------------------------
-  ✓ {timestamp}_{MIGRATION_NAME}.cs
-  ✓ {timestamp}_{MIGRATION_NAME}.Designer.cs
-  ✓ ApplicationDbContextModelSnapshot.cs (mis à jour)
-
-VALIDATIONS
--------------------------------------------------------------------------------
-  ✓ Règle "1 migration par feature" respectée
-  ✓ Convention de nommage respectée
-  ✓ Traçabilité branche/version assurée
-
-===============================================================================
-                         PROCHAINES ÉTAPES
-===============================================================================
-
-1. Vérifier le code généré:
-   Ouvrir Migrations/{timestamp}_{MIGRATION_NAME}.cs
-
-2. Appliquer la migration en local:
-   /efcore:db-deploy
-
-3. Committer les changements:
-   /gitflow:3-commit
-
-4. Avant le merge: Rebaser sur develop si nécessaire:
-   /efcore:rebase-snapshot
-
-===============================================================================
-```
-
-**Si des opérations destructives sont détectées:**
-
+MIGRATION - {MIGRATION_NAME}
+├── Branche: {CURRENT_BRANCH}
+├── Fichiers: {timestamp}_{NAME}.cs, Designer.cs, Snapshot.cs
+├── Validations: ✓ 1 migration/feature, ✓ Convention nommage
+└── Prochains: /efcore:db-deploy, /gitflow:3-commit
 ```
-===============================================================================
-                    MIGRATION CRÉÉE (AVEC AVERTISSEMENTS)
-===============================================================================
-
-DÉTAILS MIGRATION
--------------------------------------------------------------------------------
-  • Nom:        {MIGRATION_NAME}
-  • Branche:    {CURRENT_BRANCH}
-  • Version:    {VERSION}
-
-FICHIERS GÉNÉRÉS
--------------------------------------------------------------------------------
-  ✓ {timestamp}_{MIGRATION_NAME}.cs
-  ✓ {timestamp}_{MIGRATION_NAME}.Designer.cs
-  ✓ ApplicationDbContextModelSnapshot.cs (mis à jour)
-
-AVERTISSEMENTS
--------------------------------------------------------------------------------
-  ⚠ Opérations destructives détectées:
-    - DropColumn("OldField", "Users")     [Ligne 42]
-    - DropIndex("IX_Users_Email")         [Ligne 48]
-
-  Ces opérations causeront une PERTE DE DONNÉES.
-  Assurez-vous d'avoir un backup avant le déploiement.
-
-===============================================================================
-                         PROCHAINES ÉTAPES
-===============================================================================
-
-1. VÉRIFIER les opérations destructives:
-   Ouvrir Migrations/{timestamp}_{MIGRATION_NAME}.cs
-
-2. Créer un backup de la base si nécessaire:
-   Via SSMS ou Azure Portal
-
-3. Appliquer la migration en local:
-   /efcore:db-deploy
-
-4. Committer les changements:
-   /gitflow:3-commit
-
-===============================================================================
-```
-
----
-
-## Migration Conflict Handling
-
-When you rebase on develop and there are conflicts on ModelSnapshot:
-
-```bash
-# 1. Accept develop version for ModelSnapshot
-git checkout --theirs Migrations/ApplicationDbContextModelSnapshot.cs
-
-# 2. Delete your migration
-rm Migrations/*_{MIGRATION_NAME}.*
-
-# 3. Recreate the migration
-/efcore:migration
-```
-
-This command will do it automatically if it detects a conflict.
 
 ---
 
@@ -454,8 +133,6 @@ This command will do it automatically if it detects a conflict.
 
 | Option | Description |
 |--------|-------------|
-| `--name {name}` | Force a specific name |
-| `--no-apply` | Don't offer to apply |
-| `--force` | Delete existing without confirmation |
-| `--no-cross-check` | Disable cross-branch validation |
-| `--rebase-first` | Run rebase-snapshot automatically if conflict |
+| `--name {name}` | Forcer un nom spécifique |
+| `--force` | Supprimer sans confirmation |
+| `--no-cross-check` | Désactiver validation cross-branch |