refactor(assert): group the assert_*.sh files into a src/assert/ module

[Chemaclass]

🤔 Background

Related #940

Ten files and 2313 lines were already aggregated together by src/assertions.sh — the concept existed, only the directory was missing. First group under #940, which is the counterpart to #931: #931 splits files that are too large, this one collects files that belong together.

💡 Changes

src/assertions.sh -> src/assert/index.sh
src/assert.sh -> src/assert/core.sh
src/assert_<topic>.sh -> src/assert/<topic>.sh (9 files)

The assert_ prefix drops inside the directory, which now carries the concept. No function is renamed and no line of any moved file changes — git records all ten as pure renames with a zero-line diff, and the aggregator's source order is preserved one-to-one.

skip_todo.sh and test_doubles.sh stay flat and keep being sourced from the index, exactly as assertions.sh did. They're part of the same public test surface but they aren't assertions, so they don't belong inside src/assert/.

🧹 A special case this deletes

Two callers hardcoded these paths:

• tests/unit/completions_test.sh derives the public assertion list from src/assert*.sh for the completions parity contract → src/assert/*.sh.
• tests/unit/build_test.sh special-cased src/assertions.sh as the one flat-file aggregator sitting outside the src/*/index.sh glob. That branch is deleted — every aggregator is now an index.sh, so the glob covers them all. This is exactly the simplification ADR-010 predicted when the "beside" argument was withdrawn in refactor(src): move module aggregators inside their directory as index.sh #934.

✅ Verification

The built artifact's code content is identical — every difference is a comment or an embed marker — and bash build.sh bin -v prints ✅ Build verified ✅. Function count 101 before and after.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck (per file, no -x) · git check-ignore clean · no .editorconfig rule affected.

Architecture map updated — including a line that still described the pre-#934src/<module>.sh convention.

[Chemaclass]

🤔 Background

Related #940

Ten files and 2313 lines were already aggregated together by src/assertions.sh — the concept existed, only the directory was missing. First group under #940, which is the counterpart to #931: #931 splits files that are too large, this one collects files that belong together.

💡 Changes

src/assertions.sh -> src/assert/index.sh
src/assert.sh -> src/assert/core.sh
src/assert_<topic>.sh -> src/assert/<topic>.sh (9 files)

The assert_ prefix drops inside the directory, which now carries the concept. No function is renamed and no line of any moved file changes — git records all ten as pure renames with a zero-line diff, and the aggregator's source order is preserved one-to-one.

skip_todo.sh and test_doubles.sh stay flat and keep being sourced from the index, exactly as assertions.sh did. They're part of the same public test surface but they aren't assertions, so they don't belong inside src/assert/.

🧹 A special case this deletes

Two callers hardcoded these paths:

• tests/unit/completions_test.sh derives the public assertion list from src/assert*.sh for the completions parity contract → src/assert/*.sh.
• tests/unit/build_test.sh special-cased src/assertions.sh as the one flat-file aggregator sitting outside the src/*/index.sh glob. That branch is deleted — every aggregator is now an index.sh, so the glob covers them all. This is exactly the simplification ADR-010 predicted when the "beside" argument was withdrawn in refactor(src): move module aggregators inside their directory as index.sh #934.

✅ Verification

The built artifact's code content is identical — every difference is a comment or an embed marker — and bash build.sh bin -v prints ✅ Build verified ✅. Function count 101 before and after.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck (per file, no -x) · git check-ignore clean · no .editorconfig rule affected.

Architecture map updated — including a line that still described the pre-#934src/<module>.sh convention.

[Chemaclass]

🤔 Background

Related #940

Ten files and 2313 lines were already aggregated together by src/assertions.sh — the concept existed, only the directory was missing. First group under #940, which is the counterpart to #931: #931 splits files that are too large, this one collects files that belong together.

💡 Changes

src/assertions.sh -> src/assert/index.sh
src/assert.sh -> src/assert/core.sh
src/assert_<topic>.sh -> src/assert/<topic>.sh (9 files)

The assert_ prefix drops inside the directory, which now carries the concept. No function is renamed and no line of any moved file changes — git records all ten as pure renames with a zero-line diff, and the aggregator's source order is preserved one-to-one.

skip_todo.sh and test_doubles.sh stay flat and keep being sourced from the index, exactly as assertions.sh did. They're part of the same public test surface but they aren't assertions, so they don't belong inside src/assert/.

🧹 A special case this deletes

Two callers hardcoded these paths:

