marketing-shibata50/multica
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(skills): preserve all user files during bundled skill upgrades

Browse source 
Instead of only protecting .env files, use cpSync with force:true to
overlay bundle files onto the existing directory. This preserves any
user-created files (credentials.json, token.json, etc.) that don't
exist in the bundle, rather than deleting and re-copying the entire
directory.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Jiayuan Zhang 2026-02-15 14:15:31 +08:00
parent 8848f09107
commit a4b7deac3e
1 changed files with 5 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

22
packages/core/src/agent/skills/loader.ts
View file

@ -198,8 +198,8 @@ export function initializeManagedSkills(): void {
// Check if skill exists in managed
if (!existsSync(dest)) {
// Skill doesn't exist, copy it (skip .env files - those are user-specific)
cpSync(src, dest, { recursive: true, dereference: true, filter: (s) => !s.endsWith("/.env") });
// Skill doesn't exist, copy it as-is
cpSync(src, dest, { recursive: true, dereference: true });
continue;
}
@ -214,21 +214,9 @@ export function initializeManagedSkills(): void {
// Update if bundled version is higher
if (compareVersions(bundledVersion, managedVersion) > 0) {
// Preserve user's .env file across upgrades
const envPath = join(dest, ".env");
let savedEnv: string | null = null;
if (existsSync(envPath)) {
try { savedEnv = readFileSync(envPath, "utf-8"); } catch { /* ignore */ }
}
// Remove old and copy new (skip .env from bundle)
rmSync(dest, { recursive: true });
cpSync(src, dest, { recursive: true, dereference: true, filter: (s) => !s.endsWith("/.env") });
// Restore user's .env
if (savedEnv !== null) {
writeFileSync(envPath, savedEnv, "utf-8");
}
// Overwrite only files that exist in the bundle, preserving
// user-created files (e.g. .env, credentials.json, token.json)
cpSync(src, dest, { recursive: true, dereference: true, force: true });
}
}