Fix scroll behavior for inline feedback messages

[mabashian]

Jira Issue: https://issues.redhat.com/browse/AAP-44636

Assisted-by: Claude code

Description

When users clicked thumbs up/down on previous messages in the chatbot, the view would auto-scroll to the bottom instead of staying anchored on the message being rated. This caused the "Thank you for your feedback!" confirmation message to appear off-screen, disrupting the user experience.

Solution

Fixed the scroll behavior by implementing a two-part solution:

1. Conditional scroll triggering (AnsibleChatbot.tsx)

Modified the useEffect hook to only trigger auto-scroll when the scrollToHere flag is explicitly set on the last message, preventing unwanted scrolling when feedback messages are added inline.

  useEffect(() => {
      // Only auto-scroll if the last message has scrollToHere flag set
      // This prevents unwanted scrolling when feedback messages are added inline
      const lastMessage = messages[messages.length - 1];
      if (lastMessage?.scrollToHere) {
          scrollToBottom();
      }
  }, [messages]);

2. Smart flag management (useChatbot.ts)

Updated the addMessage function to only set the scrollToHere flag when messages are appended to the end, not when they're inserted inline using the addAfter parameter (which is used for feedback messages).

// Only set scrollToHere if message is added at the end (not inserted inline)
newMessage.scrollToHere = !isStreamingSupported() && !addAfter;

3. Unit tests (AnsibleChatbot.test.tsx)

Added comprehensive unit tests to verify:

• Feedback messages appear inline without triggering scroll
• New chat messages still trigger scroll as expected

Testing

All existing tests pass (41 tests in aap_chatbot, 39 tests in ansible_ai_connect_chatbot), plus 2 new tests per implementation verifying the scroll fix.

Production deployment

• This code change is ready for production on its own
• This code change requires the following considerations before going to production:

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[mabashian]

Here's the change in action: