Skip to content

Reject unknown run config fields#56

Merged
stacknil merged 1 commit into
mainfrom
stacknil/strict-run-config-fields
Jun 13, 2026
Merged

Reject unknown run config fields#56
stacknil merged 1 commit into
mainfrom
stacknil/strict-run-config-fields

Conversation

@stacknil

Copy link
Copy Markdown
Owner

Summary

  • reject unknown top-level, time, feature, and rule config fields before running the pipeline
  • keep rule unknown-field validation on the same helper path
  • add CLI and config-validation coverage for typo-like fields
  • document strict run config validation in README

Validation

  • pytest tests/test_run_config_validation.py tests/test_cli_errors.py tests/test_pipeline_e2e.py --basetemp .pytest-artifacts-strict-config-2
  • pytest --basetemp .pytest-artifacts-full-strict-config
  • python -m telemetry_window_demo.cli run --config configs/default.yaml
  • git diff --cached --check
  • staged diff privacy/secret keyword scan

@stacknil stacknil merged commit 2a9077e into main Jun 13, 2026
2 checks passed
@stacknil stacknil deleted the stacknil/strict-run-config-fields branch June 13, 2026 03:42
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.

1 participant