git-blame-ignore 1.0.0

package/src/scanner.ts

@@ -0,0 +1,159 @@
+import simpleGit, { SimpleGit } from 'simple-git';
+import { BulkCommit } from './types';
+
+export class BulkCommitScanner {
+  private git: SimpleGit;
+  private config: {
+    minFilesChanged: number;
+    minBulkScore: number;
+    whitespaceThreshold: number;
+  };
+
+  constructor(config?: {
+    minFilesChanged?: number;
+    minBulkScore?: number;
+    whitespaceThreshold?: number;
+  }) {
+    this.git = simpleGit();
+    this.config = {
+      minFilesChanged: config?.minFilesChanged || 20,
+      minBulkScore: config?.minBulkScore || 40,
+      whitespaceThreshold: config?.whitespaceThreshold || 0.8,
+    };
+  }
+
+  async scan(): Promise<BulkCommit[]> {
+    try {
+      const log = await this.git.log({ 
+        format: '%H|%s|%ai|%P',
+        file: null,
+        n: 100 
+      });
+
+      const commits: BulkCommit[] = [];
+
+      for (const commit of log.all) {
+        const [hash] = commit.split('|');
+        const bulkCommit = await this.analyzeCommit(hash);
+        if (bulkCommit.bulkScore >= this.config.minBulkScore) {
+          commits.push(bulkCommit);
+        }
+      }
+
+      return commits.sort((a, b) => b.bulkScore - a.bulkScore);
+    } catch (error) {
+      throw new Error(`Failed to scan commits: ${error}`);
+    }
+  }
+
+  private async analyzeCommit(sha: string): Promise<BulkCommit> {
+    const diff = await this.git.diff([`${sha}^`, sha, '--name-only']);
+    const files = diff.split('\n').filter(f => f.trim() !== '');
+    
+    if (files.length < this.config.minFilesChanged) {
+      return this.createEmptyCommit(sha, files.length);
+    }
+
+    const stats = await this.git.diff([`${sha}^`, sha, '--stat']);
+    const linesChanged = this.parseLinesChanged(stats);
+    
+    const whitespaceRatio = await this.calculateWhitespaceRatio(sha);
+    const message = await this.getCommitMessage(sha);
+    
+    const bulkScore = this.calculateBulkScore(files.length, linesChanged, whitespaceRatio, message);
+    
+    return {
+      sha,
+      message,
+      filesChanged: files.length,
+      linesChanged,
+      whitespaceRatio,
+      bulkScore,
+      date: await this.getCommitDate(sha),
+    };
+  }
+
+  private createEmptyCommit(sha: string, filesChanged: number): BulkCommit {
+    return {
+      sha,
+      message: 'Unknown',
+      filesChanged,
+      linesChanged: 0,
+      whitespaceRatio: 0,
+      bulkScore: 0,
+      date: 'Unknown',
+    };
+  }
+
+  private parseLinesChanged(stats: string): number {
+    const linesMatch = stats.match(/\d+ files? changed(?:, (\d+) insertions?\(\+\))?(?:, (\d+) deletions?\(-\))?/);
+    if (!linesMatch) return 0;
+    
+    const insertions = linesMatch[1] ? parseInt(linesMatch[1]) : 0;
+    const deletions = linesMatch[2] ? parseInt(linesMatch[2]) : 0;
+    return insertions + deletions;
+  }
+
+  private async calculateWhitespaceRatio(sha: string): Promise<number> {
+    try {
+      const diff = await this.git.diff([`${sha}^`, sha, '--']);
+      if (!diff) return 0;
+
+      const lines = diff.split('\n');
+      const whitespaceLines = lines.filter(line => 
+        line.trim() === '' && line !== 'diff --git' && !line.startsWith('index') && !line.startsWith('---') && !line.startsWith('+++')
+      ).length;
+
+      return lines.length > 0 ? whitespaceLines / lines.length : 0;
+    } catch {
+      return 0;
+    }
+  }
+
+  private async getCommitMessage(sha: string): Promise<string> {
+    try {
+      const result = await this.git.show([sha, '--format=%s', '--no-patch']);
+      return result.trim();
+    } catch {
+      return 'Unknown';
+    }
+  }
+
+  private async getCommitDate(sha: string): Promise<string> {
+    try {
+      const result = await this.git.show([sha, '--format=%ai', '--no-patch']);
+      return result.trim();
+    } catch {
+      return 'Unknown';
+    }
+  }
+
+  private calculateBulkScore(
+    filesChanged: number, 
+    linesChanged: number, 
+    whitespaceRatio: number, 
+    message: string
+  ): number {
+    let score = 0;
+
+    // Files changed scoring
+    if (filesChanged >= 50) score += 50;
+    else if (filesChanged >= 30) score += 40;
+    else if (filesChanged >= 20) score += 30;
+
+    // Message scoring
+    const lowerMessage = message.toLowerCase();
+    if (lowerMessage.includes('prettier') || lowerMessage.includes('format')) score += 20;
+    if (lowerMessage.includes('eslint') || lowerMessage.includes('lint')) score += 20;
+    if (lowerMessage.includes('rename') || lowerMessage.includes('refactor')) score += 20;
+
+    // Whitespace scoring
+    if (whitespaceRatio >= this.config.whitespaceThreshold) score += 20;
+
+    // Lines per file scoring (if many changes but few lines per file, likely formatting)
+    const linesPerFile = filesChanged > 0 ? linesChanged / filesChanged : 0;
+    if (linesPerFile < 5) score += 10;
+
+    return Math.min(score, 100);
+  }
+}

