From 080d81fbf1121e9945b302cb840774aa6b28bbaa Mon Sep 17 00:00:00 2001 From: fullsend-code <278716306+fullsend-ai-coder[bot]@users.noreply.github.com> Date: Tue, 7 Jul 2026 22:56:37 +0000 Subject: [PATCH] fix(#3446): remove invalid top-level return guard from inlined CSMA code The CSMA library load guard (`return 0` at top level) was left in when the library was inlined into post-prioritize.sh and its test file. Since these scripts are executed directly (not sourced), `return` at top level is invalid bash and crashes the script with exit code 2 when GITHUB_API_CSMA_SH_LOADED is set in the environment. Co-Authored-By: Claude Opus 4.6 --- .../scaffold/fullsend-repo/scripts/post-prioritize-test.sh | 3 --- internal/scaffold/fullsend-repo/scripts/post-prioritize.sh | 3 --- 2 files changed, 6 deletions(-) diff --git a/internal/scaffold/fullsend-repo/scripts/post-prioritize-test.sh b/internal/scaffold/fullsend-repo/scripts/post-prioritize-test.sh index 214b9b353..48f980250 100755 --- a/internal/scaffold/fullsend-repo/scripts/post-prioritize-test.sh +++ b/internal/scaffold/fullsend-repo/scripts/post-prioritize-test.sh @@ -283,9 +283,6 @@ run_test_failure_stderr() { # shellcheck shell=bash -[[ -n "${GITHUB_API_CSMA_SH_LOADED:-}" ]] && return 0 -GITHUB_API_CSMA_SH_LOADED=1 - _github_csma_max_attempts() { echo "${GITHUB_CSMA_MAX_ATTEMPTS:-8}" } diff --git a/internal/scaffold/fullsend-repo/scripts/post-prioritize.sh b/internal/scaffold/fullsend-repo/scripts/post-prioritize.sh index 58fa025bd..9e331cd5e 100755 --- a/internal/scaffold/fullsend-repo/scripts/post-prioritize.sh +++ b/internal/scaffold/fullsend-repo/scripts/post-prioritize.sh @@ -29,9 +29,6 @@ set -euo pipefail # shellcheck shell=bash -[[ -n "${GITHUB_API_CSMA_SH_LOADED:-}" ]] && return 0 -GITHUB_API_CSMA_SH_LOADED=1 - _github_csma_max_attempts() { echo "${GITHUB_CSMA_MAX_ATTEMPTS:-8}" }