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 | 59b0854464ef29511a0d09f1e76fd1140e675833 (patch) | |
| tree | 0c18aee463b4ff14f2fa6675e2cde6fc5f50bbdc /tests/test-vterm-toggle--buffer-filter.el | |
| parent | de555fa8b48c5ed5f17c0a8db9de7ecb946aa75d (diff) | |
| download | dotemacs-59b0854464ef29511a0d09f1e76fd1140e675833.tar.gz dotemacs-59b0854464ef29511a0d09f1e76fd1140e675833.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-vterm-toggle--buffer-filter.el')
| -rw-r--r-- | tests/test-vterm-toggle--buffer-filter.el | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/tests/test-vterm-toggle--buffer-filter.el b/tests/test-vterm-toggle--buffer-filter.el index 82afe756..d6fd2c8c 100644 --- a/tests/test-vterm-toggle--buffer-filter.el +++ b/tests/test-vterm-toggle--buffer-filter.el @@ -4,8 +4,8 @@ ;; Three closely-related helpers determine which vterm buffers F12 ;; manages: the predicate `cj/--vterm-toggle-buffer-p', the MRU list ;; `cj/--vterm-toggle-buffers', and the per-frame window finder -;; `cj/--vterm-toggle-displayed-window'. All three exclude claude- -;; prefixed buffers so claude has its own F9 surface. +;; `cj/--vterm-toggle-displayed-window'. All three exclude agent- +;; prefixed buffers so agent has its own F9 surface. ;;; Code: @@ -17,22 +17,22 @@ (require 'testutil-vterm-buffers) (defun test-vterm-toggle--cleanup () - "Kill leftover claude- and *test-vterm- prefixed buffers." - (cj/test--kill-claude-buffers) + "Kill leftover agent- and *test-vterm- prefixed buffers." + (cj/test--kill-agent-buffers) (cj/test--kill-test-vterm-buffers)) (ert-deftest test-vterm-toggle--buffer-p-accepts-vterm-mode () - "Normal: a vterm-mode buffer with non-claude name qualifies." + "Normal: a vterm-mode buffer with non-agent name qualifies." (test-vterm-toggle--cleanup) (let ((buf (cj/test--make-fake-vterm-buffer "*test-vterm-1*"))) (unwind-protect (should (cj/--vterm-toggle-buffer-p buf)) (kill-buffer buf)))) -(ert-deftest test-vterm-toggle--buffer-p-rejects-claude () - "Boundary: claude-prefixed vterm buffers are excluded from F12's set." +(ert-deftest test-vterm-toggle--buffer-p-rejects-agent () + "Boundary: agent-prefixed vterm buffers are excluded from F12's set." (test-vterm-toggle--cleanup) - (let ((buf (cj/test--make-fake-vterm-buffer "claude [project-a]"))) + (let ((buf (cj/test--make-fake-vterm-buffer "agent [project-a]"))) (unwind-protect (should-not (cj/--vterm-toggle-buffer-p buf)) (kill-buffer buf)))) @@ -52,17 +52,17 @@ (kill-buffer buf) (should-not (cj/--vterm-toggle-buffer-p buf)))) -(ert-deftest test-vterm-toggle--buffers-filters-claude () - "Normal: returns vterm buffers but excludes claude-prefixed ones." +(ert-deftest test-vterm-toggle--buffers-filters-agent () + "Normal: returns vterm buffers but excludes agent-prefixed ones." (test-vterm-toggle--cleanup) (let ((normal (cj/test--make-fake-vterm-buffer "*test-vterm-normal*")) - (claude (cj/test--make-fake-vterm-buffer "claude [for-test]"))) + (agent (cj/test--make-fake-vterm-buffer "agent [for-test]"))) (unwind-protect (let ((result (cj/--vterm-toggle-buffers))) (should (memq normal result)) - (should-not (memq claude result))) + (should-not (memq agent result))) (kill-buffer normal) - (kill-buffer claude)))) + (kill-buffer agent)))) (ert-deftest test-vterm-toggle--displayed-window-finds-vterm () "Normal: vterm in a window -> returns that window." @@ -78,17 +78,17 @@ (should (eq (window-buffer result) vt))))) (kill-buffer vt)))) -(ert-deftest test-vterm-toggle--displayed-window-skips-claude () - "Boundary: only a claude vterm is displayed -> nil (claude not F12-managed)." +(ert-deftest test-vterm-toggle--displayed-window-skips-agent () + "Boundary: only an agent vterm is displayed -> nil (agent not F12-managed)." (test-vterm-toggle--cleanup) - (let ((claude (cj/test--make-fake-vterm-buffer "claude [skip-test]"))) + (let ((agent (cj/test--make-fake-vterm-buffer "agent [skip-test]"))) (unwind-protect (save-window-excursion (delete-other-windows) (let ((win (split-window-right))) - (set-window-buffer win claude) + (set-window-buffer win agent) (should-not (cj/--vterm-toggle-displayed-window)))) - (kill-buffer claude)))) + (kill-buffer agent)))) (provide 'test-vterm-toggle--buffer-filter) ;;; test-vterm-toggle--buffer-filter.el ends here |
