@dependabit/github-client 0.1.13 → 0.1.15

package/test/auth/token.test.ts

@@ -1,97 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { TokenAuthHandler } from '../../src/auth/token';
-
-describe('TokenAuthHandler', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('constructor', () => {
-    it('should create handler with GitHub PAT token', () => {
-      const token = 'ghp_testtoken123';
-      const handler = new TokenAuthHandler(token);
-      expect(handler).toBeInstanceOf(TokenAuthHandler);
-    });
-
-    it('should create handler with API key', () => {
-      const token = 'api_key_12345';
-      const handler = new TokenAuthHandler(token);
-      expect(handler).toBeInstanceOf(TokenAuthHandler);
-    });
-
-    it('should throw error for empty token', () => {
-      expect(() => new TokenAuthHandler('')).toThrow('Token cannot be empty');
-    });
-  });
-
-  describe('authenticate', () => {
-    it('should return auth object with token', async () => {
-      const token = 'ghp_testtoken123';
-      const handler = new TokenAuthHandler(token);
-      const auth = await handler.authenticate();
-
-      expect(auth).toEqual({
-        type: 'token',
-        token: token
-      });
-    });
-
-    it('should validate token format for GitHub PAT', async () => {
-      const token = 'ghp_validtoken123456';
-      const handler = new TokenAuthHandler(token);
-      const auth = await handler.authenticate();
-
-      expect(auth.token).toBe(token);
-    });
-
-    it('should accept fine-grained tokens', async () => {
-      const token = 'github_pat_validtoken123456';
-      const handler = new TokenAuthHandler(token);
-      const auth = await handler.authenticate();
-
-      expect(auth.token).toBe(token);
-    });
-  });
-
-  describe('validate', () => {
-    it('should validate token format', () => {
-      const handler = new TokenAuthHandler('ghp_test123');
-      expect(handler.validate()).toBe(true);
-    });
-
-    it('should accept any non-empty token', () => {
-      // Token validation is lenient - allows API keys without GitHub prefix
-      const handler = new TokenAuthHandler('api_key_123');
-      expect(handler.validate()).toBe(true);
-    });
-
-    it('should accept various GitHub token prefixes', () => {
-      const validPrefixes = ['ghp_', 'gho_', 'ghu_', 'ghs_', 'ghr_', 'github_pat_'];
-
-      validPrefixes.forEach((prefix) => {
-        const handler = new TokenAuthHandler(`${prefix}test123`);
-        expect(handler.validate()).toBe(true);
-      });
-    });
-  });
-
-  describe('getType', () => {
-    it('should return token type', () => {
-      const handler = new TokenAuthHandler('ghp_test');
-      expect(handler.getType()).toBe('token');
-    });
-  });
-
-  describe('token rotation', () => {
-    it('should allow token update', () => {
-      const handler = new TokenAuthHandler('ghp_old');
-      handler.updateToken('ghp_new');
-      expect(handler.getToken()).toBe('ghp_new');
-    });
-
-    it('should throw error on empty token update', () => {
-      const handler = new TokenAuthHandler('ghp_test');
-      expect(() => handler.updateToken('')).toThrow('Token cannot be empty');
-    });
-  });
-});

package/test/commits.test.ts

