Skip to content

fix: validate nested configuration value graphs - #365

Open
codeforester wants to merge 2 commits into
mainfrom
bug/359-20260918-validate-config-graph
Open

codeforester wants to merge 2 commits into
mainfrom
bug/359-20260918-validate-config-graph

Conversation

@codeforester

Copy link
Copy Markdown
Contributor

Summary

  • Validate string keys throughout nested mapping/list values before merge or provenance traversal.
  • Reject cycles and excessive depth/traversal with source-aware ConfigurationError messages while permitting shared aliases.
  • Cover first insertion, overlays, native/attached human/JSON boundaries, cycles, and depth.

Closes #359

Validation

  • uv run --extra dev --extra typer --extra quality python -m pytest tests/test_batteries_included_config.py tests/test_explicit_config_validation.py -q
  • uv run --extra dev --extra typer --extra quality python -m mypy --strict lib/python/base_cli/config.py
  • Ruff check/format and git diff --check

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.

bug: validate nested configuration shape before merge and provenance traversal

1 participant