Skip to content

fix(bin): harden test runner for stock macOS Bash - #93

Merged
quinnbot-ai merged 16 commits into
mainfrom
fm/fm-ci-stock-bash-test-runner
Aug 6, 2026
Merged

quinnbot-ai merged 16 commits into
mainfrom
fm/fm-ci-stock-bash-test-runner

Conversation

@quinnbot-ai

Copy link
Copy Markdown
Owner

What Changed

  • Add a focused CI lane and wrapper that run the test-runner contract under stock macOS Bash 3.2, pin nested interpreter resolution, and emit typed local skip evidence when Bash 3.2 is unavailable.
  • Expand compatibility coverage for scheduling, aggregation, quoted and temporary paths, platform command differences, and signal handling without duplicating the full suite.
  • Harden serial and parallel cleanup by synchronously terminating and reaping process trees while allowing serial output to drain before reaping tee.

Updates from git push no-mistakes

@quinnbot-ai
quinnbot-ai force-pushed the fm/fm-ci-stock-bash-test-runner branch 2 times, most recently from a8e2165 to 05c0003 Compare August 6, 2026 10:41
@quinnbot-ai
quinnbot-ai force-pushed the fm/fm-ci-stock-bash-test-runner branch from 9e94721 to fd944d8 Compare August 6, 2026 11:39
@quinnbot-ai
quinnbot-ai merged commit e96ba81 into main Aug 6, 2026
13 checks passed
quinnbot-ai added a commit that referenced this pull request Aug 10, 2026
* test: add stock bash runner contract lane

* no-mistakes(review): fix: reap runner process trees synchronously

* no-mistakes(review): fix: drain serial cleanup before reaping tee

* no-mistakes(document): Document stock Bash interpreter pinning

* no-mistakes(lint): Captain: suppress indirect trap callback lint warnings

* no-mistakes(review): Bound signal cleanup and remove stale jq prerequisite

* no-mistakes(document): Document stock Bash runner wrapper

* no-mistakes(review): Bound signal escalation and descendant reaping

* no-mistakes(review): Isolate test cleanup in bounded process groups

* no-mistakes(review): Drain completed parallel worker process groups

* no-mistakes(document): Confirm stock Bash runner documentation accuracy

* no-mistakes(review): Suppress trap-only background reaper ShellCheck warning

* no-mistakes(document): Correct focused CI test count

* no-mistakes(review): Restore stock Bash snapshot runtime coverage

* no-mistakes(document): Document stock Bash snapshot runtime coverage

* no-mistakes(lint): Remove unused serial runner tee PID declaration

---------

Co-authored-by: QuinnBot <quinnbot@proton.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant