open-plan-annotator 1.0.20 → 1.1.0

package/.claude-plugin/marketplace.json

@@ -5,7 +5,7 @@
   },
   "metadata": {
     "description": "Interactive plan annotation plugin for Claude Code",
-    "version": "1.0.20"
+    "version": "1.1.0"
   },
   "plugins": [
     {
@@ -13,11 +13,11 @@
       "source": {
         "npm": {
           "package": "open-plan-annotator",
-          "version": "1.0.20"
+          "version": "1.1.0"
         }
       },
       "description": "Interactive plan annotation UI: review, strikethrough, and comment on Claude's plans before approving. Fully local, no external services.",
-      "version": "1.0.20",
+      "version": "1.1.0",
       "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": "1.0.20",
+  "version": "1.1.0",
   "author": {
     "name": "ndom91"
   },

package/bin/open-plan-annotator.mjs

@@ -6,6 +6,7 @@ import path from "node:path";
 import { fileURLToPath } from "node:url";
 import { buildCliHelpText, buildUnknownCommandPrefix } from "../shared/cliHelp.mjs";
 import { resolveCliMode } from "../shared/cliMode.mjs";
+import { detectPackageManager } from "../shared/packageManager.mjs";
 import { resolveRuntimeBinary } from "../shared/runtimeResolver.mjs";
 import { buildUpdateInstructions } from "../shared/updateHints.mjs";
 
@@ -50,19 +51,10 @@ if (cliMode === "hook") {
 }
 
 if (cliMode === "update") {
-  console.log(buildUpdateInstructions({ host: process.env.OPEN_PLAN_HOST, packageManager: detectPackageManager() }));
+  console.log(buildUpdateInstructions({ host: process.env.OPEN_PLAN_HOST, packageManager: detectPackageManager({ installPath: fileURLToPath(import.meta.url) }) }));
   process.exit(0);
 }
 
-// Detect package manager so the binary can suggest the right update command
-function detectPackageManager() {
-  const ua = process.env.npm_config_user_agent || "";
-  if (ua.startsWith("pnpm")) return "pnpm";
-  if (ua.startsWith("yarn")) return "yarn";
-  if (ua.startsWith("bun")) return "bun";
-  return "npm";
-}
-
 let runtime;
 try {
   runtime = resolveRuntimeBinary({ parentUrl: import.meta.url });
@@ -77,7 +69,7 @@ const child = spawn(runtime.binaryPath, process.argv.slice(2), {
   env: {
     ...process.env,
     OPEN_PLAN_HOST: process.env.OPEN_PLAN_HOST || "claude-code",
-    OPEN_PLAN_PKG_MANAGER: detectPackageManager(),
+    OPEN_PLAN_PKG_MANAGER: detectPackageManager({ installPath: fileURLToPath(import.meta.url) }),
   },
 });
 
@@ -136,7 +128,7 @@ function printDoctor() {
       `platform: ${platformKey}`,
       `runtime package: ${runtime.packageName}`,
       `runtime path: ${runtime.binaryPath}`,
-      `update: ${buildUpdateInstructions({ host: process.env.OPEN_PLAN_HOST, packageManager: detectPackageManager() })}`,
+      `update: ${buildUpdateInstructions({ host: process.env.OPEN_PLAN_HOST, packageManager: detectPackageManager({ installPath: fileURLToPath(import.meta.url) }) })}`,
     ].join("\n"));
   } catch (error) {
     console.log([
@@ -144,7 +136,7 @@ function printDoctor() {
       `platform: ${platformKey}`,
       `runtime: missing`,
       `error: ${error instanceof Error ? error.message : String(error)}`,
-      `update: ${buildUpdateInstructions({ host: process.env.OPEN_PLAN_HOST, packageManager: detectPackageManager() })}`,
+      `update: ${buildUpdateInstructions({ host: process.env.OPEN_PLAN_HOST, packageManager: detectPackageManager({ installPath: fileURLToPath(import.meta.url) }) })}`,
     ].join("\n"));
   }
 }

package/opencode/bridge.js

@@ -3,6 +3,7 @@ import { randomUUID } from "node:crypto";
 import { existsSync, statSync } from "node:fs";
 import { dirname } from "node:path";
 import { fileURLToPath } from "node:url";
+import { detectPackageManager } from "../shared/packageManager.mjs";
 import { resolveRuntimeBinary } from "../shared/runtimeResolver.mjs";
 
 const PKG_ROOT = fileURLToPath(new URL("..", import.meta.url));
@@ -98,14 +99,6 @@ function validateHookOutput(value) {
   throw new Error("unsupported decision payload");
 }
 
-function detectPackageManager() {
-  const ua = process.env.npm_config_user_agent || "";
-  if (ua.startsWith("pnpm")) return "pnpm";
-  if (ua.startsWith("yarn")) return "yarn";
-  if (ua.startsWith("bun")) return "bun";
-  return "npm";
-}
-
 /**
  * @param {{ plan: string, sessionId?: string, cwd?: string }} options
  */
@@ -134,7 +127,8 @@ export async function runPlanReview(options) {
       env: {
         ...process.env,
         OPEN_PLAN_HOST: "opencode",
-        OPEN_PLAN_PKG_MANAGER: process.env.OPEN_PLAN_PKG_MANAGER || detectPackageManager(),
+        OPEN_PLAN_PKG_MANAGER:
+          process.env.OPEN_PLAN_PKG_MANAGER || detectPackageManager({ installPath: fileURLToPath(import.meta.url) }),
       },
       detached: true,
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "open-plan-annotator",
-  "version": "1.0.20",
+  "version": "1.1.0",
   "type": "module",
   "description": "Fully local plugin for interactive plan annotation from your Agentic assistants",
   "author": "ndom91",
@@ -61,9 +61,9 @@
     "@opencode-ai/plugin": "^1.2.14"
   },
   "optionalDependencies": {
-    "@open-plan-annotator/runtime-darwin-arm64": "1.0.20",
-    "@open-plan-annotator/runtime-darwin-x64": "1.0.20",
-    "@open-plan-annotator/runtime-linux-arm64": "1.0.20",
-    "@open-plan-annotator/runtime-linux-x64": "1.0.20"
+    "@open-plan-annotator/runtime-darwin-arm64": "1.1.0",
+    "@open-plan-annotator/runtime-darwin-x64": "1.1.0",
+    "@open-plan-annotator/runtime-linux-arm64": "1.1.0",
+    "@open-plan-annotator/runtime-linux-x64": "1.1.0"
   }
 }

