fix(shared): Added support for global navigator check to isValidBrowser

[royanger]

Description

Checklist

• pnpm test runs as expected.
• pnpm build runs as expected.
• (If applicable) JSDoc comments have been added or updated for any package exports
• (If applicable) Documentation has been updated

Type of change

• 🐛 Bug fix
• 🌟 New feature
• 🔨 Breaking change
• 📖 Refactoring / dependency upgrade / documentation
• other:

Summary by CodeRabbit

• Bug Fixes

  • Fixed connectivity checks in MV3 extension background service workers so online/offline status is detected correctly; SSR without a navigator still reports offline.
  • Tightened WebAuthn detection to avoid false positives in environments that lack a window object.

• Tests

  • Added service-worker-like tests and expanded online/offline coverage for browser detection.

• Documentation

  • Updated changelog entry documenting the connectivity heuristic fix.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
clerk-js-sandbox	Ready	Preview, Comment	Jun 11, 2026 11:22pm
swingset	Ready	Preview, Comment	Jun 11, 2026 11:22pm

[changeset-bot]

🦋 Changeset detected

Latest commit: ca5e2d0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 21 packages

Name	Type
@clerk/shared	Patch
@clerk/astro	Patch
@clerk/backend	Patch
@clerk/chrome-extension	Patch
@clerk/clerk-js	Patch
@clerk/expo-passkeys	Patch
@clerk/expo	Patch
@clerk/express	Patch
@clerk/fastify	Patch
@clerk/hono	Patch
@clerk/localizations	Patch
@clerk/msw	Patch
@clerk/nextjs	Patch
@clerk/nuxt	Patch
@clerk/react-router	Patch
@clerk/react	Patch
@clerk/tanstack-react-start	Patch
@clerk/testing	Patch
@clerk/ui	Patch
@clerk/vue	Patch
@clerk/swingset	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository YAML (base), Repository UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 7d03d841-1ec9-4db4-9e6c-63039bfda8da

📥 Commits

Reviewing files that changed from the base of the PR and between d8b3d24 and ca5e2d0.

📒 Files selected for processing (5)

• .changeset/shared-navigator-service-worker.md
• packages/clerk-js/bundlewatch.config.json
• packages/shared/src/__tests__/browser.spec.ts
• packages/shared/src/browser.ts
• packages/shared/src/webauthn.ts

✅ Files skipped from review due to trivial changes (1)

• .changeset/shared-navigator-service-worker.md

🚧 Files skipped from review as they are similar to previous changes (4)

• packages/clerk-js/bundlewatch.config.json
• packages/shared/src/webauthn.ts
• packages/shared/src/browser.ts
• packages/shared/src/tests/browser.spec.ts

---

📝 Walkthrough

Walkthrough

This PR centralizes navigator resolution in a new getNavigator() helper so isValidBrowser() and isBrowserOnline() read from window.navigator or self.navigator (worker scopes). WebAuthn detection now requires window. Tests add a service-worker mock and cover SSR and worker online/offline cases; a changeset and a tiny bundlewatch size bump were included.

Changes

Service Worker Browser Environment Detection

Layer / File(s)	Summary
Navigator resolution refactor packages/shared/src/browser.ts	getNavigator() helper centralizes navigator retrieval from window.navigator or self.navigator for worker environments. isValidBrowser() and isBrowserOnline() now source navigator from this helper.
WebAuthn environment check packages/shared/src/webauthn.ts	isWebAuthnSupported() now explicitly requires typeof window !== 'undefined' alongside existing isValidBrowser() and PublicKeyCredential checks.
Test infrastructure and coverage packages/shared/src/__tests__/browser.spec.ts	Adds mockServiceWorkerScope() to simulate WorkerGlobalScope and extends tests for isValidBrowser and isValidBrowserOnline covering service worker (no window, self.navigator present) and SSR (no window, no WorkerGlobalScope) scenarios. Updates cleanup to call vi.unstubAllGlobals().
Changeset documentation .changeset/shared-navigator-service-worker.md	Documents navigator fallback behavior for MV3 extension background service workers and clarifies SSR and standard browser behavior remain unchanged.
Bundlewatch threshold packages/clerk-js/bundlewatch.config.json	Increases maxSize for ./dist/clerk.browser.js slightly (71.18KB → 71.24KB).

🎯 3 (Moderate) | ⏱️ ~25 minutes

🐰 In service workers bright and new,
Where window hides from distant view,
Our navigator takes the lead,
To find online when most we need,
No more misleading clerk_offline plead! 🔌

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title accurately describes the main change: adding support for reading global navigator when window is unavailable in isValidBrowser and related functions to fix service worker connectivity checks.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[pkg-pr-new]

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@8827

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@8827

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@8827

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@8827

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@8827

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@8827

@clerk/express

npm i https://pkg.pr.new/@clerk/express@8827

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@8827

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@8827

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@8827

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@8827

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@8827

@clerk/react

npm i https://pkg.pr.new/@clerk/react@8827

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@8827

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@8827

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@8827

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@8827

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@8827

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@8827

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@8827

commit: ca5e2d0

[github-actions]

API Changes Report

Generated by Break Check on 2026-06-11T23:24:00.832Z

Summary

Metric	Count
Packages analyzed	19
Packages with changes	0
🔴 Breaking changes	0
🟡 Non-breaking changes	0
🟢 Additions	0

Note
Break Check could not snapshot 3 subpaths; the diff below excludes them.

• @clerk/astro ./env: Internal Error: Unable to determine module for: /home/runner/_work/javascript/javascript/packages/astro/env.d.ts You have encountered a software defect. Please consider reporting the issue to the maintainers of this application.
• @clerk/shared ./cookie: Internal Error: Unable to follow symbol for "Cookies" You have encountered a software defect. Please consider reporting the issue to the maintainers of this application.
• @clerk/testing ./cypress: Symbol not found for identifier: Cypress

No API Changes Detected

All packages have stable APIs with no detected changes.

---

Report generated by Break Check

_{Last ran on ca5e2d0.}