• tests/unit/completions_test.sh derives the public assertion list from src/assert*.sh for the completions parity contract → src/assert/*.sh.
• tests/unit/build_test.sh special-cased src/assertions.sh as the one flat-file aggregator sitting outside the src/*/index.sh glob. That branch is deleted — every aggregator is now an index.sh, so the glob covers them all. This is exactly the simplification ADR-010 predicted when the "beside" argument was withdrawn in refactor(src): move module aggregators inside their directory as index.sh #934.

✅ Verification

The built artifact's code content is identical — every difference is a comment or an embed marker — and bash build.sh bin -v prints ✅ Build verified ✅. Function count 101 before and after.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck (per file, no -x) · git check-ignore clean · no .editorconfig rule affected.

Architecture map updated — including a line that still described the pre-#934src/<module>.sh convention.

[Chemaclass]

🤔 Background

Related #940

Ten files and 2313 lines were already aggregated together by src/assertions.sh — the concept existed, only the directory was missing. First group under #940, which is the counterpart to #931: #931 splits files that are too large, this one collects files that belong together.

💡 Changes

src/assertions.sh -> src/assert/index.sh
src/assert.sh -> src/assert/core.sh
src/assert_<topic>.sh -> src/assert/<topic>.sh (9 files)

The assert_ prefix drops inside the directory, which now carries the concept. No function is renamed and no line of any moved file changes — git records all ten as pure renames with a zero-line diff, and the aggregator's source order is preserved one-to-one.

skip_todo.sh and test_doubles.sh stay flat and keep being sourced from the index, exactly as assertions.sh did. They're part of the same public test surface but they aren't assertions, so they don't belong inside src/assert/.

🧹 A special case this deletes

Two callers hardcoded these paths:

• tests/unit/completions_test.sh derives the public assertion list from src/assert*.sh for the completions parity contract → src/assert/*.sh.
• tests/unit/build_test.sh special-cased src/assertions.sh as the one flat-file aggregator sitting outside the src/*/index.sh glob. That branch is deleted — every aggregator is now an index.sh, so the glob covers them all. This is exactly the simplification ADR-010 predicted when the "beside" argument was withdrawn in refactor(src): move module aggregators inside their directory as index.sh #934.

✅ Verification

The built artifact's code content is identical — every difference is a comment or an embed marker — and bash build.sh bin -v prints ✅ Build verified ✅. Function count 101 before and after.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck (per file, no -x) · git check-ignore clean · no .editorconfig rule affected.

Architecture map updated — including a line that still described the pre-#934src/<module>.sh convention.

[Chemaclass]

🤔 Background

Related #940

Ten files and 2313 lines were already aggregated together by src/assertions.sh — the concept existed, only the directory was missing. First group under #940, which is the counterpart to #931: #931 splits files that are too large, this one collects files that belong together.

💡 Changes

src/assertions.sh -> src/assert/index.sh
src/assert.sh -> src/assert/core.sh
src/assert_<topic>.sh -> src/assert/<topic>.sh (9 files)

The assert_ prefix drops inside the directory, which now carries the concept. No function is renamed and no line of any moved file changes — git records all ten as pure renames with a zero-line diff, and the aggregator's source order is preserved one-to-one.

skip_todo.sh and test_doubles.sh stay flat and keep being sourced from the index, exactly as assertions.sh did. They're part of the same public test surface but they aren't assertions, so they don't belong inside src/assert/.

🧹 A special case this deletes

Two callers hardcoded these paths:

• tests/unit/completions_test.sh derives the public assertion list from src/assert*.sh for the completions parity contract → src/assert/*.sh.
• tests/unit/build_test.sh special-cased src/assertions.sh as the one flat-file aggregator sitting outside the src/*/index.sh glob. That branch is deleted — every aggregator is now an index.sh, so the glob covers them all. This is exactly the simplification ADR-010 predicted when the "beside" argument was withdrawn in refactor(src): move module aggregators inside their directory as index.sh #934.

✅ Verification

The built artifact's code content is identical — every difference is a comment or an embed marker — and bash build.sh bin -v prints ✅ Build verified ✅. Function count 101 before and after.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck (per file, no -x) · git check-ignore clean · no .editorconfig rule affected.

Architecture map updated — including a line that still described the pre-#934src/<module>.sh convention.

[Chemaclass]

🤔 Background

Related #940

Ten files and 2313 lines were already aggregated together by src/assertions.sh — the concept existed, only the directory was missing. First group under #940, which is the counterpart to #931: #931 splits files that are too large, this one collects files that belong together.

💡 Changes

src/assertions.sh -> src/assert/index.sh
src/assert.sh -> src/assert/core.sh
src/assert_<topic>.sh -> src/assert/<topic>.sh (9 files)

The assert_ prefix drops inside the directory, which now carries the concept. No function is renamed and no line of any moved file changes — git records all ten as pure renames with a zero-line diff, and the aggregator's source order is preserved one-to-one.