@@ -1,169 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { fetchCommits, getCommitDiff, parseCommitFiles } from '../src/commits.js';
-
-// Mock the octokit module
-const mockListCommits = vi.fn();
-const mockGetCommit = vi.fn();
-
-vi.mock('octokit', () => {
-  class MockOctokit {
-    rest = {
-      repos: {
-        listCommits: mockListCommits,
-        getCommit: mockGetCommit
-      }
-    };
-  }
-
-  return {
-    Octokit: MockOctokit
-  };
-});
-
-describe('Commit Analysis Tests', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('fetchCommits', () => {
-    it('should fetch commits from GitHub API', async () => {
-      const mockCommits = [
-        {
-          sha: 'abc123',
-          commit: {
-            message: 'feat: add new dependency',
-            author: { name: 'Test User', date: '2024-01-01T00:00:00Z' }
-          }
-        },
-        {
-          sha: 'def456',
-          commit: {
-            message: 'fix: update README',
-            author: { name: 'Test User', date: '2024-01-01T01:00:00Z' }
-          }
-        }
-      ];
-
-      mockListCommits.mockResolvedValue({
-        data: mockCommits
-      });
-
-      const client = {
-        getOctokit: () => ({ rest: { repos: { listCommits: mockListCommits } } })
-      } as any;
-      const result = await fetchCommits(client, 'owner', 'repo', { since: '2024-01-01T00:00:00Z' });
-
-      expect(result).toHaveLength(2);
-      expect(result[0].sha).toBe('abc123');
-      expect(result[1].sha).toBe('def456');
-      expect(mockListCommits).toHaveBeenCalledWith({
-        owner: 'owner',
-        repo: 'repo',
-        since: '2024-01-01T00:00:00Z'
-      });
-    });
-
-    it('should handle empty commit list', async () => {
-      mockListCommits.mockResolvedValue({ data: [] });
-
-      const client = {
-        getOctokit: () => ({ rest: { repos: { listCommits: mockListCommits } } })
-      } as any;
-      const result = await fetchCommits(client, 'owner', 'repo');
-
-      expect(result).toHaveLength(0);
-    });
-  });
-
-  describe('getCommitDiff', () => {
-    it('should fetch commit diff with file changes', async () => {
-      const mockCommitData = {
-        sha: 'abc123',
-        files: [
-          {
-            filename: 'README.md',
-            status: 'modified',
-            additions: 5,
-            deletions: 2,
-            changes: 7,
-            patch: '@@ -1,3 +1,6 @@\n-Old line\n+New line\n+Another line'
-          },
-          {
-            filename: 'src/index.ts',
-            status: 'added',
-            additions: 10,
-            deletions: 0,
-            changes: 10,
-            patch: '@@ -0,0 +1,10 @@\n+export function test() {}'
-          }
-        ]
-      };
-
-      mockGetCommit.mockResolvedValue({ data: mockCommitData });
-
-      const client = {
-        getOctokit: () => ({ rest: { repos: { getCommit: mockGetCommit } } })
-      } as any;
-      const result = await getCommitDiff(client, 'owner', 'repo', 'abc123');
-
-      expect(result.sha).toBe('abc123');
-      expect(result.files).toHaveLength(2);
-      expect(result.files[0].filename).toBe('README.md');
-      expect(result.files[0].status).toBe('modified');
-      expect(result.files[1].filename).toBe('src/index.ts');
-      expect(result.files[1].status).toBe('added');
-    });
-
-    it('should handle commit without file changes', async () => {
-      mockGetCommit.mockResolvedValue({
-        data: { sha: 'abc123', files: [] }
-      });
-
-      const client = {
-        getOctokit: () => ({ rest: { repos: { getCommit: mockGetCommit } } })
-      } as any;
-      const result = await getCommitDiff(client, 'owner', 'repo', 'abc123');
-
-      expect(result.files).toHaveLength(0);
-    });
-  });
-
-  describe('parseCommitFiles', () => {
-    it('should extract changed files from commit', () => {
-      const files = [
-        { filename: 'README.md', status: 'modified' as const, patch: '...' },
-        { filename: 'src/index.ts', status: 'added' as const, patch: '...' },
-        { filename: 'docs/guide.md', status: 'modified' as const, patch: '...' },
-        { filename: 'test.ts', status: 'removed' as const }
-      ];
-
-      const result = parseCommitFiles(files);
-
-      expect(result.modified).toContain('README.md');
-      expect(result.modified).toContain('docs/guide.md');
-      expect(result.added).toContain('src/index.ts');
-      expect(result.removed).toContain('test.ts');
-    });
-
-    it('should handle empty file list', () => {
-      const result = parseCommitFiles([]);
-
-      expect(result.added).toHaveLength(0);
-      expect(result.modified).toHaveLength(0);
-      expect(result.removed).toHaveLength(0);
-    });
-
-    it('should group files by type of change', () => {
-      const files = [
-        { filename: 'file1.md', status: 'added' as const, patch: '...' },
-        { filename: 'file2.md', status: 'added' as const, patch: '...' },
-        { filename: 'file3.md', status: 'modified' as const, patch: '...' }
-      ];
-
-      const result = parseCommitFiles(files);
-
-      expect(result.added).toHaveLength(2);
-      expect(result.modified).toHaveLength(1);
-    });
-  });
-});

