Update to NB-25

BUILD.md

@@ -121,6 +121,10 @@ This configuration will enable you to debug both the extension’s TypeScript co
    ```json
    "jdk.serverVmOptions": ["-J-Dnetbeans.logger.console=true"]
    ```
+4. For further debugging you can set Log Level to FINEST by appending following argument to the array:
+   ```json
+   "jdk.serverVmOptions": ["-J-Dnetbeans.logger.console=true", "-J-Dorg.netbeans.modules.java.lsp.server.lsptrace.level=FINEST"]
+   ```
 
 Both options will enable logging from the NetBeans server in the VS Code Output Channel.
 

build.xml

@@ -45,21 +45,7 @@
     <loadresource property="patch-files">
         <string>
             patches/6330.diff
-            patches/7491.diff
             patches/7610.diff
-            patches/7641.diff
-            patches/7654.diff
-            patches/7670.diff
-            patches/7699.diff
-            patches/7709.diff
-            patches/7722.diff
-            patches/7724.diff
-            patches/7733.diff
-            patches/7750.diff
-            patches/7910.diff
-            patches/7921.diff
-            patches/7923.diff
-            patches/7926.diff
             patches/8036-draft.diff
             patches/8038-draft.diff
             patches/mvn-sh.diff
@@ -69,7 +55,6 @@
             patches/remove-db.diff
             patches/nbjavac-not-required.diff
             patches/l10n-licence.diff
-            patches/no-security-manager-allow.diff
             patches/dev-dependency-licenses.diff
         </string>
         <filterchain>

nbcode/integration/src/org/netbeans/modules/nbcode/integration/LspApplyEditsImplementation.java

@@ -0,0 +1,32 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.netbeans.modules.nbcode.integration;
+
+import org.netbeans.modules.java.lsp.server.ui.AbstractApplyEditsImplementation;
+import org.netbeans.spi.lsp.ApplyEditsImplementation;
+import org.openide.util.lookup.ServiceProvider;
+
+/**
+ *
+ * @author sdedic
+ */
+@ServiceProvider(service = ApplyEditsImplementation.class, position = 10000)
+public class LspApplyEditsImplementation extends AbstractApplyEditsImplementation{
+
+}

nbcode/nbproject/platform.properties

@@ -394,7 +394,6 @@ disabled.modules=\
     org.netbeans.modules.j2ee.sun.dd,\
     org.netbeans.modules.j2ee.sun.ddui,\
     org.netbeans.modules.jakarta.transformer,\
-    org.netbeans.modules.jakarta.web.beans,\
     org.netbeans.modules.jakartaee8.api,\
     org.netbeans.modules.jakartaee8.platform,\
     org.netbeans.modules.javaee7.api,\

patches/6330.diff

@@ -1,5 +1,5 @@
 diff --git a/java/java.lsp.server/src/org/netbeans/modules/java/lsp/server/ConnectionSpec.java b/java/java.lsp.server/src/org/netbeans/modules/java/lsp/server/ConnectionSpec.java
-index 9ab389564c..9b7c044c81 100644
+index bb8a4e8183..b9662fe6cd 100644
 --- a/java/java.lsp.server/src/org/netbeans/modules/java/lsp/server/ConnectionSpec.java
 +++ b/java/java.lsp.server/src/org/netbeans/modules/java/lsp/server/ConnectionSpec.java
 @@ -25,7 +25,7 @@ import java.io.OutputStream;
@@ -86,11 +86,10 @@ index 9ab389564c..9b7c044c81 100644
                          } catch (IOException ex) {
                              if (isClosed(server)) {
                                  break;
-@@ -129,26 +152,54 @@ final class ConnectionSpec implements Closeable {
-                 }
+@@ -130,25 +153,53 @@ final class ConnectionSpec implements Closeable {
              };
              listeningThread.start();
--            out.write((prefix + " listening at port " + localPort).getBytes());
+             out.write((prefix + " listening at port " + localPort + "\n").getBytes());
 +            StringBuilder message = new StringBuilder();
 +            message.append(prefix).append(" listening at port ").append(localPort);
 +            if (hash) {
@@ -128,7 +127,6 @@ index 9ab389564c..9b7c044c81 100644
 +                    if (hashContent != null) {
 +                        for (char c : hashContent) {
 +                            byte b = (byte) in.read();
-+
 +                            if (b != c) {
 +                                IOException toThrow = new IOException("Hash validation failed!");
 +                                try {
@@ -146,15 +144,21 @@ index 9ab389564c..9b7c044c81 100644
                      connectionObject.getRunningFuture().get();
                  } catch (IOException | InterruptedException | ExecutionException ex) {
 diff --git a/java/java.lsp.server/test/unit/src/org/netbeans/modules/java/lsp/server/ConnectionSpecTest.java b/java/java.lsp.server/test/unit/src/org/netbeans/modules/java/lsp/server/ConnectionSpecTest.java
-index 1035cf9bc8..55c0488d03 100644
+index f538c953f6..728228c63c 100644
 --- a/java/java.lsp.server/test/unit/src/org/netbeans/modules/java/lsp/server/ConnectionSpecTest.java
 +++ b/java/java.lsp.server/test/unit/src/org/netbeans/modules/java/lsp/server/ConnectionSpecTest.java
-@@ -118,7 +118,7 @@ public class ConnectionSpecTest {
+@@ -118,10 +118,12 @@ public class ConnectionSpecTest {
              ByteArrayInputStream in = new ByteArrayInputStream(bytes);
              ByteArrayOutputStream os = new ByteArrayOutputStream();
              conn.prepare("Pipe server", in, os, new LspSession(), ConnectionSpecTest::setCopy, ConnectionSpecTest::copy);
 -            String reply = os.toString("UTF-8");
 +            String reply = os.toString("UTF-8").replaceAll("\n$", "");
              String exp = "Pipe server listening at port ";
              assertTrue(reply, reply.startsWith(exp));
-             int port = Integer.parseInt(reply.substring(exp.length()));
+-            int port = Integer.parseInt(reply.substring(exp.length(), reply.indexOf('\n', exp.length())));
++            int nextLineIdx = reply.indexOf('\n', exp.length());
++            if (nextLineIdx < 0) nextLineIdx = reply.length();
++            int port = Integer.parseInt(reply.substring(exp.length(), nextLineIdx));
+             assertTrue("port is specified: " + port, port >= 1024);
+             try (ConnectionSpec second = ConnectionSpec.parse("connect:" + port)) {
+                 second.prepare("Pipe client", in, os, new LspSession(), ConnectionSpecTest::setCopy, ConnectionSpecTest::copy);