From 423291247a747d6e0b7aea8d9c81dae2e0796db9 Mon Sep 17 00:00:00 2001
From: ddCat <2695092045@qq.com>
Date: Tue, 22 Sep 2026 12:55:15 +0800
Subject: [PATCH] =?UTF-8?q?fix(skills):=20=E5=90=88=E5=B9=B6=E5=85=BC?=
 =?UTF-8?q?=E5=AE=B9=20-=20=E6=9B=BF=E6=8D=A2=E5=B7=B2=E5=88=A0=E9=99=A4?=
 =?UTF-8?q?=E7=A7=8D=E5=AD=90=E5=9B=9E=E5=BD=92=E5=A4=B9=E5=85=B7?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../internal/skills/skill_packages_test.go    | 31 +++++++++----------
 1 file changed, 14 insertions(+), 17 deletions(-)

diff --git a/backend/internal/skills/skill_packages_test.go b/backend/internal/skills/skill_packages_test.go
index 240fe3e13..8dcf3568c 100644
--- a/backend/internal/skills/skill_packages_test.go
+++ b/backend/internal/skills/skill_packages_test.go
@@ -3,7 +3,6 @@ package skills
 import (
 	"archive/zip"
 	"bytes"
-	"encoding/json"
 	"strings"
 	"testing"
 
@@ -28,25 +27,23 @@ func TestArchiveFromMarkdownInfersMetadata(t *testing.T) {
 	}
 }
 
-func TestReportedBuiltinSkillPackage(t *testing.T) {
-	var definitions []builtinSkillDefinition
-	if err := json.Unmarshal(builtinSkillsJSON, &definitions); err != nil {
+func TestBuiltinSkillPackageBoundsMetadata(t *testing.T) {
+	skill := builtinSkillDefinition{
+		SkillID:     "test-builtin-skill",
+		SkillName:   "测试技能",
+		Description: strings.Repeat("描述内容。", 140),
+		Instruction: "# 测试技能\n\n保留完整正文。\n",
+	}
+	archive, err := archiveFromMarkdown([]byte(skill.Instruction), skill.SkillName, skill.Description)
+	if err != nil {
 		t.Fatal(err)
 	}
-	for _, skill := range definitions {
-		if skill.SkillID != "16000000000049" {
-			continue
-		}
-		archive, err := archiveFromMarkdown([]byte(skill.Instruction), skill.SkillName, skill.Description)
-		if err != nil {
-			t.Fatal(err)
-		}
-		if len([]rune(archive.Metadata.Description)) != 500 || string(archive.Files["SKILL.md"]) != skill.Instruction {
-			t.Fatal("reported skill must have bounded metadata and unchanged instruction")
-		}
-		return
+	if got := len([]rune(archive.Metadata.Description)); got > 500 {
+		t.Fatalf("builtin skill metadata description length = %d, want <= 500", got)
+	}
+	if string(archive.Files["SKILL.md"]) != skill.Instruction {
+		t.Fatal("builtin skill package must preserve the complete instruction")
 	}
-	t.Fatal("reported builtin skill is missing")
 }
 
 func TestEnsureSkillPackagesBoundsLegacyFallbackMetadata(t *testing.T) {