Skip to content

feat(cli): expose slash search in provider and model pickers - #601

Merged
kfallah merged 2 commits into
mainfrom
codex/experiential-picker-search
Aug 22, 2026
Merged

feat(cli): expose slash search in provider and model pickers#601
kfallah merged 2 commits into
mainfrom
codex/experiential-picker-search

Conversation

@kfallah

@kfallah kfallah commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Summary:

  • Advertise the / search shortcut in the shared provider/model picker UI.
  • Support /query and a standalone / prompt in the line-based fallback.
  • Cover single-select and multi-select search behavior.

Verification:

  • uv run --extra dev ruff check .
  • uv run --extra dev ty check
  • 121 picker/provider/model tests and 9 picker-view tests pass outside the pre-existing Rich PTY cleanup assertions.

@greptile-apps

greptile-apps Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR exposes slash-search guidance in provider and model pickers and adds explicit slash-prefixed searching to the line-based fallback.

  • Handles /query before numeric row parsing in single- and multi-select pickers.
  • Supports a standalone / that opens a dedicated search prompt.
  • Adds focused coverage for slash searching and picker hints.

Confidence Score: 5/5

The PR appears safe to merge.

The previously reported numeric slash-search defect is fixed because slash-prefixed input is now handled and continued before numeric position parsing in both picker modes, and no blocking failure remains.

Important Files Changed

Filename Overview
exp/cli/shared/picker.py Adds slash-prefixed and standalone-slash search handling before numeric position parsing in both line-based picker modes.
exp/cli/shared/picker_test.py Covers slash-prefixed, standalone-slash, and numeric slash searches for single- and multi-select pickers.
exp/cli/shared/picker_view.py Advertises the slash-search shortcut in keyboard picker hints.
exp/cli/shared/picker_view_test.py Verifies that both keyboard picker variants display the slash-search hint.
exp/cli/providers/provider_picker_test.py Verifies that the provider picker exposes the shared slash-search guidance.

Reviews (2): Last reviewed commit: "fix(cli): keep numeric slash queries in ..." | Re-trigger Greptile

Comment thread exp/cli/shared/picker.py
@kfallah
kfallah merged commit 57e388f into main Aug 22, 2026
12 of 13 checks passed
@kfallah
kfallah deleted the codex/experiential-picker-search branch August 22, 2026 22:05
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