refactor(console): split results.sh into six single-purpose files

[Chemaclass]

🤔 Background

Related #931

Second half of the plan #942 set up: group first, split second. src/console/results.sh was 828 lines and 27 functions.

💡 Changes

File	Contents
console/line.sh	print_line — the primitive every result line goes through — and print_tap_line
console/duration.sh	format_duration{,_to_slot}
console/diff.sh	render_diff, first_line_ellipsis, snapshot_line_diff
console/test_line.sh	the ten per-test result lines and test_location_suffix
console/deferred.sh	flush_deferred_block and the five *_and_reset blocks
console/summary.sh	render_result, print_execution_time, print_hook_completed

Peers inside src/console/ rather than a src/console/results/ sub-module — console is the module, these are its files.

Layering from all 26 internal call edges: line, duration and diff have no console_results callees; test_line, deferred and summary build on them.

Function names are untouched, so they stay bashunit::console_results::* while files are named for what they hold — the same split learn/lessons/basics.sh already uses.

🧹 A stale directive removed

The file-wide # shellcheck disable=SC2155 was dropped, not carried. ShellCheck raises no SC2155 on the original file without it — verified explicitly before removing, the same way #928 found SC2094 had gone stale. Carrying it to all six files would have propagated dead suppression.

✅ Verification

Relocation proven: the non-blank line multiset lost only that directive, 27 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.

[Chemaclass]

🤔 Background

Related #931

Second half of the plan #942 set up: group first, split second. src/console/results.sh was 828 lines and 27 functions.

💡 Changes

File	Contents
console/line.sh	print_line — the primitive every result line goes through — and print_tap_line
console/duration.sh	format_duration{,_to_slot}
console/diff.sh	render_diff, first_line_ellipsis, snapshot_line_diff
console/test_line.sh	the ten per-test result lines and test_location_suffix
console/deferred.sh	flush_deferred_block and the five *_and_reset blocks
console/summary.sh	render_result, print_execution_time, print_hook_completed

Peers inside src/console/ rather than a src/console/results/ sub-module — console is the module, these are its files.

Layering from all 26 internal call edges: line, duration and diff have no console_results callees; test_line, deferred and summary build on them.

Function names are untouched, so they stay bashunit::console_results::* while files are named for what they hold — the same split learn/lessons/basics.sh already uses.

🧹 A stale directive removed

The file-wide # shellcheck disable=SC2155 was dropped, not carried. ShellCheck raises no SC2155 on the original file without it — verified explicitly before removing, the same way #928 found SC2094 had gone stale. Carrying it to all six files would have propagated dead suppression.

✅ Verification

Relocation proven: the non-blank line multiset lost only that directive, 27 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.

[Chemaclass]

🤔 Background

Related #931

Second half of the plan #942 set up: group first, split second. src/console/results.sh was 828 lines and 27 functions.

💡 Changes

File	Contents
console/line.sh	print_line — the primitive every result line goes through — and print_tap_line
console/duration.sh	format_duration{,_to_slot}
console/diff.sh	render_diff, first_line_ellipsis, snapshot_line_diff
console/test_line.sh	the ten per-test result lines and test_location_suffix
console/deferred.sh	flush_deferred_block and the five *_and_reset blocks
console/summary.sh	render_result, print_execution_time, print_hook_completed

Peers inside src/console/ rather than a src/console/results/ sub-module — console is the module, these are its files.

Layering from all 26 internal call edges: line, duration and diff have no console_results callees; test_line, deferred and summary build on them.

Function names are untouched, so they stay bashunit::console_results::* while files are named for what they hold — the same split learn/lessons/basics.sh already uses.

🧹 A stale directive removed

The file-wide # shellcheck disable=SC2155 was dropped, not carried. ShellCheck raises no SC2155 on the original file without it — verified explicitly before removing, the same way #928 found SC2094 had gone stale. Carrying it to all six files would have propagated dead suppression.

✅ Verification

Relocation proven: the non-blank line multiset lost only that directive, 27 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.

[Chemaclass]

🤔 Background

Related #931

Second half of the plan #942 set up: group first, split second. src/console/results.sh was 828 lines and 27 functions.

💡 Changes

File	Contents
console/line.sh	print_line — the primitive every result line goes through — and print_tap_line
console/duration.sh	format_duration{,_to_slot}
console/diff.sh	render_diff, first_line_ellipsis, snapshot_line_diff
console/test_line.sh	the ten per-test result lines and test_location_suffix
console/deferred.sh	flush_deferred_block and the five *_and_reset blocks
console/summary.sh	render_result, print_execution_time, print_hook_completed

Peers inside src/console/ rather than a src/console/results/ sub-module — console is the module, these are its files.

Layering from all 26 internal call edges: line, duration and diff have no console_results callees; test_line, deferred and summary build on them.

Function names are untouched, so they stay bashunit::console_results::* while files are named for what they hold — the same split learn/lessons/basics.sh already uses.

🧹 A stale directive removed

The file-wide # shellcheck disable=SC2155 was dropped, not carried. ShellCheck raises no SC2155 on the original file without it — verified explicitly before removing, the same way #928 found SC2094 had gone stale. Carrying it to all six files would have propagated dead suppression.

✅ Verification

Relocation proven: the non-blank line multiset lost only that directive, 27 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.

