@getcodesentinel/codesentinel 1.19.1 → 1.20.0

package/README.md

@@ -233,8 +233,9 @@ Notes:
 
 - `likely_merge` (default) may merge multiple emails that likely belong to the same person based on repository history.
 - `strict_email` treats each canonical email as a distinct author, which avoids false merges but can split the same person across multiple emails.
-- Git mailmap is enabled (`git log --use-mailmap`). Put `.mailmap` in the repository being analyzed (the `codesentinel analyze [path]` target). Git will then deterministically unify known aliases before CodeSentinel computes `authorDistribution`.
-- `authorDistribution` returns whichever identity mode is selected.
+- Git mailmap is enabled (`git log --use-mailmap`). Put `.mailmap` in the repository being analyzed (the `codesentinel analyze [path]` target). Git will then deterministically unify known aliases before CodeSentinel computes ownership distributions.
+- Evolution output includes both `authorDistributionByCommits` and `authorDistributionByChurn`, plus their derived `topAuthorShare...` and `busFactor...` fields.
+- Current scoring continues to use the commit-weighted ownership view (`...ByCommits`).
 - Logs are emitted to `stderr` and JSON output is written to `stdout`, so CI redirection still works.
 - You can set a default log level with `CODESENTINEL_LOG_LEVEL` (`silent|error|warn|info|debug`).
 - At `info`/`debug`, structural, evolution, and dependency stages report progress so long analyses are observable.

package/dist/index.js

@@ -4683,6 +4683,26 @@ var finalizeAuthorDistribution = (authorCommits) => {
     share: round44(commits / totalCommits)
   })).sort((a, b) => b.commits - a.commits || a.authorId.localeCompare(b.authorId));
 };
