From bc73b8d9977d971f852004652b86b710cc29da3f Mon Sep 17 00:00:00 2001
From: techflag <562635045@qq.com>
Date: Sat, 26 Sep 2026 17:52:44 +0800
Subject: [PATCH] fix(skills): align installed validation with upstream loader

---
 workdsh-web/packages/plugins/skills/src/services/manager.ts | 4 ++--
 workdsh-web/tests/integration/skill-manager.test.mjs        | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/workdsh-web/packages/plugins/skills/src/services/manager.ts b/workdsh-web/packages/plugins/skills/src/services/manager.ts
index 79e38de5c4..6677ad3aa5 100644
--- a/workdsh-web/packages/plugins/skills/src/services/manager.ts
+++ b/workdsh-web/packages/plugins/skills/src/services/manager.ts
@@ -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 };
   }
 
diff --git a/workdsh-web/tests/integration/skill-manager.test.mjs b/workdsh-web/tests/integration/skill-manager.test.mjs
index 31c4e814f0..604b91d267 100644
--- a/workdsh-web/tests/integration/skill-manager.test.mjs
+++ b/workdsh-web/tests/integration/skill-manager.test.mjs
@@ -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/);