@dependabit/detector 0.1.16 → 0.1.18

package/test/diff-parser.test.ts

@@ -1,187 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import {
-  parseDiff,
-  extractAddedContent,
-  extractRemovedContent,
-  getChangedFiles
-} from '../src/diff-parser.js';
-
-describe('Diff Parser Tests', () => {
-  describe('parseDiff', () => {
-    it('should parse unified diff format', () => {
-      const diff = `@@ -1,3 +1,4 @@
- Line 1
--Old line
-+New line
-+Another new line
- Line 3`;
-
-      const result = parseDiff(diff);
-
-      expect(result.additions).toContain('New line');
-      expect(result.additions).toContain('Another new line');
-      expect(result.deletions).toContain('Old line');
-    });
-
-    it('should handle diff with only additions', () => {
-      const diff = `@@ -0,0 +1,3 @@
-+First line
-+Second line
-+Third line`;
-
-      const result = parseDiff(diff);
-
-      expect(result.additions).toHaveLength(3);
-      expect(result.deletions).toHaveLength(0);
-    });
-
-    it('should handle diff with only deletions', () => {
-      const diff = `@@ -1,3 +0,0 @@
--First line
--Second line
--Third line`;
-
-      const result = parseDiff(diff);
-
-      expect(result.additions).toHaveLength(0);
-      expect(result.deletions).toHaveLength(3);
-    });
-
-    it('should ignore context lines (no +/-)', () => {
-      const diff = `@@ -1,5 +1,5 @@
- Context line 1
--Old line
-+New line
- Context line 2
- Context line 3`;
-
-      const result = parseDiff(diff);
-
-      expect(result.additions).toHaveLength(1);
-      expect(result.deletions).toHaveLength(1);
-      expect(result.additions[0]).toBe('New line');
-    });
-  });
-
-  describe('extractAddedContent', () => {
-    it('should extract URLs from added lines', () => {
-      const additions = [
-        'Check out https://example.com/api for details',
-        'See documentation at https://docs.example.com',
-        'Regular line without URL'
-      ];
-
-      const result = extractAddedContent(additions);
-
-      expect(result.urls).toContain('https://example.com/api');
-      expect(result.urls).toContain('https://docs.example.com');
-      expect(result.urls).toHaveLength(2);
-    });
-
-    it('should extract dependencies from package.json additions', () => {
-      const additions = [
-        '  "dependencies": {',
-        '    "axios": "^1.0.0",',
-        '    "lodash": "^4.17.21"',
-        '  }'
-      ];
-
-      const result = extractAddedContent(additions, 'package.json');
-
-      expect(result.packageDeps).toContain('axios');
-      expect(result.packageDeps).toContain('lodash');
-    });
-
-    it('should extract references from code comments', () => {
-      const additions = [
-        '// Based on https://github.com/example/repo',
-        '/* Reference: https://arxiv.org/abs/2301.12345 */',
-        'const x = 5; // Regular comment'
-      ];
-
-      const result = extractAddedContent(additions);
-
-      expect(result.urls).toContain('https://github.com/example/repo');
-      expect(result.urls).toContain('https://arxiv.org/abs/2301.12345');
-    });
-
-    it('should handle empty additions', () => {
-      const result = extractAddedContent([]);
-
-      expect(result.urls).toHaveLength(0);
-      expect(result.packageDeps).toHaveLength(0);
-    });
-  });
-
-  describe('extractRemovedContent', () => {
-    it('should extract URLs from removed lines', () => {
-      const deletions = [
-        'Old link: https://old-example.com',
-        'Deprecated: https://deprecated.example.com/api'
-      ];
-
-      const result = extractRemovedContent(deletions);
-
-      expect(result.urls).toContain('https://old-example.com');
-      expect(result.urls).toContain('https://deprecated.example.com/api');
-    });
-
-    it('should extract dependencies from package.json deletions', () => {
-      const deletions = ['    "old-package": "^1.0.0",', '    "deprecated-lib": "^2.0.0"'];
-
-      const result = extractRemovedContent(deletions, 'package.json');
-
-      expect(result.packageDeps).toContain('old-package');
-      expect(result.packageDeps).toContain('deprecated-lib');
-    });
-  });
-
-  describe('getChangedFiles', () => {
-    it('should identify files relevant for dependency analysis', () => {
-      const files = [
-        { filename: 'README.md', status: 'modified' as const },
-        { filename: 'src/index.ts', status: 'modified' as const },
-        { filename: 'package.json', status: 'modified' as const },
-        { filename: 'docs/guide.md', status: 'added' as const },
-        { filename: 'test/unit.test.ts', status: 'modified' as const }
-      ];
-
-      const result = getChangedFiles(files);
-
-      expect(result.relevantFiles).toContain('README.md');
-      expect(result.relevantFiles).toContain('package.json');
-      expect(result.relevantFiles).toContain('docs/guide.md');
-      expect(result.packageFiles).toContain('package.json');
-    });
-
-    it('should exclude irrelevant file types', () => {
-      const files = [
-        { filename: 'image.png', status: 'added' as const },
-        { filename: 'video.mp4', status: 'added' as const },
-        { filename: 'README.md', status: 'modified' as const }
-      ];
-
-      const result = getChangedFiles(files);
-
-      expect(result.relevantFiles).not.toContain('image.png');
-      expect(result.relevantFiles).not.toContain('video.mp4');
-      expect(result.relevantFiles).toContain('README.md');
-    });
-
-    it('should identify package manifest files', () => {
-      const files = [
-        { filename: 'package.json', status: 'modified' as const },
-        { filename: 'requirements.txt', status: 'modified' as const },
-        { filename: 'Cargo.toml', status: 'modified' as const },
-        { filename: 'go.mod', status: 'modified' as const }
-      ];
-
-      const result = getChangedFiles(files);
-
-      expect(result.packageFiles).toContain('package.json');
-      expect(result.packageFiles).toContain('requirements.txt');
-      expect(result.packageFiles).toContain('Cargo.toml');
-      expect(result.packageFiles).toContain('go.mod');
-    });
-  });
-});

