Skip to content

fix(kotlin): validate Klaxon string lengths - #3417

Merged
schani merged 1 commit into
fix/kotlin-klaxon-numeric-boundsfrom
fix/kotlin-klaxon-string-length
Sep 4, 2026
Merged

fix(kotlin): validate Klaxon string lengths#3417
schani merged 1 commit into
fix/kotlin-klaxon-numeric-boundsfrom
fix/kotlin-klaxon-string-length

Conversation

@schani

@schani schani commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

  • enforce schema minimum and maximum string lengths in Klaxon models
  • preserve optional constrained properties

Tests

  • npm run build
  • npm run lint
  • FIXTURE=schema-kotlin npm run test:fixtures -- test/inputs/schema/minmaxlength.schema test/inputs/schema/optional-constraints.schema

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

4 files differ — 4 modified, 0 new, 0 deleted
18 changed lines — +18 / −0

Open the generated-output report →

@schani
schani force-pushed the fix/kotlin-klaxon-string-length branch from 2848b40 to 2b2423d Compare September 1, 2026 21:30
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

4 files differ — 4 modified, 0 new, 0 deleted
18 changed lines — +18 / −0

Open the generated-output report →

@schani
schani force-pushed the fix/kotlin-klaxon-numeric-bounds branch from 9b2d0d5 to 8db938b Compare September 1, 2026 21:40
@schani
schani force-pushed the fix/kotlin-klaxon-string-length branch from 2b2423d to 6e512e2 Compare September 1, 2026 21:40
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

4 files differ — 4 modified, 0 new, 0 deleted
54 changed lines — +54 / −0

Open the generated-output report →

@schani
schani merged commit c6bba1f into fix/kotlin-klaxon-numeric-bounds Sep 4, 2026
34 checks passed
@schani
schani deleted the fix/kotlin-klaxon-string-length branch September 4, 2026 18:07
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