@sentry/warden 0.1.1 → 0.2.0

package/src/cli/commands/sync.ts

@@ -1,114 +0,0 @@
-import chalk from 'chalk';
-import {
-  fetchRemote,
-  listCachedRemotes,
-  parseRemoteRef,
-} from '../../skills/remote.js';
-import type { Reporter } from '../output/reporter.js';
-import type { CLIOptions } from '../args.js';
-
-/**
- * Run the sync command.
- * Updates cached remote skills to their latest versions.
- * Pinned skills (with @sha) are skipped as they're immutable.
- */
-export async function runSync(options: CLIOptions, reporter: Reporter): Promise<number> {
-  const cachedRemotes = listCachedRemotes();
-
-  if (cachedRemotes.length === 0) {
-    reporter.warning('No remote skills cached.');
-    reporter.tip('Add remote skills with: warden add --remote owner/repo --skill name');
-    return 0;
-  }
-
-  // If a specific remote is provided, only sync that one
-  const targetRepo = options.remote;
-  const remotesToSync = targetRepo
-    ? cachedRemotes.filter(({ ref }) => ref === targetRepo || ref.startsWith(`${targetRepo}@`))
-    : cachedRemotes;
-
-  if (targetRepo && remotesToSync.length === 0) {
-    reporter.error(`Remote not found in cache: ${targetRepo}`);
-    reporter.blank();
-    reporter.text('Cached remotes:');
-    for (const { ref } of cachedRemotes) {
-      reporter.text(`  - ${ref}`);
-    }
-    return 1;
-  }
-
-  // Separate pinned and unpinned remotes
-  const unpinnedRemotes = remotesToSync.filter(({ ref }) => {
-    const parsed = parseRemoteRef(ref);
-    return !parsed.sha;
-  });
-
-  const pinnedRemotes = remotesToSync.filter(({ ref }) => {
-    const parsed = parseRemoteRef(ref);
-    return !!parsed.sha;
-  });
-
-  if (unpinnedRemotes.length === 0) {
-    if (pinnedRemotes.length > 0) {
-      reporter.warning('All cached remotes are pinned to specific versions.');
-      reporter.text('Pinned remotes do not need syncing as they are immutable.');
-      for (const { ref } of pinnedRemotes) {
-        reporter.text(`  - ${ref} ${chalk.dim('(pinned)')}`);
-      }
-    } else {
-      reporter.warning('No remotes to sync.');
-    }
-    return 0;
-  }
-
-  reporter.bold('SYNC REMOTE SKILLS');
-  reporter.blank();
-
-  let updated = 0;
-  let errors = 0;
-
-  for (const { ref, entry } of unpinnedRemotes) {
-    reporter.step(`Syncing ${ref}...`);
-
-    const previousSha = entry.sha;
-
-    try {
-      const newSha = await fetchRemote(ref, { force: true });
-
-      if (newSha !== previousSha) {
-        reporter.success(`${ref}: updated ${chalk.dim(previousSha.slice(0, 7))} -> ${chalk.dim(newSha.slice(0, 7))}`);
-        updated++;
-      } else {
-        reporter.text(`${ref}: ${chalk.dim('already up to date')}`);
-      }
-    } catch (err) {
-      const message = err instanceof Error ? err.message : String(err);
-      reporter.error(`${ref}: ${message}`);
-      errors++;
-    }
-  }
-
-  // Show pinned remotes that were skipped
-  if (pinnedRemotes.length > 0 && !targetRepo) {
-    reporter.blank();
-    reporter.text(chalk.dim('Skipped pinned remotes:'));
-    for (const { ref } of pinnedRemotes) {
-      reporter.text(chalk.dim(`  - ${ref}`));
-    }
-  }
-
-  // Summary
-  reporter.blank();
-  if (errors > 0) {
-    reporter.warning(`Synced with ${errors} error${errors === 1 ? '' : 's'}.`);
-    return 1;
-  }
-
-  if (updated > 0) {
-    reporter.success(`Updated ${updated} remote${updated === 1 ? '' : 's'}.`);
-  } else {
-    reporter.success('All remotes up to date.');
-  }
-
-  return 0;
-}

