GeezerVison

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 16, 2026 6:30am

PR Summary

Low Risk
Changes are confined to UI rendering and analytics guards; main risk is minor behavior differences in modal loading and queued message layout/resize handling.

Overview
Improves landing-page modal loading by adding explicit loading: () => null fallbacks to dynamically imported AuthModal/DemoRequestModal components, reducing transient loading UI.

Hardens analytics initialization by guarding posthog.sessionRecordingStarted before calling it, preventing runtime errors in environments where the API is missing.

Refines workspace message UI: QueuedMessages now uses a callback-ref ResizeObserver pattern, and UserMessageContent adds a compact variant (used by queued messages) while memoizing mention-range computation to avoid repeated parsing during renders.

^{Reviewed by Cursor Bugbot for commit 904ab7d. Configure here.}

Greptile Summary

This PR bundles five focused fixes: a missing typeof guard on posthog.sessionRecordingStarted, loading: () => null added to all unguarded dynamic modal imports on landing pages, a callback-ref replacement for the useEffect-based ResizeObserver in QueuedMessages, a compact prop that replaces !important overrides in UserMessageContent (with computeMentionRanges memoised), and removal of duplicate stop-generation handlers left over from a rebase.

Confidence Score: 5/5

Safe to merge — all changes are targeted bug fixes and code quality improvements with no new regressions introduced.

All five fix areas are well-scoped: the PostHog typeof guard is correct, loading: () => null follows the established pattern, the callback-ref ResizeObserver pattern is a valid improvement with proper cleanup, the compact prop correctly replaces !important overrides, and the duplicate handler removal is a clean rebase fix. No P0/P1 issues found.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[SessionProvider mounts] --> B[loadSession]
    B --> C{isPending false?}
    C --> D[import posthog-js]
    D --> E{posthog.identify is function?}
    E -- no --> Z[return early]
    E -- yes --> F{data.user exists?}
    F -- no --> G[posthog.reset]
    F -- yes --> H[posthog.identify]
    H --> I{startSessionRecording AND sessionRecordingStarted are functions?}
    I -- no --> Z
    I -- yes --> J{sessionRecordingStarted returns false?}
    J -- already started --> Z
    J -- not started --> K[startSessionRecording]

_{Reviews (1): Last reviewed commit: "fix(ui): posthog guard, dynamic import l..." | Re-trigger Greptile}