From db0d2128350932dc53113da74c53bff590dfb61a Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 18 Apr 2026 22:00:27 +0100 Subject: [PATCH] test: dedupe skills prompt fixture setup --- src/agents/skills.test.ts | 26 ++++++++++++-------------- 1 file changed, 12 insertions(+), 14 deletions(-) diff --git a/src/agents/skills.test.ts b/src/agents/skills.test.ts index a1298f298ac..aca59d35826 100644 --- a/src/agents/skills.test.ts +++ b/src/agents/skills.test.ts @@ -47,6 +47,16 @@ function withWorkspaceHome(workspaceDir: string, cb: () => T): T { return withPathResolutionEnv(workspaceDir, { PATH: "" }, () => cb()); } +async function writePromptLimitSkills(workspaceDir: string) { + for (const name of ["alpha-skill", "beta-skill", "gamma-skill"]) { + await writeSkill({ + dir: path.join(workspaceDir, "skills", name), + name, + description: "D".repeat(240), + }); + } +} + const withClearedEnv = ( keys: string[], run: (original: Record) => T, @@ -334,13 +344,7 @@ describe("buildWorkspaceSkillsPrompt", () => { it("applies per-agent skillsLimits.maxSkillsPromptChars", async () => { const workspaceDir = await makeWorkspace(); - for (const name of ["alpha-skill", "beta-skill", "gamma-skill"]) { - await writeSkill({ - dir: path.join(workspaceDir, "skills", name), - name, - description: "D".repeat(240), - }); - } + await writePromptLimitSkills(workspaceDir); const prompt = withWorkspaceHome(workspaceDir, () => buildWorkspaceSkillsPrompt(workspaceDir, { @@ -372,13 +376,7 @@ describe("buildWorkspaceSkillsPrompt", () => { it("does not apply agents.list[].skillsLimits without an explicit agent id", async () => { const workspaceDir = await makeWorkspace(); - for (const name of ["alpha-skill", "beta-skill", "gamma-skill"]) { - await writeSkill({ - dir: path.join(workspaceDir, "skills", name), - name, - description: "D".repeat(240), - }); - } + await writePromptLimitSkills(workspaceDir); const prompt = withWorkspaceHome(workspaceDir, () => buildWorkspaceSkillsPrompt(workspaceDir, {