From f056ec7c96d07a6a43ee4b0451e82e0735f46ebc Mon Sep 17 00:00:00 2001 From: Wojciech Tyziniec <10180694+wtznc@users.noreply.github.com> Date: Sat, 15 Nov 2025 17:35:09 +0100 Subject: [PATCH] fix: add operation context to error handling MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Restore error message in quickUpdate to provide operation context. Users now see: 1. '✗ Quick update failed' (what operation failed) 2. 'Installation failed: [details]' (why it failed from caller) This provides better UX than generic 'Installation failed' alone. --- tools/cli/installers/lib/core/installer.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/cli/installers/lib/core/installer.js b/tools/cli/installers/lib/core/installer.js index de52b1e0..715d04d9 100644 --- a/tools/cli/installers/lib/core/installer.js +++ b/tools/cli/installers/lib/core/installer.js @@ -1800,10 +1800,12 @@ class Installer { ides: configuredIdes, }; } catch (error) { - // Ensure spinner is stopped, then let caller handle error message + // Ensure spinner is stopped if (spinner.isSpinning) { spinner.stop(); } + // Provide operation context before detailed error from caller + console.error(chalk.red('✗ Quick update failed')); throw error; } }