core: migrate to JDK17 syntax

Author: Martin7-1

langchain4j-community-core/src/main/java/dev/langchain4j/community/rag/content/aggregator/strategy/GenerateEmbeddings.java

@@ -57,8 +57,8 @@ public List<EmbeddingMatch<Content>> processContents(
 
     private String getEmbeddingId(Content content) {
         Object embeddingId = content.metadata().get(ContentMetadata.EMBEDDING_ID);
-        if (embeddingId instanceof String && !((String) embeddingId).isBlank()) {
-            return (String) embeddingId;
+        if (embeddingId instanceof String string && !string.isBlank()) {
+            return string;
         }
         return TEMP_EMBEDDING_ID_PREFIX + Math.abs(content.hashCode());
     }

langchain4j-community-core/src/main/java/dev/langchain4j/community/rag/content/aggregator/strategy/HybridEmbeddings.java

@@ -118,8 +118,8 @@ private List<EmbeddingMatch<Content>> generateMissingEmbeddings(
 
     private String getEmbeddingId(Content content) {
         Object embeddingId = content.metadata().get(ContentMetadata.EMBEDDING_ID);
-        if (embeddingId instanceof String && !((String) embeddingId).isBlank()) {
-            return (String) embeddingId;
+        if (embeddingId instanceof String string && !string.isBlank()) {
+            return string;
         }
         return TEMP_EMBEDDING_ID_PREFIX + Math.abs(content.hashCode());
     }

langchain4j-community-core/src/main/java/dev/langchain4j/community/rag/content/aggregator/strategy/UseExistingEmbeddings.java

@@ -80,8 +80,8 @@ private Embedding extractEmbeddingFromContent(Content content) {
 
     private String getEmbeddingId(Content content) {
         Object embeddingId = content.metadata().get(ContentMetadata.EMBEDDING_ID);
-        if (embeddingId instanceof String && !((String) embeddingId).isBlank()) {
-            return (String) embeddingId;
+        if (embeddingId instanceof String string && !string.isBlank()) {
+            return string;
         }
         return TEMP_EMBEDDING_ID_PREFIX + Math.abs(content.hashCode());
     }

langchain4j-community-core/src/test/java/dev/langchain4j/community/chain/RetrievalQAChainTest.java

@@ -82,7 +82,13 @@ void should_inject_retrieved_segments_when_content_available() {
 
         verify(chatModel).chat(messagesCaptor.capture());
         String expectedUserMessage =
-                "query\n" + "\n" + "Answer using the following information:\n" + "Segment 1\n" + "\n" + "Segment 2";
+                """
+                query
+                
+                Answer using the following information:
+                Segment 1
+                
+                Segment 2""";
         assertThat(messagesCaptor.getValue()).isEqualTo(expectedUserMessage);
     }
 

pom.xml

@@ -291,6 +291,7 @@
                             <activeRecipes>
                                 <!--<recipe>org.openrewrite.java.OrderImports</recipe>-->
                                 <recipe>org.openrewrite.java.testing.assertj.Assertj</recipe>
+                                <recipe>org.openrewrite.java.migrate.UpgradeToJava17</recipe>
                             </activeRecipes>
                             <failOnDryRunResults>true</failOnDryRunResults>
                         </configuration>
@@ -300,6 +301,11 @@
                                 <artifactId>rewrite-testing-frameworks</artifactId>
                                 <version>3.18.0</version>
                             </dependency>
+                            <dependency>
+                                <groupId>org.openrewrite.recipe</groupId>
+                                <artifactId>rewrite-migrate-java</artifactId>
+                                <version>3.18.0</version>
+                            </dependency>
                         </dependencies>
                     </plugin>
                 </plugins>