package/shared/packageManager.mjs

@@ -0,0 +1,39 @@
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+function detectFromValue(value) {
+  if (!value) return undefined;
+  const normalized = value.toLowerCase();
+  if (normalized.includes("pnpm")) return "pnpm";
+  if (normalized.includes("yarn")) return "yarn";
+  if (normalized.includes("bun")) return "bun";
+  if (normalized.includes("npm")) return "npm";
+  return undefined;
+}
+
+export function detectPackageManager(options = {}) {
+  const env = options.env ?? process.env;
+  const installPath = options.installPath;
+
+  const explicit = detectFromValue(env.OPEN_PLAN_PKG_MANAGER);
+  if (explicit) return explicit;
+
+  const userAgent = env.npm_config_user_agent;
+  if (typeof userAgent === "string") {
+    const name = userAgent.split("/")[0];
+    const detected = detectFromValue(name);
+    if (detected) return detected;
+  }
+
+  const packageManager = detectFromValue(env.npm_package_manager);
+  if (packageManager) return packageManager;
+
+  const execPath = detectFromValue(env.npm_execpath);
+  if (execPath) return execPath;
+
+  const resolvedInstallPath = installPath ?? fileURLToPath(import.meta.url);
+  const pathHint = detectFromValue(path.normalize(resolvedInstallPath));
+  if (pathHint) return pathHint;
+
+  return "npm";
+}

package/shared/packageManager.test.ts

@@ -0,0 +1,33 @@
+import { describe, expect, test } from "bun:test";
+import { detectPackageManager } from "./packageManager.mjs";
+
+describe("detectPackageManager", () => {
+  test("prefers explicit OPEN_PLAN_PKG_MANAGER", () => {
+    expect(detectPackageManager({ env: { OPEN_PLAN_PKG_MANAGER: "pnpm" } })).toBe("pnpm");
+  });
+
+  test("detects package manager from npm user agent", () => {
+    expect(detectPackageManager({ env: { npm_config_user_agent: "pnpm/10.0.0 node/v22.0.0" } })).toBe("pnpm");
+  });
+
+  test("detects package manager from npm execpath", () => {
+    expect(detectPackageManager({ env: { npm_execpath: "/usr/local/lib/node_modules/pnpm/bin/pnpm.cjs" } })).toBe(
+      "pnpm",
+    );
+  });
+
+  test("detects package manager from install path hints", () => {
+    expect(
+      detectPackageManager({
+        env: {},
+        installPath: "/Users/test/Library/pnpm/global/5/node_modules/open-plan-annotator/bin/open-plan-annotator.mjs",
+      }),
+    ).toBe("pnpm");
+  });
+
+  test("falls back to npm", () => {
+    expect(detectPackageManager({ env: {}, installPath: "/tmp/open-plan-annotator/bin/open-plan-annotator.mjs" })).toBe(
+      "npm",
+    );
+  });
+});