Skip to content

fix: restore chat autoscroll on send#522

Closed
Yezh1i wants to merge 5 commits into
DEEIX-AI:devfrom
Yezh1i:fix/chat-autoscroll
Closed

fix: restore chat autoscroll on send#522
Yezh1i wants to merge 5 commits into
DEEIX-AI:devfrom
Yezh1i:fix/chat-autoscroll

Conversation

@Yezh1i

@Yezh1i Yezh1i commented Jul 22, 2026

Copy link
Copy Markdown

Summary

  • anchor newly pending user messages so sending from a scrolled-up conversation returns to the active turn
  • preserve the existing parent-user anchor behavior for assistant-only retries and continuations
  • add focused regression coverage for live-message anchor selection

Root cause

Pending user messages are marked isPending, but the live-anchor lookup started one item before the first live message. As a result, the newly appended user item was skipped and the message scroller could not find a new scrollAnchor to jump to when the viewport was away from the bottom.

Testing

  • pnpm --filter @deeix/web test
  • pnpm --filter @deeix/web typecheck
  • pnpm --filter @deeix/web lint
  • pnpm --filter @deeix/web build

@Yezh1i

Yezh1i commented Jul 22, 2026

Copy link
Copy Markdown
Author

Superseded by #523, which contains the verified final implementation as a clean single commit.

@Yezh1i Yezh1i closed this Jul 22, 2026
@Yezh1i
Yezh1i deleted the fix/chat-autoscroll branch July 22, 2026 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant