From d4a9158a5c55874745e06ea263cf8c83d27b91f9 Mon Sep 17 00:00:00 2001 From: Your Name Date: Fri, 12 Jun 2026 16:11:22 +0100 Subject: [PATCH] fix(aws): use return instead of exit in state helpers to avoid killing the shell `_aws_update_state` and `_aws_clear_state` ran `... || exit 1` when the directory of `$AWS_STATE_FILE` did not exist. These helpers are sourced functions invoked from `asp`/`asr` (and profile clearing), so `exit 1` terminates the user's interactive shell (the terminal closes) instead of just aborting the helper. Use `return 1` so a missing state directory aborts only the helper, leaving the interactive shell intact. Only affects users who set `AWS_PROFILE_STATE_ENABLED=true`. --- plugins/aws/aws.plugin.zsh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/aws/aws.plugin.zsh b/plugins/aws/aws.plugin.zsh index 0c43031df..677bf3236 100644 --- a/plugins/aws/aws.plugin.zsh +++ b/plugins/aws/aws.plugin.zsh @@ -9,14 +9,14 @@ function agr() { # Update state file if enabled function _aws_update_state() { if [[ "$AWS_PROFILE_STATE_ENABLED" == true ]]; then - test -d $(dirname ${AWS_STATE_FILE}) || exit 1 + test -d $(dirname ${AWS_STATE_FILE}) || return 1 echo "${AWS_PROFILE} ${AWS_REGION}" > "${AWS_STATE_FILE}" fi } function _aws_clear_state() { if [[ "$AWS_PROFILE_STATE_ENABLED" == true ]]; then - test -d $(dirname ${AWS_STATE_FILE}) || exit 1 + test -d $(dirname ${AWS_STATE_FILE}) || return 1 echo -n > "${AWS_STATE_FILE}" fi }