From f16ddc44f9066dfbd9306d848b841346f91a2f90 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E9=9B=B7=E7=94=B5=E8=8A=BD=E8=A1=A3?=
 <zhangyanghaha0407@outlook.com>
Date: Thu, 26 Mar 2026 00:23:45 -0400
Subject: [PATCH] test(02-01): add failing test for skill type definitions

- Tests TOKEN_BUDGETS constant values for 5 adapter types
- Tests type shapes for SkillRecord, RuntimeSkillEntry, ManagedSkillRef
- Tests SkillSourceType (5 members), TrustLevel (3 members), FileInventoryEntry (6 kinds)
---
 tests/unit/skill-types.test.ts | 146 +++++++++++++++++++++++++++++++++
 1 file changed, 146 insertions(+)
 create mode 100644 tests/unit/skill-types.test.ts

diff --git a/tests/unit/skill-types.test.ts b/tests/unit/skill-types.test.ts
new file mode 100644
index 00000000..6572d96e
--- /dev/null
+++ b/tests/unit/skill-types.test.ts
@@ -0,0 +1,146 @@
+import { describe, it, expect } from "vitest";
+import {
+  TOKEN_BUDGETS,
+} from "@main/skill-types";
+import type {
+  SkillSourceType,
+  TrustLevel,
+  FileInventoryEntry,
+  SkillRecord,
+  SkillImportResult,
+  RuntimeSkillEntry,
+  ManagedSkillRef,
+  SkillAssignment,
+  SkillUpdateStatus,
+} from "@main/skill-types";
+
+describe("skill-types", () => {
+  describe("TOKEN_BUDGETS", () => {
+    it("has entries for all five adapter types", () => {
+      expect(TOKEN_BUDGETS).toHaveProperty("claude_local");
+      expect(TOKEN_BUDGETS).toHaveProperty("codex_local");
+      expect(TOKEN_BUDGETS).toHaveProperty("gemini_local");
+      expect(TOKEN_BUDGETS).toHaveProperty("opencode_local");
+      expect(TOKEN_BUDGETS).toHaveProperty("kimi_local");
+    });
+
+    it("has numeric token caps for each adapter", () => {
+      expect(TOKEN_BUDGETS.claude_local).toBe(50000);
+      expect(TOKEN_BUDGETS.codex_local).toBe(30000);
+      expect(TOKEN_BUDGETS.gemini_local).toBe(50000);
+      expect(TOKEN_BUDGETS.opencode_local).toBe(30000);
+      expect(TOKEN_BUDGETS.kimi_local).toBe(20000);
+    });
+  });
+
+  describe("type shape checks", () => {
+    it("SkillSourceType accepts all five source kinds", () => {
+      const sources: SkillSourceType[] = [
+        "github",
+        "local_path",
+        "url",
+        "skills_sh",
+        "managed",
+      ];
+      expect(sources).toHaveLength(5);
+    });
+
+    it("TrustLevel accepts all three levels", () => {
+      const levels: TrustLevel[] = [
+        "markdown_only",
+        "assets",
+        "scripts_executables",
+      ];
+      expect(levels).toHaveLength(3);
+    });
+
+    it("FileInventoryEntry has path and kind", () => {
+      const entry: FileInventoryEntry = { path: "SKILL.md", kind: "skill" };
+      expect(entry.path).toBe("SKILL.md");
+      expect(entry.kind).toBe("skill");
+    });
+
+    it("FileInventoryEntry kind accepts all six values", () => {
+      const kinds: FileInventoryEntry["kind"][] = [
+        "skill",
+        "reference",
+        "script",
+        "asset",
+        "markdown",
+        "other",
+      ];
+      expect(kinds).toHaveLength(6);
+    });
+
+    it("SkillRecord has all expected properties", () => {
+      const record: SkillRecord = {
+        id: "test-id",
+        companyId: "company-1",
+        key: "owner/repo/slug",
+        slug: "slug",
+        name: "Test Skill",
+        description: null,
+        markdown: "# Test",
+        sourceType: "github",
+        sourceLocator: "https://github.com/owner/repo",
+        sourceRef: "abc123",
+        trustLevel: "markdown_only",
+        compatibility: "compatible",
+        fileInventory: [],
+        metadata: null,
+        createdAt: "2025-01-01T00:00:00Z",
+        updatedAt: "2025-01-01T00:00:00Z",
+      };
+      expect(record.id).toBe("test-id");
+      expect(record.companyId).toBe("company-1");
+    });
+
+    it("SkillImportResult has skillId, key, name, isUpdate", () => {
+      const result: SkillImportResult = {
+        skillId: "id-1",
+        key: "owner/repo/slug",
+        name: "Test",
+        isUpdate: false,
+      };
+      expect(result.isUpdate).toBe(false);
+    });
+
+    it("RuntimeSkillEntry has runtimeName, sourcePath, name, tokenEstimate", () => {
+      const entry: RuntimeSkillEntry = {
+        runtimeName: "claude",
+        sourcePath: "/path/to/skill",
+        name: "test-skill",
+        tokenEstimate: 1000,
+      };
+      expect(entry.tokenEstimate).toBe(1000);
+    });
+
+    it("ManagedSkillRef has runtimeName and sourcePath only", () => {
+      const ref: ManagedSkillRef = {
+        runtimeName: "claude",
+        sourcePath: "/path/to/skill",
+      };
+      expect(ref.runtimeName).toBe("claude");
+      expect(ref.sourcePath).toBe("/path/to/skill");
+    });
+
+    it("SkillAssignment has agentId and skillId", () => {
+      const assignment: SkillAssignment = {
+        agentId: "agent-1",
+        skillId: "skill-1",
+      };
+      expect(assignment.agentId).toBe("agent-1");
+    });
+
+    it("SkillUpdateStatus has all expected fields", () => {
+      const status: SkillUpdateStatus = {
+        skillId: "skill-1",
+        key: "owner/repo/slug",
+        currentRef: "abc",
+        latestRef: "def",
+        hasUpdate: true,
+      };
+      expect(status.hasUpdate).toBe(true);
+    });
+  });
+});