fix(core-spring): autoconfigure metaobjects.metadata-sources loader (complete MetaDataAutoConfiguration wiring)

MetaDataLoaderConfiguration (the @ConfigurationProperties(prefix="metaobjects")
that builds a MetaDataLoader from metaobjects.metadata-sources) was never wired
into MetaDataAutoConfiguration, despite its javadoc claiming auto-discovery. A
starter-only user setting metaobjects.metadata-sources got it ignored, and
primaryMetaDataLoader then threw "No MetaDataLoader found".

- @import(MetaDataLoaderConfiguration.class) on MetaDataAutoConfiguration so the
  property-driven loader participates in autoconfiguration.
- @ConditionalOnMissingBean(MetaDataLoader.class) on applicationMetaDataLoader so
  it backs off when the app supplies its own loader.
- MetaDataSpringIntegrationTest now loads the autoconfig via
  @ImportAutoConfiguration (real autoconfig-runs-last ordering) instead of as a
  raw @ContextConfiguration class, so @ConditionalOnMissingBean evaluates against
  the app-supplied loader as it would in production.

TDD red->green via new MetaDataLoaderAutoConfigurationTest.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: dmealing

server/java/core-spring/src/main/java/com/metaobjects/spring/MetaDataAutoConfiguration.java

@@ -6,6 +6,7 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.autoconfigure.AutoConfiguration;
 import org.springframework.context.annotation.Bean;
+import org.springframework.context.annotation.Import;
 import org.springframework.context.annotation.Primary;
 
 import java.util.List;
@@ -41,6 +42,7 @@
  * @since 6.0.0
  */
 @AutoConfiguration
+@Import(MetaDataLoaderConfiguration.class)
 public class MetaDataAutoConfiguration {
 
     /**

server/java/core-spring/src/main/java/com/metaobjects/spring/MetaDataLoaderConfiguration.java

@@ -3,6 +3,7 @@
 import com.metaobjects.loader.LoaderOptions;
 import com.metaobjects.loader.MetaDataLoader;
 
+import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
 import org.springframework.boot.context.properties.ConfigurationProperties;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
@@ -63,6 +64,7 @@ public class MetaDataLoaderConfiguration {
      * {@link MetaDataAutoConfiguration}.</p>
      */
     @Bean
+    @ConditionalOnMissingBean(MetaDataLoader.class)
     public MetaDataLoader applicationMetaDataLoader() throws Exception {
         if (metadataSources.isEmpty()) {
             // Default: look for metadata files in classpath

server/java/core-spring/src/test/java/com/metaobjects/spring/MetaDataLoaderAutoConfigurationTest.java

@@ -0,0 +1,33 @@
+package com.metaobjects.spring;
+
+import com.metaobjects.loader.MetaDataLoader;
+import com.metaobjects.registry.MetaDataLoaderRegistry;
+import org.junit.Test;
+import org.springframework.boot.autoconfigure.AutoConfigurations;
+import org.springframework.boot.test.context.runner.ApplicationContextRunner;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+/**
+ * Verifies the starter honors metaobjects.metadata-sources: the autoconfiguration
+ * builds a MetaDataLoader from the property and exposes it through the registry,
+ * with no app-supplied loader bean.
+ */
+public class MetaDataLoaderAutoConfigurationTest {
+
+    private final ApplicationContextRunner runner = new ApplicationContextRunner()
+        .withConfiguration(AutoConfigurations.of(MetaDataAutoConfiguration.class));
+
+    @Test
+    public void buildsLoaderFromMetadataSourcesProperty() {
+        runner.withPropertyValues("metaobjects.metadata-sources=classpath:metadata/test-omdb-starter.json")
+            .run(ctx -> {
+                assertThat(ctx).hasSingleBean(MetaDataLoaderRegistry.class);
+                assertThat(ctx).hasBean("primaryMetaDataLoader");
+                MetaDataLoaderRegistry reg = ctx.getBean(MetaDataLoaderRegistry.class);
+                assertThat(reg.getDataLoaders()).isNotEmpty();
+                MetaDataLoader primary = ctx.getBean(MetaDataLoader.class);
+                assertThat(primary).isNotNull();
+            });
+    }
+}

server/java/core-spring/src/test/java/com/metaobjects/spring/MetaDataSpringIntegrationTest.java

@@ -10,6 +10,7 @@
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.autoconfigure.ImportAutoConfiguration;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 import org.springframework.test.context.ContextConfiguration;
@@ -29,7 +30,6 @@
  */
 @RunWith(SpringJUnit4ClassRunner.class)
 @ContextConfiguration(classes = {
-    MetaDataAutoConfiguration.class,
     MetaDataSpringIntegrationTest.TestConfiguration.class
 })
 public class MetaDataSpringIntegrationTest {
@@ -47,8 +47,9 @@ public class MetaDataSpringIntegrationTest {
      * Test Spring configuration that creates test MetaDataLoader beans
      */
     @Configuration
+    @ImportAutoConfiguration(MetaDataAutoConfiguration.class)
     static class TestConfiguration {
-        
+
         @Bean
         public MetaDataLoader testMetaDataLoader() throws Exception {
             MetaDataLoader loader = new MetaDataLoader(

server/java/core-spring/src/test/resources/metadata/test-omdb-starter.json

@@ -0,0 +1,10 @@
+{
+  "metadata.root": {
+    "package": "acme::starter",
+    "children": [
+      { "object.entity": { "name": "Widget", "children": [
+        { "field.long": { "name": "id" } }
+      ] } }
+    ]
+  }
+}