Skip to content

fix(tui): route mcp list paste into search#356

Open
OnlyYu1996 wants to merge 1 commit into
CortexLM:mainfrom
OnlyYu1996:fix-mcp-list-paste-search
Open

fix(tui): route mcp list paste into search#356
OnlyYu1996 wants to merge 1 commit into
CortexLM:mainfrom
OnlyYu1996:fix-mcp-list-paste-search

Conversation

@OnlyYu1996
Copy link
Copy Markdown

Fixes PlatformNetwork/bounty-challenge#53265.

Summary

  • Route pasted text in the MCP Servers list modal into the same searchable SelectionList used by typed filtering.
  • Make the modal consume list-mode paste events so clipboard text no longer falls through to the hidden session composer.
  • Add a regression test covering paste updating the list search query and selected filtered server.

Validation

  • rustfmt +stable --edition 2024 --check src/cortex-tui/src/modal/mcp_manager/mod.rs src/cortex-tui/src/widgets/selection_list.rs
  • cargo +stable test --package cortex-tui test_list_mode_paste_updates_search_query -- --nocapture
  • git diff --check

Note: cargo +stable fmt --package cortex-tui --check is currently blocked by an unrelated existing formatting diff in src/cortex-tui/src/question.rs; the touched files pass rustfmt directly.

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant