Skip to content

Improve repo grep UX across CLI and MCP - #31

Merged
jlitola merged 4 commits into
mainfrom
fix/grep-cli-ux
Apr 23, 2026
Merged

Improve repo grep UX across CLI and MCP#31
jlitola merged 4 commits into
mainfrom
fix/grep-cli-ux

Conversation

@jlitola

@jlitola jlitola commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the old single-file grep flow with repo-wide grepRepo across CLI and MCP via a shared request/response contract
  • polish CLI grep output with deduped same-line matches, merged context blocks, concise pagination hints, compact rg-style context rendering, and clearer summary wording
  • rewrite missing-navpack failures into actionable CLI guidance and add a narrow-scope hint for noisy broad searches

Verification

  • bun test
  • bun run typecheck
  • bun run build
  • live smoke tests against real package and repo targets using code grep

jlitola added 4 commits April 23, 2026 13:45
Replace the deprecated single-file grep flow with deterministic repo-wide grep backed by grepRepo so CLI and MCP share one contract. This aligns the public surfaces, updates docs and parity tests, and fixes validation, context rendering, and typecheck drift discovered during rollout.
Tighten repo grep terminal output with line deduping, merged context blocks, concise pagination hints, and clearer summary wording. Also rewrite missing-navpack failures into actionable CLI guidance and add a narrow-scope hint for noisy broad grep results.
Make grep context output closer to ripgrep by printing each file path once per block and emitting compact line-prefixed context rows. This keeps plain no-context output unchanged while making -C/-A/-B results much easier to scan.
Use compact heading-style output for terminal grep sessions while keeping file:line:text output for pipes and other non-TTY consumers. This makes the default interactive experience much closer to ripgrep without breaking machine-friendly output.
@jlitola
jlitola merged commit 4502838 into main Apr 23, 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