Skip to content

feat: context lines (-A/-B/-C) for standard search#1

Merged
botirk38 merged 5 commits intomasterfrom
phase-3-context-color-null
Apr 3, 2026
Merged

feat: context lines (-A/-B/-C) for standard search#1
botirk38 merged 5 commits intomasterfrom
phase-3-context-color-null

Conversation

@botirk38
Copy link
Copy Markdown
Owner

@botirk38 botirk38 commented Apr 3, 2026

  • SearchOptions: before_context, after_context; SearcherBuilder wiring and cache key
  • StandardSink: context + context_break; match vs context line-number separator (: vs -)
  • Summary/count paths disable context in searcher; only-matching clears context opts
  • CLI: resolve_context_from_args (argv order) + ContextDecl for clap acceptance
  • Tests: integration_context; update SearchOptions struct literals across bench/fuzz/profile

botirk38 added 5 commits April 3, 2026 16:45
- SearchOptions: before_context, after_context; SearcherBuilder wiring and cache key
- StandardSink: context + context_break; match vs context line-number separator (: vs -)
- Summary/count paths disable context in searcher; only-matching clears context opts
- CLI: resolve_context_from_args (argv order) + ContextDecl for clap acceptance
- Tests: integration_context; update SearchOptions struct literals across bench/fuzz/profile
- Split resident_set_bytes into cfg-specific functions to fix missing_const_for_fn on Windows
- Fix context-line prefix to use hyphen separator (path-1-context vs path:2:match)
- Add integration tests for -A, -B, filename separator, and match group separation

CI-equivalent: cargo clippy --workspace --all-targets --all-features passes
@botirk38 botirk38 merged commit 8f471dd into master Apr 3, 2026
3 checks passed
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