Keybindings refactor#2
Merged
Merged
Conversation
- Add TursoAdapter using libsql_client library - Support libsql:// and https:// URL schemes - Add Turso to DatabaseType enum and schema definitions - Add Turso badge to tree view - Add integration tests for Turso - Add libsql-server container to docker-compose for testing
- Use adapter.execute_query/execute_non_query instead of direct cursor access - Add has_cursor check in commands.py for streaming CSV/JSON output - Fixes "'ClientSync' object has no attribute 'cursor'" error with Turso
- Add SSH server container (linuxserver/openssh-server) with TCP forwarding - Add postgres-ssh container as tunnel target - Add SSH connection fixtures and test helpers - Add SSH tunnel integration tests - Configure MaxStartups for high connection throughput in tests
- Add test_cancellable_query_select for TUI async SELECT path - Add test_cancellable_query_insert for TUI async INSERT path - Add test_streaming_csv_output for CLI streaming path - Add test_streaming_json_output for CLI streaming path - Add test_adapter_interface_compliance to verify adapter methods - Add test_query_service_execution for QueryService path - These tests ensure all code paths are covered across all adapters
Keybindings: - Refactor to use centralized keybinding definitions - Add leader key menu for discoverable shortcuts - Add connection picker with fuzzy search UX improvements: - Add query execution timing to notifications (e.g. "in 156ms") - Add JSON/Python literal pretty-printing in value view - Add zebra striping to results table - Improve validation feedback in connection forms Testing: - Add validation unit tests - Add UI tests for explorer toggle and fullscreen - Add keybindings tests - Add connection picker tests
- Document Turso database support - Update uv.lock with new dependencies
- Add test-unit job for validation and UI tests - Add test-turso job for Turso integration tests
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.