refactor(core): Use promise.finally for task removal (#62674)

atscott · NothingEverHappens · commit 448a7f7e8fb6 · 2025-07-18T07:19:49.000-04:00
Updates the approach from #62631 to use Promise.finally for task removal. This is a bit cleaner and also handles rejections from the donePromise PR Close #62674
diff --git a/packages/core/src/platform/bootstrap.ts b/packages/core/src/platform/bootstrap.ts
@@ -138,8 +138,8 @@ export function bootstrap<M>(
       const initStatus = envInjector.get(ApplicationInitStatus);
       initStatus.runInitializers();
 
-      return initStatus.donePromise.then(() => {
-        try {
+      return initStatus.donePromise
+        .then(() => {
           // If the `LOCALE_ID` provider is defined at bootstrap then we set the value for ivy
           const localeId = envInjector.get(LOCALE_ID, DEFAULT_LOCALE_ID);
           setLocaleId(localeId || DEFAULT_LOCALE_ID);
@@ -172,10 +172,8 @@ export function bootstrap<M>(
             moduleBootstrapImpl?.(config.moduleRef, config.allPlatformModules);
             return config.moduleRef;
           }
-        } finally {
-          pendingTasks.remove(taskId);
-        }
-      });
+        })
+        .finally(() => void pendingTasks.remove(taskId));
     });
   });
 }
