惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

Y
Y Combinator Blog
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
博客园_首页
量子位
V
Visual Studio Blog
博客园 - Franky
宝玉的分享
宝玉的分享
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
博客园 - 【当耐特】
罗磊的独立博客
小众软件
小众软件
V
V2EX
GbyAI
GbyAI
B
Blog RSS Feed
博客园 - 三生石上(FineUI控件)
大猫的无限游戏
大猫的无限游戏
有赞技术团队
有赞技术团队
月光博客
月光博客
Recent Announcements
Recent Announcements
雷峰网
雷峰网
F
Fortinet All Blogs
M
MIT News - Artificial intelligence

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(skills): sync managed symlink skills as directories ·...
steipete · 2026-05-15 · via Recent Commits to openclaw:main

@@ -154,10 +154,13 @@ type ResolvedSkillsLimits = {

154154

type LoadedSkillRecord = {

155155

skill: Skill;

156156

frontmatter?: ParsedSkillFrontmatter;

157+

syncSourceDir?: string;

158+

syncDirName?: string;

157159

};

158160159161

type CandidateSkillDir = {

160162

skillDir: string;

163+

skillDirRealPath: string;

161164

name: string;

162165

skillMdRealPath: string;

163166

};

@@ -388,16 +391,53 @@ function loadContainedSkillRecords(params: {

388391

skillDir: string;

389392

source: string;

390393

maxSkillFileBytes: number;

394+

canonicalSkillDir?: string;

391395

}): LoadedSkillRecord[] {

392396

const expectedBaseDir = path.resolve(params.skillDir);

393397

const loaded = loadSkillsFromDirSafe({

394398

dir: params.skillDir,

395399

source: params.source,

396400

maxBytes: params.maxSkillFileBytes,

397401

});

398-

return unwrapLoadedSkillRecords(loaded).filter(

402+

const records = unwrapLoadedSkillRecords(loaded).filter(

399403

(record) => path.resolve(record.skill.baseDir) === expectedBaseDir,

400404

);

405+

const canonicalSkillDir = params.canonicalSkillDir;

406+

return canonicalSkillDir

407+

? records.map((record) => canonicalizeLoadedSkillRecord(record, canonicalSkillDir))

408+

: records;

409+

}

410+411+

function canonicalizeLoadedSkillRecord(

412+

record: LoadedSkillRecord,

413+

canonicalSkillDir: string,

414+

): LoadedSkillRecord {

415+

const originalBaseDir = path.resolve(record.skill.baseDir);

416+

const canonicalBaseDir = path.resolve(canonicalSkillDir);

417+

if (originalBaseDir === canonicalBaseDir) {

418+

return record;

419+

}

420+

const filePath = path.join(

421+

canonicalBaseDir,

422+

path.relative(originalBaseDir, record.skill.filePath),

423+

);

424+

return {

425+

...record,

426+

syncSourceDir: canonicalBaseDir,

427+

syncDirName: path.basename(originalBaseDir),

428+

skill: {

429+

...record.skill,

430+

filePath,

431+

baseDir: canonicalBaseDir,

432+

sourceInfo: record.skill.sourceInfo

433+

? {

434+

...record.skill.sourceInfo,

435+

path: filePath,

436+

baseDir: canonicalBaseDir,

437+

}

438+

: record.skill.sourceInfo,

439+

},

440+

};

401441

}

402442403443

function isPathInsideAnyRoot(rootRealPaths: readonly string[], candidateRealPath: string): boolean {

@@ -437,6 +477,10 @@ function resolveSkillRootCandidatePath(params: {

437477

});

438478

}

439479480+

function canonicalSkillDirForSource(source: string, skillDirRealPath: string): string | undefined {

481+

return shouldEnforceConfiguredSkillRootContainment(source) ? undefined : skillDirRealPath;

482+

}

483+440484

function resolveSkillFilePath(params: {

441485

source: string;

442486

skillDir: string;

@@ -624,6 +668,7 @@ function loadSkillEntries(

624668

skillDir: baseDir,

625669

source: params.source,

626670

maxSkillFileBytes: limits.maxSkillFileBytes,

671+

canonicalSkillDir: canonicalSkillDirForSource(params.source, baseDirRealPath),

627672

});

628673

}

629674

@@ -658,7 +703,12 @@ function loadSkillEntries(

658703

}

659704660705

const loadedSkills: LoadedSkillRecord[] = [];

661-

const loadCandidateSkill = ({ skillDir, name, skillMdRealPath }: CandidateSkillDir) => {

706+

const loadCandidateSkill = ({

707+

skillDir,

708+

skillDirRealPath,

709+

name,

710+

skillMdRealPath,

711+

}: CandidateSkillDir) => {

662712

try {

663713

const size = fs.statSync(skillMdRealPath).size;

664714

if (size > limits.maxSkillFileBytes) {

@@ -679,6 +729,7 @@ function loadSkillEntries(

679729

skillDir,

680730

source: params.source,

681731

maxSkillFileBytes: limits.maxSkillFileBytes,

732+

canonicalSkillDir: canonicalSkillDirForSource(params.source, skillDirRealPath),

682733

}),

683734

);

684735

};

@@ -707,7 +758,7 @@ function loadSkillEntries(

707758

candidatePath: skillMd,

708759

});

709760

if (skillMdRealPath) {

710-

loadCandidateSkill({ skillDir, name, skillMdRealPath });

761+

loadCandidateSkill({ skillDir, skillDirRealPath, name, skillMdRealPath });

711762

}

712763

} else {

713764

// No SKILL.md here — check one level deeper for grouped skill directories.

@@ -764,6 +815,7 @@ function loadSkillEntries(

764815

if (nestedDirRealPath && nestedSkillMdRealPath) {

765816

loadCandidateSkill({

766817

skillDir: nestedDir,

818+

skillDirRealPath: nestedDirRealPath,

767819

name: `${name}/${nestedName}`,

768820

skillMdRealPath: nestedSkillMdRealPath,

769821

});

@@ -884,6 +936,8 @@ function loadSkillEntries(

884936

frontmatter,

885937

metadata: resolveOpenClawMetadata(frontmatter),

886938

invocation,

939+

...(record.syncSourceDir !== undefined ? { syncSourceDir: record.syncSourceDir } : {}),

940+

...(record.syncDirName !== undefined ? { syncDirName: record.syncDirName } : {}),

887941

exposure: {

888942

includeInRuntimeRegistry: true,

889943

// Freshly loaded entries preserve the documented disable-model-invocation

@@ -1169,7 +1223,9 @@ function resolveSyncedSkillDestinationPath(params: {

11691223

entry: SkillEntry;

11701224

usedDirNames: Set<string>;

11711225

}): string | null {

1172-

const sourceDirName = path.basename(params.entry.skill.baseDir).trim();

1226+

const sourceDirName = (

1227+

params.entry.syncDirName ?? path.basename(params.entry.skill.baseDir)

1228+

).trim();

11731229

if (!sourceDirName || sourceDirName === "." || sourceDirName === "..") {

11741230

return null;

11751231

}

@@ -1233,7 +1289,7 @@ export async function syncSkillsToWorkspace(params: {

12331289

continue;

12341290

}

12351291

try {

1236-

await fsp.cp(entry.skill.baseDir, dest, {

1292+

await fsp.cp(entry.syncSourceDir ?? entry.skill.baseDir, dest, {

12371293

recursive: true,

12381294

force: true,

12391295

filter: (src) => {