@3fn/core 14.0.0 → 14.1.0

package/src/tools/release/__tests__/NpmPublisher.test.ts

@@ -1,289 +0,0 @@
-/**
- * @category evergreen
- * @purpose Verify NPM publisher handles package publishing correctly
- */
-
-import { NpmPublisher, NpmConfig } from '../publishers/NpmPublisher';
-import { PackagePublish } from '../types';
-import { execSync } from 'child_process';
-import * as fs from 'fs';
-import { NpmMockHelper } from './helpers/NpmMockHelper';
-
-jest.mock('child_process');
-const mockExecSync = execSync as jest.MockedFunction<typeof execSync>;
-
-jest.mock('fs');
-const mockFs = fs as jest.Mocked<typeof fs>;
-
-describe('NpmPublisher', () => {
-  let publisher: NpmPublisher;
-  let config: NpmConfig;
-  let npmHelper: NpmMockHelper;
-
-  beforeEach(() => {
-    jest.clearAllMocks();
-    config = { registry: 'https://registry.npmjs.org/', timeout: 60000, maxRetries: 3, retryDelay: 100, dryRun: false };
-    publisher = new NpmPublisher(config);
-    npmHelper = new NpmMockHelper(mockExecSync);
-  });
-
-  afterEach(() => { mockExecSync.mockReset(); });
-
-  describe('validateAuthentication', () => {
-    it('should return authenticated status when user is logged in', async () => {
-      mockExecSync.mockReturnValueOnce('testuser\n' as any);
-      mockExecSync.mockReturnValueOnce(JSON.stringify({ email: 'test@example.com' }) as any);
-      const result = await publisher.validateAuthentication();
-      expect(result.authenticated).toBe(true);
-      expect(result.username).toBe('testuser');
-      expect(result.email).toBe('test@example.com');
-      expect(result.registry).toBe('https://registry.npmjs.org/');
-    });
-
-    it('should return unauthenticated status when user is not logged in', async () => {
-      mockExecSync.mockImplementationOnce(() => { throw new Error('Not logged in'); });
-      const result = await publisher.validateAuthentication();
-      expect(result.authenticated).toBe(false);
-      expect(result.username).toBeUndefined();
-    });
-
-    it('should handle profile parsing errors gracefully', async () => {
-      mockExecSync.mockReturnValueOnce('testuser\n' as any);
-      mockExecSync.mockReturnValueOnce('invalid json' as any);
-      const result = await publisher.validateAuthentication();
-      expect(result.authenticated).toBe(true);
-      expect(result.username).toBe('testuser');
-      expect(result.email).toBeUndefined();
-    });
-  });
-
-  describe('packageVersionExists', () => {
-    it('should return true when package version exists', async () => {
-      mockExecSync.mockReturnValueOnce('"1.0.0"\n' as any);
-      expect(await publisher.packageVersionExists('@test/package', '1.0.0')).toBe(true);
-    });
-
-    it('should return false when package version does not exist', async () => {
-      mockExecSync.mockImplementationOnce(() => { throw new Error('Package not found'); });
-      expect(await publisher.packageVersionExists('@test/package', '1.0.0')).toBe(false);
-    });
-  });
-
-  describe('getPackageInfo', () => {
-    it('should return package info for existing package', async () => {
-      mockExecSync.mockReturnValueOnce(JSON.stringify(['1.0.0', '1.1.0', '2.0.0']) as any);
-      const info = await publisher.getPackageInfo('@test/package');
-      expect(info.name).toBe('@test/package');
-      expect(info.exists).toBe(true);
-      expect(info.version).toBe('2.0.0');
-      expect(info.publishedVersions).toEqual(['1.0.0', '1.1.0', '2.0.0']);
-    });
-
-    it('should return default info for non-existent package', async () => {
-      mockExecSync.mockImplementationOnce(() => { throw new Error('Package not found'); });
-      const info = await publisher.getPackageInfo('@test/package');
-      expect(info.exists).toBe(false);
-      expect(info.version).toBe('0.0.0');
-    });
-
-    it('should handle single version response', async () => {
-      mockExecSync.mockReturnValueOnce('"1.0.0"' as any);
-      const info = await publisher.getPackageInfo('@test/package');
-      expect(info.publishedVersions).toEqual(['1.0.0']);
-    });
-  });
-
-  describe('validatePackage', () => {
-    const testPackage: PackagePublish = { name: '@test/package', version: '1.0.0', path: '/test/path', registry: 'https://registry.npmjs.org/', access: 'public' };
-
-    it('should validate package successfully', async () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(JSON.stringify({ name: '@test/package', version: '1.0.0' }));
-      const result = await publisher.validatePackage(testPackage);
-      expect(result.valid).toBe(true);
-    });
-
-    it('should fail validation when package.json is missing', async () => {
-      mockFs.existsSync.mockReturnValue(false);
-      const result = await publisher.validatePackage(testPackage);
-      expect(result.valid).toBe(false);
-      expect(result.errors).toContain('package.json not found at /test/path/package.json');
-    });
-
-    it('should fail validation when package name mismatches', async () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(JSON.stringify({ name: '@different/package', version: '1.0.0' }));
-      const result = await publisher.validatePackage(testPackage);
-      expect(result.valid).toBe(false);
-      expect(result.errors).toContain('Package name mismatch: expected @test/package, got @different/package');
-    });
-
-    it('should fail validation when version mismatches', async () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(JSON.stringify({ name: '@test/package', version: '2.0.0' }));
-      const result = await publisher.validatePackage(testPackage);
-      expect(result.valid).toBe(false);
-      expect(result.errors).toContain('Version mismatch: expected 1.0.0, got 2.0.0');
-    });
-
-    it('should fail validation for invalid semver', async () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(JSON.stringify({ name: '@test/package', version: 'invalid-version' }));
-      const result = await publisher.validatePackage({ ...testPackage, version: 'invalid-version' });
-      expect(result.valid).toBe(false);
-      expect(result.errors).toContain('Invalid semver version: invalid-version');
-    });
-  });
-
-  describe('publishPackage', () => {
-    const testPackage: PackagePublish = { name: '@test/package', version: '1.0.0', path: '/test/path', registry: 'https://registry.npmjs.org/', access: 'public' };
-
-    beforeEach(() => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(JSON.stringify({ name: '@test/package', version: '1.0.0' }));
-    });
-
-    it('should publish package successfully', async () => {
-      npmHelper.mockPublishSuccess('@test/package', '1.0.0');
-      const result = await publisher.publishPackage(testPackage);
-      expect(result.success).toBe(true);
-      expect(result.packageName).toBe('@test/package');
-      expect(result.version).toBe('1.0.0');
-    });
-
-    it('should fail when package version already exists', async () => {
-      npmHelper.mockPackageExists('@test/package', '1.0.0');
-      const result = await publisher.publishPackage(testPackage);
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('already exists in registry');
-    });
-
-    it('should fail when not authenticated', async () => {
-      jest.clearAllMocks();
-      const unauthPublisher = new NpmPublisher(config);
-      const unauthHelper = new NpmMockHelper(mockExecSync);
-      unauthHelper.mockAuthenticationFailure();
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(JSON.stringify({ name: '@test/package', version: '1.0.0' }));
-      const result = await unauthPublisher.publishPackage(testPackage);
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('Not authenticated');
-    });
-
-    it('should retry on publish failure', async () => {
-      jest.clearAllMocks();
-      const testHelper = new NpmMockHelper(mockExecSync);
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(JSON.stringify({ name: '@test/package', version: '1.0.0' }));
-      testHelper.mockPublishWithRetry('@test/package', '1.0.0', 1);
-      const result = await publisher.publishPackage(testPackage);
-      expect(result.success).toBe(true);
-    });
-
-    it('should include access flag in publish command', async () => {
-      mockExecSync.mockReset();
-      const freshPublisher = new NpmPublisher(config);
-      const testHelper = new NpmMockHelper(mockExecSync);
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(JSON.stringify({ name: '@test/package', version: '1.0.0' }));
-      testHelper.mockPublishSuccess('@test/package', '1.0.0');
-      await freshPublisher.publishPackage(testPackage);
-      const publishCall = mockExecSync.mock.calls.find((call) => call[0].toString().includes('publish') && call[0].toString().includes('--access'));
-      expect(publishCall).toBeDefined();
-      expect(publishCall![0]).toContain('--access public');
-    });
-  });
-
-  describe('publishPackages', () => {
-    const packages: PackagePublish[] = [
-      { name: '@test/package1', version: '1.0.0', path: '/test/path1', registry: 'https://registry.npmjs.org/', access: 'public' },
-      { name: '@test/package2', version: '1.0.0', path: '/test/path2', registry: 'https://registry.npmjs.org/', access: 'public' },
-    ];
-
-    beforeEach(() => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockImplementation((path) => {
-        if (path.toString().includes('path1')) return JSON.stringify({ name: '@test/package1', version: '1.0.0' });
-        return JSON.stringify({ name: '@test/package2', version: '1.0.0' });
-      });
-    });
-
-    it('should publish multiple packages successfully', async () => {
-      mockExecSync.mockReset();
-      const freshPublisher = new NpmPublisher(config);
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockImplementation((path) => {
-        const pathStr = path.toString();
-        if (pathStr.includes('path1')) return JSON.stringify({ name: '@test/package1', version: '1.0.0' });
-        if (pathStr.includes('path2')) return JSON.stringify({ name: '@test/package2', version: '1.0.0' });
-        return JSON.stringify({ name: '@test/unknown', version: '1.0.0' });
-      });
-      const testHelper = new NpmMockHelper(mockExecSync);
-      testHelper.mockPublishSuccess('@test/package1', '1.0.0');
-      mockExecSync.mockReturnValueOnce('' as any);
-      mockExecSync.mockReturnValueOnce(`+ @test/package2@1.0.0\n` as any);
-      const results = await freshPublisher.publishPackages(packages);
-      expect(results).toHaveLength(2);
-      expect(results[0].success).toBe(true);
-      expect(results[1].success).toBe(true);
-    });
-
-    it('should stop publishing on first failure', async () => {
-      jest.clearAllMocks();
-      mockExecSync.mockReturnValueOnce('testuser\n' as any);
-      mockExecSync.mockReturnValueOnce(JSON.stringify({ email: 'test@example.com' }) as any);
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockImplementation((path) => {
-        if (path.toString().includes('package1')) return JSON.stringify({ name: '@test/package1', version: '1.0.0' });
-        return JSON.stringify({ name: '@test/package2', version: '1.0.0' });
-      });
-      mockExecSync.mockReturnValueOnce('' as any);
-      mockExecSync.mockImplementationOnce(() => { throw new Error('Publish failed'); });
-      mockExecSync.mockImplementationOnce(() => { throw new Error('Publish failed'); });
-      mockExecSync.mockImplementationOnce(() => { throw new Error('Publish failed'); });
-      const results = await publisher.publishPackages(packages);
-      expect(results).toHaveLength(2);
-      expect(results[0].success).toBe(false);
-      expect(results[1].success).toBe(false);
-      expect(results[1].error).toContain('Publishing stopped due to previous failure');
-    });
-  });
-
-  describe('unpublishPackage', () => {
-    it('should unpublish package successfully', async () => {
-      jest.clearAllMocks();
-      const testHelper = new NpmMockHelper(mockExecSync);
-      testHelper.mockUnpublish('@test/package', '1.0.0');
-      const result = await publisher.unpublishPackage('@test/package', '1.0.0');
-      expect(result).toBe(true);
-    });
-
-    it('should fail when package version does not exist', async () => {
-      mockExecSync.mockReset();
-      mockExecSync.mockReturnValueOnce('' as any);
-      await expect(publisher.unpublishPackage('@test/package', '1.0.0')).rejects.toThrow('does not exist in registry');
-    });
-
-    it('should fail when unpublish command fails', async () => {
-      mockExecSync.mockReset();
-      const testHelper = new NpmMockHelper(mockExecSync);
-      testHelper.mockUnpublishFailure('Unpublish failed', '@test/package', '1.0.0');
-      await expect(publisher.unpublishPackage('@test/package', '1.0.0')).rejects.toThrow('Failed to unpublish package');
-    });
-  });
-
-  describe('dry run mode', () => {
-    it('should not actually publish in dry run mode', async () => {
-      jest.clearAllMocks();
-      const dryRunPublisher = new NpmPublisher({ ...config, dryRun: true });
-      const dryRunHelper = new NpmMockHelper(mockExecSync);
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(JSON.stringify({ name: '@test/package', version: '1.0.0' }));
-      dryRunHelper.mockPublishSuccess('@test/package', '1.0.0');
-      const result = await dryRunPublisher.publishPackage({ name: '@test/package', version: '1.0.0', path: '/test/path', registry: 'https://registry.npmjs.org/', access: 'public' });
-      expect(result.success).toBe(true);
-      const publishCall = mockExecSync.mock.calls.find((call) => call[0].toString().includes('publish'));
-      expect(publishCall![0]).toContain('--dry-run');
-    });
-  });
-});

