fix(ChatMessages): reset scroll icon when messages are cleared

[howwohmm]

Fixes #6235

The scroll-to-bottom icon stays visible after you clear messages because the watchThrottled handler returns early when status !== 'streaming', so the reset never fires.

When messages goes empty, reset showAutoScroll, userScrolledUp, lastScrollTop, and messagesRefs before the status check.

[coderabbitai]

📝 Walkthrough

Walkthrough

A guard was added at the start of the throttled watcher in src/runtime/components/ChatMessages.vue. If props.messages is absent or empty, the watcher now sets showAutoScroll to false, userScrolledUp to false, lastScrollTop to 0, clears messagesRefs.value, and returns early, preventing subsequent streaming and auto-scroll checks from running when there are no messages.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main fix: resetting the scroll icon when messages are cleared, which is the primary objective of the changeset.
Linked Issues check	✅ Passed	The pull request fully implements the proposed fix from issue #6235: resetting showAutoScroll, userScrolledUp, and lastScrollTop when messages array is empty, before the streaming status check.
Out of Scope Changes check	✅ Passed	All changes in ChatMessages.vue are directly scoped to fixing the scroll icon visibility issue by adding an early guard for empty messages, with no unrelated modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	The pull request description clearly and accurately describes the changeset, explaining the bug fix and referencing the related issue.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/runtime/components/ChatMessages.vue`:
- Around line 146-151: When the messages array is empty the component resets
scroll-related state but doesn't clear the messagesRefs map, which can retain
stale DOM nodes; update the empty-message branch to also clear messagesRefs
(call messagesRefs.clear()) so the Map is emptied when props.messages has no
length—look for the messagesRefs symbol in ChatMessages.vue and add the clear in
the same conditional that sets showAutoScroll.value, userScrolledUp.value, and
lastScrollTop.value.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f066c515-4777-47a9-a57b-a7a6a9756ac6

📥 Commits

Reviewing files that changed from the base of the PR and between 90a94fb and 26d03bf.

📒 Files selected for processing (1)

• src/runtime/components/ChatMessages.vue

[pkg-pr-new]

npm i https://pkg.pr.new/@nuxt/ui@6239

commit: 37f1380