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

[Chemaclass]

🤔 Background

Related #931

Third module under ADR-010, and the first built to the amended index.sh convention from the start. The responsibility map was posted on #931 before any code moved.

💡 Changes

The seam is the one the file already had — a shared result store, and one writer per output format:

File	Holds
reports/collect.sh	the 7 _BASHUNIT_REPORTS_TEST_* arrays, is_enabled, add_test, the six add_test_* wrappers
reports/junit.sh	__xml_escape, generate_junit_xml
reports/tap.sh	__tap_message, generate_report_tap
reports/json.sh	__json_escape, generate_report_json
reports/gha.sh	__gha_encode, print_gha_annotations, generate_gha_log
reports/html.sh	generate_report_html

All 12 internal call edges stay inside their file: no writer calls another, and each escaper is used by exactly one format, so each travels with its format rather than into a shared helpers file. Layering is collect → {junit, tap, json, gha, html}, one level deep.

The file-wide # shellcheck disable=SC2155 moved to the two files that actually trigger it (junit.sh, html.sh) rather than being dropped or blanket-applied to all six.

✅ Verification

A relocation: the non-blank line multiset differs only by the new shebangs, module headers and aggregator source lines; function count is 18 before and after.

The built artifact is not byte-identical this time — grouping each format's functions together reorders definitions relative to the old single-file order. The sorted code content of the artifact is identical, and reordering top-level function definitions is safe because the entrypoint sources every file before anything is invoked. I verified no file-scope statement exists outside collect.sh's array initialisers.

Also checked end-to-end that JUnit/JSON/TAP still emit correct output, and ran the ADR-010 trap checklist: git check-ignore clean, no .editorconfig per-file rule lost, no line over 120, CI-mode ShellCheck (per file, no -x) clean.

Green: sequential · --parallel · --parallel --simple --strict · make sa · make lint · bash build.sh .

[Chemaclass]

🤔 Background

Related #931

Third module under ADR-010, and the first built to the amended index.sh convention from the start. The responsibility map was posted on #931 before any code moved.

💡 Changes

The seam is the one the file already had — a shared result store, and one writer per output format:

File	Holds
reports/collect.sh	the 7 _BASHUNIT_REPORTS_TEST_* arrays, is_enabled, add_test, the six add_test_* wrappers
reports/junit.sh	__xml_escape, generate_junit_xml
reports/tap.sh	__tap_message, generate_report_tap
reports/json.sh	__json_escape, generate_report_json
reports/gha.sh	__gha_encode, print_gha_annotations, generate_gha_log
reports/html.sh	generate_report_html

All 12 internal call edges stay inside their file: no writer calls another, and each escaper is used by exactly one format, so each travels with its format rather than into a shared helpers file. Layering is collect → {junit, tap, json, gha, html}, one level deep.

The file-wide # shellcheck disable=SC2155 moved to the two files that actually trigger it (junit.sh, html.sh) rather than being dropped or blanket-applied to all six.

✅ Verification

A relocation: the non-blank line multiset differs only by the new shebangs, module headers and aggregator source lines; function count is 18 before and after.

The built artifact is not byte-identical this time — grouping each format's functions together reorders definitions relative to the old single-file order. The sorted code content of the artifact is identical, and reordering top-level function definitions is safe because the entrypoint sources every file before anything is invoked. I verified no file-scope statement exists outside collect.sh's array initialisers.

Also checked end-to-end that JUnit/JSON/TAP still emit correct output, and ran the ADR-010 trap checklist: git check-ignore clean, no .editorconfig per-file rule lost, no line over 120, CI-mode ShellCheck (per file, no -x) clean.

Green: sequential · --parallel · --parallel --simple --strict · make sa · make lint · bash build.sh .

[Chemaclass]

🤔 Background

Related #931

Third module under ADR-010, and the first built to the amended index.sh convention from the start. The responsibility map was posted on #931 before any code moved.

💡 Changes

The seam is the one the file already had — a shared result store, and one writer per output format:

File	Holds
reports/collect.sh	the 7 _BASHUNIT_REPORTS_TEST_* arrays, is_enabled, add_test, the six add_test_* wrappers
reports/junit.sh	__xml_escape, generate_junit_xml
reports/tap.sh	__tap_message, generate_report_tap
reports/json.sh	__json_escape, generate_report_json
reports/gha.sh	__gha_encode, print_gha_annotations, generate_gha_log
reports/html.sh	generate_report_html

All 12 internal call edges stay inside their file: no writer calls another, and each escaper is used by exactly one format, so each travels with its format rather than into a shared helpers file. Layering is collect → {junit, tap, json, gha, html}, one level deep.

The file-wide # shellcheck disable=SC2155 moved to the two files that actually trigger it (junit.sh, html.sh) rather than being dropped or blanket-applied to all six.

✅ Verification

A relocation: the non-blank line multiset differs only by the new shebangs, module headers and aggregator source lines; function count is 18 before and after.

