fix(migrations): incorrect filtering in inject migration (#62913)

crisbeto · thePunderWoman · commit b1514d512e04 · 2025-07-31T09:15:42.000Z
The inject migration wasn't implementing file filtering properly which resulted in it passing an invalid tsconfig path to TypeScript. Fixes #62866. PR Close #62913
diff --git a/packages/core/schematics/ng-generate/inject-migration/index.ts b/packages/core/schematics/ng-generate/inject-migration/index.ts
@@ -14,19 +14,18 @@ import {canMigrateFile, createMigrationProgram} from '../../utils/typescript/com
 
 import {migrateFile} from './migration';
 import {MigrationOptions} from './analysis';
+import {getProjectTsConfigPaths} from '../../utils/project_tsconfig_paths';
 
 interface Options extends MigrationOptions {
   path: string;
 }
 
 export function migrate(options: Options): Rule {
   return async (tree: Tree) => {
+    const {buildPaths, testPaths} = await getProjectTsConfigPaths(tree);
     const basePath = process.cwd();
+    const allPaths = [...buildPaths, ...testPaths];
     const pathToMigrate = normalizePath(join(basePath, options.path));
-    let allPaths = [];
-    if (pathToMigrate.trim() !== '') {
-      allPaths.push(pathToMigrate);
-    }
 
     if (!allPaths.length) {
       throw new SchematicsException(
diff --git a/packages/core/schematics/test/inject_migration_spec.ts b/packages/core/schematics/test/inject_migration_spec.ts
@@ -411,6 +411,56 @@ describe('inject migration', () => {
     ]);
   });
 
+  it('should only migrate the specified file', async () => {
+    writeFile(
+      '/dir.ts',
+      [
+        `import { Directive } from '@angular/core';`,
+        `import { Foo } from 'foo';`,
+        ``,
+        `@Directive()`,
+        `class MyDir {`,
+        `  constructor(private foo: Foo) {}`,
+        `}`,
+      ].join('\n'),
+    );
+
+    writeFile(
+      '/other-dir.ts',
+      [
+        `import { Directive } from '@angular/core';`,
+        `import { Foo } from 'foo';`,
+        ``,
+        `@Directive()`,
+        `class MyOtherDir {`,
+        `  constructor(private foo: Foo) {}`,
+        `}`,
+      ].join('\n'),
+    );
+
+    await runMigration({path: '/dir.ts'});
+
+    expect(tree.readContent('/dir.ts').split('\n')).toEqual([
+      `import { Directive, inject } from '@angular/core';`,
+      `import { Foo } from 'foo';`,
+      ``,
+      `@Directive()`,
+      `class MyDir {`,
+      `  private foo = inject(Foo);`,
+      `}`,
+    ]);
+
+    expect(tree.readContent('/other-dir.ts').split('\n')).toEqual([
+      `import { Directive } from '@angular/core';`,
+      `import { Foo } from 'foo';`,
+      ``,
+      `@Directive()`,
+      `class MyOtherDir {`,
+      `  constructor(private foo: Foo) {}`,
+      `}`,
+    ]);
+  });
+
   it('should not migrate classes decorated with a non-Angular decorator', async () => {
     writeFile(
       '/dir.ts',
