skydive-cli 0.2.0-beta.427 → 0.2.0-beta.455

package/dist/js/bin.mjs

@@ -20,7 +20,7 @@ import semver from "semver";
 
 //#region package.json
 var name = "skydive-cli";
-var version = "0.2.0-beta.427";
+var version = "0.2.0-beta.455";
 
 //#endregion
 //#region src/types.ts
@@ -1200,7 +1200,7 @@ const importCommand = {
 				process.exit(1);
 			}
 		}
-		const { runChat } = await import("./boot-BiMPCqRg.mjs");
+		const { runChat } = await import("./boot-SxcKHfwY.mjs");
 		await runChat({
 			appUrl,
 			sessionToken: session.value.sessionToken,
@@ -1511,7 +1511,7 @@ const chatCommand = {
 			printError(`Unknown theme "${themeId}". Valid themes: ${themes.map((t) => t.id).join(", ")}`);
 			process.exit(1);
 		}
-		const { runChat } = await import("./boot-BiMPCqRg.mjs");
+		const { runChat } = await import("./boot-SxcKHfwY.mjs");
 		await runChat({
 			appUrl,
 			sessionToken: session.value.sessionToken,
@@ -1810,7 +1810,7 @@ const switchCommand = {
 				printError("The workspace picker needs the Bun runtime and it could not be set up automatically. Pass a workspace slug instead, or install Bun and retry.");
 				process.exit(1);
 			}
-			const { runWorkspacePicker } = await import("./boot-BiMPCqRg.mjs");
+			const { runWorkspacePicker } = await import("./boot-SxcKHfwY.mjs");
 			await runWorkspacePicker(session);
 			return;
 		}

package/dist/js/{boot-BiMPCqRg.mjs → boot-SxcKHfwY.mjs}

@@ -4357,6 +4357,10 @@ const keybindGroups = [
 				keys: "1 / 2",
 				action: "switch Changes / Files"
 			},
+			{
+				keys: "\\",
+				action: "toggle side / stacked layout"
+			},
 			{
 				keys: "↵ / o / l",
 				action: "focus selected file preview"
@@ -6747,7 +6751,7 @@ function gcOnce() {
 	gcStarted = true;
 	store.gc();
 }
-function ReviewPane({ agentId, conversationId, items, viewportWidth, viewportHeight, active, shellLayout, tab, onTabChange, selectedChangeFile, onSelectedChangeFileChange, selectedWorkspaceFile, onSelectedWorkspaceFileChange, navigatorSize, onNavigatorResize, onInsertToComposer, onRequestChatFocus, onClose }) {
+function ReviewPane({ agentId, conversationId, items, viewportWidth, viewportHeight, active, shellLayout, tab, onTabChange, selectedChangeFile, onSelectedChangeFileChange, selectedWorkspaceFile, onSelectedWorkspaceFileChange, navigatorSize, onNavigatorResize, onToggleOrientation, onInsertToComposer, onRequestChatFocus, onClose }) {
 	const width = viewportWidth;
 	const height = viewportHeight;
 	const shellHeight = Math.max(6, height - 2);
@@ -6898,6 +6902,10 @@ function ReviewPane({ agentId, conversationId, items, viewportWidth, viewportHei
 			onTabChange("files");
 			return;
 		}
+		if (key.name === "\\") {
+			if (onToggleOrientation() === "unavailable") setStatus("no room to change layout in this terminal");
+			return;
+		}
 		if (key.name === "s") {
 			if (comments.length === 0) {
 				setStatus("no pending comments");
@@ -7288,18 +7296,93 @@ function PaneRowView({ row, width, isCursor, inSelection, isAnchor, isCommented
 //#endregion
 //#region src/chat/tui/file-pane-layout.ts
 /**
+* Terminal cells are about twice as tall as they are wide, so a raw
+* cols-vs-rows compare would call almost any terminal "wide". Scale rows by
+* this factor to compare the *visual* aspect: a square-looking terminal has
+* `width ≈ height * CELL_ASPECT`.
+*/
+const CELL_ASPECT = 2;
+/**
+* The automatic split follows the terminal's visual aspect: wider than tall
+* places review beside the chat (a vertical divider), taller than wide stacks
+* it underneath (a horizontal divider). This is the responsive default when
+* the user hasn't pinned an orientation.
+*/
+function autoPrefersSide({ width, height }) {
+	return width >= height * CELL_ASPECT;
+}
+/**
+* Bare-minimum room to render a split at all (as opposed to the higher
+* comfort thresholds the automatic choice uses). A user pin is honored down
+* to these: chat + review each need a usable strip, but we don't insist on
+* the roomier auto breakpoints. Mirrors the clamps in clampSidePaneWidth /
+* clampBelowPaneHeight so a pinned split never renders narrower than those.
+*/
+function canRenderSide({ width, height }) {
+	return width >= 76 && height >= 8;
+}
+function canRenderBelow({ width, height }) {
+	return height >= 24 && width >= 24;
+}
+/**
 * Responsive placement for the file pane.
 *
-* Wide terminals keep the conversation readable and place review beside it.
-* Tall terminals without enough horizontal room stack review underneath.
-* Cramped terminals give review the full body rather than rendering two
-* unusably small surfaces.
+* With no user pin the split follows the terminal's visual aspect (see
+* autoPrefersSide): wider than tall -> side, taller than wide -> stacked.
+* The preferred split still has to be renderable; if it isn't we use the
+* other, and cramped terminals that can render neither get the full body.
+*
+* A user `orientation` pin always wins as long as the terminal can render
+* that split at all (the lower canRender* bar) — so you get the layout you
+* asked for even on a smallish terminal. Only when the pinned split
+* genuinely can't fit do we fall back: to the other split if it can render,
+* else to `fullscreen`.
 */
-function filePanePlacement({ width, height }) {
-	if (width >= 140 && height >= 24) return "side";
-	if (width >= 50 && height >= 40) return "below";
+function filePanePlacement({ width, height, orientation }) {
+	const renderSide = canRenderSide({
+		width,
+		height
+	});
+	const renderBelow = canRenderBelow({
+		width,
+		height
+	});
+	if (orientation === "side") {
+		if (renderSide) return "side";
+		if (renderBelow) return "below";
+		return "fullscreen";
+	}
+	if (orientation === "below") {
+		if (renderBelow) return "below";
+		if (renderSide) return "side";
+		return "fullscreen";
+	}
+	if (autoPrefersSide({
+		width,
+		height
+	})) {
+		if (renderSide) return "side";
+		if (renderBelow) return "below";
+		return "fullscreen";
+	}
+	if (renderBelow) return "below";
+	if (renderSide) return "side";
 	return "fullscreen";
 }
+/**
+* Whether toggling the orientation would actually change anything: both
+* splits must be renderable. When only one (or neither) can render, the
+* caller surfaces a hint instead of a silent no-op.
+*/
+function canToggleOrientation({ width, height }) {
+	return canRenderSide({
+		width,
+		height
+	}) && canRenderBelow({
+		width,
+		height
+	});
+}
 /** Keep a dragged side pane wide enough to use and leave the chat usable. */
 function clampSidePaneWidth(width, bodyWidth) {
 	return Math.max(44, Math.min(bodyWidth - 30, width));
@@ -7827,6 +7910,7 @@ function ChatScreen({ agent, conversation }) {
 		side: null,
 		stacked: null
 	});
+	const [reviewOrientation, setReviewOrientation] = useState(null);
 	const [reviewWorkspaceFile, setReviewWorkspaceFile] = useState(null);
 	useEffect(() => {
 		setReviewChangeFile(null);
@@ -7882,7 +7966,8 @@ function ChatScreen({ agent, conversation }) {
 	const bodyHeight = Math.max(1, height - 3);
 	const reviewPlacement = filePanePlacement({
 		width,
-		height
+		height,
+		orientation: reviewOrientation
 	});
 	const reviewWidth = reviewPlacement === "side" ? clampSidePaneWidth(reviewSizePref.side ?? sidePaneWidth(bodyWidth), bodyWidth) : bodyWidth;
 	const reviewHeight = reviewPlacement === "below" ? clampBelowPaneHeight(reviewSizePref.below ?? belowPaneHeight(bodyHeight), bodyHeight) : bodyHeight;
@@ -7925,6 +8010,7 @@ function ChatScreen({ agent, conversation }) {
 		helpOpen,
 		reviewOpen,
 		reviewFocused,
+		reviewPlacement,
 		credPromptOpen
 	]);
 	useEffect(() => {
@@ -9019,6 +9105,14 @@ function ChatScreen({ agent, conversation }) {
 			...prev,
 			side: size
 		}),
+		onToggleOrientation: () => {
+			if (!canToggleOrientation({
+				width,
+				height
+			})) return "unavailable";
+			setReviewOrientation(reviewPlacement === "below" ? "side" : "below");
+			return "toggled";
+		},
 		onInsertToComposer: (review) => {
 			const next = [inputRef.current.trimEnd(), review].filter((part) => part.length > 0).join("\n\n");
 			setInput(next);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skydive-cli",
-  "version": "0.2.0-beta.427",
+  "version": "0.2.0-beta.455",
   "description": "Skydive CLI — cloud agents from the command line",
   "homepage": "https://skydive.com",
   "license": "MIT",