Skip to content

Conversation

@kpavlov
Copy link
Contributor

@kpavlov kpavlov commented Nov 14, 2025

Optimize dependencies and update project setup

Motivation and Context

Reduce the library’s transitive footprint by making JDBC drivers and the connection pool compileOnly, so applications are no longer forced to pull in specific database drivers they may not use. It also stops the Spring Boot starter from implicitly choosing a Ktor client engine, requiring consumers to select the engine that best fits their environment and avoiding version and engine conflicts. The updated docs clarify these explicit dependency requirements, and the logging configuration helps keep test logs readable while still surfacing useful debug information.

  • Switched optional JDBC drivers to compileOnly for reduced transitive dependencies.
  • Added logback.xml to configure logging levels and prevent excessive logs.
  • Exclude Ktor client engine from spring-boot transitive dependencies and updated documentation

Breaking Changes

  1. JDBC drivers dependencies are no longer included and should be explicitly configured by users.
  2. Ktor client engine dependency should be configured by user when using Koog Spring Boot starter

Type of the changes

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Dependency update
  • Tests improvement
  • Refactoring

Checklist

  • The pull request has a description of the proposed change
  • I read the Contributing Guidelines before opening the pull request
  • The pull request uses develop as the base branch
  • Tests for the changes have been added
  • All new and existing tests passed
Additional steps for pull requests adding a new feature
  • An issue describing the proposed change exists
  • The pull request includes a link to the issue
  • The change was discussed and approved in the issue
  • Docs have been added / updated

@kpavlov kpavlov added the dependencies Update dependencies label Nov 14, 2025
@kpavlov kpavlov force-pushed the kpavlov/minimize-dependencies branch from 9c42606 to ef085d8 Compare November 14, 2025 19:34
@kpavlov kpavlov changed the title Optimize dependencies and update project setup Optimize dependencies Nov 14, 2025
@kpavlov kpavlov marked this pull request as ready for review November 14, 2025 19:43
@github-actions
Copy link

github-actions bot commented Nov 14, 2025

Qodana for JVM

1190 new problems were found

Inspection name Severity Problems
Check Kotlin and Java source code coverage 🔶 Warning 1179
Missing KDoc for public API declaration 🔶 Warning 11
@@ Code coverage @@
+ 71% total lines covered
16418 lines analyzed, 11797 lines covered
# Calculated according to the filters of your coverage tool

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

@kpavlov kpavlov force-pushed the kpavlov/minimize-dependencies branch 2 times, most recently from 8fab54d to d371fc7 Compare November 18, 2025 09:15
Reduces the library’s transitive footprint by making JDBC drivers and the connection pool `compileOnly`, so applications are no longer forced to pull in specific database drivers they may not use. It also stops the Spring Boot starter from implicitly choosing a Ktor client engine, requiring consumers to select the engine that best fits their environment and avoiding version and engine conflicts. The updated docs clarify these explicit dependency requirements, and the logging configuration helps keep test logs readable while still surfacing useful debug information.

- Switched optional JDBC drivers to `compileOnly` for reduced transitive dependencies.
- Added `logback.xml` to configure logging levels and prevent excessive logs.
- Exclude Ktor client engine from spring-boot transitive dependencies and updated documentation
@kpavlov kpavlov requested a review from EugeneTheDev December 10, 2025 13:29
@kpavlov kpavlov force-pushed the kpavlov/minimize-dependencies branch from d371fc7 to 5901a16 Compare December 10, 2025 13:30
@kpavlov kpavlov merged commit eb7dabb into develop Dec 10, 2025
12 checks passed
@kpavlov kpavlov deleted the kpavlov/minimize-dependencies branch December 10, 2025 14:34
@github-actions
Copy link

Qodana for JVM

1188 new problems were found

Inspection name Severity Problems
Check Kotlin and Java source code coverage 🔶 Warning 1168
Missing KDoc for public API declaration 🔶 Warning 20
@@ Code coverage @@
+ 72% total lines covered
17161 lines analyzed, 12496 lines covered
# Calculated according to the filters of your coverage tool

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

vova-jb pushed a commit that referenced this pull request Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Update dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants