Skip to content

feat: add v1beta1 API version with native YAML annotations#57

Merged
ghaskins merged 1 commit intomanetu:mainfrom
ghaskins:v1beta1-support
Feb 2, 2026
Merged

feat: add v1beta1 API version with native YAML annotations#57
ghaskins merged 1 commit intomanetu:mainfrom
ghaskins:v1beta1-support

Conversation

@ghaskins
Copy link
Copy Markdown
Contributor

@ghaskins ghaskins commented Feb 2, 2026

Add v1beta1 PolicyDomain schema version that allows annotation values to be specified as native YAML types (strings, numbers, arrays, objects) instead of requiring JSON-encoded strings as in v1alpha3/v1alpha4.

  • Add v1beta1 parser with interface{} annotation values
  • Update intermediate model to support both string and interface{} values
  • Add type switch in backend to handle both formats
  • Update all documentation to use v1beta1 as primary format

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 2, 2026

Codecov Report

❌ Patch coverage is 98.88889% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/policydomain/parsers/v1beta1/parse.go 98.83% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

cda0011
cda0011 previously approved these changes Feb 2, 2026
ecb
ecb previously approved these changes Feb 2, 2026
Copy link
Copy Markdown

@ecb ecb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ttu

Add v1beta1 PolicyDomain schema version that allows annotation values
to be specified as native YAML types (strings, numbers, arrays, objects)
instead of requiring JSON-encoded strings as in v1alpha3/v1alpha4.

- Add v1beta1 parser with interface{} annotation values
- Update intermediate model to support both string and interface{} values
- Add type switch in backend to handle both formats
- Update all documentation to use v1beta1 as primary format

Signed-off-by: Greg Haskins <greg@manetu.com>
@ghaskins ghaskins dismissed stale reviews from ecb and cda0011 via 7c021ab February 2, 2026 18:34
Copy link
Copy Markdown
Contributor

@muralisrini muralisrini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ghaskins ghaskins merged commit 76951c8 into manetu:main Feb 2, 2026
16 checks passed
@ghaskins ghaskins deleted the v1beta1-support branch February 2, 2026 22:53
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.

4 participants