Skip to content

[querier] Ignore diff context matches [FCOM-14] - #1138

Merged
davidrunger merged 1 commit into
mainfrom
parser/ignore-context-matches
Aug 28, 2026
Merged

[querier] Ignore diff context matches [FCOM-14]#1138
davidrunger merged 1 commit into
mainfrom
parser/ignore-context-matches

Conversation

@davidrunger

Copy link
Copy Markdown
Owner

Git emits three lines of patch context by default. Because the rg stage searched every emitted line, a match in unchanged context could make fcom print a commit or file without a matching added or removed line.

Request zero context lines with Git --unified=0 and keep the command contract covered by the existing Fcom::Querier spec. This preserves changed-line output while preventing context-only results.

codex resume 01a04984-da1c-7712-bcfb-a9a54ee38a12

Git emits three lines of patch context by default. Because the `rg` stage searched every emitted line, a match in unchanged context could make `fcom` print a commit or file without a matching added or removed line.

Request zero context lines with Git `--unified=0` and keep the command contract covered by the existing `Fcom::Querier` spec. This preserves changed-line output while preventing context-only results.

codex resume 01a04984-da1c-7712-bcfb-a9a54ee38a12
@davidrunger
davidrunger merged commit c87dd56 into main Aug 28, 2026
1 check passed
@davidrunger
davidrunger deleted the parser/ignore-context-matches branch August 28, 2026 18:03
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.

1 participant