package/src/cli/context.ts

@@ -1,131 +0,0 @@
-import { basename } from 'node:path';
-import type { EventContext, FileChange } from '../types/index.js';
-import { pluralize } from './output/index.js';
-import { expandAndCreateFileChanges } from './files.js';
-import {
-  getChangedFilesWithPatches,
-  getCurrentBranch,
-  getHeadSha,
-  getDefaultBranch,
-  getRepoRoot,
-  getRepoName,
-  getCommitMessage,
-  type GitFileChange,
-} from './git.js';
-
-/**
- * Convert git file change to EventContext FileChange format.
- */
-function toFileChange(file: GitFileChange): FileChange {
-  return {
-    filename: file.filename,
-    status: file.status,
-    additions: file.additions,
-    deletions: file.deletions,
-    patch: file.patch,
-    chunks: file.chunks,
-  };
-}
-
-export interface LocalContextOptions {
-  base?: string;
-  head?: string;
-  cwd?: string;
-  /** Override auto-detected default branch (from config) */
-  defaultBranch?: string;
-}
-
-/**
- * Build an EventContext from local git repository state.
- * Creates a synthetic pull_request event from git diff.
- *
- * When analyzing a specific commit (head is set), uses the actual commit
- * message as title/body to provide intent context to the LLM.
- */
-export function buildLocalEventContext(options: LocalContextOptions = {}): EventContext {
-  const cwd = options.cwd ?? process.cwd();
-  const repoPath = getRepoRoot(cwd);
-  const { owner, name } = getRepoName(cwd);
-  const defaultBranch = options.defaultBranch ?? getDefaultBranch(cwd);
-
-  const base = options.base ?? defaultBranch;
-  const head = options.head; // undefined means working tree
-  const currentBranch = getCurrentBranch(cwd);
-  const headSha = head ? head : getHeadSha(cwd);
-
-  const changedFiles = getChangedFilesWithPatches(base, head, cwd);
-  const files = changedFiles.map(toFileChange);
-
-  // Use actual commit message when analyzing a specific commit
-  let title: string;
-  let body: string;
-  if (head) {
-    const commitMsg = getCommitMessage(head, cwd);
-    title = commitMsg.subject || `Commit ${head}`;
-    body = commitMsg.body || `Analyzing changes in ${head}`;
-  } else {
-    title = `Local changes: ${currentBranch}`;
-    body = `Analyzing local changes from ${base} to working tree`;
-  }
-
-  return {
-    eventType: 'pull_request',
-    action: 'opened',
-    repository: {
-      owner,
-      name,
-      fullName: `${owner}/${name}`,
-      defaultBranch,
-    },
-    pullRequest: {
-      number: 0, // Local run, no real PR number
-      title,
-      body,
-      author: 'local',
-      baseBranch: base,
-      headBranch: currentBranch,
-      headSha,
-      files,
-    },
-    repoPath,
-  };
-}
-
-export interface FileContextOptions {
-  patterns: string[];
-  cwd?: string;
-}
-
-/**
- * Build an EventContext from a list of files or glob patterns.
- * Creates a synthetic pull_request event treating files as newly added.
- * This allows analysis without requiring git or a warden.toml config.
- */
-export async function buildFileEventContext(options: FileContextOptions): Promise<EventContext> {
-  const cwd = options.cwd ?? process.cwd();
-  const dirName = basename(cwd);
-
-  const files = await expandAndCreateFileChanges(options.patterns, cwd);
-
-  return {
-    eventType: 'pull_request',
-    action: 'opened',
-    repository: {
-      owner: 'local',
-      name: dirName,
-      fullName: `local/${dirName}`,
-      defaultBranch: 'main',
-    },
-    pullRequest: {
-      number: 0,
-      title: 'File analysis',
-      body: `Analyzing ${files.length} ${pluralize(files.length, 'file')}`,
-      author: 'local',
-      baseBranch: 'main',
-      headBranch: 'file-analysis',
-      headSha: 'file-analysis',
-      files,
-    },
-    repoPath: cwd,
-  };
-}