package/src/types.ts

@@ -0,0 +1,23 @@
+export interface BulkCommit {
+  sha: string;
+  message: string;
+  filesChanged: number;
+  linesChanged: number;
+  whitespaceRatio: number;
+  bulkScore: number;
+  date: string;
+}
+
+export interface IgnoreEntry {
+  sha: string;
+  message: string;
+  date: string;
+  comment?: string;
+}
+
+export interface GitBlameIgnoreConfig {
+  ignoreFile: string;
+  minFilesChanged: number;
+  minBulkScore: number;
+  whitespaceThreshold: number;
+}

package/test/scanner.test.ts

@@ -0,0 +1,57 @@
+import { BulkCommitScanner } from '../src/scanner';
+
+describe('BulkCommitScanner', () => {
+  let scanner: BulkCommitScanner;
+
+  beforeEach(() => {
+    scanner = new BulkCommitScanner();
+  });
+
+  describe('calculateBulkScore', () => {
+    it('should score high for files with many changes and formatting messages', () => {
+      const score = (scanner as any).calculateBulkScore(50, 100, 0.9, 'prettier format');
+      expect(score).toBeGreaterThan(70);
+    });
+
+    it('should score medium for files with moderate changes', () => {
+      const score = (scanner as any).calculateBulkScore(25, 50, 0.5, 'refactor');
+      expect(score).toBeGreaterThan(40);
+    });
+
+    it('should score low for files with few changes', () => {
+      const score = (scanner as any).calculateBulkScore(10, 20, 0.1, 'bug fix');
+      expect(score).toBeLessThan(40);
+    });
+
+    it('should score high for whitespace-only commits', () => {
+      const score = (scanner as any).calculateBulkScore(30, 10, 0.95, 'format');
+      expect(score).toBeGreaterThan(60);
+    });
+  });
+
+  describe('parseLinesChanged', () => {
+    it('should parse lines changed from git stat output', () => {
+      const stats = ' 3 files changed, 15 insertions(+), 10 deletions(-)';
+      const lines = (scanner as any).parseLinesChanged(stats);
+      expect(lines).toBe(25);
+    });
+
+    it('should handle output without insertions/deletions', () => {
+      const stats = ' 2 files changed';
+      const lines = (scanner as any).parseLinesChanged(stats);
+      expect(lines).toBe(0);
+    });
+
+    it('should handle output with only insertions', () => {
+      const stats = ' 1 file changed, 5 insertions(+)';
+      const lines = (scanner as any).parseLinesChanged(stats);
+      expect(lines).toBe(5);
+    });
+
+    it('should handle output with only deletions', () => {
+      const stats = ' 1 file changed, 3 deletions(-)';
+      const lines = (scanner as any).parseLinesChanged(stats);
+      expect(lines).toBe(3);
+    });
+  });
+});

