ldy26098
/
BMAD-METHOD
mirror of https://github.com/bmad-code-org/BMAD-METHOD.git
1
0
Fork 0
Code Packages Projects Releases Wiki Activity

fix: prevent spinner from hiding confirmation prompt during upgrade 

Fixes #907

When quickUpdate() called install(), two spinners ran simultaneously.
The quickUpdate spinner continued running while install() stopped its
own spinner to show prompts, causing the confirmation prompt to be
hidden behind the spinner animation.

Now quickUpdate() always stops its spinner before calling install(),
ensuring only one spinner is active and all prompts are visible.
This commit is contained in:
Wojciech Tyziniec 2025-11-15 17:13:41 +01:00
parent 05ccd1904c
commit d05f6ee420
No known key found for this signature in database
GPG Key ID: F4E491E5D0CE9709
1 changed files with 14 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
tools/cli/installers/lib/core/installer.js
View File

@ -1763,10 +1763,11 @@ class Installer {
const newBmadFolderName = this.configCollector.collectedConfig.core?.bmad_folder || existingBmadFolderName;
if (existingBmadFolderName === newBmadFolderName) {
// Normal quick update - start the spinner
spinner.start('Updating BMAD installation...');
// Normal quick update - stop spinner before calling install()
// install() will manage its own spinner
spinner.stop();
} else {
// Folder name has changed - stop spinner and let install() handle it
// Folder name has changed - notify user that install() will handle migration
spinner.stop();
console.log(chalk.yellow(`\n⚠️ Folder name will change: ${existingBmadFolderName} → ${newBmadFolderName}`));
console.log(chalk.yellow('The installer will handle the folder migration.\n'));
@ -1786,14 +1787,12 @@ class Installer {
_savedIdeConfigs: savedIdeConfigs, // Pass saved IDE configs to installer
};
// Call the standard install method
// Call the standard install method (it will manage its own spinner)
const result = await this.install(installConfig);
// Only succeed the spinner if it's still spinning
// (install method might have stopped it if folder name changed)
if (spinner.isSpinning) {
spinner.succeed('Quick update complete!');
}
// install() handles its own spinner, so no need to check if our spinner is spinning
// Just show completion message
console.log(chalk.green('✓ Quick update complete!'));
return {
success: true,
@ -1804,7 +1803,12 @@ class Installer {
ides: configuredIdes,
};
} catch (error) {
spinner.fail('Quick update failed');
// Spinner is already stopped, just show error
if (spinner.isSpinning) {
spinner.fail('Quick update failed');
} else {
console.error(chalk.red('✗ Quick update failed'));
}
throw error;
}
}