@diegopetrucci/pi-extensions 0.1.74 → 0.1.78

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

@@ -7,202 +7,202 @@ export type ReviewFileKind = "text" | "binary" | "image";
 export type ReviewCommitKind = "commit" | "working-tree";
 
 export interface ReviewFileComparison {
-	status: ChangeStatus;
-	oldPath: string | null;
-	newPath: string | null;
-	displayPath: string;
-	hasOriginal: boolean;
-	hasModified: boolean;
+  status: ChangeStatus;
+  oldPath: string | null;
+  newPath: string | null;
+  displayPath: string;
+  hasOriginal: boolean;
+  hasModified: boolean;
 }
 
 export interface ReviewFile {
-	id: string;
-	path: string;
-	worktreeStatus: ChangeStatus | null;
-	hasWorkingTreeFile: boolean;
-	/** True when the file is touched by the current branch diff (merge-base/base ref vs HEAD). */
-	inGitDiff: boolean;
-	/** Diff comparison to render for the file in the current scope.
-	 *  - branch scope: merge-base/base ref vs HEAD
-	 *  - commits scope: commit vs commit^ (populated when loaded via request-commit) */
-	gitDiff: ReviewFileComparison | null;
-	kind: ReviewFileKind;
-	mimeType: string | null;
+  id: string;
+  path: string;
+  worktreeStatus: ChangeStatus | null;
+  hasWorkingTreeFile: boolean;
+  /** True when the file is touched by the current branch diff (merge-base/base ref vs HEAD). */
+  inGitDiff: boolean;
+  /** Diff comparison to render for the file in the current scope.
+   *  - branch scope: merge-base/base ref vs HEAD
+   *  - commits scope: commit vs commit^ (populated when loaded via request-commit) */
+  gitDiff: ReviewFileComparison | null;
+  kind: ReviewFileKind;
+  mimeType: string | null;
 }
 
 export interface ReviewCommitInfo {
-	sha: string;
-	shortSha: string;
-	subject: string;
-	authorName: string;
-	authorDate: string;
-	kind: ReviewCommitKind;
+  sha: string;
+  shortSha: string;
+  subject: string;
+  authorName: string;
+  authorDate: string;
+  kind: ReviewCommitKind;
 }
 
 export interface ReviewFileContents {
-	originalContent: string;
-	modifiedContent: string;
-	kind: ReviewFileKind;
-	mimeType: string | null;
-	originalExists: boolean;
-	modifiedExists: boolean;
-	originalPreviewUrl: string | null;
-	modifiedPreviewUrl: string | null;
+  originalContent: string;
+  modifiedContent: string;
+  kind: ReviewFileKind;
+  mimeType: string | null;
+  originalExists: boolean;
+  modifiedExists: boolean;
+  originalPreviewUrl: string | null;
+  modifiedPreviewUrl: string | null;
 }
 
 export type CommentSide = "original" | "modified" | "file";
 
 export interface DiffReviewComment {
-	id: string;
-	fileId: string;
-	scope: ReviewScope;
-	/** Commit SHA when scope === "commits". */
-	commitSha?: string | null;
-	/** Short commit identifier to render in the prompt (e.g. first 7 chars of SHA). */
-	commitShort?: string | null;
-	commitKind?: ReviewCommitKind | null;
-	side: CommentSide;
-	startLine: number | null;
-	endLine: number | null;
-	body: string;
+  id: string;
+  fileId: string;
+  scope: ReviewScope;
+  /** Commit SHA when scope === "commits". */
+  commitSha?: string | null;
+  /** Short commit identifier to render in the prompt (e.g. first 7 chars of SHA). */
+  commitShort?: string | null;
+  commitKind?: ReviewCommitKind | null;
+  side: CommentSide;
+  startLine: number | null;
+  endLine: number | null;
+  body: string;
 }
 
 export interface ReviewSubmitPayload {
-	type: "submit";
-	overallComment: string;
-	comments: DiffReviewComment[];
-	/**
-	 * True means this is a close-recovery draft and must stay editor-only.
-	 * Only literal false permits an explicit submission to trigger an agent turn;
-	 * malformed or missing values are normalized to true by the host.
-	 */
-	draft: boolean;
+  type: "submit";
+  overallComment: string;
+  comments: DiffReviewComment[];
+  /**
+   * True means this is a close-recovery draft and must stay editor-only.
+   * Only literal false permits an explicit submission to trigger an agent turn;
+   * malformed or missing values are normalized to true by the host.
+   */
+  draft: boolean;
 }
 
 export interface ReviewCancelPayload {
-	type: "cancel";
+  type: "cancel";
 }
 
 export interface ReviewRequestFilePayload {
-	type: "request-file";
-	requestId: string;
-	fileId: string;
-	scope: ReviewScope;
-	commitSha?: string | null;
+  type: "request-file";
+  requestId: string;
+  fileId: string;
+  scope: ReviewScope;
+  commitSha?: string | null;
 }
 
 export interface ReviewRequestCommitPayload {
-	type: "request-commit";
-	requestId: string;
-	sha: string;
+  type: "request-commit";
+  requestId: string;
+  sha: string;
 }
 
 export interface ReviewRequestReviewDataPayload {
-	type: "request-review-data";
-	requestId: string;
+  type: "request-review-data";
+  requestId: string;
 }
 
 export interface ReviewClipboardReadPayload {
-	type: "clipboard-read";
-	requestId: string;
+  type: "clipboard-read";
+  requestId: string;
 }
 
 export interface ReviewClipboardWritePayload {
-	type: "clipboard-write";
-	text: string;
+  type: "clipboard-write";
+  text: string;
 }
 
 export type ReviewWindowMessage =
