open-plan-annotator 1.1.1 → 1.1.3

package/.claude-plugin/marketplace.json

@@ -5,14 +5,14 @@
   },
   "metadata": {
     "description": "Interactive plan annotation plugin for Claude Code",
-    "version": "1.1.1"
+    "version": "1.1.3"
   },
   "plugins": [
     {
       "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": "1.1.1",
+      "version": "1.1.3",
       "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.1.1",
+  "version": "1.1.3",
   "author": {
     "name": "ndom91"
   },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "open-plan-annotator",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "type": "module",
   "description": "Fully local plugin for interactive plan annotation from your Agentic assistants",
   "author": "ndom91",
@@ -26,9 +26,15 @@
   },
   "files": [
     "bin/open-plan-annotator.mjs",
-    "shared/",
+    "shared/cliHelp.mjs",
+    "shared/cliMode.mjs",
+    "shared/packageManager.mjs",
+    "shared/runtimeResolver.mjs",
+    "shared/updateHints.mjs",
     ".claude-plugin/",
-    "opencode/",
+    "opencode/bridge.js",
+    "opencode/config.js",
+    "opencode/index.js",
     "hooks/",
     "CLAUDE.md",
     "README.md"
@@ -61,10 +67,9 @@
     "@opencode-ai/plugin": "^1.2.14"
   },
   "optionalDependencies": {
-    "@open-plan-annotator/runtime-darwin-arm64": "1.1.1",
-    "@open-plan-annotator/runtime-darwin-x64": "1.1.1",
-    "@open-plan-annotator/runtime-linux-arm64": "1.1.1",
-    "@open-plan-annotator/runtime-linux-x64": "1.1.1"
-  },
-  "packageManager": "bun@1.3.9"
+    "@open-plan-annotator/runtime-darwin-arm64": "1.1.3",
+    "@open-plan-annotator/runtime-darwin-x64": "1.1.3",
+    "@open-plan-annotator/runtime-linux-arm64": "1.1.3",
+    "@open-plan-annotator/runtime-linux-x64": "1.1.3"
+  }
 }

package/opencode/config.test.ts

@@ -1,110 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { resolveImplementationHandoff } from "./config.js";
-
-function withEnv(key: string, value: string | undefined, fn: () => Promise<void>) {
-  const original = process.env[key];
-  if (value === undefined) {
-    delete process.env[key];
-  } else {
-    process.env[key] = value;
-  }
-
-  return fn().finally(() => {
-    if (original === undefined) {
-      delete process.env[key];
-    } else {
-      process.env[key] = original;
-    }
-  });
-}
-
-describe("resolveImplementationHandoff", () => {
-  test("returns defaults when no config files exist", async () => {
-    const tempRoot = mkdtempSync(join(tmpdir(), "open-plan-annotator-config-"));
-
-    try {
-      await withEnv("XDG_CONFIG_HOME", join(tempRoot, "xdg"), async () => {
-        const result = await resolveImplementationHandoff(join(tempRoot, "project"));
-        expect(result.enabled).toBe(true);
-        expect(result.agent).toBe("build");
-      });
-    } finally {
-      rmSync(tempRoot, { recursive: true, force: true });
-    }
-  });
-
-  test("uses global config when project config is missing", async () => {
-    const tempRoot = mkdtempSync(join(tmpdir(), "open-plan-annotator-config-"));
-    const xdgHome = join(tempRoot, "xdg");
-    const globalConfigDir = join(xdgHome, "opencode");
-    mkdirSync(globalConfigDir, { recursive: true });
-    writeFileSync(
-      join(globalConfigDir, "open-plan-annotator.json"),
-      JSON.stringify({ implementationHandoff: { enabled: false, agent: "explore" } }),
-      "utf8",
-    );
-
-    try {
-      await withEnv("XDG_CONFIG_HOME", xdgHome, async () => {
-        const result = await resolveImplementationHandoff(join(tempRoot, "project"));
-        expect(result.enabled).toBe(false);
-        expect(result.agent).toBe("explore");
-      });
-    } finally {
-      rmSync(tempRoot, { recursive: true, force: true });
-    }
-  });
-
-  test("project config overrides global config", async () => {
-    const tempRoot = mkdtempSync(join(tmpdir(), "open-plan-annotator-config-"));
-    const xdgHome = join(tempRoot, "xdg");
-    const globalConfigDir = join(xdgHome, "opencode");
-    const projectDir = join(tempRoot, "project");
-    const projectConfigDir = join(projectDir, ".opencode");
-
-    mkdirSync(globalConfigDir, { recursive: true });
-    mkdirSync(projectConfigDir, { recursive: true });
-
-    writeFileSync(
-      join(globalConfigDir, "open-plan-annotator.json"),
-      JSON.stringify({ implementationHandoff: { enabled: true, agent: "explore" } }),
-      "utf8",
-    );
-    writeFileSync(
-      join(projectConfigDir, "open-plan-annotator.json"),
-      JSON.stringify({ implementationHandoff: { enabled: false, agent: "build" } }),
-      "utf8",
-    );
-
-    try {
-      await withEnv("XDG_CONFIG_HOME", xdgHome, async () => {
-        const result = await resolveImplementationHandoff(projectDir);
-        expect(result.enabled).toBe(false);
-        expect(result.agent).toBe("build");
-      });
-    } finally {
-      rmSync(tempRoot, { recursive: true, force: true });
-    }
-  });
-
-  test("ignores malformed config and falls back", async () => {
-    const tempRoot = mkdtempSync(join(tmpdir(), "open-plan-annotator-config-"));
-    const xdgHome = join(tempRoot, "xdg");
-    const globalConfigDir = join(xdgHome, "opencode");
-    mkdirSync(globalConfigDir, { recursive: true });
-    writeFileSync(join(globalConfigDir, "open-plan-annotator.json"), "not-json", "utf8");
-
-    try {
-      await withEnv("XDG_CONFIG_HOME", xdgHome, async () => {
-        const result = await resolveImplementationHandoff(join(tempRoot, "project"));
-        expect(result.enabled).toBe(true);
-        expect(result.agent).toBe("build");
-      });
-    } finally {
-      rmSync(tempRoot, { recursive: true, force: true });
-    }
-  });
-});

