From 7358b73397858fd682de410b5b790bec23218bde Mon Sep 17 00:00:00 2001 From: zzstoatzz Date: Sun, 14 Jun 2026 17:18:18 -0500 Subject: [PATCH] ci: don't let set -e swallow wisp-cli's output on failure MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit set -e aborted the deploy step the instant wisp-cli exited non-zero, before its output flushed to the CI log — so the actual error (e.g. the auth failure) never showed. Run wisp-cli to completion, then check its exit code and fail loudly, so the error is both visible and fatal. Co-Authored-By: Claude Opus 4.8 (1M context) --- .tangled/workflows/deploy-docs.yml | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/.tangled/workflows/deploy-docs.yml b/.tangled/workflows/deploy-docs.yml index 142295d..d9c59c8 100644 --- a/.tangled/workflows/deploy-docs.yml +++ b/.tangled/workflows/deploy-docs.yml @@ -22,7 +22,6 @@ steps: - name: deploy docs to wisp command: | - set -e if [ -z "$WISP_APP_PASSWORD" ]; then echo "WISP_APP_PASSWORD is not set; cannot deploy docs" >&2 exit 1 @@ -35,4 +34,12 @@ steps: if [ -n "$INTERP" ]; then patchelf --set-interpreter "$INTERP" --set-rpath "$(dirname "$INTERP")" wisp-cli fi + # run to completion so wisp-cli's output flushes to the log, then fail + # loudly on its exit code — `set -e` here would abort mid-step and the + # error (e.g. auth failure) would never reach the log. ./wisp-cli deploy "$WISP_DID" --path ./site-out --site "$WISP_SITE_NAME" --password "$WISP_APP_PASSWORD" + rc=$? + if [ "$rc" -ne 0 ]; then + echo "wisp docs deploy failed (wisp-cli exit $rc)" >&2 + exit "$rc" + fi -- 2.51.2