@sentry/warden 0.1.1 → 0.3.0

package/src/cli/git.test.ts

@@ -1,119 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { execSync } from 'node:child_process';
-import { getDefaultBranch, getCurrentBranch, getCommitMessage } from './git.js';
-
-vi.mock('node:child_process', () => ({
-  execSync: vi.fn(),
-}));
-
-const mockExecSync = vi.mocked(execSync);
-
-beforeEach(() => {
-  mockExecSync.mockReset();
-});
-
-describe('git error handling', () => {
-  it('includes stderr in error message when git command fails', () => {
-    const error = new Error('Command failed') as Error & { stderr: string };
-    error.stderr = 'fatal: not a git repository';
-    mockExecSync.mockImplementation(() => {
-      throw error;
-    });
-
-    expect(() => getCurrentBranch()).toThrow(
-      'Git command failed: git rev-parse --abbrev-ref HEAD\nfatal: not a git repository'
-    );
-  });
-
-  it('includes command in error message when stderr is empty', () => {
-    mockExecSync.mockImplementation(() => {
-      throw new Error('Command failed');
-    });
-
-    expect(() => getCurrentBranch()).toThrow(
-      'Git command failed: git rev-parse --abbrev-ref HEAD\nCommand failed'
-    );
-  });
-});
-
-/**
- * Creates a mock that simulates git branch detection.
- * Returns success for branches in existingBranches, and optionally a config value.
- */
-function mockBranchDetection(
-  existingBranches: string[],
-  configDefault?: string
-): (cmd: string) => string {
-  return (cmd: string) => {
-    for (const branch of existingBranches) {
-      if (cmd === `git rev-parse --verify ${branch}`) {
-        return 'abc123\n';
-      }
-    }
-    if (cmd === 'git config init.defaultBranch' && configDefault) {
-      return `${configDefault}\n`;
-    }
-    throw new Error('Not found');
-  };
-}
-
-describe('getDefaultBranch', () => {
-  it('returns main when main branch exists locally', () => {
-    mockExecSync.mockImplementation(mockBranchDetection(['main']));
-    expect(getDefaultBranch()).toBe('main');
-  });
-
-  it('returns master when main does not exist but master does', () => {
-    mockExecSync.mockImplementation(mockBranchDetection(['master']));
-    expect(getDefaultBranch()).toBe('master');
-  });
-
-  it('returns develop when main and master do not exist but develop does', () => {
-    mockExecSync.mockImplementation(mockBranchDetection(['develop']));
-    expect(getDefaultBranch()).toBe('develop');
-  });
-
-  it('returns git config init.defaultBranch when no common branches exist', () => {
-    mockExecSync.mockImplementation(mockBranchDetection([], 'trunk'));
-    expect(getDefaultBranch()).toBe('trunk');
-  });
-
-  it('returns hardcoded main when no branches exist and no config is set', () => {
-    mockExecSync.mockImplementation(mockBranchDetection([]));
-    expect(getDefaultBranch()).toBe('main');
-  });
-});
-
-describe('getCommitMessage', () => {
-  it('returns subject and body from commit', () => {
-    mockExecSync.mockImplementation((cmd: string) => {
-      if (cmd === 'git log -1 --format=%s HEAD') {
-        return 'feat: Add new feature\n';
-      }
-      if (cmd === 'git log -1 --format=%b HEAD') {
-        return 'This is the commit body.\n\nWith multiple paragraphs.\n';
-      }
-      throw new Error('Unexpected command');
-    });
-
-    const result = getCommitMessage('HEAD');
-    expect(result.subject).toBe('feat: Add new feature');
-    expect(result.body).toBe('This is the commit body.\n\nWith multiple paragraphs.');
-  });
-
-  it('returns empty body when commit has no body', () => {
-    mockExecSync.mockImplementation((cmd: string) => {
-      if (cmd === 'git log -1 --format=%s abc123') {
-        return 'fix: Quick fix\n';
-      }
-      if (cmd === 'git log -1 --format=%b abc123') {
-        return '\n';
-      }
-      throw new Error('Unexpected command');
-    });
-
-    const result = getCommitMessage('abc123');
-    expect(result.subject).toBe('fix: Quick fix');
-    expect(result.body).toBe('');
-  });
-});

package/src/cli/git.ts

