@atlashub/smartstack-cli 1.11.0 → 1.12.0

package/templates/commands/efcore/_shared.md

@@ -0,0 +1,206 @@
+# EF Core - Fonctions Partagées
+
+> **Usage:** Référencé par toutes les commandes EF Core
+
+---
+
+## Détection Projet EF Core
+
+```bash
+detect_efcore_project() {
+  CSPROJ=$(find . -name "*.csproj" -exec grep -l "Microsoft.EntityFrameworkCore" {} \; | head -1)
+  PROJECT_DIR=$(dirname "$CSPROJ")
+  PROJECT_NAME=$(basename "$CSPROJ" .csproj)
+  MIGRATIONS_DIR="$PROJECT_DIR/Migrations"
+
+  STARTUP_PROJECT=$(find . -name "*.Api.csproj" -o -name "*Web.csproj" | head -1)
+  INFRA_PROJECT=$(find . -name "*Infrastructure.csproj" | head -1)
+  [ -z "$INFRA_PROJECT" ] && INFRA_PROJECT="$CSPROJ"
+}
+```
+
+---
+
+## Détection Environnement
+
+```bash
+detect_environment() {
+  API_DIR=$(find . -type d -name "*.Api" | head -1)
+  [ -z "$API_DIR" ] && API_DIR="src/SmartStack.Api"
+
+  # Priorité: --env flag > appsettings.Local.json > erreur
+  if [ -n "$ENV_FLAG" ]; then
+    SELECTED_ENV="appsettings.${ENV_FLAG}.json"
+  elif [ -f "$API_DIR/appsettings.Local.json" ]; then
+    SELECTED_ENV="appsettings.Local.json"
+  else
+    echo "ERROR: Aucun appsettings trouvé"; exit 1
+  fi
+
+  # Extraire connection string
+  CONNECTION_STRING=$(powershell -Command "
+    \$json = Get-Content '$API_DIR/$SELECTED_ENV' | ConvertFrom-Json
+    \$json.ConnectionStrings.DefaultConnection
+  ")
+  DATABASE_NAME=$(echo "$CONNECTION_STRING" | grep -oP 'Database=\K[^;]+')
+  SERVER_NAME=$(echo "$CONNECTION_STRING" | grep -oP 'Server=\K[^;]+')
+}
+```
+
+---
+
+## Protection Production
+
+```bash
+block_production() {
+  if [[ "$SELECTED_ENV" == *"Production"* ]]; then
+    echo "BLOQUÉ: Environnement Production détecté"
+    echo "Utilisez un environnement non-production"
+    exit 1
+  fi
+}
+```
+
+---
+
+## Détermination Branche Parente
+
+**Règle d'or:** Le ModelSnapshot de référence doit venir de la branche parente (état validé).
+
+```bash
+determine_base_branch() {
+  CURRENT_BRANCH=$(git branch --show-current)
+
+  # Lire config GitFlow si disponible
+  CONFIG_FILE=".claude/gitflow/config.json"
+  GF_MAIN=$(grep -oP '"main":\s*"\K[^"]+' "$CONFIG_FILE" 2>/dev/null || echo "main")
+  GF_DEVELOP=$(grep -oP '"develop":\s*"\K[^"]+' "$CONFIG_FILE" 2>/dev/null || echo "develop")
+
+  case "$CURRENT_BRANCH" in
+    feature/*|bugfix/*)
+      BASE_BRANCH="$GF_DEVELOP"
+      BRANCH_TYPE="feature"
+      ;;
+    develop)
+      BASE_BRANCH="$GF_MAIN"
+      BRANCH_TYPE="develop"
+      ;;
+    release/*)
+      BASE_BRANCH="$GF_MAIN"
+      BRANCH_TYPE="release"
+      ;;
+    hotfix/*)
+      BASE_BRANCH="$GF_MAIN"
+      BRANCH_TYPE="hotfix"
+      ;;
+    main|master)
+      echo "❌ BLOQUÉ: Opération interdite sur $CURRENT_BRANCH"
+      exit 1
+      ;;
+    *)
+      echo "⚠️ Branche non reconnue: $CURRENT_BRANCH"
+      echo "→ Utilisation de $GF_DEVELOP comme référence par défaut"
+      BASE_BRANCH="$GF_DEVELOP"
+      BRANCH_TYPE="unknown"
+      ;;
+  esac
+
+  # Vérifier que la branche parente existe
+  if ! git rev-parse --verify "origin/$BASE_BRANCH" >/dev/null 2>&1; then
+    echo "❌ Branche parente '$BASE_BRANCH' introuvable sur origin"
+    exit 1
+  fi
+
+  echo "Branche parente: $BASE_BRANCH"
+}
+```
+
+| Branche courante | Branche parente | Raison |
+|------------------|-----------------|--------|
+| `feature/*` | `develop` | État validé de develop |
+| `bugfix/*` | `develop` | État validé de develop |
+| `develop` | `main` | État en production |
+| `release/*` | `main` | État en production |
+| `hotfix/*` | `main` | État en production |
+| `main/master` | **BLOQUÉ** | Jamais squash sur branche protégée |
+
+---
+
+## Convention Nommage Migration
+
+Pattern: `{context}_v{version}_{sequence}_{Description}`
+
+| Contexte | Version | Séquence | Description | Exemple |
+|----------|---------|----------|-------------|---------|
+| core | 1.2.0 | 001 | AddUserRoles | `core_v1.2.0_001_AddUserRoles` |
+| extensions | 1.3.0 | 001 | AddCustomFields | `extensions_v1.3.0_001_AddCustomFields` |
+
+---
+
+## Règle d'Or: 1 Migration par Feature
+
+**Ne jamais accumuler plusieurs migrations sur une feature.** Si modification du modèle:
+1. Supprimer la migration existante
+2. Recréer avec `/efcore:migration`
+
+---
+
+## Règles de Conformité
+
+| ❌ INTERDIT | ✅ OBLIGATOIRE |
+|-------------|----------------|
+| Scripts SQL (.sql) dans le repo | `dotnet ef database update` |
+| `sqlcmd` avec SQL brut | `dotnet ef migrations add` |
+| `migrationBuilder.Sql()` INSERT | `HasData()` pour seeding |
+
+---
+
+## MCP Tool Reference
+
+**Tool:** `mcp__smartstack__check_migrations`
+
+| Paramètre | Type | Description |
+|-----------|------|-------------|
+| `projectPath` | string | Chemin projet EF Core (auto-detect) |
+| `branch` | string | Branche Git (current) |
+| `compareBranch` | string | Branche de comparaison |
+
+**Response:** `MigrationCheckResult { hasConflicts, migrations[], conflicts[], suggestions[] }`
+
+---
+
+## Niveaux de Risque Conflit
+
+| Condition | Risque | Action |
+|-----------|--------|--------|
+| Snapshot = develop | NONE | Merge OK |
+| Tables différentes | LOW | Merge OK |
+| FK vers même table | MEDIUM | Ordre important |
+| Même tables/colonnes | HIGH | Rebase requis |
+
+---
+
+## Configuration
+
+`.claude/gitflow/config.json`:
+
+```json
+{
+  "efcore": {
+    "database": {
+      "configFile": "appsettings.Local.json",
+      "connectionStringName": "DefaultConnection"
+    },
+    "crossBranch": {
+      "enabled": true,
+      "blockOnConflict": true
+    }
+  }
+}
+```
+
+---
+
+## Contact Support
+
+📧 **support@atlshub.ch**