The built artifact is not byte-identical this time — grouping each format's functions together reorders definitions relative to the old single-file order. The sorted code content of the artifact is identical, and reordering top-level function definitions is safe because the entrypoint sources every file before anything is invoked. I verified no file-scope statement exists outside collect.sh's array initialisers.

Also checked end-to-end that JUnit/JSON/TAP still emit correct output, and ran the ADR-010 trap checklist: git check-ignore clean, no .editorconfig per-file rule lost, no line over 120, CI-mode ShellCheck (per file, no -x) clean.

Green: sequential · --parallel · --parallel --simple --strict · make sa · make lint · bash build.sh .

[Chemaclass]

🤔 Background

Related #931

Third module under ADR-010, and the first built to the amended index.sh convention from the start. The responsibility map was posted on #931 before any code moved.

💡 Changes

The seam is the one the file already had — a shared result store, and one writer per output format:

File	Holds
reports/collect.sh	the 7 _BASHUNIT_REPORTS_TEST_* arrays, is_enabled, add_test, the six add_test_* wrappers
reports/junit.sh	__xml_escape, generate_junit_xml
reports/tap.sh	__tap_message, generate_report_tap
reports/json.sh	__json_escape, generate_report_json
reports/gha.sh	__gha_encode, print_gha_annotations, generate_gha_log
reports/html.sh	generate_report_html

All 12 internal call edges stay inside their file: no writer calls another, and each escaper is used by exactly one format, so each travels with its format rather than into a shared helpers file. Layering is collect → {junit, tap, json, gha, html}, one level deep.

The file-wide # shellcheck disable=SC2155 moved to the two files that actually trigger it (junit.sh, html.sh) rather than being dropped or blanket-applied to all six.

✅ Verification

A relocation: the non-blank line multiset differs only by the new shebangs, module headers and aggregator source lines; function count is 18 before and after.

The built artifact is not byte-identical this time — grouping each format's functions together reorders definitions relative to the old single-file order. The sorted code content of the artifact is identical, and reordering top-level function definitions is safe because the entrypoint sources every file before anything is invoked. I verified no file-scope statement exists outside collect.sh's array initialisers.

Also checked end-to-end that JUnit/JSON/TAP still emit correct output, and ran the ADR-010 trap checklist: git check-ignore clean, no .editorconfig per-file rule lost, no line over 120, CI-mode ShellCheck (per file, no -x) clean.

Green: sequential · --parallel · --parallel --simple --strict · make sa · make lint · bash build.sh .

[Chemaclass]

🤔 Background

Related #931

Third module under ADR-010, and the first built to the amended index.sh convention from the start. The responsibility map was posted on #931 before any code moved.

💡 Changes

The seam is the one the file already had — a shared result store, and one writer per output format:

File	Holds
reports/collect.sh	the 7 _BASHUNIT_REPORTS_TEST_* arrays, is_enabled, add_test, the six add_test_* wrappers
reports/junit.sh	__xml_escape, generate_junit_xml
reports/tap.sh	__tap_message, generate_report_tap
reports/json.sh	__json_escape, generate_report_json
reports/gha.sh	__gha_encode, print_gha_annotations, generate_gha_log
reports/html.sh	generate_report_html

All 12 internal call edges stay inside their file: no writer calls another, and each escaper is used by exactly one format, so each travels with its format rather than into a shared helpers file. Layering is collect → {junit, tap, json, gha, html}, one level deep.

The file-wide # shellcheck disable=SC2155 moved to the two files that actually trigger it (junit.sh, html.sh) rather than being dropped or blanket-applied to all six.

✅ Verification

A relocation: the non-blank line multiset differs only by the new shebangs, module headers and aggregator source lines; function count is 18 before and after.

The built artifact is not byte-identical this time — grouping each format's functions together reorders definitions relative to the old single-file order. The sorted code content of the artifact is identical, and reordering top-level function definitions is safe because the entrypoint sources every file before anything is invoked. I verified no file-scope statement exists outside collect.sh's array initialisers.

Also checked end-to-end that JUnit/JSON/TAP still emit correct output, and ran the ADR-010 trap checklist: git check-ignore clean, no .editorconfig per-file rule lost, no line over 120, CI-mode ShellCheck (per file, no -x) clean.

Green: sequential · --parallel · --parallel --simple --strict · make sa · make lint · bash build.sh .

[Chemaclass]

🤔 Background

Related #931

Third module under ADR-010, and the first built to the amended index.sh convention from the start. The responsibility map was posted on #931 before any code moved.

💡 Changes

The seam is the one the file already had — a shared result store, and one writer per output format:

File	Holds
reports/collect.sh	the 7 _BASHUNIT_REPORTS_TEST_* arrays, is_enabled, add_test, the six add_test_* wrappers
reports/junit.sh	__xml_escape, generate_junit_xml
reports/tap.sh	__tap_message, generate_report_tap
reports/json.sh	__json_escape, generate_report_json
reports/gha.sh	__gha_encode, print_gha_annotations, generate_gha_log
reports/html.sh	generate_report_html

