fix(desktop): finalize skill variant source metadata

Author: legeling

apps/desktop/src/main/ipc/skill/local-repo-handlers.ts

@@ -1,7 +1,10 @@
 import { ipcMain } from "electron";
 import { IPC_CHANNELS } from "@prompthub/shared/constants";
 import { SkillInstaller } from "../../services/skill-installer";
-import { buildSkillSyncUpdateFromRepo } from "../../services/skill-repo-sync";
+import {
+  buildSkillSyncUpdateFromRepo,
+  computeRepoDirectoryFingerprint,
+} from "../../services/skill-repo-sync";
 import type { SkillIPCContext } from "./shared";
 import { ensureLocalRepoPath, readCurrentFilesSnapshot } from "./shared";
 
@@ -82,7 +85,14 @@ async function syncSkillFromRepo(
     return skill;
   }
 
-  const nextUpdate = buildSkillSyncUpdateFromRepo(skill, skillMdFile.content);
+  const directoryFingerprint = await computeRepoDirectoryFingerprint(
+    resolvedRepoPath,
+  );
+  const nextUpdate = buildSkillSyncUpdateFromRepo(
+    skill,
+    skillMdFile.content,
+    directoryFingerprint,
+  );
   if (!nextUpdate) {
     return skill;
   }
@@ -301,9 +311,11 @@ export function registerSkillLocalRepoHandlers({ db }: SkillIPCContext): void {
         content,
       );
       if (relativePath.toLowerCase() === "skill.md") {
+        const nextFingerprint = await computeRepoDirectoryFingerprint(repoPath);
         db.update(skillId, {
           content,
           instructions: content,
+          directory_fingerprint: nextFingerprint,
         });
       }
       return result;

apps/desktop/src/main/services/skill-import-sanitize.ts