package/opencode/index.test.ts

@@ -1,54 +0,0 @@
-import { afterEach, describe, expect, mock, test } from "bun:test";
-
-afterEach(() => {
-  mock.restore();
-});
-
-function createPluginContext() {
-  return {
-    directory: process.cwd(),
-    client: {
-      session: {
-        messages: async () => ({ data: [] }),
-        prompt: async () => ({ data: null }),
-      },
-      app: {
-        agents: async () => ({ data: [] }),
-      },
-    },
-  };
-}
-
-describe("submit_plan tool output", () => {
-  test("returns plain text execution instructions after approval", async () => {
-    mock.module("./bridge.js", () => ({
-      runPlanReview: async () => ({ approved: true }),
-    }));
-
-    const { OpenPlanAnnotatorPlugin } = await import(`./index.js?approved-${Date.now()}`);
-    const plugin = await OpenPlanAnnotatorPlugin(createPluginContext());
-
-    const result = await plugin.tool.submit_plan.execute({ plan: "# Plan" }, { sessionID: "session-1" });
-
-    expect(typeof result).toBe("string");
-    expect(result).toContain("plan_status=approved");
-    expect(result).toContain("next_state=EXECUTION");
-    expect(result).toContain("Do not call `submit_plan` again");
-  });
-
-  test("returns plain text revision instructions after rejection", async () => {
-    mock.module("./bridge.js", () => ({
-      runPlanReview: async () => ({ approved: false, feedback: "Need rollback steps." }),
-    }));
-
-    const { OpenPlanAnnotatorPlugin } = await import(`./index.js?rejected-${Date.now()}`);
-    const plugin = await OpenPlanAnnotatorPlugin(createPluginContext());
-
-    const result = await plugin.tool.submit_plan.execute({ plan: "# Plan" }, { sessionID: "session-2" });
-
-    expect(typeof result).toBe("string");
-    expect(result).toContain("plan_status=rejected");
-    expect(result).toContain("next_state=PLAN_DRAFT");
-    expect(result).toContain("Need rollback steps.");
-  });
-});

package/shared/cliHelp.test.ts