package/test/llm/client.test.ts

@@ -1,31 +0,0 @@
-import { describe, it, expect } from 'vitest';
-
-describe('LLMProvider interface', () => {
-  it('should define analyze method signature', () => {
-    // This is a type test - we'll verify the interface exists
-    // The interface will be implemented in src/llm/client.ts
-    expect(true).toBe(true);
-  });
-
-  it('should define getSupportedModels method', () => {
-    expect(true).toBe(true);
-  });
-
-  it('should define getRateLimit method', () => {
-    expect(true).toBe(true);
-  });
-});
-
-describe('LLMResponse', () => {
-  it('should include detected dependencies', () => {
-    expect(true).toBe(true);
-  });
-
-  it('should include confidence scores', () => {
-    expect(true).toBe(true);
-  });
-
-  it('should include usage metadata (tokens, latency)', () => {
-    expect(true).toBe(true);
-  });
-});

package/test/llm/copilot.test.ts

@@ -1,334 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { promisify } from 'node:util';
-import { GitHubCopilotProvider } from '../../src/llm/copilot.js';
-import type { DetectedDependency } from '../../src/llm/client.js';
-
-// Mock child_process with proper execFile behavior
-vi.mock('node:child_process', () => {
-  const mockCallback = vi.fn();
-  
-  // The custom promisify implementation that mimics Node.js behavior
-  const customPromisify = vi.fn((...execArgs) => {
-    return new Promise((resolve, reject) => {
-      // Call the mock with a callback that converts to { stdout, stderr }
-      mockCallback(...execArgs, (err: Error | null, stdout: string, stderr: string) => {
-        if (err) {
-          const error = err as NodeJS.ErrnoException & { stdout?: string; stderr?: string };
-          error.stdout = stdout;
-          error.stderr = stderr;
-          reject(error);
-        } else {
-          resolve({ stdout, stderr });
-        }
-      });
-    });
-  });
-  
-  mockCallback[promisify.custom] = customPromisify;
-  
-  return {
-    execFile: mockCallback
-  };
-});
-
-import { execFile } from 'node:child_process';
-const mockExecFileCallback = vi.mocked(execFile);
-
-describe('GitHubCopilotProvider', () => {
-  let provider: GitHubCopilotProvider;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    provider = new GitHubCopilotProvider();
-  });
-
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  describe('CLI invocation', () => {
-    it('should use execFile with correct command and flags', async () => {
-      const mockResponse = JSON.stringify({ dependencies: [] });
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          // Use correct callback signature: callback(error, stdout, stderr)
-          callback(null, mockResponse, '');
-        }
-        return {} as any;
-      });
-
-      await provider.analyze('test content', 'test prompt');
-
-      expect(mockExecFileCallback).toHaveBeenCalledWith(
-        'gh',
-        expect.arrayContaining([
-          'copilot',
-          '-p',
-          expect.any(String),
-          '--silent',
-          '--allow-all-tools'
-        ]),
-        expect.objectContaining({
-          maxBuffer: 10 * 1024 * 1024,
-          timeout: 60000
-        }),
-        expect.any(Function)
-      );
-    });
-
-    it('should pass prompt directly without shell escaping', async () => {
-      const mockResponse = JSON.stringify({ dependencies: [] });
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, mockResponse, '');
-        }
-        return {} as any;
-      });
-
-      const promptWithSpecialChars = 'test "quotes" $dollar `backtick`';
-      await provider.analyze('content', promptWithSpecialChars);
-
-      const callArgs = mockExecFileCallback.mock.calls[0];
-      const args = callArgs?.[1] as string[];
-      const fullPrompt = args?.[2];
-
-      // Verify prompt contains special characters unescaped
-      expect(fullPrompt).toContain('test "quotes" $dollar `backtick`');
-    });
-
-    it('should handle prompts with newlines', async () => {
-      const mockResponse = JSON.stringify({ dependencies: [] });
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, mockResponse, '');
-        }
-        return {} as any;
-      });
-
-      const promptWithNewlines = 'line1\nline2\nline3';
-      await provider.analyze('content', promptWithNewlines);
-
-      const callArgs = mockExecFileCallback.mock.calls[0];
-      const args = callArgs?.[1] as string[];
-      const fullPrompt = args?.[2];
-
-      expect(fullPrompt).toContain('line1\nline2\nline3');
-    });
-
-    it('should handle prompts ending with backslash', async () => {
-      const mockResponse = JSON.stringify({ dependencies: [] });
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, mockResponse, '');
-        }
-        return {} as any;
-      });
-
-      const promptWithBackslash = 'path\\to\\file\\';
-      await provider.analyze('content', promptWithBackslash);
-
-      const callArgs = mockExecFileCallback.mock.calls[0];
-      const args = callArgs?.[1] as string[];
-      const fullPrompt = args?.[2];
-
-      expect(fullPrompt).toContain('path\\to\\file\\');
-    });
-  });
-
-  describe('response parsing', () => {
-    it('should parse JSON response correctly', async () => {
-      const mockDeps: DetectedDependency[] = [
-        {
-          name: 'test-package',
-          version: '1.0.0',
-          type: 'npm',
-          confidence: 0.9,
-          context: 'test context'
-        }
-      ];
-      const mockResponse = JSON.stringify({ dependencies: mockDeps });
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, mockResponse, '');
-        }
-        return {} as any;
-      });
-
-      const result = await provider.analyze('content', 'prompt');
-
-      expect(result.dependencies).toEqual(mockDeps);
-    });
-
-    it('should parse JSON wrapped in markdown code blocks', async () => {
-      const mockDeps: DetectedDependency[] = [
-        {
-          name: 'test-package',
-          version: '1.0.0',
-          type: 'npm',
-          confidence: 0.9,
-          context: 'test context'
-        }
-      ];
-      const mockResponse = '```json\n' + JSON.stringify({ dependencies: mockDeps }) + '\n```';
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, mockResponse, '');
-        }
-        return {} as any;
-      });
-
-      const result = await provider.analyze('content', 'prompt');
-
-      expect(result.dependencies).toEqual(mockDeps);
-    });
-
-    it('should handle malformed JSON gracefully', async () => {
-      const mockResponse = 'not valid json';
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, mockResponse, '');
-        }
-        return {} as any;
-      });
-
-      const result = await provider.analyze('content', 'prompt');
-
-      expect(result.dependencies).toEqual([]);
-    });
-
-    it('should return empty dependencies on empty response', async () => {
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, '', '');
-        }
-        return {} as any;
-      });
-
-      const result = await provider.analyze('content', 'prompt');
-
-      expect(result.dependencies).toEqual([]);
-    });
-  });
-
-  describe('error handling', () => {
-    it('should handle CLI execution errors', async () => {
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(new Error('CLI error'), '', 'Error message');
-        }
-        return {} as any;
-      });
-
-      const result = await provider.analyze('content', 'prompt');
-
-      expect(result.dependencies).toEqual([]);
-      expect(result.rawResponse).toContain('CLI error');
-    });
-
-    it('should handle stderr without stdout as error', async () => {
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, '', 'Authentication failed');
-        }
-        return {} as any;
-      });
-
-      const result = await provider.analyze('content', 'prompt');
-
-      expect(result.dependencies).toEqual([]);
-    });
-
-    it('should handle timeout errors', async () => {
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          const error = new Error('Command timeout') as NodeJS.ErrnoException;
-          error.code = 'ETIMEDOUT';
-          callback(error, '', '');
-        }
-        return {} as any;
-      });
-
-      const result = await provider.analyze('content', 'prompt');
-
-      expect(result.dependencies).toEqual([]);
-    });
-
-    it('should handle buffer overflow errors', async () => {
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          const error = new Error('maxBuffer exceeded') as NodeJS.ErrnoException;
-          error.code = 'ERR_CHILD_PROCESS_STDIO_MAXBUFFER';
-          callback(error, '', '');
-        }
-        return {} as any;
-      });
-
-      const result = await provider.analyze('content', 'prompt');
-
-      expect(result.dependencies).toEqual([]);
-    });
-  });
-
-  describe('usage metadata', () => {
-    it('should include usage metadata in response', async () => {
-      const mockResponse = JSON.stringify({ dependencies: [] });
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, mockResponse, '');
-        }
-        return {} as any;
-      });
-
-      const result = await provider.analyze('content', 'prompt');
-
-      expect(result.usage).toBeDefined();
-      expect(result.usage.promptTokens).toBeGreaterThan(0);
-      expect(result.usage.completionTokens).toBeGreaterThan(0);
-      expect(result.usage.totalTokens).toBeGreaterThan(0);
-      expect(result.usage.latencyMs).toBeGreaterThanOrEqual(0);
-    });
-
-    it('should use configured model in usage metadata', async () => {
-      const mockResponse = JSON.stringify({ dependencies: [] });
-      mockExecFileCallback.mockImplementation((file, args, options, callback) => {
-        if (typeof callback === 'function') {
-          callback(null, mockResponse, '');
-        }
-        return {} as any;
-      });
-
-      const customProvider = new GitHubCopilotProvider({ model: 'gpt-4-turbo' });
-      const result = await customProvider.analyze('content', 'prompt');
-
-      expect(result.usage.model).toBe('gpt-4-turbo');
-    });
-  });
-
-  describe('getSupportedModels', () => {
-    it('should return list of supported models', () => {
-      const models = provider.getSupportedModels();
-
-      expect(models).toBeInstanceOf(Array);
-      expect(models.length).toBeGreaterThan(0);
-      expect(models).toContain('github-copilot');
-    });
-  });
-
-  describe('getRateLimit', () => {
-    it('should return rate limit info', async () => {
-      const rateLimit = await provider.getRateLimit();
-
-      expect(rateLimit).toBeDefined();
-      expect(rateLimit.remaining).toBe(-1);
-      expect(rateLimit.limit).toBe(-1);
-    });
-  });
-
-  describe('validateConfig', () => {
-    it('should validate configuration', () => {
-      const isValid = provider.validateConfig();
-
-      expect(typeof isValid).toBe('boolean');
-    });
-  });
-});

