fix(core): wire persistence cleanup into startup — completes #218

plugins/goal-guard/guard.js

@@ -185,6 +185,16 @@ export function createGuard(input = {}, options = {}, overrides = {}) {
     /* ignore corrupt state */
   }
 
+  // Opportunistically prune orphaned snapshot files left by worktrees that no
+  // longer run (issue #218): without this the persistence dir grows forever.
+  // Best-effort — a cleanup failure must never break guard startup, and the
+  // optional call keeps injected test doubles (which omit cleanup) working.
+  try {
+    persistence.cleanup?.();
+  } catch {
+    /* best-effort */
+  }
+
   // Persist the resolved config alongside the sessions so the TUI sidebar (which reads
   // only the on-disk snapshot) evaluates completion with the SAME config the server
   // used — otherwise a non-default `contextualGates` would make the sidebar disagree

tests/integration.test.mjs

@@ -13,6 +13,18 @@ function makeGuard(options = {}) {
   );
 }
 
+test("integration: guard startup prunes orphaned persistence files (#218 wiring)", () => {
+  let cleaned = 0;
+  const persistence = { ...noopPersistence, cleanup: () => { cleaned += 1; } };
+  let t = 0;
+  createGuard(
+    { client: { app: { log: async () => undefined }, tui: { showToast: async () => undefined } } },
+    {},
+    { persistence, clock: () => (t += 1), syncIdle: true },
+  );
+  assert.equal(cleaned, 1, "createGuard must invoke persistence.cleanup() once at startup");
+});
+
 /**
  * End-to-end simulation of a realistic goal lifecycle driven entirely through
  * the plugin hooks, asserting the guard blocks completion until the full