package/src/cli/files.test.ts

@@ -1,155 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { writeFileSync, mkdirSync, rmSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import {
-  createPatchFromContent,
-  createSyntheticFileChange,
-  expandFileGlobs,
-  expandAndCreateFileChanges,
-} from './files.js';
-
-describe('createPatchFromContent', () => {
-  it('creates patch for single line content', () => {
-    const patch = createPatchFromContent('hello world');
-    expect(patch).toBe('@@ -0,0 +1,1 @@\n+hello world');
-  });
-
-  it('creates patch for multi-line content', () => {
-    const patch = createPatchFromContent('line1\nline2\nline3');
-    expect(patch).toBe('@@ -0,0 +1,3 @@\n+line1\n+line2\n+line3');
-  });
-
-  it('handles empty file', () => {
-    const patch = createPatchFromContent('');
-    expect(patch).toBe('@@ -0,0 +0,0 @@\n');
-  });
-
-  it('handles file ending with newline', () => {
-    const patch = createPatchFromContent('line1\nline2\n');
-    expect(patch).toBe('@@ -0,0 +1,3 @@\n+line1\n+line2\n+');
-  });
-});
-
-describe('createSyntheticFileChange', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = join(tmpdir(), `warden-test-${Date.now()}`);
-    mkdirSync(tempDir, { recursive: true });
-  });
-
-  afterEach(() => {
-    rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  it('creates FileChange from file', () => {
-    const filePath = join(tempDir, 'test.ts');
-    writeFileSync(filePath, 'const x = 1;\nconst y = 2;');
-
-    const change = createSyntheticFileChange(filePath, tempDir);
-
-    expect(change.filename).toBe('test.ts');
-    expect(change.status).toBe('added');
-    expect(change.additions).toBe(2);
-    expect(change.deletions).toBe(0);
-    expect(change.patch).toContain('+const x = 1;');
-    expect(change.patch).toContain('+const y = 2;');
-  });
-
-  it('handles nested files', () => {
-    const subDir = join(tempDir, 'src', 'utils');
-    mkdirSync(subDir, { recursive: true });
-    const filePath = join(subDir, 'helper.ts');
-    writeFileSync(filePath, 'export const helper = () => {};\n');
-
-    const change = createSyntheticFileChange(filePath, tempDir);
-
-    expect(change.filename).toBe('src/utils/helper.ts');
-  });
-});
-
-describe('expandFileGlobs', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = join(tmpdir(), `warden-test-${Date.now()}`);
-    mkdirSync(tempDir, { recursive: true });
-  });
-
-  afterEach(() => {
-    rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  it('expands glob pattern', async () => {
-    writeFileSync(join(tempDir, 'file1.ts'), 'content1');
-    writeFileSync(join(tempDir, 'file2.ts'), 'content2');
-    writeFileSync(join(tempDir, 'file.js'), 'content3');
-
-    const files = await expandFileGlobs(['*.ts'], tempDir);
-
-    expect(files).toHaveLength(2);
-    expect(files.some(f => f.endsWith('file1.ts'))).toBe(true);
-    expect(files.some(f => f.endsWith('file2.ts'))).toBe(true);
-    expect(files.some(f => f.endsWith('file.js'))).toBe(false);
-  });
-
-  it('expands nested glob pattern', async () => {
-    const srcDir = join(tempDir, 'src');
-    mkdirSync(srcDir, { recursive: true });
-    writeFileSync(join(tempDir, 'root.ts'), 'root');
-    writeFileSync(join(srcDir, 'nested.ts'), 'nested');
-
-    const files = await expandFileGlobs(['**/*.ts'], tempDir);
-
-    expect(files).toHaveLength(2);
-    expect(files.some(f => f.endsWith('root.ts'))).toBe(true);
-    expect(files.some(f => f.includes('src/nested.ts'))).toBe(true);
-  });
-
-  it('handles specific file path', async () => {
-    const filePath = join(tempDir, 'specific.ts');
-    writeFileSync(filePath, 'content');
-
-    const files = await expandFileGlobs(['specific.ts'], tempDir);
-
-    expect(files).toHaveLength(1);
-    expect(files[0]).toContain('specific.ts');
-  });
-
-  it('returns empty for no matches', async () => {
-    const files = await expandFileGlobs(['*.nonexistent'], tempDir);
-    expect(files).toHaveLength(0);
-  });
-});
-
-describe('expandAndCreateFileChanges', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = join(tmpdir(), `warden-test-${Date.now()}`);
-    mkdirSync(tempDir, { recursive: true });
-  });
-
-  afterEach(() => {
-    rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  it('combines glob expansion and file change creation', async () => {
-    writeFileSync(join(tempDir, 'file1.ts'), 'const a = 1;');
-    writeFileSync(join(tempDir, 'file2.ts'), 'const b = 2;\nconst c = 3;');
-
-    const changes = await expandAndCreateFileChanges(['*.ts'], tempDir);
-
-    expect(changes).toHaveLength(2);
-    expect(changes.every(c => c.status === 'added')).toBe(true);
-
-    const file1 = changes.find(c => c.filename === 'file1.ts');
-    expect(file1).toBeDefined();
-    expect(file1?.additions).toBe(1);
-
-    const file2 = changes.find(c => c.filename === 'file2.ts');
-    expect(file2).toBeDefined();
-    expect(file2?.additions).toBe(2);
-  });
-});