package/src/tools/release/__tests__/PipelineIntegration.test.ts

@@ -1,188 +0,0 @@
-/**
- * @category evergreen
- * @purpose Pipeline integration test: fixture summary docs → release notes with mocked git
- */
-
-import { SummaryScanner } from '../pipeline/SummaryScanner';
-import { ChangeExtractor } from '../pipeline/ChangeExtractor';
-import { ChangeClassifier } from '../pipeline/ChangeClassifier';
-import { NotesRenderer } from '../pipeline/NotesRenderer';
-import { ExtractedChange, ClassifiedChange } from '../types';
-import * as childProcess from 'child_process';
-import * as fs from 'fs';
-
-jest.mock('child_process');
-jest.mock('fs');
-
-const mockedExecSync = childProcess.execSync as jest.MockedFunction<typeof childProcess.execSync>;
-const mockedReadFileSync = fs.readFileSync as jest.MockedFunction<typeof fs.readFileSync>;
-
-// --- Fixture summary docs ---
-
-const FIXTURE_TOKEN_SUMMARY = `# Task 1 Summary: New Color Token
-
-**Date**: 2026-02-20
-**Type**: Implementation
-**Spec**: 070-color-tokens
-
-## What Was Done
-
-Added a new semantic color token for error states.
-
-## Key Changes
-
-- Created color.error.surface token
-- Updated theme definitions
-
-## Why It Matters
-
-Consumers need a standard error surface color.
-
-## Impact
-
-- ✅ All themes updated
-- ⚠️ Existing hardcoded error colors should migrate
-
-## Deliverables *(optional)*
-
-- 🔴 Token: New color.error.surface semantic token
-- 🟡 Documentation: Updated token reference guide
-`;
-
-const FIXTURE_GOVERNANCE_SUMMARY = `# Task 2 Summary: Process Update
-
-**Date**: 2026-02-22
-**Type**: Documentation
-**Spec**: 071-governance
-
-## What Was Done
-
-Updated the spec planning process with new template.
-
-## Key Changes
-
-- Revised summary doc template
-- Added deliverables section guidance
-
-## Why It Matters
-
-Improves consistency across specs.
-
-## Impact
-
-- ✅ Template updated
-`;
-
-describe('Pipeline Integration', () => {
-  beforeEach(() => {
-    jest.useFakeTimers().setSystemTime(new Date('2026-02-27'));
-    jest.clearAllMocks();
-  });
-
-  afterEach(() => {
-    jest.useRealTimers();
-  });
-
-  const TAG_INFO = { tag: 'v7.0.0', commit: 'abc123', date: '2026-02-15' };
-
-  it('should produce release notes from fixture summary docs end-to-end', async () => {
-    // SummaryScanner: git log --diff-filter=A (encoding: utf-8 → returns string)
-    mockedExecSync.mockReturnValueOnce(
-      'docs/specs/070-color-tokens/task-1-summary.md\ndocs/specs/071-governance/task-2-summary.md\n',
-    );
-
-    mockedReadFileSync
-      .mockReturnValueOnce(FIXTURE_TOKEN_SUMMARY)
-      .mockReturnValueOnce(FIXTURE_GOVERNANCE_SUMMARY);
-
-    // --- Run pipeline stages ---
-    const scanner = new SummaryScanner();
-    const summaries = await scanner.findSummariesSinceTag(TAG_INFO);
-
-    const extractor = new ChangeExtractor();
-    const extracted: ExtractedChange[] = summaries
-      .map((s) => extractor.extract(s))
-      .filter((e): e is ExtractedChange => e !== undefined);
-
-    const classifier = new ChangeClassifier();
-    const classified: ClassifiedChange[] = classifier.classify(extracted);
-
-    const renderer = new NotesRenderer();
-    const notes = renderer.render({
-      currentVersion: '7.0.0',
-      recommendedVersion: '8.0.0',
-      bumpType: 'major',
-      rationale: 'Breaking changes detected',
-      confidence: 0.9,
-      changes: classified,
-    });
-
-    // --- Verify public notes ---
-    expect(notes.public).toContain('# Release 8.0.0');
-    expect(notes.public).toContain('🔴 Breaking / Consumer-Facing');
-    expect(notes.public).toContain('New Color Token');
-    // Context-only items excluded from public
-    expect(notes.public).not.toContain('🔵 Internal');
-
-    // --- Verify internal notes ---
-    expect(notes.internal).toContain('New Color Token');
-    expect(notes.internal).toContain('Process Update');
-
-    // --- Verify JSON structure ---
-    expect(notes.json.version).toBe('8.0.0');
-    expect(notes.json.previousVersion).toBe('7.0.0');
-    expect(notes.json.changes).toHaveLength(2);
-    expect(notes.json.changes[0].priority).toBe('breaking');
-    expect(notes.json.changes[1].priority).toBe('context');
-  });
-
-  it('should handle pipeline with no summaries found', async () => {
-    mockedExecSync.mockReturnValueOnce('\n');
-
-    const scanner = new SummaryScanner();
-    const summaries = await scanner.findSummariesSinceTag(TAG_INFO);
-
-    const extractor = new ChangeExtractor();
-    const extracted = summaries
-      .map((s) => extractor.extract(s))
-      .filter((e): e is ExtractedChange => e !== undefined);
-
-    const classifier = new ChangeClassifier();
-    const classified = classifier.classify(extracted);
-
-    const renderer = new NotesRenderer();
-    const notes = renderer.render({
-      currentVersion: '7.0.0',
-      recommendedVersion: '7.0.0',
-      bumpType: 'none',
-      rationale: 'No changes',
-      confidence: 1.0,
-      changes: classified,
-    });
-
-    expect(notes.public).toContain('No consumer-facing changes');
-    expect(notes.json.changes).toHaveLength(0);
-  });
-
-  it('should classify by keyword when deliverables are absent', async () => {
-    mockedExecSync.mockReturnValueOnce('docs/specs/071-governance/task-2-summary.md\n');
-
-    mockedReadFileSync.mockReturnValueOnce(FIXTURE_GOVERNANCE_SUMMARY);
-
-    const scanner = new SummaryScanner();
-    const summaries = await scanner.findSummariesSinceTag(TAG_INFO);
-
-    const extractor = new ChangeExtractor();
-    const extracted = summaries
-      .map((s) => extractor.extract(s))
-      .filter((e): e is ExtractedChange => e !== undefined);
-
-    const classifier = new ChangeClassifier();
-    const classified = classifier.classify(extracted);
-
-    // "process" keyword → context
-    expect(classified).toHaveLength(1);
-    expect(classified[0].priority).toBe('context');
-    expect(classified[0].deliverableType).toBe('Process');
-  });
-});

