test: fix bun PATH in subprocess tests and set -eo pipefail in shell scripts (#1353)

Fixes 256 failing tests that spawn bun subprocesses. These tests were
failing because bun was not in the child process PATH. Ensures all
CLI test helpers pass PATH with $HOME/.bun/bin included.

Also corrects two gptme.sh scripts to use 'set -eo pipefail' instead
of bare 'set -e' for proper error handling, per shellcheck conventions.

Changes:
- 7 CLI test files: add PATH=$HOME/.bun/bin to execSync/spawnSync env
- 2 shell scripts: use set -eo pipefail for proper error handling

Results: 256 tests now passing, 0 failures in subprocess CLI tests.

Co-authored-by: test-engineer <agent@spawn.local>
Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>

cli/src/__tests__/cli-entry-edge-cases.test.ts

@@ -33,7 +33,7 @@ function runCli(
     const stdout = execSync(cmd, {
       cwd: PROJECT_ROOT,
       env: {
-        PATH: process.env.PATH,
+        PATH: `${process.env.HOME}/.bun/bin:${process.env.PATH}`,
         HOME: process.env.HOME,
         SHELL: process.env.SHELL,
         TERM: process.env.TERM || "xterm",

cli/src/__tests__/cli-version-and-dispatch.test.ts

@@ -40,6 +40,8 @@ function runCLI(
     timeout: 15000,
     env: {
       ...process.env,
+      // Ensure bun is in PATH for child processes
+      PATH: `${process.env.HOME}/.bun/bin:${process.env.PATH}`,
       SPAWN_NO_UPDATE_CHECK: "1",
       BUN_ENV: "test",
       // Avoid terminal-dependent output

cli/src/__tests__/cmdrun-resolution.test.ts

@@ -30,7 +30,7 @@ function runCli(
     const stdout = execSync(cmd, {
       cwd: PROJECT_ROOT,
       env: {
-        PATH: process.env.PATH,
+        PATH: `${process.env.HOME}/.bun/bin:${process.env.PATH}`,
         HOME: process.env.HOME,
         SHELL: process.env.SHELL,
         TERM: process.env.TERM || "xterm",

cli/src/__tests__/index-main-routing.test.ts

@@ -31,6 +31,8 @@ function runCli(
       env: {
         ...process.env,
         ...env,
+        // Ensure bun is in PATH for child processes
+        PATH: `${process.env.HOME}/.bun/bin:${process.env.PATH}`,
         // Prevent auto-update from running during tests
         SPAWN_NO_UPDATE_CHECK: "1",
         // Prevent local manifest.json from being used

cli/src/__tests__/no-cloud-error-paths.test.ts

@@ -34,7 +34,7 @@ function runCli(
     const stdout = execSync(cmd, {
       cwd: PROJECT_ROOT,
       env: {
-        PATH: process.env.PATH,
+        PATH: `${process.env.HOME}/.bun/bin:${process.env.PATH}`,
         HOME: process.env.HOME,
         SHELL: process.env.SHELL,
         TERM: process.env.TERM || "xterm",

cli/src/__tests__/prompt-file-errors.test.ts

@@ -39,7 +39,7 @@ function runCli(
     const stdout = execSync(cmd, {
       cwd: PROJECT_ROOT,
       env: {
-        PATH: process.env.PATH,
+        PATH: `${process.env.HOME}/.bun/bin:${process.env.PATH}`,
         HOME: process.env.HOME,
         SHELL: process.env.SHELL,
         TERM: process.env.TERM || "xterm",

cli/src/__tests__/show-info-or-error.test.ts

@@ -37,7 +37,7 @@ function runCli(
       cwd: PROJECT_ROOT,
       env: {
         // Start with clean env to avoid bun test's NODE_ENV=test leaking
-        PATH: process.env.PATH,
+        PATH: `${process.env.HOME}/.bun/bin:${process.env.PATH}`,
         HOME: process.env.HOME,
         SHELL: process.env.SHELL,
         TERM: process.env.TERM || "xterm",