@agimon-ai/po-git 0.0.0-alpha.34 → 0.0.0-alpha.35

package/dist/extensions/native.mjs

@@ -1,12 +1,399 @@
 import { createApiClient, defineActivityGroup, defineChannelFile, defineFill, defineGlobalStore, defineNativePlugin, defineSessionStore, defineWebLifecycle, nativeApiTransport, sessionApiAddress } from "@agimon-ai/po-core/native";
-import { AlertIcon, Badge, BranchIcon, Button, ChevronRightIcon, Dot, FileIcon, Input, MessageIcon, Sheet, SheetBody, SheetContent, SheetHeader, SheetTitle, Spinner, SyntaxLine, Textarea, fontOf } from "@agimon-ai/po-native-components";
-import { createElement, useCallback, useEffect, useRef, useState, useSyncExternalStore } from "react";
 import { View } from "react-native-unistyles/components/native/View";
 import { Text } from "react-native-unistyles/components/native/Text";
-import { ScrollView } from "react-native-unistyles/components/native/ScrollView";
-import { Pressable } from "react-native-unistyles/components/native/Pressable";
+import { AlertIcon, Badge, BranchIcon, Button, ChevronRightIcon, Dot, FileIcon, Input, Label, ListRow, MessageIcon, SegmentedControl, Sheet, SheetBody, SheetContent, SheetHeader, SheetTitle, Spinner, SyntaxLine, Textarea, fontOf } from "@agimon-ai/po-native-components";
+import { createElement, memo, useCallback, useEffect, useMemo, useRef, useState, useSyncExternalStore } from "react";
 import { StyleSheet, useUnistyles } from "react-native-unistyles";
 import { Fragment, jsx, jsxs } from "react/jsx-runtime";
