Llewellyn/starship
1
0
Fork 0
mirror of https://github.com/Llewellynvdm/starship.git synced 2024-11-28 07:46:28 +00:00
Code Issues Releases Activity

refactor: Improve compatibility with Zsh prompt theme system (#3480)

Browse Source 
Zsh's `promptinit` expects a theme's hook functions to be named `prompt_<theme>_<hook>`. See 2876c25a28/Functions/Prompts/promptinit (L155)
This commit is contained in:
Marlon Richert 2022-02-03 22:53:38 +02:00 committed by GitHub
parent 5efb78bcd3
commit aaf4c17122
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 17 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
src/init/starship.zsh
View File

@ -23,8 +23,13 @@ else
} }
fi fi
# Will be run before every prompt draw
starship_precmd() { # The two functions below follow the naming convention `prompt_<theme>_<hook>`
# for compatibility with Zsh's prompt system. See
# https://github.com/zsh-users/zsh/blob/2876c25a28b8052d6683027998cc118fc9b50157/Functions/Prompts/promptinit#L155
# Runs before each new command line.
prompt_starship_precmd() {
# Save the status, because commands in this pipeline will change $? # Save the status, because commands in this pipeline will change $?
STARSHIP_CMD_STATUS=$? STARSHIP_PIPE_STATUS=(${pipestatus[@]}) STARSHIP_CMD_STATUS=$? STARSHIP_PIPE_STATUS=(${pipestatus[@]})
@ -41,23 +46,26 @@ starship_precmd() {
# quotes so we set it here and then use the value later on. # quotes so we set it here and then use the value later on.
STARSHIP_JOBS_COUNT=${#jobstates} STARSHIP_JOBS_COUNT=${#jobstates}
} }
starship_preexec() {
# Runs after the user submits the command line, but before it is executed.
prompt_starship_preexec() {
__starship_get_time && STARSHIP_START_TIME=$STARSHIP_CAPTURED_TIME __starship_get_time && STARSHIP_START_TIME=$STARSHIP_CAPTURED_TIME
} }
# If precmd/preexec arrays are not already set, set them. If we don't do this, # If precmd/preexec arrays are not already set, set them. If we don't do this,
# the code to detect whether starship_precmd is already in precmd_functions will # the code to detect whether prompt_starship_precmd is already in precmd_functions will
# fail because the array doesn't exist (and same for starship_preexec) # fail because the array doesn't exist (and same for prompt_starship_preexec)
(( ! ${+precmd_functions} )) && precmd_functions=() (( ! ${+precmd_functions} )) && precmd_functions=()
(( ! ${+preexec_functions} )) && preexec_functions=() (( ! ${+preexec_functions} )) && preexec_functions=()
# If starship precmd/preexec functions are already hooked, don't double-hook them # If starship precmd/preexec functions are already hooked, don't double-hook them
# to avoid unnecessary performance degradation in nested shells # to avoid unnecessary performance degradation in nested shells
if [[ -z ${precmd_functions[(re)starship_precmd]} ]]; then if [[ -z ${precmd_functions[(re)prompt_starship_precmd]} ]]; then
precmd_functions+=(starship_precmd) precmd_functions+=(prompt_starship_precmd)
fi fi
if [[ -z ${preexec_functions[(re)starship_preexec]} ]]; then if [[ -z ${preexec_functions[(re)prompt_starship_preexec]} ]]; then
preexec_functions+=(starship_preexec) preexec_functions+=(prompt_starship_preexec)
fi fi
# Set up a function to redraw the prompt if the user switches vi modes # Set up a function to redraw the prompt if the user switches vi modes