Skip to content

fix #36 validation does not support list elements#37

Merged
bristermitten merged 19 commits into
masterfrom
36-validation-does-not-support-list-elements
Jun 12, 2026
Merged

fix #36 validation does not support list elements#37
bristermitten merged 19 commits into
masterfrom
36-validation-does-not-support-list-elements

Conversation

@bristermitten

@bristermitten bristermitten commented Jun 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner
  • fix validation does not support list elements etc #36 by refactoring generated serdes code to be lower level and rely more on mittenlib serdes code rather than outsourcing
  • add jacoco and improve test coverage quite a bit
  • move to palantir javapoet and add a bit of javadocs to generated code
  • add proper public interface for saving configs

@bristermitten
build: configure JaCoCo code coverage reporting
09f1ca9
@bristermitten
feat(core): support TYPE_USE target for validation annotations
eb207ed
@bristermitten
feat(core): add Set deserialization helper and fix potential NPE
be1f612
@bristermitten
feat(processor): support collection and map element validation
9fe5964
@bristermitten
test(processor): add compilation and integration tests for collection…
4863dee 
…/map validation
@bristermitten
build: exclude JMH benchmark package from SpotBugs analysis
481d694
@bristermitten
fix(processor): register DTO mappings in GeneratedTypeCache
dbb40ed
@bristermitten
test(processor): add CustomSerializerTest and non-static CustomDeseri…
3cb97fd 
…alizer test
@bristermitten
test(processor): add VerifierTest for compiler error/warning verifica…
caa168d 
…tion
@bristermitten
test(processor): add unit tests for utility and exception coverage
77689f1
@bristermitten
build: exclude generated files from jacoco
6e74c44
@bristermitten
fix(annotation-processor): improve error reporting in parsers
20fb687 
Replaced thrown IllegalStateException and IllegalArgumentException with
MessagerUtils.error in ConfigClassParser, CustomDeserializers, and
CustomSerializers to ensure compilation gracefully fails and displays
helpful messages. Also fixed ConfigClassParser ignoring enclosed
non-config classes and added related tests.
@bristermitten
test(annotation-processor): fix compilation and execution of existing…
f3b9c34 
… tests

Split multiple public classes into separate files in DeserializerGeneratorTest.
Corrected EnumParsingSchemes imports and assertions in TypesUtilTest to
ensure they compile properly and pass.
@bristermitten
test(annotation-processor): improve test coverage for utility classes…
bf8d8ec 
… and code generators

Made several internal component constructors public to facilitate unit
testing. Added tests for GenericTypeDeserializerGenerator, ElementsFinder,
PrivateAnnotations, and TypeSpecUtil, bringing overall annotation-processor
test coverage significantly higher.
@bristermitten
fix: improve quality of generated serialization code
bb39a78
@bristermitten
refactor: take the palantir pill
6af3c79
@bristermitten
refactor(annotation-processor): general tidy up
b84f74e
@bristermitten bristermitten linked an issue Jun 11, 2026 that may be closed by this pull request
validation does not support list elements etc #36
Closed
@bristermitten bristermitten added the enhancement New feature or request label Jun 11, 2026
@bristermitten bristermitten merged commit 4c3233b into master Jun 12, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

validation does not support list elements etc

1 participant

@bristermitten