fix(SourceFile): ignore try/finally statements for inTryStatement (#313)

Author: fraxken

Diff for: eslint.config.mjs

@@ -4,7 +4,8 @@ export default [
   {
     ignores: [
       "**/test/fixtures/**/*",
-      "**/test/probes/fixtures/**/*.js"
+      "**/test/probes/fixtures/**/*.js",
+      "**/examples/*.js"
     ]
   },
   ...ESLintConfig,

Diff for: src/SourceFile.js

@@ -129,7 +129,7 @@ export class SourceFile {
     this.deobfuscator.walk(node);
 
     // Detect TryStatement and CatchClause to known which dependency is required in a Try {} clause
-    if (node.type === "TryStatement" && typeof node.handler !== "undefined") {
+    if (node.type === "TryStatement" && node.handler) {
       this.inTryStatement = true;
     }
     else if (node.type === "CatchClause") {

Diff for: test/issues/312-try-finally.spec.js

@@ -0,0 +1,28 @@
+// Import Node.js Dependencies
+import { test } from "node:test";
+import assert from "node:assert";
+
+// Import Internal Dependencies
+import { AstAnalyser } from "../../index.js";
+
+/**
+ * @see https://github.com/NodeSecure/js-x-ray/issues/312
+ */
+test("SourceFile inTryStatement must ignore try/finally statements", () => {
+  const { dependencies } = new AstAnalyser().analyse(`
+    try {
+      // do something
+    }
+    finally {
+
+    }
+
+    var import_ts = __toESM(require("foobar"), 1);
+  `);
+  assert.strictEqual(dependencies.size, 1);
+  assert.ok(dependencies.has("foobar"));
+
+  const dependency = dependencies.get("foobar");
+  assert.strictEqual(dependency.unsafe, false);
+  assert.strictEqual(dependency.inTry, false);
+});