technical-debt-radar 1.6.1 → 1.6.3

package/dist/index.js

@@ -16219,7 +16219,23 @@ var require_reliability_detector = __commonJS({
         detectUnboundedParallelCalls(sourceFile, file.path, fns, policy, violations);
         project.removeSourceFile(sourceFile);
       }
-      return applyExceptions(violations, policy);
+      return applyExceptions(deduplicateOverlapping(violations), policy);
+    }
+    function deduplicateOverlapping(violations) {
+      const hasMissingTryCatch = /* @__PURE__ */ new Set();
+      for (const v of violations) {
+        if (v.ruleId === shared_1.RELIABILITY_RULES.MISSING_TRY_CATCH) {
+          hasMissingTryCatch.add(`${v.file}:${v.function ?? ""}`);
+        }
+      }
+      return violations.filter((v) => {
+        if (v.ruleId === shared_1.RELIABILITY_RULES.UNHANDLED_PROMISE_REJECTION) {
+          const key = `${v.file}:${v.function ?? ""}`;
+          if (hasMissingTryCatch.has(key))
+            return false;
+        }
+        return true;
+      });
     }
     function collectFunctions(sourceFile, filePath) {
       const scopes = [];
@@ -16597,39 +16613,39 @@ var require_reliability_detector = __commonJS({
           continue;
         if (isNestJSControllerMethod(fn.node, _sourceFile))
           continue;
-        let hasAwait = false;
-        fn.node.forEachDescendant((child) => {
-          if (ts_morph_1.Node.isAwaitExpression(child))
-            hasAwait = true;
-        });
-        if (!hasAwait)
-          continue;
         const body = fn.node.getBody();
         if (!body || !ts_morph_1.Node.isBlock(body))
           continue;
-        const hasTryAtTop = body.getStatements().some((s) => ts_morph_1.Node.isTryStatement(s));
-        if (hasTryAtTop)
-          continue;
-        let hasTryNested = false;
-        body.forEachDescendant((child) => {
-          if (hasTryNested)
-            return;
+        let unhandledAwaitCount = 0;
+        let totalAwaitCount = 0;
+        fn.node.forEachDescendant((child) => {
           if (child !== fn.node && (ts_morph_1.Node.isFunctionDeclaration(child) || ts_morph_1.Node.isMethodDeclaration(child) || ts_morph_1.Node.isArrowFunction(child) || ts_morph_1.Node.isFunctionExpression(child)))
             return;
-          if (ts_morph_1.Node.isTryStatement(child))
-            hasTryNested = true;
-        });
-        if (hasTryNested)
-          continue;
-        let awaitCount = 0;
-        fn.node.forEachDescendant((child) => {
-          if (ts_morph_1.Node.isFunctionDeclaration(child) || ts_morph_1.Node.isMethodDeclaration(child) || ts_morph_1.Node.isArrowFunction(child) || ts_morph_1.Node.isFunctionExpression(child)) {
-            if (child !== fn.node)
-              return;
+          if (!ts_morph_1.Node.isAwaitExpression(child))
+            return;
+          totalAwaitCount++;
+          let current = child.getParent();
+          let isHandled = false;
+          while (current && current !== fn.node) {
+            if (ts_morph_1.Node.isTryStatement(current)) {
+              const tryBlock = current.getTryBlock();
+              if (tryBlock && child.getPos() >= tryBlock.getPos() && child.getEnd() <= tryBlock.getEnd()) {
+                isHandled = true;
+              }
+              break;
+            }
+            if (ts_morph_1.Node.isFunctionDeclaration(current) || ts_morph_1.Node.isMethodDeclaration(current) || ts_morph_1.Node.isArrowFunction(current) || ts_morph_1.Node.isFunctionExpression(current))
+              break;
+            current = current.getParent();
           }
-          if (ts_morph_1.Node.isAwaitExpression(child))
-            awaitCount++;
+          if (!isHandled)
+            unhandledAwaitCount++;
         });
+        if (totalAwaitCount === 0)
+          continue;
+        if (unhandledAwaitCount === 0)
+          continue;
+        const awaitCount = unhandledAwaitCount;
         if (awaitCount < 2)
           continue;
         violations.push(makeViolation(shared_1.RELIABILITY_RULES.MISSING_TRY_CATCH, filePath, fn.node.getStartLineNumber(), `Async function '${fn.name}' has multiple awaits but no try/catch`, policy, fn.name, "Add try/catch around the async operations or use an error-handling wrapper"));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "technical-debt-radar",
-  "version": "1.6.1",
+  "version": "1.6.3",
   "description": "Stop Node.js production crashes before merge. 47 detection patterns across 5 categories.",
   "bin": {
     "radar": "dist/index.js",