@wbern/obscene 0.1.1 → 0.2.0

package/dist/cli.js

@@ -5,6 +5,7 @@ import { Command } from "commander";
 
 // src/analyze.ts
 import { execSync } from "child_process";
+import { readFileSync } from "fs";
 var DEFAULT_EXCLUDES = [
   /\.test\./,
   /\.spec\./,
@@ -64,31 +65,123 @@ function runScc(excludes = []) {
   }
   return files.sort((a, b) => b.complexity - a.complexity);
 }
+function gitFileCount(gitArgs, errorMessage) {
+  let raw;
+  try {
+    raw = execSync(gitArgs, {
+      maxBuffer: 50 * 1024 * 1024,
+      stdio: ["pipe", "pipe", "pipe"]
+    });
+  } catch {
+    throw new Error(errorMessage);
+  }
+  const counts = /* @__PURE__ */ new Map();
+  for (const line of raw.toString().split("\n")) {
+    const trimmed = normalizePath(line.trim());
+    if (!trimmed) continue;
+    counts.set(trimmed, (counts.get(trimmed) ?? 0) + 1);
+  }
+  return counts;
+}
 function getChurn(months) {
+  return gitFileCount(
+    `git log --since="${months} months ago" --format="" --name-only`,
+    "Not a git repository or git is not installed."
+  );
+}
+function getDefects(months) {
+  return gitFileCount(
+    `git log --since="${months} months ago" --grep="^fix" --format="" --name-only`,
+    "Not a git repository or git is not installed."
+  );
+}
+function getAuthors(months) {
   let raw;
   try {
     raw = execSync(
-      `git log --since="${months} months ago" --format="" --name-only`,
+      `git log --since="${months} months ago" --format="COMMIT_SEP%n%aN" --name-only`,
       { maxBuffer: 50 * 1024 * 1024, stdio: ["pipe", "pipe", "pipe"] }
     );
   } catch {
     throw new Error("Not a git repository or git is not installed.");
   }
+  const authorSets = /* @__PURE__ */ new Map();
+  const blocks = raw.toString().split("COMMIT_SEP\n");
+  for (const block of blocks) {
+    if (!block.trim()) continue;
+    const lines = block.split("\n");
+    const author = lines[0].trim();
+    if (!author) continue;
+    for (let i = 1; i < lines.length; i++) {
+      const file = normalizePath(lines[i].trim());
+      if (!file) continue;
+      let set = authorSets.get(file);
+      if (!set) {
+        set = /* @__PURE__ */ new Set();
+        authorSets.set(file, set);
+      }
+      set.add(author);
+    }
+  }
   const counts = /* @__PURE__ */ new Map();
-  for (const line of raw.toString().split("\n")) {
-    const trimmed = normalizePath(line.trim());
-    if (!trimmed) continue;
-    counts.set(trimmed, (counts.get(trimmed) ?? 0) + 1);
+  for (const [file, set] of authorSets) {
+    counts.set(file, set.size);
   }
   return counts;
 }
-function computeHotspots(files, churn) {
+function getNestingDepths(filePaths) {
+  const depths = /* @__PURE__ */ new Map();
+  for (const filePath of filePaths) {
+    let content;
+    try {
+      content = readFileSync(filePath, "utf-8");
+    } catch {
+      depths.set(filePath, 0);
+      continue;
+    }
+    let minSpaces = Number.POSITIVE_INFINITY;
+    const leadings = [];
+    for (const line of content.split("\n")) {
+      if (!line.trim()) continue;
+      const match = line.match(/^(\s+)/);
+      if (!match) continue;
+      const leading = match[1];
+      leadings.push(leading);
+      const spaceCount = (leading.match(/ /g) ?? []).length;
+      if (spaceCount > 0 && !leading.includes("	") && spaceCount < minSpaces) {
+        minSpaces = spaceCount;
+      }
+    }
+    const indentUnit = minSpaces === Number.POSITIVE_INFINITY ? 4 : minSpaces;
+    let maxDepth = 0;
+    for (const leading of leadings) {
+      let depth = 0;
+      for (const ch of leading) {
+        if (ch === "	") {
+          depth += 1;
+        } else if (ch === " ") {
+          depth += 1 / indentUnit;
+        }
+      }
+      depth = Math.floor(depth);
+      if (depth > maxDepth) maxDepth = depth;
+    }
+    depths.set(filePath, maxDepth);
+  }
+  return depths;
+}
+function computeHotspots(files, churn, defects = /* @__PURE__ */ new Map(), nestingDepths = /* @__PURE__ */ new Map(), authors = /* @__PURE__ */ new Map()) {
   const scored = files.map((f) => {
     const fileChurn = churn.get(f.file) ?? 0;
+    const fileDefects = defects.get(f.file) ?? 0;
     return {
       ...f,
       churn: fileChurn,
-      hotspotScore: f.complexity * fileChurn
+      hotspotScore: f.complexity * fileChurn,
+      defects: fileDefects,
+      defectDensity: f.code > 0 ? Math.round(fileDefects / f.code * 1e4) / 1e4 : 0,
+      maxNesting: nestingDepths.get(f.file) ?? 0,
+      authors: authors.get(f.file) ?? 0
     };
   }).filter((h) => h.hotspotScore > 0).sort((a, b) => b.hotspotScore - a.hotspotScore);
   const totalScore = scored.reduce((sum, h) => sum + h.hotspotScore, 0);
@@ -144,13 +237,13 @@ function formatHotspotsTable(output) {
   lines.push(`Showing: ${output.showing} of ${output.totalHotspots}`);
   lines.push("");
   lines.push(
-    padRight("File", 50) + padLeft("Score", 8) + padLeft("%", 7) + padLeft("Churn", 7) + padLeft("Cmplx", 7) + padLeft("Density", 9) + padLeft("Tier", 8)
+    padRight("File", 50) + padLeft("Score", 8) + padLeft("%", 7) + padLeft("Churn", 7) + padLeft("Cmplx", 7) + padLeft("Dens", 7) + padLeft("Dfcts", 6) + padLeft("Nest", 6) + padLeft("Auth", 6) + padLeft("Tier", 8)
   );
-  lines.push("\u2500".repeat(96));
+  lines.push("\u2500".repeat(112));
   for (const h of hotspots) {
     const tierLabel = h.tier === "danger" ? "DANGER" : h.tier === "watch" ? "WATCH" : "stable";
     lines.push(
-      padRight(truncate(h.file, 48), 50) + padLeft(h.hotspotScore.toLocaleString(), 8) + padLeft(h.percentOfTotal.toFixed(1), 7) + padLeft(String(h.churn), 7) + padLeft(String(h.complexity), 7) + padLeft(h.complexityDensity.toFixed(2), 9) + padLeft(tierLabel, 8)
+      padRight(truncate(h.file, 48), 50) + padLeft(h.hotspotScore.toLocaleString(), 8) + padLeft(h.percentOfTotal.toFixed(1), 7) + padLeft(String(h.churn), 7) + padLeft(String(h.complexity), 7) + padLeft(h.complexityDensity.toFixed(2), 7) + padLeft(String(h.defects), 6) + padLeft(String(h.maxNesting), 6) + padLeft(String(h.authors), 6) + padLeft(tierLabel, 8)
     );
   }
   return lines.join("\n");
@@ -167,7 +260,7 @@ function truncate(s, max) {
 
 // src/cli.ts
 var program = new Command();
-program.name("obscene").description("Identify hotspot files \u2014 complex code that changes frequently").version("0.1.1");
+program.name("obscene").description("Identify hotspot files \u2014 complex code that changes frequently").version("0.2.0");
 function addSharedOptions(cmd) {
   return cmd.option("--top <n>", "limit to top N entries (0 = all)", "20").option("--format <type>", "output format: json | table", "json").option(
     "--exclude <patterns...>",
@@ -227,7 +320,16 @@ function runHotspots(opts) {
   const months = parseInt(opts.months, 10);
   const files = runScc(opts.exclude);
   const churn = getChurn(months);
-  const hotspots = computeHotspots(files, churn);
+  const defects = getDefects(months);
+  const authors = getAuthors(months);
+  const nestingDepths = getNestingDepths(files.map((f) => f.file));
+  const hotspots = computeHotspots(
+    files,
+    churn,
+    defects,
+    nestingDepths,
+    authors
+  );
   const limited = top > 0 ? hotspots.slice(0, top) : hotspots;
   const tierCounts = { danger: 0, watch: 0, stable: 0 };
   for (const h of hotspots) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wbern/obscene",
-  "version": "0.1.1",
+  "version": "0.2.0",
   "description": "Identify hotspot files — complex code that changes frequently. Churn × complexity analysis for any git repo.",
   "type": "module",
   "bin": {