gitnexus 1.6.11-rc.31 → 1.6.11-rc.32

package/dist/core/ingestion/entry-point-scoring.d.ts

@@ -29,8 +29,12 @@ export interface EntryPointScoreResult {
  */
 export declare function calculateEntryPointScore(name: string, language: SupportedLanguages, isExported: boolean, callerCount: number, calleeCount: number, filePath?: string): EntryPointScoreResult;
 /**
- * Check if a file path is a test file (should be excluded from entry points)
- * Covers common test file patterns across all supported languages
+ * Check if a file path is a test file (should be excluded from entry points).
+ *
+ * Delegates to the shared predicate in `utils/test-file-path.ts`. This used to be
+ * a second, hand-maintained copy that had drifted from the one backing the MCP
+ * `includeTests` flag — see that module's header. Re-exported under this name so
+ * existing importers are unaffected.
  */
 export declare function isTestFile(filePath: string): boolean;
 /**

package/dist/core/ingestion/entry-point-scoring.js

@@ -11,6 +11,7 @@
  */
 import { detectFrameworkFromPath } from './framework-detection.js';
 import { providers } from './languages/index.js';
+import { isTestFilePath } from './utils/test-file-path.js';
 // ============================================================================
 // NAME PATTERNS
 // ============================================================================
@@ -127,52 +128,15 @@ export function calculateEntryPointScore(name, language, isExported, callerCount
 // HELPER FUNCTIONS
 // ============================================================================
 /**
- * Check if a file path is a test file (should be excluded from entry points)
- * Covers common test file patterns across all supported languages
+ * Check if a file path is a test file (should be excluded from entry points).
+ *
+ * Delegates to the shared predicate in `utils/test-file-path.ts`. This used to be
+ * a second, hand-maintained copy that had drifted from the one backing the MCP
+ * `includeTests` flag — see that module's header. Re-exported under this name so
+ * existing importers are unaffected.
  */
 export function isTestFile(filePath) {
-    const p = filePath.toLowerCase().replace(/\\/g, '/');
-    return (
-    // JavaScript/TypeScript test patterns
-    p.includes('.test.') ||
-        p.includes('.spec.') ||
-        p.includes('__tests__/') ||
-        p.includes('__mocks__/') ||
-        // Generic test folders
-        p.includes('/test/') ||
-        p.includes('/tests/') ||
-        p.includes('/testing/') ||
-        // Python test patterns
-        p.endsWith('_test.py') ||
-        p.includes('/test_') ||
-        // Go test patterns
-        p.endsWith('_test.go') ||
-        // Java test patterns
-        p.includes('/src/test/') ||
-        // Rust test patterns (inline tests are different, but test files)
-        p.includes('/tests/') ||
-        // Swift/iOS test patterns
-        p.endsWith('tests.swift') ||
-        p.endsWith('test.swift') ||
-        p.includes('uitests/') ||
-        // C# test patterns
-        p.endsWith('tests.cs') ||
-        p.endsWith('test.cs') ||
-        p.includes('.tests/') ||
-        p.includes('.test/') ||
-        p.includes('.integrationtests/') ||
-        p.includes('.unittests/') ||
-        p.includes('/testproject/') ||
-        // PHP/Laravel test patterns
-        p.endsWith('test.php') ||
-        p.endsWith('spec.php') ||
-        p.includes('/tests/feature/') ||
-        p.includes('/tests/unit/') ||
-        // Ruby test patterns
-        p.endsWith('_spec.rb') ||
-        p.endsWith('_test.rb') ||
-        p.includes('/spec/') ||
-        p.includes('/test/fixtures/'));
+    return isTestFilePath(filePath);
 }
 /**
  * Check if a file path is likely a utility/helper file

package/dist/core/ingestion/utils/test-file-path.d.ts

@@ -0,0 +1,28 @@
+/**
+ * Test-file path classification — the single source of truth.
+ *
+ * WHY THIS MODULE EXISTS
+ *
+ * Two independent copies of this predicate existed and had drifted apart:
+ *
+ *   - `core/ingestion/entry-point-scoring.ts` `isTestFile`  — excludes test
+ *     files from process entry-point detection.
+ *   - `mcp/local/local-backend.ts` `isTestFilePath`         — backs the
+ *     `includeTests` flag on `impact` / `trace` / `context`.
+ *
+ * They answered "is this a test file?" differently, so the same path could be a
+ * test in one code path and not the other. The MCP copy recognized no C#, Java,
+ * or Swift test convention at all, meaning `includeTests: false` silently failed
+ * to filter them; the scoring copy missed `/conftest.` (it already matched
+ * `/test/`, so `/test/fixtures/` was never a scoring gap).
+ *
+ * The duplication was not gratuitous: `entry-point-scoring.ts` imports the
+ * language-provider registry, and #2802 deliberately cut that closure out of MCP
+ * server startup. Importing it back into `local-backend.ts` would reintroduce
+ * that cost. So the shared predicate lives here instead, with NO imports — pure
+ * string matching — and both callers delegate to it.
+ *
+ * Keep it dependency-free. Anything imported here lands in MCP startup.
+ */
+/** Absent / empty paths are not test paths. */
+export declare function isTestFilePath(filePath: string | null | undefined): boolean;

package/dist/core/ingestion/utils/test-file-path.js

@@ -0,0 +1,87 @@
+/**
+ * Test-file path classification — the single source of truth.
+ *
+ * WHY THIS MODULE EXISTS
+ *
+ * Two independent copies of this predicate existed and had drifted apart:
+ *
+ *   - `core/ingestion/entry-point-scoring.ts` `isTestFile`  — excludes test
+ *     files from process entry-point detection.
+ *   - `mcp/local/local-backend.ts` `isTestFilePath`         — backs the
+ *     `includeTests` flag on `impact` / `trace` / `context`.
+ *
+ * They answered "is this a test file?" differently, so the same path could be a
+ * test in one code path and not the other. The MCP copy recognized no C#, Java,
+ * or Swift test convention at all, meaning `includeTests: false` silently failed
+ * to filter them; the scoring copy missed `/conftest.` (it already matched
+ * `/test/`, so `/test/fixtures/` was never a scoring gap).
+ *
+ * The duplication was not gratuitous: `entry-point-scoring.ts` imports the
+ * language-provider registry, and #2802 deliberately cut that closure out of MCP
+ * server startup. Importing it back into `local-backend.ts` would reintroduce
+ * that cost. So the shared predicate lives here instead, with NO imports — pure
+ * string matching — and both callers delegate to it.
+ *
+ * Keep it dependency-free. Anything imported here lands in MCP startup.
+ */
+/**
+ * Lowercase forward-slash path substrings. Directory needles include a leading
+ * slash so they match path components after the caller slash-prefixes relative
+ * paths. `/test/` already covers Maven `src/test` and `/test/fixtures/`;
+ * `/tests/` covers Laravel `tests/Feature` and `/tests/fixtures/`.
+ */
+const TEST_PATH_SUBSTRINGS = [
+    '.test.',
+    '.spec.',
+    '__tests__/',
+    '__mocks__/',
+    '/test/',
+    '/tests/',
+    '/testing/',
+    '/spec/',
+    '/test_',
+    '/conftest.',
+    '/uitests/',
+    '.tests/',
+    '.test/',
+    '.integrationtests/',
+    '.unittests/',
+    '/testproject/',
+];
+/** Case-insensitive suffixes that already include a delimiter (`_test.py`, not `test.py`). */
+const TEST_PATH_DELIMITED_SUFFIXES = [
+    '_test.py',
+    '_test.go',
+    '_spec.rb',
+    '_test.rb',
+];
+/**
+ * Case-sensitive `Test`/`Tests`/`Spec` suffixes. Lowercasing first would also
+ * match production names such as `Contest.swift` and `Latest.php`.
+ */
+const TEST_PATH_CASED_SUFFIXES = [
+    'Tests.swift',
+    'Test.swift',
+    'Tests.cs',
+    'Test.cs',
+    'Test.php',
+    'Spec.php',
+];
+/** Absent / empty paths are not test paths. */
+export function isTestFilePath(filePath) {
+    if (!filePath)
+        return false;
+    const slashed = filePath.replace(/\\/g, '/');
+    const prefixed = slashed.startsWith('/') ? slashed : `/${slashed}`;
+    const lower = prefixed.toLowerCase();
+    if (TEST_PATH_SUBSTRINGS.some((needle) => lower.includes(needle)))
+        return true;
+    if (TEST_PATH_DELIMITED_SUFFIXES.some((suffix) => lower.endsWith(suffix)))
+        return true;
+    // Xcode `{Product}UITests` targets. Slash-anchored `/uitests/` does not match
+    // `MyAppUITests`; an unanchored `uitests/` substring also matches `fruitests`.
+    if (prefixed.split('/').some((seg) => seg.endsWith('UITests')))
+        return true;
+    const basename = prefixed.slice(prefixed.lastIndexOf('/') + 1);
+    return TEST_PATH_CASED_SUFFIXES.some((suffix) => basename.endsWith(suffix));
+}

package/dist/mcp/local/local-backend.d.ts

@@ -5,14 +5,12 @@
  * Supports multiple indexed repositories via a global registry.
  * LadybugDB connections are opened lazily per repo on first query.
  */
+import { isTestFilePath } from '../../core/ingestion/utils/test-file-path.js';
 import { type RegistryEntry, type BranchSummary } from '../../storage/repo-manager.js';
 import { GroupService } from '../../core/group/service.js';
 import { type StalenessInfo } from '../../core/git-staleness.js';
-/**
- * Quick test-file detection for filtering impact results.
- * Matches common test file patterns across all supported languages.
- */
-export declare function isTestFilePath(filePath: string | null | undefined): boolean;
+/** Shared predicate; re-exported so MCP importers keep the old public name. */
+export { isTestFilePath };
 /** Valid LadybugDB node labels for safe Cypher query construction */
 export declare const VALID_NODE_LABELS: Set<string>;
 /** Valid relation types for impact analysis filtering */
@@ -859,4 +857,3 @@ export declare class LocalBackend {
     queryProcessDetail(name: string, repoName?: string): Promise<any>;
     disconnect(): Promise<void>;
 }
-export {};

package/dist/mcp/local/local-backend.js

@@ -19,6 +19,7 @@ import { LBUG_ID_PROBE_BATCH_SIZE, LBUG_QUERY_BATCH_SIZE } from '../../core/lbug
 import { chunk, mapConcurrent } from '../../lib/utils.js';
 import { pathSuffixOf } from './path-predicate.js';
 import { toOneBasedLine } from '../../core/ingestion/utils/line-base.js';
+import { isTestFilePath } from '../../core/ingestion/utils/test-file-path.js';
 import { isWalCorruptionError, WAL_RECOVERY_SUGGESTION } from '../../core/lbug/lbug-config.js';
 // Embedding imports are lazy (dynamic import) to avoid loading onnxruntime-node
 // at MCP server startup — crashes on unsupported Node ABI versions (#89)
@@ -190,30 +191,8 @@ function normalizeToolParams(method, params) {
 }
 // AI context generation is CLI-only (gitnexus analyze)
 // import { generateAIContextFiles } from '../../cli/ai-context.js';
-/**
- * Quick test-file detection for filtering impact results.
- * Matches common test file patterns across all supported languages.
- */
-export function isTestFilePath(filePath) {
-    if (!filePath)
-        return false;
-    const p = filePath.toLowerCase().replace(/\\/g, '/');
-    return (p.includes('.test.') ||
-        p.includes('.spec.') ||
-        p.includes('__tests__/') ||
-        p.includes('__mocks__/') ||
-        p.includes('/test/') ||
-        p.includes('/tests/') ||
-        p.includes('/testing/') ||
-        p.includes('/fixtures/') ||
-        p.endsWith('_test.go') ||
-        p.endsWith('_test.py') ||
-        p.endsWith('_spec.rb') ||
-        p.endsWith('_test.rb') ||
-        p.includes('/spec/') ||
-        p.includes('/test_') ||
-        p.includes('/conftest.'));
-}
+/** Shared predicate; re-exported so MCP importers keep the old public name. */
+export { isTestFilePath };
 /** Valid LadybugDB node labels for safe Cypher query construction */
 export const VALID_NODE_LABELS = new Set([
     'File',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitnexus",
-  "version": "1.6.11-rc.31",
+  "version": "1.6.11-rc.32",
   "description": "Graph-powered code intelligence for AI agents. Index any codebase, query via MCP or CLI.",
   "author": "Abhigyan Patwari",
   "license": "PolyForm-Noncommercial-1.0.0",