Skip to content

Refactor RuntimeHints to scan entire packages for JsonInclude annotat… #2681

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 9, 2025

Conversation

sobychacko
Copy link
Contributor

@sobychacko sobychacko commented Apr 9, 2025

…ions

  • Replace class-by-class scanning with comprehensive package-level scanning to capture all JSON-annotated classes within each model's package hierarchy.
  • Update tests to verify registered types and add specific type assertions.

@sobychacko sobychacko added this to the 1.0.0-M7 milestone Apr 9, 2025
…ions

- Replace class-by-class scanning with comprehensive package-level scanning to capture
  all JSON-annotated classes within each model's package hierarchy.
- Update tests to verify registered types and add specific type assertions.

Signed-off-by: Soby Chacko <soby.chacko@broadcom.com>
@sobychacko sobychacko force-pushed the aot-jsoninclude-fix branch from 10b00ce to 923c5cc Compare April 9, 2025 23:11
@sobychacko sobychacko merged commit 186175a into spring-projects:main Apr 9, 2025
2 checks passed
TheovanKraay pushed a commit to TheovanKraay/spring-ai that referenced this pull request Apr 11, 2025
…ions (spring-projects#2681)

- Replace class-by-class scanning with comprehensive package-level scanning to capture
  all JSON-annotated classes within each model's package hierarchy.
- Update tests to verify registered types and add specific type assertions.

Signed-off-by: Soby Chacko <soby.chacko@broadcom.com>
Signed-off-by: Theo van Kraay <theo.van@microsoft.com>
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