From d4cb57b3a9245c81afbdb2818af73c27dfd49778 Mon Sep 17 00:00:00 2001 From: Gergo Magyar Date: Tue, 1 Sep 2026 15:50:08 +0000 Subject: [PATCH] Address PR review feedback (#2866) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Correct the module header: scoring already matched /test/, so /test/fixtures/ was never the scoring gap — only /conftest. was. Co-authored-by: Cursor --- gitnexus/src/core/ingestion/utils/test-file-path.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/gitnexus/src/core/ingestion/utils/test-file-path.ts b/gitnexus/src/core/ingestion/utils/test-file-path.ts index d2db3db1e..456920a89 100644 --- a/gitnexus/src/core/ingestion/utils/test-file-path.ts +++ b/gitnexus/src/core/ingestion/utils/test-file-path.ts @@ -13,7 +13,8 @@ * 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 `/test/fixtures/` and `/conftest.`. + * 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