* mongo: update driver to 5.5.1

Signed-off-by: neo <1100909+neowu@users.noreply.github.com>

Author: neowu

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 * db: support postgres enum type
 * http_client: update okhttp to 5.1.0
+* mongo: update driver to 5.5.1
 
 ### 9.2.2 (5/21/2025 - 6/26/2025)
 

build.gradle.kts

@@ -20,8 +20,8 @@ subprojects {
 
 val elasticVersion = "8.18.1"
 val jacksonVersion = "2.18.4"
-val junitVersion = "5.12.2"
-val mockitoVersion = "5.17.0"
+val junitVersion = "5.13.4"
+val mockitoVersion = "5.18.0"
 val assertjVersion = "3.27.3"
 
 project("core-ng-api") {
@@ -42,7 +42,7 @@ project("core-ng") {
             exclude("org.xerial.snappy")
         }
         compileOnly("org.jboss.logging:jboss-logging-annotations:2.2.1.Final")
-        compileOnly("com.github.spotbugs:spotbugs-annotations:4.8.3")
+        compileOnly("com.github.spotbugs:spotbugs-annotations:4.9.3")
         testImplementation("org.junit.jupiter:junit-jupiter-api:${junitVersion}")
         testImplementation("org.mockito:mockito-junit-jupiter:${mockitoVersion}")
         testImplementation("org.assertj:assertj-core:${assertjVersion}")
@@ -69,7 +69,7 @@ project("core-ng-mongo") {
     apply(plugin = "lib")
     dependencies {
         api(project(":core-ng"))
-        api("org.mongodb:mongodb-driver-sync:5.2.1")
+        api("org.mongodb:mongodb-driver-sync:5.5.1")
         testImplementation(project(":core-ng-test"))
     }
 }
@@ -79,7 +79,7 @@ project("core-ng-mongo-test") {
     dependencies {
         implementation(project(":core-ng-test"))
         implementation(project(":core-ng-mongo"))
-        implementation("de.bwaldvogel:mongo-java-server:1.46.0")
+        implementation("de.bwaldvogel:mongo-java-server:1.47.0")
     }
 }
 

buildSrc/src/main/check/checkstyle.xml

@@ -70,9 +70,11 @@
         </module>
         <module name="OverloadMethodsDeclarationOrder"/>    <!--Checks that overload methods are grouped together.-->
         <module name="ParameterAssignment"/>
+        <module name="PatternVariableAssignment"/>          <!--Checks for assignment of pattern variables.-->
         <module name="SimplifyBooleanExpression"/>          <!--Checks for overly complicated boolean expressions.-->
         <module name="SimplifyBooleanReturn"/>              <!--Checks for overly complicated boolean return statements.-->
         <module name="StringLiteralEquality"/>
+        <module name="UnnecessaryNullCheckWithInstanceOf"/> <!--Checks for redundant null checks with the instanceof operator.-->
         <module name="UnnecessaryParentheses"/>             <!--Checks if unnecessary parentheses are used in a statement or expression.-->
         <module name="UnnecessarySemicolonAfterOuterTypeDeclaration"/>
         <module name="UnnecessarySemicolonAfterTypeMemberDeclaration"/>

buildSrc/src/main/check/pmd.xml

@@ -29,6 +29,7 @@
     <rule ref="category/java/bestpractices.xml/SimplifiableTestAssertion"/>
     <rule ref="category/java/bestpractices.xml/SystemPrintln"/>
     <rule ref="category/java/bestpractices.xml/UnnecessaryVarargsArrayCreation"/>
+    <rule ref="category/java/bestpractices.xml/UnnecessaryWarningSuppression"/>
     <rule ref="category/java/bestpractices.xml/UnusedAssignment"/>
     <rule ref="category/java/bestpractices.xml/UnusedFormalParameter"/>
     <rule ref="category/java/bestpractices.xml/UnusedLocalVariable"/>

buildSrc/src/main/kotlin/check.gradle.kts

@@ -19,7 +19,7 @@ subprojects {
 
     checkstyle {
         dependencies {
-            checkstyle("com.puppycrawl.tools:checkstyle:10.23.0")
+            checkstyle("com.puppycrawl.tools:checkstyle:10.26.1")
             checkstyle("com.github.sevntu-checkstyle:sevntu-checks:1.44.1")
         }
 
@@ -47,7 +47,7 @@ subprojects {
     pmd {
         ruleSets = listOf()
         ruleSetFiles = rootProject.files("buildSrc/src/main/check/pmd.xml")
-        toolVersion = "7.13.0"
+        toolVersion = "7.16.0"
         isConsoleOutput = true
 
         tasks.withType<Pmd> {
@@ -57,7 +57,7 @@ subprojects {
 
     spotbugs {
         dependencies {
-            spotbugsPlugins("com.mebigfatguy.sb-contrib:sb-contrib:7.6.9")
+            spotbugsPlugins("com.mebigfatguy.sb-contrib:sb-contrib:7.6.12")
         }
 
         toolVersion = "4.9.3"

core-ng/src/main/java/core/framework/internal/web/HTTPErrorHandler.java

@@ -78,9 +78,9 @@ Object errorResponse(Throwable e, String userAgent, String actionId) {
             response.id = actionId;
             response.message = e.getMessage();
             response.stackTrace = Exceptions.stackTrace(e);
-            if (e instanceof ErrorCode errorCode) {
-                response.errorCode = errorCode.errorCode();
-                response.severity = errorCode.severity().name();
+            if (e instanceof ErrorCode code) {
+                response.errorCode = code.errorCode();
+                response.severity = code.severity().name();
             } else {
                 // internally LogManager logs exception class name as error code if not impl ErrorCode,
                 // here it uses INTERNAL_ERROR as it's border of internal, we don't want to expose internal details (exception class) to external which is part of api contract

core-ng/src/main/java/core/framework/internal/web/service/ErrorResponse.java

@@ -11,8 +11,8 @@ public static ErrorResponse errorResponse(Throwable e, String actionId) {
         var response = new ErrorResponse();
         response.id = actionId;
         response.message = e.getMessage();
-        if (e instanceof ErrorCode errorCode) {
-            response.errorCode = errorCode.errorCode();
+        if (e instanceof ErrorCode code) {
+            response.errorCode = code.errorCode();
         } else {
             response.errorCode = "INTERNAL_ERROR";
         }