+var finalizeAuthorChurnDistribution = (authorChurn) => {
+  const entries = [...authorChurn.entries()].map(([authorId, churn]) => {
+    const churnAdded = churn.churnAdded;
+    const churnDeleted = churn.churnDeleted;
+    return {
+      authorId,
+      churnAdded,
+      churnDeleted,
+      churnTotal: churnAdded + churnDeleted
+    };
+  });
+  const totalChurn = entries.reduce((sum, entry) => sum + entry.churnTotal, 0);
+  if (totalChurn === 0) {
+    return [];
+  }
+  return entries.map((entry) => ({
+    ...entry,
+    share: round44(entry.churnTotal / totalChurn)
+  })).sort((a, b) => b.churnTotal - a.churnTotal || a.authorId.localeCompare(b.authorId));
+};
 var buildCouplingMatrix = (coChangeByPair, fileCommitCount, consideredCommits, skippedLargeCommits, maxCouplingPairs) => {
   const allPairs = [];
   for (const [key, coChangeCommits] of coChangeByPair.entries()) {
@@ -4751,10 +4771,19 @@ var computeRepositoryEvolutionSummary = (targetPath, commits, config) => {
         recentCommitCount: 0,
         churnAdded: 0,
         churnDeleted: 0,
-        authors: /* @__PURE__ */ new Map()
+        authorsByCommits: /* @__PURE__ */ new Map(),
+        authorsByChurn: /* @__PURE__ */ new Map()
       };
       current.churnAdded += fileChange.additions;
       current.churnDeleted += fileChange.deletions;
+      const effectiveAuthorId = authorAliasById.get(commit.authorId) ?? commit.authorId;
+      const authorChurn = current.authorsByChurn.get(effectiveAuthorId) ?? {
+        churnAdded: 0,
+        churnDeleted: 0
+      };
+      authorChurn.churnAdded += fileChange.additions;
+      authorChurn.churnDeleted += fileChange.deletions;
+      current.authorsByChurn.set(effectiveAuthorId, authorChurn);
       fileStats.set(fileChange.filePath, current);
     }
     for (const filePath of uniqueFiles) {
@@ -4767,7 +4796,10 @@ var computeRepositoryEvolutionSummary = (targetPath, commits, config) => {
         current.recentCommitCount += 1;
       }
       const effectiveAuthorId = authorAliasById.get(commit.authorId) ?? commit.authorId;
-      current.authors.set(effectiveAuthorId, (current.authors.get(effectiveAuthorId) ?? 0) + 1);
+      current.authorsByCommits.set(
+        effectiveAuthorId,
+        (current.authorsByCommits.get(effectiveAuthorId) ?? 0) + 1
+      );
     }
     const orderedFiles = [...uniqueFiles].sort((a, b) => a.localeCompare(b));
     if (orderedFiles.length > 1) {
@@ -4790,8 +4822,10 @@ var computeRepositoryEvolutionSummary = (targetPath, commits, config) => {
     }
   }
   const files = [...fileStats.entries()].map(([filePath, stats]) => {
-    const authorDistribution = finalizeAuthorDistribution(stats.authors);
-    const topAuthorShare = authorDistribution[0]?.share ?? 0;
+    const authorDistributionByCommits = finalizeAuthorDistribution(stats.authorsByCommits);
+    const authorDistributionByChurn = finalizeAuthorChurnDistribution(stats.authorsByChurn);
+    const topAuthorShareByCommits = authorDistributionByCommits[0]?.share ?? 0;
+    const topAuthorShareByChurn = authorDistributionByChurn[0]?.share ?? 0;
     return {
       filePath,
       commitCount: stats.commitCount,
@@ -4801,9 +4835,18 @@ var computeRepositoryEvolutionSummary = (targetPath, commits, config) => {
       churnTotal: stats.churnAdded + stats.churnDeleted,
       recentCommitCount: stats.recentCommitCount,
       recentVolatility: stats.commitCount === 0 ? 0 : round44(stats.recentCommitCount / stats.commitCount),
-      topAuthorShare,
-      busFactor: computeBusFactor(authorDistribution, config.busFactorCoverageThreshold),
-      authorDistribution
+      topAuthorShareByCommits,
+      busFactorByCommits: computeBusFactor(
+        authorDistributionByCommits,
+        config.busFactorCoverageThreshold
+      ),
+      authorDistributionByCommits,
+      topAuthorShareByChurn,
+      busFactorByChurn: computeBusFactor(
+        authorDistributionByChurn,
+        config.busFactorCoverageThreshold
+      ),
+      authorDistributionByChurn
     };
   }).sort((a, b) => a.filePath.localeCompare(b.filePath));
   const fileCommitCount = new Map(files.map((file) => [file.filePath, file.commitCount]));
@@ -5558,15 +5601,15 @@ var computeRepositoryHealthSummary = (input) => {
     let singleContributorFiles = 0;
     let trackedFiles = 0;
     for (const file of evolutionSourceFiles) {
-      if (file.commitCount <= 0 || file.authorDistribution.length === 0) {
+      if (file.commitCount <= 0 || file.authorDistributionByCommits.length === 0) {
         continue;
       }
       trackedFiles += 1;
-      const dominantShare = clamp01(file.authorDistribution[0]?.share ?? 0);
-      if (file.authorDistribution.length === 1 || dominantShare >= 0.9) {
+      const dominantShare = clamp01(file.authorDistributionByCommits[0]?.share ?? 0);
+      if (file.authorDistributionByCommits.length === 1 || dominantShare >= 0.9) {
         singleContributorFiles += 1;
       }
-      for (const author of file.authorDistribution) {
+      for (const author of file.authorDistributionByCommits) {
         const commits = Math.max(0, author.commits);
         if (commits <= 0) {
           continue;
@@ -6329,7 +6372,7 @@ var computeEvolutionScales = (evolutionByFile, config) => {
       config.quantileClamp.upper
     ),
     busFactor: buildQuantileScale(
-      evolutionFiles.map((metrics) => metrics.busFactor),
+      evolutionFiles.map((metrics) => metrics.busFactorByCommits),
       config.quantileClamp.lower,
       config.quantileClamp.upper
     )
@@ -6518,9 +6561,9 @@ var computeRiskSummary = (structural, evolution, external, config, traceCollecto
         evolutionScales.churnTotal
       );
       volatilityRisk = toUnitInterval(evolutionMetrics.recentVolatility);
-      ownershipConcentrationRisk = toUnitInterval(evolutionMetrics.topAuthorShare);
+      ownershipConcentrationRisk = toUnitInterval(evolutionMetrics.topAuthorShareByCommits);
       busFactorRisk = toUnitInterval(
-        1 - normalizeWithScale(evolutionMetrics.busFactor, evolutionScales.busFactor)
+        1 - normalizeWithScale(evolutionMetrics.busFactorByCommits, evolutionScales.busFactor)
       );
       const evolutionWeights = config.evolutionFactorWeights;
       evolutionFactor = toUnitInterval(
@@ -6577,8 +6620,8 @@ var computeRiskSummary = (structural, evolution, external, config, traceCollecto
         commitCount: evolutionMetrics?.commitCount ?? null,
         churnTotal: evolutionMetrics?.churnTotal ?? null,
         recentVolatility: evolutionMetrics?.recentVolatility ?? null,
-        topAuthorShare: evolutionMetrics?.topAuthorShare ?? null,
-        busFactor: evolutionMetrics?.busFactor ?? null,
+        topAuthorShareByCommits: evolutionMetrics?.topAuthorShareByCommits ?? null,
+        busFactorByCommits: evolutionMetrics?.busFactorByCommits ?? null,
         dependencyAffinity: round46(dependencyAffinity),
         repositoryExternalPressure: round46(dependencyComputation.repositoryExternalPressure),
         structuralAttenuation: round46(structuralAttenuation)
@@ -6646,8 +6689,8 @@ var computeRiskSummary = (structural, evolution, external, config, traceCollecto
             commitCount: context.rawMetrics.commitCount,
             churnTotal: context.rawMetrics.churnTotal,
             recentVolatility: context.rawMetrics.recentVolatility,
-            topAuthorShare: context.rawMetrics.topAuthorShare,
-            busFactor: context.rawMetrics.busFactor
+            topAuthorShareByCommits: context.rawMetrics.topAuthorShareByCommits,
+            busFactorByCommits: context.rawMetrics.busFactorByCommits
           },
           normalizedMetrics: {
             frequencyRisk: context.normalizedMetrics.frequencyRisk,