@diegopetrucci/pi-extensions 0.1.74 → 0.1.76

package/extensions/annotate-git-diff/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-annotate-git-diff",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "description": "A standalone pi extension that adds /annotate-git-diff, a native Glimpse UI for annotating git diffs and sending explicit feedback directly to the agent; close recovery keeps unsent drafts in the editor.",
   "keywords": [
     "pi-package",

package/extensions/annotate-git-diff/prompt.ts

@@ -1,66 +1,66 @@
 import type { DiffReviewComment, ReviewFile, ReviewSubmitPayload } from "./types.js";
 
 function formatScopeLabel(comment: DiffReviewComment): string {
-	switch (comment.scope) {
-		case "branch":
-			return "branch diff";
-		case "commits":
-			return comment.commitKind === "working-tree"
-				? "working tree changes"
-				: comment.commitShort
-					? `commit ${comment.commitShort}`
-					: "commit";
-		default:
-			return "all files";
-	}
+  switch (comment.scope) {
+    case "branch":
+      return "branch diff";
+    case "commits":
+      return comment.commitKind === "working-tree"
+        ? "working tree changes"
+        : comment.commitShort
+          ? `commit ${comment.commitShort}`
+          : "commit";
+    default:
+      return "all files";
+  }
 }
 
 function getCommentFilePath(file: ReviewFile | undefined): string {
-	if (file == null) return "(unknown file)";
-	return file.gitDiff?.displayPath ?? file.path;
+  if (file == null) return "(unknown file)";
+  return file.gitDiff?.displayPath ?? file.path;
 }
 
 function formatLocation(comment: DiffReviewComment, file: ReviewFile | undefined): string {
-	const filePath = getCommentFilePath(file);
-	const scopePrefix = `[${formatScopeLabel(comment)}] `;
+  const filePath = getCommentFilePath(file);
+  const scopePrefix = `[${formatScopeLabel(comment)}] `;
 
-	if (comment.side === "file" || comment.startLine == null) {
-		return `${scopePrefix}${filePath}`;
-	}
+  if (comment.side === "file" || comment.startLine == null) {
+    return `${scopePrefix}${filePath}`;
+  }
 
-	const range =
-		comment.endLine != null && comment.endLine !== comment.startLine
-			? `${comment.startLine}-${comment.endLine}`
-			: `${comment.startLine}`;
+  const range =
+    comment.endLine != null && comment.endLine !== comment.startLine
+      ? `${comment.startLine}-${comment.endLine}`
+      : `${comment.startLine}`;
 
-	if (comment.scope === "all") {
-		return `${scopePrefix}${filePath}:${range}`;
-	}
+  if (comment.scope === "all") {
+    return `${scopePrefix}${filePath}:${range}`;
+  }
 
-	const suffix = comment.side === "original" ? " (old)" : " (new)";
-	return `${scopePrefix}${filePath}:${range}${suffix}`;
+  const suffix = comment.side === "original" ? " (old)" : " (new)";
+  return `${scopePrefix}${filePath}:${range}${suffix}`;
 }
 
 export function composeReviewPrompt(files: ReviewFile[], payload: ReviewSubmitPayload): string {
-	const fileMap = new Map(files.map((file) => [file.id, file]));
-	const comments = payload.comments.filter((comment) => comment.body.trim().length > 0);
-	const lines: string[] = [];
+  const fileMap = new Map(files.map((file) => [file.id, file]));
+  const comments = payload.comments.filter((comment) => comment.body.trim().length > 0);
+  const lines: string[] = [];
 
-	lines.push("Please address the following feedback");
-	lines.push("");
+  lines.push("Please address the following feedback");
+  lines.push("");
 
-	const overallComment = payload.overallComment.trim();
-	if (overallComment.length > 0) {
-		lines.push(overallComment);
-		lines.push("");
-	}
+  const overallComment = payload.overallComment.trim();
+  if (overallComment.length > 0) {
+    lines.push(overallComment);
+    lines.push("");
+  }
 
-	comments.forEach((comment, index) => {
-		const file = fileMap.get(comment.fileId);
-		lines.push(`${index + 1}. ${formatLocation(comment, file)}`);
-		lines.push(`   ${comment.body.trim()}`);
-		lines.push("");
-	});
+  comments.forEach((comment, index) => {
+    const file = fileMap.get(comment.fileId);
+    lines.push(`${index + 1}. ${formatLocation(comment, file)}`);
+    lines.push(`   ${comment.body.trim()}`);
+    lines.push("");
+  });
 
-	return lines.join("\n").trim();
+  return lines.join("\n").trim();
 }

package/extensions/annotate-git-diff/quiet-glimpse.ts

@@ -7,173 +7,179 @@ import { createInterface } from "node:readline";
 import type { GlimpseOpenOptions } from "glimpseui";
 
 interface NativeHostInfo {
-	path: string;
-	extraArgs?: string[];
-	buildHint?: string;
+  path: string;
+  extraArgs?: string[];
+  buildHint?: string;
 }
 
 interface GlimpseProtocolMessage {
-	type?: string;
-	data?: unknown;
-	screen?: unknown;
-	screens?: unknown;
-	appearance?: unknown;
-	cursor?: unknown;
-	cursorTip?: unknown;
+  type?: string;
+  data?: unknown;
+  screen?: unknown;
+  screens?: unknown;
+  appearance?: unknown;
+  cursor?: unknown;
+  cursorTip?: unknown;
 }
 
 export interface QuietGlimpseWindow {
-	readonly closed: boolean;
-	readonly failure: Error | null;
-	on(event: "message", listener: (data: unknown) => void): this;
-	on(event: "closed", listener: () => void): this;
-	on(event: "error", listener: (error: Error) => void): this;
-	removeListener(event: "message", listener: (data: unknown) => void): this;
-	removeListener(event: "closed", listener: () => void): this;
-	removeListener(event: "error", listener: (error: Error) => void): this;
-	send(js: string): void;
-	close(): void;
+  readonly closed: boolean;
+  readonly failure: Error | null;
+  on(event: "message", listener: (data: unknown) => void): this;
+  on(event: "closed", listener: () => void): this;
+  on(event: "error", listener: (error: Error) => void): this;
+  removeListener(event: "message", listener: (data: unknown) => void): this;
+  removeListener(event: "closed", listener: () => void): this;
+  removeListener(event: "error", listener: (error: Error) => void): this;
+  send(js: string): void;
+  close(): void;
 }
 
 class QuietGlimpseWindowImpl extends EventEmitter implements QuietGlimpseWindow {
-	#proc: ChildProcessWithoutNullStreams;
-	#closed = false;
-	#pendingHTML: string | null;
-	#stderr = "";
-	#failure: Error | null = null;
-
-	constructor(proc: ChildProcessWithoutNullStreams, initialHTML: string) {
-		super();
-		this.#proc = proc;
-		this.#pendingHTML = initialHTML;
-		// EventEmitter treats an unobserved `error` as fatal. Keep a permanent
-		// fallback while callers attach and detach their lifecycle listeners.
-		this.on("error", () => {});
-
-		proc.stdin.on("error", () => {});
-		proc.stderr.on("data", (chunk) => {
-			this.#stderr += chunk.toString();
-		});
-
-		const rl = createInterface({ input: proc.stdout, crlfDelay: Infinity });
-		rl.on("line", (line) => {
-			let parsed: unknown;
-			try {
-				parsed = JSON.parse(line);
-			} catch {
-				this.#reportError(new Error(`Malformed glimpse protocol line: ${line}`));
-				return;
-			}
-			if (parsed === null || typeof parsed !== "object" || Array.isArray(parsed)) return;
-			const message = parsed as GlimpseProtocolMessage;
-
-			switch (message.type) {
-				case "ready":
-					if (this.#pendingHTML != null) {
-						this.setHTML(this.#pendingHTML);
-						this.#pendingHTML = null;
-					}
-					break;
-				case "message":
-					this.emit("message", message.data);
-					break;
-				case "closed":
-					this.#markClosed();
-					break;
-				default:
-					break;
-			}
-		});
-
-		proc.on("error", (error) => this.#reportError(error));
-		proc.on("exit", (code, signal) => {
-			const stderr = this.#stderr.trim();
-			const failed = signal != null || (code != null && code !== 0);
-			if (!this.#closed && failed) {
-				const message = stderr || `Glimpse process exited abnormally (code: ${code}, signal: ${signal}).`;
-				this.#reportError(new Error(message));
-			}
-			this.#markClosed();
-		});
-		// Spawn failures can emit `error` and then `close` without `exit`.
-		proc.on("close", () => this.#markClosed());
-	}
-
-	send(js: string): void {
-		this.#write({ type: "eval", js });
-	}
-
-	close(): void {
-		this.#write({ type: "close" });
-	}
-
-	get closed(): boolean {
-		return this.#closed;
-	}
-
-	get failure(): Error | null {
-		return this.#failure;
-	}
-
-	#reportError(error: Error): void {
-		this.#failure ??= error;
-		this.emit("error", error);
-	}
-
-	#markClosed(): void {
-		if (this.#closed) return;
-		this.#closed = true;
-		this.emit("closed");
-	}
-
-	#write(obj: Record<string, unknown>): void {
-		if (this.#closed) return;
-		this.#proc.stdin.write(`${JSON.stringify(obj)}\n`);
-	}
-
-	setHTML(html: string): void {
-		this.#write({ type: "html", html: Buffer.from(html).toString("base64") });
-	}
+  #proc: ChildProcessWithoutNullStreams;
+  #closed = false;
+  #pendingHTML: string | null;
+  #stderr = "";
+  #failure: Error | null = null;
+
+  constructor(proc: ChildProcessWithoutNullStreams, initialHTML: string) {
+    super();
+    this.#proc = proc;
+    this.#pendingHTML = initialHTML;
+    // EventEmitter treats an unobserved `error` as fatal. Keep a permanent
+    // fallback while callers attach and detach their lifecycle listeners.
+    this.on("error", () => {});
+
+    proc.stdin.on("error", () => {});
+    proc.stderr.on("data", (chunk) => {
+      this.#stderr += chunk.toString();
+    });
+
+    const rl = createInterface({ input: proc.stdout, crlfDelay: Infinity });
+    rl.on("line", (line) => {
+      let parsed: unknown;
+      try {
+        parsed = JSON.parse(line);
+      } catch {
+        this.#reportError(new Error(`Malformed glimpse protocol line: ${line}`));
+        return;
+      }
+      if (parsed === null || typeof parsed !== "object" || Array.isArray(parsed)) return;
+      const message = parsed as GlimpseProtocolMessage;
+
+      switch (message.type) {
+        case "ready":
+          if (this.#pendingHTML != null) {
+            this.setHTML(this.#pendingHTML);
+            this.#pendingHTML = null;
+          }
+          break;
+        case "message":
+          this.emit("message", message.data);
+          break;
+        case "closed":
+          this.#markClosed();
+          break;
+        default:
+          break;
+      }
+    });
+
+    proc.on("error", (error) => this.#reportError(error));
+    proc.on("exit", (code, signal) => {
+      const stderr = this.#stderr.trim();
+      const failed = signal != null || (code != null && code !== 0);
+      if (!this.#closed && failed) {
+        const message =
+          stderr || `Glimpse process exited abnormally (code: ${code}, signal: ${signal}).`;
+        this.#reportError(new Error(message));
+      }
+      this.#markClosed();
+    });
+    // Spawn failures can emit `error` and then `close` without `exit`.
+    proc.on("close", () => this.#markClosed());
+  }
+
+  send(js: string): void {
+    this.#write({ type: "eval", js });
+  }
+
+  close(): void {
+    this.#write({ type: "close" });
+  }
+
+  get closed(): boolean {
+    return this.#closed;
+  }
+
+  get failure(): Error | null {
+    return this.#failure;
+  }
+
+  #reportError(error: Error): void {
+    this.#failure ??= error;
+    this.emit("error", error);
+  }
+
+  #markClosed(): void {
+    if (this.#closed) return;
+    this.#closed = true;
+    this.emit("closed");
+  }
+
+  #write(obj: Record<string, unknown>): void {
+    if (this.#closed) return;
+    this.#proc.stdin.write(`${JSON.stringify(obj)}\n`);
+  }
+
+  setHTML(html: string): void {
+    this.#write({ type: "html", html: Buffer.from(html).toString("base64") });
+  }
 }
 
 async function getNativeHostInfo(): Promise<NativeHostInfo> {
-	const glimpseModule = (await import("glimpseui")) as unknown as { getNativeHostInfo: () => NativeHostInfo };
-	return glimpseModule.getNativeHostInfo();
+  const glimpseModule = (await import("glimpseui")) as unknown as {
+    getNativeHostInfo: () => NativeHostInfo;
+  };
+  return glimpseModule.getNativeHostInfo();
 }
 
-export async function openQuietGlimpse(html: string, options: GlimpseOpenOptions = {}): Promise<QuietGlimpseWindow> {
-	const host = await getNativeHostInfo();
-	if (!existsSync(host.path)) {
-		const hint = host.buildHint ? ` ${host.buildHint}` : "";
-		throw new Error(`Glimpse host not found at '${host.path}'.${hint}`);
-	}
-	const args: string[] = [];
-
-	if (options.width != null) args.push("--width", String(options.width));
-	if (options.height != null) args.push("--height", String(options.height));
-	if (options.title != null) args.push("--title", options.title);
-	if (options.frameless) args.push("--frameless");
-	if (options.floating) args.push("--floating");
-	if (options.transparent) args.push("--transparent");
-	if (options.clickThrough) args.push("--click-through");
-	if (options.hidden) args.push("--hidden");
-	if (options.autoClose) args.push("--auto-close");
-	if (options.x != null) args.push(`--x=${options.x}`);
-	if (options.y != null) args.push(`--y=${options.y}`);
-	if (options.cursorOffset?.x != null) args.push(`--cursor-offset-x=${options.cursorOffset.x}`);
-	if (options.cursorOffset?.y != null) args.push(`--cursor-offset-y=${options.cursorOffset.y}`);
-	if (options.cursorAnchor != null) args.push("--cursor-anchor", options.cursorAnchor);
-	if (options.followMode != null) args.push("--follow-mode", options.followMode);
-	if (options.followCursor) args.push("--follow-cursor");
-
-	const proc = spawn(host.path, [...(host.extraArgs ?? []), ...args], {
-		stdio: ["pipe", "pipe", "pipe"],
-		windowsHide: process.platform === "win32",
-		env: {
-			...process.env,
-			OS_ACTIVITY_MODE: process.env.OS_ACTIVITY_MODE ?? "disable",
-		},
-	});
-
-	return new QuietGlimpseWindowImpl(proc, html);
+export async function openQuietGlimpse(
+  html: string,
+  options: GlimpseOpenOptions = {},
+): Promise<QuietGlimpseWindow> {
+  const host = await getNativeHostInfo();
+  if (!existsSync(host.path)) {
+    const hint = host.buildHint ? ` ${host.buildHint}` : "";
+    throw new Error(`Glimpse host not found at '${host.path}'.${hint}`);
+  }
+  const args: string[] = [];
+
+  if (options.width != null) args.push("--width", String(options.width));
+  if (options.height != null) args.push("--height", String(options.height));
+  if (options.title != null) args.push("--title", options.title);
+  if (options.frameless) args.push("--frameless");
+  if (options.floating) args.push("--floating");
+  if (options.transparent) args.push("--transparent");
+  if (options.clickThrough) args.push("--click-through");
+  if (options.hidden) args.push("--hidden");
+  if (options.autoClose) args.push("--auto-close");
+  if (options.x != null) args.push(`--x=${options.x}`);
+  if (options.y != null) args.push(`--y=${options.y}`);
+  if (options.cursorOffset?.x != null) args.push(`--cursor-offset-x=${options.cursorOffset.x}`);
+  if (options.cursorOffset?.y != null) args.push(`--cursor-offset-y=${options.cursorOffset.y}`);
+  if (options.cursorAnchor != null) args.push("--cursor-anchor", options.cursorAnchor);
+  if (options.followMode != null) args.push("--follow-mode", options.followMode);
+  if (options.followCursor) args.push("--follow-cursor");
+
+  const proc = spawn(host.path, [...(host.extraArgs ?? []), ...args], {
+    stdio: ["pipe", "pipe", "pipe"],
+    windowsHide: process.platform === "win32",
+    env: {
+      ...process.env,
+      OS_ACTIVITY_MODE: process.env.OS_ACTIVITY_MODE ?? "disable",
+    },
+  });
+
+  return new QuietGlimpseWindowImpl(proc, html);
 }