technical-debt-radar 1.7.0 → 1.8.0

package/dist/index.js

@@ -12889,6 +12889,7 @@ var require_boundary_checker = __commonJS({
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.checkBoundaries = checkBoundaries;
     exports2._resetPathAliasCache = _resetPathAliasCache;
+    exports2._resetModuleParsingCache = _resetModuleParsingCache;
     var shared_1 = require_dist();
     var minimatch_1 = require_commonjs3();
     var ts_morph_1 = require("ts-morph");
@@ -13222,6 +13223,12 @@ var require_boundary_checker = __commonJS({
             continue;
           if (typeOnly)
             continue;
+          if (isNestJS) {
+            const importedNames = getImportedNamesFromSpecifier(sourceFile, specifier);
+            const suppressed = importedNames.length > 0 && importedNames.every((name) => isLegitimateNestJSModuleImport(sourceModule, targetModule, name, input));
+            if (suppressed)
+              continue;
+          }
           violations.push({
             category: "architecture",
             type: "module-boundary-violation",
@@ -13461,6 +13468,117 @@ var require_boundary_checker = __commonJS({
     function _resetPathAliasCache() {
       _cachedAliases = void 0;
     }
+    var _cachedModuleParsing;
+    function parseNestJSModules(input) {
+      if (_cachedModuleParsing)
+        return _cachedModuleParsing;
+      const result = /* @__PURE__ */ new Map();
+      const project = new ts_morph_1.Project({ useInMemoryFileSystem: true });
+      for (const file of input.changedFiles) {
+        if (file.status === "deleted")
+          continue;
+        if (!/\.module\.(ts|js)$/.test(file.path))
+          continue;
+        const sf = project.createSourceFile(file.path, file.content);
+        for (const cls of sf.getClasses()) {
+          const moduleDecorator = cls.getDecorators().find((d) => d.getName() === "Module");
+          if (!moduleDecorator)
+            continue;
+          const className = cls.getName() ?? "";
+          const pathMatch = file.path.match(/src\/([^/]+)\//);
+          const moduleName = pathMatch ? pathMatch[1] : "";
+          if (!moduleName)
+            continue;
+          const callExpr = moduleDecorator.getCallExpression?.();
+          if (!callExpr)
+            continue;
+          const args = callExpr.getArguments();
+          if (args.length === 0 || !ts_morph_1.Node.isObjectLiteralExpression(args[0]))
+            continue;
+          const objLit = args[0];
+          const imports = [];
+          const exports3 = [];
+          const hasForwardRef = /* @__PURE__ */ new Map();
+          const importsProp = objLit.getProperty("imports");
+          if (importsProp && ts_morph_1.Node.isPropertyAssignment(importsProp)) {
+            const init = importsProp.getInitializer();
+            if (init && ts_morph_1.Node.isArrayLiteralExpression(init)) {
+              for (const el of init.getElements()) {
+                if (ts_morph_1.Node.isIdentifier(el)) {
+                  imports.push(el.getText());
+                  hasForwardRef.set(el.getText(), false);
+                } else if (ts_morph_1.Node.isCallExpression(el)) {
+                  const callText = el.getText();
+                  const fwdMatch = callText.match(/forwardRef\s*\(\s*\(\)\s*=>\s*(\w+)\s*\)/);
+                  if (fwdMatch) {
+                    imports.push(fwdMatch[1]);
+                    hasForwardRef.set(fwdMatch[1], true);
+                  } else {
+                    const expr = el.getExpression();
+                    if (ts_morph_1.Node.isPropertyAccessExpression(expr)) {
+                      const obj = expr.getExpression();
+                      if (ts_morph_1.Node.isIdentifier(obj)) {
+                        imports.push(obj.getText());
+                        hasForwardRef.set(obj.getText(), false);
+                      }
+                    }
+                  }
+                }
+              }
+            }
+          }
+          const exportsProp = objLit.getProperty("exports");
+          if (exportsProp && ts_morph_1.Node.isPropertyAssignment(exportsProp)) {
+            const init = exportsProp.getInitializer();
+            if (init && ts_morph_1.Node.isArrayLiteralExpression(init)) {
+              for (const el of init.getElements()) {
+                if (ts_morph_1.Node.isIdentifier(el)) {
+                  exports3.push(el.getText());
+                }
+              }
+            }
+          }
+          result.set(moduleName, { moduleName, className, imports, exports: exports3, hasForwardRef });
+        }
+        project.removeSourceFile(sf);
+      }
+      _cachedModuleParsing = result;
+      return result;
+    }
+    function getImportedNamesFromSpecifier(sourceFile, specifier) {
+      const names = [];
+      for (const decl of sourceFile.getImportDeclarations()) {
+        if (decl.getModuleSpecifierValue() !== specifier)
+          continue;
+        const defaultImport = decl.getDefaultImport();
+        if (defaultImport)
+          names.push(defaultImport.getText());
+        for (const named of decl.getNamedImports()) {
+          names.push(named.getName());
+        }
+      }
+      return names;
+    }
+    function isLegitimateNestJSModuleImport(sourceModuleName, targetModuleName, importedClassName, input) {
+      const modules = parseNestJSModules(input);
+      const importingMod = modules.get(sourceModuleName);
+      if (!importingMod)
+        return false;
+      const targetMod = modules.get(targetModuleName);
+      if (!targetMod)
+        return false;
+      const targetClassName = targetMod.className;
+      if (!importingMod.imports.includes(targetClassName))
+        return false;
+      if (importingMod.hasForwardRef.get(targetClassName))
+        return false;
+      if (!targetMod.exports.includes(importedClassName))
+        return false;
+      return true;
+    }
+    function _resetModuleParsingCache() {
+      _cachedModuleParsing = void 0;
+    }
     function normalizeForMatching(filePath) {
       let normalized = filePath.replace(/\\/g, "/");
       if (normalized.startsWith("/")) {
@@ -19209,7 +19327,10 @@ var require_dead_code_detector = __commonJS({
             for (const [otherFilePath, otherText] of allFileTexts) {
               if (otherFilePath === filePath)
                 continue;
-              if (callPattern.test(otherText)) {
+              if (!callPattern.test(otherText))
+                continue;
+              const importPattern = new RegExp(`import\\s+.*\\b${className}\\b.*from\\s+`);
+              if (importPattern.test(otherText)) {
                 hasExternalCall = true;
                 break;
               }
@@ -19650,7 +19771,7 @@ var require_orchestrator = __commonJS({
         (0, reliability_detector_1.detectReliabilityIssues)(filteredInput, policy),
         (0, duplication_detector_1.detectDuplication)(filteredInput),
         projectRoot ? (0, missing_tests_detector_1.detectMissingTests)(filteredInput, projectRoot, buildMissingTestsConfig(policy)) : Promise.resolve(null),
-        (0, dead_code_detector_1.detectDeadCode)(filteredInput, {}, projectRoot),
+        (0, dead_code_detector_1.detectDeadCode)(filteredInput, { excludeBarrels: false, excludeTypes: false }, projectRoot),
         projectRoot ? Promise.resolve((0, coverage_delta_detector_1.detectCoverageDelta)(projectRoot)) : Promise.resolve(null)
       ]);
       const runtimeViolations = runtimeResult.violations;
@@ -20355,19 +20476,35 @@ var RadarApiClient = class _RadarApiClient {
     return null;
   }
   async fetch(path9, init) {
-    const res = await fetch(`${this.apiUrl}${path9}`, {
-      ...init,
-      headers: {
-        "Authorization": `Bearer ${this.token}`,
-        "Content-Type": "application/json",
-        ...init?.headers
-      }
-    });
+    const url = `${this.apiUrl}${path9}`;
+    let res;
+    try {
+      res = await fetch(url, {
+        ...init,
+        headers: {
+          "Authorization": `Bearer ${this.token}`,
+          "Content-Type": "application/json",
+          ...init?.headers
+        }
+      });
+    } catch (err) {
+      const cause = err?.cause?.message || err?.cause?.code || "";
+      throw new Error(`Network error connecting to ${url}: ${err.message}${cause ? ` (${cause})` : ""}`);
+    }
     if (!res.ok) {
       const body = await res.text().catch(() => "");
-      throw new Error(`API error ${res.status}: ${body}`);
+      throw new Error(`API error ${res.status} on ${path9}: ${body}`);
+    }
+    const contentType = res.headers.get("content-type") ?? "";
+    const text = await res.text();
+    if (!text || text.trim().length === 0) {
+      return void 0;
+    }
+    try {
+      return JSON.parse(text);
+    } catch {
+      return void 0;
     }
-    return res.json();
   }
   async verifyToken() {
     return this.fetch("/cli/auth/verify");
@@ -20674,8 +20811,9 @@ async function scanCommand(targetPath, options) {
       }).catch(() => {
       });
       console.log(import_chalk.default.green("  \u2713 Results synced to dashboard: https://www.technicaldebtradar.com/dashboard"));
-    } catch {
-      console.log(import_chalk.default.yellow("  \u26A0\uFE0F  Could not sync results to dashboard. Check your connection."));
+    } catch (err) {
+      const detail = err?.message ? `: ${err.message}` : "";
+      console.log(import_chalk.default.yellow(`  \u26A0\uFE0F  Could not sync results to dashboard${detail}`));
     }
   }
   if (mode === "warn") {

package/package.json

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