Skip to content

improve: overhaul config validation architecture -WoC#183

Merged
1Lucas1apk merged 1 commit intoPerformanC:devfrom
aroradev1010:refactor/config-validation-architecture
Mar 7, 2026
Merged

improve: overhaul config validation architecture -WoC#183
1Lucas1apk merged 1 commit intoPerformanC:devfrom
aroradev1010:refactor/config-validation-architecture

Conversation

@aroradev1010
Copy link
Contributor

Changes

  • Replaced lazy getter pattern (get: () => value) — simpler and more direct
  • Validation methods now return ValidationRule[] and aggregate into a single pass, removing the per-domain runRules() + validateProperty() indirection
  • Added crossFieldIntRule helper to reduce repetition across cross-field comparisons
  • Guarded all cluster sub-field validation behind cluster.enabled === true

Why

Write here why you think this should be merged

Checkmarks

  • The modified endpoints have been tested.
  • Used the same indentation as the rest of the project.
  • Still compatible with LavaLink clients.

Additional information

If you have any additional information, write it here

@1Lucas1apk 1Lucas1apk merged commit e7cf58a into PerformanC:dev Mar 7, 2026
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.

2 participants