package/src/tools/release/__tests__/ReleasePipeline.test.ts

@@ -1,192 +0,0 @@
-/**
- * @category evergreen
- * @purpose Verify ReleasePipeline orchestrates pipeline components correctly
- */
-
-import { ReleasePipeline } from '../cli/ReleasePipeline';
-import * as childProcess from 'child_process';
-import * as fs from 'fs';
-
-jest.mock('child_process');
-jest.mock('fs');
-
-const mockedExecSync = childProcess.execSync as jest.MockedFunction<typeof childProcess.execSync>;
-const mockedReadFileSync = fs.readFileSync as jest.MockedFunction<typeof fs.readFileSync>;
-const mockedExistsSync = fs.existsSync as jest.MockedFunction<typeof fs.existsSync>;
-const mockedWriteFileSync = fs.writeFileSync as jest.MockedFunction<typeof fs.writeFileSync>;
-const mockedMkdirSync = fs.mkdirSync as jest.MockedFunction<typeof fs.mkdirSync>;
-const mockedReaddirSync = fs.readdirSync as jest.MockedFunction<typeof fs.readdirSync>;
-
-const FIXTURE_SUMMARY = `# Task 1 Summary: New Token
-
-**Date**: 2026-02-20
-**Type**: Implementation
-**Spec**: 070-tokens
-
-## What Was Done
-
-Added a new token.
-
-## Key Changes
-
-- Created new token
-
-## Why It Matters
-
-Consumers need it.
-
-## Impact
-
-- ✅ Done
-
-## Deliverables *(optional)*
-
-- 🔴 Token: New semantic token
-`;
-
-const CONFIG = JSON.stringify({ npmPublishEnabled: false, repoUrl: '', outputDir: 'docs/releases' });
-
-describe('ReleasePipeline', () => {
-  beforeEach(() => {
-    jest.useFakeTimers().setSystemTime(new Date('2026-02-27'));
-    jest.clearAllMocks();
-    // Default: config file read returns config
-    mockedReadFileSync.mockImplementation((p: any) => {
-      if (String(p).includes('release-config.json')) return CONFIG;
-      return FIXTURE_SUMMARY;
-    });
-    mockedExistsSync.mockReturnValue(true);
-  });
-
-  afterEach(() => {
-    jest.useRealTimers();
-  });
-
-  function mockTagAndSummaries(summaryPaths: string[]): void {
-    // TagResolver: describe, rev-list, log (all return strings via encoding: utf-8)
-    mockedExecSync
-      .mockReturnValueOnce('v7.0.0\n')
-      .mockReturnValueOnce('abc123\n')
-      .mockReturnValueOnce('2026-02-15\n')
-      // SummaryScanner: git log
-      .mockReturnValueOnce(summaryPaths.join('\n') + '\n');
-  }
-
-  describe('analyze', () => {
-    it('should return recommendation with changes', async () => {
-      mockTagAndSummaries(['docs/specs/070-tokens/task-1-summary.md']);
-      const pipeline = new ReleasePipeline();
-      const rec = await pipeline.analyze();
-
-      expect(rec.currentVersion).toBe('7.0.0');
-      expect(rec.bumpType).toBe('major');
-      expect(rec.changes).toHaveLength(1);
-      expect(rec.changes[0].priority).toBe('breaking');
-    });
-
-    it('should return none bump when no summaries', async () => {
-      mockTagAndSummaries([]);
-      const pipeline = new ReleasePipeline();
-      const rec = await pipeline.analyze();
-
-      expect(rec.bumpType).toBe('none');
-      expect(rec.changes).toHaveLength(0);
-    });
-  });
-
-  describe('generateNotes', () => {
-    it('should write files to output directory', async () => {
-      mockTagAndSummaries(['docs/specs/070-tokens/task-1-summary.md']);
-      const pipeline = new ReleasePipeline();
-      await pipeline.generateNotes();
-
-      expect(mockedMkdirSync).toHaveBeenCalled();
-      expect(mockedWriteFileSync).toHaveBeenCalledTimes(3); // public, internal, json
-    });
-  });
-
-  describe('checkAccumulation', () => {
-    it('should warn when >5 files in output dir', () => {
-      mockedReaddirSync.mockReturnValue(['a', 'b', 'c', 'd', 'e', 'f'] as any);
-      const pipeline = new ReleasePipeline();
-      const warning = pipeline.checkAccumulation();
-
-      expect(warning).toContain('6 files');
-    });
-
-    it('should return null when <=5 files', () => {
-      mockedReaddirSync.mockReturnValue(['a', 'b'] as any);
-      const pipeline = new ReleasePipeline();
-      expect(pipeline.checkAccumulation()).toBeNull();
-    });
-  });
-
-  describe('release', () => {
-    it('should return notes without tagging or publishing in dry-run mode', async () => {
-      mockTagAndSummaries(['docs/specs/070-tokens/task-1-summary.md']);
-      const pipeline = new ReleasePipeline();
-      const result = await pipeline.release(true);
-
-      expect(result.published).toBe(false);
-      expect(result.notes.json.version).toBe('8.0.0');
-      // generateNotes writes 3 files, but no tag creation call beyond that
-      const execCalls = mockedExecSync.mock.calls;
-      const tagCalls = execCalls.filter((c) => String(c[0]).includes('git tag'));
-      expect(tagCalls).toHaveLength(0);
-    });
-
-    it('should create git tag when not dry-run', async () => {
-      // First call: analyze (tag + scanner)
-      mockTagAndSummaries(['docs/specs/070-tokens/task-1-summary.md']);
-      // Second call inside release: generateNotes calls analyze again
-      mockedExecSync
-        .mockReturnValueOnce('v7.0.0\n')
-        .mockReturnValueOnce('abc123\n')
-        .mockReturnValueOnce('2026-02-15\n')
-        .mockReturnValueOnce('docs/specs/070-tokens/task-1-summary.md\n')
-        // createTag call
-        .mockReturnValueOnce('');
-
-      const pipeline = new ReleasePipeline();
-      const result = await pipeline.release(false);
-
-      const tagCalls = mockedExecSync.mock.calls.filter((c) => String(c[0]).includes('git tag'));
-      expect(tagCalls).toHaveLength(1);
-      expect(String(tagCalls[0][0])).toContain('v8.0.0');
-      // No repoUrl configured, so not published
-      expect(result.published).toBe(false);
-      expect(result.error).toContain('No repoUrl configured');
-    });
-
-    it('should skip GitHub publish when GITHUB_TOKEN is not set', async () => {
-      const configWithRepo = JSON.stringify({ npmPublishEnabled: false, repoUrl: 'https://github.com/3fn/DesignerPunk', outputDir: 'docs/releases' });
-      mockedReadFileSync.mockImplementation((p: any) => {
-        if (String(p).includes('release-config.json')) return configWithRepo;
-        return FIXTURE_SUMMARY;
-      });
-
-      // analyze pass
-      mockTagAndSummaries(['docs/specs/070-tokens/task-1-summary.md']);
-      // generateNotes calls analyze again
-      mockedExecSync
-        .mockReturnValueOnce('v7.0.0\n')
-        .mockReturnValueOnce('abc123\n')
-        .mockReturnValueOnce('2026-02-15\n')
-        .mockReturnValueOnce('docs/specs/070-tokens/task-1-summary.md\n')
-        .mockReturnValueOnce(''); // createTag
-
-      const originalToken = process.env.GITHUB_TOKEN;
-      delete process.env.GITHUB_TOKEN;
-
-      try {
-        const pipeline = new ReleasePipeline();
-        const result = await pipeline.release(false);
-
-        expect(result.published).toBe(false);
-        expect(result.error).toContain('GITHUB_TOKEN not set');
-      } finally {
-        if (originalToken) process.env.GITHUB_TOKEN = originalToken;
-      }
-    });
-  });
-});