diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-11 07:18:20 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-11 07:18:20 -0500 |
| commit | e221bf4c61be057c9408e4b11550d1e365ec065f (patch) | |
| tree | bb0ad65fb1cb9b1dba629998fe2983a9a3020fa7 /tests/test-ai-vterm--displayed-claude-window.el | |
| parent | 5f65d6f32820cab4b7486d90bae3020dcb478428 (diff) | |
| download | dotemacs-e221bf4c61be057c9408e4b11550d1e365ec065f.tar.gz dotemacs-e221bf4c61be057c9408e4b11550d1e365ec065f.zip | |
refactor(ai-vterm): rename Claude-specific names to a generic "agent"
I may add other terminal agents to this launcher (aider, an open-source LLM TUI), so the buffer prefix, the user knob, and the internal helpers shouldn't say "Claude". The module name (ai-vterm) and the `cj/ai-vterm-*` customs were already generic. This finishes the job:
- buffer prefix `claude [<basename>]` -> `agent [<basename>]` (the `defconst` and the matching display-buffer-alist regex move together)
- `cj/ai-vterm-claude-command` -> `cj/ai-vterm-agent-command` (the default still runs the `claude` CLI, with a docstring note on swapping it)
- `cj/--ai-vterm-claude-buffers` / `-displayed-claude-window` / `-reuse-existing-claude` -> `-agent-*`, and their test files renamed to match
- prose in the module commentary and docstrings, plus the matching test docstrings and buffer-name literals
`vterm-config.el` hardcodes the same buffer prefix in `cj/--vterm-toggle-buffer-p` (F12 excludes agent buffers from its candidate set), so that literal moved too. Collapsing it into the shared `cj/--ai-vterm-name-prefix` is a cleanup for another day.
After a reload, a project's buffer opens as `agent [foo]` instead of `claude [foo]`. Old buffers keep their names until killed. I also corrected two stale `eshell-vterm-config.el` references in ai-vterm.el docstrings (that module was split into `vterm-config.el`).
Two things keep saying "Claude": the `cj/ai-vterm-agent-command` default value (the actual CLI), and the "Claude Code" example in `vterm-config.el`'s cursor-restore docstring (a concrete TUI example, not branding).
90 tests pass. `make validate-modules` clean.
Diffstat (limited to 'tests/test-ai-vterm--displayed-claude-window.el')
| -rw-r--r-- | tests/test-ai-vterm--displayed-claude-window.el | 60 |
1 files changed, 0 insertions, 60 deletions
diff --git a/tests/test-ai-vterm--displayed-claude-window.el b/tests/test-ai-vterm--displayed-claude-window.el deleted file mode 100644 index 9f84f87b..00000000 --- a/tests/test-ai-vterm--displayed-claude-window.el +++ /dev/null @@ -1,60 +0,0 @@ -;;; test-ai-vterm--displayed-claude-window.el --- Tests for the displayed-window helper -*- lexical-binding: t; -*- - -;;; Commentary: -;; The helper returns a window in the selected frame whose buffer -;; satisfies `cj/--ai-vterm-buffer-p', or nil when no such window -;; exists. Used by F9 dispatch and M-F9 in-place replacement. - -;;; Code: - -(require 'ert) - -(add-to-list 'load-path (expand-file-name "modules" user-emacs-directory)) -(add-to-list 'load-path (expand-file-name "tests" user-emacs-directory)) -(require 'ai-vterm) -(require 'testutil-vterm-buffers) - -(ert-deftest test-ai-vterm--displayed-claude-window-no-buffers-returns-nil () - "Boundary: no claude buffers anywhere -> nil." - (cj/test--kill-claude-buffers) - (save-window-excursion - (delete-other-windows) - (should-not (cj/--ai-vterm-displayed-claude-window)))) - -(ert-deftest test-ai-vterm--displayed-claude-window-not-displayed-returns-nil () - "Boundary: claude buffer exists but not in any window -> nil." - (cj/test--kill-claude-buffers) - (let ((b1 (get-buffer-create "claude [hidden]"))) - (unwind-protect - (save-window-excursion - (delete-other-windows) - (should-not (cj/--ai-vterm-displayed-claude-window))) - (kill-buffer b1)))) - -(ert-deftest test-ai-vterm--displayed-claude-window-returns-window-when-displayed () - "Normal: claude buffer in a window -> returns that window." - (cj/test--kill-claude-buffers) - (let ((b1 (get-buffer-create "claude [shown]"))) - (unwind-protect - (save-window-excursion - (delete-other-windows) - (let ((win (split-window-right))) - (set-window-buffer win b1) - (let ((result (cj/--ai-vterm-displayed-claude-window))) - (should (windowp result)) - (should (eq (window-buffer result) b1))))) - (kill-buffer b1)))) - -(ert-deftest test-ai-vterm--displayed-claude-window-ignores-non-claude-windows () - "Boundary: only a non-claude buffer is displayed -> nil." - (cj/test--kill-claude-buffers) - (let ((other (get-buffer-create "regular-displayed-buffer"))) - (unwind-protect - (save-window-excursion - (delete-other-windows) - (set-window-buffer (selected-window) other) - (should-not (cj/--ai-vterm-displayed-claude-window))) - (kill-buffer other)))) - -(provide 'test-ai-vterm--displayed-claude-window) -;;; test-ai-vterm--displayed-claude-window.el ends here |
