Remove e/E cursor motions; keep operator usage

Author: Maxteabag

README.md

@@ -205,7 +205,6 @@ Use with operators like `y`, `d`, `c` (e.g. `dw`, `y$`).
 | `h` / `j` / `k` / `l` | Move cursor left / down / up / right |
 | `w` / `W` | Next word / WORD |
 | `b` / `B` | Previous word / WORD |
-| `e` / `E` | End of word / WORD |
 | `0` / `$` | Line start / end |
 | `gg` / `G` | File start / end |
 | `f{c}` / `F{c}` | Find char forward / backward |

sqlit/core/keymap.py

@@ -350,8 +350,6 @@ def _build_action_keys(self) -> list[ActionKeyDef]:
             ActionKeyDef("W", "cursor_WORD_forward", "query_normal"),
             ActionKeyDef("b", "cursor_word_back", "query_normal"),
             ActionKeyDef("B", "cursor_WORD_back", "query_normal"),
-            ActionKeyDef("e", "cursor_word_end", "query_normal"),
-            ActionKeyDef("E", "cursor_WORD_end", "query_normal"),
             ActionKeyDef("0", "cursor_line_start", "query_normal"),
             ActionKeyDef("dollar_sign", "cursor_line_end", "query_normal"),
             ActionKeyDef("G", "cursor_last_line", "query_normal"),

sqlit/domains/query/state/query_normal.py

@@ -30,8 +30,6 @@ def _setup_actions(self) -> None:
         self.allows("cursor_WORD_forward", help="Move to next WORD")
         self.allows("cursor_word_back", help="Move to previous word")
         self.allows("cursor_WORD_back", help="Move to previous WORD")
-        self.allows("cursor_word_end", help="Move to end of word")
-        self.allows("cursor_WORD_end", help="Move to end of WORD")
         self.allows("cursor_line_start", help="Move to line start")
         self.allows("cursor_line_end", help="Move to line end")
         self.allows("cursor_last_line", help="Move to last line")

sqlit/domains/query/ui/mixins/query_editing_cursor.py

@@ -40,6 +40,7 @@ def action_g_WORD_end_back(self: QueryMixinHost) -> None:
         self._clear_leader_pending()
         self._move_with_motion("gE")
 
+
     def action_cursor_left(self: QueryMixinHost) -> None:
         """Move cursor left (h in normal mode)."""
         row, col = self.query_input.cursor_location
@@ -84,13 +85,6 @@ def action_cursor_WORD_back(self: QueryMixinHost) -> None:
         """Move cursor to previous WORD (B)."""
         self._move_with_motion("B")
 
-    def action_cursor_word_end(self: QueryMixinHost) -> None:
-        """Move cursor to end of word (e)."""
-        self._move_with_motion("e")
-
-    def action_cursor_WORD_end(self: QueryMixinHost) -> None:
-        """Move cursor to end of WORD (E)."""
-        self._move_with_motion("E")
 
     def action_cursor_line_start(self: QueryMixinHost) -> None:
         """Move cursor to start of line (0)."""

sqlit/domains/shell/state/machine.py

@@ -212,7 +212,6 @@ def binding(key: str, desc: str, indent: int = 4) -> str:
         lines.append(binding("h/j/k/l", "Cursor left/down/up/right"))
         lines.append(binding("w/W", "Word forward"))
         lines.append(binding("b/B", "Word backward"))
-        lines.append(binding("e/E", "End of word"))
         lines.append(binding("0/$", "Line start/end"))
         lines.append(binding("gg/G", "File start/end"))
         lines.append(binding("f{c}/F{c}", "Find char forward/back"))

tests/ui/keybindings/test_vim_motions.py

@@ -49,29 +49,6 @@ async def test_word_motions_w_W_b_B(self) -> None:
             await pilot.pause()
             assert app.query_input.cursor_location == (0, 0)
 
-    @pytest.mark.asyncio
-    async def test_word_end_motions_e_E(self) -> None:
-        app = _make_app()
-
-        async with app.run_test(size=(100, 35)) as pilot:
-            app.action_focus_query()
-            await pilot.pause()
-
-            app.query_input.text = "foo-bar baz"
-            app.query_input.cursor_location = (0, 0)
-            await pilot.pause()
-
-            await pilot.press("e")
-            await pilot.pause()
-            assert app.query_input.cursor_location == (0, 2)
-
-            app.query_input.cursor_location = (0, 0)
-            await pilot.pause()
-
-            await pilot.press("E")
-            await pilot.pause()
-            assert app.query_input.cursor_location == (0, 6)
-
     @pytest.mark.asyncio
     async def test_line_motions_0_dollar_G(self) -> None:
         app = _make_app()