Skip to content

feat: validate governance configuration at load time - #24

Merged
vitormattos merged 4 commits into
mainfrom
feat/config-validation
Sep 20, 2026
Merged

vitormattos merged 4 commits into
mainfrom
feat/config-validation

Conversation

@vitormattos

Copy link
Copy Markdown
Member

Summary

Validate caller-owned governance configuration before repository probing or reconciliation.

The validator:

  • rejects unknown keys instead of silently ignoring typos;
  • validates nested rule parameters and bypass actors;
  • rejects unsupported rule types;
  • reports the exact JSON path that is invalid;
  • runs before any GitHub API interaction.

This implements #19 without adding a runtime validation dependency.

Signed-off-by: Vitor Mattos <vitor@php.rio>
Signed-off-by: Vitor Mattos <vitor@php.rio>
Signed-off-by: Vitor Mattos <vitor@php.rio>
Signed-off-by: Vitor Mattos <vitor@php.rio>
@vitormattos
vitormattos merged commit e292fe2 into main Sep 20, 2026
6 checks passed
@vitormattos
vitormattos deleted the feat/config-validation branch September 20, 2026 01:27
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