Fix TikTok browser crashes

[thomshutt]

PR Type

Bug fix

---

Description

• Guard against undefined navigator in share handler

• Add navigator check in TikTok browser hook

• Prevent SSR crashes in analytics tracking

• Log missing API errors instead of throwing

---

Changes walkthrough 📝

Relevant files

Error handling

ClipShareContent.tsx Guard navigator in share handler                                                  apps/app/components/daydream/Clipping/ClipShareContent.tsx Added undefined navigator guard in share handler Logged error and early returned if API missing +5/-0      useIsTikTokBrowser.ts Include navigator check in hook                                                    apps/app/hooks/useIsTikTokBrowser.ts Included navigator undefined check in useEffect Prevent SSR errors accessing navigator.userAgent +1/-1      track.ts Guard window/navigator in tracking                                              apps/app/lib/track.ts Guarded window and navigator in getBrowserInfo Early return in track if APIs undefined Kept webdriver skip logic intact +8/-5

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
pipelines-app	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 3, 2025 8:05pm

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Web Share API The code checks if navigator is defined but does not verify support for the Web Share API (navigator.share), which may lead to runtime errors in browsers without this feature. Include feature detection or a fallback. if (typeof navigator === "undefined") { console.error("Navigator API not available"); return; } SSR Hydration On server-side renders the hook leaves isTikTokBrowser at its initial false, which can cause hydration mismatches and UI flicker. Consider deferring the check or using a hydration-safe pattern. if (typeof window !== "undefined" && typeof navigator !== "undefined") { const userAgent = window.navigator.userAgent.toLowerCase(); setIsTikTokBrowser(identifyTikTokInAppBrowser(userAgent)); } Silent Skipped Tracks The tracking skip logic returns false under SSR or bot detection without logging, making it hard to debug why events aren’t sent. Consider adding logs when analytics calls are skipped. if (typeof window === "undefined" || typeof navigator === "undefined") { return false; } if (navigator.webdriver) { return false;

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Validate Web Share API support Instead of only checking for the existence of the global navigator object, verify support for the Web Share API (navigator.share) to avoid runtime errors on browsers where navigator exists but the share method does not. apps/app/components/daydream/Clipping/ClipShareContent.tsx [47-50] -if (typeof navigator === "undefined") { - console.error("Navigator API not available"); +if (!navigator?.share) { + console.error("Web Share API is not supported in this browser."); return; } Suggestion importance[1-10]: 7 __ Why: Checking only for navigator existence misses cases where navigator.share is unsupported; verifying navigator.share prevents runtime errors and improves robustness.	Medium