diff --git a/package.json b/package.json index 875d788f5..a26398fdf 100644 --- a/package.json +++ b/package.json @@ -70,7 +70,6 @@ "chalk": "^4.1.2", "commander": "^14.0.0", "csv-parse": "^6.1.0", - "fs-extra": "^11.3.0", "glob": "^11.0.3", "ignore": "^7.0.5", "js-yaml": "^4.1.0", diff --git a/src/bmm-skills/2-plan-workflows/bmad-create-prd/steps-c/step-08-scoping.md b/src/bmm-skills/2-plan-workflows/bmad-create-prd/steps-c/step-08-scoping.md index b060dda8d..c35289145 100644 --- a/src/bmm-skills/2-plan-workflows/bmad-create-prd/steps-c/step-08-scoping.md +++ b/src/bmm-skills/2-plan-workflows/bmad-create-prd/steps-c/step-08-scoping.md @@ -1,4 +1,4 @@ -# Step 8: Scoping Exercise - MVP & Future Features +# Step 8: Scoping Exercise - Scope Definition (Phased or Single-Release) **Progress: Step 8 of 11** - Next: Functional Requirements @@ -12,6 +12,8 @@ - π YOU ARE A FACILITATOR, not a content generator - π¬ FOCUS on strategic scope decisions that keep projects viable - π― EMPHASIZE lean MVP thinking while preserving long-term vision +- β οΈ NEVER de-scope, defer, or phase out requirements that the user explicitly included in their input documents without asking first +- β οΈ NEVER invent phasing (MVP/Growth/Vision) unless the user requests phased delivery β if input documents define all components as core requirements, they are ALL in scope - β YOU MUST ALWAYS SPEAK OUTPUT In your Agent communication style with the config `{communication_language}` - β YOU MUST ALWAYS WRITE all artifact and document content in `{document_output_language}` @@ -34,7 +36,7 @@ ## YOUR TASK: -Conduct comprehensive scoping exercise to define MVP boundaries and prioritize features across development phases. +Conduct comprehensive scoping exercise to define release boundaries and prioritize features based on the user's chosen delivery mode (phased or single-release). ## SCOPING SEQUENCE: @@ -75,30 +77,41 @@ Use structured decision-making for scope: - Advanced functionality that builds on MVP - Ask what features could be added in versions 2, 3, etc. +**β οΈ SCOPE CHANGE CONFIRMATION GATE:** +- If you believe any user-specified requirement should be deferred or de-scoped, you MUST present this to the user and get explicit confirmation BEFORE removing it from scope +- Frame it as a recommendation, not a decision: "I'd recommend deferring X because [reason]. Do you agree, or should it stay in scope?" +- NEVER silently move user requirements to a later phase or exclude them from MVP +- Before creating any consequential phase-based artifacts (e.g., phase tags, labels, or follow-on prompts), present artifact creation as a recommendation and proceed only after explicit user approval + ### 4. Progressive Feature Roadmap -Create phased development approach: -- Guide mapping of features across development phases -- Structure as Phase 1 (MVP), Phase 2 (Growth), Phase 3 (Vision) -- Ensure clear progression and dependencies +**CRITICAL: Phasing is NOT automatic. Check the user's input first.** -- Core user value delivery -- Essential user journeys -- Basic functionality that works reliably +Before proposing any phased approach, review the user's input documents: -**Phase 2: Growth** +- **If the input documents define all components as core requirements with no mention of phases:** Present all requirements as a single release scope. Do NOT invent phases or move requirements to fabricated future phases. +- **If the input documents explicitly request phased delivery:** Guide mapping of features across the phases the user defined. +- **If scope is unclear:** ASK the user whether they want phased delivery or a single release before proceeding. -- Additional user types -- Enhanced features -- Scale improvements +**When the user requests phased delivery**, guide mapping of features across the phases the user defines: -**Phase 3: Expansion** +- Use user-provided phase labels and count; if none are provided, propose a default (e.g., MVP/Growth/Vision) and ask for confirmation +- Ensure clear progression and dependencies between phases -- Advanced capabilities -- Platform features -- New markets or use cases +**Each phase should address:** -**Where does your current vision fit in this development sequence?**" +- Core user value delivery and essential journeys for that phase +- Clear boundaries on what ships in each phase +- Dependencies on prior phases + +**When the user chooses a single release**, define the complete scope: + +- All user-specified requirements are in scope +- Focus must-have vs nice-to-have analysis on what ships in this release +- Do NOT create phases β use must-have/nice-to-have priority within the single release + +**If phased delivery:** "Where does your current vision fit in this development sequence?" +**If single release:** "How does your current vision map to this upcoming release?" ### 5. Risk-Based Scoping @@ -129,6 +142,8 @@ Prepare comprehensive scoping section: #### Content Structure: +**If user chose phased delivery:** + ```markdown ## Project Scoping & Phased Development @@ -160,11 +175,39 @@ Prepare comprehensive scoping section: **Resource Risks:** {{contingency_approach}} ``` +**If user chose single release (no phasing):** + +```markdown +## Project Scoping + +### Strategy & Philosophy + +**Approach:** {{chosen_approach}} +**Resource Requirements:** {{team_size_and_skills}} + +### Complete Feature Set + +**Core User Journeys Supported:** +{{all_journeys}} + +**Must-Have Capabilities:** +{{list_of_must_have_features}} + +**Nice-to-Have Capabilities:** +{{list_of_nice_to_have_features}} + +### Risk Mitigation Strategy + +**Technical Risks:** {{mitigation_approach}} +**Market Risks:** {{validation_approach}} +**Resource Risks:** {{contingency_approach}} +``` + ### 7. Present MENU OPTIONS Present the scoping decisions for review, then display menu: - Show strategic scoping plan (using structure from step 6) -- Highlight MVP boundaries and phased roadmap +- Highlight release boundaries and prioritization (phased roadmap only if phased delivery was selected) - Ask if they'd like to refine further, get other perspectives, or proceed - Present menu options naturally as part of conversation @@ -173,7 +216,7 @@ Display: "**Select:** [A] Advanced Elicitation [P] Party Mode [C] Continue to Fu #### Menu Handling Logic: - IF A: Invoke the `bmad-advanced-elicitation` skill with the current scoping analysis, process the enhanced insights that come back, ask user if they accept the improvements, if yes update content then redisplay menu, if no keep original content then redisplay menu - IF P: Invoke the `bmad-party-mode` skill with the scoping context, process the collaborative insights on MVP and roadmap decisions, ask user if they accept the changes, if yes update content then redisplay menu, if no keep original content then redisplay menu -- IF C: Append the final content to {outputFile}, update frontmatter by adding this step name to the end of the stepsCompleted array, then read fully and follow: ./step-09-functional.md +- IF C: Append the final content to {outputFile}, update frontmatter by adding this step name to the end of the stepsCompleted array (also add `releaseMode: phased` or `releaseMode: single-release` to frontmatter based on user's choice), then read fully and follow: ./step-09-functional.md - IF Any other: help user respond, then redisplay menu #### EXECUTION RULES: @@ -189,8 +232,9 @@ When user selects 'C', append the content directly to the document using the str β Complete PRD document analyzed for scope implications β Strategic MVP approach defined and justified -β Clear MVP feature boundaries established -β Phased development roadmap created +β Clear feature boundaries established (phased or single-release, per user preference) +β All user-specified requirements accounted for β none silently removed or deferred +β Any scope reduction recommendations presented to user with rationale and explicit confirmation obtained β Key risks identified and mitigation strategies defined β User explicitly agrees to scope decisions β A/P/C menu presented and handled correctly @@ -202,8 +246,11 @@ When user selects 'C', append the content directly to the document using the str β Making scope decisions without strategic rationale β Not getting explicit user agreement on MVP boundaries β Missing critical risk analysis -β Not creating clear phased development approach β Not presenting A/P/C menu after content generation +β **CRITICAL**: Silently de-scoping or deferring requirements that the user explicitly included in their input documents +β **CRITICAL**: Inventing phasing (MVP/Growth/Vision) when the user did not request phased delivery +β **CRITICAL**: Making consequential scoping decisions (what is in/out of scope) without explicit user confirmation +β **CRITICAL**: Creating phase-based artifacts (tags, labels, follow-on prompts) without explicit user approval β **CRITICAL**: Reading only partial step file - leads to incomplete understanding and poor decisions β **CRITICAL**: Proceeding with 'C' without fully reading and understanding the next step file diff --git a/src/bmm-skills/2-plan-workflows/bmad-create-prd/steps-c/step-11-polish.md b/src/bmm-skills/2-plan-workflows/bmad-create-prd/steps-c/step-11-polish.md index c63ae5b29..6d33abd5c 100644 --- a/src/bmm-skills/2-plan-workflows/bmad-create-prd/steps-c/step-11-polish.md +++ b/src/bmm-skills/2-plan-workflows/bmad-create-prd/steps-c/step-11-polish.md @@ -138,7 +138,7 @@ Make targeted improvements: - All user success criteria - All functional requirements (capability contract) - All user journey narratives -- All scope decisions (MVP, Growth, Vision) +- All scope decisions (whether phased or single-release), including consent-critical evidence (explicit user confirmations and rationales for any scope changes from step 8) - All non-functional requirements - Product differentiator and vision - Domain-specific requirements diff --git a/src/core-skills/module.yaml b/src/core-skills/module.yaml index 48e7a58f7..5ac3cd887 100644 --- a/src/core-skills/module.yaml +++ b/src/core-skills/module.yaml @@ -1,5 +1,6 @@ code: core name: "BMad Core Module" +description: "Core configuration and shared resources" header: "BMad Core Configuration" subheader: "Configure the core settings for your BMad installation.\nThese settings will be used across all installed bmad skills, workflows, and agents." diff --git a/test/test-installation-components.js b/test/test-installation-components.js index 10639bab8..f1c1be486 100644 --- a/test/test-installation-components.js +++ b/test/test-installation-components.js @@ -13,7 +13,7 @@ const path = require('node:path'); const os = require('node:os'); -const fs = require('fs-extra'); +const fs = require('../tools/installer/fs-native'); const { Installer } = require('../tools/installer/core/installer'); const { ManifestGenerator } = require('../tools/installer/core/manifest-generator'); const { OfficialModules } = require('../tools/installer/modules/official-modules'); diff --git a/tools/installer/commands/status.js b/tools/installer/commands/status.js index 49c0afd73..c7f4a816c 100644 --- a/tools/installer/commands/status.js +++ b/tools/installer/commands/status.js @@ -19,7 +19,7 @@ module.exports = { const { bmadDir } = await installer.findBmadDir(projectDir); // Check if bmad directory exists - const fs = require('fs-extra'); + const fs = require('../fs-native'); if (!(await fs.pathExists(bmadDir))) { await prompts.log.warn('No BMAD installation found in the current directory.'); await prompts.log.message(`Expected location: ${bmadDir}`); diff --git a/tools/installer/commands/uninstall.js b/tools/installer/commands/uninstall.js index d0e168a15..727b7b0ef 100644 --- a/tools/installer/commands/uninstall.js +++ b/tools/installer/commands/uninstall.js @@ -1,5 +1,5 @@ const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('../fs-native'); const prompts = require('../prompts'); const { Installer } = require('../core/installer'); diff --git a/tools/installer/core/existing-install.js b/tools/installer/core/existing-install.js index 643f1d946..6bbf191d1 100644 --- a/tools/installer/core/existing-install.js +++ b/tools/installer/core/existing-install.js @@ -1,5 +1,5 @@ const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('../fs-native'); const yaml = require('yaml'); const { Manifest } = require('./manifest'); diff --git a/tools/installer/core/install-paths.js b/tools/installer/core/install-paths.js index f1c50ee43..e7fb98b6d 100644 --- a/tools/installer/core/install-paths.js +++ b/tools/installer/core/install-paths.js @@ -1,5 +1,5 @@ const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('../fs-native'); const { getProjectRoot } = require('../project-root'); const { BMAD_FOLDER_NAME } = require('../ide/shared/path-utils'); diff --git a/tools/installer/core/installer.js b/tools/installer/core/installer.js index 95e16adfe..2a9ff3272 100644 --- a/tools/installer/core/installer.js +++ b/tools/installer/core/installer.js @@ -1,5 +1,5 @@ const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('../fs-native'); const { Manifest } = require('./manifest'); const { OfficialModules } = require('../modules/official-modules'); const { IdeManager } = require('../ide/manager'); diff --git a/tools/installer/core/manifest-generator.js b/tools/installer/core/manifest-generator.js index 13e33af56..477142888 100644 --- a/tools/installer/core/manifest-generator.js +++ b/tools/installer/core/manifest-generator.js @@ -1,5 +1,5 @@ const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('../fs-native'); const yaml = require('yaml'); const crypto = require('node:crypto'); const csv = require('csv-parse/sync'); diff --git a/tools/installer/core/manifest.js b/tools/installer/core/manifest.js index aaa86649a..2dc94ae9f 100644 --- a/tools/installer/core/manifest.js +++ b/tools/installer/core/manifest.js @@ -1,5 +1,5 @@ const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('../fs-native'); const crypto = require('node:crypto'); const { getProjectRoot } = require('../project-root'); const prompts = require('../prompts'); diff --git a/tools/installer/file-ops.js b/tools/installer/file-ops.js index 5cd7970d8..2a2869930 100644 --- a/tools/installer/file-ops.js +++ b/tools/installer/file-ops.js @@ -1,4 +1,4 @@ -const fs = require('fs-extra'); +const fs = require('./fs-native'); const path = require('node:path'); const crypto = require('node:crypto'); diff --git a/tools/installer/fs-native.js b/tools/installer/fs-native.js new file mode 100644 index 000000000..b6a4abfa5 --- /dev/null +++ b/tools/installer/fs-native.js @@ -0,0 +1,111 @@ +// Drop-in replacement for fs-extra using native node:fs APIs. +// Eliminates graceful-fs monkey-patching that causes non-deterministic +// file loss during multi-module installs on macOS (issue #1779). +const fsp = require('node:fs/promises'); +const fs = require('node:fs'); +const path = require('node:path'); + +async function pathExists(p) { + try { + await fsp.access(p); + return true; + } catch { + return false; + } +} + +async function ensureDir(dir) { + await fsp.mkdir(dir, { recursive: true }); +} + +async function remove(p) { + await fsp.rm(p, { recursive: true, force: true }); +} + +async function copy(src, dest, options = {}) { + const filterFn = options.filter; + const overwrite = options.overwrite !== false; + const srcStat = await fsp.stat(src); + + if (srcStat.isFile()) { + if (filterFn && !(await filterFn(src, dest))) return; + await fsp.mkdir(path.dirname(dest), { recursive: true }); + if (!overwrite) { + try { + await fsp.access(dest); + if (options.errorOnExist) throw new Error(`${dest} already exists`); + return; + } catch (error) { + if (error.message.includes('already exists')) throw error; + } + } + await fsp.copyFile(src, dest); + return; + } + + if (srcStat.isDirectory()) { + if (filterFn && !(await filterFn(src, dest))) return; + await fsp.mkdir(dest, { recursive: true }); + const entries = await fsp.readdir(src, { withFileTypes: true }); + for (const entry of entries) { + await copy(path.join(src, entry.name), path.join(dest, entry.name), options); + } + } +} + +async function move(src, dest) { + try { + await fsp.rename(src, dest); + } catch (error) { + if (error.code === 'EXDEV') { + await copy(src, dest); + await fsp.rm(src, { recursive: true, force: true }); + } else { + throw error; + } + } +} + +function readJsonSync(p) { + return JSON.parse(fs.readFileSync(p, 'utf8')); +} + +async function writeJson(p, data, options = {}) { + const spaces = options.spaces ?? 2; + await fsp.writeFile(p, JSON.stringify(data, null, spaces) + '\n', 'utf8'); +} + +module.exports = { + // Native async (node:fs/promises) + readFile: fsp.readFile, + writeFile: fsp.writeFile, + stat: fsp.stat, + readdir: fsp.readdir, + access: fsp.access, + rename: fsp.rename, + unlink: fsp.unlink, + chmod: fsp.chmod, + mkdir: fsp.mkdir, + mkdtemp: fsp.mkdtemp, + copyFile: fsp.copyFile, + rm: fsp.rm, + + // fs-extra compatible helpers (native implementations) + pathExists, + ensureDir, + remove, + copy, + move, + readJsonSync, + writeJson, + + // Sync methods from core node:fs + existsSync: fs.existsSync.bind(fs), + readFileSync: fs.readFileSync.bind(fs), + writeFileSync: fs.writeFileSync.bind(fs), + createReadStream: fs.createReadStream.bind(fs), + pathExistsSync: fs.existsSync.bind(fs), + + // Constants + constants: fs.constants, +}; diff --git a/tools/installer/ide/_config-driven.js b/tools/installer/ide/_config-driven.js index 9c7df4bc5..563818f67 100644 --- a/tools/installer/ide/_config-driven.js +++ b/tools/installer/ide/_config-driven.js @@ -1,6 +1,6 @@ const os = require('node:os'); const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('../fs-native'); const yaml = require('yaml'); const prompts = require('../prompts'); const csv = require('csv-parse/sync'); diff --git a/tools/installer/ide/platform-codes.js b/tools/installer/ide/platform-codes.js index 32d82e9cc..f29be8fcb 100644 --- a/tools/installer/ide/platform-codes.js +++ b/tools/installer/ide/platform-codes.js @@ -1,4 +1,4 @@ -const fs = require('fs-extra'); +const fs = require('../fs-native'); const path = require('node:path'); const yaml = require('yaml'); diff --git a/tools/installer/ide/shared/skill-manifest.js b/tools/installer/ide/shared/skill-manifest.js index 746d5d16f..1dfc7eb35 100644 --- a/tools/installer/ide/shared/skill-manifest.js +++ b/tools/installer/ide/shared/skill-manifest.js @@ -1,5 +1,5 @@ const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('../../fs-native'); const yaml = require('yaml'); /** diff --git a/tools/installer/message-loader.js b/tools/installer/message-loader.js index 03ba7eca1..97f02d6e4 100644 --- a/tools/installer/message-loader.js +++ b/tools/installer/message-loader.js @@ -1,4 +1,4 @@ -const fs = require('fs-extra'); +const fs = require('./fs-native'); const path = require('node:path'); const yaml = require('yaml'); const prompts = require('./prompts'); diff --git a/tools/installer/modules/community-manager.js b/tools/installer/modules/community-manager.js index 0f88cffff..3e0217688 100644 --- a/tools/installer/modules/community-manager.js +++ b/tools/installer/modules/community-manager.js @@ -1,4 +1,4 @@ -const fs = require('fs-extra'); +const fs = require('../fs-native'); const os = require('node:os'); const path = require('node:path'); const { execSync } = require('node:child_process'); diff --git a/tools/installer/modules/custom-module-manager.js b/tools/installer/modules/custom-module-manager.js index e0f8b7085..482c4dc43 100644 --- a/tools/installer/modules/custom-module-manager.js +++ b/tools/installer/modules/custom-module-manager.js @@ -1,4 +1,4 @@ -const fs = require('fs-extra'); +const fs = require('../fs-native'); const os = require('node:os'); const path = require('node:path'); const { execSync } = require('node:child_process'); diff --git a/tools/installer/modules/external-manager.js b/tools/installer/modules/external-manager.js index 0b8f5074c..5169ffb50 100644 --- a/tools/installer/modules/external-manager.js +++ b/tools/installer/modules/external-manager.js @@ -1,4 +1,4 @@ -const fs = require('fs-extra'); +const fs = require('../fs-native'); const os = require('node:os'); const path = require('node:path'); const { execSync } = require('node:child_process'); diff --git a/tools/installer/modules/official-modules.js b/tools/installer/modules/official-modules.js index 6158a7863..19dc0f4dc 100644 --- a/tools/installer/modules/official-modules.js +++ b/tools/installer/modules/official-modules.js @@ -1,5 +1,5 @@ const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('../fs-native'); const yaml = require('yaml'); const prompts = require('../prompts'); const { getProjectRoot, getSourcePath, getModulePath } = require('../project-root'); diff --git a/tools/installer/modules/plugin-resolver.js b/tools/installer/modules/plugin-resolver.js index 9fbf325a2..58e20ab88 100644 --- a/tools/installer/modules/plugin-resolver.js +++ b/tools/installer/modules/plugin-resolver.js @@ -1,4 +1,4 @@ -const fs = require('fs-extra'); +const fs = require('../fs-native'); const path = require('node:path'); const yaml = require('yaml'); diff --git a/tools/installer/project-root.js b/tools/installer/project-root.js index 26063f81f..037f1a430 100644 --- a/tools/installer/project-root.js +++ b/tools/installer/project-root.js @@ -1,5 +1,5 @@ const path = require('node:path'); -const fs = require('fs-extra'); +const fs = require('./fs-native'); /** * Find the BMAD project root directory by looking for package.json diff --git a/tools/installer/ui.js b/tools/installer/ui.js index 527708494..d1c5189e9 100644 --- a/tools/installer/ui.js +++ b/tools/installer/ui.js @@ -1,6 +1,6 @@ const path = require('node:path'); const os = require('node:os'); -const fs = require('fs-extra'); +const fs = require('./fs-native'); const { CLIUtils } = require('./cli-utils'); const { ExternalModuleManager } = require('./modules/external-manager'); const { getProjectRoot } = require('./project-root'); @@ -598,7 +598,7 @@ class UI { const officialCodes = new Set(officialSelected); const externalManager = new ExternalModuleManager(); const registryModules = await externalManager.listAvailable(); - const officialRegistryCodes = new Set(registryModules.map((m) => m.code)); + const officialRegistryCodes = new Set(['core', 'bmm', ...registryModules.map((m) => m.code)]); const installedNonOfficial = [...installedModuleIds].filter((id) => !officialRegistryCodes.has(id)); // Phase 2: Community modules (category drill-down) @@ -630,6 +630,11 @@ class UI { * @returns {Array} Selected official module codes */ async _selectOfficialModules(installedModuleIds = new Set()) { + // Built-in modules (core, bmm) come from local source, not the registry + const { OfficialModules } = require('./modules/official-modules'); + const builtInModules = (await new OfficialModules().listAvailable()).modules || []; + + // External modules come from the registry (with fallback) const externalManager = new ExternalModuleManager(); const registryModules = await externalManager.listAvailable(); @@ -637,20 +642,34 @@ class UI { const initialValues = []; const lockedValues = ['core']; - const buildModuleEntry = async (mod) => { - const isInstalled = installedModuleIds.has(mod.code); - const version = await getMarketplaceVersion(mod.code); - const label = version ? `${mod.name} (v${version})` : mod.name; + const buildModuleEntry = async (code, name, description, isDefault) => { + const isInstalled = installedModuleIds.has(code); + const version = await getMarketplaceVersion(code); + const label = version ? `${name} (v${version})` : name; return { label, - value: mod.code, - hint: mod.description, - selected: isInstalled, + value: code, + hint: description, + selected: isInstalled || isDefault, }; }; + // Add built-in modules first (always available regardless of network) + const builtInCodes = new Set(); + for (const mod of builtInModules) { + const code = mod.id; + builtInCodes.add(code); + const entry = await buildModuleEntry(code, mod.name, mod.description, mod.defaultSelected); + allOptions.push({ label: entry.label, value: entry.value, hint: entry.hint }); + if (entry.selected) { + initialValues.push(code); + } + } + + // Add external registry modules (skip built-in duplicates) for (const mod of registryModules) { - const entry = await buildModuleEntry(mod); + if (mod.builtIn || builtInCodes.has(mod.code)) continue; + const entry = await buildModuleEntry(mod.code, mod.name, mod.description, mod.defaultSelected); allOptions.push({ label: entry.label, value: entry.value, hint: entry.hint }); if (entry.selected) { initialValues.push(mod.code); @@ -1122,12 +1141,26 @@ class UI { * @returns {Array} Default module codes */ async getDefaultModules(installedModuleIds = new Set()) { + // Built-in modules with default_selected come from local source + const { OfficialModules } = require('./modules/official-modules'); + const builtInModules = (await new OfficialModules().listAvailable()).modules || []; + + const defaultModules = []; + const seen = new Set(); + + for (const mod of builtInModules) { + if (mod.defaultSelected || installedModuleIds.has(mod.id)) { + defaultModules.push(mod.id); + seen.add(mod.id); + } + } + + // Add external registry defaults const externalManager = new ExternalModuleManager(); const registryModules = await externalManager.listAvailable(); - const defaultModules = []; - for (const mod of registryModules) { + if (mod.builtIn || seen.has(mod.code)) continue; if (mod.defaultSelected || installedModuleIds.has(mod.code)) { defaultModules.push(mod.code); } diff --git a/tools/migrate-custom-module-paths.js b/tools/migrate-custom-module-paths.js index 13aa3e710..b199e8bfe 100755 --- a/tools/migrate-custom-module-paths.js +++ b/tools/migrate-custom-module-paths.js @@ -3,7 +3,7 @@ * This should be run once to update existing installations */ -const fs = require('fs-extra'); +const fs = require('./installer/fs-native'); const path = require('node:path'); const yaml = require('yaml'); const chalk = require('chalk'); diff --git a/website/public/workflow-map-diagram-fr.html b/website/public/workflow-map-diagram-fr.html index bc59f23a9..1fde3c038 100644 --- a/website/public/workflow-map-diagram-fr.html +++ b/website/public/workflow-map-diagram-fr.html @@ -93,7 +93,6 @@ .agent-icon.john { background: linear-gradient(135deg, #60a5fa, #3b82f6); } .agent-icon.sally { background: linear-gradient(135deg, #fbbf24, #f59e0b); color: #000; } .agent-icon.winston { background: linear-gradient(135deg, #a78bfa, #8b5cf6); } - .agent-icon.bob { background: linear-gradient(135deg, #34d399, #10b981); color: #000; } .agent-icon.amelia { background: linear-gradient(135deg, #fb7185, #ef4444); } .agent-name { font-size: 0.65rem; } @@ -261,7 +260,7 @@ sprint-planning
@@ -270,7 +269,7 @@ create-story @@ -308,7 +307,7 @@ par Epic diff --git a/website/public/workflow-map-diagram.html b/website/public/workflow-map-diagram.html index 897492700..0a17cc2eb 100644 --- a/website/public/workflow-map-diagram.html +++ b/website/public/workflow-map-diagram.html @@ -93,7 +93,6 @@ .agent-icon.john { background: linear-gradient(135deg, #60a5fa, #3b82f6); } .agent-icon.sally { background: linear-gradient(135deg, #fbbf24, #f59e0b); color: #000; } .agent-icon.winston { background: linear-gradient(135deg, #a78bfa, #8b5cf6); } - .agent-icon.bob { background: linear-gradient(135deg, #34d399, #10b981); color: #000; } .agent-icon.amelia { background: linear-gradient(135deg, #fb7185, #ef4444); } .agent-name { font-size: 0.65rem; } @@ -272,7 +271,7 @@ sprint-planning @@ -281,7 +280,7 @@ create-story @@ -319,7 +318,7 @@ per epic