@@ -1,318 +0,0 @@
-import { execSync } from 'node:child_process';
-import { countPatchChunks } from '../types/index.js';
-
-export interface GitFileChange {
-  filename: string;
-  status: 'added' | 'removed' | 'modified' | 'renamed' | 'copied';
-  additions: number;
-  deletions: number;
-  patch?: string;
-  chunks?: number;
-}
-
-/**
- * Execute a git command and return stdout.
- */
-function git(args: string, cwd: string = process.cwd()): string {
-  try {
-    return execSync(`git ${args}`, {
-      cwd,
-      encoding: 'utf-8',
-      stdio: ['pipe', 'pipe', 'pipe'],
-    }).trim();
-  } catch (error) {
-    const execError = error as { stderr?: Buffer | string; message: string };
-    const stderr = execError.stderr?.toString() ?? '';
-    throw new Error(`Git command failed: git ${args}\n${stderr || execError.message}`);
-  }
-}
-
-/**
- * Get the current branch name.
- */
-export function getCurrentBranch(cwd: string = process.cwd()): string {
-  return git('rev-parse --abbrev-ref HEAD', cwd);
-}
-
-/**
- * Get the HEAD commit SHA.
- */
-export function getHeadSha(cwd: string = process.cwd()): string {
-  return git('rev-parse HEAD', cwd);
-}
-
-/**
- * Detect the default branch by checking common branch names locally.
- * Does not perform any remote operations to avoid SSH prompts.
- */
-export function getDefaultBranch(cwd: string = process.cwd()): string {
-  // Check common default branches locally (no remote operations)
-  for (const branch of ['main', 'master', 'develop']) {
-    try {
-      git(`rev-parse --verify ${branch}`, cwd);
-      return branch;
-    } catch {
-      // Try next branch
-    }
-  }
-
-  // Check git config for user-configured default branch
-  try {
-    const configuredDefault = git('config init.defaultBranch', cwd);
-    if (configuredDefault) {
-      return configuredDefault;
-    }
-  } catch {
-    // Config not set
-  }
-
-  return 'main'; // Default fallback
-}
-
-/**
- * Get the repository root path.
- */
-export function getRepoRoot(cwd: string = process.cwd()): string {
-  return git('rev-parse --show-toplevel', cwd);
-}
-
-/**
- * Get the repository name from the git remote or directory name.
- */
-export function getRepoName(cwd: string = process.cwd()): { owner: string; name: string } {
-  try {
-    const remoteUrl = git('config --get remote.origin.url', cwd);
-    // Handle SSH: git@github.com:owner/repo.git
-    // Handle HTTPS: https://github.com/owner/repo.git
-    const match = remoteUrl.match(/[/:]([\w.-]+)\/([\w.-]+?)(?:\.git)?$/);
-    if (match && match[1] && match[2]) {
-      return { owner: match[1], name: match[2] };
-    }
-  } catch {
-    // No remote configured
-  }
-
-  // Fall back to directory name
-  const repoRoot = getRepoRoot(cwd);
-  const dirName = repoRoot.split('/').pop() ?? 'unknown';
-  return { owner: 'local', name: dirName };
-}
-
-/**
- * Get the GitHub repository URL if the remote is on GitHub.
- * Returns null if the remote is not GitHub or not configured.
- */
-export function getGitHubRepoUrl(cwd: string = process.cwd()): string | null {
-  try {
-    const remoteUrl = git('config --get remote.origin.url', cwd);
-    // Handle SSH: git@github.com:owner/repo.git
-    const sshMatch = remoteUrl.match(/git@github\.com:([\w.-]+)\/([\w.-]+?)(?:\.git)?$/);
-    if (sshMatch && sshMatch[1] && sshMatch[2]) {
-      return `https://github.com/${sshMatch[1]}/${sshMatch[2]}`;
-    }
-    // Handle HTTPS: https://github.com/owner/repo.git
-    const httpsMatch = remoteUrl.match(/https:\/\/github\.com\/([\w.-]+)\/([\w.-]+?)(?:\.git)?$/);
-    if (httpsMatch && httpsMatch[1] && httpsMatch[2]) {
-      return `https://github.com/${httpsMatch[1]}/${httpsMatch[2]}`;
-    }
-  } catch {
-    // No remote configured
-  }
-  return null;
-}
-
-/**
- * Map git status letter to FileChange status.
- */
-function mapStatus(status: string): GitFileChange['status'] {
-  switch (status[0]) {
-    case 'A':
-      return 'added';
-    case 'D':
-      return 'removed';
-    case 'M':
-      return 'modified';
-    case 'R':
-      return 'renamed';
-    case 'C':
-      return 'copied';
-    default:
-      return 'modified';
-  }
-}
-
-/**
- * Get list of changed files between two refs.
- * If head is undefined, compares against the working tree.
- */
-export function getChangedFiles(
-  base: string,
-  head?: string,
-  cwd: string = process.cwd()
-): GitFileChange[] {
-  // Get file statuses
-  const diffArgs = head ? `${base}...${head}` : base;
-  const nameStatusOutput = git(`diff --name-status ${diffArgs}`, cwd);
-
-  if (!nameStatusOutput) {
-    return [];
-  }
-
-  const files: GitFileChange[] = [];
-
-  for (const line of nameStatusOutput.split('\n')) {
-    if (!line.trim()) continue;
-
-    const parts = line.split('\t');
-    const status = parts[0] ?? '';
-    // For renames, format is "R100\told-name\tnew-name"
-    const filename = parts.length > 2 ? (parts[2] ?? '') : (parts[1] ?? '');
-    if (!filename) continue;
-
-    files.push({
-      filename,
-      status: mapStatus(status),
-      additions: 0,
-      deletions: 0,
-    });
-  }
-
-  // Get numstat for additions/deletions
-  const numstatOutput = git(`diff --numstat ${diffArgs}`, cwd);
-  if (numstatOutput) {
-    for (const line of numstatOutput.split('\n')) {
-      if (!line.trim()) continue;
-      const parts = line.split('\t');
-      const additions = parts[0] ?? '0';
-      const deletions = parts[1] ?? '0';
-      const filename = parts[2] ?? '';
-      const file = files.find((f) => f.filename === filename);
-      if (file) {
-        file.additions = additions === '-' ? 0 : parseInt(additions, 10);
-        file.deletions = deletions === '-' ? 0 : parseInt(deletions, 10);
-      }
-    }
-  }
-
-  return files;
-}
-
-/**
- * Get the patch for a specific file.
- */
-export function getFilePatch(
-  base: string,
-  head: string | undefined,
-  filename: string,
-  cwd: string = process.cwd()
-): string | undefined {
-  try {
-    const diffArgs = head ? `${base}...${head}` : base;
-    return git(`diff ${diffArgs} -- "${filename}"`, cwd);
-  } catch {
-    return undefined;
-  }
-}
-
-/**
- * Parse a combined diff output into individual file patches.
- */
-function parseCombinedDiff(diffOutput: string): Map<string, string> {
-  const patches = new Map<string, string>();
-  if (!diffOutput) return patches;
-
-  // Split by "diff --git" but keep the delimiter
-  const parts = diffOutput.split(/(?=^diff --git )/m);
-
-  for (const part of parts) {
-    if (!part.trim()) continue;
-
-    // Extract filename from "diff --git a/path b/path" line
-    const match = part.match(/^diff --git a\/(.+?) b\/(.+?)\n/);
-    if (match) {
-      // Use the "b" path (destination) as the filename
-      const filename = match[2];
-      if (filename) {
-        patches.set(filename, part);
-      }
-    }
-  }
-
-  return patches;
-}
-
-/**
- * Get patches for all changed files in a single git command.
- */
-export function getChangedFilesWithPatches(
-  base: string,
-  head?: string,
-  cwd: string = process.cwd()
-): GitFileChange[] {
-  const files = getChangedFiles(base, head, cwd);
-
-  if (files.length === 0) {
-    return files;
-  }
-
-  // Get all patches in a single git diff command
-  try {
-    const diffArgs = head ? `${base}...${head}` : base;
-    const combinedDiff = git(`diff ${diffArgs}`, cwd);
-    const patches = parseCombinedDiff(combinedDiff);
-
-    for (const file of files) {
-      file.patch = patches.get(file.filename);
-      file.chunks = countPatchChunks(file.patch);
-    }
-  } catch {
-    // Fall back to per-file patches if combined diff fails
-    for (const file of files) {
-      file.patch = getFilePatch(base, head, file.filename, cwd);
-      file.chunks = countPatchChunks(file.patch);
-    }
-  }
-
-  return files;
-}
-
-/**
- * Check if there are uncommitted changes in the working tree.
- */
-export function hasUncommittedChanges(cwd: string = process.cwd()): boolean {
-  const status = git('status --porcelain', cwd);
-  return status.length > 0;
-}
-
-/**
- * Check if a ref exists.
- */
-export function refExists(ref: string, cwd: string = process.cwd()): boolean {
-  try {
-    git(`rev-parse --verify ${ref}`, cwd);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Commit message with subject and body separated.
- */
-export interface CommitMessage {
-  /** First line of the commit message */
-  subject: string;
-  /** Remaining lines after the subject (may be empty) */
-  body: string;
-}
-
-/**
- * Get the commit message for a specific ref.
- * Returns subject (first line) and body (remaining lines) separately.
- */
-export function getCommitMessage(ref: string, cwd: string = process.cwd()): CommitMessage {
-  // %s = subject, %b = body
-  const subject = git(`log -1 --format=%s ${ref}`, cwd);
-  const body = git(`log -1 --format=%b ${ref}`, cwd);
-  return { subject, body };
-}

package/src/cli/index.ts

@@ -1,14 +0,0 @@
-#!/usr/bin/env node
-import { main, abortController } from './main.js';
-
-process.on('SIGINT', () => {
-  // Abort any running SDK queries
-  abortController.abort();
-  process.stderr.write('\n');
-  process.exit(130);
-});
-
-main().catch((error) => {
-  console.error('Fatal error:', error);
-  process.exit(1);
-});