@3fn/core 14.0.0 → 15.0.0

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

@@ -1,133 +0,0 @@
-/**
- * @category evergreen
- * @purpose Verify ChangeClassifier maps changes to correct priority tiers
- */
-
-import { ChangeClassifier } from '../pipeline/ChangeClassifier';
-import { ExtractedChange } from '../types';
-
-describe('ChangeClassifier', () => {
-  let classifier: ChangeClassifier;
-
-  beforeEach(() => {
-    classifier = new ChangeClassifier();
-  });
-
-  const makeChange = (overrides: Partial<ExtractedChange> = {}): ExtractedChange => ({
-    specName: '065-test',
-    taskTitle: 'Test task',
-    taskType: 'Implementation',
-    whatWasDone: 'Did something',
-    whyItMatters: 'It matters',
-    keyChanges: ['change 1'],
-    impact: ['impact 1'],
-    ...overrides,
-  });
-
-  describe('structured deliverables path', () => {
-    it('should use deliverable priority directly', () => {
-      const change = makeChange({
-        deliverables: [{ priority: '🔴', type: 'Component', description: 'New ButtonCTA' }],
-      });
-
-      const results = classifier.classify([change]);
-
-      expect(results[0].priority).toBe('breaking');
-      expect(results[0].deliverableType).toBe('Component');
-    });
-
-    it('should use highest priority when multiple deliverables', () => {
-      const change = makeChange({
-        deliverables: [
-          { priority: '🔵', type: 'Governance', description: 'Process update' },
-          { priority: '🔴', type: 'Token', description: 'New color token' },
-          { priority: '🟡', type: 'Tool', description: 'Build change' },
-        ],
-      });
-
-      const results = classifier.classify([change]);
-
-      expect(results[0].priority).toBe('breaking');
-      expect(results[0].deliverableType).toBe('Token');
-    });
-
-    it('should map all three emoji priorities', () => {
-      const changes = [
-        makeChange({ deliverables: [{ priority: '🔴', type: 'Token', description: 'x' }] }),
-        makeChange({ deliverables: [{ priority: '🟡', type: 'Tool', description: 'x' }] }),
-        makeChange({ deliverables: [{ priority: '🔵', type: 'Governance', description: 'x' }] }),
-      ];
-
-      const results = classifier.classify(changes);
-
-      expect(results[0].priority).toBe('breaking');
-      expect(results[1].priority).toBe('prominent');
-      expect(results[2].priority).toBe('context');
-    });
-  });
-
-  describe('keyword heuristics fallback', () => {
-    it('should classify token changes as breaking', () => {
-      const change = makeChange({ taskTitle: 'Add new color token' });
-      const results = classifier.classify([change]);
-      expect(results[0].priority).toBe('breaking');
-      expect(results[0].deliverableType).toBe('Token');
-    });
-
-    it('should classify component changes as breaking', () => {
-      const change = makeChange({ whatWasDone: 'Created new component scaffold' });
-      const results = classifier.classify([change]);
-      expect(results[0].priority).toBe('breaking');
-      expect(results[0].deliverableType).toBe('Component');
-    });
-
-    it('should classify tool changes as prominent', () => {
-      const change = makeChange({ taskTitle: 'Build tool improvements' });
-      const results = classifier.classify([change]);
-      expect(results[0].priority).toBe('prominent');
-    });
-
-    it('should classify governance changes as context', () => {
-      const change = makeChange({ keyChanges: ['Updated governance documentation'] });
-      const results = classifier.classify([change]);
-      expect(results[0].priority).toBe('context');
-    });
-
-    it('should default to context when no keywords match', () => {
-      const change = makeChange({ taskTitle: 'Miscellaneous cleanup', whatWasDone: 'Cleaned up files', keyChanges: ['removed old stuff'] });
-      const results = classifier.classify([change]);
-      expect(results[0].priority).toBe('context');
-      expect(results[0].deliverableType).toBe('Other');
-    });
-  });
-
-  describe('sorting', () => {
-    it('should sort by priority: breaking > prominent > context', () => {
-      const changes = [
-        makeChange({ taskTitle: 'Governance update', deliverables: [{ priority: '🔵', type: 'Governance', description: 'x' }] }),
-        makeChange({ taskTitle: 'New token', deliverables: [{ priority: '🔴', type: 'Token', description: 'x' }] }),
-        makeChange({ taskTitle: 'New tool', deliverables: [{ priority: '🟡', type: 'Tool', description: 'x' }] }),
-      ];
-
-      const results = classifier.classify(changes);
-
-      expect(results[0].priority).toBe('breaking');
-      expect(results[1].priority).toBe('prominent');
-      expect(results[2].priority).toBe('context');
-    });
-  });
-
-  describe('mixed structured and heuristic', () => {
-    it('should handle mix of deliverable and no-deliverable changes', () => {
-      const changes = [
-        makeChange({ taskTitle: 'Misc cleanup', whatWasDone: 'Cleaned things', keyChanges: ['stuff'] }),
-        makeChange({ deliverables: [{ priority: '🔴', type: 'Component', description: 'New component' }] }),
-      ];
-
-      const results = classifier.classify(changes);
-
-      expect(results[0].priority).toBe('breaking');
-      expect(results[1].priority).toBe('context');
-    });
-  });
-});

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

