GitNexus/gitnexus/test/unit/analyze-self-commit-bridge.test.ts

import { beforeEach, describe, expect, it, vi } from 'vitest';

const {
  runFullAnalysisMock,
  generateAIContextFilesMock,
  generateSkillFilesMock,
  cliErrorMock,
  selfCommitContextFilesMock,
  snapshotSelfCommitSafetyMock,
} = vi.hoisted(() => {
  const runFullAnalysisMock = vi.fn();
  const generateAIContextFilesMock = vi.fn(async () => ({ files: [] as string[] }));
  const generateSkillFilesMock = vi.fn(async () => ({
    skills: [{ name: 'c', label: 'Community', symbolCount: 1, fileCount: 1 }],
    outputPath: '/repo/.claude/skills',
  }));
  const cliErrorMock = vi.fn();
  const selfCommitContextFilesMock = vi.fn();
  const snapshotSelfCommitSafetyMock = vi.fn(
    () =>
      new Map([
        ['AGENTS.md', true],
        ['CLAUDE.md', true],
      ]),
  );
  return {
    runFullAnalysisMock,
    generateAIContextFilesMock,
    generateSkillFilesMock,
    cliErrorMock,
    selfCommitContextFilesMock,
    snapshotSelfCommitSafetyMock,
  };
});

vi.mock('../../src/core/run-analyze.js', () => ({
  runFullAnalysis: runFullAnalysisMock,
}));

vi.mock('../../src/cli/ai-context.js', () => ({
  generateAIContextFiles: generateAIContextFilesMock,
}));

vi.mock('../../src/cli/skill-gen.js', () => ({
  generateSkillFiles: generateSkillFilesMock,
}));

vi.mock('../../src/cli/cli-message.js', () => ({
  cliError: cliErrorMock,
}));

vi.mock('../../src/core/lbug/lbug-adapter.js', () => ({
  closeLbug: vi.fn(async () => undefined),
  closeLbugBeforeExit: vi.fn(async () => undefined),
  isLbugReady: vi.fn(() => false),
}));

vi.mock('../../src/storage/repo-manager.js', () => ({
  getStoragePaths: vi.fn(() => ({ storagePath: '.gitnexus', lbugPath: '.gitnexus/lbug' })),
  getGlobalRegistryPath: vi.fn(() => 'registry.json'),
  RegistryNameCollisionError: class RegistryNameCollisionError extends Error {},
  AnalysisNotFinalizedError: class AnalysisNotFinalizedError extends Error {},
  assertAnalysisFinalized: vi.fn(async () => undefined),
}));

vi.mock('../../src/storage/git.js', () => ({
  getGitRoot: vi.fn(() => '/repo'),
  hasGitDir: vi.fn(() => true),
  getDefaultBranch: vi.fn(() => null),
  selfCommitContextFiles: selfCommitContextFilesMock,
  snapshotSelfCommitSafety: snapshotSelfCommitSafetyMock,
}));

vi.mock('../../src/core/ingestion/utils/max-file-size.js', () => ({
  getMaxFileSizeBannerMessage: vi.fn(() => null),
}));

describe('analyzeCommand --self-commit bridge (#2639)', () => {
  beforeEach(() => {
    vi.resetModules();
    runFullAnalysisMock.mockReset();
    runFullAnalysisMock.mockResolvedValue({
      repoName: 'repo',
      repoPath: '/repo',
      stats: {},
      alreadyUpToDate: true,
    });
    generateAIContextFilesMock.mockReset();
    generateAIContextFilesMock.mockResolvedValue({ files: [] });
    generateSkillFilesMock.mockReset();
    generateSkillFilesMock.mockResolvedValue({
      skills: [{ name: 'c', label: 'Community', symbolCount: 1, fileCount: 1 }],
      outputPath: '/repo/.claude/skills',
    });
    cliErrorMock.mockReset();
    selfCommitContextFilesMock.mockReset();
    snapshotSelfCommitSafetyMock.mockClear();
    process.exitCode = undefined;
    process.env.NODE_OPTIONS = `${process.env.NODE_OPTIONS ?? ''} --max-old-space-size=8192`.trim();
  });

  it('does not call selfCommitContextFiles when --self-commit is omitted (default off)', async () => {
    const { analyzeCommand } = await import('../../src/cli/analyze.js');

    await analyzeCommand(undefined, {});

    expect(selfCommitContextFilesMock).not.toHaveBeenCalled();
  });

  it('does not call selfCommitContextFiles when --self-commit is explicitly false', async () => {
    const { analyzeCommand } = await import('../../src/cli/analyze.js');

    await analyzeCommand(undefined, { selfCommit: false });

    expect(selfCommitContextFilesMock).not.toHaveBeenCalled();
  });

  it('calls selfCommitContextFiles scoped to AGENTS.md/CLAUDE.md on the already-up-to-date fast path', async () => {
    const { analyzeCommand } = await import('../../src/cli/analyze.js');

    await analyzeCommand(undefined, { selfCommit: true });

    expect(selfCommitContextFilesMock).toHaveBeenCalledTimes(1);
    expect(selfCommitContextFilesMock).toHaveBeenCalledWith(
      '/repo',
      ['AGENTS.md', 'CLAUDE.md'],
      expect.any(Map),
    );
  });

  it('calls selfCommitContextFiles on the primary (non-fast-path) analyze run', async () => {
    runFullAnalysisMock.mockResolvedValueOnce({
      repoName: 'repo',
      repoPath: '/repo',
      stats: {
        files: 1,
        nodes: 10,
        edges: 20,
        communities: 0,
        processes: 5,
      },
      alreadyUpToDate: false,
      pipelineResult: { communityResult: undefined },
    });

    const exitSpy = vi.spyOn(process, 'exit').mockImplementation(() => undefined as never);
    try {
      const { analyzeCommand } = await import('../../src/cli/analyze.js');

      await analyzeCommand(undefined, { selfCommit: true });

      expect(selfCommitContextFilesMock).toHaveBeenCalledTimes(1);
      expect(selfCommitContextFilesMock).toHaveBeenCalledWith(
        '/repo',
        ['AGENTS.md', 'CLAUDE.md'],
        expect.any(Map),
      );
    } finally {
      exitSpy.mockRestore();
    }
  });

  it('does not call selfCommitContextFiles on the primary run when --self-commit is omitted', async () => {
    runFullAnalysisMock.mockResolvedValueOnce({
      repoName: 'repo',
      repoPath: '/repo',
      stats: {
        files: 1,
        nodes: 10,
        edges: 20,
        communities: 0,
        processes: 5,
      },
      alreadyUpToDate: false,
      pipelineResult: { communityResult: undefined },
    });

    const exitSpy = vi.spyOn(process, 'exit').mockImplementation(() => undefined as never);
    try {
      const { analyzeCommand } = await import('../../src/cli/analyze.js');

      await analyzeCommand(undefined, {});

      expect(selfCommitContextFilesMock).not.toHaveBeenCalled();
    } finally {
      exitSpy.mockRestore();
    }
  });

  it('composes with --no-stats (both flags threaded independently)', async () => {
    const { analyzeCommand } = await import('../../src/cli/analyze.js');

    await analyzeCommand(undefined, { selfCommit: true, stats: false });

    const opts = runFullAnalysisMock.mock.calls[0][1];
    expect(opts.noStats).toBe(true);
    expect(selfCommitContextFilesMock).toHaveBeenCalledWith(
      '/repo',
      ['AGENTS.md', 'CLAUDE.md'],
      expect.any(Map),
    );
  });
});