package/test/feedback.test.ts

@@ -1,203 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { FeedbackListener } from '../src/feedback.js';
-import type { IssueManagerInterface } from '../src/feedback.js';
-
-describe('FeedbackListener', () => {
-  let mockIssueManager: IssueManagerInterface;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-
-    // Mock IssueManager
-    mockIssueManager = {
-      listIssues: vi.fn(),
-      getIssue: vi.fn()
-    } as IssueManagerInterface;
-  });
-
-  describe('constructor', () => {
-    it('should create listener with issue manager', () => {
-      const listener = new FeedbackListener(mockIssueManager);
-      expect(listener).toBeInstanceOf(FeedbackListener);
-    });
-
-    it('should accept custom labels', () => {
-      const config = {
-        truePositiveLabel: 'confirmed',
-        falsePositiveLabel: 'not-a-bug'
-      };
-      const listener = new FeedbackListener(mockIssueManager, config);
-      expect(listener).toBeInstanceOf(FeedbackListener);
-    });
-  });
-
-  describe('collectFeedback', () => {
-    it('should collect issues with feedback labels', async () => {
-      const mockIssues = [
-        { number: 1, labels: ['false-positive'], title: 'Test 1', created_at: '2026-01-01' },
-        { number: 2, labels: ['true-positive'], title: 'Test 2', created_at: '2026-01-02' },
-        { number: 3, labels: ['false-positive'], title: 'Test 3', created_at: '2026-01-03' }
-      ];
-
-      vi.mocked(mockIssueManager.listIssues).mockResolvedValue(mockIssues as any);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const feedback = await listener.collectFeedback();
-
-      expect(feedback.truePositives).toHaveLength(1);
-      expect(feedback.falsePositives).toHaveLength(2);
-      expect(feedback.total).toBe(3);
-    });
-
-    it('should filter by date range', async () => {
-      const mockIssues = [
-        {
-          number: 1,
-          labels: ['false-positive'],
-          title: 'Old issue',
-          created_at: '2025-12-01T00:00:00Z'
-        },
-        {
-          number: 2,
-          labels: ['false-positive'],
-          title: 'Recent issue',
-          created_at: '2026-01-25T00:00:00Z'
-        }
-      ];
-
-      vi.mocked(mockIssueManager.listIssues).mockResolvedValue(mockIssues as any);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const startDate = new Date('2026-01-01');
-      const feedback = await listener.collectFeedback({ startDate });
-
-      expect(feedback.falsePositives).toHaveLength(1);
-      expect(feedback.falsePositives[0].number).toBe(2);
-    });
-
-    it('should handle empty results', async () => {
-      vi.mocked(mockIssueManager.listIssues).mockResolvedValue([]);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const feedback = await listener.collectFeedback();
-
-      expect(feedback.truePositives).toHaveLength(0);
-      expect(feedback.falsePositives).toHaveLength(0);
-      expect(feedback.total).toBe(0);
-    });
-
-    it('should filter by repository', async () => {
-      const mockIssues = [
-        { number: 1, labels: ['false-positive'], title: 'Test', repository: 'owner/repo1' },
-        { number: 2, labels: ['false-positive'], title: 'Test', repository: 'owner/repo2' }
-      ];
-
-      vi.mocked(mockIssueManager.listIssues).mockResolvedValue(mockIssues as any);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const feedback = await listener.collectFeedback({ repository: 'owner/repo1' });
-
-      expect(feedback.falsePositives).toHaveLength(1);
-    });
-  });
-
-  describe('getFeedbackRate', () => {
-    it('should calculate false positive rate', async () => {
-      const mockIssues = [
-        { number: 1, labels: ['false-positive'], title: 'FP1' },
-        { number: 2, labels: ['true-positive'], title: 'TP1' },
-        { number: 3, labels: ['true-positive'], title: 'TP2' },
-        { number: 4, labels: ['false-positive'], title: 'FP2' }
-      ];
-
-      vi.mocked(mockIssueManager.listIssues).mockResolvedValue(mockIssues as any);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const rate = await listener.getFeedbackRate();
-
-      expect(rate.falsePositiveRate).toBe(0.5); // 2 FP out of 4 total
-      expect(rate.truePositiveRate).toBe(0.5); // 2 TP out of 4 total
-      expect(rate.totalFeedback).toBe(4);
-    });
-
-    it('should return 0 for empty feedback', async () => {
-      vi.mocked(mockIssueManager.listIssues).mockResolvedValue([]);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const rate = await listener.getFeedbackRate();
-
-      expect(rate.falsePositiveRate).toBe(0);
-      expect(rate.truePositiveRate).toBe(0);
-      expect(rate.totalFeedback).toBe(0);
-    });
-  });
-
-  describe('getRecentFeedback', () => {
-    it('should get feedback from last 30 days', async () => {
-      const now = new Date('2026-01-31');
-      const mockIssues = [
-        {
-          number: 1,
-          labels: ['false-positive'],
-          title: 'Recent',
-          created_at: '2026-01-20T00:00:00Z'
-        },
-        {
-          number: 2,
-          labels: ['false-positive'],
-          title: 'Old',
-          created_at: '2025-12-15T00:00:00Z'
-        }
-      ];
-
-      vi.mocked(mockIssueManager.listIssues).mockResolvedValue(mockIssues as any);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const feedback = await listener.getRecentFeedback(30, now);
-
-      expect(feedback.falsePositives).toHaveLength(1);
-      expect(feedback.falsePositives[0].number).toBe(1);
-    });
-
-    it('should default to current date', async () => {
-      vi.mocked(mockIssueManager.listIssues).mockResolvedValue([]);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const feedback = await listener.getRecentFeedback(30);
-
-      expect(feedback).toBeDefined();
-    });
-  });
-
-  describe('monitorIssue', () => {
-    it('should check if issue has feedback label', async () => {
-      const issue = {
-        number: 1,
-        labels: ['false-positive'],
-        title: 'Test'
-      };
-
-      vi.mocked(mockIssueManager.getIssue).mockResolvedValue(issue as any);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const hasFeedback = await listener.monitorIssue(1);
-
-      expect(hasFeedback).toBe(true);
-    });
-
-    it('should return false for issues without feedback labels', async () => {
-      const issue = {
-        number: 1,
-        labels: ['bug', 'enhancement'],
-        title: 'Test'
-      };
-
-      vi.mocked(mockIssueManager.getIssue).mockResolvedValue(issue as any);
-
-      const listener = new FeedbackListener(mockIssueManager);
-      const hasFeedback = await listener.monitorIssue(1);
-
-      expect(hasFeedback).toBe(false);
-    });
-  });
-});