+import { ScrollView } from "react-native-unistyles/components/native/ScrollView";
+import { Pressable } from "react-native-unistyles/components/native/Pressable";
+const api = createApiClient({
+	/** Workspace: the new-session dialog's branch list. */
+	branches: {
+		method: "GET",
+		path: "/branches",
+		response: void 0
+	},
+	/** Workspace: a top-level worktree session on an existing or new branch. */
+	createSession: {
+		method: "POST",
+		path: "/sessions",
+		response: void 0
+	},
+	/** Workspace: that workspace's saved remote auth, never the token itself. */
+	auth: {
+		method: "GET",
+		path: "/auth",
+		query: ["repositoryId"],
+		response: void 0
+	},
+	/** Workspace: replaces that workspace's remote auth. Passkey step-up gated for remote devices. */
+	saveAuth: {
+		method: "PUT",
+		path: "/auth",
+		query: ["repositoryId"],
+		response: void 0
+	},
+	/** Session: the files this session's checkout changed against its base. */
+	review: {
+		method: "GET",
+		path: "/review",
+		response: void 0
+	},
+	/** Session: one changed file's hunks. */
+	reviewFile: {
+		method: "GET",
+		path: "/review/file",
+		query: ["path"],
+		response: void 0
+	}
+}, {
+	scopes: [
+		"global",
+		"workspace",
+		"session"
+	],
+	basePath: "git",
+	transport: nativeApiTransport,
+	sessionAddress: sessionApiAddress
+});
+//#endregion
+//#region src/extensions/workspaces/(frontend)/_lib/newSessionForm.ts
+const MAX_LISTED = 200;
+/** Local branches first, then remote-only ones, filtered by a case-insensitive substring. */
+function branchOptions(branches, query) {
+	const needle = query.trim().toLowerCase();
+	const matches = (text) => needle === "" || text.toLowerCase().includes(needle);
+	const local = branches.local.filter((branch) => matches(branch.name)).map((branch) => ({
+		key: `local:${branch.name}`,
+		name: branch.name,
+		current: branch.name === branches.current,
+		...branch.checkedOutAt === void 0 ? {} : { busyAt: branch.checkedOutAt }
+	}));
+	const remote = branches.remote.filter((branch) => matches(`${branch.remote}/${branch.name}`)).map((branch) => ({
+		key: `remote:${branch.remote}/${branch.name}`,
+		name: branch.name,
+		remote: branch.remote,
+		current: false
+	}));
+	return [...local, ...remote].slice(0, MAX_LISTED);
+}
+/** Why the form cannot be submitted yet, or undefined when it can. */
+function formProblem(state) {
+	if (state.mode === "existing") {
+		if (state.selected === void 0) return "Pick a branch.";
+		if (state.selected.busyAt !== void 0) return `That branch is checked out at ${state.selected.busyAt}.`;
+	}
+	if (state.mode === "new" && state.newBranch.trim() === "") return "Name the new branch.";
+}
+/** The worktree create request for the form, or undefined in plain mode. */
+function createRequest(state) {
+	const name = state.name.trim();
+	const named = name === "" ? {} : { name };
+	if (state.mode === "existing" && state.selected !== void 0) return {
+		mode: "existing-branch",
+		branch: state.selected.name,
+		...state.selected.remote === void 0 ? {} : { remote: state.selected.remote },
+		...named
+	};
+	if (state.mode === "new") {
+		const baseRef = state.baseRef.trim();
+		return {
+			mode: "new-branch",
+			branch: state.newBranch.trim(),
+			...baseRef === "" ? {} : { baseRef },
+			...named
+		};
+	}
+}
+//#endregion
+//#region src/extensions/workspaces/(frontend)/fill/_components/GitNewSessionSheet.native.tsx
+const MODES = [
+	{
+		value: "existing",
+		label: "Existing"
+	},
+	{
+		value: "new",
+		label: "New branch"
+	},
+	{
+		value: "plain",
+		label: "No branch"
+	}
+];
+/** The native twin of GitNewSessionDialogView: the same form state and checks, in sheet parts. */
+function GitNewSessionSheetView({ workspaceRoot, branches, loadError = "", initialForm, busy = false, error = "", onCreate }) {
+	const [form, setForm] = useState({
+		mode: "existing",
+		newBranch: "",
+		baseRef: "",
+		name: "",
+		...initialForm
+	});
+	const [query, setQuery] = useState("");
+	const ready = branches?.repository === true && loadError === "";
+	const loading = branches === null && loadError === "";
+	const mode = ready || loading ? form.mode : "plain";
+	const state = {
+		...form,
+		mode,
+		baseRef: form.baseRef || branches?.defaultBase || ""
+	};
+	const problem = formProblem(state);
+	const update = (patch) => setForm((current) => ({
+		...current,
+		...patch
+	}));
+	const options = branches === null ? [] : branchOptions(branches, query);
+	return /* @__PURE__ */ jsxs(Fragment, { children: [/* @__PURE__ */ jsx(SheetHeader, { children: /* @__PURE__ */ jsx(SheetTitle, { children: "New session" }) }), /* @__PURE__ */ jsxs(SheetBody, {
+		style: styles$3.body,
+		children: [
+			/* @__PURE__ */ jsx(Text, {
+				style: styles$3.root,
+				numberOfLines: 1,
+				ellipsizeMode: "middle",
+				children: workspaceRoot
+			}),
+			!ready ? /* @__PURE__ */ jsx(Text, {
+				testID: "git-new-session-plain-note",
+				style: styles$3.dim,
+				children: loading ? "Reading branches..." : loadError || "This workspace is not a git checkout. Start a plain session in this folder."
+			}) : /* @__PURE__ */ jsxs(Fragment, { children: [/* @__PURE__ */ jsx(SegmentedControl, {
+				accessibilityLabel: "Session kind",
+				options: MODES,
+				value: mode,
+				disabled: busy,
+				onValueChange: (value) => update({ mode: value })
+			}), mode === "existing" ? /* @__PURE__ */ jsxs(View, {
+				style: styles$3.field,
+				children: [
+					/* @__PURE__ */ jsx(Label, { children: "Branch" }),
+					/* @__PURE__ */ jsx(Input, {
+						testID: "git-new-session-branch-search",
+						accessibilityLabel: "Find a branch",
+						placeholder: "Find a branch",
+						autoCapitalize: "none",
+						autoCorrect: false,
+						value: query,
+						onChangeText: setQuery,
+						disabled: busy
+					}),
+					/* @__PURE__ */ jsxs(View, {
+						style: styles$3.list,
+						accessibilityLabel: "Branches",
+						children: [options.length === 0 ? /* @__PURE__ */ jsx(Text, {
+							style: [styles$3.dim, styles$3.pad],
+							children: "No matching branches"
+						}) : null, options.map((branch, index) => /* @__PURE__ */ jsxs(View, { children: [branch.remote !== void 0 && (index === 0 || options[index - 1]?.remote === void 0) ? /* @__PURE__ */ jsx(Text, {
+							style: [
+								styles$3.dim,
+								styles$3.pad,
+								styles$3.divider
+							],
+							children: "Remote branches"
+						}) : null, /* @__PURE__ */ jsx(ListRow, {
+							title: /* @__PURE__ */ jsx(Text, {
+								style: styles$3.mono,
+								children: branch.remote === void 0 ? branch.name : `${branch.remote}/${branch.name}`
+							}),
+							description: branch.busyAt !== void 0 ? `Checked out in ${branch.busyAt}` : branch.current ? "Current" : void 0,
+							selected: form.selected?.key === branch.key,
+							disabled: branch.busyAt !== void 0 || busy,
+							disclosure: false,
+							onPress: () => update({ selected: branch })
+						})] }, branch.key))]
+					})
+				]
+			}) : mode === "new" ? /* @__PURE__ */ jsxs(Fragment, { children: [/* @__PURE__ */ jsxs(View, {
+				style: styles$3.field,
+				children: [/* @__PURE__ */ jsx(Label, { children: "Branch name" }), /* @__PURE__ */ jsx(Input, {
+					testID: "git-new-session-new-branch",
+					accessibilityLabel: "Branch name",
+					placeholder: "feature/my-change",
+					autoCapitalize: "none",
+					autoCorrect: false,
+					style: styles$3.mono,
+					value: form.newBranch,
+					onChangeText: (newBranch) => update({ newBranch }),
+					disabled: busy
+				})]
+			}), /* @__PURE__ */ jsxs(View, {
+				style: styles$3.field,
+				children: [/* @__PURE__ */ jsx(Label, { children: "From" }), /* @__PURE__ */ jsx(Input, {
+					testID: "git-new-session-base",
+					accessibilityLabel: "From",
+					placeholder: branches?.defaultBase ?? "Current branch",
+					autoCapitalize: "none",
+					autoCorrect: false,
+					style: styles$3.mono,
+					value: form.baseRef,
+					onChangeText: (baseRef) => update({ baseRef }),
+					disabled: busy
+				})]
+			})] }) : /* @__PURE__ */ jsx(Text, {
+				style: styles$3.dim,
+				children: "Start in this workspace folder, without creating a worktree."
+			})] }),
+			/* @__PURE__ */ jsxs(View, {
+				style: styles$3.field,
+				children: [/* @__PURE__ */ jsx(Label, { children: "Name" }), /* @__PURE__ */ jsx(Input, {
+					testID: "new-session-name",
+					accessibilityLabel: "Session name",
+					placeholder: mode === "plain" ? "Optional" : "Optional, defaults to the branch",
+					value: form.name,
+					onChangeText: (name) => update({ name }),
+					disabled: busy
+				})]
+			}),
+			mode !== "plain" ? /* @__PURE__ */ jsx(Text, {
+				style: styles$3.dim,
+				children: "Runs in its own worktree, so this checkout stays as it is."
+			}) : null,
+			error ? /* @__PURE__ */ jsx(Text, {
+				testID: "new-session-error",
+				style: styles$3.error,
+				accessibilityRole: "alert",
+				children: error
+			}) : null,
+			/* @__PURE__ */ jsx(Button, {
+				testID: "new-session-create",
+				variant: "primary",
+				size: "lg",
+				loading: busy,
+				loadingLabel: mode === "plain" ? "Creating session" : "Creating worktree",
+				disabled: loading || problem !== void 0,
+				accessibilityHint: problem,
+				onPress: () => onCreate(state),
+				children: "Create session"
+			})
+		]
+	})] });
+}
+function errorOf(body, status) {
+	return typeof body === "object" && body !== null && typeof body.error === "string" ? body.error : `The cockpit answered ${String(status)}.`;
+}
+/** The native new-session sheet when git is installed; mirrors GitNewSessionDialog's calls. */
+function GitNewSessionSheet({ newSession }) {
+	const [branches, setBranches] = useState(null);
+	const [loadError, setLoadError] = useState("");
+	const [error, setError] = useState("");
+	const [busy, setBusy] = useState(false);
+	const workspaceId = newSession?.workspaceId;
+	const request = newSession?.requestWithStepUp;
+	useEffect(() => {
+		if (workspaceId === void 0 || request === void 0) return;
+		let stale = false;
+		request(api.workspace(workspaceId).branches.url()).then(async (response) => {
+			const body = await response.json().catch(() => void 0);
+			if (stale) return;
+			if (!response.ok) setLoadError(errorOf(body, response.status));
+			else setBranches(body);
+		}).catch(() => {
+			if (!stale) setLoadError("The branches could not be loaded.");
+		});
+		return () => {
+			stale = true;
+		};
+	}, [workspaceId, request]);
+	if (newSession === void 0) return null;
+	const submit = async (state) => {
+		if (busy || formProblem(state) !== void 0) return;
+		setBusy(true);
+		setError("");
+		const body = createRequest(state);
+		let failure;
+		try {
+			if (body === void 0) failure = await newSession.createPlain(state.name);
+			else {
+				const response = await newSession.requestWithStepUp(api.workspace(newSession.workspaceId).createSession.url(), {
+					method: "POST",
+					headers: { "Content-Type": "application/json" },
+					body: JSON.stringify(body)
+				});
+				const answer = await response.json().catch(() => void 0);
+				failure = response.ok ? await newSession.openCreated(answer.sessionId) : errorOf(answer, response.status);
+			}
+		} catch {
+			failure = "The cockpit is unreachable.";
+		}
+		if (failure === void 0) return;
+		setError(failure);
+		setBusy(false);
+	};
+	return /* @__PURE__ */ jsx(GitNewSessionSheetView, {
+		workspaceRoot: newSession.workspaceRoot,
+		branches,
+		loadError,
+		busy,
+		error,
+		onCreate: (state) => void submit(state)
+	});
+}
+const styles$3 = StyleSheet.create((theme) => {
+	const sans = fontOf(theme.fontFamily.sans);
+	const mono = fontOf(theme.fontFamily.mono);
+	return {
+		body: {
+			gap: theme.spacing(4),
+			uni__dependencies: [0]
+		},
+		root: {
+			fontFamily: mono,
+			fontSize: theme.fontSize.sm,
+			lineHeight: theme.lineHeight.sm,
+			color: theme.colors.dim,
+			textAlign: "center",
+			uni__dependencies: [0]
+		},
+		field: {
+			gap: theme.spacing(1.5),
+			uni__dependencies: [0]
+		},
+		dim: {
+			fontFamily: sans,
+			fontSize: theme.fontSize.sm,
+			lineHeight: theme.lineHeightRelaxed.sm,
+			color: theme.colors.dim,
+			uni__dependencies: [0]
+		},
+		error: {
+			fontFamily: sans,
+			fontSize: theme.fontSize.sm,
+			lineHeight: theme.lineHeightRelaxed.sm,
+			color: theme.colors.red,
+			uni__dependencies: [0]
+		},
+		mono: {
+			fontFamily: mono,
+			fontSize: theme.fontSize.base,
+			lineHeight: theme.lineHeight.base,
+			color: theme.colors.hi,
+			uni__dependencies: [0]
+		},
+		list: {
+			borderRadius: theme.borderRadius.lg,
+			borderWidth: 1,
+			borderColor: theme.colors["border-soft"],
+			overflow: "hidden",
+			uni__dependencies: [0]
+		},
+		pad: {
+			paddingHorizontal: theme.spacing(4),
+			paddingVertical: theme.spacing(2),
+			uni__dependencies: [0]
+		},
+		divider: {
+			borderTopWidth: StyleSheet.hairlineWidth,
+			borderTopColor: theme.colors["border-soft"],
+			uni__dependencies: [0]
+		}
+	};
+});
+//#endregion
+//#region src/extensions/workspaces/(frontend)/fill/GitNewSession.new-session.native.tsx
+var GitNewSession_new_session_native_default = defineFill({ component: GitNewSessionSheet });
+//#endregion
 //#region src/types/webWorktrees.ts
 /**
 * The cockpit's view of this session's worktrees.
@@ -858,56 +1245,6 @@ var git_changes_native_default = defineChannelFile(gitChangesChannel);
 //#endregion
 //#region src/extensions/workspaces/sessions/(frontend)/channel/git-worktrees.native.ts
 var git_worktrees_native_default = defineChannelFile(worktreesChannel);
-const api = createApiClient({
-	/** Workspace: the new-session dialog's branch list. */
-	branches: {
-		method: "GET",
-		path: "/branches",
-		response: void 0
-	},
-	/** Workspace: a top-level worktree session on an existing or new branch. */
-	createSession: {
-		method: "POST",
-		path: "/sessions",
-		response: void 0
-	},
-	/** Workspace: that workspace's saved remote auth, never the token itself. */
-	auth: {
-		method: "GET",
-		path: "/auth",
-		query: ["repositoryId"],
-		response: void 0
-	},
-	/** Workspace: replaces that workspace's remote auth. Passkey step-up gated for remote devices. */
-	saveAuth: {
-		method: "PUT",
-		path: "/auth",
-		query: ["repositoryId"],
-		response: void 0
-	},
-	/** Session: the files this session's checkout changed against its base. */
-	review: {
-		method: "GET",
-		path: "/review",
-		response: void 0
-	},
-	/** Session: one changed file's hunks. */
-	reviewFile: {
-		method: "GET",
-		path: "/review/file",
-		query: ["path"],
-		response: void 0
-	}
-}, {
-	scopes: [
-		"global",
-		"workspace",
-		"session"
-	],
-	basePath: "git",
-	transport: nativeApiTransport,
-	sessionAddress: sessionApiAddress
-});
 //#endregion
 //#region src/extensions/workspaces/sessions/(frontend)/_lib/gitReviewApi.native.ts
 /**
@@ -1318,10 +1655,21 @@ function FileHeader({ entry, index, total, onPrevious, onNext, onChanges }) {
 		]
 	});
 }
-function DiffRows({ rows, comments = [], selection, onLine, showInlineComments = true }) {
+const DiffRows = memo(function DiffRows({ rows, comments = [], selection, onLine, showInlineComments = true, gutterDigits }) {
+	const anchoredComments = useMemo(() => {
+		const index = /* @__PURE__ */ new Map();
+		for (const comment of comments) {
+			const key = `${comment.side}:${comment.endLine ?? comment.startLine}`;
+			const entries = index.get(key);
+			if (entries) entries.push(comment);
+			else index.set(key, [comment]);
+		}
+		return index;
+	}, [comments]);
+	const digits = gutterDigits ?? rows.reduce((max, row) => Math.max(max, String(row.line).length), 1);
 	return /* @__PURE__ */ jsx(View, { children: rows.map((row, i) => {
 		const side = row.marker === "-" ? "old" : "new";
-		const anchored = comments.filter((comment) => comment.side === side && (comment.endLine ?? comment.startLine) === row.line);
+		const anchored = anchoredComments.get(`${side}:${row.line}`) ?? [];
 		return /* @__PURE__ */ jsxs(View, { children: [/* @__PURE__ */ jsxs(Pressable, {
 			accessibilityRole: "button",
 			accessibilityLabel: `Select ${side} line ${row.line}`,
@@ -1334,8 +1682,9 @@ function DiffRows({ rows, comments = [], selection, onLine, showInlineComments =
 			],
 			children: [
 				/* @__PURE__ */ jsx(Text, {
+					numberOfLines: 1,
 					style: [styles$1.mono, styles$1.gutter],
-					children: row.line
+					children: String(row.line).padStart(digits, "\xA0")
 				}),
 				/* @__PURE__ */ jsx(Text, {
 					style: [styles$1.dim, styles$1.mono],
@@ -1361,7 +1710,7 @@ function DiffRows({ rows, comments = [], selection, onLine, showInlineComments =
 			})]
 		}, comment.id))] }, i);
 	}) });
-}
+});
 function lineLabel(comment) {
 	return comment.startLine === void 0 ? "File comment" : comment.endLine === comment.startLine || comment.endLine === void 0 ? `Line ${comment.startLine}` : `Lines ${comment.startLine} to ${comment.endLine}`;
 }