@@ -1,29 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { buildCliHelpText, buildUnknownCommandPrefix } from "./cliHelp.mjs";
-
-describe("buildCliHelpText", () => {
-  test("builds canonical help text for all entrypoints", () => {
-    expect(buildCliHelpText("1.2.3")).toBe(`open-plan-annotator v1.2.3
-
-Usage:
-  open-plan-annotator              Show this help
-  open-plan-annotator < event.json Run as a Claude Code hook (debug)
-  open-plan-annotator doctor       Show resolved runtime details
-  open-plan-annotator update       Show package-managed update guidance
-  open-plan-annotator upgrade      Alias for update
-  open-plan-annotator --version    Print version
-  open-plan-annotator --help       Show this help
-
-https://github.com/ndom91/open-plan-annotator`);
-  });
-});
-
-describe("buildUnknownCommandPrefix", () => {
-  test("formats unknown command prefix consistently", () => {
-    expect(buildUnknownCommandPrefix("wat")).toBe("open-plan-annotator: unknown command `wat`");
-  });
-
-  test("handles missing command safely", () => {
-    expect(buildUnknownCommandPrefix(undefined)).toBe("open-plan-annotator: unknown command ``");
-  });
-});

package/shared/cliMode.test.ts

@@ -1,35 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { resolveCliMode } from "./cliMode.mjs";
-
-describe("resolveCliMode", () => {
-  test("returns help for no-arg interactive invocation", () => {
-    expect(resolveCliMode(undefined, { stdinIsTTY: true })).toBe("help");
-  });
-
-  test("returns hook for no-arg piped invocation", () => {
-    expect(resolveCliMode(undefined, { stdinIsTTY: false })).toBe("hook");
-  });
-
-  test("defaults no-arg invocation to hook mode", () => {
-    expect(resolveCliMode(undefined)).toBe("hook");
-  });
-
-  test("treats upgrade as update alias", () => {
-    expect(resolveCliMode("upgrade")).toBe("update");
-  });
-
-  test("recognizes doctor subcommand", () => {
-    expect(resolveCliMode("doctor")).toBe("doctor");
-  });
-
-  test("recognizes help and version flags", () => {
-    expect(resolveCliMode("--help")).toBe("help");
-    expect(resolveCliMode("-h")).toBe("help");
-    expect(resolveCliMode("--version")).toBe("version");
-    expect(resolveCliMode("-v")).toBe("version");
-  });
-
-  test("marks unknown commands explicitly", () => {
-    expect(resolveCliMode("noop")).toBe("unknown");
-  });
-});

package/shared/packageManager.test.ts

@@ -1,33 +0,0 @@
-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",
-    );
-  });
-});

package/shared/runtimeResolver.test.ts

@@ -1,36 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import fs from "node:fs";
-import os from "node:os";
-import path from "node:path";
-import { pathToFileURL } from "node:url";
-import { getRuntimePackageName, resolveRuntimeBinary } from "./runtimeResolver.mjs";
-
-describe("runtimeResolver", () => {
-  test("maps supported platforms to runtime packages", () => {
-    expect(getRuntimePackageName("darwin", "arm64")).toBe("@open-plan-annotator/runtime-darwin-arm64");
-    expect(getRuntimePackageName("linux", "x64")).toBe("@open-plan-annotator/runtime-linux-x64");
-  });
-
-  test("throws for unsupported platforms", () => {
-    expect(() => resolveRuntimeBinary({ platform: "win32", arch: "x64" })).toThrow("Unsupported platform win32-x64");
-  });
-
-  test("resolves binary from installed runtime package", () => {
-    const tempRoot = fs.mkdtempSync(path.join(os.tmpdir(), "opa-runtime-"));
-    const packageRoot = path.join(tempRoot, "node_modules", "@open-plan-annotator", "runtime-linux-x64");
-    const binaryPath = path.join(packageRoot, "bin", "open-plan-annotator");
-
-    fs.mkdirSync(path.dirname(binaryPath), { recursive: true });
-    fs.writeFileSync(path.join(packageRoot, "package.json"), '{"name":"@open-plan-annotator/runtime-linux-x64"}');
-    fs.writeFileSync(binaryPath, "binary");
-
-    const resolved = resolveRuntimeBinary({
-      platform: "linux",
-      arch: "x64",
-      parentUrl: pathToFileURL(path.join(tempRoot, "index.mjs")).href,
-    });
-
-    expect(resolved.packageName).toBe("@open-plan-annotator/runtime-linux-x64");
-    expect(fs.realpathSync(resolved.binaryPath)).toBe(fs.realpathSync(binaryPath));
-  });
-});