@groma/scanner-angular 0.2.2 → 0.2.3

package/README.md

@@ -16,7 +16,7 @@ bun plugins/scanners/angular/build.ts
 From the project being scanned:
 
 ```sh
-groma scanner add /absolute/path/to/groma/plugins/scanners/angular/dist/package
+groma plugin add /absolute/path/to/groma/plugins/scanners/angular/dist/package
 groma scan
 ```
 

package/dist/index.js

@@ -254,31 +254,33 @@ function configOwners(wanted, configs) {
   }
   return owners;
 }
+function programWithOwnedSources(program, paths) {
+  return { program, owned: new Set(paths.map((file) => program.getSourceFile(file))) };
+}
 function angularPrograms(root, files, configs) {
   const wanted = new Set(files.map((file) => path3.resolve(root, file)));
   const owners = configOwners(wanted, configs);
   const groups = new Map;
   for (const [file, owner] of owners)
     groups.set(owner, [...groups.get(owner) ?? [], file]);
-  const programs = [...groups].map(([owner, rootNames]) => ({
-    program: ts.createProgram(rootNames, { ...owner.config.options, noEmit: true }),
-    files: new Set(rootNames)
-  }));
+  const programs = [...groups].map(([owner, rootNames]) => programWithOwnedSources(ts.createProgram(rootNames, { ...owner.config.options, noEmit: true }), rootNames));
   const loose = [];
   for (const file of wanted) {
     if (owners.has(file))
       continue;
-    const importer = programs.find(({ program }) => program.getSourceFile(file) !== undefined);
+    const readers = programs.filter(({ program }) => program.getSourceFile(file) !== undefined);
+    if (readers.some(({ program, owned }) => owned.has(program.getSourceFile(file))))
+      continue;
+    const [importer] = readers;
     if (importer)
-      importer.files.add(file);
+      importer.owned.add(importer.program.getSourceFile(file));
     else
       loose.push(file);
   }
   if (loose.length)
-    programs.push({ program: ts.createProgram(loose, { noEmit: true }), files: new Set(loose) });
-  return programs.map(({ program, files: own }) => {
-    const owned = [...own].map((file) => program.getSourceFile(file));
-    failDiagnostics(owned.flatMap((source) => program.getSyntacticDiagnostics(source)));
+    programs.push(programWithOwnedSources(ts.createProgram(loose, { noEmit: true }), loose));
+  return programs.map(({ program, owned }) => {
+    failDiagnostics([...owned].flatMap((source) => program.getSyntacticDiagnostics(source)));
     return { program, owned, sources: program.getSourceFiles().filter((source) => inRepository(root, source)) };
   });
 }
@@ -28710,20 +28712,32 @@ function reachedFiles(entry, compiled, imports, packages, units) {
 }
 async function declaredEntries(root, declarations, packages, workspaces) {
   const entries = [];
+  const diagnostics = [];
   for (const file of declarations) {
     const text = await readFile3(path5.join(root, file), "utf8");
     const name = path5.posix.basename(file);
-    if (name === "package.json") {
-      const manifest = JSON.parse(text);
-      entries.push(...declaredBins(file, manifest), ...declaredScripts(file, manifest.scripts));
-    } else if (name === "angular.json")
-      entries.push(...angularEntries(file, text));
-    else if (name === "project.json")
-      entries.push(...nxEntries(file, text, workspaces));
-    else if (file.endsWith(".html"))
-      entries.push(...browserEntries(file, text, packages));
+    try {
+      if (name === "package.json") {
+        const manifest = JSON.parse(text);
+        entries.push(...declaredBins(file, manifest), ...declaredScripts(file, manifest.scripts));
+      } else if (name === "angular.json")
+        entries.push(...angularEntries(file, text));
+      else if (name === "project.json")
+        entries.push(...nxEntries(file, text, workspaces));
+      else if (file.endsWith(".html"))
+        entries.push(...browserEntries(file, text, packages));
+    } catch (error) {
+      if (!(error instanceof SyntaxError))
+        throw error;
+      diagnostics.push({
+        severity: "warning",
+        code: "JAVASCRIPT_MANIFEST_INVALID",
+        file,
+        message: `The file is not valid JSON; the entry points it declares are not read. ${error.message}`
+      });
+    }
   }
-  return entries;
+  return { entries, diagnostics };
 }
 function sourced(entry, files, outputs = []) {
   const file = sourceOf(entry.file, outputs, files);
@@ -28737,7 +28751,8 @@ async function withJavaScriptEntries(root, observation, inputs, files) {
   const packages = new Set(inventory.filter((file) => path5.posix.basename(file) === "package.json").map((file) => path5.posix.dirname(file)));
   const active = new Set(observation.files.map((file) => packageFor(file.file, packages)));
   const declarations = inventory.filter((file) => path5.posix.basename(file) !== "nx.json" && active.has(packageFor(file, packages)));
-  const entries = [...inputs.entries, ...await declaredEntries(root, declarations, packages, workspaces)];
+  const declared = await declaredEntries(root, declarations, packages, workspaces);
+  const entries = [...inputs.entries, ...declared.entries];
   const inventoryFiles = [...observation.files];
   const visible = new Set(inventoryFiles.map((file) => file.file));
   const readable = new Set(files);
@@ -28754,7 +28769,12 @@ async function withJavaScriptEntries(root, observation, inputs, files) {
     }
     return [{ ...entry, files: [...new Set([entry.file, ...members.map((file) => file.file)])] }];
   });
-  return createScanObservation({ ...observation, files: inventoryFiles, entryPoints });
+  return createScanObservation({
+    ...observation,
+    files: inventoryFiles,
+    entryPoints,
+    diagnostics: [...observation.diagnostics, ...declared.diagnostics]
+  });
 }
 
 // plugins/scanners/entry-points/source.ts
@@ -30039,17 +30059,16 @@ async function scanProject(root, directory, programs, inputs, readable) {
   const httpRequests = [];
   for (const { program, owned, sources } of programs) {
     const checker = program.getTypeChecker();
-    const mine = new Set(owned);
     const directives = sources.filter((source) => readable.has(relative(root, source.fileName))).flatMap((source) => source.statements.filter(ts6.isClassDeclaration).flatMap((node) => sourceDirective(node, checker) ?? []));
-    for (const component of directives.filter((directive) => directive.view && mine.has(directive.declaration.getSourceFile()))) {
+    for (const component of directives.filter((directive) => directive.view && owned.has(directive.declaration.getSourceFile()))) {
       templates.bindOutputs(component, directives, checker);
       sourceUnits.push(templates.unit(component));
     }
     httpRequests.push(...await angularHttpRequests(sources, checker, (node) => {
-      const caller = mine.has(node.getSourceFile()) ? enclosingOperation(node) : undefined;
+      const caller = owned.has(node.getSourceFile()) ? enclosingOperation(node) : undefined;
       return caller && evidence.operationId(caller);
     }));
-    files.push(...owned.map((source) => ({
+    files.push(...[...owned].map((source) => ({
       file: relative(root, source.fileName),
       symbols: source.statements.filter(ts6.isClassDeclaration).map((node) => ({
         id: `${relative(root, source.fileName)}#${node.getStart()}`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groma/scanner-angular",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Complementary Angular framework evidence with its own compatible TypeScript tooling.",
   "private": false,
   "type": "module",