mirror of
https://github.com/bmad-code-org/BMAD-METHOD.git
synced 2026-09-19 08:11:52 +08:00
Brian Madison
a6d075bd0b
fs-extra routes all operations through graceful-fs, which globally monkey-patches node:fs with a deferred retry queue. During multi-module installs (~500+ file ops), retried unlink operations from one module's remove phase can fire after the next module's copy phase has written files, silently deleting them non-deterministically. Replace fs-extra with a thin fs-native.js wrapper over node:fs/promises and node:fs. All 21 consumers now use native APIs with no global monkey-patching, eliminating the retry-queue race condition entirely. Closes #1779
58 lines
2.3 KiB
JavaScript
58 lines
2.3 KiB
JavaScript
const path = require('node:path');
|
|
const fs = require('../../fs-native');
|
|
const yaml = require('yaml');
|
|
|
|
/**
|
|
* Load bmad-skill-manifest.yaml from a directory.
|
|
* Single-entry manifests (canonicalId at top level) apply to all files in the directory.
|
|
* Multi-entry manifests are keyed by source filename.
|
|
* @param {string} dirPath - Directory to check for bmad-skill-manifest.yaml
|
|
* @returns {Object|null} Parsed manifest or null
|
|
*/
|
|
async function loadSkillManifest(dirPath) {
|
|
const manifestPath = path.join(dirPath, 'bmad-skill-manifest.yaml');
|
|
try {
|
|
if (!(await fs.pathExists(manifestPath))) return null;
|
|
const content = await fs.readFile(manifestPath, 'utf8');
|
|
const parsed = yaml.parse(content);
|
|
if (!parsed || typeof parsed !== 'object') return null;
|
|
if (parsed.canonicalId || parsed.type) return { __single: parsed };
|
|
return parsed;
|
|
} catch (error) {
|
|
console.warn(`Warning: Failed to parse bmad-skill-manifest.yaml in ${dirPath}: ${error.message}`);
|
|
return null;
|
|
}
|
|
}
|
|
|
|
/**
|
|
* Get the canonicalId for a specific file from a loaded skill manifest.
|
|
* @param {Object|null} manifest - Loaded manifest (from loadSkillManifest)
|
|
* @param {string} filename - Source filename to look up (e.g., 'pm.md', 'help.md')
|
|
* @returns {string} canonicalId or empty string
|
|
*/
|
|
function getCanonicalId(manifest, filename) {
|
|
if (!manifest) return '';
|
|
// Single-entry manifest applies to all files in the directory
|
|
if (manifest.__single) return manifest.__single.canonicalId || '';
|
|
// Multi-entry: look up by filename directly
|
|
if (manifest[filename]) return manifest[filename].canonicalId || '';
|
|
return '';
|
|
}
|
|
|
|
/**
|
|
* Get the artifact type for a specific file from a loaded skill manifest.
|
|
* @param {Object|null} manifest - Loaded manifest (from loadSkillManifest)
|
|
* @param {string} filename - Source filename to look up
|
|
* @returns {string|null} type or null
|
|
*/
|
|
function getArtifactType(manifest, filename) {
|
|
if (!manifest) return null;
|
|
// Single-entry manifest applies to all files in the directory
|
|
if (manifest.__single) return manifest.__single.type || null;
|
|
// Multi-entry: look up by filename directly
|
|
if (manifest[filename]) return manifest[filename].type || null;
|
|
return null;
|
|
}
|
|
|
|
module.exports = { loadSkillManifest, getCanonicalId, getArtifactType };
|