refactor(state): split src/state.sh into a src/state/ module

[Chemaclass]

🤔 Background

Related #931

Seventh module under ADR-010. 404 lines, 42 functions, five concerns.

💡 Changes

File	Contents
state/counters.sh	the _BASHUNIT_TESTS_*/_ASSERTIONS_* tallies and their 22 get/add accessors
state/duplicates.sh	duplicate test-function detection state
state/context.sh	per-test output buffer, exit code, title, interpolated name, hook failure/message, initialize_assertions_count
state/payload.sh	encode_field, export_subshell_context, and the base64 -w capability probe they depend on
state/parallel.sh	aggregate_parallel_results

counters.sh and duplicates.sh source before context.sh, which owns the per-test reset that clears their globals.

⚠️ A contract this split silently retired — and restored

tests/unit/state_test.sh carries the layering contract from #868/#862: state must never call console_results, console_header, parallel or runner. It enforced that by grepping src/state.sh with || true (no match = pass).

With that file gone the grep matched nothing, so the test kept passing while checking nothing. All suites were green and it would have merged unnoticed.

It now globs src/state/*.sh, and the comment records why a single-file target is the wrong shape for this kind of check. Verified by appending a call to bashunit::parallel::stop to one of the new files — red with it, green without.

✅ Verification

Relocation proven: non-blank line multiset lost nothing, 42 functions before and after, artifact code content identical, bash build.sh bin -v → ✅ Build verified ✅.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck · git check-ignore clean.

[Chemaclass]

🤔 Background

Related #931

Seventh module under ADR-010. 404 lines, 42 functions, five concerns.

💡 Changes

File	Contents
state/counters.sh	the _BASHUNIT_TESTS_*/_ASSERTIONS_* tallies and their 22 get/add accessors
state/duplicates.sh	duplicate test-function detection state
state/context.sh	per-test output buffer, exit code, title, interpolated name, hook failure/message, initialize_assertions_count
state/payload.sh	encode_field, export_subshell_context, and the base64 -w capability probe they depend on
state/parallel.sh	aggregate_parallel_results

counters.sh and duplicates.sh source before context.sh, which owns the per-test reset that clears their globals.

⚠️ A contract this split silently retired — and restored

tests/unit/state_test.sh carries the layering contract from #868/#862: state must never call console_results, console_header, parallel or runner. It enforced that by grepping src/state.sh with || true (no match = pass).

With that file gone the grep matched nothing, so the test kept passing while checking nothing. All suites were green and it would have merged unnoticed.

It now globs src/state/*.sh, and the comment records why a single-file target is the wrong shape for this kind of check. Verified by appending a call to bashunit::parallel::stop to one of the new files — red with it, green without.

✅ Verification

Relocation proven: non-blank line multiset lost nothing, 42 functions before and after, artifact code content identical, bash build.sh bin -v → ✅ Build verified ✅.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck · git check-ignore clean.

[Chemaclass]

🤔 Background

Related #931

Seventh module under ADR-010. 404 lines, 42 functions, five concerns.

💡 Changes

File	Contents
state/counters.sh	the _BASHUNIT_TESTS_*/_ASSERTIONS_* tallies and their 22 get/add accessors
state/duplicates.sh	duplicate test-function detection state
state/context.sh	per-test output buffer, exit code, title, interpolated name, hook failure/message, initialize_assertions_count
state/payload.sh	encode_field, export_subshell_context, and the base64 -w capability probe they depend on
state/parallel.sh	aggregate_parallel_results

counters.sh and duplicates.sh source before context.sh, which owns the per-test reset that clears their globals.

⚠️ A contract this split silently retired — and restored

tests/unit/state_test.sh carries the layering contract from #868/#862: state must never call console_results, console_header, parallel or runner. It enforced that by grepping src/state.sh with || true (no match = pass).

With that file gone the grep matched nothing, so the test kept passing while checking nothing. All suites were green and it would have merged unnoticed.

It now globs src/state/*.sh, and the comment records why a single-file target is the wrong shape for this kind of check. Verified by appending a call to bashunit::parallel::stop to one of the new files — red with it, green without.

✅ Verification

Relocation proven: non-blank line multiset lost nothing, 42 functions before and after, artifact code content identical, bash build.sh bin -v → ✅ Build verified ✅.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck · git check-ignore clean.

[Chemaclass]

🤔 Background

Related #931

Seventh module under ADR-010. 404 lines, 42 functions, five concerns.

💡 Changes

File	Contents
state/counters.sh	the _BASHUNIT_TESTS_*/_ASSERTIONS_* tallies and their 22 get/add accessors
state/duplicates.sh	duplicate test-function detection state
state/context.sh	per-test output buffer, exit code, title, interpolated name, hook failure/message, initialize_assertions_count
state/payload.sh	encode_field, export_subshell_context, and the base64 -w capability probe they depend on
state/parallel.sh	aggregate_parallel_results

counters.sh and duplicates.sh source before context.sh, which owns the per-test reset that clears their globals.

⚠️ A contract this split silently retired — and restored

tests/unit/state_test.sh carries the layering contract from #868/#862: state must never call console_results, console_header, parallel or runner. It enforced that by grepping src/state.sh with || true (no match = pass).

With that file gone the grep matched nothing, so the test kept passing while checking nothing. All suites were green and it would have merged unnoticed.

