jb/paseo
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

Lock skills update status invariant

Browse Source
This commit is contained in:
Mohamed Boudra
2026-06-04 15:08:28 +07:00
parent de9d796567
commit 1b350ac09e
1 changed files with 61 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/desktop/src/integrations/skills/operations.test.ts
+61 -2
View File
@@ -133,7 +133,7 @@ describe("getSkillsStatus", () => {
expect(status).toEqual({ state: "up-to-date", ops: [] });
});
it("ignores user-added files inside current managed skill dirs", async () => {
it("ignores user-added files inside current managed skill dirs in every target", async () => {
await writeCurrentBundle(sandbox.targets.sourceDir);
await writeOnDiskSkillToAllTargets(sandbox.targets, "paseo", { "SKILL.md": "paseo-v1" });
await writeOnDiskSkillToAllTargets(sandbox.targets, "paseo-loop", { "SKILL.md": "loop-v1" });
@@ -141,6 +141,14 @@ describe("getSkillsStatus", () => {
"SKILL.md": "paseo-v1",
"my-context.md": "user context",
});
await writeOnDiskSkill(sandbox.targets.claudeDir, "paseo", {
"SKILL.md": "paseo-v1",
"commands/local.md": "user command",
});
await writeOnDiskSkill(sandbox.targets.codexDir, "paseo", {
"SKILL.md": "paseo-v1",
"hooks/guard.sh": "user guard",
});
const status = await getSkillsStatus(sandbox.targets);
@@ -272,6 +280,47 @@ describe("installSkills / updateSkills", () => {
}
});
it("defines updated as the state reached after preserving user files", async () => {
await writeCurrentBundle(sandbox.targets.sourceDir);
await writeOnDiskSkillToAllTargets(sandbox.targets, "paseo", { "SKILL.md": "paseo-v1" });
await writeOnDiskSkillToAllTargets(sandbox.targets, "paseo-loop", { "SKILL.md": "loop-v1" });
await writeOnDiskSkill(sandbox.targets.agentsDir, "paseo", {
"SKILL.md": "stale",
"hooks/guard.sh": "user guard",
});
await writeOnDiskSkill(sandbox.targets.claudeDir, "paseo", {
"SKILL.md": "paseo-v1",
"notes/local.md": "claude notes",
});
await writeOnDiskSkill(sandbox.targets.codexDir, "paseo", {
"SKILL.md": "paseo-v1",
"prompts/local.md": "codex prompt",
});
const status = await updateSkills(sandbox.targets);
expect(status).toEqual({ state: "up-to-date", ops: [] });
expect(await getSkillsStatus(sandbox.targets)).toEqual({ state: "up-to-date", ops: [] });
expect(
await fs.readFile(
path.join(sandbox.targets.agentsDir, "paseo", "hooks", "guard.sh"),
"utf-8",
),
).toBe("user guard");
expect(
await fs.readFile(
path.join(sandbox.targets.claudeDir, "paseo", "notes", "local.md"),
"utf-8",
),
).toBe("claude notes");
expect(
await fs.readFile(
path.join(sandbox.targets.codexDir, "paseo", "prompts", "local.md"),
"utf-8",
),
).toBe("codex prompt");
});
it("repairs secondary agent targets even when agents skills are current", async () => {
await writeCurrentBundle(sandbox.targets.sourceDir);
await writeOnDiskSkill(sandbox.targets.agentsDir, "paseo", { "SKILL.md": "paseo-v1" });
@@ -290,14 +339,24 @@ describe("installSkills / updateSkills", () => {
it("auto-updates drifted installed skills", async () => {
await writeCurrentBundle(sandbox.targets.sourceDir);
await writeOnDiskSkill(sandbox.targets.agentsDir, "paseo", { "SKILL.md": "stale" });
await writeOnDiskSkill(sandbox.targets.agentsDir, "paseo", {
"SKILL.md": "stale",
"hooks/guard.sh": "user guard",
});
const status = await autoUpdateInstalledSkills(sandbox.targets);
expect(status).toEqual({ state: "up-to-date", ops: [] });
expect(await getSkillsStatus(sandbox.targets)).toEqual({ state: "up-to-date", ops: [] });
expect(
await fs.readFile(path.join(sandbox.targets.agentsDir, "paseo", "SKILL.md"), "utf-8"),
).toBe("paseo-v1");
expect(
await fs.readFile(
path.join(sandbox.targets.agentsDir, "paseo", "hooks", "guard.sh"),
"utf-8",
),
).toBe("user guard");
});
it("does not auto-install skills on a clean machine", async () => {