From 7c4495623f968ea110721ab86a4156ab38f023ec Mon Sep 17 00:00:00 2001
From: ddCat <2695092045@qq.com>
Date: Tue, 22 Sep 2026 13:13:26 +0800
Subject: [PATCH] =?UTF-8?q?fix(skills):=20=E7=A7=8D=E5=AD=90=E6=B5=8B?=
 =?UTF-8?q?=E8=AF=95=20-=20=E8=B7=9F=E9=9A=8F=E6=9C=80=E6=96=B0=E5=9F=9F?=
 =?UTF-8?q?=E5=8C=85=E5=90=8D=E7=A7=B0=E6=A0=A1=E9=AA=8C=E5=85=AC=E5=BC=80?=
 =?UTF-8?q?=E6=90=9C=E7=B4=A2?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 backend/internal/skills/skills_seed_test.go | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/backend/internal/skills/skills_seed_test.go b/backend/internal/skills/skills_seed_test.go
index 07f0f3983..0cc10b100 100644
--- a/backend/internal/skills/skills_seed_test.go
+++ b/backend/internal/skills/skills_seed_test.go
@@ -35,6 +35,8 @@ func TestBuiltinSeedSyncPreservesUserState(t *testing.T) {
 	if err := json.Unmarshal(builtinSkillsJSON, &definitions); err != nil {
 		t.Fatal(err)
 	}
+	const communitySkillID = "16000000000081"
+	communitySkillName := ""
 	community := 0
 	for _, def := range definitions {
 		if def.OwnerUID == "" || def.OwnerUID != def.EffectiveUser.UID {
@@ -48,17 +50,23 @@ func TestBuiltinSeedSyncPreservesUserState(t *testing.T) {
 		if def.OwnerUID == "community-itswyatt-k" {
 			community++
 		}
+		if def.SkillID == communitySkillID {
+			communitySkillName = def.SkillName
+		}
 	}
 	if community != 35 {
 		t.Fatalf("community skills = %d, want 35", community)
 	}
+	if communitySkillName == "" {
+		t.Fatalf("community seed skill %s is missing", communitySkillID)
+	}
 	if err := svc.EnsureBuiltinSkills(); err != nil {
 		t.Fatal(err)
 	}
 	if err := svc.EnsureSkillPackages(); err != nil {
 		t.Fatal(err)
 	}
-	state := model.UserSkillState{ID: "test-state", UserID: "test-user", SkillID: "16000000000081", Added: true, Liked: true}
+	state := model.UserSkillState{ID: "test-state", UserID: "test-user", SkillID: communitySkillID, Added: true, Liked: true}
 	if err := db.Create(&state).Error; err != nil {
 		t.Fatal(err)
 	}
@@ -83,7 +91,7 @@ func TestBuiltinSeedSyncPreservesUserState(t *testing.T) {
 		t.Fatalf("user state lost: %#v", saved)
 	}
 	var skill model.Skill
-	if err := db.First(&skill, "id = ?", state.SkillID).Error; err != nil {
+	if err := db.First(&skill, "id = ?", communitySkillID).Error; err != nil {
 		t.Fatal(err)
 	}
 	if skill.OwnerID != "community-itswyatt-k" || skill.CreatedAt.Year() != 2026 {
@@ -110,11 +118,11 @@ func TestBuiltinSeedSyncPreservesUserState(t *testing.T) {
 			t.Fatalf("skill %s package instruction was changed", item.ID)
 		}
 	}
-	list, err := svc.Skills("test-user", SkillListRequest{Scope: "public", Search: "shortform-drama-playbook"})
+	list, err := svc.Skills("test-user", SkillListRequest{Scope: "public", Search: communitySkillName})
 	if err != nil {
 		t.Fatal(err)
 	}
-	if list.TotalCount != 1 || len(list.Skills) != 1 || list.Skills[0].SkillID != "16000000000081" {
+	if list.TotalCount != 1 || len(list.Skills) != 1 || list.Skills[0].SkillID != communitySkillID {
 		t.Fatalf("community skill not visible in public search: %#v", list)
 	}
 }