feat(bazel): allow ignoring strict deps using @ts-ignore comment

Author: josephperrott

bazel/ts_project/strict_deps/index.mts

@@ -56,9 +56,7 @@ function checkPathsForMatch(moduleSpecifier: string, paths?: ts.MapLike<string[]
 const knownModuleSpecifiersWithExtensions = new Set(['highlight.js', 'zone.js']);
 
 for (const fileExecPath of manifest.testFiles) {
-  const content = await fs.readFile(fileExecPath, 'utf8');
-  const sf = ts.createSourceFile(fileExecPath, content, ts.ScriptTarget.ESNext, true);
-  const imports = getImportsInSourceFile(sf);
+  const imports = getImportsInSourceFile(fileExecPath);
 
   for (const i of imports) {
     const moduleSpecifier = knownModuleSpecifiersWithExtensions.has(i.moduleSpecifier)

bazel/ts_project/strict_deps/visitor.mts

@@ -7,17 +7,28 @@
  */
 
 import ts from 'typescript';
+import fs from 'node:fs';
 
 export interface Import {
   diagnosticNode: ts.Node;
   moduleSpecifier: string;
 }
 
-export function getImportsInSourceFile(sf: ts.SourceFile): Import[] {
+export function getImportsInSourceFile(fileExecPath: string): Import[] {
+  const content = fs.readFileSync(fileExecPath, 'utf8');
+  const sf = ts.createSourceFile(fileExecPath, content, ts.ScriptTarget.ESNext, true);
   const result: Import[] = [];
 
   const visitor = (node: ts.Node) => {
     if (ts.isImportDeclaration(node) || ts.isExportDeclaration(node)) {
+      const leadingComments = ts.getLeadingCommentRanges(content, node.pos) || [];
+      for (const comment of leadingComments) {
+        const commentText = content.substring(comment.pos, comment.end);
+        if (/@ts-ignore.*strict-deps/.test(commentText)) {
+          return;
+        }
+      }
+
       const moduleSpecifier = node.moduleSpecifier as ts.StringLiteral;
       // If not moduleSpecifier is included in the declaration, it is infered to be the local file,
       // essentially a self import and can be ignored.