klinsmaya/workdsh
1
0
Fork 0
mirror of https://gh-proxy.org/https://github.com/klinsmaya/workdsh.git synced 2026-11-08 01:17:05 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(skills): align installed validation with upstream loader

Browse Source
This commit is contained in:
techflag
2026-09-26 17:52:44 +08:00
parent dd5d1ca003
commit bc73b8d997
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
workdsh-web/packages/plugins/skills/src/services/manager.ts
+2 -2
View File
@@ -293,7 +293,7 @@ export class SkillManager extends Service implements SkillManagementService {
validateDocument(document: string, expectedName?: string): SkillValidationResult {
const diagnostics: SkillDiagnostic[] = [];
if (Buffer.byteLength(document) > maximumDocumentBytes) diagnostics.push({ code: 'document-too-large', message: 'SKILL.md 超过 1 MiB 上限。', path: 'SKILL.md' });
if (expectedName && Buffer.byteLength(document) > maximumDocumentBytes) diagnostics.push({ code: 'document-too-large', message: 'SKILL.md 超过 1 MiB 上限。', path: 'SKILL.md' });
let metadata: Record<string, unknown> | undefined;
try { metadata = frontmatter(document); }
catch { diagnostics.push({ code: 'invalid-frontmatter', message: 'SKILL.md 必须以有效的 YAML frontmatter 开头。', path: 'SKILL.md' }); }
@@ -305,7 +305,7 @@ export class SkillManager extends Service implements SkillManagementService {
if (expectedName && name && name !== expectedName) diagnostics.push({ code: 'name-mismatch', message: `frontmatter name 必须是 ${expectedName}。`, path: 'SKILL.md' });
if (!description) diagnostics.push({ code: 'description-required', message: 'description 不能为空。', path: 'SKILL.md' });
const body = metadata ? document.replace(/^---\s*\r?\n[\s\S]*?\r?\n---(?:\r?\n|$)/, '').trim() : '';
if (metadata && !body) diagnostics.push({ code: 'instructions-required', message: 'frontmatter 后必须包含可执行的技能说明。', path: 'SKILL.md' });
if (expectedName && metadata && !body) diagnostics.push({ code: 'instructions-required', message: 'frontmatter 后必须包含可执行的技能说明。', path: 'SKILL.md' });
return { valid: diagnostics.length === 0, ...(name ? { name } : {}), ...(description ? { description } : {}), diagnostics };
}
workdsh-web/tests/integration/skill-manager.test.mjs
+1 -1
View File
@@ -186,7 +186,7 @@ test('skill manager surfaces invalid local skills with actionable diagnostics',
const summary = (await ctx.workdshSkills.list()).find(row => row.name === 'broken-skill');
assert.equal(summary.state, 'invalid');
assert.equal(summary.modelInvocable, false);
assert.deepEqual(summary.diagnostics.map(item => item.code), ['description-required', 'instructions-required']);
assert.deepEqual(summary.diagnostics.map(item => item.code), ['description-required']);
const detail = await ctx.workdshSkills.detail('broken-skill');
assert.equal(detail.state, 'invalid');
assert.match(detail.diagnostics[0].message, /description/);