package/templates/commands/efcore/conflicts.md

@@ -4,43 +4,29 @@ agent: efcore-conflicts
 model: sonnet
 ---
 
-# EF Core Conflicts - Cross-Branch Conflict Analysis
+# EF Core Conflicts
 
-Analyzes potential conflicts between current branch and target branch using **SmartStack MCP**.
-
-**USAGE:** Before a merge or commit to ensure there are no migration conflicts.
-
-**BEHAVIOR:** Returns exit code 1 if conflict detected (blocks merge/commit).
-
-**INTEGRATION:** Uses `mcp__smartstack__check_migrations` for structured analysis.
+> **Ref:** [_shared.md](_shared.md)
+> **MCP:** `mcp__smartstack__check_migrations`
+> **Comportement:** Exit code 1 si conflit (bloque merge/commit)
 
 ---
 
-## STEP 1: Invoke MCP check_migrations
-
-> **MCP INTEGRATION:** This command delegates conflict analysis to the SmartStack MCP server
-> which provides structured, consistent analysis across all SmartStack projects.
-
-```
-CALL mcp__smartstack__check_migrations WITH:
-  - branch: {current branch}
-  - compareBranch: "develop" (or specified target)
-```
-
-**Claude instruction:** Use the MCP tool `mcp__smartstack__check_migrations` with parameters:
+## STEP 1: Invoquer MCP
 
 ```json
 {
-  "branch": "<current_branch>",
-  "compareBranch": "develop"
+  "tool": "mcp__smartstack__check_migrations",
+  "parameters": {
+    "branch": "<current_branch>",
+    "compareBranch": "develop"
+  }
 }
 ```
 
 ---
 