@@ -1637,10 +1986,20 @@ const styles$1 = StyleSheet.create((theme) => ({
 	},
 	gutter: {
 		color: theme.colors.dim,
-		width: 24,
+		width: "auto",
+		minWidth: 24,
+		flexShrink: 0,
 		textAlign: "right",
 		uni__dependencies: [0]
 	},
+	disclosure: {
+		padding: theme.spacing(3),
+		gap: theme.spacing(2),
+		borderTopWidth: 1,
+		borderColor: theme.colors.border,
+		backgroundColor: theme.colors.bg,
+		uni__dependencies: [0]
+	},
 	code: {
 		flex: 1,
 		minWidth: 0,
@@ -1676,8 +2035,25 @@ const styles$1 = StyleSheet.create((theme) => ({
 	},
 	remove: { alignSelf: "flex-end" }
 }));
+const DIFF_ROW_STEP = 200;
 function GitReviewView(props) {
 	const { screen, activity, summary, file, path, comments } = props;
+	const rows = useMemo(() => file?.hunks.flatMap((hunk) => hunk.rows) ?? [], [file?.hunks]);
+	const gutterDigits = useMemo(() => rows.reduce((max, row) => Math.max(max, String(row.line).length), 1), [rows]);
+	const fileComments = useMemo(() => comments.filter((comment) => comment.path === path), [comments, path]);
+	const initialLimit = props.initialVisibleRows ?? DIFF_ROW_STEP;
+	const [disclosure, setDisclosure] = useState({
+		rows,
+		path,
+		limit: initialLimit
+	});
+	const current = disclosure.rows === rows && disclosure.path === path ? disclosure : {
+		rows,
+		path,
+		limit: initialLimit
+	};
+	if (current !== disclosure) setDisclosure(current);
+	const visibleRows = useMemo(() => rows.slice(0, current.limit), [rows, current.limit]);
 	const index = summary?.files.findIndex((entry) => entry.path === path) ?? -1;
 	const entry = summary?.files[index];
 	return /* @__PURE__ */ jsxs(View, {
@@ -1789,13 +2165,14 @@ function GitReviewView(props) {
 							style: styles$1.section,
 							children: file.binary ? "Binary file" : "Diff too large to display"
 						})
-					}) : file.hunks.map((hunk, i) => /* @__PURE__ */ jsx(DiffRows, {
-						rows: hunk.rows,
-						comments: comments.filter((comment) => comment.path === file.path),
+					}) : /* @__PURE__ */ jsx(DiffRows, {
+						rows: visibleRows,
+						comments: fileComments,
 						selection: props.selection,
-						onLine: props.onLine
-					}, i)),
-					comments.filter((comment) => comment.path === path && comment.startLine === void 0).map((comment) => /* @__PURE__ */ jsxs(View, {
+						onLine: props.onLine,
+						gutterDigits
+					}),
+					fileComments.filter((comment) => comment.startLine === void 0).map((comment) => /* @__PURE__ */ jsxs(View, {
 						style: styles$1.inlineComment,
 						children: [/* @__PURE__ */ jsx(Text, {
 							style: styles$1.blue,
@@ -1819,7 +2196,7 @@ function GitReviewView(props) {
 					style: [styles$1.red, styles$1.content],
 					children: props.error
 				}) : null]
-			}),
+			}, `${screen}:${path ?? ""}`),
 			/* @__PURE__ */ jsx(Footer, {
 				count: comments.length,
 				confirmation: screen === "changes" && activity.changes?.rebase !== void 0,
@@ -1827,7 +2204,35 @@ function GitReviewView(props) {
 				onReview: () => props.onScreen("review"),
 				onSend: props.onSend,
 				onDiscard: props.onDiscard
-			})
+			}),
+			screen === "file" && file && !file.binary && !file.tooLarge ? /* @__PURE__ */ jsxs(View, {
+				style: styles$1.disclosure,
+				children: [/* @__PURE__ */ jsxs(Text, {
+					style: styles$1.dim,
+					accessibilityLiveRegion: "polite",
+					children: [
+						Math.min(current.limit, rows.length),
+						" of ",
+						rows.length,
+						" diff rows shown",
+						props.selection ? ` · ${props.selection.side} line ${props.selection.startLine} selected` : ""
+					]
+				}), current.limit < rows.length ? /* @__PURE__ */ jsxs(Button, {
+					variant: "outline",
+					onPress: () => setDisclosure({
+						...current,
+						limit: Math.min(rows.length, current.limit + DIFF_ROW_STEP)
+					}),
+					children: [
+						"Show ",
+						Math.min(DIFF_ROW_STEP, rows.length - current.limit),
+						" more rows"
+					]
+				}) : /* @__PURE__ */ jsx(Text, {
+					style: styles$1.dim,
+					children: "All diff rows shown"
+				})]
+			}) : null
 		]
 	});
 }
