@silvestv/migration-planificator 7.1.2 → 7.1.3

package/dist/src/autofix/prompts/constitution.js

@@ -12,9 +12,9 @@ Tu es un expert Angular spécialisé dans les migrations. Tu travailles sur un p
 
 ### 0. AUTONOMIE
 En restant critique et sécurisant sur tes actions dangereuses tu DOIS:
-- Etre le plus autonome possible
-- Professionel, sérieux et sécure
-- Eviter à L'EXTREME MAXIMUM les interactions avec l'utilisateur
+- Suivre à la lettre les commandes écrites + suivre à la lettre les phases
+- Professionnel, sérieux et sécure
+- Éviter à L'EXTREME MAXIMUM les interactions avec l'utilisateur
 
 ### 1. INTERDICTION DES PLACEHOLDERS
 Tu ne dois JAMAIS :
@@ -57,20 +57,17 @@ Exemples courants à ne PAS oublier :
 est TRÈS COÛTEUSE en tokens et déclenche un nouveau cycle de validation.
 Ne modifie en Phase 6 que si ABSOLUMENT NÉCESSAIRE (bug critique, violation flagrante).
 
-### 6. EXECUTION DE COMMANDE SHELL ET AUTONOMIE
-- Execute toujours de manière autonome les commandes définies dans contexte.md
+### 6. EXECUTION DE COMMANDE SHELL
+- Éxecutes toujours de manière autonome et exacte les commandes définies dans file-prompt.md & contexte.md
 - Sois toujours le plus autonome possible
 - Privilégie TOUJOURS les commandes autorisées dans le settings.json
-- Evite à tout prix de créer des commandes ou d'ajouter des paramètres ou chainer des commandes inutiles
-- Si un commande te parait particulièrement dangereuse ou critique, demande à l'utilisateur l'autorisation
+- Évites à tout prix de créer des commandes
+- Si un commande te parait particulièrement DANGEREUSE ou critique, demande à l'utilisateur l'autorisation
 
 ### 7. COMMANDES SHELL - RÈGLE ABSOLUE
-⛔ **COPIE-COLLE EXACT** : Utilise les commandes EXACTEMENT comme       
-écrites dans context.md
-- !!! ESSAIE DE NE PAS INVENTER DE COMMANDES SHELL UTILISE CELLE DEMANDEES !!!
+⛔ **COPIE-COLLE EXACT** : Utilise les commandes EXACTEMENT comme écrites dans file-prompte.md & context.md
+- !!! ESSAIE DE NE PAS INVENTER DE COMMANDES SHELL UTILISE CELLES DEMANDEES !!!
 - NE JAMAIS adapter pour Windows/Linux/Mac
-- NE JAMAIS ajouter de flags ou paramètres
-- NE JAMAIS chaîner différemment
 - Si la commande échoue, DEMANDE à l'utilisateur au lieu d'inventer 
 
 ## CONTEXTE

package/dist/src/autofix/prompts/file-prompt-phases.js