-	| ReviewSubmitPayload
-	| ReviewCancelPayload
-	| ReviewRequestFilePayload
-	| ReviewRequestCommitPayload
-	| ReviewRequestReviewDataPayload
-	| ReviewClipboardReadPayload
-	| ReviewClipboardWritePayload;
+  | ReviewSubmitPayload
+  | ReviewCancelPayload
+  | ReviewRequestFilePayload
+  | ReviewRequestCommitPayload
+  | ReviewRequestReviewDataPayload
+  | ReviewClipboardReadPayload
+  | ReviewClipboardWritePayload;
 
 export interface ReviewFileDataMessage {
-	type: "file-data";
-	requestId: string;
-	fileId: string;
-	scope: ReviewScope;
-	commitSha?: string | null;
-	originalContent: string;
-	modifiedContent: string;
-	kind: ReviewFileKind;
-	mimeType: string | null;
-	originalExists: boolean;
-	modifiedExists: boolean;
-	originalPreviewUrl: string | null;
-	modifiedPreviewUrl: string | null;
+  type: "file-data";
+  requestId: string;
+  fileId: string;
+  scope: ReviewScope;
+  commitSha?: string | null;
+  originalContent: string;
+  modifiedContent: string;
+  kind: ReviewFileKind;
+  mimeType: string | null;
+  originalExists: boolean;
+  modifiedExists: boolean;
+  originalPreviewUrl: string | null;
+  modifiedPreviewUrl: string | null;
 }
 
 export interface ReviewFileErrorMessage {
-	type: "file-error";
-	requestId: string;
-	fileId: string;
-	scope: ReviewScope;
-	commitSha?: string | null;
-	message: string;
+  type: "file-error";
+  requestId: string;
+  fileId: string;
+  scope: ReviewScope;
+  commitSha?: string | null;
+  message: string;
 }
 
 export interface ReviewCommitDataMessage {
-	type: "commit-data";
-	requestId: string;
-	sha: string;
-	files: ReviewFile[];
+  type: "commit-data";
+  requestId: string;
+  sha: string;
+  files: ReviewFile[];
 }
 
 export interface ReviewCommitErrorMessage {
-	type: "commit-error";
-	requestId: string;
-	sha: string;
-	message: string;
+  type: "commit-error";
+  requestId: string;
+  sha: string;
+  message: string;
 }
 
 export interface ReviewReviewDataMessage {
-	type: "review-data";
-	requestId: string;
-	files: ReviewFile[];
-	commits: ReviewCommitInfo[];
-	branchBaseRef: string | null;
-	branchMergeBaseSha: string | null;
-	repositoryHasHead: boolean;
+  type: "review-data";
+  requestId: string;
+  files: ReviewFile[];
+  commits: ReviewCommitInfo[];
+  branchBaseRef: string | null;
+  branchMergeBaseSha: string | null;
+  repositoryHasHead: boolean;
 }
 
 export interface ReviewReviewDataErrorMessage {
-	type: "review-data-error";
-	requestId: string;
-	message: string;
+  type: "review-data-error";
+  requestId: string;
+  message: string;
 }
 
 export interface ReviewClipboardDataMessage {
-	type: "clipboard-data";
-	requestId: string;
-	text: string;
-	message?: string;
+  type: "clipboard-data";
+  requestId: string;
+  text: string;
+  message?: string;
 }
 
 export interface ReviewWorkingTreeChangedMessage {
-	type: "working-tree-changed";
-	changedAt: number;
+  type: "working-tree-changed";
+  changedAt: number;
 }
 
 export type ReviewHostMessage =
