Skip to content

Comments

Local model#68

Merged
alexheifetz merged 3 commits intomainfrom
local-model
Sep 15, 2025
Merged

Local model#68
alexheifetz merged 3 commits intomainfrom
local-model

Conversation

@alexheifetz
Copy link
Contributor

This pull request improves how model dependencies are managed and configures property scanning for Spring Boot applications in both the Java and Kotlin example projects. It makes Ollama the default always-included model backend, while OpenAI and Anthropic dependencies are now activated only if the corresponding API keys are set. Additionally, it updates the Spring Boot configuration to use @ConfigurationPropertiesScan for property binding. The environment check script is also updated to better handle missing API keys.

Dependency management improvements:

  • Ollama is now always included as a dependency for local model hosting in both examples-java/pom.xml and examples-kotlin/pom.xml. OpenAI and Anthropic dependencies are removed from the default dependencies and instead added under Maven profiles that activate only if the respective API keys are present in the environment (OPENAI_API_KEY, ANTHROPIC_API_KEY). [1] [2] [3] [4]

Spring Boot configuration enhancements:

  • Replaces @EnableConfigurationProperties with @ConfigurationPropertiesScan (scanning the com.embabel.example package) in all Java and Kotlin example applications to improve property binding and configuration. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]

Other improvements:

  • Updates the environment check batch script (scripts/support/check_env.bat) to remove API key instructions and clarify that Ollama will be used if both API keys are missing, with a reminder to check Ollama installation and set the default model. [1] [2]
  • Fixes a class naming issue in the Kotlin MCP server application, renaming JavaAgentMcpServerApplication to KotlinAgentMcpServerApplication. [1] [2]

@alexheifetz alexheifetz merged commit 2517273 into main Sep 15, 2025
2 checks passed
@alexheifetz alexheifetz deleted the local-model branch September 15, 2025 08:25
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