modelcontextprotocol/mcpb
Watch
1
0
Fork
You've already forked mcpb
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: allow repeat runs of "dxt clean" (#83)

Browse source
This commit is contained in:
Felix Rieseberg 2025-07-31 22:07:28 +02:00 • committed by GitHub
commit bf01918080
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 20 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

25
src/node/validate.ts
View file

@ -70,10 +70,8 @@ export async function cleanDxt(inputPath: string) {
await unpackExtension({ dxtPath, silent: true, outputDir: unpackPath });
const manifestPath = resolve(unpackPath, "manifest.json");
const originalManifest = await fs.readFile(manifestPath, "utf-8");
const manifestData = JSON.parse(originalManifest);
const result = LooseDxtManifestSchema.safeParse(manifestData);
if (!result.success) {
@ -94,14 +92,31 @@ export async function cleanDxt(inputPath: string) {
const nodeModulesPath = resolve(unpackPath, "node_modules");
if (existsSync(nodeModulesPath)) {
console.log(" -- node_modules found, running galactus");
console.log(" -- node_modules found, deleting development dependencies");
const destroyer = new DestroyerOfModules({
rootDirectory: unpackPath,
});
await destroyer.destroy();
console.log(" -- Galactus pruned node_modules");
try {
await destroyer.destroy();
} catch (error) {
// If modules have already been deleted in a previous clean, the walker
// will fail when it can't find required dependencies. This is expected
// and safe to ignore.
if (
error instanceof Error &&
error.message.includes("Failed to locate module")
) {
console.log(
" -- Some modules already removed, skipping remaining cleanup",
);
} else {
throw error;
}
}
console.log(" -- Removed development dependencies from node_modules");
} else {
console.log(" -- No node_modules, not pruning");
}