Merge pull request #2829 from patternfly/fix/core/internals-ssr

fix(core): make internals controller ssrable

Author: adamjohnson

.changeset/thin-aliens-smoke.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/pfe-core": patch
+---
+`InternalsController`: prevent Safari-detector from breaking SSR

.github/workflows/tests.yml

@@ -95,7 +95,7 @@ jobs:
     name: SSR Tests (Playwright)
     runs-on: ubuntu-latest
     container:
-      image: mcr.microsoft.com/playwright:v1.45.1-jammy
+      image: mcr.microsoft.com/playwright:v1.46.1-jammy
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4

core/pfe-core/controllers/internals-controller.ts

@@ -83,7 +83,8 @@ export class InternalsController implements ReactiveController, ARIAMixin {
     return Array.from(this.instances.get(host)?.internals.labels ?? []) as Element[];
   }
 
-  public static isSafari: boolean = /^((?!chrome|android).)*safari/i.test(navigator.userAgent);
+  public static isSafari: boolean =
+    !isServer && /^((?!chrome|android).)*safari/i.test(navigator.userAgent);
 
   public static of(
     host: ReactiveControllerHost,