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: harden install XML guard and remove no-op placeholder replacement

This commit is contained in:
Dicky Moore 2026-02-07 17:37:51 +00:00
parent 87c4292021
commit 454ae910f9
2 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
test/test-installation-components.js
View File

@ -32,8 +32,9 @@ let failed = 0;
/**
* Recursively collect files from a mix of files/directories.
*/
async function collectFiles(targets, allowedExtensions) {
async function collectFiles(targets, allowedExtensions, excludedFiles = new Set()) {
const files = [];
const normalizedExcludes = new Set([...excludedFiles].map((p) => path.resolve(p)));
const walk = async (targetPath) => {
if (!(await fs.pathExists(targetPath))) {
@ -42,6 +43,10 @@ async function collectFiles(targets, allowedExtensions) {
const stat = await fs.stat(targetPath);
if (stat.isFile()) {
const normalizedTargetPath = path.resolve(targetPath);
if (normalizedExcludes.has(normalizedTargetPath)) {
return;
}
if (allowedExtensions.has(path.extname(targetPath))) {
files.push(targetPath);
}
@ -58,6 +63,9 @@ async function collectFiles(targets, allowedExtensions) {
await walk(fullPath);
continue;
}
if (normalizedExcludes.has(path.resolve(fullPath))) {
continue;
}
if (allowedExtensions.has(path.extname(entry.name))) {
files.push(fullPath);
}
@ -273,9 +281,10 @@ async function runTests() {
];
const allowedExtensions = new Set(['.md', '.yaml', '.yml', '.xml']);
const forbiddenRef = 'validate-workflow.xml';
const excludedFile = path.join(projectRoot, 'src', 'core', 'tasks', 'validate-workflow.xml');
const offenders = [];
const files = await collectFiles(searchTargets, allowedExtensions);
const files = await collectFiles(searchTargets, allowedExtensions, new Set([excludedFile]));
for (const fullPath of files) {
const content = await fs.readFile(fullPath, 'utf8');
if (content.includes(forbiddenRef)) {

3
tools/cli/installers/lib/ide/shared/agent-command-generator.js
View File

@ -79,8 +79,7 @@ class AgentCommandGenerator {
.replaceAll('{{module}}', agent.module)
.replaceAll('{{path}}', agentPathInModule)
.replaceAll('{{description}}', agent.description || `${agent.name} agent`)
.replaceAll('_bmad', this.bmadFolderName)
.replaceAll('_bmad', '_bmad');
.replaceAll('_bmad', this.bmadFolderName);
}
/**