fix(cli): make every run-mode flag this-process-only by Chemaclass · Pull Request #839 · TypedDevs/bashunit

Chemaclass · 2026-07-19T23:40:53Z

🤔 Background

Related #837

#835 fixed five exported flags leaking into nested bashunit runs; the flag-parse loops still exported ~40 more, silently reconfiguring any bashunit invocation spawned by a script under test.

💡 Changes

Convert every remaining export BASHUNIT_* flag site (main parse loops + entrypoint early scan) to assign + export -n; audited each flag's readers — all in-process, no exec'd consumer exists
Extend the leak-probe acceptance test to fifteen flag/probe pairs, kept 1:1 so export -n also clears allexport-.env attributes
Env-var configuration (BASHUNIT_*=… bashunit) unchanged

https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Completes the audit started in #835: all ~40 remaining `export BASHUNIT_*` sites in the flag-parse loops (and the entrypoint's early scan) now assign without the export attribute and strip an inherited one with `export -n`. Every reader — runner, reporters, parallel workers — lives in the main shell or its subshells, which inherit unexported variables; exec'd children (nested bashunit runs) must not silently inherit the parent's output mode, strictness, parallelism, timeouts, seeds or report paths. Verified no exec'd consumer exists for any converted flag. The leak-probe acceptance test now covers fifteen flag/probe pairs; each probed name is paired with the matching parent flag because the branch's `export -n` is also what clears an export attribute stamped by an allexport .env load. Env-var configuration is unchanged. Closes#837 Claude-Session: https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Chemaclass · 2026-07-19T23:40:53Z

🤔 Background

Related #837

#835 fixed five exported flags leaking into nested bashunit runs; the flag-parse loops still exported ~40 more, silently reconfiguring any bashunit invocation spawned by a script under test.

💡 Changes

Convert every remaining export BASHUNIT_* flag site (main parse loops + entrypoint early scan) to assign + export -n; audited each flag's readers — all in-process, no exec'd consumer exists
Extend the leak-probe acceptance test to fifteen flag/probe pairs, kept 1:1 so export -n also clears allexport-.env attributes
Env-var configuration (BASHUNIT_*=… bashunit) unchanged

https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Completes the audit started in #835: all ~40 remaining `export BASHUNIT_*` sites in the flag-parse loops (and the entrypoint's early scan) now assign without the export attribute and strip an inherited one with `export -n`. Every reader — runner, reporters, parallel workers — lives in the main shell or its subshells, which inherit unexported variables; exec'd children (nested bashunit runs) must not silently inherit the parent's output mode, strictness, parallelism, timeouts, seeds or report paths. Verified no exec'd consumer exists for any converted flag. The leak-probe acceptance test now covers fifteen flag/probe pairs; each probed name is paired with the matching parent flag because the branch's `export -n` is also what clears an export attribute stamped by an allexport .env load. Env-var configuration is unchanged. Closes#837 Claude-Session: https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Chemaclass · 2026-07-19T23:40:53Z

🤔 Background

Related #837

#835 fixed five exported flags leaking into nested bashunit runs; the flag-parse loops still exported ~40 more, silently reconfiguring any bashunit invocation spawned by a script under test.

💡 Changes

Convert every remaining export BASHUNIT_* flag site (main parse loops + entrypoint early scan) to assign + export -n; audited each flag's readers — all in-process, no exec'd consumer exists
Extend the leak-probe acceptance test to fifteen flag/probe pairs, kept 1:1 so export -n also clears allexport-.env attributes
Env-var configuration (BASHUNIT_*=… bashunit) unchanged

https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Completes the audit started in #835: all ~40 remaining `export BASHUNIT_*` sites in the flag-parse loops (and the entrypoint's early scan) now assign without the export attribute and strip an inherited one with `export -n`. Every reader — runner, reporters, parallel workers — lives in the main shell or its subshells, which inherit unexported variables; exec'd children (nested bashunit runs) must not silently inherit the parent's output mode, strictness, parallelism, timeouts, seeds or report paths. Verified no exec'd consumer exists for any converted flag. The leak-probe acceptance test now covers fifteen flag/probe pairs; each probed name is paired with the matching parent flag because the branch's `export -n` is also what clears an export attribute stamped by an allexport .env load. Env-var configuration is unchanged. Closes#837 Claude-Session: https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Chemaclass · 2026-07-19T23:40:53Z

🤔 Background

Related #837

#835 fixed five exported flags leaking into nested bashunit runs; the flag-parse loops still exported ~40 more, silently reconfiguring any bashunit invocation spawned by a script under test.

💡 Changes

Convert every remaining export BASHUNIT_* flag site (main parse loops + entrypoint early scan) to assign + export -n; audited each flag's readers — all in-process, no exec'd consumer exists
Extend the leak-probe acceptance test to fifteen flag/probe pairs, kept 1:1 so export -n also clears allexport-.env attributes
Env-var configuration (BASHUNIT_*=… bashunit) unchanged

https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Completes the audit started in #835: all ~40 remaining `export BASHUNIT_*` sites in the flag-parse loops (and the entrypoint's early scan) now assign without the export attribute and strip an inherited one with `export -n`. Every reader — runner, reporters, parallel workers — lives in the main shell or its subshells, which inherit unexported variables; exec'd children (nested bashunit runs) must not silently inherit the parent's output mode, strictness, parallelism, timeouts, seeds or report paths. Verified no exec'd consumer exists for any converted flag. The leak-probe acceptance test now covers fifteen flag/probe pairs; each probed name is paired with the matching parent flag because the branch's `export -n` is also what clears an export attribute stamped by an allexport .env load. Env-var configuration is unchanged. Closes#837 Claude-Session: https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Chemaclass · 2026-07-19T23:40:53Z

