fix: add operation context to error handling
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.
This commit is contained in:
parent
59ed774154
commit
f056ec7c96
|
|
@ -1800,10 +1800,12 @@ class Installer {
|
||||||
ides: configuredIdes,
|
ides: configuredIdes,
|
||||||
};
|
};
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
// Ensure spinner is stopped, then let caller handle error message
|
// Ensure spinner is stopped
|
||||||
if (spinner.isSpinning) {
|
if (spinner.isSpinning) {
|
||||||
spinner.stop();
|
spinner.stop();
|
||||||
}
|
}
|
||||||
|
// Provide operation context before detailed error from caller
|
||||||
|
console.error(chalk.red('✗ Quick update failed'));
|
||||||
throw error;
|
throw error;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue