mirror of
https://github.com/abhigyanpatwari/GitNexus.git
synced 2026-10-07 02:58:02 +00:00
* refactor: migrate from KuzuDB to LadybugDB v0.15 KuzuDB was archived (Apple acquisition, Oct 2025). LadybugDB is the community fork with full API compatibility. - Package swap: kuzu → @ladybugdb/core, kuzu-wasm → @ladybugdb/wasm-core - Rename all internal paths: kuzu → lbug (adapters, schema, storage) - Storage path: .gitnexus/kuzu → .gitnexus/lbug (with auto-cleanup) - Add explicit VECTOR extension loading (required in v0.15) - Update CI workflow, documentation, and all tests - 1151 unit + 27 integration tests passing Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: address code review findings (P1-P3) P1: Fix WASM adapter to use getAll() API, wire cleanupOldKuzuFiles into analyze command, add symlink path traversal protection. P2: Cache VECTOR extension load state, batch augmentation engine queries (20→4), fix web getCopyQuery for multi-language tables, fix stale KuzuDB references, correct brainstorm package names. P3: Complete lbug-wasm.d.ts type declarations, batch semantic search per-label, update stale BM25 comment. * chore: remove outdated KuzuDB migration brainstorming document * fix: load FTS extension in MCP pool adapter on init The read-only pool adapter never loaded the FTS extension, so all QUERY_FTS_INDEX calls failed silently. This broke search-pool and augmentation integration tests, and caused empty results in the web UI server mode. * feat: implement shared Database caching and connection reference counting * feat: enhance KuzuDB migration handling and status reporting * fix: mock cleanupOldKuzuFiles in local backend callTool tests * fix: update mock for cleanupOldKuzuFiles and adjust imports in callTool tests --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
121 lines
4.4 KiB
TypeScript
121 lines
4.4 KiB
TypeScript
/**
|
|
* P0 Integration Tests: BM25/FTS Search against real LadybugDB
|
|
*
|
|
* Tests: searchFTSFromLbug via core adapter (no repoId) path against
|
|
* indexed test data. Verifies ranked result ordering, score merging,
|
|
* and empty-match behavior.
|
|
*
|
|
* Uses withTestLbugDB wrapper for full lifecycle management.
|
|
*/
|
|
import { describe, it, expect } from 'vitest';
|
|
import { withTestLbugDB } from '../helpers/test-indexed-db.js';
|
|
import { searchFTSFromLbug } from '../../src/core/search/bm25-index.js';
|
|
import { SEARCH_SEED_DATA, SEARCH_FTS_INDEXES } from '../fixtures/search-seed.js';
|
|
|
|
// ─── Core adapter path (no repoId) ──────────────────────────────────
|
|
|
|
withTestLbugDB('search-core', (_handle) => {
|
|
describe('searchFTSFromLbug — core adapter (no repoId)', () => {
|
|
it('returns ranked results for a matching query', async () => {
|
|
const results = await searchFTSFromLbug('user authentication', 10);
|
|
|
|
expect(results.length).toBeGreaterThan(0);
|
|
|
|
for (const r of results) {
|
|
expect(r).toHaveProperty('filePath');
|
|
expect(r).toHaveProperty('score');
|
|
expect(r).toHaveProperty('rank');
|
|
expect(typeof r.filePath).toBe('string');
|
|
expect(typeof r.score).toBe('number');
|
|
expect(typeof r.rank).toBe('number');
|
|
expect(r.score).toBeGreaterThan(0);
|
|
}
|
|
|
|
// Ranks should be sequential starting from 1
|
|
results.forEach((r, i) => {
|
|
expect(r.rank).toBe(i + 1);
|
|
});
|
|
});
|
|
|
|
it('results are ordered by descending score', async () => {
|
|
const results = await searchFTSFromLbug('user authentication', 10);
|
|
|
|
for (let i = 1; i < results.length; i++) {
|
|
expect(results[i - 1].score).toBeGreaterThanOrEqual(results[i].score);
|
|
}
|
|
});
|
|
|
|
it('auth-related files rank higher than unrelated files', async () => {
|
|
const results = await searchFTSFromLbug('user authentication', 10);
|
|
const filePaths = results.map((r) => r.filePath);
|
|
|
|
expect(filePaths).toContain('src/auth.ts');
|
|
|
|
const authIdx = filePaths.indexOf('src/auth.ts');
|
|
const utilsIdx = filePaths.indexOf('src/utils.ts');
|
|
if (utilsIdx !== -1) {
|
|
expect(authIdx).toBeLessThan(utilsIdx);
|
|
}
|
|
});
|
|
|
|
it('merges scores from multiple node types for the same filePath', async () => {
|
|
const results = await searchFTSFromLbug('user authentication', 20);
|
|
|
|
const authResult = results.find((r) => r.filePath === 'src/auth.ts');
|
|
expect(authResult).toBeDefined();
|
|
|
|
const routerResult = results.find((r) => r.filePath === 'src/router.ts');
|
|
if (routerResult) {
|
|
expect(authResult!.score).toBeGreaterThan(routerResult.score);
|
|
}
|
|
});
|
|
|
|
it('respects limit parameter', async () => {
|
|
const results = await searchFTSFromLbug('user authentication', 2);
|
|
expect(results.length).toBeLessThanOrEqual(2);
|
|
});
|
|
|
|
it('returns empty array for a non-matching query', async () => {
|
|
const results = await searchFTSFromLbug('xyzzyplughtwisty', 10);
|
|
expect(results).toEqual([]);
|
|
});
|
|
});
|
|
|
|
// ─── Unhappy paths ──────────────────────────────────────────────────
|
|
|
|
describe('unhappy paths', () => {
|
|
it('returns empty array for empty query string', async () => {
|
|
const results = await searchFTSFromLbug('', 10);
|
|
expect(results).toEqual([]);
|
|
});
|
|
|
|
it('returns empty array for whitespace-only query', async () => {
|
|
const results = await searchFTSFromLbug(' ', 10);
|
|
expect(results).toEqual([]);
|
|
});
|
|
|
|
it('handles special characters in query gracefully', async () => {
|
|
const results = await searchFTSFromLbug('user* OR auth+', 10);
|
|
expect(Array.isArray(results)).toBe(true);
|
|
});
|
|
|
|
it('handles limit of 0', async () => {
|
|
const results = await searchFTSFromLbug('user authentication', 0);
|
|
expect(results).toEqual([]);
|
|
});
|
|
|
|
it('handles negative limit gracefully', async () => {
|
|
const results = await searchFTSFromLbug('user authentication', -1);
|
|
expect(Array.isArray(results)).toBe(true);
|
|
});
|
|
|
|
it('handles very large limit', async () => {
|
|
const results = await searchFTSFromLbug('user authentication', 100000);
|
|
expect(results.length).toBeLessThanOrEqual(100000);
|
|
expect(results.length).toBeGreaterThan(0);
|
|
});
|
|
});
|
|
}, {
|
|
seed: SEARCH_SEED_DATA,
|
|
ftsIndexes: SEARCH_FTS_INDEXES,
|
|
});
|