fix: correct visibility scope issues in Spring AI components

- Change return types from implementation to interface in AutoConfiguration beans
  - OllamaConnectionDetails in OllamaApiAutoConfiguration
  - WeaviateConnectionDetails in WeaviateVectorStoreAutoConfiguration

- Fix visibility modifiers for:
  - Make Aws class public in OpenSearchVectorStoreProperties
  - Set OpenAIAutoConfigurationUtil.resolveConnectionProperties to package-private
  - Change static NumericBoundary constants to private in RedisFilterExpressionConverter
  - Explicitly mark metadataFields as final in RedisFilterExpressionConverter

These changes ensure components are properly encapsulated and follow API design best practices
by exposing only what's necessary through interfaces rather than implementations.

Signed-off-by: Soby Chacko <[email protected]>

Author: sobychacko

auto-configurations/models/spring-ai-autoconfigure-model-ollama/src/main/java/org/springframework/ai/model/ollama/autoconfigure/OllamaApiAutoConfiguration.java

@@ -42,7 +42,7 @@ public class OllamaApiAutoConfiguration {
 
 	@Bean
 	@ConditionalOnMissingBean(OllamaConnectionDetails.class)
-	public PropertiesOllamaConnectionDetails ollamaConnectionDetails(OllamaConnectionProperties properties) {
+	public OllamaConnectionDetails ollamaConnectionDetails(OllamaConnectionProperties properties) {
 		return new PropertiesOllamaConnectionDetails(properties);
 	}
 

auto-configurations/models/spring-ai-autoconfigure-model-openai/src/main/java/org/springframework/ai/model/openai/autoconfigure/OpenAIAutoConfigurationUtil.java

@@ -33,8 +33,8 @@ private OpenAIAutoConfigurationUtil() {
 		// Avoids instantiation
 	}
 
-	public static @NotNull ResolvedConnectionProperties resolveConnectionProperties(
-			OpenAiParentProperties commonProperties, OpenAiParentProperties modelProperties, String modelType) {
+	static @NotNull ResolvedConnectionProperties resolveConnectionProperties(OpenAiParentProperties commonProperties,
+			OpenAiParentProperties modelProperties, String modelType) {
 
 		String baseUrl = StringUtils.hasText(modelProperties.getBaseUrl()) ? modelProperties.getBaseUrl()
 				: commonProperties.getBaseUrl();

auto-configurations/vector-stores/spring-ai-autoconfigure-vector-store-opensearch/src/main/java/org/springframework/ai/vectorstore/opensearch/autoconfigure/OpenSearchVectorStoreProperties.java

@@ -144,7 +144,7 @@ public void setAws(Aws aws) {
 		this.aws = aws;
 	}
 
-	static class Aws {
+	public static class Aws {
 
 		private String domainName;
 

auto-configurations/vector-stores/spring-ai-autoconfigure-vector-store-weaviate/src/main/java/org/springframework/ai/vectorstore/weaviate/autoconfigure/WeaviateVectorStoreAutoConfiguration.java

@@ -52,7 +52,7 @@ public class WeaviateVectorStoreAutoConfiguration {
 
 	@Bean
 	@ConditionalOnMissingBean(WeaviateConnectionDetails.class)
-	public PropertiesWeaviateConnectionDetails weaviateConnectionDetails(WeaviateVectorStoreProperties properties) {
+	public WeaviateConnectionDetails weaviateConnectionDetails(WeaviateVectorStoreProperties properties) {
 		return new PropertiesWeaviateConnectionDetails(properties);
 	}
 

vector-stores/spring-ai-redis-store/src/main/java/org/springframework/ai/vectorstore/redis/RedisFilterExpressionConverter.java

@@ -38,11 +38,11 @@
  */
 public class RedisFilterExpressionConverter extends AbstractFilterExpressionConverter {
 
-	public static final NumericBoundary POSITIVE_INFINITY = new NumericBoundary(Double.POSITIVE_INFINITY, true);
+	static final NumericBoundary POSITIVE_INFINITY = new NumericBoundary(Double.POSITIVE_INFINITY, true);
 
-	public static final NumericBoundary NEGATIVE_INFINITY = new NumericBoundary(Double.NEGATIVE_INFINITY, true);
+	static final NumericBoundary NEGATIVE_INFINITY = new NumericBoundary(Double.NEGATIVE_INFINITY, true);
 
-	private Map<String, MetadataField> metadataFields;
+	private final Map<String, MetadataField> metadataFields;
 
 	public RedisFilterExpressionConverter(List<MetadataField> metadataFields) {
 		this.metadataFields = metadataFields.stream()
@@ -168,11 +168,11 @@ private NumericBoundary exclusive(Value value) {
 		return new NumericBoundary(value.value(), true);
 	}
 
-	static record Numeric(NumericBoundary lower, NumericBoundary upper) {
+	record Numeric(NumericBoundary lower, NumericBoundary upper) {
 
 	}
 
-	static record NumericBoundary(Object value, boolean exclusive) {
+	record NumericBoundary(Object value, boolean exclusive) {
 
 		private static final String INFINITY = "inf";