technical-debt-radar 1.6.0 → 1.6.2

package/dist/index.js

@@ -16282,12 +16282,21 @@ var require_reliability_detector = __commonJS({
       }
       return false;
     }
+    function isNestJSInjectable(classNode) {
+      return classNode.getDecorators().some((d) => d.getName() === "Injectable");
+    }
     function isNestJSController(classNode) {
       return classNode.getDecorators().some((d) => {
         const name = d.getName();
         return name === "Controller" || name === "Resolver";
       });
     }
+    function isInsideNestJSInjectable(node) {
+      const classDecl = node.getFirstAncestorByKind(ts_morph_1.SyntaxKind.ClassDeclaration);
+      if (!classDecl)
+        return false;
+      return isNestJSInjectable(classDecl);
+    }
     function fileImportsNestJS(sourceFile) {
       for (const decl of sourceFile.getImportDeclarations()) {
         const spec = decl.getModuleSpecifierValue();
@@ -16398,6 +16407,8 @@ var require_reliability_detector = __commonJS({
           return;
         if (isNestJSControllerMethod(node, sourceFile))
           return;
+        if (fileImportsNestJS(sourceFile) && isInsideNestJSInjectable(node))
+          return;
         if (functionHasTryCatch(fn.node))
           return;
         const isRisky = isRiskyAwaitCall(node);
@@ -16586,39 +16597,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.0",
+  "version": "1.6.2",
   "description": "Stop Node.js production crashes before merge. 47 detection patterns across 5 categories.",
   "bin": {
     "radar": "dist/index.js",