technical-debt-radar 1.7.1 → 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;

package/package.json

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