mirror of
https://github.com/yamadashy/repomix.git
synced 2026-02-03 11:33:39 +01:00
0fd8812b01
Created BaseParseStrategy abstract class to reduce code duplication: - Common helper methods (getCaptureTypes, checkAndAddToProcessed, validateLineExists) - Shared ParseResult type and CommonCaptureTypes constants - All strategies (Default, TypeScript, Python, Go, CSS, Vue) now extend base class Unified language configuration in languageConfig.ts: - Consolidated ext2Lang.ts and lang2Query.ts functionality - Single source of truth for language settings (extensions, queries, strategies) - Efficient Map-based lookup system for better performance - Removed switch statement in createParseStrategy() Updated all Strategy classes to extend BaseParseStrategy: - Eliminated ~200 lines of duplicate code - Improved maintainability and extensibility - Easier to add new languages (single configuration change) This refactoring was motivated by the need to reduce code duplication across language-specific strategies and simplify the process of adding new language support. The previous implementation had language configurations scattered across multiple files, making it difficult to maintain and extend.
65 lines
2.0 KiB
TypeScript
65 lines
2.0 KiB
TypeScript
import { describe, expect, test } from 'vitest';
|
|
import { CHUNK_SEPARATOR, parseFile } from '../../../src/core/treeSitter/parseFile.js';
|
|
import { createMockConfig } from '../../testing/testUtils.js';
|
|
|
|
describe('parseFile for JavaScript', () => {
|
|
test('should filter captures with same start row', async () => {
|
|
const fileContent = `
|
|
/**
|
|
* Greeting function
|
|
* @param name The name to greet
|
|
*/
|
|
function sayHello(name) { // inline comment
|
|
console.log("Hello, " + name);
|
|
}
|
|
|
|
// next function
|
|
function sayGoodbye(name) {
|
|
console.log("Goodbye, " + name);
|
|
}
|
|
`;
|
|
const filePath = 'dummy.js';
|
|
const config = {};
|
|
const result = await parseFile(fileContent, filePath, createMockConfig(config));
|
|
expect(typeof result).toBe('string');
|
|
// Check content
|
|
expect(result).toContain('/**\n * Greeting function\n * @param name The name to greet\n */');
|
|
expect(result).toContain('// inline comment');
|
|
expect(result).not.toBeUndefined();
|
|
if (result) {
|
|
// Check separator
|
|
const parts = result.split(`${CHUNK_SEPARATOR}\n`);
|
|
expect(parts.length).toBeGreaterThan(1);
|
|
for (const part of parts) {
|
|
expect(part.trim()).not.toBe('');
|
|
}
|
|
}
|
|
});
|
|
|
|
test('should parse JSX correctly', async () => {
|
|
const fileContent = `
|
|
// React component function
|
|
/**
|
|
* A hello world component
|
|
* @param {string} name - The name to display
|
|
*/
|
|
function sayHello(name) { console.log("Hello, " + name); }
|
|
`;
|
|
const filePath = 'dummy.jsx';
|
|
const config = {};
|
|
const result = await parseFile(fileContent, filePath, createMockConfig(config));
|
|
expect(typeof result).toBe('string');
|
|
|
|
const expectContents = [
|
|
'// React component function',
|
|
'* A hello world component',
|
|
'* @param {string} name - The name to display',
|
|
'function sayHello(name)',
|
|
];
|
|
|
|
for (const expectContent of expectContents) {
|
|
expect(result).toContain(expectContent);
|
|
}
|
|
});
|
|
});
|