From a51e8a21b67db40307afc9daa9e4e808b7a68aea Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Sat, 30 May 2026 00:01:10 +0200
Subject: [PATCH] fix(ci): break skills loading cycle

---
 src/cli/skills-cli.commands.test.ts  |  2 +-
 src/skills/loading/skill-contract.ts | 13 +++++++++----
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/src/cli/skills-cli.commands.test.ts b/src/cli/skills-cli.commands.test.ts
index b6fe0577adf..834e0af6691 100644
--- a/src/cli/skills-cli.commands.test.ts
+++ b/src/cli/skills-cli.commands.test.ts
@@ -85,7 +85,7 @@ const mocks = vi.hoisted(() => {
     updateSkillsFromClawHubMock: vi.fn(),
     readTrackedClawHubSkillSlugsMock: vi.fn(),
     resolveClawHubSkillVerificationTargetMock: vi.fn(),
-    readClawHubSkillsLockfileStatusSyncMock: vi.fn(() => ({ kind: "missing" })),
+    readClawHubSkillsLockfileStatusSyncMock: vi.fn((..._args: unknown[]) => ({ kind: "missing" })),
     resolveClawHubSkillStatusLinkSyncMock: vi.fn(),
     resolveLocalSkillCardStatusSyncMock: vi.fn(),
     fetchClawHubSkillVerificationMock: vi.fn(),
diff --git a/src/skills/loading/skill-contract.ts b/src/skills/loading/skill-contract.ts
index 5a024c076a1..320ee945e5e 100644
--- a/src/skills/loading/skill-contract.ts
+++ b/src/skills/loading/skill-contract.ts
@@ -1,13 +1,18 @@
 import type { SourceInfo } from "../../agents/sessions/source-info.js";
-import type { Skill as CanonicalSkill } from "./session.js";
 
 export type SourceScope = "user" | "project" | "temporary";
 export type SourceOrigin = "package" | "top-level";
 
-export type Skill = CanonicalSkill & {
+export interface Skill {
+  name: string;
+  description: string;
+  filePath: string;
+  baseDir: string;
+  sourceInfo: SourceInfo;
+  disableModelInvocation: boolean;
   // Preserve legacy source reads while keeping the canonical upstream shape.
-  source?: string;
-};
+  source: string;
+}
 
 export function createSyntheticSourceInfo(
   path: string,