technical-debt-radar 1.8.0 → 1.9.0

package/dist/index.js

@@ -13176,6 +13176,9 @@ var require_boundary_checker = __commonJS({
         const filePath = file.path;
         const sourceModule = findModuleForFile(filePath, policy);
         if (!sourceModule) {
+          if (isNestJSFramework(policy)) {
+            checkSharedFileImports(sourceFile, filePath, policy, input, violations);
+          }
           project.removeSourceFile(sourceFile);
           continue;
         }
@@ -13463,6 +13466,9 @@ var require_boundary_checker = __commonJS({
         return "src/" + specifier.slice(2);
       if (specifier.startsWith("~/"))
         return "src/" + specifier.slice(2);
+      if (specifier.startsWith("src/") || specifier.startsWith("lib/") || specifier.startsWith("app/")) {
+        return specifier;
+      }
       return void 0;
     }
     function _resetPathAliasCache() {
@@ -13579,6 +13585,53 @@ var require_boundary_checker = __commonJS({
     function _resetModuleParsingCache() {
       _cachedModuleParsing = void 0;
     }
+    function checkSharedFileImports(sourceFile, filePath, policy, input, violations) {
+      const basename2 = filePath.replace(/^.*\//, "");
+      if (/^(main|cli|bootstrap|server)\.(ts|js)$/.test(basename2))
+        return;
+      const normalizedPath = filePath.replace(/\\/g, "/");
+      const isSharedTypeFile = /\/(interfaces|types|shared|common|contracts)\//i.test(normalizedPath);
+      if (!isSharedTypeFile)
+        return;
+      for (const decl of sourceFile.getImportDeclarations()) {
+        const specifier = decl.getModuleSpecifierValue();
+        const line = decl.getStartLineNumber();
+        let resolvedTarget;
+        if (specifier.startsWith(".") || specifier.startsWith("/")) {
+          const sourceDir = filePath.replace(/\/[^/]+$/, "");
+          resolvedTarget = resolveRelativePath(sourceDir, specifier);
+        } else {
+          resolvedTarget = resolvePathAliasForBoundary(specifier, input);
+        }
+        if (!resolvedTarget)
+          continue;
+        const targetModule = findModuleForFile(resolvedTarget, policy);
+        if (!targetModule)
+          continue;
+        if (/\.module\.(ts|js)$/.test(resolvedTarget))
+          continue;
+        const targetBasename = resolvedTarget.replace(/^.*\//, "");
+        if (/\.(interface|dto|enum|types?)\.(ts|js)$/.test(targetBasename))
+          continue;
+        if (decl.isTypeOnly())
+          continue;
+        violations.push({
+          category: "architecture",
+          type: "module-boundary-violation",
+          ruleId: shared_1.ARCHITECTURE_RULES.MODULE_BOUNDARY,
+          severity: "critical",
+          source: "deterministic",
+          confidence: "high",
+          file: filePath,
+          line,
+          module: "shared",
+          message: `Shared file imports from "${targetModule}" module \u2014 shared files should not depend on feature modules (${filePath})`,
+          explanation: `Shared/interfaces files should contain plain types, not dependencies on feature module internals.`,
+          debtPoints: 5,
+          gateAction: "block"
+        });
+      }
+    }
     function normalizeForMatching(filePath) {
       let normalized = filePath.replace(/\\/g, "/");
       if (normalized.startsWith("/")) {
@@ -18864,7 +18917,7 @@ var require_dead_code_detector = __commonJS({
           const sf = sourceFiles.get(filePath);
           if (sf && exp.type === "class" && isNestJSDIRegistered(sf, exp.name))
             continue;
-          if (sf && isReferencedInOwnFile(sf, exp.name))
+          if (sf && !isTypeExport(exp.type) && isReferencedInOwnFile(sf, exp.name))
             continue;
           unusedExports.push({
             export: exp,
@@ -19221,6 +19274,8 @@ var require_dead_code_detector = __commonJS({
         for (const ext of extensions) {
           if (fp.endsWith(suffix + ext))
             return normalizeFilePath(fp);
+          if (fp.endsWith(suffix + "/index" + ext))
+            return normalizeFilePath(fp);
         }
       }
       return void 0;
@@ -19337,8 +19392,9 @@ var require_dead_code_detector = __commonJS({
             }
             if (!hasExternalCall) {
               const ownText = allFileTexts.get(filePath) ?? "";
-              const matches = ownText.match(new RegExp(`\\.${methodName}\\s*\\(`, "g"));
-              const internalCallCount = matches?.length ?? 0;
+              const selfCallPattern = new RegExp(`this\\.${methodName}\\s*\\(`, "g");
+              const selfCallMatches = ownText.match(selfCallPattern);
+              const internalCallCount = selfCallMatches?.length ?? 0;
               if (internalCallCount === 0) {
                 violations.push({
                   category: "maintainability",

package/package.json

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