GeezerVison

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Aug 11, 2026 3:41am

PR Summary

Medium Risk
Touches scroll pinning and virtualized transcript layout during streaming—user-visible and timing-sensitive—but logic is isolated with unit tests and preserves tall-transcript behavior.

Overview
Fixes phantom scroll space during streaming when the transcript is shorter than the viewport. The sizer min-height floor was tied to the viewport bottom (scrollTop + clientHeight), which can sit below real content height—so the floor invented empty scrollable area that only shows up when the scroll container shrinks mid-turn (composer growth, queued-message banner, resize), pulling the pinned transcript upward until end-of-turn drain.

nextSizerFloor centralizes the floor: a per-turn high-water mark (max of prior high water, virtualizer total size, and applied floor) caps the floor so it never exceeds space content has actually held. Mid-turn shrinks still hold via high water; undrained floor debt still carries when the floor re-engages mid-drain.

mothership-chat wires that helper, tracks high water in a ref, and resets floor state on chat switch (not when a pending chat gets its id) so a short chat doesn’t inherit another chat’s floor.

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

Greptile Summary

The PR prevents short streaming transcripts from creating phantom scroll space and adds explicit floor-state cleanup when switching chats.

• Clamps the active sizer floor to the transcript’s retained high-water mark.
• Preserves undrained floor debt when a new turn interrupts the drain.
• Clears transcript-specific floor state on chat changes.
• Adds focused unit coverage for floor bounds, shrinking content, fractional offsets, and interrupted drains.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Transcript layout commit] --> B{Chat changed?}
  B -->|Yes, distinct chat| C[Cancel drain and clear floor state]
  B -->|No or pending chat persisted| D{Floor active?}
  C --> D
  D -->|Yes| E[Compute content high-water mark]
  E --> F[Clamp viewport extent to high-water mark]
  F --> G[Apply bounded min-height]
  D -->|No| H[Drain or clear applied floor]

_{Reviews (3): Last reviewed commit: "fix(chat): release the transcript floor ..." | Re-trigger Greptile}

Greptile Summary

The PR extracts the transcript floor calculation and clamps it to content held during the active turn, with focused unit coverage.

• Adds nextSizerFloor and tests for short transcripts, fractional extents, row shrinkage, growth, and interrupted drains.
• Integrates a turn-level high-water mark into the streaming transcript floor and preserves undrained floor debt.
• The high-water state currently lacks a reset at chat-identity boundaries.

Confidence Score: 4/5

The PR should not merge until the floor’s high-water state is reset when the active chat changes; the import-convention issue is non-blocking.

A retained component can apply the previous tall chat’s high-water mark to a newly selected short chat for at least one layout pass, recreating artificial scroll space that then drains visibly.

Files Needing Attention: apps/sim/app/workspace/[workspaceId]/home/components/mothership-chat/mothership-chat.tsx; apps/sim/app/workspace/[workspaceId]/home/components/mothership-chat/sizer-floor.test.ts

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Tall chat is streaming] --> B[heldHighWater records tall content]
  B --> C[User switches to cached short chat]
  C --> D[MothershipChat instance and refs persist]
  D --> E[Layout effect runs while lastRowAnimating is still true]
  E --> F[nextSizerFloor retains old high-water]
  F --> G[Short chat receives artificial min-height]
  G --> H[Floor drains over subsequent frames]

_{Reviews (1): Last reviewed commit: "fix(chat): stop the streaming transcript..." | Re-trigger Greptile}

@greptile

@cursor review

@greptile