Skip to content

fix: reject invalid explicit config paths - #78

Merged
codeforester merged 1 commit into
mainfrom
bug/56-20260802-explicit-config-validation
Aug 2, 2026
Merged

codeforester merged 1 commit into
mainfrom
bug/56-20260802-explicit-config-validation

Conversation

@codeforester

Copy link
Copy Markdown
Contributor

Summary

  • validate explicit --config values as expanded, existing, readable regular files before profile or runtime startup
  • keep absent profile-owned implicit files optional while making the generic explicit loader strict and race-aware
  • preserve stable malformed YAML and non-mapping errors and surface invalid UTF-8 as a safe configuration error
  • document the strict explicit-path and optional implicit-file contracts

Validation

  • full 253-test suite on Click 8.4.2
  • full 253-test suite on Click 8.1.8
  • isolated wheel and sdist build and artifact validation
  • 314 downstream Base engine tests pass; four unrelated baseline failures reproduce unchanged on current main
  • adversarial missing, unreadable, non-regular, tilde, parse-to-load race, YAML-shape, encoding, and no-side-effect coverage

Closes #56

@codeforester
codeforester marked this pull request as ready for review August 2, 2026 20:49
@codeforester
codeforester merged commit ce9f896 into main Aug 2, 2026
17 checks passed
@codeforester
codeforester deleted the bug/56-20260802-explicit-config-validation branch August 2, 2026 20:50
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.

Reject missing explicit configuration files

1 participant