open-plan-annotator 0.2.13 → 0.2.15

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.13",
+      "version": "0.2.15",
       "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.13",
+  "version": "0.2.15",
   "author": {
     "name": "ndom91"
   },

package/bin/open-plan-annotator.cjs

@@ -65,13 +65,12 @@ child.stdout.on("data", (chunk) => {
     if (!trimmed) continue;
     try {
       JSON.parse(trimmed);
-      // Valid JSON — forward it and exit, letting the binary run in the background
+      // Valid JSON — write directly to fd 1 (bypasses Node stream buffering),
+      // detach child, and exit immediately.
       forwarded = true;
-      process.stdout.write(trimmed + "\n", () => {
-        child.unref();
-        process.exit(0);
-      });
-      return;
+      fs.writeSync(1, trimmed + "\n");
+      child.unref();
+      process.exit(0);
     } catch {
       // Not JSON yet, keep buffering
     }
@@ -81,7 +80,9 @@ child.stdout.on("data", (chunk) => {
 child.on("close", (code) => {
   if (!forwarded) {
     // Binary exited without producing valid JSON — forward whatever we have
-    if (stdout.trim()) process.stdout.write(stdout);
+    if (stdout.trim()) {
+      fs.writeSync(1, stdout);
+    }
     process.exit(code || 1);
   }
 });

package/opencode/bridge.js

@@ -1,10 +1,12 @@
-import { spawn } from "node:child_process";
+import { execFileSync, spawn } from "node:child_process";
 import { randomUUID } from "node:crypto";
 import { existsSync, statSync } from "node:fs";
-import { dirname } from "node:path";
+import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 
-const WRAPPER_PATH = fileURLToPath(new URL("../bin/open-plan-annotator.cjs", import.meta.url));
+const PKG_ROOT = fileURLToPath(new URL("..", import.meta.url));
+const BINARY_PATH = join(PKG_ROOT, "bin", "open-plan-annotator-binary");
+const INSTALL_SCRIPT = join(PKG_ROOT, "install.cjs");
 
 /**
  * @typedef {{
@@ -97,10 +99,42 @@ function validateHookOutput(value) {
   throw new Error("unsupported decision payload");
 }
 
+/** Ensure the compiled binary exists, downloading if necessary. */
+function ensureBinary() {
+  if (existsSync(BINARY_PATH)) return;
+
+  // Try to find node on PATH for running the install script
+  try {
+    execFileSync(process.execPath, [INSTALL_SCRIPT], {
+      cwd: PKG_ROOT,
+      stdio: ["ignore", "pipe", "pipe"],
+    });
+  } catch {
+    // Retry with "node" explicitly in case process.execPath is bun
+    try {
+      execFileSync("node", [INSTALL_SCRIPT], {
+        cwd: PKG_ROOT,
+        stdio: ["ignore", "pipe", "pipe"],
+      });
+    } catch {
+      // ignore — we'll check below
+    }
+  }
+
+  if (!existsSync(BINARY_PATH)) {
+    throw new Error(
+      `open-plan-annotator: binary not found at ${BINARY_PATH}. ` +
+        `Try running: node ${INSTALL_SCRIPT}`,
+    );
+  }
+}
+
 /**
  * @param {{ plan: string, sessionId?: string, cwd?: string }} options
  */
 export async function runPlanReview(options) {
+  ensureBinary();
+
   const payload = buildHookPayload(options);
 
   const result = await new Promise((resolve, reject) => {
@@ -112,11 +146,12 @@ export async function runPlanReview(options) {
         cwd = dirname(cwd);
       }
     } catch {
-      // Fall back to wrapper's directory if all else fails
-      cwd = dirname(WRAPPER_PATH);
+      cwd = PKG_ROOT;
     }
 
-    const child = spawn(process.execPath, [WRAPPER_PATH], {
+    // Spawn the compiled binary directly (skip the Node wrapper).
+    // This avoids issues with bun vs node runtime differences.
+    const child = spawn(BINARY_PATH, [], {
       cwd,
       stdio: ["pipe", "pipe", "pipe"],
       env: process.env,
@@ -177,4 +212,4 @@ export async function runPlanReview(options) {
     approved: false,
     feedback: decision.message,
   };
-}
+}

package/package.json

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