@@ -1866,7 +2271,6 @@ function GitReviewPanelSession({ sessionId, sendSessionFrame }) {
 	const [path, setPath] = useState();
 	const [selection, setSelection] = useState();
 	const [draft, setDraft] = useState();
-	const [comment, setComment] = useState("");
 	const request = useRef(void 0);
 	const fingerprint = JSON.stringify(activity.changes);
 	useEffect(() => {
@@ -1976,20 +2380,16 @@ function GitReviewPanelSession({ sessionId, sendSessionFrame }) {
 				if (!open) {
 					setDraft(void 0);
 					setSelection(void 0);
-					setComment("");
 				}
 			},
-			children: /* @__PURE__ */ jsx(SheetContent, { children: /* @__PURE__ */ jsx(SheetBody, { children: /* @__PURE__ */ jsx(CommentSheet, {
+			children: /* @__PURE__ */ jsx(SheetContent, { children: /* @__PURE__ */ jsx(SheetBody, { children: /* @__PURE__ */ jsx(CommentEditor, {
 				range: draft?.range,
 				snippet: draft?.snippet ?? "",
-				value: comment,
-				onChange: setComment,
 				onCancel: () => {
 					setDraft(void 0);
 					setSelection(void 0);
-					setComment("");
 				},
-				onAdd: () => {
+				onAdd: (comment) => {
 					if (!draft || !comment.trim()) return;
 					addReviewComment(sessionId, {
 						path: draft.path,
@@ -2000,9 +2400,8 @@ function GitReviewPanelSession({ sessionId, sendSessionFrame }) {
 					});
 					setDraft(void 0);
 					setSelection(void 0);
-					setComment("");
 				}
-			}) }) })
+			}, draft ? "open" : "closed") }) })
 		}),
 		/* @__PURE__ */ jsx(Sheet, {
 			open: confirm !== void 0,
@@ -2034,6 +2433,15 @@ function GitReviewPanelSession({ sessionId, sendSessionFrame }) {
 		})
 	] });
 }
+function CommentEditor({ onAdd, ...props }) {
+	const [body, setBody] = useState("");
+	return /* @__PURE__ */ jsx(CommentSheet, {
+		...props,
+		value: body,
+		onChange: setBody,
+		onAdd: () => onAdd(body)
+	});
+}
 //#endregion
 //#region src/extensions/workspaces/sessions/(frontend)/fill/_components/WorktreesActivitySection.native.tsx
 function GitChangesRow({ state = "changes", branch = "Detached HEAD", added = 0, removed = 0, onReview }) {
@@ -2241,6 +2649,12 @@ const via = (identity, value) => value === void 0 ? void 0 : {
 };
 const nativePlugin = defineNativePlugin({
 	id: "git",
+	workspace: { get fills() {
+		return [via({
+			slot: "new-session",
+			id: "git-new-session"
+		}, GitNewSession_new_session_native_default)];
+	} },
 	session: {
 		get activityGroups() {
 			return [via({ name: "git" }, git_native_default)];