@silvestv/migration-planificator 3.0.0 → 4.1.0

package/dist/src/data/rules/to18/rules-18-optionnelle.json

@@ -1,188 +1,188 @@
-[
-  {
-    "key": "guards_RedirectCommand",
-    "summary": "Guards peuvent retourner RedirectCommand",
-    "description": "Les guards de routage peuvent maintenant retourner RedirectCommand pour des redirections plus explicites et typées, offrant un meilleur contrôle que les simples UrlTree. Mettre à jour les signatures de type des guards.",
-    "estimated_time_per_occurrence": 8,
-    "onFile": null,
-    "fileTypes": [
-      "*.ts"
-    ],
-    "regex": "canActivate.*:.*boolean\\s*\\|\\s*UrlTree",
-    "category": "routing",
-    "auto_fixable": false,
-    "migration_command": null,
-    "risk_level": "medium",
-    "code_description": "// Avant:\ncanActivate(): boolean | UrlTree\n\n// Après:\ncanActivate(): boolean | UrlTree | RedirectCommand {\n  return new RedirectCommand('/login');\n}",
-    "astPattern": {
-      "nodeType": "MethodDeclaration",
-      "name": [
-        "canActivate",
-        "canActivateChild",
-        "canDeactivate",
-        "canMatch"
-      ],
-      "returnType": {
-        "union": {
-          "contains": [
-            "boolean",
-            "UrlTree"
-          ],
-          "missing": "RedirectCommand"
-        }
-      },
-      "excludeContext": [
-        "StringLiteral",
-        "Comment"
-      ]
-    },
-    "doc_url": "https://angular.dev/reference/migrations"
-  },
-  {
-    "key": "guards_RedirectCommand_fn",
-    "summary": "Guards fonctionnels peuvent retourner RedirectCommand",
-    "description": "Les guards fonctionnels (CanActivateFn, CanMatchFn...) peuvent maintenant retourner RedirectCommand pour une meilleure expressivité des redirections.",
-    "estimated_time_per_occurrence": 6,
-    "fileTypes": [
-      "*.ts"
-    ],
-    "regex": "\\bCan(?:Activate|Match|Deactivate|Load)Fn\\b[\\s\\S]*?boolean\\s*\\|\\s*UrlTree(?![\\s\\S]*RedirectCommand)",
-    "category": "routing",
-    "auto_fixable": false,
-    "risk_level": "medium",
-    "code_description": "// Avant:\nexport const authGuard: CanActivateFn = (route, state) => {\n  return Authservice.isAuth() ? true : inject(Router).createUrlTree(['/login']);\n};\n\n// Après:\nexport const authGuard: CanActivateFn = (route, state) => {\n  return Authservice.isAuth() ? true : new RedirectCommand('/login');\n};",
-    "astPattern": {
-      "nodeType": "VariableDeclaration",
-      "type": {
-        "containsAny": [
-          "CanActivateFn",
-          "CanMatchFn",
-          "CanDeactivateFn",
-          "CanLoadFn"
-        ]
-      },
-      "initializer": {
-        "nodeType": "ArrowFunction"
-      },
-      "returnType": {
-        "union": {
-          "contains": [
-            "boolean",
-            "UrlTree"
-          ],
-          "missing": "RedirectCommand"
-        }
-      }
-    },
-    "doc_url": "https://angular.dev/guide/router#redirectcommand"
-  },
-  {
-    "key": "ng_content_fallback",
-    "summary": "Utiliser fallback content dans ng-content",
-    "description": "Angular 18 permet de définir un contenu par défaut à l'intérieur de ng-content qui sera affiché si aucun contenu n'est projeté. Utile pour créer des composants avec des valeurs par défaut sans logique conditionnelle complexe.",
-    "estimated_time_per_occurrence": 3,
-    "onFile": null,
-    "fileTypes": [
-      "*.html",
-      "*.ts"
-    ],
-    "regex": "<ng-content[^>]*></ng-content>",
-    "astPattern": {
-      "nodeType": "Element",
-      "name": "ng-content",
-      "hasChildren": false,
-      "excludeContext": [
-        "Comment"
-      ]
-    },
-    "category": "template",
-    "auto_fixable": false,
-    "migration_command": null,
-    "risk_level": "low",
-    "code_description": "// Avant:\n<ng-content></ng-content>\n\n// Après:\n<ng-content>\n  <div class=\"default\">Contenu par défaut</div>\n</ng-content>",
-    "doc_url": "https://angular.dev/reference/migrations"
-  },
-  {
-    "key": "http_cache_auth",
-    "summary": "Cache HTTP avec headers auth",
-    "description": "Par défaut, le cache HTTP de transfert d'état exclut les requêtes avec headers d'authentification. Activer includeRequestsWithAuthHeaders: true si votre application nécessite le cache de ces requêtes.",
-    "estimated_time_per_occurrence": 5,
-    "fileTypes": [
-      "*.ts"
-    ],
-    "category": "http",
-    "risk_level": "low",
-    "auto_fixable": false,
-    "regex": "provideHttpClient\\s*\\([^)]*withHttpTransferCache\\s*\\([^)]*\\)",
-    "code_description": "provideHttpClient(\n  withHttpTransferCache({ \n    includeRequestsWithAuthHeaders: true \n  })\n)",
-    "astPattern": {
-      "nodeType": "CallExpression",
-      "functionName": "provideHttpClient",
-      "arguments": {
-        "containsCall": {
-          "functionName": "withHttpTransferCache",
-          "object": {
-            "properties": {
-              "includeRequestsWithAuthHeaders": {
-                "missing": true
-              }
-            }
-          }
-        }
-      }
-    },
-    "doc_url": "https://angular.dev/reference/migrations"
-  },
-  {
-    "key": "partial_hydration_defer",
-    "summary": "Hydratation partielle avec @defer SSR",
-    "description": "L'hydratation partielle avec @defer permet de rendre des composants côté serveur mais de différer leur hydratation côté client, réduisant le JavaScript initial et améliorant le TTI. Cette règle identifie les blocs @defer dont la configuration est sous-optimale (absence de déclencheur, de placeholder, ou usage non différé), afin d'améliorer la qualité du rendu progressif et la performance SSR.",
-    "estimated_time_per_occurrence": 15,
-    "fileTypes": [
-      "*.html",
-      "*.ts"
-    ],
-    "category": "ssr",
-    "risk_level": "medium",
-    "auto_fixable": false,
-    "regex": "(?<!<!--[^>]*?)@defer\\s*\\(",
-    "astPattern": {
-      "nodeType": "DeferredBlock",
-      "excludeContext": [
-        "Comment"
-      ]
-    },
-    "code_description": "@defer (on viewport) {\n  <heavy-component />\n} @placeholder {\n  <div>Chargement...</div>\n}",
-    "doc_url": "https://angular.dev/reference/migrations"
-  },
-  {
-    "key": "decouple_legacy_providers",
-    "summary": "Découpler les services fournis dans des modules ou composants",
-    "description": "Détecte les services Angular encore déclarés dans des tableaux `providers` d’un `@NgModule` ou d’un `@Component`. Depuis Angular standalone, il est recommandé que les services soient autonomes et marqués `providedIn: 'root'` pour être des singletons globaux. Les providers locaux créent des instances dupliquées et complexifient la gestion d’état.",
-    "estimated_time_per_occurrence": 25,
-    "onFile": null,
-    "fileTypes": ["*.ts"],
-    "regex": "@(?:NgModule|Component)\\s*\\([^)]*\\bproviders\\s*:\\s*\\[[^\\]]*\\]",
-    "category": "architecture",
-    "auto_fixable": false,
-    "migration_command": null,
-    "risk_level": "high",
-    "code_description": "// Avant (ancien modèle):\n@NgModule({\n  providers: [UserService, AuthService]\n})\nexport class UserModule {}\n\n@Component({\n  selector: 'app-profile',\n  providers: [ProfileService]\n})\nexport class ProfileComponent {}\n\n// Après (standalone / moderne):\n@Injectable({ providedIn: 'root' })\nexport class UserService {}\n\n@Injectable({ providedIn: 'root' })\nexport class ProfileService {}\n\n// Puis retirer les providers des décorateurs.",
-    "astPattern": {
-      "nodeType": ["Identifier", "CallExpression"],
-      "context": "ProvidersArray",
-      "referTo": {
-        "decorator": "Injectable",
-        "elementNodeType": ["Identifier", "CallExpression"],
-        "propertyNotEqual": {
-          "key": "providedIn",
-          "value": "root"
-        }
-      },
-      "excludeContext": [
-        "StringLiteral",
-        "Comment"
-      ]
-    }
-  }
+[
+  {
+    "key": "guards_RedirectCommand",
+    "summary": "Guards peuvent retourner RedirectCommand",
+    "description": "Les guards de routage peuvent maintenant retourner RedirectCommand pour des redirections plus explicites et typées, offrant un meilleur contrôle que les simples UrlTree. Mettre à jour les signatures de type des guards.",
+    "estimated_time_per_occurrence": 8,
+    "onFile": null,
+    "fileTypes": [
+      "*.ts"
+    ],
+    "regex": "canActivate.*:.*boolean\\s*\\|\\s*UrlTree",
+    "category": "routing",
+    "isAutoFixable": false,
+    "migration_command": null,
+    "risk_level": "medium",
+    "code_description": "// Avant:\ncanActivate(): boolean | UrlTree\n\n// Après:\ncanActivate(): boolean | UrlTree | RedirectCommand {\n  return new RedirectCommand('/login');\n}",
+    "astPattern": {
+      "nodeType": "MethodDeclaration",
+      "name": [
+        "canActivate",
+        "canActivateChild",
+        "canDeactivate",
+        "canMatch"
+      ],
+      "returnType": {
+        "union": {
+          "contains": [
+            "boolean",
+            "UrlTree"
+          ],
+          "missing": "RedirectCommand"
+        }
+      },
+      "excludeContext": [
+        "StringLiteral",
+        "Comment"
+      ]
+    },
+    "doc_url": "https://angular.dev/reference/migrations"
+  },
+  {
+    "key": "guards_RedirectCommand_fn",
+    "summary": "Guards fonctionnels peuvent retourner RedirectCommand",
+    "description": "Les guards fonctionnels (CanActivateFn, CanMatchFn...) peuvent maintenant retourner RedirectCommand pour une meilleure expressivité des redirections.",
+    "estimated_time_per_occurrence": 6,
+    "fileTypes": [
+      "*.ts"
+    ],
+    "regex": "\\bCan(?:Activate|Match|Deactivate|Load)Fn\\b[\\s\\S]*?boolean\\s*\\|\\s*UrlTree(?![\\s\\S]*RedirectCommand)",
+    "category": "routing",
+    "isAutoFixable": false,
+    "risk_level": "medium",
+    "code_description": "// Avant:\nexport const authGuard: CanActivateFn = (route, state) => {\n  return Authservice.isAuth() ? true : inject(Router).createUrlTree(['/login']);\n};\n\n// Après:\nexport const authGuard: CanActivateFn = (route, state) => {\n  return Authservice.isAuth() ? true : new RedirectCommand('/login');\n};",
+    "astPattern": {
+      "nodeType": "VariableDeclaration",
+      "type": {
+        "containsAny": [
+          "CanActivateFn",
+          "CanMatchFn",
+          "CanDeactivateFn",
+          "CanLoadFn"
+        ]
+      },
+      "initializer": {
+        "nodeType": "ArrowFunction"
+      },
+      "returnType": {
+        "union": {
+          "contains": [
+            "boolean",
+            "UrlTree"
+          ],
+          "missing": "RedirectCommand"
+        }
+      }
+    },
+    "doc_url": "https://angular.dev/guide/router#redirectcommand"
+  },
+  {
+    "key": "ng_content_fallback",
+    "summary": "Utiliser fallback content dans ng-content",
+    "description": "Angular 18 permet de définir un contenu par défaut à l'intérieur de ng-content qui sera affiché si aucun contenu n'est projeté. Utile pour créer des composants avec des valeurs par défaut sans logique conditionnelle complexe.",
+    "estimated_time_per_occurrence": 3,
+    "onFile": null,
+    "fileTypes": [
+      "*.html",
+      "*.ts"
+    ],
+    "regex": "<ng-content[^>]*></ng-content>",
+    "astPattern": {
+      "nodeType": "Element",
+      "name": "ng-content",
+      "hasChildren": false,
+      "excludeContext": [
+        "Comment"
+      ]
+    },
+    "category": "template",
+    "isAutoFixable": false,
+    "migration_command": null,
+    "risk_level": "low",
+    "code_description": "// Avant:\n<ng-content></ng-content>\n\n// Après:\n<ng-content>\n  <div class=\"default\">Contenu par défaut</div>\n</ng-content>",
+    "doc_url": "https://angular.dev/reference/migrations"
+  },
+  {
+    "key": "http_cache_auth",
+    "summary": "Cache HTTP avec headers auth",
+    "description": "Par défaut, le cache HTTP de transfert d'état exclut les requêtes avec headers d'authentification. Activer includeRequestsWithAuthHeaders: true si votre application nécessite le cache de ces requêtes.",
+    "estimated_time_per_occurrence": 5,
+    "fileTypes": [
+      "*.ts"
+    ],
+    "category": "http",
+    "risk_level": "low",
+    "isAutoFixable": false,
+    "regex": "provideHttpClient\\s*\\([^)]*withHttpTransferCache\\s*\\([^)]*\\)",
+    "code_description": "provideHttpClient(\n  withHttpTransferCache({ \n    includeRequestsWithAuthHeaders: true \n  })\n)",
+    "astPattern": {
+      "nodeType": "CallExpression",
+      "functionName": "provideHttpClient",
+      "arguments": {
+        "containsCall": {
+          "functionName": "withHttpTransferCache",
+          "object": {
+            "properties": {
+              "includeRequestsWithAuthHeaders": {
+                "missing": true
+              }
+            }
+          }
+        }
+      }
+    },
+    "doc_url": "https://angular.dev/reference/migrations"
+  },
+  {
+    "key": "partial_hydration_defer",
+    "summary": "Hydratation partielle avec @defer SSR",
+    "description": "L'hydratation partielle avec @defer permet de rendre des composants côté serveur mais de différer leur hydratation côté client, réduisant le JavaScript initial et améliorant le TTI. Cette règle identifie les blocs @defer dont la configuration est sous-optimale (absence de déclencheur, de placeholder, ou usage non différé), afin d'améliorer la qualité du rendu progressif et la performance SSR.",
+    "estimated_time_per_occurrence": 15,
+    "fileTypes": [
+      "*.html",
+      "*.ts"
+    ],
+    "category": "ssr",
+    "risk_level": "medium",
+    "isAutoFixable": false,
+    "regex": "(?<!<!--[^>]*?)@defer\\s*\\(",
+    "astPattern": {
+      "nodeType": "DeferredBlock",
+      "excludeContext": [
+        "Comment"
+      ]
+    },
+    "code_description": "@defer (on viewport) {\n  <heavy-component />\n} @placeholder {\n  <div>Chargement...</div>\n}",
+    "doc_url": "https://angular.dev/reference/migrations"
+  },
+  {
+    "key": "decouple_legacy_providers",
+    "summary": "Découpler les services fournis dans des modules ou composants",
+    "description": "Détecte les services Angular encore déclarés dans des tableaux `providers` d’un `@NgModule` ou d’un `@Component`. Depuis Angular standalone, il est recommandé que les services soient autonomes et marqués `providedIn: 'root'` pour être des singletons globaux. Les providers locaux créent des instances dupliquées et complexifient la gestion d’état.",
+    "estimated_time_per_occurrence": 25,
+    "onFile": null,
+    "fileTypes": ["*.ts"],
+    "regex": "@(?:NgModule|Component)\\s*\\([^)]*\\bproviders\\s*:\\s*\\[[^\\]]*\\]",
+    "category": "architecture",
+    "isAutoFixable": false,
+    "migration_command": null,
+    "risk_level": "high",
+    "code_description": "// Avant (ancien modèle):\n@NgModule({\n  providers: [UserService, AuthService]\n})\nexport class UserModule {}\n\n@Component({\n  selector: 'app-profile',\n  providers: [ProfileService]\n})\nexport class ProfileComponent {}\n\n// Après (standalone / moderne):\n@Injectable({ providedIn: 'root' })\nexport class UserService {}\n\n@Injectable({ providedIn: 'root' })\nexport class ProfileService {}\n\n// Puis retirer les providers des décorateurs.",
+    "astPattern": {
+      "nodeType": ["Identifier", "CallExpression"],
+      "context": "ProvidersArray",
+      "referTo": {
+        "decorator": "Injectable",
+        "elementNodeType": ["Identifier", "CallExpression"],
+        "propertyNotEqual": {
+          "key": "providedIn",
+          "value": "root"
+        }
+      },
+      "excludeContext": [
+        "StringLiteral",
+        "Comment"
+      ]
+    }
+  }
 ]