technical-debt-radar 1.8.0 → 1.10.0

package/dist/index.js

@@ -231,11 +231,11 @@ var require_plans = __commonJS({
       [PlanId.FREE]: {
         id: PlanId.FREE,
         name: "Free",
-        description: "Unlimited scans, warn mode",
+        description: "5 scans/month, warn mode",
         priceMonthly: 0,
         priceAnnual: 0,
         priceMonthlyIfAnnual: 0,
-        limits: { maxRepos: 2, maxMembers: 1, aiCreditsPerMonth: 0, maxOrgs: 1, scansPerMonth: -1 },
+        limits: { maxRepos: 2, maxMembers: 1, aiCreditsPerMonth: 0, maxOrgs: 1, scansPerMonth: 5 },
         features: {
           ...ALL_CLI,
           cliFixAI: false,
@@ -270,7 +270,7 @@ var require_plans = __commonJS({
         priceMonthly: 15,
         priceAnnual: 144,
         priceMonthlyIfAnnual: 12,
-        limits: { maxRepos: 5, maxMembers: 1, aiCreditsPerMonth: 50, maxOrgs: 1, scansPerMonth: -1 },
+        limits: { maxRepos: 5, maxMembers: 1, aiCreditsPerMonth: 50, maxOrgs: 1, scansPerMonth: 200 },
         features: {
           ...ALL_CLI,
           cliFixAI: true,
@@ -305,7 +305,7 @@ var require_plans = __commonJS({
         priceMonthly: 49,
         priceAnnual: 468,
         priceMonthlyIfAnnual: 39,
-        limits: { maxRepos: -1, maxMembers: -1, aiCreditsPerMonth: 500, maxOrgs: 3, scansPerMonth: -1 },
+        limits: { maxRepos: -1, maxMembers: -1, aiCreditsPerMonth: 500, maxOrgs: 3, scansPerMonth: 1e3 },
         features: {
           ...ALL_CLI,
           cliFixAI: true,
@@ -340,7 +340,7 @@ var require_plans = __commonJS({
         priceMonthly: 99,
         priceAnnual: 948,
         priceMonthlyIfAnnual: 79,
-        limits: { maxRepos: -1, maxMembers: -1, aiCreditsPerMonth: 1800, maxOrgs: 10, scansPerMonth: -1 },
+        limits: { maxRepos: -1, maxMembers: -1, aiCreditsPerMonth: 1800, maxOrgs: 10, scansPerMonth: 5e3 },
         features: {
           ...ALL_FEATURES_TRUE,
           sso: false,
@@ -399,6 +399,145 @@ var require_plans = __commonJS({
   }
 });
 
+// ../../packages/shared/dist/graph-builder.js
+var require_graph_builder = __commonJS({
+  "../../packages/shared/dist/graph-builder.js"(exports2) {
+    "use strict";
+    Object.defineProperty(exports2, "__esModule", { value: true });
+    exports2.buildImportGraphData = buildImportGraphData2;
+    var LAYER_COLORS = {
+      api: "#4CAF50",
+      application: "#2196F3",
+      domain: "#FF9800",
+      infrastructure: "#F44336",
+      shared: "#9C27B0",
+      common: "#9C27B0",
+      presentation: "#4CAF50",
+      business: "#2196F3",
+      "data-access": "#F44336",
+      models: "#FF9800",
+      controllers: "#4CAF50",
+      views: "#9E9E9E",
+      ports: "#2196F3",
+      adapters: "#F44336",
+      handlers: "#4CAF50",
+      commands: "#2196F3",
+      queries: "#FF9800",
+      events: "#9C27B0",
+      config: "#607D8B"
+    };
+    function buildImportGraphData2(edges, violations, files, modules) {
+      const nodeMap = /* @__PURE__ */ new Map();
+      const violationsByFile = /* @__PURE__ */ new Map();
+      for (const v of violations) {
+        violationsByFile.set(v.file, (violationsByFile.get(v.file) ?? 0) + 1);
+      }
+      const fileLoc = /* @__PURE__ */ new Map();
+      for (const f of files) {
+        if (f.linesOfCode) {
+          fileLoc.set(f.path, f.linesOfCode);
+        }
+      }
+      const ensureNode = (filePath, layer, module3) => {
+        if (nodeMap.has(filePath))
+          return;
+        const parts = filePath.split("/");
+        const fileName = parts[parts.length - 1] ?? filePath;
+        const label = fileName.replace(/\.(ts|tsx|js|jsx)$/, "");
+        const ext = fileName.match(/\.(service|controller|module|entity|repository|guard|pipe|middleware|dto|resolver|gateway)/);
+        nodeMap.set(filePath, {
+          id: filePath,
+          label,
+          module: module3 ?? void 0,
+          layer: layer ?? void 0,
+          type: ext ? ext[1] : "file",
+          violations: violationsByFile.get(filePath) ?? 0,
+          complexity: 0,
+          linesOfCode: fileLoc.get(filePath) ?? 0,
+          isHotspot: (violationsByFile.get(filePath) ?? 0) >= 3
+        });
+      };
+      for (const edge of edges) {
+        ensureNode(edge.source, edge.sourceLayer, edge.sourceModule);
+        ensureNode(edge.target, edge.targetLayer, edge.targetModule);
+      }
+      const circularFiles = /* @__PURE__ */ new Set();
+      for (const v of violations) {
+        if (v.type === "circular_dependency") {
+          circularFiles.add(v.file);
+        }
+      }
+      const graphEdges = edges.map((e) => {
+        const isCircular = circularFiles.has(e.source) && circularFiles.has(e.target);
+        const matchingViolation = violations.find((v) => v.category === "architecture" && v.file === e.source && v.message.includes(e.target.split("/").pop() ?? ""));
+        const isViolation = !!matchingViolation || isCircular;
+        let type = "import";
+        let violationType = null;
+        let message = null;
+        if (isCircular) {
+          type = "circular";
+          violationType = "circular-dependency";
+          message = `Circular dependency involving ${e.source} and ${e.target}`;
+        } else if (matchingViolation) {
+          type = "violation";
+          violationType = matchingViolation.type;
+          message = matchingViolation.message;
+        }
+        return { source: e.source, target: e.target, type, isViolation, violationType, message };
+      });
+      const moduleMap = /* @__PURE__ */ new Map();
+      for (const node of nodeMap.values()) {
+        if (node.module) {
+          const existing = moduleMap.get(node.module);
+          if (existing) {
+            existing.nodeCount++;
+            existing.violations += node.violations;
+          } else {
+            const policyMod = modules?.find((m) => m.name === node.module);
+            moduleMap.set(node.module, {
+              nodeCount: 1,
+              violations: node.violations,
+              path: policyMod?.pathPattern ?? ""
+            });
+          }
+        }
+      }
+      const graphModules = Array.from(moduleMap.entries()).map(([name, data]) => ({
+        name,
+        path: data.path,
+        nodeCount: data.nodeCount,
+        violations: data.violations
+      }));
+      const layerSet = /* @__PURE__ */ new Set();
+      for (const node of nodeMap.values()) {
+        if (node.layer)
+          layerSet.add(node.layer);
+      }
+      const layers = Array.from(layerSet).map((name) => ({
+        name,
+        color: LAYER_COLORS[name] ?? "#9E9E9E"
+      }));
+      const nodes = Array.from(nodeMap.values());
+      const violationEdgeCount = graphEdges.filter((e) => e.isViolation && e.type !== "circular").length;
+      const circularEdgeCount = graphEdges.filter((e) => e.type === "circular").length;
+      return {
+        nodes,
+        edges: graphEdges,
+        modules: graphModules,
+        layers,
+        stats: {
+          totalNodes: nodes.length,
+          totalEdges: graphEdges.length,
+          violationEdges: violationEdgeCount,
+          circularEdges: circularEdgeCount,
+          modules: graphModules.length,
+          layers: layers.length
+        }
+      };
+    }
+  }
+});
+
 // ../../packages/shared/dist/index.js
 var require_dist = __commonJS({
   "../../packages/shared/dist/index.js"(exports2) {
@@ -420,9 +559,14 @@ var require_dist = __commonJS({
       for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports3, p)) __createBinding(exports3, m, p);
     };
     Object.defineProperty(exports2, "__esModule", { value: true });
+    exports2.buildImportGraphData = void 0;
     __exportStar(require_types(), exports2);
     __exportStar(require_constants(), exports2);
     __exportStar(require_plans(), exports2);
+    var graph_builder_1 = require_graph_builder();
+    Object.defineProperty(exports2, "buildImportGraphData", { enumerable: true, get: function() {
+      return graph_builder_1.buildImportGraphData;
+    } });
   }
 });
 
@@ -13176,6 +13320,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 +13610,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 +13729,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 +19061,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 +19418,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 +19536,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",
@@ -20205,6 +20405,7 @@ var os2 = __toESM(require("os"));
 var import_chalk = __toESM(require("chalk"));
 var import_policy_engine = __toESM(require_dist2());
 var import_analyzers = __toESM(require_dist3());
+var import_shared = __toESM(require_dist());
 var import_analyzers2 = __toESM(require_dist3());
 
 // src/ai/scan-summary-generator.ts
@@ -20763,6 +20964,12 @@ async function scanCommand(targetPath, options) {
     const blocking = result.violations.filter((v) => v.gateAction === "block");
     const warns = result.violations.filter((v) => v.gateAction === "warn");
     const byCat = countByCategory(result.violations);
+    const graphData = result.importGraph && result.importGraph.length > 0 ? (0, import_shared.buildImportGraphData)(
+      result.importGraph,
+      result.violations,
+      changedFiles.map((f) => ({ path: f.path, linesOfCode: f.content?.split("\n").length })),
+      policy.modules
+    ) : void 0;
     const reportData = {
       repoName: path2.basename(path2.resolve(targetPath)),
       violations: result.violations.map((v) => ({
@@ -20786,6 +20993,7 @@ async function scanCommand(targetPath, options) {
       duration: Date.now() - scanStart,
       usedAi: shouldRunAi,
       aiCreditsUsed: 0,
+      importGraphData: graphData,
       // PR metadata from CI environment (set by GitHub Action or GitLab CI)
       ...process.env.RADAR_PR_NUMBER ? {
         prNumber: parseInt(process.env.RADAR_PR_NUMBER, 10),

package/package.json

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