From 57259a8455e668d0f41680b16eea518202f5f1c0 Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 20 May 2026 05:39:06 +0000 Subject: [PATCH 1/2] configure revapi to run as part of the audits job - Move revapi out of the code-audits profile and into a dedicated audits profile in core/pom.xml, activated by -Daudits (matching audits.yml) - Remove the none lifecycle-suppression hack from root pluginManagement; let the profile-scoped execution control when it runs - Fix the config file element name to per the revapi-maven-plugin docs - Rewrite config/revapi.json from scratch in the modern array format, keeping only revapi.filter with targeted package exclusions https://claude.ai/code/session_01GkmqCe8N2JvJB7tqCD3w3q --- config/revapi.json | 54 ++++------------------------------------------ core/pom.xml | 20 +++++++++++++++++ pom.xml | 12 +---------- 3 files changed, 25 insertions(+), 61 deletions(-) diff --git a/config/revapi.json b/config/revapi.json index cddd4fc1ccc..33d6a4037c9 100644 --- a/config/revapi.json +++ b/config/revapi.json @@ -1,23 +1,4 @@ [ - { - "extension": "revapi.versions", - "configuration": { - "enabled": true, - "semantic0": true, - "strictSemver": true, - "versionIncreaseAllows": { - "major": { - "severity": "BREAKING" - }, - "minor": { - "severity": "NON_BREAKING" - }, - "patch": { - "severity": "EQUIVALENT" - } - } - } - }, { "extension": "revapi.filter", "configuration": { @@ -31,47 +12,20 @@ "matcher": "java", "match": "@dev.morphia.annotations.internal.MorphiaExperimental ^*;" }, - { - "matcher": "java", - "match": "class dev.morphia.sofia.Sofia {}" - }, - { - "matcher": "java", - "match": "type ^* implements com.mongodb.session.ClientSession {}" - }, - { - "matcher": "java-package", - "match": "/.*codec.*/" - }, - { - "matcher": "java-package", - "match": "/com.mongodb.*/" - }, { "matcher": "java-package", - "match": "/org.bson.*/" + "match": "/.*\\.internal(\\..*)?/" }, { "matcher": "java-package", - "match": "/.*internal.*/" + "match": "/.*\\.experimental(\\..*)?/" }, { "matcher": "java-package", - "match": "/.*experimental.*/" - }, - { - "matcher": "java-package", - "match": "dev.morphia.annotations.internal" + "match": "/.*\\.codec(\\..*)?/" } ] } } - }, - { - "extension": "revapi.differences", - "configuration": { - "ignore": true, - "differences": [] - } } -] \ No newline at end of file +] diff --git a/core/pom.xml b/core/pom.xml index c9f9312b04b..dcbe030961f 100644 --- a/core/pom.xml +++ b/core/pom.xml @@ -249,9 +249,29 @@ com.github.spotbugs spotbugs-maven-plugin + + + + + audits + + + audits + + + + org.revapi revapi-maven-plugin + + + api-check + + check + + + diff --git a/pom.xml b/pom.xml index 75e4f2ab105..d5902cf16dd 100644 --- a/pom.xml +++ b/pom.xml @@ -109,19 +109,9 @@ ${revapi.java.version} - - - revapi - - check - - none - - - true - ${maven.multiModuleProjectDirectory}/config/revapi.json + ${maven.multiModuleProjectDirectory}/config/revapi.json From f81e97864b1d9b993e514eb88f2e65ca3ccf6894 Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 20 May 2026 05:42:56 +0000 Subject: [PATCH 2/2] fix codec package filter to match both codec and codecs The previous regex /.*\.codec(\..*)?/ missed plural forms like dev.morphia.aggregation.codecs and dev.morphia.aggregation.codecs.stages. Use /.*codec.*/ to cover all codec-related packages regardless of plurality. https://claude.ai/code/session_01GkmqCe8N2JvJB7tqCD3w3q --- config/revapi.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/revapi.json b/config/revapi.json index 33d6a4037c9..e64c399a615 100644 --- a/config/revapi.json +++ b/config/revapi.json @@ -22,7 +22,7 @@ }, { "matcher": "java-package", - "match": "/.*\\.codec(\\..*)?/" + "match": "/.*codec.*/" } ] }