riched 0.4.2__tar.gz → 0.4.3__tar.gz

{riched-0.4.2 → riched-0.4.3}/AGENTS.md

@@ -19,6 +19,8 @@ Use Python 3.12-compatible code with type hints, `from __future__ import annotat
 
 Keep key bindings in `src/riched/bindings.yaml`, not hardcoded in Python. The F1 hotkey popup is generated from this YAML and displays one user-facing key per binding, preferring `cmd` aliases over `super`.
 
+When changing app-level key bindings, test both file-open and no-buffer/directory-start states. For Command-key regressions, verify both Textual names (`cmd+...` and terminal-normalized `super+...`) because Pilot can pass for synthetic keys while a real terminal path or missing app state still breaks. Do not patch Command-key behavior by adding Python-side hardcoded fallbacks; add aliases to `bindings.yaml` and e2e coverage instead.
+
 ## Testing Guidelines
 
 Only add end-to-end tests in `tests/`; do not create unit tests. Tests use Textual's Pilot harness and should be named `test_<behavior>`. Keep each test focused on user-visible behavior such as file open/save, dirty prompts, file tree switching, quick-open search/indexing, or syntax highlighting. Use temporary files as existing tests do.

{riched-0.4.2 → riched-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: riched
-Version: 0.4.2
+Version: 0.4.3
 Summary: A Rich library powered editor(hence the name riched) that implements VS Code keybindings in TUI
 Project-URL: Repository, https://github.com/hoishing/riched
 Requires-Python: >=3.12

{riched-0.4.2 → riched-0.4.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "riched"
-version = "0.4.2"
+version = "0.4.3"
 description = "A Rich library powered editor(hence the name riched) that implements VS Code keybindings in TUI"
 readme = "README.md"
 requires-python = ">=3.12"

{riched-0.4.2 → riched-0.4.3}/src/riched/app.py

@@ -545,10 +545,9 @@ class RichedApp(App):
     def action_toggle_file_tree(self) -> None:
         editor = self._editor_or_none()
         if self._is_file_tree_visible():
-            if editor is None:
-                return
             self._hide_file_tree()
-            editor.focus()
+            if editor is not None:
+                editor.focus()
             return
         self._show_file_tree()
 

{riched-0.4.2 → riched-0.4.3}/tests/runner.py

@@ -32,6 +32,7 @@ from .test_editor_shortcuts import (
     test_cmd_backspace_deletes_selection,
     test_cmd_backspace_deletes_to_line_start,
     test_cmd_b_toggles_file_tree,
+    test_cmd_b_toggles_file_tree_without_open_buffer,
     test_cmd_l_repeats_expand_line_selection,
     test_cmd_l_selects_current_line_with_newline,
     test_cmd_l_selects_final_line_without_newline,
@@ -56,6 +57,7 @@ from .test_editor_shortcuts import (
     test_parser_order_super_shift_line_selection_aliases,
     test_super_l_selects_current_line_alias,
     test_super_b_toggles_file_tree_alias,
+    test_super_b_toggles_file_tree_without_open_buffer,
     test_super_shift_k_deletes_current_line_alias,
     test_super_shift_line_selection_aliases,
     test_undo_multiline_insert_that_removes_scrollbar,
@@ -183,7 +185,15 @@ TESTS: list[tuple[str, Callable[[], Awaitable[None]]]] = [
     ),
     ("editor: alt+z toggles word wrap", test_alt_z_toggles_word_wrap),
     ("app: cmd+b toggles file tree", test_cmd_b_toggles_file_tree),
+    (
+        "app: cmd+b toggles file tree without open buffer",
+        test_cmd_b_toggles_file_tree_without_open_buffer,
+    ),
     ("app: super+b toggles file tree alias", test_super_b_toggles_file_tree_alias),
+    (
+        "app: super+b toggles file tree without open buffer",
+        test_super_b_toggles_file_tree_without_open_buffer,
+    ),
     (
         "editor: undo multiline insert that removes scrollbar",
         test_undo_multiline_insert_that_removes_scrollbar,

{riched-0.4.2 → riched-0.4.3}/tests/test_editor_shortcuts.py

@@ -201,6 +201,21 @@ async def test_cmd_b_toggles_file_tree() -> None:
         assert tree.styles.display == "block"
 
 
+async def test_cmd_b_toggles_file_tree_without_open_buffer() -> None:
+    tmp, _ = _fresh_env()
+    app = _make_app(tmp, root=tmp)
+    async with app.run_test() as pilot:
+        await pilot.pause()
+        tree = app.query_one("#file-tree", DirectoryTree)
+        assert tree.styles.display == "block"
+        await pilot.press("cmd+b")
+        await pilot.pause()
+        assert tree.styles.display == "none"
+        await pilot.press("cmd+b")
+        await pilot.pause()
+        assert tree.styles.display == "block"
+
+
 async def test_super_b_toggles_file_tree_alias() -> None:
     tmp, _ = _fresh_env()
     f = tmp / "tree.txt"
@@ -217,6 +232,21 @@ async def test_super_b_toggles_file_tree_alias() -> None:
         assert tree.styles.display == "block"
 
 
+async def test_super_b_toggles_file_tree_without_open_buffer() -> None:
+    tmp, _ = _fresh_env()
+    app = _make_app(tmp, root=tmp)
+    async with app.run_test() as pilot:
+        await pilot.pause()
+        tree = app.query_one("#file-tree", DirectoryTree)
+        assert tree.styles.display == "block"
+        await pilot.press("super+b")
+        await pilot.pause()
+        assert tree.styles.display == "none"
+        await pilot.press("super+b")
+        await pilot.pause()
+        assert tree.styles.display == "block"
+
+
 async def test_undo_multiline_insert_that_removes_scrollbar() -> None:
     tmp, _ = _fresh_env()
     f = tmp / "undo.txt"

{riched-0.4.2 → riched-0.4.3}/uv.lock

@@ -131,7 +131,7 @@ wheels = [
 
 [[package]]
 name = "riched"
-version = "0.4.2"
+version = "0.4.3"
 source = { editable = "." }
 dependencies = [
     { name = "pyyaml" },