notebooklm-py-analysis/scripts
Teng Lin 7939795fa2
fix(ci): fix RPC health check workflow failures (#149)
* fix(ci): capture exit code correctly in RPC health check workflow

GitHub Actions `shell: bash` implicitly uses `set -eo pipefail`. The
previous `set -o pipefail` was redundant, and `set -e` caused the shell
to exit immediately when the health check script returned non-zero,
before `exit_code=${PIPESTATUS[0]}` could run. This meant the exit_code
output was never set, so the conditional issue-creation steps (for RPC
mismatch or auth failure) never fired.

Fix: use `set +e` before the pipeline so the script's exit code is
captured into PIPESTATUS, then `set -e` to restore strict mode.

https://claude.ai/code/session_01Bbbf9yHDaWv6gvvqEZwZqH

* fix(ci): replace removed LIST_CONVERSATIONS with GET_LAST_CONVERSATION_ID in health check

LIST_CONVERSATIONS was renamed to GET_LAST_CONVERSATION_ID and
GET_CONVERSATION_TURNS in the chat refactor (#141). The health check
script still referenced the old name, causing an AttributeError.

https://claude.ai/code/session_01Bbbf9yHDaWv6gvvqEZwZqH

* chore(ci): remove redundant set -e in health check workflow

The only statements after exit_code capture are echo and exit,
so restoring strict mode adds no value.

https://claude.ai/code/session_01Bbbf9yHDaWv6gvvqEZwZqH

---------

Co-authored-by: Claude <noreply@anthropic.com>
2026-03-04 10:09:40 -05:00
..
check_rpc_health.py fix(ci): fix RPC health check workflow failures (#149) 2026-03-04 10:09:40 -05:00
diagnose_get_notebook.py feat: add slide deck revision and PPTX download (issue #128) (#129) 2026-03-01 14:30:02 -05:00