From e0e59e66bfc123904f9c008e0f42c4ed0d757a1b Mon Sep 17 00:00:00 2001 From: Alex Verkhovsky Date: Sat, 21 Mar 2026 21:51:48 -0600 Subject: [PATCH] refactor(installer): remove logo display and setBmadFolderName from install() MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Logo display belongs in ui.js, not install(). setBmadFolderName call was redundant — IdeManager constructor already defaults to the same constant. Remove dead CLIUtils import. --- tools/cli/installers/lib/core/installer.js | 7 ------- 1 file changed, 7 deletions(-) diff --git a/tools/cli/installers/lib/core/installer.js b/tools/cli/installers/lib/core/installer.js index ff96e5ba5..6e91d7603 100644 --- a/tools/cli/installers/lib/core/installer.js +++ b/tools/cli/installers/lib/core/installer.js @@ -9,7 +9,6 @@ const { FileOps } = require('../../../lib/file-ops'); const { Config } = require('../../../lib/config'); const { ConfigCollector } = require('./config-collector'); const { getProjectRoot, getSourcePath, getModulePath } = require('../../../lib/project-root'); -const { CLIUtils } = require('../../../lib/cli-utils'); const { ManifestGenerator } = require('./manifest-generator'); const { IdeConfigManager } = require('./ide-config-manager'); const { CustomHandler } = require('../custom-handler'); @@ -47,18 +46,12 @@ class Installer { // Everything else — custom modules, quick-update state, the whole mess const customConfig = { ...originalConfig }; - // if core config isn't collected, we haven't run the UI -> display logo/version - if (!config.hasCoreConfig()) { - await CLIUtils.displayLogo(); - } - const paths = await InstallPaths.create(config); // Collect configurations for official modules const moduleConfigs = await this._collectConfigs(config, paths); await this.customModules.discoverPaths(config, paths); - this.ideManager.setBmadFolderName(BMAD_FOLDER_NAME); try { const existingInstall = await this.detector.detect(paths.bmadDir);