@agimon-ai/po-git 0.0.0-alpha.26 → 0.0.0-alpha.28

package/dist/extensions/web.mjs

@@ -1,5 +1,5 @@
 import { createApiClient, defineActivityGroup, defineChannelFile, defineFill, defineGlobalStore, defineLeaderBinding, defineRepositorySettingsPanel, defineSessionStore, defineToolRenderer, defineWebLifecycle, defineWebPlugin, sessionApiAddress } from "@agimon-ai/po-core/web";
-import { BranchIcon, Button, CheckIcon, ChevronDownIcon, ChevronUpIcon, Dialog, DialogContent, DialogTitle, DiffView, Dot, DropdownMenuItem, EmptyState, Input, MessageIcon, MessageItem, MessageItemBody, ReviewCommentDraft, Select, SelectContent, SelectItem, SelectTrigger, SelectValue, SettingsActions, SettingsCard, SettingsNote, SettingsRow, SettingsRowDescription, SettingsRowText, SettingsRowTitle, SettingsSection, SettingsSectionDescription, SettingsSectionHeader, SettingsSectionTitle, ShieldIcon, Spinner, StatusBadge, Tabs, TabsContent, TabsList, TabsTrigger, buildReviewPrompt } from "@agimon-ai/po-web-components";
+import { ArrowRightIcon, BranchIcon, Button, CheckIcon, ChevronDownIcon, ChevronUpIcon, Dialog, DialogContent, DialogTitle, DiffView, Dot, DropdownMenuItem, EmptyState, Input, MessageIcon, MessageItem, MessageItemBody, ReviewCommentDraft, Select, SelectContent, SelectItem, SelectTrigger, SelectValue, SettingsActions, SettingsCard, SettingsNote, SettingsRow, SettingsRowDescription, SettingsRowText, SettingsRowTitle, SettingsSection, SettingsSectionDescription, SettingsSectionHeader, SettingsSectionTitle, ShieldIcon, Spinner, StatusBadge, Tabs, TabsContent, TabsList, TabsTrigger, buildReviewPrompt } from "@agimon-ai/po-web-components";
 import { createElement, useCallback, useEffect, useId, useRef, useState } from "react";
 import { sealedTransport } from "@agimon-ai/po-web-security/browser";
 import { Fragment, jsx, jsxs } from "react/jsx-runtime";
