fix: implement isESMExport probe to consider export stmts as dependen…

…cies

src/ProbeRunner.js

@@ -11,6 +11,7 @@ import isImportDeclaration from "./probes/isImportDeclaration.js";
 import isWeakCrypto from "./probes/isWeakCrypto.js";
 import isBinaryExpression from "./probes/isBinaryExpression.js";
 import isArrayExpression from "./probes/isArrayExpression.js";
+import isESMExport from "./probes/isESMExport.js";
 
 // Import Internal Dependencies
 import { SourceFile } from "./SourceFile.js";
@@ -39,6 +40,7 @@ export class ProbeRunner {
    */
   static Defaults = [
     isRequire,
+    isESMExport,
     isUnsafeCallee,
     isLiteral,
     isLiteralRegex,

src/probes/isESMExport.js

@@ -0,0 +1,31 @@
+/**
+ * @description Search for ESM Export
+ *
+ * @example
+ * export { bar } from "./foo.js";
+ * export * from "./bar.js";
+ */
+function validateNode(node) {
+  return [
+    /**
+     * We must be sure that the source property is a Literal to not fall in a trap
+     * export const foo = "bar";
+     */
+    (node.type === "ExportNamedDeclaration" && node.source?.type === "Literal") ||
+    node.type === "ExportAllDeclaration"
+  ];
+}
+
+function main(node, { sourceFile }) {
+  sourceFile.addDependency(
+    node.source.value,
+    node.loc
+  );
+}
+
+export default {
+  name: "isESMExport",
+  validateNode,
+  main,
+  breakOnMatch: true
+};

test/EntryFilesAnalyser.spec.js

@@ -41,6 +41,30 @@ describe("EntryFilesAnalyser", () => {
     assert.strictEqual(calls.length, 6);
   });
 
+  it("should analyze ESM export statements recursively", async(t) => {
+    const entryFilesAnalyser = new EntryFilesAnalyser();
+    const entryUrl = new URL("export.js", FIXTURE_URL);
+
+    t.mock.method(AstAnalyser.prototype, "analyseFile");
+
+    const generator = entryFilesAnalyser.analyse([
+      entryUrl
+    ]);
+    const reports = await fromAsync(generator);
+
+    assert.deepEqual(
+      reports.map((report) => report.file),
+      [
+        entryUrl,
+        new URL("shared.js", FIXTURE_URL)
+      ].map((url) => fileURLToPath(url))
+    );
+
+    // Check that shared dependencies are not analyzed several times
+    const calls = AstAnalyser.prototype.analyseFile.mock.calls;
+    assert.strictEqual(calls.length, 2);
+  });
+
   it("should detect internal deps that failed to be analyzed", async() => {
     const entryFilesAnalyser = new EntryFilesAnalyser();
     const entryUrl = new URL("entryWithInvalidDep.js", FIXTURE_URL);

test/fixtures/entryFiles/export.js

@@ -0,0 +1 @@
+export * from "./shared.js";

test/probes/isESMExport.spec.js

@@ -0,0 +1,34 @@
+// Import Node.js Dependencies
+import { describe, it } from "node:test";
+import assert from "node:assert";
+
+// Import Internal Dependencies
+import { AstAnalyser } from "../../index.js";
+
+describe("probe: isESMExport", () => {
+  it("should detect ExportNamedDeclaration statement with a Literal source as dependency", () => {
+    const code = `
+      export { foo } from "./bar.js";
+      export const bar = "foo";
+    `;
+    const { dependencies } = new AstAnalyser().analyse(code);
+
+    assert.deepEqual(
+      [...dependencies.keys()],
+      ["./bar.js"]
+    );
+  });
+
+  it("should detect ExportAllDeclaration statement as dependency", () => {
+    const code = `
+      export * from "./bar.js";
+    `;
+    const { dependencies } = new AstAnalyser().analyse(code);
+
+    assert.deepEqual(
+      [...dependencies.keys()],
+      ["./bar.js"]
+    );
+  });
+});
+