open-plan-annotator 1.0.15 → 1.0.17

package/.claude-plugin/marketplace.json

@@ -5,14 +5,14 @@
   },
   "metadata": {
     "description": "Interactive plan annotation plugin for Claude Code",
-    "version": "1.0.15"
+    "version": "1.0.17"
   },
   "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.0.15",
+      "version": "1.0.17",
       "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.15",
+  "version": "1.0.17",
   "author": {
     "name": "ndom91"
   },

package/README.md

@@ -22,6 +22,13 @@ Everything runs locally. Nothing leaves your machine.
 
 ## Install
 
+> [!NOTE]
+> The first run might take up to 30s to pop open the UI. If your agentic coding
+> tool installed it via pnpm, they block post-install scripts by default so
+> Claude / OpenCode will trigger the download of the correct binary for your
+> platform upon first use, i.e. when first transitioning out of plan mode.
+> All subsequent runs will be instant.
+
 ### Claude Code
 
 From within Claude Code, add the marketplace and install the plugin:
@@ -73,10 +80,6 @@ If you want to run the binary standalone or build from source:
 npm install -g open-plan-annotator
 ```
 
-> [!NOTE]
-> The first run might take a few seconds if you use pnpm, as it blocks postinstall scripts.
-> Claude / OpenCode will trigger the download upon first use then.
-
 #### From Source
 
 ```sh

package/opencode/index.js

@@ -162,7 +162,7 @@ export const OpenPlanAnnotatorPlugin = async (ctx) => {
     tool: {
       submit_plan: tool({
         description:
-          "Submit a markdown plan for interactive user review. Returns a structured result with plan_status, next_state, approved, and feedback fields.",
+          "Submit a markdown plan for interactive user review. Returns a structured result with plan_status, next_state, feedback, and guidance fields.",
 
         args: {
           plan: tool.schema.string().describe("The complete implementation plan in markdown format"),
@@ -176,11 +176,12 @@ export const OpenPlanAnnotatorPlugin = async (ctx) => {
             cwd: ctx.directory,
           });
 
+          const feedback = result.approved ? "" : (result.feedback ?? "Plan changes requested.");
+
           const basePayload = {
             plan_status: result.approved ? "approved" : "rejected",
             next_state: result.approved ? "EXECUTION" : "PLAN_DRAFT",
-            approved: result.approved,
-            feedback: result.approved ? null : (result.feedback ?? "Plan changes requested."),
+            feedback,
           };
 
           if (result.approved) {
@@ -216,7 +217,7 @@ export const OpenPlanAnnotatorPlugin = async (ctx) => {
               "",
               "## User feedback",
               "",
-              basePayload.feedback,
+              feedback,
               "",
               "Revise the plan using this feedback, then submit the revised draft once via `submit_plan`.",
             ].join("\n"),

package/opencode/index.test.ts

@@ -32,8 +32,8 @@ describe("submit_plan return schema contract", () => {
 
     expect(result.plan_status).toBe("approved");
     expect(result.next_state).toBe("EXECUTION");
-    expect(result.approved).toBe(true);
-    expect(result.feedback).toBeNull();
+    expect(result.feedback).toBe("");
+    expect(result.guidance).toContain("Plan approved by the user.");
   });
 
   test("rejected decision maps to plan redraft payload with bridge feedback", async () => {
@@ -50,7 +50,27 @@ describe("submit_plan return schema contract", () => {
 
     expect(result.plan_status).toBe("rejected");
     expect(result.next_state).toBe("PLAN_DRAFT");
-    expect(result.approved).toBe(false);
     expect(result.feedback).toBe(bridgeFeedback);
+    expect(result.guidance).toContain("## User feedback");
+  });
+
+  test("all top-level submit_plan fields are strings for display safety", async () => {
+    mock.module("./bridge.js", () => ({
+      runPlanReview: async () => ({ approved: true }),
+    }));
+
+    const { OpenPlanAnnotatorPlugin } = await import(`./index.js?display-safe-${Date.now()}`);
+    const plugin = await OpenPlanAnnotatorPlugin(createPluginContext());
+
+    const result = await plugin.tool.submit_plan.execute(
+      { plan: "# Plan", summary: "Short summary" },
+      { sessionID: "session-3" },
+    );
+
+    for (const value of Object.values(result)) {
+      expect(typeof value).toBe("string");
+    }
+
+    expect(result).not.toHaveProperty("approved");
   });
 });

package/package.json

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