package/tests/index.test.ts

@@ -0,0 +1,67 @@
+import { describe, it, expect, beforeEach, afterEach } from '@jest/globals';
+import { execSync } from 'child_process';
+import fs from 'fs/promises';
+import path from 'path';
+
+const TEST_REPO_DIR = path.join(__dirname, 'test-repo');
+const GIT_BLAME_IGNORE_CMD = 'node dist/index.js';
+
+describe('git-blame-ignore', () => {
+  beforeEach(async () => {
+    // Create a test git repository
+    await fs.mkdir(TEST_REPO_DIR, { recursive: true });
+    process.chdir(TEST_REPO_DIR);
+    
+    // Initialize git repo
+    execSync('git init', { stdio: 'inherit' });
+    execSync('git config user.name "Test User"', { stdio: 'inherit' });
+    execSync('git config user.email "test@example.com"', { stdio: 'inherit' });
+    
+    // Create some files and commits
+    await fs.writeFile('test1.txt', 'content1');
+    execSync('git add test1.txt', { stdio: 'inherit' });
+    execSync('git commit -m "Initial commit"', { stdio: 'inherit' });
+    
+    await fs.writeFile('test2.txt', 'content2');
+    execSync('git add test2.txt', { stdio: 'inherit' });
+    execSync('git commit -m "Add test2"', { stdio: 'inherit' });
+  });
+
+  afterEach(async () => {
+    // Clean up test repository
+    process.chdir(__dirname);
+    await fs.rm(TEST_REPO_DIR, { recursive: true, force: true });
+  });
+
+  it('should show help when no command is provided', () => {
+    const output = execSync(`node dist/index.js --help`, { encoding: 'utf-8' });
+    expect(output).toContain('git-blame-ignore');
+  });
+
+  it('should scan for bulk-change commits', () => {
+    // Create a commit with multiple files
+    const files = ['file1.txt', 'file2.txt', 'file3.txt'];
+    files.forEach(file => {
+      execSync(`echo "content" > ${file}`, { stdio: 'inherit' });
+    });
+    execSync('git add .', { stdio: 'inherit' });
+    execSync('git commit -m "Bulk commit with many files"', { stdio: 'inherit' });
+    
+    const output = execSync(`node dist/index.js scan`, { encoding: 'utf-8' });
+    expect(output).toContain('Bulk-change commits detected');
+  });
+
+  it('should list ignored commits when file exists', async () => {
+    // Create .git-blame-ignore-revs file
+    await fs.writeFile('.git-blame-ignore-revs', 'abc123def456\nxyz789uvw456\n');
+    
+    const output = execSync(`node dist/index.js list`, { encoding: 'utf-8' });
+    expect(output).toContain('abc123def456');
+    expect(output).toContain('xyz789uvw456');
+  });
+
+  it('should handle missing .git-blame-ignore-revs file gracefully', () => {
+    const output = execSync(`node dist/index.js list`, { encoding: 'utf-8' });
+    expect(output).toContain('No commits are currently ignored');
+  });
+});

package/tsconfig.json

@@ -0,0 +1,25 @@
+{
+  "compilerOptions": {
+    "target": "ES2020",
+    "module": "CommonJS",
+    "lib": ["ES2020"],
+    "outDir": "./dist",
+    "rootDir": "./src",
+    "strict": true,
+    "esModuleInterop": true,
+    "skipLibCheck": true,
+    "forceConsistentCasingInFileNames": true,
+    "declaration": true,
+    "declarationMap": true,
+    "sourceMap": true,
+    "resolveJsonModule": true,
+    "noImplicitAny": true,
+    "noImplicitReturns": true,
+    "noImplicitThis": true,
+    "noUnusedLocals": true,
+    "noUnusedParameters": true,
+    "exactOptionalPropertyTypes": true
+  },
+  "include": ["src/**/*"],
+  "exclude": ["node_modules", "dist", "**/*.test.ts"]
+}