fix: restore Pi plan file hint and vendor template for source installs

Add optional planFilePath to planDenyFeedback so Pi can tell the agent
to read the plan file before editing. Check in a vendored copy of the
template so Pi source installs work without running build:pi first.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: backnotprop

apps/pi-extension/feedback-templates.ts

@@ -0,0 +1,20 @@
+/**
+ * Vendored copy of packages/shared/feedback-templates.ts for source installs.
+ * Keep this file in sync with the shared source via `bun run build:pi`.
+ */
+
+export interface PlanDenyFeedbackOptions {
+  planFilePath?: string;
+}
+
+export const planDenyFeedback = (
+  feedback: string,
+  toolName: string = "ExitPlanMode",
+  options?: PlanDenyFeedbackOptions,
+): string => {
+  const planFileRule = options?.planFilePath
+    ? `- Read ${options.planFilePath} to see the current plan before editing it.\n`
+    : "";
+
+  return `YOUR PLAN WAS NOT APPROVED.\n\nYou MUST revise the plan to address ALL of the feedback below before calling ${toolName} again.\n\nRules:\n${planFileRule}- Use the Edit tool to make targeted changes to the plan — do not resubmit the same plan unchanged.\n- Do NOT change the plan title (first # heading) unless the user explicitly asks you to.\n\n${feedback || "Plan changes requested"}`;
+};

apps/pi-extension/index.ts

@@ -446,7 +446,7 @@ export default function plannotator(pi: ExtensionAPI): void {
         content: [
           {
             type: "text",
-            text: planDenyFeedback(feedbackText, "exit_plan_mode"),
+            text: planDenyFeedback(feedbackText, "exit_plan_mode", { planFilePath }),
           },
         ],
         details: { approved: false, feedback: feedbackText },

packages/shared/feedback-templates.test.ts

@@ -51,4 +51,13 @@ describe("feedback-templates", () => {
     expect(result.toLowerCase()).toContain("title");
     expect(result.toLowerCase()).toContain("heading");
   });
+
+  test("plan deny can include a plan file hint for file-based integrations", () => {
+    const result = planDenyFeedback("feedback", "exit_plan_mode", {
+      planFilePath: "plans/auth.md",
+    });
+
+    expect(result).toContain("Read plans/auth.md to see the current plan before editing it.");
+    expect(result).toContain("exit_plan_mode");
+  });
 });

packages/shared/feedback-templates.ts

@@ -5,8 +5,18 @@
  * directive framing — Claude was ignoring softer phrasing.
  */
 
+export interface PlanDenyFeedbackOptions {
+  planFilePath?: string;
+}
+
 export const planDenyFeedback = (
   feedback: string,
   toolName: string = "ExitPlanMode",
-): string =>
-  `YOUR PLAN WAS NOT APPROVED.\n\nYou MUST revise the plan to address ALL of the feedback below before calling ${toolName} again.\n\nRules:\n- Use the Edit tool to make targeted changes to the plan — do not resubmit the same plan unchanged.\n- Do NOT change the plan title (first # heading) unless the user explicitly asks you to.\n\n${feedback || "Plan changes requested"}`;
+  options?: PlanDenyFeedbackOptions,
+): string => {
+  const planFileRule = options?.planFilePath
+    ? `- Read ${options.planFilePath} to see the current plan before editing it.\n`
+    : "";
+
+  return `YOUR PLAN WAS NOT APPROVED.\n\nYou MUST revise the plan to address ALL of the feedback below before calling ${toolName} again.\n\nRules:\n${planFileRule}- Use the Edit tool to make targeted changes to the plan — do not resubmit the same plan unchanged.\n- Do NOT change the plan title (first # heading) unless the user explicitly asks you to.\n\n${feedback || "Plan changes requested"}`;
+};