@gavin-hu/my-pi-agent 0.15.1 → 0.17.0

package/extensions/plan/tui.ts

@@ -37,7 +37,7 @@ import {
 import { isCloseKey } from "../../lib/tui/cursor.ts";
 import { gaugeBar } from "../../lib/tui/gauge.ts";
 import { GLYPHS } from "../../lib/tui/glyphs.ts";
-import { titleRule } from "../../lib/tui/rules.ts";
+import { plainRule, titleRule } from "../../lib/tui/rules.ts";
 import { emptyState, checkbox } from "../../lib/tui/rows.ts";
 import { paint } from "../../lib/tui/tokens.ts";
 import { planTitle, type StoredPlan } from "./plans.ts";
@@ -45,7 +45,7 @@ import { changedSections, sanitizePlanText, type SectionDigest } from "./plan-te
 import { extractPlanSections } from "./steps.ts";
 
 /** What the user chose from the read/review screen. */
-export type PlanViewAction = "approve" | "refine" | "keep" | "browser";
+export type PlanViewAction = "approve" | "refine" | "keep" | "browser" | "cancel";
 
 /** A plan section's rendered start line, for navigation and the current-section label. */
 interface SectionOffset {
@@ -87,7 +87,7 @@ export interface PlanViewOptions {
 
 /** Rows the screen shows when the terminal height is unknown. */
 const DEFAULT_ROWS = 20;
-/** Header, blanks, footer, and trailing rows around a non-empty plan. */
+/** Header, blanks, footer, and closing rule around a non-empty plan. */
 const CHROME_ROWS = 5;
 /** Narrowest width where Markdown's 2-column padding still fits the body. */
 const MIN_PADDED_WIDTH = 6;
@@ -97,13 +97,14 @@ const BODY_PADDING = 2;
 const GAUGE_SEGMENTS = 10;
 /** Footer hints shared by the review and the inline editor's primary actions. */
 const PRIMARY_HINTS: ScreenHint[] = [
-	"Esc keep planning",
-	{ text: "a approve & run", color: "success" },
+	"Esc keep plan",
+	{ text: "a approve", color: "success" },
 	{ text: "r refine", color: "accent" },
 	{ text: "w browser", color: "accent" },
+	{ text: "x exit plan", color: "warning" },
 ];
 /** Scroll keys appended to the review footer while the plan overflows. */
-const SCROLL_HINTS: ScreenHint[] = ["↑/↓ or k/j scroll", "space/b page", "g/G ends"];
+const SCROLL_HINTS: ScreenHint[] = ["j/k scroll", "space page", "g/G ends"];
 /** Section-jump hint, shown when the plan has more than one heading section. */
 const SECTION_HINTS: ScreenHint[] = ["n/p section"];
 
@@ -332,6 +333,7 @@ export class PlanViewComponent implements Component {
 		else if (data === "a") this.options.onClose("approve");
 		else if (data === "r") this.startRefine();
 		else if (data === "w") this.options.onClose("browser");
+		else if (data === "x") this.options.onClose("cancel");
 	}
 
 	/** Wheel scrolling in fullscreen; regular mode leaves the wheel to the terminal. */
@@ -403,7 +405,7 @@ export class PlanViewComponent implements Component {
 				this.maxOffset > 0 ? [...PRIMARY_HINTS, ...SCROLL_HINTS, ...sectionHints] : PRIMARY_HINTS;
 			lines.push(screenHint(theme, w, hints));
 		}
-		lines.push("");
+		lines.push(plainRule(theme, w));
 		return lines;
 	}
 }

package/extensions/plan/types.ts

@@ -74,6 +74,14 @@ export interface ExitPlanModeDetails {
 	refinement?: string;
 	/** Comments the review carried when it approved or requested a refinement. */
 	annotations?: PlanAnnotation[];
+	/** True when the user left plan mode without approving the plan. */
+	cancelled?: boolean;
 	/** True when no interactive UI was available to ask for approval. */
 	unavailable?: boolean;
 }
+
+/** Structured result carried in `cancel_plan_mode`'s `details`. */
+export interface CancelPlanModeDetails {
+	/** True when plan mode was on and has now been turned off. */
+	cancelled: boolean;
+}

package/extensions/plan/web-chrome.ts

@@ -0,0 +1,101 @@
+/**
+ * The review page's chrome: what the page says about itself.
+ *
+ * It is a fragment of the same IIFE `web-client.ts` ships (`WEB_CHROME_JS` is
+ * interpolated into `REVIEW_CLIENT_JS`), kept in its own module so the state
+ * chip, the status line, and the header are read together and away from the
+ * document rendering. It paints only text it was handed.
+ */
+
+export const WEB_CHROME_JS = String.raw`  function setStatus(text, kind) {
+    if (!statusEl) return;
+    statusEl.textContent = text || "";
+    statusEl.title = text || "";
+    statusEl.className = kind ? "bf-status bf-status--" + kind : "bf-status";
+  }
+
+  // The always-visible header chip: connection, comment count, and whether
+  // there are edits the file does not have yet.
+  function updateState() {
+    if (!stateEl) return;
+    var label;
+    var tone = "";
+    if (approved) {
+      label = "Approved";
+      tone = "success";
+    } else if (saved) {
+      label = "Saved";
+      tone = "success";
+    } else if (closed) {
+      // Settled by the user, not by a dropped stream: never flash Offline.
+      label = "Closed";
+    } else if (connected === true) {
+      label = "Connected";
+      tone = "success";
+    } else if (everConnected) {
+      // Only after the stream has opened once: an error before that is a
+      // transient initial connect, so keep showing Connecting rather than Offline.
+      label = "Offline";
+      tone = "danger";
+    } else {
+      label = "Connecting\u2026";
+    }
+    var count = annotations.length;
+    label += " \u00b7 " + count + (count === 1 ? " comment" : " comments");
+    if (dirty && !saved && !approved && !closed) {
+      label += " \u00b7 unsaved";
+      if (!tone) tone = "warn";
+    }
+    stateEl.textContent = label;
+    stateEl.className = "bf-chip bf-chip--accent review-state" + (tone ? " bf-chip--" + tone : "");
+  }
+
+  function renderHeader() {
+    var titleEl = document.getElementById("plan-title");
+    if (titleEl && data.title) {
+      titleEl.textContent = data.title;
+      titleEl.title = data.title;
+    }
+    if (data.title) document.title = "Plan review \u00b7 " + data.title;
+  }
+
+  function updateProgress() {
+    if (!progressEl) return;
+    var doc = document.scrollingElement || document.documentElement;
+    var top = doc ? doc.scrollTop : 0;
+    var height = doc ? doc.scrollHeight : 0;
+    var view = doc ? doc.clientHeight : 0;
+    if (planEl && planEl.scrollHeight > planEl.clientHeight + 4) {
+      top = planEl.scrollTop; height = planEl.scrollHeight; view = planEl.clientHeight;
+    }
+    var max = height - view;
+    var percent = max > 0 ? Math.min(100, Math.max(0, (top / max) * 100)) : 0;
+    progressEl.style.width = percent + "%";
+  }
+
+  function lockActions() {
+    if (approveButton) approveButton.disabled = true;
+    if (saveButton) saveButton.disabled = true;
+    if (cancelButton) cancelButton.disabled = true;
+  }
+
+  // A conflict means the file moved under the page; only a reload can re-anchor.
+  function offerReload() {
+    if (document.getElementById("reload")) return;
+    var controls = document.querySelector(".review-controls");
+    if (!controls) return;
+    var reload = el("button");
+    reload.type = "button";
+    reload.id = "reload";
+    reload.className = "bf-btn bf-btn--primary";
+    reload.textContent = "Reload";
+    reload.addEventListener("click", function () { window.location.reload(); });
+    controls.appendChild(reload);
+  }
+
+  // The stream exists only to notice a closed tab. Once the review is settled it
+  // has no job, and leaving it open retries a dead port until the tab closes.
+  function settleStream() {
+    if (stream && stream.close) { stream.close(); stream = null; }
+  }
+`;