package/src/cli/files.ts

@@ -1,89 +0,0 @@
-import { readFileSync } from 'node:fs';
-import { resolve, relative } from 'node:path';
-import fg from 'fast-glob';
-import { countPatchChunks } from '../types/index.js';
-import type { FileChange } from '../types/index.js';
-
-/**
- * Expand glob patterns to a list of file paths.
- */
-export async function expandFileGlobs(
-  patterns: string[],
-  cwd: string = process.cwd()
-): Promise<string[]> {
-  const files = await fg(patterns, {
-    cwd,
-    onlyFiles: true,
-    absolute: true,
-    dot: false,
-  });
-
-  return files.sort();
-}
-
-/**
- * Create a unified diff patch for a file, treating entire content as added.
- */
-export function createPatchFromContent(content: string): string {
-  const lines = content.split('\n');
-  const lineCount = lines.length;
-
-  // Handle empty files
-  if (lineCount === 0 || (lineCount === 1 && lines[0] === '')) {
-    return '@@ -0,0 +0,0 @@\n';
-  }
-
-  // Create patch header showing all lines as additions
-  const patchLines = [`@@ -0,0 +1,${lineCount} @@`];
-
-  for (const line of lines) {
-    patchLines.push(`+${line}`);
-  }
-
-  return patchLines.join('\n');
-}
-
-/**
- * Read a file and create a synthetic FileChange treating it as newly added.
- */
-export function createSyntheticFileChange(
-  absolutePath: string,
-  basePath: string
-): FileChange {
-  const content = readFileSync(absolutePath, 'utf-8');
-  const lines = content.split('\n');
-  const lineCount = lines.length;
-  const relativePath = relative(basePath, absolutePath);
-  const patch = createPatchFromContent(content);
-
-  return {
-    filename: relativePath,
-    status: 'added',
-    additions: lineCount,
-    deletions: 0,
-    patch,
-    chunks: countPatchChunks(patch),
-  };
-}
-
-/**
- * Process a list of file paths into FileChange objects.
- */
-export function createSyntheticFileChanges(
-  absolutePaths: string[],
-  basePath: string
-): FileChange[] {
-  return absolutePaths.map((filePath) => createSyntheticFileChange(filePath, basePath));
-}
-
-/**
- * Expand glob patterns and create FileChange objects for all matching files.
- */
-export async function expandAndCreateFileChanges(
-  patterns: string[],
-  cwd: string = process.cwd()
-): Promise<FileChange[]> {
-  const resolvedCwd = resolve(cwd);
-  const files = await expandFileGlobs(patterns, resolvedCwd);
-  return createSyntheticFileChanges(files, resolvedCwd);
-}