awesome-emacs/doomemacs
1
0
Fork 0
mirror of https://github.com/doomemacs/doomemacs synced 2025-08-01 12:17:25 -05:00
Code Issues Packages Projects Releases Wiki Activity

feat(:term): confine undo to comint/eshell prompts

Browse Source 
Otherwise, undo could delete entire chunks of a shell buffer. This
addresses the issue in eshell, comint shells, and derivatives (like
ielm, shell, or inferior-* shells).

Ref: #8410
This commit is contained in:
Henrik Lissner
2025-06-30 20:54:21 +02:00
parent 1b125ddf7b
commit 5a690fc54f
2 changed files with 22 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
lisp/doom-ui.el
View File

@ -350,6 +350,20 @@ windows, switch to `doom-fallback-buffer'. Otherwise, delegate to original
(after! comint
(setq-default comint-buffer-maximum-size 2048) ; double the default
;; UX: Temporarily disable undo history between command executions. Otherwise,
;; undo could destroy output while it's being printed or delete buffer
;; contents past the boundaries of the current prompt.
(add-hook 'comint-exec-hook #'buffer-disable-undo)
(defadvice! doom--comint-enable-undo-a (process _string)
:after #'comint-output-filter
(let ((start-marker comint-last-output-start))
(when (and (< start-marker
(or (if process (process-mark process))
(point-max-marker)))
(eq (char-before start-marker) ?\n)) ;; Account for some of the IELM’s wilderness.
(buffer-enable-undo)
(setq buffer-undo-list nil))))
;; Protect prompts from accidental modifications.
(setq-default comint-prompt-read-only t)

15
modules/term/eshell/config.el
View File

@ -85,15 +85,14 @@ You should use `set-eshell-alias!' to change this.")
(add-hook 'eshell-mode-hook #'+eshell-init-h)
(add-hook 'eshell-exit-hook #'+eshell-cleanup-h)
;; UX: Temporarily disable undo history between command executions. Undo can
;; destroy output while it's being printed to stdout.
(let (old-undo-list)
(add-hook! 'eshell-pre-command-hook
(setq old-undo-list buffer-undo-list
buffer-undo-list nil))
;; UX: Temporarily disable undo history between command executions. Otherwise,
;; undo could destroy output while it's being printed or delete buffer
;; contents past the boundaries of the current prompt.
(add-hook 'eshell-pre-command-hook #'buffer-disable-undo)
(add-hook! 'eshell-post-command-hook
(setq buffer-undo-list old-undo-list)
(clrhash undo-equiv-table)))
(defun +eshell--enable-undo-h ()
(buffer-enable-undo (current-buffer))
(setq buffer-undo-list nil)))
;; UX: Prior output in eshell buffers should be read-only. Otherwise, it's
;; trivial to make edits in visual modes (like evil's or term's