-	| ReviewFileDataMessage
-	| ReviewFileErrorMessage
-	| ReviewCommitDataMessage
-	| ReviewCommitErrorMessage
-	| ReviewReviewDataMessage
-	| ReviewReviewDataErrorMessage
-	| ReviewClipboardDataMessage
-	| ReviewWorkingTreeChangedMessage;
+  | ReviewFileDataMessage
+  | ReviewFileErrorMessage
+  | ReviewCommitDataMessage
+  | ReviewCommitErrorMessage
+  | ReviewReviewDataMessage
+  | ReviewReviewDataErrorMessage
+  | ReviewClipboardDataMessage
+  | ReviewWorkingTreeChangedMessage;
 
 export interface ReviewWindowData {
-	repoRoot: string;
-	files: ReviewFile[];
-	commits: ReviewCommitInfo[];
-	branchBaseRef: string | null;
-	branchMergeBaseSha: string | null;
-	repositoryHasHead: boolean;
+  repoRoot: string;
+  files: ReviewFile[];
+  commits: ReviewCommitInfo[];
+  branchBaseRef: string | null;
+  branchMergeBaseSha: string | null;
+  repositoryHasHead: boolean;
 }

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

@@ -7,41 +7,41 @@ const __dirname = dirname(fileURLToPath(import.meta.url));
 const webDir = join(__dirname, "web");
 
 function escapeForInlineScript(value: string): string {
-	return value.replace(/</g, "\\u003c").replace(/>/g, "\\u003e").replace(/&/g, "\\u0026");
+  return value.replace(/</g, "\\u003c").replace(/>/g, "\\u003e").replace(/&/g, "\\u0026");
 }
 
 function escapeInlineScriptSource(value: string): string {
-	return value.replace(/<\/(script)/gi, "<\\/$1");
+  return value.replace(/<\/(script)/gi, "<\\/$1");
 }
 
 export interface ReviewUiAssets {
-	tailwindBrowserJs: string;
-	monacoLoaderJs: string;
-	monacoVsBaseUrl: string;
+  tailwindBrowserJs: string;
+  monacoLoaderJs: string;
+  monacoVsBaseUrl: string;
 }
 
 export function buildReviewHtml(data: ReviewWindowData, assets: ReviewUiAssets): string {
-	const templateHtml = readFileSync(join(webDir, "index.html"), "utf8");
-	const appJs = escapeInlineScriptSource(readFileSync(join(webDir, "app.js"), "utf8"));
-	const payload = escapeForInlineScript(JSON.stringify(data));
-	const assetConfig = escapeForInlineScript(
-		JSON.stringify({
-			monacoVsBaseUrl: assets.monacoVsBaseUrl,
-			bootstrapError: null,
-		}),
-	);
-	const replacements = new Map([
-		['"__INLINE_DATA__"', payload],
-		["__INLINE_ASSET_CONFIG__", assetConfig],
-		["__INLINE_TAILWIND_JS__", escapeInlineScriptSource(assets.tailwindBrowserJs)],
-		["__INLINE_MONACO_LOADER_JS__", escapeInlineScriptSource(assets.monacoLoaderJs)],
-		["__INLINE_JS__", appJs],
-	]);
+  const templateHtml = readFileSync(join(webDir, "index.html"), "utf8");
+  const appJs = escapeInlineScriptSource(readFileSync(join(webDir, "app.js"), "utf8"));
+  const payload = escapeForInlineScript(JSON.stringify(data));
+  const assetConfig = escapeForInlineScript(
+    JSON.stringify({
+      monacoVsBaseUrl: assets.monacoVsBaseUrl,
+      bootstrapError: null,
+    }),
+  );
+  const replacements = new Map([
+    ['"__INLINE_DATA__"', payload],
+    ["__INLINE_ASSET_CONFIG__", assetConfig],
+    ["__INLINE_TAILWIND_JS__", escapeInlineScriptSource(assets.tailwindBrowserJs)],
+    ["__INLINE_MONACO_LOADER_JS__", escapeInlineScriptSource(assets.monacoLoaderJs)],
+    ["__INLINE_JS__", appJs],
+  ]);
 
-	// Replace the original template in one pass so marker-like repository data
-	// or runtime source is never scanned as another template placeholder.
-	return templateHtml.replace(
-		/"__INLINE_DATA__"|__INLINE_ASSET_CONFIG__|__INLINE_TAILWIND_JS__|__INLINE_MONACO_LOADER_JS__|__INLINE_JS__/g,
-		(marker) => replacements.get(marker) ?? marker,
-	);
+  // Replace the original template in one pass so marker-like repository data
+  // or runtime source is never scanned as another template placeholder.
+  return templateHtml.replace(
+    /"__INLINE_DATA__"|__INLINE_ASSET_CONFIG__|__INLINE_TAILWIND_JS__|__INLINE_MONACO_LOADER_JS__|__INLINE_JS__/g,
+    (marker) => replacements.get(marker) ?? marker,
+  );
 }

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

