Skip to content

JacksonJavaxToJakarta keeps newer Jackson versions and shared version properties when relocating artifacts - #1275

Merged
jkschneider merged 2 commits into
jakarta/jetty-12-baselinefrom
jakarta/jackson-version-preserved
Oct 3, 2026
Merged

jkschneider merged 2 commits into
jakarta/jetty-12-baselinefrom
jakarta/jackson-version-preserved

Conversation

@jkschneider

Copy link
Copy Markdown
Member

Each relocation in JacksonJavaxToJakarta set the new artifact to 2.13.x, which lowered a build already on 2.16 to 2.13.5, and for a provider versioned by a shared ${jackson.version} it wrote a literal 2.13.5 while jackson-databind kept the old property value. The recipe now first raises the old jackson-module-jaxb-annotations, jackson-jaxrs-* and jackson-datatype-jsr353 dependencies to at least 2.13.x with UpgradeDependencyVersion, which never downgrades and updates a shared property in place, and then relocates them without a version, so the new artifact keeps the version reference it had. The second commit adds a test for the shared property at 2.2.2, which is raised to 2.13.5 for the provider and jackson-databind together.

Found by a team of coding agents reviewing a Moderne run of org.openrewrite.java.migrate.jakarta.JavaxMigrationToJakarta over the 31 Arquillian repositories (run 20261003120559-YQoma). The issues are jackson-version-downgrade in the run's io.moderne.RecipeIssues data table (1 row) and jackson-shared-version-property-split (1 row), and the fix was discussed and reviewed on the run's message board. A row from the table, as a generalized example:

Changing the Jackson provider artifact downgrades a newer shared Jackson version. (arquillian/arquillian-cube, pom.xml, recipe org.openrewrite.java.dependencies.ChangeDependency)

Before:

com.fasterxml.jackson.jaxrs:jackson-jaxrs-json-provider:2.16.0

What the recipe produced:

com.fasterxml.jackson.jakarta.rs:jackson-jakarta-rs-json-provider:2.13.5

Expected:

com.fasterxml.jackson.jakarta.rs:jackson-jakarta-rs-json-provider:2.16.0

Stacked on #1274 (12 of 19); merge that one first.

@jkschneider
jkschneider force-pushed the jakarta/jackson-version-preserved branch from f2e5287 to 423c787 Compare October 3, 2026 22:26
@jkschneider
jkschneider added this pull request to stack #1283 October 3, 2026 22:39
@jkschneider
jkschneider merged commit 689d569 into jakarta/reflective-type-names Oct 3, 2026
@jkschneider
jkschneider deleted the jakarta/jackson-version-preserved branch October 3, 2026 22:41
jkschneider added a commit that referenced this pull request Oct 4, 2026
#1271)

* javax-reflective-names-retained: migrate runtime Jakarta type names

* maven-coordinate-literal-retained: migrate validation artifact lookups (#1272)

* weld-javax-provider-retained: align Weld core with Jakarta CDI (#1273)

* `JettyUpgradeEE9` also upgrades the other Jetty dependencies, the servlet packages and the Java version (#1274)

* jetty-ee9-java-baseline: align Jetty dependencies packages and Java baseline

* jetty-ee9-java-baseline: leave standalone Jetty utility dependencies unchanged

* jetty-ee9-java-baseline: require a direct dependency before upgrading Java

* `JacksonJavaxToJakarta` keeps newer Jackson versions and shared version properties when relocating artifacts (#1275)

* jackson-version-downgrade: preserve newer versions and shared properties

* jackson-shared-version-property-split: cover older shared Jackson versions

* The Jakarta JAXB migration keeps the `javax.xml.bind` API for Arquillian Recorder 1.x binaries (#1276)

* recorder-javax-jaxb-api-removed: retain the API needed by Recorder binaries

* recorder-javax-jaxb-api-removed: limit retention to external binaries after migration

* hibernate-validator-javax-provider-retained: migrate legacy validation providers (#1277)

* legacy-javaee-api-coordinate-retained: migrate legacy annotation EL and EJB aliases (#1278)

* The Jakarta migration keeps a provided API provided instead of adding it again in compile scope (#1279)

* cdi-provided-scope-promoted: preserve provided CDI API scope

* cdi-provided-scope-promoted: distinguish provided declarations from runtime APIs

* provided-api-scope-overwritten: preserve provided Mail API scope

* provided-api-scope-overwritten: preserve container API scopes across Jakarta migrations

* The Jackson Jakarta migration adds the `defaultView` parameter to `ProviderBase._configForWriting` overrides (#1280)

* jackson-provider-override-signature: forward the default view parameter

* jackson-provider-override-signature: update callers of migrated overrides

* invocation-context-get-constructor-missing: complete legacy decorators (#1281)

* jakarta-api-duplicates-platform: respect provided platform APIs (#1282)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant