technical-debt-radar 1.0.6 → 1.0.8

package/dist/index.js

@@ -499,6 +499,7 @@ var require_parser = __commonJS({
       const standards = parseStandards(raw.standards);
       const scoring = parseScoring(raw.scoring);
       const shared_infrastructure = parseSharedInfrastructure(raw.shared_infrastructure);
+      const maintainability = parseMaintainability(raw.maintainability);
       const result = {
         mode,
         stack,
@@ -512,7 +513,8 @@ var require_parser = __commonJS({
         exceptions,
         standards,
         scoring,
-        shared_infrastructure
+        shared_infrastructure,
+        maintainability
       };
       if (architecture !== void 0) {
         result.architecture = architecture;
@@ -863,6 +865,34 @@ var require_parser = __commonJS({
         return item;
       });
     }
+    function parseMaintainability(value) {
+      if (value === void 0 || value === null)
+        return void 0;
+      if (typeof value !== "object") {
+        throw new PolicyValidationError("maintainability must be an object");
+      }
+      const m = value;
+      const result = {};
+      if (m.require_tests !== void 0) {
+        if (typeof m.require_tests !== "boolean") {
+          throw new PolicyValidationError("maintainability.require_tests must be a boolean");
+        }
+        result.require_tests = m.require_tests;
+      }
+      if (m.require_tests_for !== void 0) {
+        if (!Array.isArray(m.require_tests_for)) {
+          throw new PolicyValidationError("maintainability.require_tests_for must be an array of strings");
+        }
+        result.require_tests_for = m.require_tests_for.map(String);
+      }
+      if (m.ignore_tests_for !== void 0) {
+        if (!Array.isArray(m.ignore_tests_for)) {
+          throw new PolicyValidationError("maintainability.ignore_tests_for must be an array of strings");
+        }
+        result.ignore_tests_for = m.ignore_tests_for.map(String);
+      }
+      return result;
+    }
     function parseScoring(value) {
       if (value === void 0 || value === null)
         return {};
@@ -7511,6 +7541,15 @@ var require_validator = __commonJS({
         shared_infrastructure: {
           type: "array",
           items: { type: "string" }
+        },
+        maintainability: {
+          type: "object",
+          properties: {
+            require_tests: { type: "boolean" },
+            require_tests_for: { type: "array", items: { type: "string" } },
+            ignore_tests_for: { type: "array", items: { type: "string" } }
+          },
+          additionalProperties: false
         }
       },
       additionalProperties: false
@@ -9554,7 +9593,8 @@ var require_compiler = __commonJS({
         standards: config.standards,
         scoring,
         architecture: config.architecture,
-        sharedInfrastructure: config.shared_infrastructure ?? []
+        sharedInfrastructure: config.shared_infrastructure ?? [],
+        maintainability: config.maintainability
       };
     }
     function compileLayers(layers) {
@@ -11625,6 +11665,14 @@ var require_rules_generator = __commonJS({
         lines.push("  default: S");
       }
       lines.push("");
+      lines.push("maintainability:");
+      lines.push("  require_tests: true");
+      lines.push("  ignore_tests_for:");
+      lines.push("    - guard");
+      lines.push("    - interceptor");
+      lines.push("    - decorator");
+      lines.push("    - strategy");
+      lines.push("");
       return lines.join("\n");
     }
     function extractExceptions2(rulesYmlContent) {
@@ -17144,6 +17192,7 @@ var require_missing_tests_detector = __commonJS({
         "handler",
         "repository"
       ],
+      ignoreTestsFor: [],
       severity: "warning"
     };
     var EXCLUDED_PATTERNS = [
@@ -17222,7 +17271,17 @@ var require_missing_tests_detector = __commonJS({
         const basename2 = path9.basename(f.path);
         if (isExcludedFromTesting(basename2))
           return false;
-        return isTestableFile(basename2, cfg.requireTestsFor);
+        if (!isTestableFile(basename2, cfg.requireTestsFor))
+          return false;
+        if (cfg.ignoreTestsFor.length > 0) {
+          const fileType = extractFileType(f.path);
+          if (cfg.ignoreTestsFor.includes(fileType))
+            return false;
+          const normalizedPath = f.path.replace(/\\/g, "/");
+          if (cfg.ignoreTestsFor.some((pattern) => normalizedPath.includes(`/${pattern}/`) || normalizedPath.includes(`/${pattern}s/`)))
+            return false;
+        }
+        return true;
       });
       const filesWithoutTests = [];
       for (const file of testableFiles) {
@@ -17804,6 +17863,8 @@ 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))
+            continue;
           unusedExports.push({
             export: exp,
             file: filePath,
@@ -17985,6 +18046,30 @@ var require_dead_code_detector = __commonJS({
     function isTypeExport(type) {
       return type === "type" || type === "interface" || type === "enum";
     }
+    function isReferencedInOwnFile(sourceFile, exportName) {
+      const identifiers = sourceFile.getDescendantsOfKind(ts_morph_1.SyntaxKind.Identifier).filter((id) => id.getText() === exportName);
+      let declarationCount = 0;
+      let totalCount = 0;
+      for (const id of identifiers) {
+        totalCount++;
+        const parent = id.getParent();
+        if (!parent)
+          continue;
+        const parentKind = parent.getKind();
+        if (parentKind === ts_morph_1.SyntaxKind.ClassDeclaration || parentKind === ts_morph_1.SyntaxKind.FunctionDeclaration || parentKind === ts_morph_1.SyntaxKind.VariableDeclaration || parentKind === ts_morph_1.SyntaxKind.EnumDeclaration || parentKind === ts_morph_1.SyntaxKind.InterfaceDeclaration || parentKind === ts_morph_1.SyntaxKind.TypeAliasDeclaration) {
+          const nameNode = parent.getNameNode?.();
+          if (nameNode && nameNode.getPos() === id.getPos()) {
+            declarationCount++;
+            continue;
+          }
+        }
+        if (parentKind === ts_morph_1.SyntaxKind.ExportSpecifier) {
+          declarationCount++;
+          continue;
+        }
+      }
+      return totalCount > declarationCount;
+    }
     function parsePathAliases(input, projectRoot) {
       let tsconfigContent;
       const tsconfigFile = input.changedFiles.find((f) => f.status !== "deleted" && /tsconfig(?:\.build)?\.json$/.test(f.path));
@@ -18096,8 +18181,13 @@ var require_dead_code_detector = __commonJS({
         return resolveInProject(resolved, project) ?? resolved;
       }
       if (aliases.length > 0) {
-        return resolvePathAlias(specifier, aliases, project);
+        const aliasResolved = resolvePathAlias(specifier, aliases, project);
+        if (aliasResolved)
+          return aliasResolved;
       }
+      const directResolved = resolveInProject(specifier, project);
+      if (directResolved)
+        return directResolved;
       return void 0;
     }
     function resolveInProject(resolved, project) {
@@ -18543,7 +18633,7 @@ var require_orchestrator = __commonJS({
         (0, perf_pattern_detector_1.detectPerformanceRisks)(filteredInput, policy),
         (0, reliability_detector_1.detectReliabilityIssues)(filteredInput, policy),
         (0, duplication_detector_1.detectDuplication)(filteredInput),
-        projectRoot ? (0, missing_tests_detector_1.detectMissingTests)(filteredInput, projectRoot) : Promise.resolve(null),
+        projectRoot ? (0, missing_tests_detector_1.detectMissingTests)(filteredInput, projectRoot, buildMissingTestsConfig(policy)) : Promise.resolve(null),
         (0, dead_code_detector_1.detectDeadCode)(filteredInput, {}, projectRoot),
         projectRoot ? Promise.resolve((0, coverage_delta_detector_1.detectCoverageDelta)(projectRoot)) : Promise.resolve(null)
       ]);
@@ -18783,6 +18873,22 @@ var require_orchestrator = __commonJS({
         analysisTimeMs: Date.now() - startMs
       };
     }
+    function buildMissingTestsConfig(policy) {
+      const m = policy.maintainability;
+      if (!m)
+        return {};
+      const config = {};
+      if (m.require_tests === false) {
+        config.enabled = false;
+      }
+      if (m.require_tests_for && m.require_tests_for.length > 0) {
+        config.requireTestsFor = m.require_tests_for;
+      }
+      if (m.ignore_tests_for && m.ignore_tests_for.length > 0) {
+        config.ignoreTestsFor = m.ignore_tests_for;
+      }
+      return config;
+    }
     function deriveProjectRoot(input) {
       const paths = input.changedFiles.filter((f) => f.status !== "deleted" && path9.isAbsolute(f.path)).map((f) => f.path);
       if (paths.length === 0)

package/package.json

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