package/test/issues.test.ts

@@ -1,197 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { IssueManager } from '../src/issues.js';
-
-// Mock the octokit module
-vi.mock('octokit', () => {
-  const mockIssuesCreate = vi.fn().mockResolvedValue({
-    data: {
-      number: 123,
-      html_url: 'https://github.com/test-owner/test-repo/issues/123',
-      labels: []
-    }
-  });
-
-  const mockIssuesGet = vi.fn().mockResolvedValue({
-    data: {
-      number: 123,
-      body: 'Existing issue body',
-      labels: [{ name: 'existing-label' }]
-    }
-  });
-
-  const mockIssuesUpdate = vi.fn().mockResolvedValue({
-    data: {
-      number: 123,
-      html_url: 'https://github.com/test-owner/test-repo/issues/123',
-      labels: []
-    }
-  });
-
-  const mockSearchIssues = vi.fn().mockResolvedValue({
-    data: {
-      items: []
-    }
-  });
-
-  class MockOctokit {
-    rest = {
-      issues: {
-        create: mockIssuesCreate,
-        get: mockIssuesGet,
-        update: mockIssuesUpdate
-      },
-      search: {
-        issuesAndPullRequests: mockSearchIssues
-      }
-    };
-  }
-
-  return {
-    Octokit: MockOctokit
-  };
-});
-
-describe('IssueManager', () => {
-  let issueManager: IssueManager;
-
-  beforeEach(() => {
-    issueManager = new IssueManager();
-    vi.clearAllMocks();
-  });
-
-  describe('createIssue', () => {
-    it('should create issue with severity label', async () => {
-      const issueData = {
-        owner: 'test-owner',
-        repo: 'test-repo',
-        title: 'Dependency Update: test-package',
-        body: 'New version detected',
-        severity: 'major' as const,
-        dependency: {
-          id: 'test-id',
-          url: 'https://github.com/owner/repo'
-        }
-      };
-
-      const result = await issueManager.createIssue(issueData);
-
-      expect(result).toBeDefined();
-      expect(result.number).toBeDefined();
-      expect(result.url).toBeDefined();
-      expect(result.labels).toContain('severity:major');
-    });
-
-    it('should include dependabit label', async () => {
-      const issueData = {
-        owner: 'test-owner',
-        repo: 'test-repo',
-        title: 'Dependency Update',
-        body: 'Changes detected',
-        severity: 'minor' as const,
-        dependency: {
-          id: 'test-id',
-          url: 'https://example.com'
-        }
-      };
-
-      const result = await issueManager.createIssue(issueData);
-
-      expect(result.labels).toContain('dependabit');
-    });
-
-    it('should handle breaking changes with appropriate label', async () => {
-      const issueData = {
-        owner: 'test-owner',
-        repo: 'test-repo',
-        title: 'Breaking Change Detected',
-        body: 'Major version update with breaking changes',
-        severity: 'breaking' as const,
-        dependency: {
-          id: 'test-id',
-          url: 'https://github.com/owner/repo'
-        }
-      };
-
-      const result = await issueManager.createIssue(issueData);
-
-      expect(result.labels).toContain('severity:breaking');
-    });
-
-    it('should assign issue to AI agent when specified', async () => {
-      const issueData = {
-        owner: 'test-owner',
-        repo: 'test-repo',
-        title: 'Dependency Update',
-        body: 'Changes detected',
-        severity: 'major' as const,
-        dependency: {
-          id: 'test-id',
-          url: 'https://example.com'
-        },
-        assignee: 'copilot'
-      };
-
-      const result = await issueManager.createIssue(issueData);
-
-      expect(result).toBeDefined();
-      expect(result.assignees).toContain('copilot');
-    });
-  });
-
-  describe('findExistingIssue', () => {
-    it('should find existing issue for dependency', async () => {
-      const params = {
-        owner: 'test-owner',
-        repo: 'test-repo',
-        dependencyId: 'test-id'
-      };
-
-      const result = await issueManager.findExistingIssue(params);
-
-      expect(result).toBeDefined();
-    });
-
-    it('should return null when no existing issue found', async () => {
-      const params = {
-        owner: 'test-owner',
-        repo: 'test-repo',
-        dependencyId: 'non-existent-id'
-      };
-
-      const result = await issueManager.findExistingIssue(params);
-
-      expect(result).toBeNull();
-    });
-  });
-
-  describe('updateIssue', () => {
-    it('should update existing issue with new information', async () => {
-      const updateData = {
-        owner: 'test-owner',
-        repo: 'test-repo',
-        issueNumber: 123,
-        body: 'Updated information',
-        severity: 'breaking' as const
-      };
-
-      const result = await issueManager.updateIssue(updateData);
-
-      expect(result).toBeDefined();
-      expect(result.number).toBe(123);
-    });
-
-    it('should append to existing issue body when specified', async () => {
-      const updateData = {
-        owner: 'test-owner',
-        repo: 'test-repo',
-        issueNumber: 123,
-        body: 'Additional update',
-        append: true
-      };
-
-      const result = await issueManager.updateIssue(updateData);
-
-      expect(result).toBeDefined();
-    });
-  });
-});