7.17.0

[tkvangorder]

New Java Type Cache

One of the strengths of OpenRewrite is that its tree elements include semantic type information. In the case of the Java parser, type attribution is achieved by mapping type information from the Javac compiler into Rewrite's Java-type system. In previous versions of Rewrite, the framework used the flyweight pattern to cache the type information and promote referential de-duplication. The flyweights prevented memory exhaustion and reduced the size of serialized ASTs. An additional feature of Rewrite's Java-type system was that it would artificially break cycles by introducing a Cyclic reference.

As the library has evolved (and we have started collecting more type information), it has exposed limitations and gaps within the legacy type system. This release introduces a new Java Type caching system that is better suited to large-scale, automated refactoring. Most of the changes are "behind-the-scenes" and lead to a much more accurate type-attribution system.

Other Enhancements

• New Recipe: Add new properties to a properties file. Recipe for adding new properties in properties files. #1152
• New Recipe: Change Maven plugin configuration Add Recipes for manipulating maven plugin configuration, executions, and dependencies #1190
• New Recipe: Change Maven packaging Add Recipe to change maven project packaging #1208 Visitors and Recipes for 7.11.0 #1134
• New Recipe: Add dependencies to maven plugins Create an AddPluginDependency Maven recipe for adding dependencies to maven plugins #1214 Visitors and Recipes for 7.11.0 #1134
• New Recipe: Use Map notation for Gradle dependency declarations 2802d60
• New Recipe: Removing interfaces from class declarations Add RemoveImplements recipe for removing interfaces from class declarations #1241
• Utilities for normalizing names between kebab-case, snake-case, camel-case, etc Name case convention utilities #1188
• Add support to ImportLayoutStyle for specific packages to fold. Add support to ImportLayoutStyle for specific packages to fold. #1231 Added serialization and deserialization of sub packages option to ImportLayoutStyle #1206
• Add support to parse DTD Markup in XML. Add support to parse DTD Markup in XML. #1226
• Refactored the Java cleanup recipes Organize cleanup recipes #1194
• Build is now using Gradle enterprise
• GenerateGetter does not work for primitives GenerateGetter does not work for primitives #1301

Fixes

