A prune-only run summarizes as removal, never as an empty state

Signed-off-by: willbot <w.a.madden+machine@gmail.com>
Signed-off-by: Will Madden <madden@prisma.io>

Author: wmadden-electric

packages/cli/src/commands/skills/presentation.ts

@@ -24,14 +24,19 @@ function unmanagedClause(count: number): string {
 }
 
 function syncSummary(result: SkillsSyncResult): string {
-  if (result.agents.length === 0) {
-    return "No agents are configured to sync skills for.";
-  }
-  if (result.packages.length === 0) {
-    return "No Prisma packages with agent skills are installed.";
-  }
-  if (result.skills.length === 0) {
-    return "No Prisma dependencies in your project ship agent skills to sync.";
+  // The empty-state sentences hold only when this run also removed
+  // nothing; a prune is work done, and its summary must match the
+  // Removed table rendered beneath it.
+  if (result.pruned.length === 0) {
+    if (result.agents.length === 0) {
+      return "No agents are configured to sync skills for.";
+    }
+    if (result.packages.length === 0) {
+      return "No Prisma packages with agent skills are installed.";
+    }
+    if (result.skills.length === 0) {
+      return "No Prisma dependencies in your project ship agent skills to sync.";
+    }
   }
   const refusedDirs = result.refused.reduce(
     (count, skill) => count + skill.dirs.length,
@@ -40,6 +45,10 @@ function syncSummary(result: SkillsSyncResult): string {
   if (result.synced.length === 0 && result.pruned.length === 0) {
     return `Agent skills are up to date${unmanagedClause(refusedDirs)}.`;
   }
+  const removed = `${result.pruned.length} skill${result.pruned.length === 1 ? "" : "s"}`;
+  if (result.synced.length === 0 && result.pruned.length > 0) {
+    return `Removed ${removed}${unmanagedClause(refusedDirs)}.`;
+  }
   const synced = `${result.synced.length} skill${result.synced.length === 1 ? "" : "s"}`;
   const base =
     result.pruned.length === 0

packages/cli/tests/skills-sync.test.ts

@@ -330,6 +330,25 @@ describe("skills sync", () => {
     ).toBe(true);
   });
 
+  it("summarizes a prune-only run as removal, not as an empty state", async () => {
+    const root = await makeProjectRoot();
+    await seedSyncedSkill(root, ".claude/skills", {
+      skill: "prisma-8",
+      library: "@prisma/orm-postgres",
+      version: "8.1.0",
+    });
+
+    const run = await makeCli().run(["skills", "sync"], {
+      cwd: root,
+      isTty: { stdout: true, stderr: true },
+    });
+
+    expect(run.exitCode).toBe(0);
+    expect(run.stderr).toContain("Removed 1 skill.");
+    expect(run.stderr).not.toContain("are installed");
+    expect(run.stderr).not.toContain("ship agent skills");
+  });
+
   it("keeps a skill still shipped by another installed package", async () => {
     const root = await makeProjectRoot();
     await installPackage(root, {