Preserve selected message in inspector on updates - #249
Conversation
Stop always jumping to the latest row when new LSP traffic arrives. Auto-follow only when the cursor is already on the last message (swyddfa#247).
|
pre-commit.ci is failing with the same ruff/mypy issues present on develop (not introduced here). Happy to help clean those up in a follow-up if needed. |
|
Thanks for opening this!
Yeah, it's a sign that the code is relatively immature still, happy to ignore any existing errors for now. I've not had chance to try the branch out myself yet, but the implementation looks good. If you're adding tests though, I'd prefer it them to use a |
Drop the _FakeDb stub in favor of an in-memory SqlHandler, as requested in review. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Thanks for the review! Updated the tests to use |
|
Thanks! I've finally had a chance to try it out and can confirm it works as expected 😄 |
Summary
lsp-devtools inspectand the client inspectorTest plan
cd lib/lsp-devtools && hatch test(or pytest ontests/inspector/test_message_browser.py)lsp-devtools inspect, feed chatty traffic, select a middle row — selection should stick