fix(opencode): store active plan backing file outside workspace (#743)

The backing file used for edit-based plan submission was stored at
.opencode/plans/_active-plan.md inside the workspace, causing it to
appear in git status and editor file trees. It is now stored at
~/.plannotator/active/{project}/_active-plan.md alongside the version
history.

- Move getPlanBackingPath to derive path from project name under
  ~/.plannotator/active/
- Derive project name from ctx.directory basename via sanitizeTag at
  call site
- Delete backing file on approval since it is no longer needed after the
  session ends
- Update tests to reflect new path contract

Author: rcdailey

apps/opencode-plugin/index.ts

@@ -16,7 +16,8 @@
  */
 
 import { type Plugin, tool } from "@opencode-ai/plugin";
-import { existsSync, mkdirSync, readFileSync, writeFileSync } from "fs";
+import { existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from "fs";
+import { homedir } from "os";
 import path from "path";
 
 // OpenCode's @hono/node-server patches global.Response with a polyfill that
@@ -67,6 +68,7 @@ import { composeImproveContext } from "@plannotator/shared/pfm-reminder";
 import {
   stripConflictingPlanModeRules,
 } from "./plan-mode";
+import { sanitizeTag } from "@plannotator/shared/project";
 import {
   applyWorkflowConfig,
   isPlanningAgent,
@@ -127,12 +129,13 @@ interface PlanEdit {
 }
 
 /**
- * Backing file for the current plan. Managed entirely by the plugin;
+ * Backing file for the current plan. Stored outside the workspace in
+ * `~/.plannotator/active/{project}/_active-plan.md` so it never appears
+ * in git status or editor file trees. Managed entirely by the plugin;
  * the agent never sees or touches this file directly.
  */
-export function getPlanBackingPath(directory: string): string {
-  const planDir = path.join(directory, ".opencode", "plans");
-  return path.join(planDir, "_active-plan.md");
+export function getPlanBackingPath(project: string): string {
+  return path.join(homedir(), ".plannotator", "active", project, "_active-plan.md");
 }
 
 /**
@@ -564,7 +567,8 @@ Use /plannotator-last or /plannotator-annotate for manual review, or set workflo
           }
 
           // Read existing backing file (empty on first call)
-          const backingPath = getPlanBackingPath(ctx.directory);
+          const project = sanitizeTag(path.basename(ctx.directory)) || "_unknown";
+          const backingPath = getPlanBackingPath(project);
           const backingDir = path.dirname(backingPath);
           mkdirSync(backingDir, { recursive: true });
 
@@ -640,6 +644,9 @@ Use /plannotator-last or /plannotator-annotate for manual review, or set workflo
           server.stop();
 
           if (result.approved) {
+            // Clean up backing file after approval
+            try { unlinkSync(backingPath); } catch { /* already gone */ }
+
             const shouldSwitchAgent = result.agentSwitch && result.agentSwitch !== 'disabled';
             const targetAgent = result.agentSwitch || 'build';
 

apps/opencode-plugin/submit-plan.test.ts

@@ -1,4 +1,6 @@
 import { describe, expect, test } from "bun:test";
+import { homedir } from "os";
+import path from "path";
 import {
   applyEdits,
   formatWithLineNumbers,
@@ -173,14 +175,14 @@ describe("formatWithLineNumbers", () => {
 // ── getPlanBackingPath ─────────────────────────────────────────────────────
 
 describe("getPlanBackingPath", () => {
-  test("returns path inside .opencode/plans within the given directory", () => {
-    const result = getPlanBackingPath("/some/project");
-    expect(result).toBe("/some/project/.opencode/plans/_active-plan.md");
+  test("returns path inside ~/.plannotator/active/{project}/_active-plan.md", () => {
+    const result = getPlanBackingPath("myproject");
+    expect(result).toBe(path.join(homedir(), ".plannotator", "active", "myproject", "_active-plan.md"));
   });
 
-  test("uses the provided directory as the root", () => {
-    const result = getPlanBackingPath("/home/user/myproject");
-    expect(result).toContain("/home/user/myproject");
+  test("uses the provided project name as the directory segment", () => {
+    const result = getPlanBackingPath("some-project");
+    expect(result).toContain(path.join(".plannotator", "active", "some-project"));
     expect(result).toContain("_active-plan.md");
   });
 });