@@ -1,104 +1,104 @@
 import { type FSWatcher, watch } from "node:fs";
 
 export interface RepoChangeWatcher {
-	dispose(): void;
+  dispose(): void;
 }
 
 interface RepoChangeWatcherOptions {
-	debounceMs?: number;
-	onError?: (error: Error) => void;
+  debounceMs?: number;
+  onError?: (error: Error) => void;
 }
 
 const DEFAULT_DEBOUNCE_MS = 2000;
 
 const ignoredPathSegments = new Set([
-	".cache",
-	".git",
-	".hg",
-	".next",
-	".nuxt",
-	".svn",
-	".turbo",
-	"build",
-	"coverage",
-	"dist",
-	"node_modules",
-	"out",
-	"target",
-	"tmp",
+  ".cache",
+  ".git",
+  ".hg",
+  ".next",
+  ".nuxt",
+  ".svn",
+  ".turbo",
+  "build",
+  "coverage",
+  "dist",
+  "node_modules",
+  "out",
+  "target",
+  "tmp",
 ]);
 
 const ignoredFileNames = new Set([".DS_Store"]);
 
 function normalizeWatchPath(path: string): string {
-	return path.replace(/\\/g, "/").replace(/^\/+/, "");
+  return path.replace(/\\/g, "/").replace(/^\/+/, "");
 }
 
 export function isIgnoredWatchPath(path: string | Buffer | null | undefined): boolean {
-	if (path == null) return false;
-	const normalized = normalizeWatchPath(path.toString());
-	if (normalized.length === 0) return false;
+  if (path == null) return false;
+  const normalized = normalizeWatchPath(path.toString());
+  if (normalized.length === 0) return false;
 
-	const segments = normalized.split("/").filter((segment) => segment.length > 0);
-	if (segments.some((segment) => ignoredPathSegments.has(segment))) return true;
+  const segments = normalized.split("/").filter((segment) => segment.length > 0);
+  if (segments.some((segment) => ignoredPathSegments.has(segment))) return true;
 
-	const fileName = segments.at(-1) ?? normalized;
-	if (ignoredFileNames.has(fileName)) return true;
-	if (fileName.endsWith("~") || fileName.endsWith(".swp") || fileName.endsWith(".tmp")) return true;
+  const fileName = segments.at(-1) ?? normalized;
+  if (ignoredFileNames.has(fileName)) return true;
+  if (fileName.endsWith("~") || fileName.endsWith(".swp") || fileName.endsWith(".tmp")) return true;
 
-	return false;
+  return false;
 }
 
 export function createRepoChangeWatcher(
-	repoRoot: string,
-	onChange: () => void,
-	options: RepoChangeWatcherOptions = {},
+  repoRoot: string,
+  onChange: () => void,
+  options: RepoChangeWatcherOptions = {},
 ): RepoChangeWatcher {
-	const debounceMs = options.debounceMs ?? DEFAULT_DEBOUNCE_MS;
-	let disposed = false;
-	let timer: ReturnType<typeof setTimeout> | null = null;
-	let watcher: FSWatcher | null = null;
-
-	const clearPending = (): void => {
-		if (timer == null) return;
-		clearTimeout(timer);
-		timer = null;
-	};
-
-	const scheduleChange = (path: string | Buffer | null): void => {
-		if (disposed || isIgnoredWatchPath(path)) return;
-		clearPending();
-		timer = setTimeout(() => {
-			timer = null;
-			if (!disposed) onChange();
-		}, debounceMs);
-	};
-
-	try {
-		watcher = watch(repoRoot, { recursive: true }, (_eventType, fileName) => {
-			scheduleChange(fileName);
-		});
-		watcher.on("error", (error) => {
-			if (!disposed) options.onError?.(error);
-		});
-	} catch (error) {
-		options.onError?.(error instanceof Error ? error : new Error(String(error)));
-	}
-
-	return {
-		dispose() {
-			disposed = true;
-			clearPending();
-			try {
-				watcher?.close();
-			} catch {
-				// Ignore watcher shutdown races.
-			}
-			watcher = null;
-		},
-	};
+  const debounceMs = options.debounceMs ?? DEFAULT_DEBOUNCE_MS;
+  let disposed = false;
+  let timer: ReturnType<typeof setTimeout> | null = null;
+  let watcher: FSWatcher | null = null;
+
+  const clearPending = (): void => {
+    if (timer == null) return;
+    clearTimeout(timer);
+    timer = null;
+  };
+
+  const scheduleChange = (path: string | Buffer | null): void => {
+    if (disposed || isIgnoredWatchPath(path)) return;
+    clearPending();
+    timer = setTimeout(() => {
+      timer = null;
+      if (!disposed) onChange();
+    }, debounceMs);
+  };
+
+  try {
+    watcher = watch(repoRoot, { recursive: true }, (_eventType, fileName) => {
+      scheduleChange(fileName);
+    });
+    watcher.on("error", (error) => {
+      if (!disposed) options.onError?.(error);
+    });
+  } catch (error) {
+    options.onError?.(error instanceof Error ? error : new Error(String(error)));
+  }
+
+  return {
+    dispose() {
+      disposed = true;
+      clearPending();
+      try {
+        watcher?.close();
+      } catch {
+        // Ignore watcher shutdown races.
+      }
+      watcher = null;
+    },
+  };
 }
 
 export const __testing = {
-	isIgnoredWatchPath,
+  isIgnoredWatchPath,
 };