@@ -231,7 +231,10 @@ function buildPhase5Section(params) {
 | **5. MAJ Ledger** | **⛔ AVANT rebuild - JAMAIS APRÈS** |
 | 6. Re-exec | \`log_N+1.txt\` et boucle |
 
-> 🔍 **Erreur Angular (NGXXXX)** : TOUJOURS rechercher \`angular.dev/errors/NGXXXX\` avant de deviner la cause.
+  ### ⛔ RÈGLE BLOQUANTE - Erreurs NGXXXX
+> Avant TOUTE correction d'une erreur Angular (NGXXXX) :
+> 1. WebFetch https://angular.dev/errors/NGXXXX
+> 2. SI tu corriges SANS WebFetch → Migration INVALIDE
 > Exemple : NG0401 = BootstrapContext manquant (Angular 20 SSR), pas un problème d'inject().
 
 #### Calcul du BLAME (via ledger.json)
@@ -271,8 +274,18 @@ npx @silvestv/migration-planificator --no-report --onlyRule=${params.ruleKey} --
 > ⏭️ **Skippé** (--skip-build)` : `
 ### 5.B : Build
 
+\`\`\`bash
+# Si log court 
+npm run build
+\`\`\`
+
+> **Pourquoi cette commande complexe ?** Si \`npm run build\` produit 5000+ lignes, tu ne peux pas tout lire.
+> Ce pipe : strip ANSI colors → filtre erreurs + 60 lignes contexte → fichier lisible via \`cat\`.
+> **⛔ OBLIGATOIRE si logs longs** - aucune simplification autorisée.
+
 \`\`\`bash
 mkdir -p ${logDir}/build_logs
+# COPIER-COLLER EXACT - NE DEMANDE PAS CONFIRMATION EXECUTE !
 ${(0, bash_commands_helpers_1.logCmd)(params.migrationNumber, params.ruleKey, 'build')}
 \`\`\`
 
@@ -284,9 +297,12 @@ ${(0, bash_commands_helpers_1.logCmd)(params.migrationNumber, params.ruleKey, 'b
 > ⏭️ **Skippé** (--skip-tests)` : `
 ### 5.C : Tests
 
+> **Même logique que 5.B** : logs longs → pipe obligatoire pour filtrer erreurs.
+
 \`\`\`bash
 npm pkg get scripts.test  # Si {} → skip
 mkdir -p ${logDir}/test_logs
+# COPIER-COLLER EXACT - NE DEMANDE PAS CONFIRMATION EXECUTE !
 ${(0, bash_commands_helpers_1.logCmd)(params.migrationNumber, params.ruleKey, 'test')}
 \`\`\`
 
@@ -317,12 +333,12 @@ function buildPhase7Section(params) {
     const hasSchematic = !!params.migrationCommand;
     const validationSummary = [!params.skipBuild && 'Build OK', !params.skipTests && 'Tests OK']
         .filter(Boolean).join(', ') || 'Modifications terminées';
-    return `## ✅ PHASE 7 : COMMIT & PUSH (Si succès uniquement)
+    return `## ✅ PHASE 7 : COMMIT & PUSH & MAJ REPORT (Si succès uniquement)
 
 Une fois la validation complète (${validationSummary}) :
 
 ### Commit
-
+** !!!  ne jamais git add le dossier OUTPUT il est GITIGNORE !!! **
 \`\`\`bash
 git add .
 # Ne demande pas d'approbation de l'utilisateur commit direct !
@@ -354,6 +370,11 @@ git push origin feat/ai-migration-${params.ruleKey}
 }
 \`\`\`
 
+### Mise à jour du report 
+\`\`\`bash
+npx @silvestv/migration-planificator
+\`\`\`
+
 ### Message final
 
 \`\`\`

package/dist/src/data/rules/rearchitecture/rearchitecture-rules.json

@@ -92,5 +92,41 @@
         "Comment"
       ]
     }
+  },
+  {
+    "key": "nx_schematic_config_fix",
+    "summary": "Corriger la configuration Nx pour les schematics Angular",
+    "description": "Les schematics Angular (@angular/core:signal-input-migration, output-migration, signal-queries-migration) ne scannent que les projets ayant un target 'build' ou 'test' avec 'options.tsConfig' défini. Les projets Nx utilisant des noms de targets non-standards (browser, compile, etc.) ou les libraries sans target build exposant tsConfig sont ignorés silencieusement. Cette règle détecte les configurations manquantes.",
+    "estimated_time_per_occurrence": 5,
+    "onFile": "project.json",
+    "fileTypes": [
+      "project.json"
+    ],
+    "regex": "\"executor\":\\s*\"@angular-devkit/build-angular:(browser|server)\"",
+    "category": "architecture",
+    "isAutoFixable": false,
+    "minAngularVersion": 17,
+    "migration_command": null,
+    "risk_level": "low",
+    "code_description": "// Problème 1: App avec target 'browser' au lieu de 'build'\n// project.json (avant):\n\"browser\": { \"executor\": \"@angular-devkit/build-angular:browser\", ... }\n\n// project.json (après):\n\"build\": { \"executor\": \"@angular-devkit/build-angular:browser\", ... }\n\n// Problème 2: Library sans target build exposant tsConfig\n// project.json (avant):\n{ \"name\": \"my-lib\", \"targets\": { \"lint\": {...} } }\n\n// project.json (après):\n{ \"name\": \"my-lib\", \"targets\": { \"build\": { \"executor\": \"nx:run-commands\", \"options\": { \"command\": \"echo 'Not buildable'\", \"tsConfig\": \"libs/my-lib/tsconfig.lib.json\" } }, \"lint\": {...} } }",
+    "astPattern": null,
+    "doc_url": "https://github.com/angular/angular/blob/main/packages/core/schematics/utils/project_tsconfig_paths.ts",
+    "detection_patterns": [
+      {
+        "type": "missing_build_target",
+        "description": "Library sans target 'build' avec tsConfig",
+        "check": "project.json sans 'targets.build.options.tsConfig' ni 'targets.test.options.tsConfig'"
+      },
+      {
+        "type": "non_standard_target_name",
+        "description": "App utilisant 'browser' au lieu de 'build'",
+        "check": "project.json avec 'targets.browser' mais sans 'targets.build'"
+      },
+      {
+        "type": "missing_tsconfig_lib",
+        "description": "Library sans tsconfig.lib.json",
+        "check": "Dossier lib/ sans fichier tsconfig.lib.json"
+      }
+    ]
   }
 ]

package/dist/src/data/rules/to19/rules-19-recommande.json

@@ -12,8 +12,8 @@
     "category": "signals",
     "isAutoFixable": true,
     "minAngularVersion": 19,
-    "migration_command": "npx ng generate @angular/core:signal-input-migration --path=./ --best-effort-mode=false",
-    "nxMigrationCommand": "npx nx generate @angular/core:signal-input-migration --path=./ --best-effort-mode=false",
+    "migration_command": "npx ng generate @angular/core:signal-input-migration --path=./ --insert-todos --best-effort-mode=false",
+    "nxMigrationCommand": "npx nx generate @angular/core:signal-input-migration --path=./ --insert-todos --best-effort-mode=false",
     "risk_level": "medium",
     "code_description": "// Avant:\n@Input() name: string;\n@Input({ required: true }) age: number;\n\n// Après:\nreadonly name = input<string>();\nreadonly age = input.required<number>();",
     "astPattern": {
@@ -105,8 +105,8 @@
     "category": "signals",
     "isAutoFixable": true,
     "minAngularVersion": 19,
-    "migration_command": "npx ng generate @angular/core:signal-queries-migration --path=./ --best-effort-mode=false",
-    "nxMigrationCommand": "npx nx generate @angular/core:signal-queries-migration --path=./ --best-effort-mode=false",
+    "migration_command": "npx ng generate @angular/core:signal-queries-migration --path=./ --insert-todos --best-effort-mode=false",
+    "nxMigrationCommand": "npx nx generate @angular/core:signal-queries-migration --path=./ --insert-todos --best-effort-mode=false",
     "risk_level": "medium",
     "code_description": "// Avant:\n@ViewChild('ref') ref: ElementRef;\n\n// Après:\nreadonly ref = viewChild<ElementRef>('ref');",
     "astPattern": {

package/dist/src/data/rules/to21/rules-21-recommande.json

@@ -82,7 +82,7 @@
     },
     "category": "template",
     "isAutoFixable": true,
-    "minAngularVersion": 17,
+    "minAngularVersion": 21,
     "migration_command": "npx ng generate @angular/core:ngclass-to-class --path=./",
     "nxMigrationCommand": "npx nx generate @angular/core:ngclass-to-class --path=./",
     "risk_level": "low",
@@ -104,7 +104,7 @@
     },
     "category": "template",
     "isAutoFixable": true,
-    "minAngularVersion": 17,
+    "minAngularVersion": 21,
     "migration_command": "npx ng generate @angular/core:ngstyle-to-style --path=./",
     "nxMigrationCommand": "npx nx generate @angular/core:ngstyle-to-style --path=./",
     "risk_level": "low",

package/dist/src/models/constants/fix-ai-cli-permissions.js

@@ -43,6 +43,8 @@ exports.CLI_ALLOWED_COMMANDS = [
     'Bash(npx nx g @angular/core:*)',
     // === Nx CLI - Add ===
     'Bash(npx nx add *)',
+    // === Nx CLI - Run ===
+    'Bash(npx nx run:*)',
     // === npm - Install packages ===
     'Bash(npm install typescript@*)',
     'Bash(npm install @angular/*)',
@@ -59,11 +61,14 @@ exports.CLI_ALLOWED_COMMANDS = [
     'Bash(npm pkg *)',
     'Bash(npm pkg get *)',
     // === Build & Test ===
-    'Bash(npm run build*)',
+    'Bash(npm run build)',
+    'Bash(npm run build *)',
     'Bash(npm run build:*)',
-    'Bash(npm test*)',
+    'Bash(npm test)',
+    'Bash(npm test *)',
     'Bash(npm test:*)',
-    'Bash(npm run test*)',
+    'Bash(npm run test)',
+    'Bash(npm run test *)',
     'Bash(npm run test:*)',
     'Bash(npm ls *)',
     'Bash(npm view*)',
@@ -91,15 +96,21 @@ exports.CLI_ALLOWED_COMMANDS = [
     'Bash(git checkout -b *)',
     'Bash(git add *)',
     "Bash(git commit*)",
+    "Bash(git commit:*)",
     'Bash(git push *)',
     'Bash(git push -u *)',
     // === Nx / Angular - Versions ===
     'Bash(npx nx --version)',
     // === AST Engine ===
+    'Bash(npx @silvestv/migration-planificator)',
     'Bash(npx @silvestv/migration-planificator *)',
-    // === FileSystem Visualisation ===
+    // === FileSystem Operations ===
     'Bash(ls *)',
+    'Bash(pwd)',
     'Bash(pwd *)',
+    'Bash(mkdir *)',
+    'Bash(mkdir -p *)',
+    'Bash(cat *)',
     // === Web Search - Angular Documentation ===
     'WebFetch(domain:angular.dev)',
 ];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@silvestv/migration-planificator",
-  "version": "7.1.2",
+  "version": "7.1.3",
   "description": "Professional Angular migration analysis tool with AST precision for version upgrades (17→18, 18→19, 19→20, 20→21), Nx monorepo refactoring, workload estimation, and technical debt assessment. Interactive HTML reports with Gantt timeline and real-time editing.",
   "main": "dist/src/index.js",
   "publishConfig": {