@@ -27,6 +27,10 @@ export interface ImportedSkillDraft {
   fallbackTags?: unknown;
   instructions?: unknown;
   source_url?: unknown;
+  source_label?: unknown;
+  source_branch?: unknown;
+  source_directory?: unknown;
+  canonical_skill_path?: unknown;
   local_repo_path?: unknown;
   icon_url?: unknown;
   icon_emoji?: unknown;
@@ -45,6 +49,10 @@ export interface SanitizedImportedSkill {
   tags: string[];
   instructions?: string;
   source_url?: string;
+  source_label?: string;
+  source_branch?: string;
+  source_directory?: string;
+  canonical_skill_path?: string;
   local_repo_path?: string;
   icon_url?: string;
   icon_emoji?: string;
@@ -135,6 +143,18 @@ export function sanitizeImportedSkillDraft(
     tags: sanitizeImportedTags(draft.tags, draft.fallbackTags, defaultTags),
     instructions: sanitizeImportedString(draft.instructions),
     source_url: sanitizeImportedString(draft.source_url, undefined, 500000),
+    source_label: sanitizeImportedString(draft.source_label, undefined, 500000),
+    source_branch: sanitizeImportedString(draft.source_branch, undefined, 256),
+    source_directory: sanitizeImportedString(
+      draft.source_directory,
+      undefined,
+      500000,
+    ),
+    canonical_skill_path: sanitizeImportedString(
+      draft.canonical_skill_path,
+      undefined,
+      500000,
+    ),
     local_repo_path: sanitizeImportedString(
       draft.local_repo_path,
       undefined,

apps/desktop/src/main/services/skill-installer-export.ts

@@ -73,6 +73,10 @@ export function exportAsJson(skill: {
   instructions?: string;
   protocol_type?: string;
   source_url?: string;
+  source_label?: string;
+  source_branch?: string;
+  source_directory?: string;
+  canonical_skill_path?: string;
   icon_url?: string;
   icon_emoji?: string;
   icon_background?: string;
@@ -86,6 +90,10 @@ export function exportAsJson(skill: {
     instructions: skill.instructions || "",
     protocol_type: skill.protocol_type || "skill",
     source_url: skill.source_url || "",
+    source_label: skill.source_label || "",
+    source_branch: skill.source_branch || "",
+    source_directory: skill.source_directory || "",
+    canonical_skill_path: skill.canonical_skill_path || "",
     icon_url: skill.icon_url || "",
     icon_emoji: skill.icon_emoji || "",
     icon_background: skill.icon_background || "",
@@ -121,6 +129,10 @@ export async function importFromJson(
         compatibility: data.compatibility,
         protocol_type: data.protocol_type,
         source_url: data.source_url,
+        source_label: data.source_label,
+        source_branch: data.source_branch,
+        source_directory: data.source_directory,
+        canonical_skill_path: data.canonical_skill_path,
       },
       { defaultTags: ["imported"] },
     );
@@ -146,6 +158,10 @@ export async function importFromJson(
       prerequisites: sanitized.prerequisites,
       compatibility: sanitized.compatibility,
       source_url: sanitized.source_url,
+      source_label: sanitized.source_label,
+      source_branch: sanitized.source_branch,
+      source_directory: sanitized.source_directory,
+      canonical_skill_path: sanitized.canonical_skill_path,
     });
 
     return skill.id;

apps/desktop/src/main/services/skill-installer-repo.ts

@@ -8,6 +8,7 @@ import * as fs from "fs/promises";
 import * as path from "path";
 import type {
   Skill,
+  SkillLocalFileBufferEntry,
   SkillLocalFileEntry,
   SkillLocalFileTreeEntry,
 } from "@prompthub/shared/types";
@@ -25,11 +26,6 @@ import {
   validateSkillName,
 } from "./skill-installer-internal";
 
-export interface SkillLocalFileBufferEntry {
-  path: string;
-  data: Uint8Array;
-}
-
 export interface CopyRepoByPathToDirectoryOptions {
   ifExists?: "overwrite" | "skip" | "error";
   mode?: "copy" | "symlink";

apps/desktop/src/main/services/skill-installer.ts

@@ -347,6 +347,8 @@ export class SkillInstaller {
 
       // Create Skill in DB first, then move the cloned repo into the managed
       // variant container so all My Skills entries share one disk layout.
+      const repoFiles = await this.readLocalRepoFileBuffersByPath(skillDir);
+      const sourceDirectory = path.relative(installDir, skillDir).replace(/\\/g, "/") || undefined;
       const skill = db.create({
         name: manifest.name || repoName,
         description: manifest.description || `Installed from ${url}`,
@@ -356,7 +358,11 @@ export class SkillInstaller {
         instructions: manifest.instructions || "",
         protocol_type: "skill",
         source_url: url,
+        source_label: `${userDir}/${repoName}`,
+        source_directory: sourceDirectory,
+        canonical_skill_path: sourceDirectory ? `${sourceDirectory}/SKILL.md` : "SKILL.md",
         local_repo_path: installDir,
+        directory_fingerprint: computeDirectoryFingerprint(repoFiles),
         is_favorite: false,
         tags: [],
         original_tags: manifest.tags || ["github"],
@@ -474,9 +480,9 @@ export class SkillInstaller {
       );
     }
 
-    const sanitized = sanitizeImportedSkillDraft(
-      {
-        name: skillName,
+      const sanitized = sanitizeImportedSkillDraft(
+        {
+          name: skillName,
         description: parsed?.frontmatter.description,
         fallbackDescription:
           manifest.description ||
@@ -494,6 +500,7 @@ export class SkillInstaller {
       },
       { defaultTags: [] },
     );
+    const canonicalSkillPath = options?.repoSourceDir ? "SKILL.md" : undefined;
 
     // Save files first, then create DB record to avoid orphaned records
     const createdSkill = db.create({
@@ -508,7 +515,9 @@ export class SkillInstaller {
       original_tags: sanitized.tags,
       is_favorite: false,
       source_url: sanitized.source_url,
+      source_label: options?.sourceUrl,
       local_repo_path: sanitized.local_repo_path,
+      canonical_skill_path: canonicalSkillPath,
     });
 
     let localRepoPath: string | undefined;
@@ -526,7 +535,16 @@ export class SkillInstaller {
     }
 
     if (localRepoPath && createdSkill.local_repo_path !== localRepoPath) {
-      db.update(createdSkill.id, { local_repo_path: localRepoPath });
+      const repoFiles = await this.readLocalRepoFileBuffersByPath(localRepoPath);
+      db.update(createdSkill.id, {
+        local_repo_path: localRepoPath,
+        directory_fingerprint: computeDirectoryFingerprint(repoFiles),
+      });
+    } else if (localRepoPath) {
+      const repoFiles = await this.readLocalRepoFileBuffersByPath(localRepoPath);
+      db.update(createdSkill.id, {
+        directory_fingerprint: computeDirectoryFingerprint(repoFiles),
+      });
     }
 
     return createdSkill.id;
@@ -659,9 +677,7 @@ export class SkillInstaller {
           source_branch: normalizedBranch,
           source_directory: normalizedDirectory || undefined,
           canonical_skill_path: skill.filePath,
-          directory_fingerprint: computeDirectoryFingerprint([
-            { path: "SKILL.md", content: skill.instructions, isDirectory: false },
-          ]),
+          directory_fingerprint: skill.directory_fingerprint,
           description: builtin?.description || skill.description || `${skill.name} skill`,
           category: builtin?.category || "general",
           icon_url: builtin?.icon_url,
@@ -873,6 +889,9 @@ export class SkillInstaller {
               );
 
               skillMap.set(skillFolderPath, {
+                directory_fingerprint: computeDirectoryFingerprint(
+                  await this.readLocalRepoFileBuffersByPath(skillFolderPath),
+                ),
                 name: sanitized.name!,
                 description: sanitized.description || manifest.description,
                 version: sanitized.version,

apps/desktop/src/main/services/skill-repo-sync.ts

@@ -1,4 +1,5 @@
 import type { Skill, UpdateSkillParams } from "@prompthub/shared/types";
+import { computeDirectoryFingerprint } from "@prompthub/shared/utils/skill-identity";
 import { sanitizeImportedSkillDraft } from "./skill-import-sanitize";
 import { parseSkillMd } from "./skill-validator";
 import { SkillInstaller } from "./skill-installer";
@@ -29,6 +30,7 @@ function normalizeCompatibility(compatibility?: string): string[] | undefined {
 export function buildSkillSyncUpdateFromRepo(
   skill: Skill,
   skillMdContent: string,
+  directoryFingerprint?: string,
 ): UpdateSkillParams | null {
   const parsed = parseSkillMd(skillMdContent);
   const sanitized = sanitizeImportedSkillDraft(
@@ -91,9 +93,21 @@ export function buildSkillSyncUpdateFromRepo(
     changed = true;
   }
 
+  if (directoryFingerprint !== undefined && directoryFingerprint !== skill.directory_fingerprint) {
+    update.directory_fingerprint = directoryFingerprint;
+    changed = true;
+  }
+
   return changed ? update : null;
 }
 
+export async function computeRepoDirectoryFingerprint(
+  repoPath: string,
+): Promise<string> {
+  const entries = await SkillInstaller.readLocalRepoFileBuffersByPath(repoPath);
+  return computeDirectoryFingerprint(entries);
+}
+
 /**
  * Check whether metadata-only fields (description, author, name, tags) changed.
  * Returns true when the SKILL.md frontmatter should be rewritten to stay in sync

apps/desktop/src/renderer/components/skill/SkillGalleryCard.tsx

@@ -11,6 +11,8 @@ import { useTranslation } from "react-i18next";
 import type { Skill } from "@prompthub/shared/types";
 import { SkillIcon } from "./SkillIcon";
 import { getRuntimeCapabilities } from "../../runtime";
+import { SkillVariantBadgeList } from "./SkillVariantBadgeList";
+import { buildSkillVariantBadges } from "../../services/skill-variant-badges";
 
 function normalizeStringArray(value: unknown): string[] {
   if (Array.isArray(value)) {
@@ -59,6 +61,7 @@ function SkillGalleryCardComponent({
   const { t } = useTranslation();
   const runtimeCapabilities = getRuntimeCapabilities();
   const visibleTags = normalizeStringArray(skill.tags).slice(0, 4);
+  const variantBadges = buildSkillVariantBadges(skill, t);
 
   return (
     <div
@@ -181,6 +184,7 @@ function SkillGalleryCardComponent({
         {skill.description ||
           t("skill.defaultDescription", "技能描述，帮助 AI 理解何时使用此技能")}
       </p>
+      <SkillVariantBadgeList badges={variantBadges} className="mb-3 flex flex-wrap gap-1.5" />
       {visibleTags.length > 0 ? (
         <div className="flex flex-wrap gap-1.5">
           {visibleTags.map((tag) => (

apps/desktop/src/renderer/components/skill/SkillListView.tsx

@@ -21,6 +21,8 @@ import { filterDetectedPlatforms } from "../../services/platform-visibility";
 import type { Skill, SkillSafetyLevel } from "@prompthub/shared/types";
 import type { SkillPlatform } from "@prompthub/shared/constants/platforms";
 import { getRuntimeCapabilities } from "../../runtime";
+import { SkillVariantBadgeList } from "./SkillVariantBadgeList";
+import { buildSkillVariantBadges } from "../../services/skill-variant-badges";
 
 function normalizeStringArray(value: unknown): string[] {
   if (Array.isArray(value)) {
@@ -438,19 +440,23 @@ export function SkillListView({
                 <p className="text-xs text-muted-foreground truncate mt-0.5">
                   {skill.description || t("skill.defaultDescription")}
                 </p>
-                {visibleTags.length > 0 ? (
-                  <div className="mt-2 flex flex-wrap gap-1.5">
-                    {visibleTags.map((tag) => (
+                  {visibleTags.length > 0 ? (
+                    <div className="mt-2 flex flex-wrap gap-1.5">
+                      {visibleTags.map((tag) => (
                       <span
                         key={tag}
                         className="rounded-full bg-primary/10 px-2 py-0.5 text-[10px] font-medium text-primary"
                       >
                         {tag}
                       </span>
-                    ))}
-                  </div>
-                ) : null}
-              </div>
+                      ))}
+                    </div>
+                  ) : null}
+                  <SkillVariantBadgeList
+                    badges={buildSkillVariantBadges(skill, t)}
+                    className="mt-2 flex flex-wrap gap-1.5"
+                  />
+                </div>
 
               {/* Platform indicators */}
               {runtimeCapabilities.skillPlatformIntegration && totalPlatforms > 0 && (

apps/desktop/src/renderer/components/skill/SkillStoreCard.tsx

@@ -2,6 +2,8 @@ import { CheckIcon, DownloadIcon, Loader2Icon, PlusIcon } from "lucide-react";
 import { useTranslation } from "react-i18next";
 import type { RegistrySkill } from "@prompthub/shared/types";
 import { SkillIcon } from "./SkillIcon";
+import { SkillVariantBadgeList } from "./SkillVariantBadgeList";
+import { buildSkillVariantBadges } from "../../services/skill-variant-badges";
 
 const MAX_STAGGERED_STORE_CARDS = 12;
 
@@ -26,6 +28,10 @@ export function SkillStoreCard({
 }: SkillStoreCardProps) {
   const { t } = useTranslation();
   const isInstallingThis = installingSourceId === skill.source_id;
+  const badges = buildSkillVariantBadges(skill, t, {
+    hasUpdate,
+    isInstalled,
+  });
 
   return (
     <div
@@ -52,6 +58,7 @@ export function SkillStoreCard({
         <p className="text-[11px] text-muted-foreground truncate mt-0.5">
           {skill.description}
         </p>
+        <SkillVariantBadgeList badges={badges} className="mt-2 flex flex-wrap gap-1" />
         {skill.weekly_installs && (
           <div className="mt-1.5 inline-flex items-center rounded-full bg-primary/10 px-2 py-0.5 text-[10px] font-medium text-primary">
             {skill.weekly_installs}/wk