Skip to content

[codex] Honor file_pattern in BM25 graph search#555

Merged
DeusData merged 1 commit into
DeusData:mainfrom
pcristin:codex/552-search-graph-file-pattern
Jun 22, 2026
Merged

[codex] Honor file_pattern in BM25 graph search#555
DeusData merged 1 commit into
DeusData:mainfrom
pcristin:codex/552-search-graph-file-pattern

Conversation

@pcristin

@pcristin pcristin commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #552

Validation

  • git diff --check
  • make -f Makefile.cbm build/c/test-runner
  • build/c/test-runner | awk 'BEGIN{show=0} /^=== mcp ===/{show=1} /^=== language ===/{exit} show{print}'
  • Actual CLI usage with patched checkout binary and isolated CBM_CACHE_DIR:
    • Indexed a throwaway repo containing both src/lib/status.c and src/components/status.c.
    • search_graph with query:"status" returned both BM25 results.
    • search_graph with query:"status", file_pattern:"src/lib/*" returned only src/lib/status.c with total:1.

Signed-off-by: pcristin <xxxokzxxx@protonmail.com>
@pcristin pcristin marked this pull request as ready for review June 22, 2026 11:02
@DeusData DeusData merged commit 0618eaa into DeusData:main Jun 22, 2026
13 checks passed
@pcristin pcristin deleted the codex/552-search-graph-file-pattern branch June 22, 2026 21:01
@studyzy studyzy mentioned this pull request Jun 23, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

search_graph: file_pattern returns unfiltered results on 0.8.1 (follow-up to #200)

2 participants