From c83f9d2a7bf4722a4066132baa4e47b14ae571e4 Mon Sep 17 00:00:00 2001 From: chinesepowered Date: Sat, 4 Apr 2026 15:16:59 -0700 Subject: [PATCH] fix(extensions): handle individual extension update check failures checkForAllExtensionUpdates() used Promise.all without per-promise catch handlers. A single extension's update check failure would reject the entire batch, leaving remaining extensions stuck in CHECKING_FOR_UPDATES state. Now catches per-extension and reports ERROR. --- packages/core/src/extension/extensionManager.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/core/src/extension/extensionManager.ts b/packages/core/src/extension/extensionManager.ts index ca57c26ed..375628bf2 100644 --- a/packages/core/src/extension/extensionManager.ts +++ b/packages/core/src/extension/extensionManager.ts @@ -1232,9 +1232,9 @@ export class ExtensionManager { } callback(extension.name, ExtensionUpdateState.CHECKING_FOR_UPDATES); promises.push( - checkForExtensionUpdate(extension, this).then((state) => - callback(extension.name, state), - ), + checkForExtensionUpdate(extension, this) + .then((state) => callback(extension.name, state)) + .catch(() => callback(extension.name, ExtensionUpdateState.ERROR)), ); } await Promise.all(promises);