@@ -1,222 +0,0 @@
-/**
- * @category evergreen
- * @purpose Verify ChangeExtractor parses summary docs correctly
- */
-
-import { ChangeExtractor } from '../pipeline/ChangeExtractor';
-import { SummaryDoc } from '../types';
-
-describe('ChangeExtractor', () => {
-  let extractor: ChangeExtractor;
-
-  beforeEach(() => {
-    extractor = new ChangeExtractor();
-  });
-
-  const makeSummaryDoc = (raw: string, overrides: Partial<SummaryDoc> = {}): SummaryDoc => ({
-    path: 'docs/specs/065-release-system-rebuild/task-1-summary.md',
-    specName: '065-release-system-rebuild',
-    taskNumber: 1,
-    raw,
-    ...overrides,
-  });
-
-  const fullDoc = makeSummaryDoc(`# Task 1 Summary: Summary Format Enhancement
-
-**Date**: 2026-02-27
-**Spec**: 065-release-system-rebuild
-**Type**: Architecture
-
----
-
-## What Was Done
-
-Added an optional Deliverables section to the summary template.
-
-## Why It Matters
-
-The release tool needs structured deliverable classification.
-
-## Key Changes
-
-- Added Deliverables section to summary template
-- Added guidance note explaining three tiers
-- Updated example summary
-
-## Impact
-
-- ✅ Release tool gets structured data
-- ✅ Fallback to section-based extraction when absent
-- ✅ Minimal per-task overhead
-
-## Deliverables *(optional)*
-
-- 🔵 Governance: Deliverables field added to summary template
-- 🟡 Tool: Release tool scaffold created
-
----
-
-*For detailed notes, see completion doc*
-`);
-
-  describe('full document parsing', () => {
-    it('should extract all sections', () => {
-      const result = extractor.extract(fullDoc);
-
-      expect(result.specName).toBe('065-release-system-rebuild');
-      expect(result.taskTitle).toBe('Summary Format Enhancement');
-      expect(result.taskType).toBe('Architecture');
-      expect(result.whatWasDone).toContain('Added an optional Deliverables section');
-      expect(result.whyItMatters).toContain('structured deliverable classification');
-      expect(result.keyChanges).toHaveLength(3);
-      expect(result.keyChanges[0]).toBe('Added Deliverables section to summary template');
-      expect(result.impact).toHaveLength(3);
-      expect(result.impact[0]).toBe('Release tool gets structured data');
-    });
-
-    it('should extract deliverables with correct priority and type', () => {
-      const result = extractor.extract(fullDoc);
-
-      expect(result.deliverables).toHaveLength(2);
-      expect(result.deliverables![0]).toEqual({
-        priority: '🔵',
-        type: 'Governance',
-        description: 'Deliverables field added to summary template',
-      });
-      expect(result.deliverables![1]).toEqual({
-        priority: '🟡',
-        type: 'Tool',
-        description: 'Release tool scaffold created',
-      });
-    });
-  });
-
-  describe('document without deliverables', () => {
-    it('should return undefined deliverables when section missing', () => {
-      const doc = makeSummaryDoc(`# Task 2 Summary: Some Task
-
-**Type**: Implementation
-
-## What Was Done
-
-Did something useful.
-
-## Why It Matters
-
-It matters a lot.
-
-## Key Changes
-
-- Change one
-- Change two
-
-## Impact
-
-- ✅ Good outcome
-`);
-
-      const result = extractor.extract(doc);
-
-      expect(result.deliverables).toBeUndefined();
-      expect(result.taskTitle).toBe('Some Task');
-      expect(result.taskType).toBe('Implementation');
-      expect(result.keyChanges).toHaveLength(2);
-      expect(result.impact).toHaveLength(1);
-    });
-  });
-
-  describe('missing sections', () => {
-    it('should handle minimal document gracefully', () => {
-      const doc = makeSummaryDoc(`# Task 3 Summary: Minimal
-
-**Type**: Setup
-
-## What Was Done
-
-Just this.
-`);
-
-      const result = extractor.extract(doc);
-
-      expect(result.taskTitle).toBe('Minimal');
-      expect(result.taskType).toBe('Setup');
-      expect(result.whatWasDone).toBe('Just this.');
-      expect(result.whyItMatters).toBe('');
-      expect(result.keyChanges).toHaveLength(0);
-      expect(result.impact).toHaveLength(0);
-      expect(result.deliverables).toBeUndefined();
-    });
-
-    it('should handle empty document', () => {
-      const doc = makeSummaryDoc('');
-
-      const result = extractor.extract(doc);
-
-      expect(result.taskTitle).toBe('');
-      expect(result.taskType).toBe('Unknown');
-      expect(result.whatWasDone).toBe('');
-    });
-  });
-
-  describe('deliverable parsing edge cases', () => {
-    it('should parse all three priority levels', () => {
-      const doc = makeSummaryDoc(`# Task 4 Summary: Multi-tier
-
-**Type**: Architecture
-
-## Deliverables *(optional)*
-
-- 🔴 Component: New ButtonCTA component
-- 🟡 Build: Updated build pipeline
-- 🔵 Governance: Process documentation
-`);
-
-      const result = extractor.extract(doc);
-
-      expect(result.deliverables).toHaveLength(3);
-      expect(result.deliverables![0].priority).toBe('🔴');
-      expect(result.deliverables![1].priority).toBe('🟡');
-      expect(result.deliverables![2].priority).toBe('🔵');
-    });
-
-    it('should skip malformed deliverable lines', () => {
-      const doc = makeSummaryDoc(`# Task 5 Summary: Mixed
-
-**Type**: Implementation
-
-## Deliverables *(optional)*
-
-- 🔴 Token: Valid entry
-- This is not a deliverable
-- 🟡 Tool: Another valid entry
-`);
-
-      const result = extractor.extract(doc);
-
-      expect(result.deliverables).toHaveLength(2);
-    });
-  });
-
-  describe('impact line cleaning', () => {
-    it('should strip emoji prefixes from impact lines', () => {
-      const doc = makeSummaryDoc(`# Task 6 Summary: Emoji
-
-**Type**: Implementation
-
-## Impact
-
-- ✅ Good thing happened
-- ⚠️ Watch out for this
-- ❌ This broke
-`);
-
-      const result = extractor.extract(doc);
-
-      expect(result.impact).toEqual([
-        'Good thing happened',
-        'Watch out for this',
-        'This broke',
-      ]);
-    });
-  });
-});

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

