fix: Do not try to refresh app while it is initializing (#20504) (#20512

) When the app is bootstrapping, it only contains `initializing`, `isActive` and `productionMode`. We should not call `sendEventMessage` at this point or might cause `Uncaught TypeError: l.sendEventMessage is not a function`. This is unlikely to happen in many real world scenarios but happens in Copilot tests Co-authored-by: Artur <[email protected]>
vaadin · Nov 19, 2024 · 73586c9 · 73586c9
1 parent da9b46d
commit 73586c9
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/vaadin-dev-server/src/main/frontend/vaadin-dev-tools.ts b/vaadin-dev-server/src/main/frontend/vaadin-dev-tools.ts
@@ -656,9 +656,13 @@ export class VaadinDevTools extends LitElement {
             .filter((key) => key !== 'TypeScript')
             .map((id) => anyVaadin.Flow.clients[id])
             .forEach((client) => {
-              client.sendEventMessage(1, "ui-refresh", {
-                fullRefresh: strategy === 'full-refresh'
-              })
+              if (client.sendEventMessage) {
+                client.sendEventMessage(1, "ui-refresh", {
+                  fullRefresh: strategy === 'full-refresh'
+               })
+              } else {
+                console.warn("Ignoring ui-refresh event for application ",id);
+              }
             });
       } else {
         const lastReload = window.sessionStorage.getItem(VaadinDevTools.TRIGGERED_COUNT_KEY_IN_SESSION_STORAGE);