Skip to content

[AI-8th] add Spring Boot configuration metadata support#1422

Open
pmupkin wants to merge 1 commit into
sofastack:masterfrom
pmupkin:Add-Configuration-Metadata
Open

[AI-8th] add Spring Boot configuration metadata support#1422
pmupkin wants to merge 1 commit into
sofastack:masterfrom
pmupkin:Add-Configuration-Metadata

Conversation

@pmupkin
Copy link
Copy Markdown

@pmupkin pmupkin commented Apr 25, 2026

This PR(related issue #1404) adds Spring Boot configuration metadata support for SOFABoot IDE auto-completion.

Changes

  1. Added additional-spring-configuration-metadata.json to sofa-boot-autoconfigure
  • Covers RPC, Runtime, ISLE, Ark, and Tracer configuration properties.
  • Adds descriptions, defaults, value hints, and conditional auto-configuration switches.
  • Adds metadata for Tracer integration toggles such as Kafka, Redis, RabbitMQ, RocketMQ, Zipkin, Spring MVC, and others.
  • Ignores sofa.boot.rpc.environment, which is injected internally and should not be exposed as a user-facing configuration property.
  • Added additional-spring-configuration-metadata.json to sofa-boot-actuator-autoconfigure
  1. Covers health check, startup, and virtual-thread startup configuration metadata.
  • Adds default values and hints for health status mapping/order.
  • Added metadata validation tests for both modules
  1. Verifies additional metadata structure.
  • Verifies expected groups, properties, and hints.
  • Verifies generated spring-configuration-metadata.json contains merged metadata items.

@CrazyHZM
Copy link
Copy Markdown
Member

@pmupkin Please fix ut with jdk25.

@pmupkin
Copy link
Copy Markdown
Author

pmupkin commented Apr 28, 2026

@pmupkin Please fix ut with jdk25.

ok

CrazyHZM
CrazyHZM previously approved these changes May 8, 2026
@CrazyHZM
Copy link
Copy Markdown
Member

CrazyHZM commented May 8, 2026

@pmupkin Please fix ut for jdk25:
Error: ConfigurationMetadataTests.generatedMetadataShouldContainMergedAdditionalItems:65

@CrazyHZM CrazyHZM self-requested a review May 8, 2026 09:54
@CrazyHZM CrazyHZM dismissed their stale review May 8, 2026 09:54

re-review

@pmupkin
Copy link
Copy Markdown
Author

pmupkin commented May 11, 2026

@pmupkin Please fix ut for jdk25: Error: ConfigurationMetadataTests.generatedMetadataShouldContainMergedAdditionalItems:65

OK, I'll do it this week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants