From 4000330e5e6536f64f404c1cabc8dc86d9556bb5 Mon Sep 17 00:00:00 2001 From: Craig Jennings Date: Mon, 11 May 2026 07:18:20 -0500 Subject: 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 []` -> `agent []` (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. --- tests/test-ai-vterm--claude-buffers.el | 59 ---------------------------------- 1 file changed, 59 deletions(-) delete mode 100644 tests/test-ai-vterm--claude-buffers.el (limited to 'tests/test-ai-vterm--claude-buffers.el') diff --git a/tests/test-ai-vterm--claude-buffers.el b/tests/test-ai-vterm--claude-buffers.el deleted file mode 100644 index f975b64e..00000000 --- a/tests/test-ai-vterm--claude-buffers.el +++ /dev/null @@ -1,59 +0,0 @@ -;;; test-ai-vterm--claude-buffers.el --- Tests for cj/--ai-vterm-claude-buffers -*- lexical-binding: t; -*- - -;;; Commentary: -;; The helper returns the list of buffers whose names start with the -;; literal prefix "claude [". Order is the same order `buffer-list' -;; gives them (most-recently-selected first). Non-claude buffers and -;; buffers whose names merely contain the prefix as a substring are -;; excluded. - -;;; 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--claude-buffers-empty-when-none-exist () - "Boundary: no claude-prefixed buffers anywhere -> empty list." - (cj/test--kill-claude-buffers) - (unwind-protect - (should (null (cj/--ai-vterm-claude-buffers))) - (cj/test--kill-claude-buffers))) - -(ert-deftest test-ai-vterm--claude-buffers-returns-only-claude-buffers () - "Normal: filters to only claude-prefixed buffers, leaves others alone." - (cj/test--kill-claude-buffers) - (let ((c1 (get-buffer-create "claude [a]")) - (c2 (get-buffer-create "claude [b]")) - (other (get-buffer-create "regular-buffer"))) - (unwind-protect - (let ((result (cj/--ai-vterm-claude-buffers))) - (should (memq c1 result)) - (should (memq c2 result)) - (should-not (memq other result)) - (should (= (length result) 2))) - (kill-buffer c1) - (kill-buffer c2) - (kill-buffer other)))) - -(ert-deftest test-ai-vterm--claude-buffers-anchors-prefix-not-substring () - "Boundary: 'foo claude [bar]' is not a claude buffer -- prefix anchored." - (cj/test--kill-claude-buffers) - (let ((not-claude (get-buffer-create "foo claude [bar]"))) - (unwind-protect - (should-not (memq not-claude (cj/--ai-vterm-claude-buffers))) - (kill-buffer not-claude)))) - -(ert-deftest test-ai-vterm--claude-buffers-bare-claude-not-included () - "Boundary: 'claude' alone (no bracket) doesn't match the 'claude [' prefix." - (cj/test--kill-claude-buffers) - (let ((bare (get-buffer-create "claude"))) - (unwind-protect - (should-not (memq bare (cj/--ai-vterm-claude-buffers))) - (kill-buffer bare)))) - -(provide 'test-ai-vterm--claude-buffers) -;;; test-ai-vterm--claude-buffers.el ends here -- cgit v1.2.3