open-plan-annotator 0.2.10 → 0.2.12

package/.claude-plugin/marketplace.json

@@ -12,7 +12,7 @@
       "name": "open-plan-annotator",
       "source": "./",
       "description": "Interactive plan annotation UI: review, strikethrough, and comment on Claude's plans before approving. Fully local, no external services.",
-      "version": "0.2.10",
+      "version": "0.2.12",
       "author": {
         "name": "ndom91"
       },

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "open-plan-annotator",
   "description": "Interactive plan annotation UI: review, strikethrough, and comment on Claude's plans before approving. Fully local, no external services.",
-  "version": "0.2.10",
+  "version": "0.2.12",
   "author": {
     "name": "ndom91"
   },

package/bin/open-plan-annotator.cjs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-const { execFileSync } = require("child_process");
+const { execFileSync, spawn } = require("child_process");
 const path = require("path");
 const fs = require("fs");
 
@@ -39,11 +39,54 @@ if (!fs.existsSync(binaryPath)) {
   }
 }
 
-try {
-  execFileSync(binaryPath, process.argv.slice(2), {
-    stdio: ["pipe", "inherit", "inherit"],
-    input: stdinBuffer,
-  });
-} catch (e) {
-  process.exit(e.status || 1);
-}
+// Spawn the binary with detached so it can outlive this wrapper.
+// We pipe stdout to detect the JSON hook output, then forward it and exit
+// immediately — the binary keeps its server alive in the background.
+const child = spawn(binaryPath, process.argv.slice(2), {
+  stdio: ["pipe", "pipe", "inherit"],
+  detached: true,
+});
+
+child.stdin.write(stdinBuffer);
+child.stdin.end();
+
+let stdout = "";
+let forwarded = false;
+
+child.stdout.on("data", (chunk) => {
+  stdout += chunk;
+
+  if (forwarded) return;
+
+  // Look for a complete JSON line (the hook output)
+  const lines = stdout.split("\n");
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (!trimmed) continue;
+    try {
+      JSON.parse(trimmed);
+      // Valid JSON — forward it and exit, letting the binary run in the background
+      forwarded = true;
+      process.stdout.write(trimmed + "\n", () => {
+        child.unref();
+        process.exit(0);
+      });
+      return;
+    } catch {
+      // Not JSON yet, keep buffering
+    }
+  }
+});
+
+child.on("close", (code) => {
+  if (!forwarded) {
+    // Binary exited without producing valid JSON — forward whatever we have
+    if (stdout.trim()) process.stdout.write(stdout);
+    process.exit(code || 1);
+  }
+});
+
+child.on("error", (err) => {
+  console.error("open-plan-annotator: failed to spawn binary:", err.message);
+  process.exit(1);
+});

package/opencode/bridge.js

@@ -1,5 +1,7 @@
 import { spawn } from "node:child_process";
 import { randomUUID } from "node:crypto";
+import { existsSync, statSync } from "node:fs";
+import { dirname } from "node:path";
 import { fileURLToPath } from "node:url";
 
 const WRAPPER_PATH = fileURLToPath(new URL("../bin/open-plan-annotator.cjs", import.meta.url));
@@ -102,7 +104,18 @@ export async function runPlanReview(options) {
   const payload = buildHookPayload(options);
 
   const result = await new Promise((resolve, reject) => {
-    const cwd = options.cwd ?? process.cwd();
+    let cwd = options.cwd ?? process.cwd();
+
+    // Guard: ensure cwd is a directory, not a file
+    try {
+      if (existsSync(cwd) && !statSync(cwd).isDirectory()) {
+        cwd = dirname(cwd);
+      }
+    } catch {
+      // Fall back to wrapper's directory if all else fails
+      cwd = dirname(WRAPPER_PATH);
+    }
+
     const child = spawn(process.execPath, [WRAPPER_PATH], {
       cwd,
       stdio: ["pipe", "pipe", "pipe"],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "open-plan-annotator",
-  "version": "0.2.10",
+  "version": "0.2.12",
   "type": "module",
   "description": "Fully local plugin for interactive plan annotation from your Agentic assistants",
   "author": "ndom91",