fix: persist zsh history on Kali look, skip TerminalLook module cleanly on missing prereqs
- write_zshrc_kali now sets HISTFILE/HISTSIZE/SAVEHIST and share_history/ hist_ignore_dups so command history survives across sessions (previously unset, so history was in-memory only despite shipping zsh-history-substring-search and fzf history search). - apply_terminal_look now guards install_terminal_prereqs and returns 0 with a skip message on failure, matching apply_hyprland's existing pattern, so an unrecognized package manager no longer aborts the whole script under set -euo pipefail and blocks a Hyprland module that would otherwise succeed. Addresses findings 1 and 2 from the final whole-branch review.
This commit is contained in:
1 parent
2c5fe486fe
commit
6183c7859b
1 file changed
+10
-1
+10
-1
@@ -421,6 +421,11 @@ apply_terminal_look_p10k() {
|
||||
write_zshrc_kali() {
|
||||
local dest="$1"
|
||||
cat > "$dest" <<'EOF'
|
||||
HISTFILE=~/.zsh_history
|
||||
HISTSIZE=10000
|
||||
SAVEHIST=10000
|
||||
setopt share_history hist_ignore_dups
|
||||
|
||||
source "$HOME/.dotevo/kali-prompt.zsh"
|
||||
source "$HOME/.dotevo-zsh-custom/plugins/zsh-autosuggestions/zsh-autosuggestions.zsh"
|
||||
source "$HOME/.dotevo-zsh-custom/plugins/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh"
|
||||
@@ -454,7 +459,11 @@ chsh_to_zsh() {
|
||||
}
|
||||
|
||||
apply_terminal_look() {
|
||||
install_terminal_prereqs
|
||||
if ! install_terminal_prereqs; then
|
||||
echo "dotevo: skipping Terminal look -- prerequisites unavailable"
|
||||
return 0
|
||||
fi
|
||||
|
||||
run_terminal_look_picker
|
||||
|
||||
case "$PICK_TERMINAL_LOOK" in
|
||||
|
||||
Reference in new issue
Block a user