🤔 Background

Related #837

#835 fixed five exported flags leaking into nested bashunit runs; the flag-parse loops still exported ~40 more, silently reconfiguring any bashunit invocation spawned by a script under test.

💡 Changes

Convert every remaining export BASHUNIT_* flag site (main parse loops + entrypoint early scan) to assign + export -n; audited each flag's readers — all in-process, no exec'd consumer exists
Extend the leak-probe acceptance test to fifteen flag/probe pairs, kept 1:1 so export -n also clears allexport-.env attributes
Env-var configuration (BASHUNIT_*=… bashunit) unchanged

https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Completes the audit started in #835: all ~40 remaining `export BASHUNIT_*` sites in the flag-parse loops (and the entrypoint's early scan) now assign without the export attribute and strip an inherited one with `export -n`. Every reader — runner, reporters, parallel workers — lives in the main shell or its subshells, which inherit unexported variables; exec'd children (nested bashunit runs) must not silently inherit the parent's output mode, strictness, parallelism, timeouts, seeds or report paths. Verified no exec'd consumer exists for any converted flag. The leak-probe acceptance test now covers fifteen flag/probe pairs; each probed name is paired with the matching parent flag because the branch's `export -n` is also what clears an export attribute stamped by an allexport .env load. Env-var configuration is unchanged. Closes#837 Claude-Session: https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Chemaclass · 2026-07-19T23:40:53Z

🤔 Background

Related #837

#835 fixed five exported flags leaking into nested bashunit runs; the flag-parse loops still exported ~40 more, silently reconfiguring any bashunit invocation spawned by a script under test.

💡 Changes

Convert every remaining export BASHUNIT_* flag site (main parse loops + entrypoint early scan) to assign + export -n; audited each flag's readers — all in-process, no exec'd consumer exists
Extend the leak-probe acceptance test to fifteen flag/probe pairs, kept 1:1 so export -n also clears allexport-.env attributes
Env-var configuration (BASHUNIT_*=… bashunit) unchanged

https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Completes the audit started in #835: all ~40 remaining `export BASHUNIT_*` sites in the flag-parse loops (and the entrypoint's early scan) now assign without the export attribute and strip an inherited one with `export -n`. Every reader — runner, reporters, parallel workers — lives in the main shell or its subshells, which inherit unexported variables; exec'd children (nested bashunit runs) must not silently inherit the parent's output mode, strictness, parallelism, timeouts, seeds or report paths. Verified no exec'd consumer exists for any converted flag. The leak-probe acceptance test now covers fifteen flag/probe pairs; each probed name is paired with the matching parent flag because the branch's `export -n` is also what clears an export attribute stamped by an allexport .env load. Env-var configuration is unchanged. Closes#837 Claude-Session: https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Chemaclass · 2026-07-19T23:40:53Z

🤔 Background

Related #837

#835 fixed five exported flags leaking into nested bashunit runs; the flag-parse loops still exported ~40 more, silently reconfiguring any bashunit invocation spawned by a script under test.

💡 Changes

Convert every remaining export BASHUNIT_* flag site (main parse loops + entrypoint early scan) to assign + export -n; audited each flag's readers — all in-process, no exec'd consumer exists
Extend the leak-probe acceptance test to fifteen flag/probe pairs, kept 1:1 so export -n also clears allexport-.env attributes
Env-var configuration (BASHUNIT_*=… bashunit) unchanged

https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Completes the audit started in #835: all ~40 remaining `export BASHUNIT_*` sites in the flag-parse loops (and the entrypoint's early scan) now assign without the export attribute and strip an inherited one with `export -n`. Every reader — runner, reporters, parallel workers — lives in the main shell or its subshells, which inherit unexported variables; exec'd children (nested bashunit runs) must not silently inherit the parent's output mode, strictness, parallelism, timeouts, seeds or report paths. Verified no exec'd consumer exists for any converted flag. The leak-probe acceptance test now covers fifteen flag/probe pairs; each probed name is paired with the matching parent flag because the branch's `export -n` is also what clears an export attribute stamped by an allexport .env load. Env-var configuration is unchanged. Closes#837 Claude-Session: https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Chemaclass · 2026-07-19T23:40:53Z

🤔 Background

Related #837

#835 fixed five exported flags leaking into nested bashunit runs; the flag-parse loops still exported ~40 more, silently reconfiguring any bashunit invocation spawned by a script under test.

💡 Changes

Convert every remaining export BASHUNIT_* flag site (main parse loops + entrypoint early scan) to assign + export -n; audited each flag's readers — all in-process, no exec'd consumer exists
Extend the leak-probe acceptance test to fifteen flag/probe pairs, kept 1:1 so export -n also clears allexport-.env attributes
Env-var configuration (BASHUNIT_*=… bashunit) unchanged

https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED

Completes the audit started in #835: all ~40 remaining `export BASHUNIT_*` sites in the flag-parse loops (and the entrypoint's early scan) now assign without the export attribute and strip an inherited one with `export -n`. Every reader — runner, reporters, parallel workers — lives in the main shell or its subshells, which inherit unexported variables; exec'd children (nested bashunit runs) must not silently inherit the parent's output mode, strictness, parallelism, timeouts, seeds or report paths. Verified no exec'd consumer exists for any converted flag. The leak-probe acceptance test now covers fifteen flag/probe pairs; each probed name is paired with the matching parent flag because the branch's `export -n` is also what clears an export attribute stamped by an allexport .env load. Env-var configuration is unchanged. Closes#837 Claude-Session: https://claude.ai/code/session_01JSeB8UzLCpqst55hAK6dED