package/test/parsers/code-comments.test.ts

@@ -1,98 +0,0 @@
-import { describe, it, expect } from 'vitest';
-
-describe('CodeCommentParser', () => {
-  describe('parse', () => {
-    it('should extract URLs from single-line comments', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should extract URLs from multi-line comments', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should extract JSDoc @see tags', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should include file path and line number', () => {
-      // Expected: Include { file: 'src/foo.ts', line: 5 }
-      expect(true).toBe(true);
-    });
-
-    it('should handle TypeScript files', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should handle JavaScript files', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should handle Python files', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should skip commented-out code', () => {
-      expect(true).toBe(true);
-    });
-  });
-
-  describe('extractDocReferences', () => {
-    it('should identify specification references', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should identify API endpoint references', () => {
-      const code = '// API endpoint: https://example.com/api';
-      expect(true).toBe(true);
-      expect(code).toContain('https://example.com/api');
-    });
-
-    it('should extract URLs from multi-line comments', () => {
-      const code = '/* Reference: https://example.com */';
-      expect(code).toContain('https://example.com');
-    });
-
-    it('should extract JSDoc @see tags', () => {
-      const code = '/** @see https://example.com/docs */';
-      expect(code).toContain('https://example.com/docs');
-    });
-
-    it('should include file path and line number', () => {
-      const code = '// Based on https://spec.example.com';
-      // Expected: Include { file: 'src/foo.ts', line: 5 }
-      expect(code).toContain('https://spec.example.com');
-    });
-
-    it('should handle TypeScript files', () => {
-      const code = '// Implementation from https://github.com/example/repo';
-      expect(code).toContain('https://github.com/example/repo');
-    });
-
-    it('should handle JavaScript files', () => {
-      const code = '/* Spec: https://openapi.example.com/spec.yaml */';
-      expect(code).toContain('https://openapi.example.com/spec.yaml');
-    });
-
-    it('should handle Python files', () => {
-      const code = '# Reference: https://arxiv.org/abs/1234.5678';
-      expect(code).toContain('https://arxiv.org/abs/1234.5678');
-    });
-
-    it('should skip commented-out code', () => {
-      const code = '// const url = "https://example.com"; // unused';
-      expect(code).toContain('https://example.com');
-    });
-  });
-
-  describe('extractDocReferences', () => {
-    it('should identify specification references', () => {
-      const code = '// Implements RFC 7231 specification';
-      expect(code).toContain('RFC 7231');
-    });
-
-    it('should identify API endpoint references', () => {
-      const code = '// Calls POST /api/v1/users';
-      expect(code).toContain('/api/v1/users');
-    });
-  });
-});

package/test/parsers/package-files.test.ts

@@ -1,52 +0,0 @@
-import { describe, it, expect } from 'vitest';
-
-describe('PackageFileParser', () => {
-  describe('parsePackageJson', () => {
-    it('should extract repository URLs', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should extract homepage URL', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should extract documentation URL', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should mark dependencies as tracked by dependabot', () => {
-      // Expected: Return empty (dependabot handles this)
-      expect(true).toBe(true);
-    });
-
-    it('should extract URLs from package description', () => {
-      expect(true).toBe(true);
-    });
-  });
-
-  describe('parseRequirementsTxt', () => {
-    it('should extract GitHub repository URLs from comments', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should skip PyPI packages', () => {
-      // Expected: Return empty (dependabot handles this)
-      expect(true).toBe(true);
-    });
-
-    it('should extract documentation URLs from comments', () => {
-      expect(true).toBe(true);
-    });
-  });
-
-  describe('parseCargoToml', () => {
-    it('should extract repository URLs', () => {
-      expect(true).toBe(true);
-    });
-
-    it('should skip crates.io dependencies', () => {
-      // Expected: Return empty (dependabot handles this)
-      expect(true).toBe(true);
-    });
-  });
-});