All 12 internal call edges stay inside their file: no writer calls another, and each escaper is used by exactly one format, so each travels with its format rather than into a shared helpers file. Layering is collect → {junit, tap, json, gha, html}, one level deep.

The file-wide # shellcheck disable=SC2155 moved to the two files that actually trigger it (junit.sh, html.sh) rather than being dropped or blanket-applied to all six.

✅ Verification

A relocation: the non-blank line multiset differs only by the new shebangs, module headers and aggregator source lines; function count is 18 before and after.

The built artifact is not byte-identical this time — grouping each format's functions together reorders definitions relative to the old single-file order. The sorted code content of the artifact is identical, and reordering top-level function definitions is safe because the entrypoint sources every file before anything is invoked. I verified no file-scope statement exists outside collect.sh's array initialisers.

Also checked end-to-end that JUnit/JSON/TAP still emit correct output, and ran the ADR-010 trap checklist: git check-ignore clean, no .editorconfig per-file rule lost, no line over 120, CI-mode ShellCheck (per file, no -x) clean.

Green: sequential · --parallel · --parallel --simple --strict · make sa · make lint · bash build.sh .

[Chemaclass]

🤔 Background

Related #931

Third module under ADR-010, and the first built to the amended index.sh convention from the start. The responsibility map was posted on #931 before any code moved.

💡 Changes

The seam is the one the file already had — a shared result store, and one writer per output format:

File	Holds
reports/collect.sh	the 7 _BASHUNIT_REPORTS_TEST_* arrays, is_enabled, add_test, the six add_test_* wrappers
reports/junit.sh	__xml_escape, generate_junit_xml
reports/tap.sh	__tap_message, generate_report_tap
reports/json.sh	__json_escape, generate_report_json
reports/gha.sh	__gha_encode, print_gha_annotations, generate_gha_log
reports/html.sh	generate_report_html

All 12 internal call edges stay inside their file: no writer calls another, and each escaper is used by exactly one format, so each travels with its format rather than into a shared helpers file. Layering is collect → {junit, tap, json, gha, html}, one level deep.

The file-wide # shellcheck disable=SC2155 moved to the two files that actually trigger it (junit.sh, html.sh) rather than being dropped or blanket-applied to all six.

✅ Verification

A relocation: the non-blank line multiset differs only by the new shebangs, module headers and aggregator source lines; function count is 18 before and after.

The built artifact is not byte-identical this time — grouping each format's functions together reorders definitions relative to the old single-file order. The sorted code content of the artifact is identical, and reordering top-level function definitions is safe because the entrypoint sources every file before anything is invoked. I verified no file-scope statement exists outside collect.sh's array initialisers.

Also checked end-to-end that JUnit/JSON/TAP still emit correct output, and ran the ADR-010 trap checklist: git check-ignore clean, no .editorconfig per-file rule lost, no line over 120, CI-mode ShellCheck (per file, no -x) clean.

Green: sequential · --parallel · --parallel --simple --strict · make sa · make lint · bash build.sh .

[Chemaclass]

🤔 Background

Related #931

Third module under ADR-010, and the first built to the amended index.sh convention from the start. The responsibility map was posted on #931 before any code moved.

💡 Changes

The seam is the one the file already had — a shared result store, and one writer per output format:

File	Holds
reports/collect.sh	the 7 _BASHUNIT_REPORTS_TEST_* arrays, is_enabled, add_test, the six add_test_* wrappers
reports/junit.sh	__xml_escape, generate_junit_xml
reports/tap.sh	__tap_message, generate_report_tap
reports/json.sh	__json_escape, generate_report_json
reports/gha.sh	__gha_encode, print_gha_annotations, generate_gha_log
reports/html.sh	generate_report_html

All 12 internal call edges stay inside their file: no writer calls another, and each escaper is used by exactly one format, so each travels with its format rather than into a shared helpers file. Layering is collect → {junit, tap, json, gha, html}, one level deep.

The file-wide # shellcheck disable=SC2155 moved to the two files that actually trigger it (junit.sh, html.sh) rather than being dropped or blanket-applied to all six.

✅ Verification

A relocation: the non-blank line multiset differs only by the new shebangs, module headers and aggregator source lines; function count is 18 before and after.

The built artifact is not byte-identical this time — grouping each format's functions together reorders definitions relative to the old single-file order. The sorted code content of the artifact is identical, and reordering top-level function definitions is safe because the entrypoint sources every file before anything is invoked. I verified no file-scope statement exists outside collect.sh's array initialisers.

Also checked end-to-end that JUnit/JSON/TAP still emit correct output, and ran the ADR-010 trap checklist: git check-ignore clean, no .editorconfig per-file rule lost, no line over 120, CI-mode ShellCheck (per file, no -x) clean.

Green: sequential · --parallel · --parallel --simple --strict · make sa · make lint · bash build.sh .