v0.6.46: mothership queueing, web vitals#4194
Conversation
* fix(ui): fix attachment logic on queued mothership messages * Add focus after hitting pencil button for queued message * fix copilot layout
* improvement(landing): optimize core web vitals and accessibility Code-split AuthModal and DemoRequestModal via next/dynamic across 7 landing components to move auth-client bundle (~150-250KB) out of the initial JS payload. Replace useSession import in navbar with direct SessionContext read to avoid pulling the entire better-auth client into the landing page bundle. Add immutable cache header for content-hashed _next/static assets. Defer PostHog session recording until user identification to avoid loading the recorder (~80KB) on anonymous visits. Fix accessibility issues flagged by Lighthouse: add missing aria-label on preview submit button, add inert to aria-hidden ReactFlow wrapper, set decorative alt on logos inside labeled links, disambiguate duplicate footer API links. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix(posthog): guard startSessionRecording against repeated calls on refetch The effect fires on every session reload (e.g., subscription upgrade). Calling startSessionRecording() while already recording fragments the session in the analytics dashboard. Add sessionRecordingStarted() guard so recording only starts once per page lifecycle. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix(config): remove redundant _next/static cache header Next.js already sets Cache-Control: public, max-age=31536000, immutable on _next/static assets natively and this cannot be overridden. The custom rule was redundant on Vercel and conflicted with the extension-based rule on self-hosted deployments due to last-match-wins ordering. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Optimizes the landing experience for Core Web Vitals/accessibility by converting Updates analytics config to disable PostHog session recording by default and only start recording after a user session is identified. Reviewed by Cursor Bugbot for commit 23ccd4a. Configure here. |
Greptile SummaryThis PR bundles two improvements: (1) a fix for the Mothership queued-message edit flow so that file attachments and context mentions are properly restored when a queued message is pulled back into the input, and (2) a set of landing-page Core Web Vitals / accessibility improvements (lazy-loading modals with The queued-message fix uses Confidence Score: 5/5Safe to merge — the queued-message attachment fix is well-architected and the landing optimisations are straightforward. All findings are P2 (one missing typeof guard that is already inside a try/catch). No data loss, no broken paths, no security issues. apps/sim/app/_shell/providers/session-provider.tsx — minor defensive guard missing around sessionRecordingStarted(). Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant QueuedMessages
participant MothershipChat
participant Store as home.tsx / panel.tsx store
participant UserInput
User->>QueuedMessages: click ✏️ edit
QueuedMessages->>MothershipChat: onEdit(id)
MothershipChat->>Store: onEditQueuedMessage(id)
Note over Store: removes message from queue,<br/>returns QueuedMessage
Store-->>MothershipChat: QueuedMessage { content, fileAttachments, contexts }
MothershipChat->>UserInput: userInputRef.loadQueuedMessage(msg)
UserInput->>UserInput: setValue(msg.content)
UserInput->>useFileAttachments: restoreAttachedFiles(msg.fileAttachments)
UserInput->>useContextManagement: setSelectedContexts(msg.contexts)
UserInput->>textarea: focus() + setSelectionRange(end, end)
Note over UserInput: ArrowUp (empty input) also<br/>triggers edit of tail message
Reviews (1): Last reviewed commit: "improvement(landing): optimize core web ..." | Re-trigger Greptile |
…se cleanup (#4196) * v0.6.29: login improvements, posthog telemetry (#4026) * feat(posthog): Add tracking on mothership abort (#4023) Co-authored-by: Theodore Li <theo@sim.ai> * fix(login): fix captcha headers for manual login (#4025) * fix(signup): fix turnstile key loading * fix(login): fix captcha header passing * Catch user already exists, remove login form captcha * fix(ui): posthog guard, dynamic import loading, compact variant, rebase cleanup --------- Co-authored-by: Theodore Li <theodoreqili@gmail.com>
Uh oh!
There was an error while loading. Please reload this page.