Move response ready notification to trigger only after generation completes and remove sessionStorage check

Author: robertpiosik

apps/browser/src/content-scripts/send-prompt-content-script/utils/add-apply-response-button.ts

@@ -81,15 +81,14 @@ export function observe_for_responses(params: {
         action: 'finished-responding'
       })
       has_sent_finished_responding = true
+      show_response_ready_notification({ chatbot_name: params.chatbot_name })
     }
 
     const all_footers = document.querySelectorAll(params.footer_selector)
     if (all_footers.length == 0) {
       return
     }
 
-    show_response_ready_notification({ chatbot_name: params.chatbot_name })
-
     all_footers.forEach((footer) => {
       params.add_buttons(footer)
     })

apps/browser/src/content-scripts/send-prompt-content-script/utils/show-response-ready-notification.ts

@@ -1,11 +1,6 @@
 export const show_response_ready_notification = async (params: {
   chatbot_name: string
 }) => {
-  const NOTIFICATION_SHOWN_KEY = 'cwc_notification_shown'
-
-  if (sessionStorage.getItem(NOTIFICATION_SHOWN_KEY)) return
-  sessionStorage.setItem(NOTIFICATION_SHOWN_KEY, 'true')
-
   if (!('Notification' in window) || document.hasFocus()) {
     return
   }