Fix notification polling race condition (aws#5142)

* Add framework for processing notifications (aws#5112)

* Add deserialization for notification messages retrieved from the notification file and criteria on whether it should be displayed (aws#5093)

* Display toast notifications with actions

* Condition matcher for displaying notifications

* Modified deserialization cases and added tests

* not required file change

* feedback 1

* modified the base class

* modified test instance lifecycle

* Show toasts for notifications and notification banner on critical notifications(aws#5097)

* Display toast notifications with actions

* Condition matcher for displaying notifications

* Show notification banner

* feedback 1

* Modified deserialization cases and added tests

* not required file change

* feedback 1

* feedback 1

* modified the base class

* merge conflicts resolved

* rearranged call site

* show notifications when panel is opened

* fixed tests

* detekt

* feedback

* convert panels into wrappers

* fixed test

* Adding update/restart action to notifications (aws#5136)

* Poll for new notifications (aws#5119)

* initial commit

* run on startup

* detekt

* move vals

* remote resource implementation

* comments

* detekt

* Validate file before saving

* cache path

* observer implementation

* deserialize notifs from file

* detekt

* remove unused interface

* internal class

* Fix observer

* etag singleton state component

* add telemetry

* atomicBoolean

* initialize once per IDE startup

* code scan

* Omit (Unit)

* specify etag storage location

* detekt

* fix detekt issues

* basic tests

* no star imports

* coroutine scope delay instead of thread.sleep

* feedback fixes

* test fix

* Application Exists for tests

* endpoint object

* detekt

* detekt fixes

* boolean flag

* boolean flag

* update tests

* move startup flag handling to processBase

* fix delay

* fix delay

* Notification dismissal state tracking  (aws#5129)

* split notifications into separated lists.

* add persistent notification dismissal state logic

* boolean changes

* group persistant states

* comments

* Service initialized automatically

* isStartup global

* Deserialized notification schedule type

* tests

* persistent state syntax

* convert to light services

* Remove state from companion object

* detekt

* endpoint as registryKey

* detekt

* fix startup issues

* Expiry issues

* Add logging info to IDE notification polling and processing (aws#5138)

* add logs for polling and processing notifs

* redundant

* finish log

* show message instead of yesNoDialogue

* initialize ProcessNotificationsBase listener before Polling service

* fix isFirstPoll not setting to false on first pass

---------

Co-authored-by: aws-toolkit-automation <[email protected]>
Co-authored-by: manodnyab <[email protected]>
Co-authored-by: Bryce Ito <[email protected]>

Author: 4 people

plugins/core/jetbrains-community/src/software/aws/toolkits/jetbrains/core/notifications/CustomizeNotificationsUi.kt

@@ -59,12 +59,10 @@ object NotificationManager {
         add(
             NotificationActionList(AwsCoreBundle.message("general.more_dialog")) {
                 if (url == null) {
-                    Messages.showYesNoDialog(
+                    Messages.showMessageDialog(
                         project,
                         message,
                         title,
-                        AwsCoreBundle.message("general.acknowledge"),
-                        AwsCoreBundle.message("general.cancel"),
                         AllIcons.General.Error
                     )
                 } else {

plugins/core/jetbrains-community/src/software/aws/toolkits/jetbrains/core/notifications/NotificationServiceInitializer.kt

@@ -13,6 +13,7 @@ internal class NotificationServiceInitializer : ProjectActivity {
     private val initialized = AtomicBoolean(false)
 
     override suspend fun execute(project: Project) {
+        ProcessNotificationsBase.getInstance(project)
         if (ApplicationManager.getApplication().isUnitTestMode) return
         if (initialized.compareAndSet(false, true)) {
             val service = NotificationPollingService.getInstance()

plugins/core/jetbrains-community/src/software/aws/toolkits/jetbrains/core/notifications/ProcessNotificationsBase.kt

@@ -31,6 +31,7 @@ class ProcessNotificationsBase(
 ) {
     private val notifListener = mutableListOf<NotifListener>()
     init {
+        LOG.info { "Initializing ProcessNotificationsBase" }
         NotificationPollingService.getInstance().addObserver {
             retrieveStartupAndEmergencyNotifications()
         }