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(installer): restore currentProjectDir writes for placeholder expansion 

The previous commit removed the three assignments to
OfficialModules.currentProjectDir as dead code, but buildQuestion()
still reads the property to resolve {directory_name} placeholders in
module config defaults during interactive collection. Without the
writes, any module default containing {directory_name} would surface
the literal placeholder to users.
This commit is contained in:
Alex Verkhovsky 2026-04-10 20:15:34 -07:00
parent 67a3c4c6f7
commit 70b32f96a4
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
tools/installer/modules/official-modules.js
View File

@ -12,6 +12,9 @@ class OfficialModules {
// Config collection state (merged from ConfigCollector) // Config collection state (merged from ConfigCollector)
this.collectedConfig = {}; this.collectedConfig = {};
this._existingConfig = null; this._existingConfig = null;
// Tracked during interactive config collection so {directory_name}
// placeholder defaults can be resolved in buildQuestion().
this.currentProjectDir = null;
} }
/** /**
@ -1040,6 +1043,7 @@ class OfficialModules {
* @returns {boolean} True if new fields were prompted, false if all fields existed * @returns {boolean} True if new fields were prompted, false if all fields existed
*/ */
async collectModuleConfigQuick(moduleName, projectDir, silentMode = true) { async collectModuleConfigQuick(moduleName, projectDir, silentMode = true) {
this.currentProjectDir = projectDir;
// Load existing config if not already loaded // Load existing config if not already loaded
if (!this._existingConfig) { if (!this._existingConfig) {
await this.loadExistingConfig(projectDir); await this.loadExistingConfig(projectDir);
@ -1330,6 +1334,7 @@ class OfficialModules {
* @param {boolean} skipCompletion - Skip showing completion message (for early core collection) * @param {boolean} skipCompletion - Skip showing completion message (for early core collection)
*/ */
async collectModuleConfig(moduleName, projectDir, skipLoadExisting = false, skipCompletion = false) { async collectModuleConfig(moduleName, projectDir, skipLoadExisting = false, skipCompletion = false) {
this.currentProjectDir = projectDir;
// Load existing config if needed and not already loaded // Load existing config if needed and not already loaded
if (!skipLoadExisting && !this._existingConfig) { if (!skipLoadExisting && !this._existingConfig) {
await this.loadExistingConfig(projectDir); await this.loadExistingConfig(projectDir);