skip_todo.sh and test_doubles.sh stay flat and keep being sourced from the index, exactly as assertions.sh did. They're part of the same public test surface but they aren't assertions, so they don't belong inside src/assert/.

🧹 A special case this deletes

Two callers hardcoded these paths:

• tests/unit/completions_test.sh derives the public assertion list from src/assert*.sh for the completions parity contract → src/assert/*.sh.
• tests/unit/build_test.sh special-cased src/assertions.sh as the one flat-file aggregator sitting outside the src/*/index.sh glob. That branch is deleted — every aggregator is now an index.sh, so the glob covers them all. This is exactly the simplification ADR-010 predicted when the "beside" argument was withdrawn in refactor(src): move module aggregators inside their directory as index.sh #934.

✅ Verification

The built artifact's code content is identical — every difference is a comment or an embed marker — and bash build.sh bin -v prints ✅ Build verified ✅. Function count 101 before and after.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck (per file, no -x) · git check-ignore clean · no .editorconfig rule affected.

Architecture map updated — including a line that still described the pre-#934src/<module>.sh convention.

[Chemaclass]

🤔 Background

Related #940

Ten files and 2313 lines were already aggregated together by src/assertions.sh — the concept existed, only the directory was missing. First group under #940, which is the counterpart to #931: #931 splits files that are too large, this one collects files that belong together.

💡 Changes

src/assertions.sh -> src/assert/index.sh
src/assert.sh -> src/assert/core.sh
src/assert_<topic>.sh -> src/assert/<topic>.sh (9 files)

The assert_ prefix drops inside the directory, which now carries the concept. No function is renamed and no line of any moved file changes — git records all ten as pure renames with a zero-line diff, and the aggregator's source order is preserved one-to-one.

skip_todo.sh and test_doubles.sh stay flat and keep being sourced from the index, exactly as assertions.sh did. They're part of the same public test surface but they aren't assertions, so they don't belong inside src/assert/.

🧹 A special case this deletes

Two callers hardcoded these paths:

• tests/unit/completions_test.sh derives the public assertion list from src/assert*.sh for the completions parity contract → src/assert/*.sh.
• tests/unit/build_test.sh special-cased src/assertions.sh as the one flat-file aggregator sitting outside the src/*/index.sh glob. That branch is deleted — every aggregator is now an index.sh, so the glob covers them all. This is exactly the simplification ADR-010 predicted when the "beside" argument was withdrawn in refactor(src): move module aggregators inside their directory as index.sh #934.

✅ Verification

The built artifact's code content is identical — every difference is a comment or an embed marker — and bash build.sh bin -v prints ✅ Build verified ✅. Function count 101 before and after.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck (per file, no -x) · git check-ignore clean · no .editorconfig rule affected.

Architecture map updated — including a line that still described the pre-#934src/<module>.sh convention.

[Chemaclass]

🤔 Background

Related #940

Ten files and 2313 lines were already aggregated together by src/assertions.sh — the concept existed, only the directory was missing. First group under #940, which is the counterpart to #931: #931 splits files that are too large, this one collects files that belong together.

💡 Changes

src/assertions.sh -> src/assert/index.sh
src/assert.sh -> src/assert/core.sh
src/assert_<topic>.sh -> src/assert/<topic>.sh (9 files)

The assert_ prefix drops inside the directory, which now carries the concept. No function is renamed and no line of any moved file changes — git records all ten as pure renames with a zero-line diff, and the aggregator's source order is preserved one-to-one.

skip_todo.sh and test_doubles.sh stay flat and keep being sourced from the index, exactly as assertions.sh did. They're part of the same public test surface but they aren't assertions, so they don't belong inside src/assert/.

🧹 A special case this deletes

Two callers hardcoded these paths:

• tests/unit/completions_test.sh derives the public assertion list from src/assert*.sh for the completions parity contract → src/assert/*.sh.
• tests/unit/build_test.sh special-cased src/assertions.sh as the one flat-file aggregator sitting outside the src/*/index.sh glob. That branch is deleted — every aggregator is now an index.sh, so the glob covers them all. This is exactly the simplification ADR-010 predicted when the "beside" argument was withdrawn in refactor(src): move module aggregators inside their directory as index.sh #934.

✅ Verification

The built artifact's code content is identical — every difference is a comment or an embed marker — and bash build.sh bin -v prints ✅ Build verified ✅. Function count 101 before and after.

Green: sequential (1604 passed, 0 failed) · --parallel --simple --strict · make sa · make lint · CI-mode ShellCheck (per file, no -x) · git check-ignore clean · no .editorconfig rule affected.

Architecture map updated — including a line that still described the pre-#934src/<module>.sh convention.