Do not enable it in non-interactive shells

Author: akinomyoga

bash-preexec.sh

@@ -47,6 +47,11 @@ if [[ -z "${BASH_VERSINFO-}" ]] || (( BASH_VERSINFO[0] < 3 || (BASH_VERSINFO[0]
     return 1
 fi
 
+# We do not enable bash-preexc in non-interactive shells (except in tests)
+if [[ $- != *i* && ! ${__bp_inside_test-} ]]; then
+   return 0
+fi
+
 # Avoid duplicate inclusion
 if [[ -n "${bash_preexec_imported:-}" || -n "${__bp_imported:-}" ]]; then
     return 0

test/bash-preexec.bats

@@ -1,5 +1,11 @@
 #!/usr/bin/env bats
 
+# This variable tells bash-preexec.sh that it is loaded for the testing
+# purposes.  bash-preexec.sh is intended to be used in interactive shell
+# sessions, so it is disabled in non-interactive shells by default.  However,
+# it still needs to be loaded in non-interactive shells for the Bats tests,
+__bp_inside_test=yes
+
 setup() {
   PROMPT_COMMAND=''        # in case the invoking shell has set this
   history -s fake command  # preexec requires there be some history

test/include-test.bats

@@ -1,5 +1,11 @@
 #!/usr/bin/env bats
 
+# This variable tells bash-preexec.sh that it is loaded for the testing
+# purposes.  bash-preexec.sh is intended to be used in interactive shell
+# sessions, so it is disabled in non-interactive shells by default.  However,
+# it still needs to be loaded in non-interactive shells for the Bats tests,
+__bp_inside_test=yes
+
 @test "should not import if it's already defined" {
   bash_preexec_imported="defined"
   source "${BATS_TEST_DIRNAME}/../bash-preexec.sh"