-## STEP 2: Parse MCP Response
-
-The MCP returns a structured `MigrationCheckResult`:
+## STEP 2: Parser Réponse
 
 ```typescript
 interface MigrationCheckResult {
@@ -49,162 +35,49 @@ interface MigrationCheckResult {
   conflicts: MigrationConflict[];
   suggestions: string[];
 }
-
-interface MigrationConflict {
-  type: 'order' | 'snapshot' | 'dependency' | 'naming';
-  description: string;
-  files: string[];
-  resolution: string;
-}
 ```
 
 ---
 
-## STEP 3: Evaluate Conflict Level
-
-Based on MCP response:
+## STEP 3: Évaluer Niveau
 
-| Conflict Type | Level | Action |
-|---------------|-------|--------|
-| `naming` | LOW | Warning only |
-| `order` | MEDIUM | Warning, suggest reorder |
-| `dependency` | HIGH | Block merge |
-| `snapshot` | CRITICAL | Block merge, require rebase |
+| Type Conflit | Niveau | Action |
+|--------------|--------|--------|
+| `naming` | LOW | Warning |
+| `order` | MEDIUM | Suggérer réordonnancement |
+| `dependency` | HIGH | Bloquer merge |
+| `snapshot` | CRITICAL | Bloquer, exiger rebase |
 
 ---
 
-## STEP 4: Resume
-
-**Si aucun conflit (`hasConflicts: false`):**
+## Résumé
 
+**Si aucun conflit:**
 ```
-===============================================================================
-                         ANALYSE DES CONFLITS
-===============================================================================
-
-CONTEXTE
--------------------------------------------------------------------------------
-  • Branche:      {current_branch}
-  • Cible:        develop
-  • Migrations:   {count}
-
-RÉSULTAT
--------------------------------------------------------------------------------
-  ✓ Aucun conflit détecté
-  ✓ ModelSnapshot synchronisé avec develop
-
-===============================================================================
-                         PROCHAINES ÉTAPES
-===============================================================================
-
-1. Continuer le développement:
-   /gitflow:3-commit
-
-2. Quand prêt, créer une Pull Request:
-   /gitflow:7-pull-request
-
-===============================================================================
+CONFLICTS - {branch}
+├── Cible: develop
+├── Résultat: ✓ Aucun conflit
+└── Prochains: /gitflow:3-commit, /gitflow:7-pull-request
 ```
 
