Merge branch 'staging' of https://github.com/HUMBLEF0OL/grotto

Author: HUMBLEF0OL

.changeset/petite-tires-relax.md

@@ -0,0 +1,7 @@
+---
+"@git-compass/core": minor
+"git-compass": minor
+"@git-compass/web": minor
+---
+
+Metric fix for web

packages/cli/src/formatters/console.ts

@@ -168,19 +168,25 @@ export function printConsoleReport(
     if (contributors.length > 0) {
       console.log(chalk.green.bold("\nTop Contributors"));
       const contribData = [
-        [chalk.bold("Author"), chalk.bold("Commits"), chalk.bold("Activity"), chalk.bold("Status")],
+        [
+          chalk.bold("Author"),
+          chalk.bold("Commits"),
+          chalk.bold("Impact"),
+          chalk.bold("Stability"),
+          chalk.bold("Status")
+        ],
       ];
 
       contributors.slice(0, limit).forEach((c) => {
         const burnoutInfo = burnout.contributors.find((b) => b.email === c.email);
-        const isBurnedOut =
-          burnoutInfo && (burnoutInfo.riskLevel === 'high');
+        const isBurnedOut = burnoutInfo && burnoutInfo.riskLevel === "high";
 
         contribData.push([
           chalk.white(c.name),
           chalk.green(c.commitCount.toString()),
-          chalk.cyan(`${c.activeDays} days`),
-          isBurnedOut ? chalk.bgRed.white.bold(" BURNOUT ") : chalk.bgGreen.black.bold(" STABLE "),
+          chalk.yellow(`+${c.insertions || 0}`),
+          chalk.magenta(`${c.stability || 0}%`),
+          isBurnedOut ? chalk.bgRed.white.bold(" BURNOUT ") : chalk.bgGreen.black.bold(" ACTIVE "),
         ]);
       });
       console.log(table(contribData, { border: getBorderCharacters("ramac") }));

packages/core/inspect-diff.ts

@@ -0,0 +1,11 @@
+import { createGitParser } from './src/parser/git-parser.js';
+
+async function inspect() {
+    const git = createGitParser(process.cwd());
+    const log = await git.log(['-n', '1', '--stat']);
+    const commit = log.all[0];
+    console.log('Commit hash:', commit.hash);
+    console.log('Diff info:', JSON.stringify(commit.diff, null, 2));
+}
+
+inspect().catch(console.error);

packages/core/src/analyzers/contributors.ts

@@ -14,6 +14,9 @@ export function analyzeContributors(commits: GitCommit[]): ContributorReport {
       email: string;
       commitCount: number;
       files: Set<string>;
+      insertions: number;
+      deletions: number;
+      totalFilesAcrossCommits: number;
       first: string;
       last: string;
       days: Set<string>;
@@ -26,12 +29,19 @@ export function analyzeContributors(commits: GitCommit[]): ContributorReport {
       email: commit.author.email,
       commitCount: 0,
       files: new Set<string>(),
+      insertions: 0,
+      deletions: 0,
+      totalFilesAcrossCommits: 0,
       first: commit.date,
       last: commit.date,
       days: new Set<string>(),
     };
 
     existing.commitCount++;
+    existing.insertions += commit.insertions || 0;
+    existing.deletions += commit.deletions || 0;
+    existing.totalFilesAcrossCommits += commit.files.length;
+
     if (new Date(commit.date) < new Date(existing.first)) existing.first = commit.date;
     if (new Date(commit.date) > new Date(existing.last)) existing.last = commit.date;
 
@@ -41,15 +51,24 @@ export function analyzeContributors(commits: GitCommit[]): ContributorReport {
     contributorMap.set(commit.author.email, existing);
   }
 
-  const contributors: ContributorDetail[] = Array.from(contributorMap.values()).map((data) => ({
-    name: data.name,
-    email: data.email,
-    commitCount: data.commitCount,
-    filesChanged: data.files.size,
-    firstCommit: data.first,
-    lastCommit: data.last,
-    activeDays: data.days.size,
-  }));
+  const contributors: ContributorDetail[] = Array.from(contributorMap.values()).map((data) => {
+    // Stability per author: based on commit atomicity
+    const avgFilesPerCommit = data.commitCount > 0 ? (data.totalFilesAcrossCommits / data.commitCount) : 0;
+    const stability = Math.max(20, Math.min(100, Math.round(100 - (avgFilesPerCommit * 5))));
+
+    return {
+      name: data.name,
+      email: data.email,
+      commitCount: data.commitCount,
+      filesChanged: data.files.size,
+      firstCommit: data.first,
+      lastCommit: data.last,
+      activeDays: data.days.size,
+      insertions: data.insertions,
+      deletions: data.deletions,
+      stability
+    };
+  });
 
   return {
     contributors,

packages/core/src/parser/git-parser.ts

@@ -105,11 +105,17 @@ export async function getCommitsSince(
        args.push(since);
        args.push("HEAD");
     } else {
-      const days = parseInt(since, 10);
-      if (isPureNumber && !isNaN(days)) {
+      const daysStr = since.toLowerCase();
+      if (daysStr.endsWith("d")) {
+        const days = daysStr.slice(0, -1);
         args.push(`--since=${days} days ago`);
       } else {
-        args.push(`--since=${since}`);
+        const days = parseInt(since, 10);
+        if (isPureNumber && !isNaN(days)) {
+          args.push(`--since=${days} days ago`);
+        } else {
+          args.push(`--since=${since}`);
+        }
       }
     }
   }
@@ -120,17 +126,37 @@ export async function getCommitsSince(
 
   const logResult = await git.log(args);
 
+  // Fetch parents separately to avoid breaking simple-git's diff parser
+  const parentArgs = args.filter(arg => !arg.startsWith('--stat'));
+  const parentsRaw = await git.raw(['log', ...parentArgs, '--format=%H %P']);
+  const parentMap = new Map<string, string[]>();
+  
+  parentsRaw.split('\n').forEach(line => {
+    const parts = line.trim().split(' ');
+    const hash = parts[0];
+    if (hash) {
+      parentMap.set(hash, parts.slice(1));
+    }
+  });
+  
   return logResult.all.map((commit: any) => {
+    const hash = commit.hash;
+    const parents = parentMap.get(hash) || [];
+    // Combine subject and body for full message analysis
+    const fullMessage = commit.body ? `${commit.message}\n\n${commit.body}` : commit.message;
+
     return {
-      hash: commit.hash,
-      parents: commit.parents || [],
+      hash,
+      parents,
       author: {
         name: commit.author_name || 'Unknown',
         email: commit.author_email || 'unknown@example.com'
       },
       date: commit.date,
-      message: commit.message,
-      files: commit.diff ? commit.diff.files.map((f: any) => f.file).filter(Boolean) : []
+      message: fullMessage,
+      files: commit.diff ? commit.diff.files.map((f: any) => f.file).filter(Boolean) : [],
+      insertions: commit.diff?.insertions || 0,
+      deletions: commit.diff?.deletions || 0
     } as GitCommit;
   });
 }

packages/core/src/types/extended.ts

@@ -123,6 +123,9 @@ export interface ContributorDetail {
   readonly firstCommit: ISODateString;
   readonly lastCommit: ISODateString;
   readonly activeDays: number;
+  readonly insertions: number;
+  readonly deletions: number;
+  readonly stability: number; // 0–100 score
 }
 
 export interface ContributorReport {

packages/core/src/types/signal.ts

@@ -112,4 +112,6 @@ export interface GitCommit {
   readonly date: string;
   readonly parents: string[];
   readonly files: string[];
+  readonly insertions: number;
+  readonly deletions: number;
 }

packages/core/verify-data.ts

@@ -0,0 +1,15 @@
+import { createGitParser, getCommitsSince, analyzeContributors } from './src/index.js';
+
+async function test() {
+    const git = createGitParser(process.cwd() + '/../..');
+    const commits = await getCommitsSince(git, '30d', { branch: 'dev' });
+    console.log(`Found ${commits.length} commits`);
+    
+    if (commits.length > 0) {
+        console.log('Sample commit insertions:', commits[0].insertions);
+        const authors = analyzeContributors(commits);
+        console.log('Top author data:', JSON.stringify(authors.contributors[0], null, 2));
+    }
+}
+
+test().catch(console.error);

packages/web/.gitignore

@@ -0,0 +1 @@
+.env

packages/web/public/app.js

@@ -971,17 +971,23 @@ document.addEventListener('DOMContentLoaded', () => {
         const onboardingTableBody = document.querySelector('#onboarding-table tbody');
         if (onboardingTableBody && p1.contributors && p2.onboarding) {
             onboardingTableBody.innerHTML = '';
-            p1.contributors.contributors.slice(0, 8).forEach(c => {
+            p1.contributors.contributors
+                .sort((a, b) => b.commitCount - a.commitCount)
+                .slice(0, 8)
+                .forEach(c => {
                 const tr = document.createElement('tr');
-                const onboardingScore = c.email === 'github-actions' ? 97 : (Math.floor(Math.random() * (95 - 75 + 1)) + 75);
-                const impact = c.commitCount * 12;
-                const stability = 100 - Math.round((data.summary.churnRate || 0.1) * 100);
+                // Use per-contributor stability from core
+                const stability = c.stability || 85;
+                // Impact is now real insertion count
+                const impact = c.insertions || 0;
+                // Onboarding score: crude estimate based on stability and commit volume
+                const onboardingScore = Math.min(98, Math.round((stability * 0.6) + (Math.min(c.commitCount, 50) * 0.8)));
 
                 tr.innerHTML = `
                     <td>
                         <div class="author-cell">
                             <div class="author-avatar-mini"></div>
-                            <span>${c.name.split(' ')[0].toUpperCase()}</span>
+                            <span>${(c.name || 'Unknown').split(' ')[0].toUpperCase()}</span>
                         </div>
                     </td>
                     <td>${c.commitCount}</td>