@atlashub/smartstack-cli 1.9.0 → 1.10.0

package/templates/commands/gitflow/7-pull-request.md

@@ -199,6 +199,63 @@ if (NEEDS_PR.length > 0) {
 
 ---
 
+## ETAPE 2: SYNCHRONISATION ET VERIFICATION AHEAD/BEHIND
+
+> **CRITIQUE:** Vérifier la synchronisation avec le remote avant de créer une PR.
+
+```bash
+# Fetch pour avoir les dernières infos (si pas déjà fait en scan mode)
+git fetch origin --quiet
+
+CURRENT=$(git rev-parse --abbrev-ref HEAD)
+
+# Déterminer la branche cible
+if [[ "$CURRENT" == hotfix/* || "$CURRENT" == release/* ]]; then
+  TARGET="main"
+else
+  TARGET="develop"
+fi
+
+# Vérifier synchronisation de la branche source
+AHEAD=$(git rev-list --count origin/$CURRENT..HEAD 2>/dev/null || echo "0")
+BEHIND=$(git rev-list --count HEAD..origin/$CURRENT 2>/dev/null || echo "0")
+
+echo ""
+echo "SYNCHRONISATION"
+echo "───────────────────────────────────────"
+echo "  Branche: $CURRENT → $TARGET"
+echo "  AHEAD:   $AHEAD commit(s) non poussé(s)"
+echo "  BEHIND:  $BEHIND commit(s) non récupéré(s)"
+echo ""
+
+# Avertissements
+if [ "$BEHIND" -gt 0 ]; then
+  echo "⚠️ ATTENTION: Votre branche locale est en retard de $BEHIND commit(s)"
+  echo "→ Recommandation: git pull origin $CURRENT avant de créer la PR"
+  echo ""
+fi
+
+if [ "$AHEAD" -eq 0 ]; then
+  echo "ℹ️ Tous vos commits sont déjà poussés sur origin"
+fi
+
+# Vérifier si branche à jour par rapport à la cible
+BEHIND_TARGET=$(git rev-list --count HEAD..origin/$TARGET 2>/dev/null || echo "0")
+if [ "$BEHIND_TARGET" -gt 0 ]; then
+  echo "⚠️ Votre branche est en retard de $BEHIND_TARGET commit(s) par rapport à $TARGET"
+  echo "→ Recommandation: git rebase origin/$TARGET pour intégrer les derniers changements"
+  echo ""
+fi
+```
+
+| État | Signification | Action |
+|------|---------------|--------|
+| AHEAD > 0, BEHIND = 0 | Commits prêts à être dans la PR | ✅ Continuer |
+| BEHIND > 0 | Commits remote non récupérés | `git pull origin {branch}` |
+| BEHIND_TARGET > 0 | Branche cible a avancé | `git rebase origin/{target}` |
+
+---
+
 ## Pre-checks obligatoires
 
 ### 1. Validation branche et detection cible
@@ -289,45 +346,237 @@ echo "Cible: $TARGET"
 
 ### Titre automatique
 
-| Type branche | Format titre |
-|--------------|--------------|
-| `feature/*` | `feat: {description from branch name}` |
-| `hotfix/*` | `fix: {description from branch name}` |
-| `release/*` | `release: v{version}` |
-| Autre | `{branch name}` |
+> **IMPORTANT:** Le titre de PR doit etre standardise et sans fautes d'orthographe.
+>
+> **Prefixes TOUJOURS en anglais (Conventional Commits):**
+> - `feat:` pour les features (jamais "fonctionnalite:")
+> - `fix:` pour les corrections (jamais "corriger:")
+> - `release:` pour les releases
+>
+> Seule la **description** (apres le prefix) utilise la langue configuree.
+
+| Type branche | Format titre | Exemple FR |
+|--------------|--------------|------------|
+| `feature/*` | `feat: {description}` | `feat: Ajouter Authentification` |
+| `hotfix/*` | `fix: {description}` | `fix: Corriger Bug Login` |
+| `release/*` | `release: v{version}` | `release: v1.5.0` |
+
+**Extraction et normalisation de la description:**
 
-**Extraction description:**
+```bash
+# ═══════════════════════════════════════════════════════════════
+# EXTRACTION DU TITRE DE PR DEPUIS LE NOM DE BRANCHE
+# ═══════════════════════════════════════════════════════════════
+
+# Fonction pour convertir le nom de branche en titre lisible
+branch_to_title() {
+  local branch="$1"
+
+  # Extraire la partie apres feature/, hotfix/, release/
+  local name=$(echo "$branch" | sed 's|^[^/]*/||')
+
+  # Remplacer les tirets par des espaces
+  local title=$(echo "$name" | tr '-' ' ')
+
+  # Mettre en majuscule la premiere lettre de chaque mot significatif
+  # (garder les mots courts comme "de", "le", "la" en minuscules si FR)
+  title=$(echo "$title" | sed 's/\b\(.\)/\u\1/g')
+
+  echo "$title"
+}
+
+# Exemples:
+# feature/add-user-authentication → "Add User Authentication"
+# hotfix/fix-login-bug → "Fix Login Bug"
+# feature/ameliorer-performance-api → "Ameliorer Performance Api"
 ```
-feature/add-user-authentication → "add user authentication"
-hotfix/fix-login-bug → "fix login bug"
+
+**Generer le titre selon la langue configuree:**
+
+```bash
+# Lire la configuration de langue
+CONFIG_FILE=".claude/gitflow/config.json"
+LANG_CODE="en"
+if [ -f "$CONFIG_FILE" ]; then
+  LANG_CODE=$(grep -oP '"code":\s*"\K[^"]+' "$CONFIG_FILE" 2>/dev/null | head -1)
+  LANG_CODE=${LANG_CODE:-en}
+fi
+
+# Extraire le type de branche
+if [[ "$CURRENT" == feature/* ]]; then
+  BRANCH_TYPE="feature"
+  PREFIX="feat"
+elif [[ "$CURRENT" == hotfix/* ]]; then
+  BRANCH_TYPE="hotfix"
+  PREFIX="fix"
+elif [[ "$CURRENT" == release/* ]]; then
+  BRANCH_TYPE="release"
+  PREFIX="release"
+  # Pour les releases, extraire la version
+  VERSION=$(echo "$CURRENT" | sed 's|release/v\?||')
+fi
+
+# Generer le titre
+if [ "$BRANCH_TYPE" = "release" ]; then
+  PR_TITLE="${PREFIX}: v${VERSION}"
+else
+  DESCRIPTION=$(branch_to_title "$CURRENT")
+  PR_TITLE="${PREFIX}: ${DESCRIPTION}"
+fi
+
+echo "Titre PR genere: $PR_TITLE"
+```
+
+**Validation du titre:**
+
+```bash
+# ═══════════════════════════════════════════════════════════════
+# VALIDATION DU TITRE DE PR
+# ═══════════════════════════════════════════════════════════════
+# Regles:
+# - Longueur max: 72 caracteres
+# - Format: {prefix}: {description}
+# - Pas de caracteres speciaux interdits
+# - Premiere lettre de description en majuscule
+# ═══════════════════════════════════════════════════════════════
+
+validate_pr_title() {
+  local title="$1"
+
+  # 1. Verifier la longueur
+  if [ ${#title} -gt 72 ]; then
+    echo "⚠️ Titre trop long (${#title} > 72 chars), sera tronque"
+    title=$(echo "$title" | cut -c1-72)
+  fi
+
+  # 2. Verifier le format prefix:
+  if ! echo "$title" | grep -qE '^(feat|fix|release|chore|docs|refactor|test|perf|ci):'; then
+    echo "⚠️ Titre sans prefix valide, ajout de 'feat:'"
+    title="feat: $title"
+  fi
+
+  # 3. S'assurer que la description commence par une majuscule
+  title=$(echo "$title" | sed 's/: \([a-z]\)/: \U\1/')
+
+  echo "$title"
+}
+
+PR_TITLE=$(validate_pr_title "$PR_TITLE")
 ```
 
 ### Description automatique
 
+> **CLAUDE INSTRUCTION - REDACTION EXPERTE DES PR**
+>
+> La description de PR doit etre **auto-explicative** et permettre a un reviewer de comprendre:
+> 1. **Le contexte** - Pourquoi ce changement?
+> 2. **La solution** - Quelle approche technique?
+> 3. **L'impact** - Quel benefice concret?
+>
+> **Chaque point majeur = 1 paragraphe autonome** (comprehensible sans lire le reste)
+
+#### Format expert de description PR
+
 ```markdown
-## Summary
+## Resume
 
-{Liste des commits depuis divergence avec develop}
+{1 phrase: Quoi + Pourquoi - pas de liste, juste l'essentiel}
 
-## Changes
+## Changements
 
-- {Fichiers modifies groupes par type}
+{Pour CHAQUE modification majeure, un paragraphe structure:}
 
-## Checklist
+### {Nom du changement 1}
+
+**CONTEXTE:** {Situation problematique avant - quel etait le manque ou le bug?}
+
+**SOLUTION:** {Approche technique choisie - comment ca fonctionne maintenant?}
+
+**BENEFICE:** {Impact concret mesurable - qu'est-ce que l'utilisateur/dev gagne?}
+
+### {Nom du changement 2}
+
+**CONTEXTE:** {Description de la situation initiale}
+
+**SOLUTION:** {Implementation technique}
+
+**BENEFICE:** {Gain apporte}
+
+## Fichiers cles
+
+| Fichier | Role |
+|---------|------|
+| `path/file1.ext` | {role en 5 mots max} |
+| `path/file2.ext` | {role en 5 mots max} |
 
-- [ ] Tests ajoutés/mis à jour
-- [ ] Documentation mise à jour
-- [ ] Breaking changes documentés
-- [ ] Migration EF Core testée (si applicable)
+## Migrations EF Core
 
-## EF Core Migrations
+{Si applicable:}
+| Migration | Apport |
+|-----------|--------|
+| `{Timestamp}_{Name}` | {Ce qu'elle ajoute/modifie} |
 
-{Liste des migrations incluses, ou "Aucune migration"}
+{Sinon: "Aucune migration incluse"}
+
+## Test
+
+{Comment valider que ca fonctionne - etapes concretes}
+
+## Checklist
+
+- [ ] Tests ajoutes/mis a jour
+- [ ] Documentation mise a jour si necessaire
+- [ ] Breaking changes documentes
+- [ ] Migration EF Core testee (si applicable)
 
 ---
 🤖 Generated with GitFlow
 ```
 
+#### Regles de redaction PR
+
+| Principe | Application |
+|----------|-------------|
+| **Contexte d'abord** | Toujours expliquer le POURQUOI avant le COMMENT |
+| **Paragraphes autonomes** | Chaque section comprehensible independamment |
+| **Impact mesurable** | Benefices concrets, pas de vagues "ameliorations" |
+| **Fichiers avec role** | Pas juste la liste, mais CE QUE fait chaque fichier |
+| **Zero jargon sans explication** | Si terme technique, l'expliquer brievement |
+
+#### Verbes d'impact pour titres de sections
+
+| Eviter | Utiliser |
+|--------|----------|
+| "Ajout de X" | "Implemente X pour permettre Y" |
+| "Modification de X" | "Optimise X - gain de Y" |
+| "Correction de X" | "Resout X qui causait Y" |
+| "Mise a jour de X" | "Ameliore X en ajoutant Y" |
+
+#### Exemples de paragraphes bien rediges
+
+**BON:**
+```
+### Rescue to Feature
+
+**CONTEXTE:** Les developpeurs qui commitaient par erreur sur develop perdaient
+leur travail lors du reset, sans moyen simple de le recuperer.
+
+**SOLUTION:** Detection automatique des modifications sur develop (commits,
+fichiers stages, untracked) avec creation d'un tag de sauvegarde avant toute
+operation destructive, puis migration transparente vers une feature branch.
+
+**BENEFICE:** Zero perte de travail - meme en cas d'erreur de workflow,
+le code est preserve et accessible via le tag rescue.
+```
+
+**MAUVAIS:**
+```
+### Rescue to Feature
+
+Ajout d'une fonctionnalite de rescue. Permet de sauvegarder le travail.
+Modifie plusieurs fichiers.
+```
+
 ### Labels automatiques
 
 | Condition | Label |

package/templates/commands/gitflow/9-merge.md

@@ -37,6 +37,42 @@ fi
 
 ---
 
+## ETAPE 0.5: SYNCHRONISATION AVEC REMOTE
+
+> **CRITIQUE:** Toujours synchroniser avec le remote avant toute opération de merge.
+
+```bash
+echo "Synchronisation avec remote..."
+git fetch --all --quiet
+
+# Vérifier l'état de synchronisation
+CURRENT=$(git rev-parse --abbrev-ref HEAD)
+echo "Branche courante: $CURRENT"
+
+# Afficher état AHEAD/BEHIND pour information
+if [[ "$CURRENT" != "main" && "$CURRENT" != "develop" ]]; then
+  AHEAD=$(git rev-list --count origin/$CURRENT..HEAD 2>/dev/null || echo "0")
+  BEHIND=$(git rev-list --count HEAD..origin/$CURRENT 2>/dev/null || echo "0")
+
+  if [ "$BEHIND" -gt 0 ]; then
+    echo "⚠️ Attention: Votre branche locale est en retard de $BEHIND commit(s)"
+    echo "→ Recommandation: git pull origin $CURRENT"
+  fi
+
+  if [ "$AHEAD" -gt 0 ]; then
+    echo "ℹ️ Votre branche locale a $AHEAD commit(s) non poussé(s)"
+  fi
+fi
+```
+
+| État | Signification | Action recommandée |
+|------|---------------|-------------------|
+| BEHIND > 0 | Commits remote non récupérés | `git pull origin {branch}` |
+| AHEAD > 0 | Commits locaux non poussés | `git push origin {branch}` |
+| BEHIND = AHEAD = 0 | Parfaitement synchronisé | ✅ Prêt |
+
+---
+
 ## ETAPE 1: SCAN PROACTIF (si pas d'argument)
 
 > **COMPORTEMENT PROACTIF:** Si aucun numero de PR fourni, scanner TOUTES les PRs ouvertes pour proposer celles qui sont pretes a merger.