[Chemaclass]

🤔 Background

Related #931

Second half of the plan #942 set up: group first, split second. src/console/results.sh was 828 lines and 27 functions.

💡 Changes

File	Contents
console/line.sh	print_line — the primitive every result line goes through — and print_tap_line
console/duration.sh	format_duration{,_to_slot}
console/diff.sh	render_diff, first_line_ellipsis, snapshot_line_diff
console/test_line.sh	the ten per-test result lines and test_location_suffix
console/deferred.sh	flush_deferred_block and the five *_and_reset blocks
console/summary.sh	render_result, print_execution_time, print_hook_completed

Peers inside src/console/ rather than a src/console/results/ sub-module — console is the module, these are its files.

Layering from all 26 internal call edges: line, duration and diff have no console_results callees; test_line, deferred and summary build on them.

Function names are untouched, so they stay bashunit::console_results::* while files are named for what they hold — the same split learn/lessons/basics.sh already uses.

🧹 A stale directive removed

The file-wide # shellcheck disable=SC2155 was dropped, not carried. ShellCheck raises no SC2155 on the original file without it — verified explicitly before removing, the same way #928 found SC2094 had gone stale. Carrying it to all six files would have propagated dead suppression.

✅ Verification

Relocation proven: the non-blank line multiset lost only that directive, 27 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.

[Chemaclass]

🤔 Background

Related #931

Second half of the plan #942 set up: group first, split second. src/console/results.sh was 828 lines and 27 functions.

💡 Changes

File	Contents
console/line.sh	print_line — the primitive every result line goes through — and print_tap_line
console/duration.sh	format_duration{,_to_slot}
console/diff.sh	render_diff, first_line_ellipsis, snapshot_line_diff
console/test_line.sh	the ten per-test result lines and test_location_suffix
console/deferred.sh	flush_deferred_block and the five *_and_reset blocks
console/summary.sh	render_result, print_execution_time, print_hook_completed

Peers inside src/console/ rather than a src/console/results/ sub-module — console is the module, these are its files.

Layering from all 26 internal call edges: line, duration and diff have no console_results callees; test_line, deferred and summary build on them.

Function names are untouched, so they stay bashunit::console_results::* while files are named for what they hold — the same split learn/lessons/basics.sh already uses.

🧹 A stale directive removed

The file-wide # shellcheck disable=SC2155 was dropped, not carried. ShellCheck raises no SC2155 on the original file without it — verified explicitly before removing, the same way #928 found SC2094 had gone stale. Carrying it to all six files would have propagated dead suppression.

✅ Verification

Relocation proven: the non-blank line multiset lost only that directive, 27 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.

[Chemaclass]

🤔 Background

Related #931

Second half of the plan #942 set up: group first, split second. src/console/results.sh was 828 lines and 27 functions.

💡 Changes

File	Contents
console/line.sh	print_line — the primitive every result line goes through — and print_tap_line
console/duration.sh	format_duration{,_to_slot}
console/diff.sh	render_diff, first_line_ellipsis, snapshot_line_diff
console/test_line.sh	the ten per-test result lines and test_location_suffix
console/deferred.sh	flush_deferred_block and the five *_and_reset blocks
console/summary.sh	render_result, print_execution_time, print_hook_completed

Peers inside src/console/ rather than a src/console/results/ sub-module — console is the module, these are its files.

Layering from all 26 internal call edges: line, duration and diff have no console_results callees; test_line, deferred and summary build on them.

Function names are untouched, so they stay bashunit::console_results::* while files are named for what they hold — the same split learn/lessons/basics.sh already uses.

🧹 A stale directive removed

The file-wide # shellcheck disable=SC2155 was dropped, not carried. ShellCheck raises no SC2155 on the original file without it — verified explicitly before removing, the same way #928 found SC2094 had gone stale. Carrying it to all six files would have propagated dead suppression.

✅ Verification

Relocation proven: the non-blank line multiset lost only that directive, 27 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.

[Chemaclass]

🤔 Background

Related #931

Second half of the plan #942 set up: group first, split second. src/console/results.sh was 828 lines and 27 functions.

💡 Changes

File	Contents
console/line.sh	print_line — the primitive every result line goes through — and print_tap_line
console/duration.sh	format_duration{,_to_slot}
console/diff.sh	render_diff, first_line_ellipsis, snapshot_line_diff
console/test_line.sh	the ten per-test result lines and test_location_suffix
console/deferred.sh	flush_deferred_block and the five *_and_reset blocks
console/summary.sh	render_result, print_execution_time, print_hook_completed

Peers inside src/console/ rather than a src/console/results/ sub-module — console is the module, these are its files.

Layering from all 26 internal call edges: line, duration and diff have no console_results callees; test_line, deferred and summary build on them.

Function names are untouched, so they stay bashunit::console_results::* while files are named for what they hold — the same split learn/lessons/basics.sh already uses.

🧹 A stale directive removed

The file-wide # shellcheck disable=SC2155 was dropped, not carried. ShellCheck raises no SC2155 on the original file without it — verified explicitly before removing, the same way #928 found SC2094 had gone stale. Carrying it to all six files would have propagated dead suppression.

✅ Verification

Relocation proven: the non-blank line multiset lost only that directive, 27 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.