-**Si conflits détectés (`hasConflicts: true`):**
-
+**Si conflits:**
 ```
-===============================================================================
-                    CONFLITS DÉTECTÉS - MERGE BLOQUÉ
-===============================================================================
-
-CONTEXTE
--------------------------------------------------------------------------------
-  • Branche:      {current_branch}
-  • Cible:        develop
-
-CONFLITS TROUVÉS
--------------------------------------------------------------------------------
-{for each conflict in conflicts}
-  [{conflict.type.toUpperCase()}] {conflict.description}
-    Fichiers: {conflict.files.join(', ')}
-    Résolution: {conflict.resolution}
-{end for}
-
-SUGGESTIONS MCP
--------------------------------------------------------------------------------
-{for each suggestion in suggestions}
-  → {suggestion}
-{end for}
-
-===============================================================================
-                         ACTIONS REQUISES
-===============================================================================
-
-  ⛔ Le merge est bloqué jusqu'à résolution des conflits.
-
-1. RECOMMANDÉ - Rebaser le ModelSnapshot:
-   /efcore:rebase-snapshot
-
-2. NON RECOMMANDÉ - Ignorer les conflits:
-   /efcore:conflicts --force
-
-3. Besoin d'aide?
-   support@atlshub.ch
-
-===============================================================================
-```
-
-**Si conflit critique (snapshot divergent):**
-
-```
-===============================================================================
-                    CONFLIT CRITIQUE - SNAPSHOT DIVERGENT
-===============================================================================
-
-CONTEXTE
--------------------------------------------------------------------------------
-  • Branche:      {current_branch}
-  • Cible:        develop
-
-PROBLÈME
--------------------------------------------------------------------------------
-  ⛔ Le ModelSnapshot de votre branche a divergé significativement
-     de develop ({DIFF_LINES} lignes de différence).
-
-  Cela indique que des migrations ont été ajoutées sur develop
-  depuis la création de votre branche.
-
-IMPACT
--------------------------------------------------------------------------------
-  • Le merge créera des conflits impossibles à résoudre automatiquement
-  • Les migrations pourraient s'exécuter dans le mauvais ordre
-  • La base de données pourrait être corrompue
-
-===============================================================================
-                         ACTIONS REQUISES
-===============================================================================
-
-  ⛔ MERGE BLOQUÉ - Résolution obligatoire avant de continuer.
-
-1. Rebaser le snapshot sur develop (OBLIGATOIRE):
-   /efcore:rebase-snapshot
-
-2. Recréer votre migration après le rebase:
-   /efcore:migration
-
-3. Vérifier que tout fonctionne:
-   /efcore:db-deploy
-
-===============================================================================
+CONFLICTS - {branch} - BLOQUÉ
+├── Conflits: {N} trouvés
+├── Types: {snapshot/dependency/order}
+├── Action: /efcore:rebase-snapshot (OBLIGATOIRE)
+└── Force: /efcore:conflicts --force (non recommandé)
 ```
 
 ---
 
-## STEP 5: Exit Code
+## Exit Codes
 
-```bash
-if [ "$HAS_CONFLICTS" = "true" ]; then
-  exit 1  # BLOCKING
-else
-  exit 0  # OK
-fi
-```
+| Code | Signification |
+|------|---------------|
+| 0 | Pas de conflit ou mineur |
+| 1 | Conflit détecté - merge bloqué |
+| 2 | Erreur connexion MCP |
 
 ---
 
@@ -212,41 +85,6 @@ fi
 
 | Option | Description |
 |--------|-------------|
-| `--force` | Ignore conflict (not recommended) |
-| `--target <branch>` | Compare with branch other than develop |
-| `--json` | Output raw MCP JSON response |
-
----
-
-## Exit Codes
-
-| Code | Meaning |
-|------|---------|
-| 0 | No conflict or minor conflict |
-| 1 | Conflict detected - merge blocked |
-| 2 | MCP connection error |
-
----
-
-## MCP Tool Reference
-
-**Tool:** `mcp__smartstack__check_migrations`
-
-**Description:** Analyze EF Core migrations for conflicts, ordering issues, and ModelSnapshot discrepancies between branches
-
-**Parameters:**
-
-| Parameter | Type | Description |
-|-----------|------|-------------|
-| `projectPath` | string | EF Core project path (default: auto-detect) |
-| `branch` | string | Git branch to check (default: current) |
-| `compareBranch` | string | Branch to compare against |
-
----
-
-## Why MCP Integration?
-
-1. **Consistency:** Same analysis logic across CLI, CI/CD, and IDE
-2. **Maintainability:** Single source of truth in MCP server
-3. **Structured Output:** Typed responses instead of bash text parsing
-4. **Extensibility:** Easy to add new conflict types in one place
+| `--force` | Ignorer conflit (non recommandé) |
+| `--target <branch>` | Comparer avec autre branche |
+| `--json` | Output JSON brut MCP |