From 2c15960ac2c980ba5494c06eb5bcc96eb0bdd6a4 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 28 Mar 2026 02:03:07 +0000 Subject: [PATCH] fix: replay skill fixture source drift --- src/agents/skills-install.download.test.ts | 3 +-- src/agents/skills-status.test.ts | 3 +-- src/agents/skills.buildworkspaceskillstatus.test.ts | 2 +- src/agents/skills.resolveskillspromptforrun.test.ts | 3 +-- src/agents/skills/compact-format.test.ts | 8 ++------ src/cli/skills-cli.formatting.test.ts | 3 +-- 6 files changed, 7 insertions(+), 15 deletions(-) diff --git a/src/agents/skills-install.download.test.ts b/src/agents/skills-install.download.test.ts index acbd0e15b73..a222e55cfb3 100644 --- a/src/agents/skills-install.download.test.ts +++ b/src/agents/skills-install.download.test.ts @@ -1,7 +1,6 @@ import fs from "node:fs/promises"; import os from "node:os"; import path from "node:path"; -import { createSyntheticSourceInfo } from "@mariozechner/pi-coding-agent"; import { afterAll, beforeAll, beforeEach, describe, expect, it, vi } from "vitest"; import { installDownloadSpec } from "./skills-install-download.js"; import { setTempStateDir } from "./skills-install.download-test-utils.js"; @@ -80,7 +79,7 @@ function createFixtureSkill(params: { description: params.description, filePath: params.filePath, baseDir: params.baseDir, - sourceInfo: createSyntheticSourceInfo(params.filePath, { source: params.source }), + source: params.source, disableModelInvocation: false, }; } diff --git a/src/agents/skills-status.test.ts b/src/agents/skills-status.test.ts index 0e279327316..1c36e967523 100644 --- a/src/agents/skills-status.test.ts +++ b/src/agents/skills-status.test.ts @@ -1,4 +1,3 @@ -import { createSyntheticSourceInfo } from "@mariozechner/pi-coding-agent"; import { describe, expect, it } from "vitest"; import { buildWorkspaceSkillStatus } from "./skills-status.js"; import type { SkillEntry } from "./skills/types.js"; @@ -54,7 +53,7 @@ function createFixtureSkill(params: { description: params.description, filePath: params.filePath, baseDir: params.baseDir, - sourceInfo: createSyntheticSourceInfo(params.filePath, { source: params.source }), + source: params.source, disableModelInvocation: false, }; } diff --git a/src/agents/skills.buildworkspaceskillstatus.test.ts b/src/agents/skills.buildworkspaceskillstatus.test.ts index c2e111dca3a..1f8eed0d400 100644 --- a/src/agents/skills.buildworkspaceskillstatus.test.ts +++ b/src/agents/skills.buildworkspaceskillstatus.test.ts @@ -62,7 +62,7 @@ function createFixtureSkill(params: { description: params.description, filePath: params.filePath, baseDir: params.baseDir, - sourceInfo: { source: params.source } as never, + source: params.source, disableModelInvocation: false, } as SkillEntry["skill"]; } diff --git a/src/agents/skills.resolveskillspromptforrun.test.ts b/src/agents/skills.resolveskillspromptforrun.test.ts index 15c47b742a5..9713cac9b37 100644 --- a/src/agents/skills.resolveskillspromptforrun.test.ts +++ b/src/agents/skills.resolveskillspromptforrun.test.ts @@ -1,4 +1,3 @@ -import { createSyntheticSourceInfo } from "@mariozechner/pi-coding-agent"; import { describe, expect, it } from "vitest"; import { resolveSkillsPromptForRun } from "./skills.js"; import type { SkillEntry } from "./skills/types.js"; @@ -43,7 +42,7 @@ function createFixtureSkill(params: { description: params.description, filePath: params.filePath, baseDir: params.baseDir, - sourceInfo: createSyntheticSourceInfo(params.filePath, { source: params.source }), + source: params.source, disableModelInvocation: false, }; } diff --git a/src/agents/skills/compact-format.test.ts b/src/agents/skills/compact-format.test.ts index f532ce1d199..cd9d6f42f15 100644 --- a/src/agents/skills/compact-format.test.ts +++ b/src/agents/skills/compact-format.test.ts @@ -1,9 +1,5 @@ import os from "node:os"; -import { - createSyntheticSourceInfo, - formatSkillsForPrompt, - type Skill, -} from "@mariozechner/pi-coding-agent"; +import { formatSkillsForPrompt, type Skill } from "@mariozechner/pi-coding-agent"; import { describe, expect, it } from "vitest"; import type { OpenClawConfig } from "../../config/config.js"; import type { SkillEntry } from "./types.js"; @@ -19,7 +15,7 @@ function makeSkill(name: string, desc = "A skill", filePath = `/skills/${name}/S description: desc, filePath, baseDir: `/skills/${name}`, - sourceInfo: createSyntheticSourceInfo(filePath, { source: "workspace" }), + source: "workspace", disableModelInvocation: false, }; } diff --git a/src/cli/skills-cli.formatting.test.ts b/src/cli/skills-cli.formatting.test.ts index 832660b29d1..33cb0747485 100644 --- a/src/cli/skills-cli.formatting.test.ts +++ b/src/cli/skills-cli.formatting.test.ts @@ -1,7 +1,6 @@ import fs from "node:fs"; import os from "node:os"; import path from "node:path"; -import { createSyntheticSourceInfo } from "@mariozechner/pi-coding-agent"; import { afterAll, beforeAll, describe, expect, it } from "vitest"; import { buildWorkspaceSkillStatus } from "../agents/skills-status.js"; import type { SkillEntry } from "../agents/skills.js"; @@ -98,7 +97,7 @@ function createFixtureSkill(params: { description: params.description, filePath: params.filePath, baseDir: params.baseDir, - sourceInfo: createSyntheticSourceInfo(params.filePath, { source: params.source }), + source: params.source, disableModelInvocation: false, }; }