From c74a0089d0cbc219bba054ed1b83884eb9669537 Mon Sep 17 00:00:00 2001
From: Jesse <15653378+squarezw@user.noreply.gitee.com>
Date: Wed, 16 Sep 2026 11:12:28 +0800
Subject: [PATCH] perf(skills): return list summaries only

---
 app/apps/components/AppSkillsSection.tsx |  2 +-
 app/skills/page.tsx                      | 12 ++++++------
 test/skillListSummary.test.ts            | 17 +++++++++++++++++
 types/skill.ts                           |  4 ++++
 4 files changed, 28 insertions(+), 7 deletions(-)
 create mode 100644 test/skillListSummary.test.ts

diff --git a/app/apps/components/AppSkillsSection.tsx b/app/apps/components/AppSkillsSection.tsx
index bfec4bd..02c3c96 100644
--- a/app/apps/components/AppSkillsSection.tsx
+++ b/app/apps/components/AppSkillsSection.tsx
@@ -267,7 +267,7 @@ function BindSkillsDialog({
         ) : (
           <div className="space-y-2">
             {available.map((skill) => {
-              const isPublished = skill.published_content !== null;
+              const isPublished = skill.is_published === true;
               return (
                 <label
                   key={skill.id}
diff --git a/app/skills/page.tsx b/app/skills/page.tsx
index e943c1a..48914a3 100644
--- a/app/skills/page.tsx
+++ b/app/skills/page.tsx
@@ -32,7 +32,7 @@ import { Loader2, Plus, Search, Sparkles, Terminal, Upload } from "lucide-react"
 import { useDebounce } from "use-debounce";
 import { useCurrentUser } from "@/hooks/useCurrentUser";
 import { useSkills } from "@/hooks/useSkills";
-import { hasUnpublishedChanges, resolveReviewStatus, reviewStatusBadge } from "@/lib/reviewStatus";
+import { resolveReviewStatus, reviewStatusBadge } from "@/lib/reviewStatus";
 import { canEditSkill } from "@/lib/skillPermissions";
 import { filterSkillsByCreator, getSkillCreators } from "@/lib/skillCreatorFilter";
 import { checkDeptAdmin, checkSuperAdmin, checkTenantAdmin } from "@/lib/clientPermissions";
@@ -218,12 +218,12 @@ export default function SkillsPage() {
                   </TableHeader>
                   <TableBody>
                     {filteredSkills.map((skill) => {
-                      const status = resolveReviewStatus(skill.status, skill.published_content);
-                      const badge = reviewStatusBadge(status);
-                      const unpublishedChanges = hasUnpublishedChanges(
-                        skill.content,
-                        skill.published_content
+                      const status = resolveReviewStatus(
+                        skill.status,
+                        skill.is_published ? "" : null
                       );
+                      const badge = reviewStatusBadge(status);
+                      const unpublishedChanges = skill.has_unpublished_changes === true;
                       const canEdit = canEditSkill(skill, user, isSuperAdmin, isTenantAdmin);
                       return (
                         <TableRow
diff --git a/test/skillListSummary.test.ts b/test/skillListSummary.test.ts
new file mode 100644
index 0000000..0dc1960
--- /dev/null
+++ b/test/skillListSummary.test.ts
@@ -0,0 +1,17 @@
+import assert from "node:assert/strict";
+import { readFileSync } from "node:fs";
+import { join } from "node:path";
+import { test } from "node:test";
+
+const listPage = readFileSync(join(process.cwd(), "app/skills/page.tsx"), "utf8");
+const bindDialog = readFileSync(
+  join(process.cwd(), "app/apps/components/AppSkillsSection.tsx"),
+  "utf8"
+);
+
+test("列表消费服务端摘要状态，不依赖未返回的正文", () => {
+  assert.match(listPage, /has_unpublished_changes/);
+  assert.doesNotMatch(listPage, /resolveReviewStatus\(skill\.status, skill\.published_content\)/);
+  assert.match(bindDialog, /skill\.is_published === true/);
+  assert.doesNotMatch(bindDialog, /skill\.published_content !== null/);
+});
diff --git a/types/skill.ts b/types/skill.ts
index d8c3d07..09087e0 100644
--- a/types/skill.ts
+++ b/types/skill.ts
@@ -21,6 +21,10 @@ export interface Skill {
   content: string;
   /** 已发布正文；null = 从未发布 */
   published_content: string | null;
+  /** 列表接口直接给出，避免返回正文后再由浏览器计算。 */
+  has_unpublished_changes?: boolean;
+  /** 列表接口直接给出，避免以未返回的 published_content 推断。 */
+  is_published?: boolean;
   requires: SkillRequires | null;
   visibility: SkillVisibility;
   /** 审核生命周期状态；后端并行开发中可能缺失，用 resolveReviewStatus 容错归一化 */