It now globs src/state/*.sh, and the comment records why a single-file target is the wrong shape for this kind of check. Verified by appending a call to bashunit::parallel::stop to one of the new files — red with it, green without.

✅ Verification

Relocation proven: non-blank line multiset lost nothing, 42 functions before and after, artifact code content identical, bash build.sh bin -v → ✅ Build verified ✅.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck · git check-ignore clean.

[Chemaclass]

🤔 Background

Related #931

Seventh module under ADR-010. 404 lines, 42 functions, five concerns.

💡 Changes

File	Contents
state/counters.sh	the _BASHUNIT_TESTS_*/_ASSERTIONS_* tallies and their 22 get/add accessors
state/duplicates.sh	duplicate test-function detection state
state/context.sh	per-test output buffer, exit code, title, interpolated name, hook failure/message, initialize_assertions_count
state/payload.sh	encode_field, export_subshell_context, and the base64 -w capability probe they depend on
state/parallel.sh	aggregate_parallel_results

counters.sh and duplicates.sh source before context.sh, which owns the per-test reset that clears their globals.

⚠️ A contract this split silently retired — and restored

tests/unit/state_test.sh carries the layering contract from #868/#862: state must never call console_results, console_header, parallel or runner. It enforced that by grepping src/state.sh with || true (no match = pass).

With that file gone the grep matched nothing, so the test kept passing while checking nothing. All suites were green and it would have merged unnoticed.

It now globs src/state/*.sh, and the comment records why a single-file target is the wrong shape for this kind of check. Verified by appending a call to bashunit::parallel::stop to one of the new files — red with it, green without.

✅ Verification

Relocation proven: non-blank line multiset lost nothing, 42 functions before and after, artifact code content identical, bash build.sh bin -v → ✅ Build verified ✅.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck · git check-ignore clean.

[Chemaclass]

🤔 Background

Related #931

Seventh module under ADR-010. 404 lines, 42 functions, five concerns.

💡 Changes

File	Contents
state/counters.sh	the _BASHUNIT_TESTS_*/_ASSERTIONS_* tallies and their 22 get/add accessors
state/duplicates.sh	duplicate test-function detection state
state/context.sh	per-test output buffer, exit code, title, interpolated name, hook failure/message, initialize_assertions_count
state/payload.sh	encode_field, export_subshell_context, and the base64 -w capability probe they depend on
state/parallel.sh	aggregate_parallel_results

counters.sh and duplicates.sh source before context.sh, which owns the per-test reset that clears their globals.

⚠️ A contract this split silently retired — and restored

tests/unit/state_test.sh carries the layering contract from #868/#862: state must never call console_results, console_header, parallel or runner. It enforced that by grepping src/state.sh with || true (no match = pass).

With that file gone the grep matched nothing, so the test kept passing while checking nothing. All suites were green and it would have merged unnoticed.

It now globs src/state/*.sh, and the comment records why a single-file target is the wrong shape for this kind of check. Verified by appending a call to bashunit::parallel::stop to one of the new files — red with it, green without.

✅ Verification

Relocation proven: non-blank line multiset lost nothing, 42 functions before and after, artifact code content identical, bash build.sh bin -v → ✅ Build verified ✅.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck · git check-ignore clean.

[Chemaclass]

🤔 Background

Related #931

Seventh module under ADR-010. 404 lines, 42 functions, five concerns.

💡 Changes

File	Contents
state/counters.sh	the _BASHUNIT_TESTS_*/_ASSERTIONS_* tallies and their 22 get/add accessors
state/duplicates.sh	duplicate test-function detection state
state/context.sh	per-test output buffer, exit code, title, interpolated name, hook failure/message, initialize_assertions_count
state/payload.sh	encode_field, export_subshell_context, and the base64 -w capability probe they depend on
state/parallel.sh	aggregate_parallel_results

counters.sh and duplicates.sh source before context.sh, which owns the per-test reset that clears their globals.

⚠️ A contract this split silently retired — and restored

tests/unit/state_test.sh carries the layering contract from #868/#862: state must never call console_results, console_header, parallel or runner. It enforced that by grepping src/state.sh with || true (no match = pass).

With that file gone the grep matched nothing, so the test kept passing while checking nothing. All suites were green and it would have merged unnoticed.

It now globs src/state/*.sh, and the comment records why a single-file target is the wrong shape for this kind of check. Verified by appending a call to bashunit::parallel::stop to one of the new files — red with it, green without.

✅ Verification

Relocation proven: non-blank line multiset lost nothing, 42 functions before and after, artifact code content identical, bash build.sh bin -v → ✅ Build verified ✅.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck · git check-ignore clean.

[Chemaclass]

🤔 Background

Related #931

Seventh module under ADR-010. 404 lines, 42 functions, five concerns.

💡 Changes

File	Contents
state/counters.sh	the _BASHUNIT_TESTS_*/_ASSERTIONS_* tallies and their 22 get/add accessors
state/duplicates.sh	duplicate test-function detection state
state/context.sh	per-test output buffer, exit code, title, interpolated name, hook failure/message, initialize_assertions_count
state/payload.sh	encode_field, export_subshell_context, and the base64 -w capability probe they depend on
state/parallel.sh	aggregate_parallel_results

counters.sh and duplicates.sh source before context.sh, which owns the per-test reset that clears their globals.

⚠️ A contract this split silently retired — and restored

tests/unit/state_test.sh carries the layering contract from #868/#862: state must never call console_results, console_header, parallel or runner. It enforced that by grepping src/state.sh with || true (no match = pass).

With that file gone the grep matched nothing, so the test kept passing while checking nothing. All suites were green and it would have merged unnoticed.

It now globs src/state/*.sh, and the comment records why a single-file target is the wrong shape for this kind of check. Verified by appending a call to bashunit::parallel::stop to one of the new files — red with it, green without.

✅ Verification

Relocation proven: non-blank line multiset lost nothing, 42 functions before and after, artifact code content identical, bash build.sh bin -v → ✅ Build verified ✅.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck · git check-ignore clean.