Merge branch 'main' into javascript-type-module-attribute

Author: Artur-

flow-build-tools/src/main/java/com/vaadin/flow/server/frontend/FrontendTools.java

@@ -70,7 +70,7 @@ public class FrontendTools {
      */
     public static final String DEFAULT_NPM_VERSION = "11.12.1";
 
-    public static final String DEFAULT_PNPM_VERSION = "10.30.2";
+    public static final String DEFAULT_PNPM_VERSION = "11.0.4";
 
     private static final String MSG_PREFIX = "%n%n======================================================================================================";
     private static final String MSG_SUFFIX = "%n======================================================================================================%n";

flow-server/src/main/java/com/vaadin/flow/component/UI.java

@@ -133,7 +133,7 @@ public class UI extends Component
 
     private final UIInternals internals;
 
-    private final Page page = new Page(this);
+    private final Page page;
 
     private final Geolocation geolocation;
 
@@ -166,6 +166,7 @@ protected UI(UIInternalUpdater internalsHandler) {
         getNode().getFeature(ElementData.class).setTag("body");
         Component.setElement(this, Element.get(getNode()));
         pushConfiguration = new PushConfigurationImpl(this);
+        page = new Page(this);
         geolocation = new Geolocation(this);
     }
 

flow-server/src/main/java/com/vaadin/flow/component/page/ExtendedClientDetails.java

@@ -17,6 +17,7 @@
 
 import java.io.Serializable;
 import java.util.Date;
+import java.util.Objects;
 import java.util.TimeZone;
 import java.util.function.Function;
 
@@ -442,21 +443,23 @@ void setColorScheme(ColorScheme.Value colorScheme) {
     }
 
     /**
-     * Creates an ExtendedClientDetails instance from browser details JSON
-     * object. This is intended for internal use when browser details are
-     * provided as JSON (e.g., during UI initialization or refresh).
+     * Parses browser details from the given JSON and updates the UI from them:
+     * stores the resulting {@link ExtendedClientDetails} on the UI's internals
+     * and seeds the page-visibility and geolocation-availability signals from
+     * the same payload.
      * <p>
      * For internal use only.
      *
      * @param ui
-     *            the UI instance that owns this ExtendedClientDetails
+     *            the UI to update, not {@code null}
      * @param json
      *            the JSON object containing browser details parameters
-     * @return a new ExtendedClientDetails instance
+     * @return the parsed details
      * @throws RuntimeException
      *             if the JSON is not a valid object
      */
-    public static ExtendedClientDetails fromJson(UI ui, JsonNode json) {
+    public static ExtendedClientDetails updateFromJson(UI ui, JsonNode json) {
+        Objects.requireNonNull(ui, "UI must not be null");
         if (!(json instanceof ObjectNode)) {
             throw new RuntimeException("Expected a JSON object");
         }
@@ -494,8 +497,9 @@ public static ExtendedClientDetails fromJson(UI ui, JsonNode json) {
                 getStringElseNull.apply("v-np"),
                 getStringElseNull.apply("v-cs"),
                 getStringElseNull.apply("v-tn"));
+        ui.getInternals().setExtendedClientDetails(details);
         String ga = getStringElseNull.apply("v-ga");
-        if (ga != null && ui != null) {
+        if (ga != null) {
             try {
                 ui.getInternals().setGeolocationAvailability(
                         GeolocationAvailability.valueOf(ga));
@@ -519,8 +523,7 @@ public static ExtendedClientDetails fromJson(UI ui, JsonNode json) {
     public void refresh(SerializableConsumer<ExtendedClientDetails> callback) {
         final String js = "return Vaadin.Flow.getBrowserDetailsParameters();";
         final SerializableConsumer<JsonNode> resultHandler = json -> {
-            ExtendedClientDetails details = fromJson(ui, json);
-            ui.getInternals().setExtendedClientDetails(details);
+            ExtendedClientDetails details = updateFromJson(ui, json);
             if (callback != null) {
                 callback.accept(details);
             }

flow-server/src/main/java/com/vaadin/flow/server/BootstrapHandler.java

@@ -1388,9 +1388,7 @@ private void extractAndStoreBrowserDetails(VaadinRequest request, UI ui) {
         if (browserDetailsJson != null && !browserDetailsJson.isEmpty()) {
             try {
                 JsonNode json = JacksonUtils.readTree(browserDetailsJson);
-                ExtendedClientDetails details = ExtendedClientDetails
-                        .fromJson(ui, json);
-                ui.getInternals().setExtendedClientDetails(details);
+                ExtendedClientDetails.updateFromJson(ui, json);
             } catch (Exception e) {
                 // Log and continue without browser details
                 getLogger().debug(