@@ -2478,8 +2478,10 @@ function GitReviewPanel({ sessionId, sendSessionFrame }) {
 * The '# git' group's branch row: this session's code change against its base.
 *
 * DESIGN PATTERNS:
-* - One lane, branch and `+A -R`. Files, base, upstream and a paused rebase live
-*   in the review tab's header, which this lane opens.
+* - One lane, branch and `+A -R`, straight on the group body like every other
+*   activity group: the dock already draws the card, so no inner card or heading.
+*   Files, base, upstream and a paused rebase live in the review tab's header,
+*   which this lane opens.
 * - Stays clickable at zero, because the review tab is also where pull, push
 *   and rebase live.
 * - Reads the focused session's checkout, whether it is a worktree or not.
@@ -2501,42 +2503,41 @@ function DiffActivityView({ changes, pending, onReview, branch }) {
 	const zero = changes?.added === 0 && changes?.removed === 0;
 	const against = changes?.base === void 0 ? "uncommitted" : `vs ${changes.base}`;
 	return /* @__PURE__ */ jsxs("div", {
-		className: "rounded-lg border border-po-border-soft bg-po-panel p-4",
-		children: [
-			/* @__PURE__ */ jsx("h2", {
-				className: "mb-3 text-lg font-semibold text-po-hi",
-				children: "Changes"
-			}),
-			/* @__PURE__ */ jsxs(Button, {
-				variant: "ghost",
-				size: "xs",
-				"data-testid": "activity-diff-changes",
-				className: "h-auto min-h-0 w-full min-w-0 gap-2 px-0 py-0 hover:bg-transparent",
-				"aria-label": changes ? `review ${changes.branch ?? "detached HEAD"} changes: ${changes.added} added, ${changes.removed} removed ${against}` : "Changes not reported yet",
-				title: changes ? `${changes.files} ${changes.files === 1 ? "file" : "files"} ${against}` : void 0,
-				onClick: onReview,
-				children: [/* @__PURE__ */ jsx("span", {
-					className: "min-w-0 flex-1 truncate text-left font-mono text-base font-normal text-po-hi",
-					children: changes?.branch ?? branch ?? "detached HEAD"
-				}), !changes || zero ? /* @__PURE__ */ jsx("span", {
-					className: "text-sm font-normal text-po-dim",
-					children: zero ? "No changes yet" : "Not reported yet"
-				}) : /* @__PURE__ */ jsxs(Fragment, { children: [/* @__PURE__ */ jsxs("span", {
+		className: "flex min-w-0 flex-col gap-1",
+		children: [/* @__PURE__ */ jsxs(Button, {
+			variant: "ghost",
+			size: "card",
+			"data-testid": "activity-diff-changes",
+			className: "min-w-0 flex-row items-center gap-2 rounded-md px-1 py-1 font-normal hover:bg-po-panel",
+			"aria-label": changes ? `review ${changes.branch ?? "detached HEAD"} changes: ${changes.added} added, ${changes.removed} removed ${against}` : "Changes not reported yet",
+			title: changes ? `${changes.files} ${changes.files === 1 ? "file" : "files"} ${against}` : void 0,
+			onClick: onReview,
+			children: [/* @__PURE__ */ jsx("span", {
+				className: "min-w-0 flex-1 truncate text-left font-mono text-sm text-po-hi",
+				children: changes?.branch ?? branch ?? "detached HEAD"
+			}), !changes || zero ? /* @__PURE__ */ jsx("span", {
+				className: "shrink-0 text-sm text-po-dim",
+				children: zero ? "No changes yet" : "Not reported yet"
+			}) : /* @__PURE__ */ jsxs("span", {
+				className: "flex shrink-0 gap-2 text-sm",
+				children: [/* @__PURE__ */ jsxs("span", {
 					className: "text-po-green",
 					children: ["+", changes.added]
 				}), /* @__PURE__ */ jsxs("span", {
 					className: "text-po-red",
 					children: ["-", changes.removed]
-				})] })]
-			}),
-			pending && /* @__PURE__ */ jsxs("p", {
-				className: "mt-2 flex items-center gap-2 text-sm text-po-dim",
-				children: [/* @__PURE__ */ jsx(Spinner, {
-					className: "size-3",
-					label: pending
-				}), pending]
-			})
-		]
+				})]
+			})]
+		}), pending && /* @__PURE__ */ jsxs("p", {
+			className: "flex items-center gap-2 px-1 text-sm text-po-dim",
+			children: [/* @__PURE__ */ jsx(Spinner, {
+				className: "size-3",
+				label: pending
+			}), /* @__PURE__ */ jsx("span", {
+				className: "inline-block first-letter:uppercase",
+				children: pending
+			})]
+		})]
 	});
 }
 //#endregion
@@ -2544,7 +2545,7 @@ function DiffActivityView({ changes, pending, onReview, branch }) {
 function WorktreesActivitySection(props) {
 	const session = useStore(worktreeActivity.store, (state) => worktreeActivity.select(state, props.sessionId));
 	return /* @__PURE__ */ jsxs("div", {
-		className: "flex min-w-0 flex-col gap-6",
+		className: "flex min-w-0 flex-col gap-1",
 		children: [/* @__PURE__ */ jsx(DiffActivitySection, { ...props }), /* @__PURE__ */ jsx(WorktreesActivityView, {
 			worktrees: session.worktrees.filter((w) => w.sessionId !== props.sessionId),
 			pending: session.pending,
@@ -2558,52 +2559,45 @@ function WorktreesActivitySection(props) {
 }
 function WorktreesActivityView({ worktrees, pending, onOpen, onCreate, startingIds = [] }) {
 	return /* @__PURE__ */ jsxs("div", {
-		className: "rounded-lg border border-po-border-soft bg-po-panel p-4",
-		children: [
-			/* @__PURE__ */ jsx("h2", {
-				className: `${worktrees.length === 0 ? "mb-3" : "mb-2"} text-lg font-semibold text-po-hi`,
-				children: "Worktrees"
-			}),
-			pending && /* @__PURE__ */ jsxs("p", {
-				className: "flex items-center gap-2 text-sm text-po-dim",
-				children: [/* @__PURE__ */ jsx(Spinner, {
-					className: "size-3",
-					label: pending
-				}), /* @__PURE__ */ jsx("span", {
-					className: "inline-block first-letter:uppercase",
-					children: pending
-				})]
-			}),
-			worktrees.length === 0 ? /* @__PURE__ */ jsxs("div", {
-				className: "flex items-center justify-between",
-				children: [/* @__PURE__ */ jsx("span", {
-					"data-testid": "activity-summary-git",
-					className: "text-sm text-po-dim",
-					children: "Idle"
-				}), onCreate && /* @__PURE__ */ jsx(Button, {
-					variant: "link",
-					size: "sm",
-					"data-testid": "activity-git-create",
-					className: "text-po-blue",
-					onClick: onCreate,
-					children: "Create a worktree"
-				})]
-			}) : /* @__PURE__ */ jsx("div", {
-				"data-testid": "activity-git-worktrees",
-				children: worktrees.map((w) => /* @__PURE__ */ jsx(Button, {
-					variant: "ghost",
-					size: "card",
-					className: "block w-full min-w-0 p-0 text-left font-normal",
-					"data-testid": `activity-git-worktree-${w.id}`,
-					onClick: onOpen,
-					children: /* @__PURE__ */ jsx(WorktreeRow, {
-						compact: true,
-						worktree: w,
-						starting: startingIds.includes(w.id) || !!pending?.includes("starting") && w.sessionId === null && !w.orphaned && !w.unowned
-					})
-				}, w.id))
-			})
-		]
+		className: "flex min-w-0 flex-col gap-1",
+		children: [pending && /* @__PURE__ */ jsxs("p", {
+			className: "flex items-center gap-2 px-1 text-sm text-po-dim",
+			children: [/* @__PURE__ */ jsx(Spinner, {
+				className: "size-3",
+				label: pending
+			}), /* @__PURE__ */ jsx("span", {
+				className: "inline-block first-letter:uppercase",
+				children: pending
+			})]
+		}), worktrees.length === 0 ? onCreate ? /* @__PURE__ */ jsxs(Button, {
+			variant: "link",
+			size: "sm",
+			"data-testid": "activity-git-create",
+			className: "h-auto self-start gap-1 px-1 text-po-green",
+			onClick: onCreate,
+			children: ["Create a worktree", /* @__PURE__ */ jsx(ArrowRightIcon, {
+				"aria-hidden": true,
+				className: "size-3.5"
+			})]
+		}) : /* @__PURE__ */ jsx("span", {
+			"data-testid": "activity-summary-git",
+			className: "px-1 text-sm text-po-dim",
+			children: "Idle"
+		}) : /* @__PURE__ */ jsx("div", {
+			"data-testid": "activity-git-worktrees",
+			children: worktrees.map((w) => /* @__PURE__ */ jsx(Button, {
+				variant: "ghost",
+				size: "card",
+				className: "block w-full min-w-0 p-0 text-left font-normal",
+				"data-testid": `activity-git-worktree-${w.id}`,
+				onClick: onOpen,
+				children: /* @__PURE__ */ jsx(WorktreeRow, {
+					compact: true,
+					worktree: w,
+					starting: startingIds.includes(w.id) || !!pending?.includes("starting") && w.sessionId === null && !w.orphaned && !w.unowned
+				})
+			}, w.id))
+		})]
 	});
 }
 //#endregion