DEV: Cleanup

Author: janzenisaac

assets/javascripts/initializers/ai-conversations-sidebar.js

@@ -137,9 +137,7 @@ export default {
               this.loadedThirtyDayLabel = false;
               this.loadedMonthLabels.clear();
 
-              // Build links again with the new topic
-              const allTopics = [topic, ...this.topics];
-              this.topics = allTopics;
+              this.topics = [topic, ...this.topics];
               this.buildSidebarLinks();
 
               this.watchForTitleUpdate(topic);

spec/system/ai_bot/homepage_spec.rb

@@ -179,9 +179,7 @@
       header.click_bot_button
 
       expect(ai_pm_homepage).to have_homepage
-      sidebar.find(
-        ".sidebar-section[data-section-name='ai-conversations-history'] a.sidebar-section-link",
-      ).click
+      ai_pm_homepage.click_fist_sidebar_conversation
       expect(topic_page).to have_topic_title(pm.title)
     end
 
@@ -193,9 +191,7 @@
       expect(header).to have_icon_in_bot_button(icon: "shuffle")
 
       # Go to a PM and assert that the icon is still shuffle
-      sidebar.find(
-        ".sidebar-section[data-section-name='ai-conversations-history'] a.sidebar-section-link",
-      ).click
+      ai_pm_homepage.click_fist_sidebar_conversation
       expect(header).to have_icon_in_bot_button(icon: "shuffle")
 
       # Go back home and assert that the icon is now robot again

spec/system/page_objects/components/ai_pm_homepage.rb

@@ -40,6 +40,12 @@ def click_new_question_button
         page.find(".ai-new-question-button").click
       end
 
+      def click_fist_sidebar_conversation
+        page.find(
+          ".sidebar-section[data-section-name='ai-conversations-history'] a.sidebar-section-link:not(.date-heading)",
+        ).click
+      end
+
       def persona_selector
         PageObjects::Components::SelectKit.new(".persona-llm-selector__persona-dropdown")
       end