@@ -1,240 +0,0 @@
-/**
- * @category evergreen
- * @purpose Verify GitHub publisher handles package publishing correctly
- */
-
-import { GitHubPublisher, GitHubConfig } from '../publishers/GitHubPublisher';
-import { GitHubRelease, GitTag, Artifact } from '../types';
-import * as fs from 'fs';
-
-jest.mock('@octokit/rest');
-const { Octokit } = require('@octokit/rest');
-
-jest.mock('fs');
-const mockFs = fs as jest.Mocked<typeof fs>;
-
-describe('GitHubPublisher', () => {
-  let publisher: GitHubPublisher;
-  let mockOctokit: any;
-  let config: GitHubConfig;
-
-  beforeEach(() => {
-    jest.clearAllMocks();
-    mockOctokit = {
-      rest: {
-        users: { getAuthenticated: jest.fn() },
-        repos: { get: jest.fn(), getReleaseByTag: jest.fn(), createRelease: jest.fn(), deleteRelease: jest.fn(), uploadReleaseAsset: jest.fn() },
-        git: { getCommit: jest.fn(), createTag: jest.fn(), createRef: jest.fn(), deleteRef: jest.fn() },
-      },
-    };
-    Octokit.mockImplementation(() => mockOctokit);
-    config = { token: 'test-token', owner: 'test-owner', repo: 'test-repo' };
-    publisher = new GitHubPublisher(config);
-  });
-
-  describe('Authentication', () => {
-    it('should validate authentication successfully', async () => {
-      mockOctokit.rest.users.getAuthenticated.mockResolvedValue({ data: { login: 'test-user' } });
-      const result = await publisher.validateAuthentication();
-      expect(result).toBe(true);
-      expect(mockOctokit.rest.users.getAuthenticated).toHaveBeenCalled();
-    });
-
-    it('should throw error on authentication failure', async () => {
-      mockOctokit.rest.users.getAuthenticated.mockRejectedValue(new Error('Invalid token'));
-      await expect(publisher.validateAuthentication()).rejects.toThrow('GitHub authentication failed');
-    });
-  });
-
-  describe('Repository Information', () => {
-    it('should get repository information', async () => {
-      mockOctokit.rest.repos.get.mockResolvedValue({
-        data: { owner: { login: 'test-owner' }, name: 'test-repo', full_name: 'test-owner/test-repo', default_branch: 'main', private: false, html_url: 'https://github.com/test-owner/test-repo' },
-      });
-      const info = await publisher.getRepositoryInfo();
-      expect(info).toEqual({ owner: 'test-owner', name: 'test-repo', fullName: 'test-owner/test-repo', defaultBranch: 'main', private: false, url: 'https://github.com/test-owner/test-repo' });
-    });
-
-    it('should throw error when repository not found', async () => {
-      mockOctokit.rest.repos.get.mockRejectedValue(new Error('Not Found'));
-      await expect(publisher.getRepositoryInfo()).rejects.toThrow('Failed to get repository info');
-    });
-  });
-
-  describe('Release Existence Check', () => {
-    it('should return true when release exists', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockResolvedValue({ data: { id: 123, tag_name: 'v1.0.0' } });
-      expect(await publisher.releaseExists('v1.0.0')).toBe(true);
-    });
-
-    it('should return false when release does not exist', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockRejectedValue({ status: 404, message: 'Not Found' });
-      expect(await publisher.releaseExists('v1.0.0')).toBe(false);
-    });
-
-    it('should throw error on non-404 errors', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockRejectedValue(new Error('Server error'));
-      await expect(publisher.releaseExists('v1.0.0')).rejects.toThrow('Failed to check if release exists');
-    });
-  });
-
-  describe('Tag Creation', () => {
-    it('should create tags successfully', async () => {
-      const tags: GitTag[] = [{ name: 'v1.0.0', message: 'Release v1.0.0', commit: 'abc123' }];
-      mockOctokit.rest.git.getCommit.mockResolvedValue({ data: { sha: 'abc123' } });
-      mockOctokit.rest.git.createTag.mockResolvedValue({ data: { sha: 'def456' } });
-      mockOctokit.rest.git.createRef.mockResolvedValue({ data: { ref: 'refs/tags/v1.0.0' } });
-      const results = await publisher.createTags(tags);
-      expect(results).toHaveLength(1);
-      expect(results[0]).toEqual({ success: true, tagName: 'v1.0.0' });
-    });
-
-    it('should handle tag creation failures', async () => {
-      mockOctokit.rest.git.getCommit.mockRejectedValue(new Error('Commit not found'));
-      const results = await publisher.createTags([{ name: 'v1.0.0', message: 'Release v1.0.0', commit: 'abc123' }]);
-      expect(results[0]).toEqual({ success: false, tagName: 'v1.0.0', error: 'Commit not found' });
-    });
-
-    it('should create multiple tags', async () => {
-      mockOctokit.rest.git.getCommit.mockResolvedValue({ data: { sha: 'abc123' } });
-      mockOctokit.rest.git.createTag.mockResolvedValue({ data: { sha: 'tag-sha' } });
-      mockOctokit.rest.git.createRef.mockResolvedValue({ data: { ref: 'refs/tags/v1.0.0' } });
-      const results = await publisher.createTags([
-        { name: 'v1.0.0', message: 'Release v1.0.0', commit: 'abc123' },
-        { name: 'v1.0.1', message: 'Release v1.0.1', commit: 'def456' },
-      ]);
-      expect(results).toHaveLength(2);
-      expect(results.every((r) => r.success)).toBe(true);
-    });
-  });
-
-  describe('Release Creation', () => {
-    beforeEach(() => {
-      mockOctokit.rest.users.getAuthenticated.mockResolvedValue({ data: { login: 'test-user' } });
-    });
-
-    it('should create release successfully', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockRejectedValue({ status: 404 });
-      mockOctokit.rest.repos.createRelease.mockResolvedValue({
-        data: { id: 123, tag_name: 'v1.0.0', html_url: 'https://github.com/test-owner/test-repo/releases/tag/v1.0.0' },
-      });
-      const result = await publisher.createRelease({ tagName: 'v1.0.0', name: 'Release v1.0.0', body: '## Changes\n- Feature 1', draft: false, prerelease: false, artifacts: [] });
-      expect(result.success).toBe(true);
-      expect(result.version).toBe('v1.0.0');
-      expect(result.githubReleaseUrl).toBe('https://github.com/test-owner/test-repo/releases/tag/v1.0.0');
-      expect(result.errors).toHaveLength(0);
-    });
-
-    it('should fail when release already exists', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockResolvedValue({ data: { id: 123 } });
-      const result = await publisher.createRelease({ tagName: 'v1.0.0', name: 'Release v1.0.0', body: 'Release notes', draft: false, prerelease: false, artifacts: [] });
-      expect(result.success).toBe(false);
-      expect(result.errors[0].message).toContain('already exists');
-    });
-
-    it('should create draft release', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockRejectedValue({ status: 404 });
-      mockOctokit.rest.repos.createRelease.mockResolvedValue({ data: { id: 123, tag_name: 'v1.0.0', html_url: 'https://github.com/test-owner/test-repo/releases/tag/v1.0.0' } });
-      const result = await publisher.createRelease({ tagName: 'v1.0.0', name: 'Release v1.0.0', body: 'Release notes', draft: true, prerelease: false, artifacts: [] });
-      expect(result.success).toBe(true);
-      expect(mockOctokit.rest.repos.createRelease).toHaveBeenCalledWith(expect.objectContaining({ draft: true }));
-    });
-
-    it('should create prerelease', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockRejectedValue({ status: 404 });
-      mockOctokit.rest.repos.createRelease.mockResolvedValue({ data: { id: 123, tag_name: 'v1.0.0-beta.1', html_url: 'https://github.com/test-owner/test-repo/releases/tag/v1.0.0-beta.1' } });
-      const result = await publisher.createRelease({ tagName: 'v1.0.0-beta.1', name: 'Release v1.0.0-beta.1', body: 'Beta release', draft: false, prerelease: true, artifacts: [] });
-      expect(result.success).toBe(true);
-      expect(mockOctokit.rest.repos.createRelease).toHaveBeenCalledWith(expect.objectContaining({ prerelease: true }));
-    });
-
-    it('should handle release creation errors', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockRejectedValue({ status: 404 });
-      mockOctokit.rest.repos.createRelease.mockRejectedValue(new Error('API error'));
-      const result = await publisher.createRelease({ tagName: 'v1.0.0', name: 'Release v1.0.0', body: 'Release notes', draft: false, prerelease: false, artifacts: [] });
-      expect(result.success).toBe(false);
-      expect(result.errors[0].code).toBe('RELEASE_CREATION_FAILED');
-    });
-  });
-
-  describe('Artifact Upload', () => {
-    beforeEach(() => { jest.clearAllMocks(); });
-
-    it('should upload artifacts successfully', async () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(Buffer.from('test content'));
-      mockOctokit.rest.repos.uploadReleaseAsset.mockResolvedValue({ data: { browser_download_url: 'https://github.com/test-owner/test-repo/releases/download/v1.0.0/build.zip' } });
-      const results = await publisher.uploadArtifacts('123', [{ name: 'build.zip', path: '/path/to/build.zip', contentType: 'application/zip', size: 1024 }]);
-      expect(results).toHaveLength(1);
-      expect(results[0]).toEqual({ success: true, artifactName: 'build.zip', url: 'https://github.com/test-owner/test-repo/releases/download/v1.0.0/build.zip' });
-    });
-
-    it('should handle missing artifact files', async () => {
-      mockFs.existsSync.mockReturnValue(false);
-      const results = await publisher.uploadArtifacts('123', [{ name: 'missing.zip', path: '/path/to/missing.zip', contentType: 'application/zip', size: 1024 }]);
-      expect(results[0]).toEqual({ success: false, artifactName: 'missing.zip', error: expect.stringContaining('not found') });
-    });
-
-    it('should handle upload failures', async () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(Buffer.from('test content'));
-      mockOctokit.rest.repos.uploadReleaseAsset.mockRejectedValue(new Error('Upload failed'));
-      const results = await publisher.uploadArtifacts('123', [{ name: 'build.zip', path: '/path/to/build.zip', contentType: 'application/zip', size: 1024 }]);
-      expect(results[0]).toEqual({ success: false, artifactName: 'build.zip', error: 'Upload failed' });
-    });
-
-    it('should upload multiple artifacts', async () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockReturnValue(Buffer.from('test content'));
-      mockOctokit.rest.repos.uploadReleaseAsset.mockResolvedValue({ data: { browser_download_url: 'https://github.com/test-owner/test-repo/releases/download/v1.0.0/build.zip' } });
-      const results = await publisher.uploadArtifacts('123', [
-        { name: 'build1.zip', path: '/path/to/build1.zip', contentType: 'application/zip', size: 1024 },
-        { name: 'build2.zip', path: '/path/to/build2.zip', contentType: 'application/zip', size: 2048 },
-      ]);
-      expect(results).toHaveLength(2);
-      expect(results.every((r) => r.success)).toBe(true);
-    });
-  });
-
-  describe('Rollback Operations', () => {
-    it('should delete release successfully', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockResolvedValue({ data: { id: 123 } });
-      mockOctokit.rest.repos.deleteRelease.mockResolvedValue({});
-      expect(await publisher.deleteRelease('v1.0.0')).toBe(true);
-      expect(mockOctokit.rest.repos.deleteRelease).toHaveBeenCalledWith({ owner: 'test-owner', repo: 'test-repo', release_id: 123 });
-    });
-
-    it('should handle release deletion errors', async () => {
-      mockOctokit.rest.repos.getReleaseByTag.mockRejectedValue(new Error('Release not found'));
-      await expect(publisher.deleteRelease('v1.0.0')).rejects.toThrow('Failed to delete release');
-    });
-
-    it('should delete tag successfully', async () => {
-      mockOctokit.rest.git.deleteRef.mockResolvedValue({});
-      expect(await publisher.deleteTag('v1.0.0')).toBe(true);
-      expect(mockOctokit.rest.git.deleteRef).toHaveBeenCalledWith({ owner: 'test-owner', repo: 'test-repo', ref: 'tags/v1.0.0' });
-    });
-
-    it('should handle tag deletion errors', async () => {
-      mockOctokit.rest.git.deleteRef.mockRejectedValue(new Error('Tag not found'));
-      await expect(publisher.deleteTag('v1.0.0')).rejects.toThrow('Failed to delete tag');
-    });
-  });
-
-  describe('Configuration', () => {
-    it('should use default timeout', () => {
-      new GitHubPublisher(config);
-      expect(Octokit).toHaveBeenCalledWith(expect.objectContaining({ request: expect.objectContaining({ timeout: 30000 }) }));
-    });
-
-    it('should use custom timeout', () => {
-      new GitHubPublisher({ ...config, timeout: 60000 });
-      expect(Octokit).toHaveBeenCalledWith(expect.objectContaining({ request: expect.objectContaining({ timeout: 60000 }) }));
-    });
-
-    it('should use custom base URL for GitHub Enterprise', () => {
-      new GitHubPublisher({ ...config, baseUrl: 'https://github.enterprise.com/api/v3' });
-      expect(Octokit).toHaveBeenCalledWith(expect.objectContaining({ baseUrl: 'https://github.enterprise.com/api/v3' }));
-    });
-  });
-});