scriptoriumadmin/GitNexus
1
0
Fork 0
mirror of https://github.com/abhigyanpatwari/GitNexus.git synced 2026-11-14 09:01:33 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

Address PR review feedback (#2866)

Browse source 
Correct the module header: scoring already matched /test/, so
/test/fixtures/ was never the scoring gap — only /conftest. was.

Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
Gergo Magyar 2026-09-01 15:50:08 +00:00
parent 17fc30cdf7
commit d4cb57b3a9
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
gitnexus/src/core/ingestion/utils/test-file-path.ts
View file

@ -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