• Git provenance is optional and may be null when source files are not in a Git repository. nullable git provenance #1148
• Do not left-shift YAML prefix comments following the previous element on the same line. Do not left-shift yaml prefix comments following previous element on the same line #1149 Error processing yaml comments #1142
• XmlParser throws NPE when parsing a DTC without an externalId XmlParser throws NPE when parsing a DTC without an externalId #1150
• JsonParserVisitor parses longs with parseInt JsonParserVisitor parses longs with parseInt. #1145
• Scope which types of SourceFile are accepted by visitors. Scope which types of SourceFile are accepted by visitors #1151
• ChangePackage.updateType is never called ChangePackage updateType is never called #1140
• Mismatched parenthesis in MissingOverrideAnnotation Mismatched parenthesis in MissingOverrideAnnotation #1153
• AddImport should not add java.lang to classes including classes in the default package AddImport should not add java.lang to classes including classes in the default package #1158 AddImports adds java.lang.* imports within default packages #1156
• Add Anchor and Alias models to YAML, and alias support for Yaml.Scalar values. Add Anchor and Alias models to Yaml, and alias support for Yaml.Scalar values. #1155
• Space between annotation arguments after the comma is empty Space between annotation arguments after comma is empty #1162
• Fix import folding collisions in AddImport 7007421
• Fix JavaTemplate method argument substitution and Java8 type mapping cad4aed 9ab35a9
• UpgradeParentPom may downgrade versions UpgradeParentPom may downgrade versions #1107
• If a Maven visitor successfully changes a non-Maven Xml.Document, it is transformed into a Maven with an empty model If a Maven visitor successfully changes a non-Maven Xml.Document, it is transformed it into a a Maven with an empty model #1167
• Upgrade dependency adds an overridden version property by Upgrade dependency adds an overridden version property #1170
• Fix TypeMapping.getClasspathElement() so that it also works with Windows paths 5cdd6e0
• Several fixes to provenance information in Maven 413ded9 8a0264d abc3978
• JavaVisitor.visitFieldAccess() does not allow a FieldAccess to be replaced by any other AST element JavaVisitor.visitFieldAccess() does not allow a FieldAccess to be replaced by any other AST element #1174
• Fix in ChangeMethodTargetToStaticVisitor to remove potential namespace conflicts. 2be7989
• Fixes for enum types in BlankLinesVisitor fefdc00 35bb53f
• XML AutoFormat always formats maven poms with 4 space indentation Xml AutoFormat always formats maven poms with 4 space indentation #1186
• Enhance Maven AddPlugin to correctly format any executions/configuration/dependencies Enhance Maven AddPlugin to correctly format any executions/configuration/dependencies #1187
• YamlParser throws EmptyStackException YamlParser throws EmptyStackException #1180
• Expand LatestRelease to support timestamped suffixes 5503aac
• Parse empty Java source file Parse empty Java source file #1192
• Make sure to use a continuation indent on a lambda when ANY method argument has a newline. Formatting issue discovered with src/main/java/org/openrewrite/concourse/ChangeValue.java #1173
• FindProperties in rewrite-properties missing visitMarkers FindProperties in rewrite-properties missing visitMarkers #1195
• FindProperties should have a marker on the value, same as YAML and other search results FindProperties should have marker on the value, same as yaml and other search results #1196
• Relaxed binding on YAML and properties Relaxed binding on yaml and properties #1182 problem when deleting certain properties Kebab Case vs Camel Case #1168
• Do not remove blocks that only contain comments in EmptyBlockVisitor ef074f5
• NoValueOfOnStringType removes necessary call to String#valueOf in string concatenation NoValueOfOnStringType removes necessary call to String#valueOf in string concatenation #1200
• MethodNameCasing should not rename overridden methods by MethodNameCasing should not rename overridden methods #1217
• Autodetect counts prefixes with neither tabs nor spaces as spaces Autodetect counts prefixes with neither tabs or spaces as spaces #1221
• UnnecessaryExplicitTypeArguments can lead to uncompilable code due to introduced ambiguity UnnecessaryExplicitTypeArguments can lead to uncompilable code due to introduced ambiguity #1211
• MinimumSwitchCases should prepend enum class to values or statically import them MinimumSwitchCases should prepend enum class to values or statically import them #1212
• XML AutoFormat under-indents tag values Xml AutoFormat under-indents tag values #1189
• NestedEnumsAreNotStatic reformats whole enum NestedEnumsAreNotStatic reformats whole enum #1222
• EmptyBlock recipe should not add throw RuntimeException. EmptyBlock recipe should not add throw RuntimeException #1210
• SpacesVisitor can throw an NPE when operating on Groovy Closures which, unlike Java, can have empty parameters list SpacesVisitor can throw an NPE when operating on Groovy Closures which, unlike Java, can have empty parameters list #1224
• IndexOfChecksShouldUseAStartPosition should ignore cases where the start position is undefined and the index expected to be greater than 0 IndexOfChecksShouldUseAStartPosition should ignore cases where the start position is undefined and the index expected to be greater than 0 #1225
• UnnecessaryThrows removes throws from the parent class and breaks subclasses overriding methods UnnecessaryThrows removes throws from parent class and breaks subclasses overriding methods #1207
• OrderPomElements removes valid tags while reordering GAVs. OrderPomElements removes valid tags while reordering gavs. #1228
• Preserve new line layout of pom's reordered by OrderPomElements. Preserve new line layout of pom's reordered by OrderPomElements. #1230
• MethodMatcher should acknowledge matchOverrides MethodMatcher should acknowledge matchOverrides #1219
• Add JavaTemplate coordinate for replacing just the arguments of an annotation Add JavaTemplate coordinate for replacing just the arguments of an annotation #1234
• Refactor JavadocVisitor initialization to make it easier to customize Refactor JavadocVisitor initialization to make it easier to customize #1233
• JavaTemplate throws Index 0 out of bounds for length 0 JavaTemplate throws Index 0 out of bounds for length 0 #1198
• WrappingAndBraces for Gradle creates incorrect spacing WrappingAndBraces for Gradle creates incorrect spacing #1237
• Performance fixes in 'MethodParamPad` fd469b4 77af643
• ExplicitLambdaArgumentTypes should add import if necessary ExplicitLambdaArgumentTypes should add import if necessary #1242
• Avoid Class.forName for primitives in MethodMatcher, better toString on JavaType
• ChangeMethodAccessLevel should pass on matchOverrides ChangeMethodAccessLevel should pass on matchOverrides #1240
• <?m2e ...?> tag results extraneous input warning when parsing pom <?m2e ...?> tag results extraneous input warning when parsing pom #1243
• Converge ChangeDependencyVersion along with the other ChangeDependency* recipes. a0f37c4
• ExplicitLamdaArgumentTypes inserts null as type ExplicitLamdaArgumentTypes inserts null as type #1246
• The diff on Result should account for relative paths b9f2c25
• ExplicitInitialization removes initialization expression interface fields ExplicitInitialization removes initialization expression interface fields #1247
• Use the correct prefix if the value is a Sequence in Yaml ChangeValue 67ca7f5
• Handle exact version ranges in VersionRangeParser Handle exact version ranges in VersionRangeParser #1250
• RemoveUnusedImports is incorrectly removing an import for an annotation on an enum value. RemoveUnusedImports is incorrectly removing an import for an annotation on an enum value. #1251
• Update ChangePropertyKey to prevent moving entries to nonscalar nodes or those nested under a sequence Update ChangePropertyKey to prevent moving entries to nonscalar nodes or those nested under a sequence #1257
• Add isAcceptable override to JavaVisitor to fix ClassCastExceptions ad96bdf
• Fix type attribution on references in JavaDocs Fix type attribution on references in JavaDocs #1273
• The RocksdbMavenPomCache can lead to heap exhaustion. The RocksdbMavenPomCache can lead to heap exhaustion. #1284
• Update YAML IndentsVisitor to maintain correct sequence entry indent. Merge recipe with sequences bad indent #1275
• Refactor AddSerialVersionUidToSerializable to improve performance only checking ClassDeclaration Variables
• Fixes to Java templates and reduce heap pressure Fix to better re-use Java templates and reduce heap pressure #1294 JavaTemplateParser cache keys lead to heap pressure #1288
• Refactor build scripts to use toolchain instead of javac executable Refactor build scripts to use toolchain instead of javac executable #1299
• Merge recipe with sequences within map bad indent Merge recipe with sequences within map bad indent  #1292
• Rework MavenPomCache and introduce TTL Rework MavenPomCache and introduce TTL #1304
• When using ListUtils.flatMap to replace an item with a collection, subsequent items may be incorrect When using ListUtils.flatMap to replace an item with a collection, subsequent items may be incorrect  #1309
• Fix "too many files open" issues eb92f81 079e715
• MavenDownloader was exiting on first download failure 448094d

Full Changelog: v7.16.0...v7.17.0

---

This discussion was created from the release 7.17.0.