Skip to content

chore(deps) : Upgrade to spring 4 #2079

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

chore(deps) : Upgrade to spring 4 #2079

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
4ff3bc7
chore(deps) : Upgrade to spring 4
rajadilipkolli Oct 2, 2025
eb6b16f
attempt to fix issue with traceId
rajadilipkolli Oct 2, 2025
5d09974
fix : json data
rajadilipkolli Oct 2, 2025
66efd55
Merge branch 'main' into strategy
rajadilipkolli Oct 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/boot-strategy-plugin.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ jobs:
- uses: graalvm/setup-graalvm@v1
with:
distribution: 'graalvm'
java-version: 21
java-version: 25
cache: "maven"

- name: Printing versions
Expand Down
19 changes: 14 additions & 5 deletions boot-strategy-plugin/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,15 @@
<parent>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-parent</artifactId>
<version>3.5.6</version>
<version>4.0.0-M3</version>
<relativePath/> <!-- lookup parent from repository -->
</parent>
<groupId>com.example.strategy.plugin</groupId>
<artifactId>boot-strategy-plugin</artifactId>
<version>0.0.1-SNAPSHOT</version>

<properties>
<java.version>21</java.version>
<java.version>25</java.version>
<spring.plugin.version>3.0.0</spring.plugin.version>
<springdoc-openapi.version>2.8.13</springdoc-openapi.version>
<spotless.version>3.0.0</spotless.version>
Expand All @@ -36,15 +36,15 @@
</dependency>
<dependency>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-aop</artifactId>
<artifactId>spring-boot-starter-aspectj</artifactId>
</dependency>
<dependency>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-validation</artifactId>
</dependency>
<dependency>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-web</artifactId>
<artifactId>spring-boot-starter-webmvc</artifactId>
</dependency>

<!-- swagger-->
Expand All @@ -53,6 +53,11 @@
<artifactId>springdoc-openapi-starter-webmvc-ui</artifactId>
<version>${springdoc-openapi.version}</version>
</dependency>
<!-- to be removed when SB4 is supported -->
<dependency>
<groupId>org.jetbrains.kotlin</groupId>
<artifactId>kotlin-reflect</artifactId>
</dependency>

<!-- For Tracing -->
<!-- OpenTelemetry version -->
Expand All @@ -61,6 +66,10 @@
<artifactId>micrometer-tracing-bridge-otel</artifactId>
</dependency>
<!-- For Latency Visualization -->
<dependency>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-opentelemetry</artifactId>
</dependency>
<dependency>
<groupId>io.opentelemetry</groupId>
<artifactId>opentelemetry-exporter-otlp</artifactId>
Expand Down Expand Up @@ -120,7 +129,7 @@
<configuration>
<java>
<googleJavaFormat>
<version>1.27.0</version>
<version>1.28.0</version>
<style>AOSP</style>
</googleJavaFormat>
<importOrder />
Expand Down
2 changes: 1 addition & 1 deletion ...lugin/src/test/java/com/example/plugin/strategyplugin/StrategyPluginApplicationTests.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@

import com.example.plugin.strategyplugin.common.AbstractIntegrationTest;
import com.example.plugin.strategyplugin.domain.GenericDTO;
import com.fasterxml.jackson.databind.JsonNode;
import java.util.concurrent.TimeUnit;
import org.junit.jupiter.api.Test;
import org.slf4j.Logger;
Expand All @@ -14,6 +13,7 @@
import org.springframework.http.MediaType;
import org.springframework.http.ProblemDetail;
import org.springframework.web.client.RestClient;
import tools.jackson.databind.JsonNode;
Copy link
Contributor

@coderabbitai coderabbitai bot Oct 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion | 🟠 Major

🧩 Analysis chain

LGTM! Jackson 3.x package migration.

The import has been correctly updated from com.fasterxml.jackson.databind.JsonNode to tools.jackson.databind.JsonNode to align with Jackson 3.x, which is included in Spring Boot 4.0.0-M3.

Verify that all other Jackson imports across the codebase have been migrated to the new tools.jackson package:

🏁 Script executed:

#!/bin/bash
# Description: Find any remaining imports from the old com.fasterxml.jackson package.

# Search for old Jackson imports that should be migrated
rg -n "import com\.fasterxml\.jackson\." --type=java

Length of output: 16871

Migrate all remaining Jackson imports to tools.jackson
Detected numerous import com.fasterxml.jackson.* statements across tests and main sources (e.g., scheduler, r2dbc, jpa, graphql modules). Update all to tools.jackson.* to complete the Jackson 3.x migration.

🤖 Prompt for AI Agents 
In
boot-strategy-plugin/src/test/java/com/example/plugin/strategyplugin/StrategyPluginApplicationTests.java
around line 16, replace the existing Jackson import with the tools.jackson
package (e.g., change import com.fasterxml.jackson.* to import tools.jackson.*);
update this file's import to tools.jackson.databind.JsonNode and then search the
repository for any remaining com.fasterxml.jackson imports and replace them with
the corresponding tools.jackson.* equivalents, re-run the build/tests to ensure
no remaining compilation errors and fix any updated package/API differences.


class StrategyPluginApplicationTests extends AbstractIntegrationTest {

Expand Down