Skip to content

Harden stdlib range validation#44

Merged
codeforester merged 1 commit into
masterfrom
enhancement/41-20260619-stdlib-file-docs
Jun 20, 2026
Merged

Harden stdlib range validation#44
codeforester merged 1 commit into
masterfrom
enhancement/41-20260619-stdlib-file-docs

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

  • Refactor assert_integer_range to validate literal bounds directly instead of relying on Bash dynamic scoping.
  • Add focused BATS coverage for invalid range bounds and clearer bound-specific error messages.
  • Remove a production # FIX: development comment and guard against future # FIX: comments in validation.
  • Document the stdlib root-layout assumption and make the Bash 4.2+ requirement prominent in the README.

Validation

  • bats lib/bash/std/tests/lib_std.bats --filter "assert_integer_range rejects non-integer bounds directly"
  • bats lib/bash/std/tests/lib_std.bats lib/bash/file/tests/lib_file.bats
  • ./tests/validate.sh
  • git diff --check

Fixes #41

@codeforester codeforester merged commit 8069f7a into master Jun 20, 2026
2 checks passed
@codeforester codeforester deleted the enhancement/41-20260619-stdlib-file-docs branch June 20, 2026 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Harden stdlib/file cleanup and adoption docs

1 participant