prreviewbuddy 0.82.1 → 0.83.3

package/CHANGELOG.md

@@ -1,5 +1,63 @@
 # Changelog
 
+## 0.83.3
+
+**Peek at a collapsed sidebar.** Rest the pointer on the collapsed sidebar, or tab into it, and it
+slides out to full width over the page, with every name and section, until you move away. The page
+underneath stays where it was, and the sidebar stays collapsed.
+
+## 0.83.2
+
+**Send feedback is only in the sidebar.** A narrow window no longer draws a Send feedback link in
+the top toolbar; it is in the sidebar, expanded or collapsed, and nowhere else.
+
+## 0.83.1
+
+**New review is the sidebar's one purple button, and Collapse sidebar moves to the sidebar's foot.**
+New review is filled in purple wherever it is: the full-width button, the square + of the collapsed
+sidebar, and the + in the top toolbar. Collapse sidebar is now the last row of the sidebar, below
+Send feedback, and expands it again from the same place. While the sidebar is hidden, Show sidebar
+in the top toolbar brings it back, and Send feedback is hidden with it.
+
+## 0.83.0
+
+**Start a review from New review in the sidebar, and collapse the sidebar to icons.** A New review
+button, with its shortcut beside it, sits above Reviews in the sidebar on every workspace page. It
+opens the same panel as ⌘P (Ctrl+P on other platforms), which works as before; that panel is now
+called New review too. The button at the far left of the top toolbar collapses the sidebar to a
+column of icons, with New review as a + and each name shown on hover, and expands it again. ⌘K H
+hides the sidebar completely; the same toolbar button then brings it back, and a + in the toolbar
+starts a review while it is hidden. Your choice is remembered.
+
+## 0.82.5
+
+**Help explains Collections more plainly.** The Collections topic in Workspace Help now says what
+collections are for, how to add reviews, what happens when one is marked Done, and that collapsing
+helps you stay focused. The note on how long reviews in a collection are kept is unchanged.
+
+## 0.82.4
+
+**⌘K L C lists your collections, to expand or collapse one.** Each row shows the collection's
+name, its count and whether it is expanded. Move with J and K or the arrows and press Enter, or
+press a row's number. Expanding a collection from here also lets J and K walk into its reviews.
+New collection moves to ⌘K L N.
+
+## 0.82.3
+
+**⌘K L C makes a new collection.** It opens the same New collection dialog as the button beside
+Select. After you create or rename a collection, its heading takes focus without the keyboard ring
+the name field had; Tab brings the ring back. While a new collection is being made, its name is
+locked and the dialog cannot be opened again, so a slow answer can never close or change the next
+one you start.
+
+## 0.82.2
+
+**Collections sit in the list like any other section.** Each collection is now a section between
+Pinned and Active, with no Collections heading above them, and nothing is drawn for collections
+until you make one. New collection moves up beside Select and opens a small dialog that says what a
+collection is for, with Cancel and Create. A collection whose reviews are all Done now says "No
+active reviews".
+
 ## 0.82.1
 
 **Back no longer draws a focus ring you never had.** Going Back or Forward puts focus where you

package/dist/server.js

@@ -376,7 +376,7 @@ function collectionEmptyText(counts, filtered) {
 	if (filtered) return "Nothing in this collection matches the filters.";
 	if (counts.members === 0) return "Nothing here yet. Add reviews from a card’s menu, or select several and choose Manage collections.";
 	if (counts.pinned > 0 && counts.completed === 0) return "Its reviews are pinned, so they are listed under Pinned.";
-	return "Nothing here needs attention.";
+	return "No active reviews";
 }
 /** Said under a collection whose oldest members are not drawn, so it never looks smaller than it is. */
 function omittedText(omitted) {
@@ -803,7 +803,10 @@ var PATHS = {
 	closed: "<circle cx=\"12\" cy=\"12\" r=\"9\"/><path d=\"m5.6 5.6 12.8 12.8\"/>",
 	reopened: "<circle cx=\"12\" cy=\"12\" r=\"9\"/><circle cx=\"12\" cy=\"12\" r=\"2\"/>",
 	warning: "<path d=\"M10.3 3.9 1.8 18a2 2 0 0 0 1.7 3h17a2 2 0 0 0 1.7-3L13.7 3.9a2 2 0 0 0-3.4 0z\"/><path d=\"M12 9v4M12 17h.01\"/>",
-	forward: "<path d=\"M5 12h14\"/><path d=\"m12 5 7 7-7 7\"/>"
+	forward: "<path d=\"M5 12h14\"/><path d=\"m12 5 7 7-7 7\"/>",
+	plus: "<path d=\"M12 5v14M5 12h14\"/>",
+	"sidebar-collapse": "<rect x=\"3\" y=\"4\" width=\"18\" height=\"16\" rx=\"2\"/><path d=\"M9 4v16\"/><path d=\"m16 9-3 3 3 3\"/>",
+	"sidebar-expand": "<rect x=\"3\" y=\"4\" width=\"18\" height=\"16\" rx=\"2\"/><path d=\"M9 4v16\"/><path d=\"m13 9 3 3-3 3\"/>"
 };
 /** The icon slot for one rail row. Hidden from assistive technology: the label beside it says it. */
 function railIcon(icon) {
@@ -863,6 +866,18 @@ function serverStatusLabel(status) {
 	return "Workspace server running";
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/sidebar_state.ts
+/**
+* What the sidebar's own toggle, at its foot, will do when pressed, which is what it is called.
+* Only Expanded and Compact: while the sidebar is hidden that toggle is not drawn, and the strip's
+* restore control, which only ever says Show sidebar, stands in for it.
+*/
+function sidebarToggleLabel(sidebar) {
+	return sidebar?.compact ? "Expand sidebar" : "Collapse sidebar";
+}
+/** The strip's restore control, drawn only while the sidebar is hidden. */
+var SHOW_SIDEBAR = "Show sidebar";
+//#endregion
 //#region ../../packages/review-harness/src/workspace/workspace_nav.ts
 /**
 * The workspace's own places, and the one model every page draws them from.
@@ -905,6 +920,13 @@ var WORKSPACE_PLACES = [
 	}
 ];
 /**
+* The sidebar's state, on `<html>` beside the theme, so the page is drawn that way from its first
+* paint. The stylesheet reads it; `client/sidebar.ts` changes it.
+*/
+function sidebarAttributes(sidebar) {
+	return `${sidebar?.compact ? " data-sidebar-compact" : ""}${sidebar?.hidden ? " data-sidebar-hidden" : ""}`;
+}
+/**
 * The count beside Notifications. Always rendered, hidden at zero, so the page can reveal it when
 * `/revision` or Mark all read reports a new number. See `client/unread_counts.ts`.
 */
@@ -934,7 +956,8 @@ function renderGlobalBar(options) {
 	const live = options.live ?? true;
 	return `<header class="prb-strip">
   <span class="prb-wordmark"><b>pr</b>review<b>buddy</b></span>
-  ${live ? renderHeaderNav(options) : ""}
+  ${live ? renderSidebarRestore() : ""}
+  ${live ? renderNewReview() : ""}
   ${live ? renderHistoryControls() : ""}
   ${live ? renderStripReviews(options) : ""}
   ${live ? renderStripRuns(options) : ""}
@@ -980,6 +1003,37 @@ function renderNotificationsBell(options) {
   </a>`;
 }
 /**
+* Show sidebar: the way back from Hidden, and drawn only then (see `shell.css`). Among the strip's
+* controls rather than beside the wordmark, so it reads as a control and not as part of the brand.
+* Named in words and in the strip's own tooltip, never a `title`.
+*/
+function renderSidebarRestore() {
+	return `<button type="button" class="prb-strip-action prb-sidebar-restore" data-sidebar-toggle="strip" aria-label="${SHOW_SIDEBAR}" data-tip="${SHOW_SIDEBAR}">${iconSvg("sidebar-expand")}</button>`;
+}
+/**
+* The sidebar's own toggle, the last row of its foot: Expanded and Compact, back and forth.
+*
+* In the same place in both, so it never moves under the pointer that pressed it. Hidden is not a
+* third press: it is its own act (⌘K H), and the strip's Show sidebar brings it back at the width it
+* had. The row's words say what pressing it will do, and in Compact they are its tooltip, as every
+* row's are. Both drawings are rendered and the stylesheet shows the one for the state, so nothing
+* needs repainting but the words. `client/sidebar.ts` keeps those true.
+*/
+function renderSidebarToggle(sidebar) {
+	return `<button type="button" class="prb-rail-item prb-rail-link prb-sidebar-toggle" data-sidebar-toggle="rail" aria-expanded="${!sidebar?.compact}">
+      <span class="prb-rail-icon" aria-hidden="true"><span class="prb-sidebar-collapse-icon">${iconSvg("sidebar-collapse")}</span><span class="prb-sidebar-expand-icon">${iconSvg("sidebar-expand")}</span></span>
+      <span class="prb-rail-label"><span class="prb-rail-title">${sidebarToggleLabel(sidebar)}</span></span>
+    </button>`;
+}
+/**
+* New review, in the strip: the square stand-in for the rail's labelled button, shown only while
+* the rail is hidden (a narrow window, a review's focus mode). See `renderNewReviewButton`, and
+* `shell.css` for the one rule that decides which of the two is showing.
+*/
+function renderNewReview() {
+	return `<button type="button" class="prb-strip-action prb-strip-new-review" aria-label="New review" data-new-review="strip">${iconSvg("plus")}</button>`;
+}
+/**
 * Back and Forward through this tab's workspace history, ahead of the places.
 *
 * Workspace-wide, so here in the strip rather than in a review's own bar: they move between views,
@@ -1058,9 +1112,20 @@ function renderServerStatus() {
     <span class="prb-visually-hidden" data-server-status-label>${escapeHtml$1(serverStatusLabel("running"))}</span>
   </span>`;
 }
-/** The workspace rows, for a rail group. */
+/**
+* New review, at the head of the workspace rows: starting a review is the product's primary action,
+* so it is a labelled button rather than an icon to interpret, set apart from the places below it.
+*
+* What it does is wired by the command layer (`client/command_layer.ts`), which runs the very
+* command ⌘P runs, so the two cannot open different things. The shortcut beside the label is
+* painted there too, because only the browser knows the platform.
+*/
+function renderNewReviewButton() {
+	return `<div class="prb-rail-new"><button type="button" class="prb-new-review" data-new-review="rail">${iconSvg("plus")}<span class="prb-new-review-label">New review</span><kbd class="prb-new-review-key" data-new-review-key aria-hidden="true"></kbd></button></div>`;
+}
+/** New review, then the workspace rows, for a rail group. */
 function renderWorkspaceNav(options) {
-	return WORKSPACE_PLACES.map((item) => `<a class="prb-rail-item prb-rail-link" href="${item.href}"${item.place === options.current ? " aria-current=\"true\"" : ""}>
+	return renderNewReviewButton() + WORKSPACE_PLACES.map((item) => `<a class="prb-rail-item prb-rail-link" href="${item.href}"${item.place === options.current ? " aria-current=\"true\"" : ""}>
       ${railIcon(item.icon)}
       <span class="prb-rail-label"><span class="prb-rail-title">${escapeHtml$1(item.label)}</span></span>
       ${item.place === "notifications" ? unreadCount(options.unread, "rail") : ""}
@@ -1097,20 +1162,9 @@ function renderSendFeedback() {
 function renderWorkspaceRail(options) {
 	return `<nav class="prb-rail" aria-label="Workspace">
   <div class="prb-rail-group">${renderWorkspaceNav(options)}</div>
-  <div class="prb-rail-group prb-rail-foot">${renderSendFeedback()}</div>
+  <div class="prb-rail-group prb-rail-foot">${renderSendFeedback()}${renderSidebarToggle(options.sidebar)}</div>
 </nav>`;
 }
-/**
-* What the rail carries that the strip does not, shown only while a narrow window hides the rail.
-*
-* Which is now the feedback form alone. Every workspace place is an icon in this same strip at
-* every width, so a text link beside one of them would be the same destination twice; before the
-* Reviews icon existed this is where Reviews had to go, and the filter that put it here is gone
-* with the reason for it.
-*/
-function renderHeaderNav(_options) {
-	return `<nav class="prb-header-nav" aria-label="Workspace"><a class="prb-feedback" href="${escapeHtml$1(feedbackUrl())}" target="_blank" rel="noopener noreferrer"><span class="prb-feedback-icon" aria-hidden="true">${iconSvg("feedback")}</span><span class="prb-feedback-label">Send feedback</span></a></nav>`;
-}
 //#endregion
 //#region ../../packages/review-harness/src/domain/utils/file_path.ts
 /**
@@ -6088,7 +6142,7 @@ function renderWorkspace(view) {
 	*/
 	const freshness = result ? view.freshness : void 0;
 	return `<!doctype html>
-<html lang="en"${themeAttribute(view.theme)}>
+<html lang="en"${themeAttribute(view.theme)}${sidebarAttributes(view.sidebar)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -6681,6 +6735,7 @@ function renderRailFoot(view) {
 	return `<div class="prb-rail-group prb-rail-foot">
     ${view.token ? renderWorkspaceNav({ unread: view.unreadNotifications ?? 0 }) : ""}
     ${renderSendFeedback()}
+    ${view.token ? renderSidebarToggle(view.sidebar) : ""}
   </div>`;
 }
 /**
@@ -9676,7 +9731,7 @@ function renderReviewsPage(view) {
 	const storedReviews = view.groups.reduce((total, group) => total + group.history.length, 0);
 	const renderSectionAt = (section) => renderSection(section, sections[section], (groups) => renderCards(groups, "home", (group) => collectedHome.has(group)), sectionOpen(section, view.sectionsCollapsed), sections[section].filter((group) => !inProgressOnly.has(group.latest.id) && !collectedHome.has(group)).length);
 	return `<!doctype html>
-<html lang="en"${themeAttribute(view.theme)}>
+<html lang="en"${themeAttribute(view.theme)}${sidebarAttributes(view.sidebar)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -9699,12 +9754,16 @@ ${renderContextBar("Reviews", ordered.length === 0 ? "" : renderControls$1(queue
 <div class="prb-shell">
 ${renderWorkspaceRail({
 		current: "reviews",
-		unread: view.unreadNotifications ?? 0
+		unread: view.unreadNotifications ?? 0,
+		sidebar: view.sidebar
 	})}
 <main class="prb-index">
   ${ordered.length === 0 ? "" : `<div class="prb-index-summary">
     <p class="prb-index-count" data-count>${escapeHtml$1(countLabel(storedReviews, ordered.length))}</p>
-    <button type="button" class="prb-index-select" data-select-mode aria-pressed="false">Select</button>
+    <div class="prb-index-summary-actions">
+      <button type="button" class="prb-index-select" data-collection-new><span aria-hidden="true">+</span> New collection</button>
+      <button type="button" class="prb-index-select" data-select-mode aria-pressed="false">Select</button>
+    </div>
   </div>`}
   ${renderInProgress$1(operations, ordered, positions, view.prStates, now, sectionOpen("in-progress", view.sectionsCollapsed), memberOf)}
   ${renderSectionAt("pinned")}
@@ -9713,6 +9772,7 @@ ${renderWorkspaceRail({
   ${ordered.length === 0 ? `<p class="prb-index-empty">No reviews stored yet. Ask Claude Code to review a change and one will appear here.</p>` : `<p class="prb-index-none" data-no-matches hidden>Nothing matches that.</p>`}
   ${ordered.length === 0 ? "" : renderBulkBar()}
   ${ordered.length === 0 ? "" : renderCollectionsDialog()}
+  ${ordered.length === 0 ? "" : renderNewCollectionDialog()}
   <p class="prb-index-note">Reviews are kept for thirty days, on this machine only. Pinned, Done and queued reviews, and reviews in a collection, are kept for ninety days instead, counting from the last time you opened or marked them. Every review has a fixed address, so a link to one is worth keeping and still opens after a restart. Stopping the server deletes nothing; it only stops anything being served until the next review starts it again.</p>
 </main>
 </div>
@@ -9849,14 +9909,9 @@ function collectionsArea(context, collapsed) {
 			completed: renderCards(completed, { collection: collection.id })
 		});
 	});
-	return `<section class="prb-collections" data-collections-area aria-label="Collections">
+	return `<section class="prb-collections" data-collections-area aria-label="Collections"${collections.length === 0 ? " hidden" : ""}>
     <script type="application/json" data-collections-registry>${registry}<\/script>
-    <div class="prb-collections-head">
-      <h2 class="prb-collections-title">Collections</h2>
-      ${collections.length === 0 ? "<p class=\"prb-collections-hint\">Group reviews that belong together, such as one refactor or one release.</p>" : ""}
-    </div>
     ${blocks.join("\n    ")}
-    ${renderNewCollection()}
   </section>`;
 }
 function renderCollection(collection, counts, omitted, collapsed, cards) {
@@ -9918,17 +9973,25 @@ function renderCollectionMenu(collection) {
         </div>
       </div>`;
 }
-/** New collection, which opens a name field in place. */
-function renderNewCollection() {
-	return `<div class="prb-collection-new">
-      <button type="button" class="prb-index-more prb-collection-new-button" data-collection-new><span aria-hidden="true">+</span> New collection</button>
-      <form class="prb-collection-new-form" data-collection-new-form hidden>
-        <input type="text" class="prb-collection-name" data-collection-name-input placeholder="Collection name" aria-label="New collection name" maxlength="60" autocomplete="off" spellcheck="false">
-        <button type="submit" class="prb-bulk-button">Create</button>
-        <button type="button" class="prb-bulk-button" data-collection-cancel>Cancel</button>
-        <p class="prb-collection-error" data-collection-error hidden role="alert"></p>
-      </form>
-    </div>`;
+/**
+* New collection, opened from beside Select. The description lives here, where somebody is deciding
+* what to make, rather than on the list for good.
+*/
+function renderNewCollectionDialog() {
+	return `<dialog class="prb-dialog prb-collection-new-dialog" data-collection-new-dialog aria-labelledby="prb-collection-new-title">
+  <form method="dialog" data-collection-new-form>
+    <h2 id="prb-collection-new-title">New collection</h2>
+    <p>Group reviews that belong together, such as one refactor or one release.</p>
+    <label class="prb-field"><span>Collection name</span>
+      <input type="text" data-collection-name-input placeholder="e.g. Refactor" maxlength="60" autocomplete="off" spellcheck="false" required>
+    </label>
+    <p class="prb-collection-error" data-collection-error hidden role="alert"></p>
+    <div class="prb-dialog-actions">
+      <button type="button" data-collection-cancel>Cancel</button>
+      <button type="submit" class="prb-dialog-primary" data-collection-create disabled>Create</button>
+    </div>
+  </form>
+</dialog>`;
 }
 /**
 * Manage collections, for one card or for a selection. Built once; the browser fills the list from
@@ -10610,7 +10673,7 @@ function readPreferences() {
 	try {
 		const parsed = JSON.parse(readFileSync(PREFERENCES_PATH, "utf8"));
 		if (!parsed || typeof parsed !== "object") return {};
-		const { askWidth, dismissedNotices, promptsCollapsed, changeMapRegister, theme, reviewSections, runsSections, reviewLayout } = parsed;
+		const { askWidth, dismissedNotices, promptsCollapsed, changeMapRegister, theme, reviewSections, runsSections, reviewLayout, sidebar } = parsed;
 		return {
 			...typeof askWidth === "number" && Number.isFinite(askWidth) ? { askWidth: clampAskWidth(askWidth) } : {},
 			...Array.isArray(dismissedNotices) ? { dismissedNotices: dismissedNotices.filter((key) => typeof key === "string" && NOTICE_KEY.test(key)) } : {},
@@ -10619,7 +10682,8 @@ function readPreferences() {
 			...isTheme(theme) ? { theme } : {},
 			...readReviewSections(reviewSections),
 			...readRunsSections(runsSections),
-			...readLayout(reviewLayout)
+			...readLayout(reviewLayout),
+			...readSidebar(sidebar)
 		};
 	} catch {
 		return {};
@@ -10652,6 +10716,35 @@ function writeReviewLayout(patch) {
 		writeFileSync(PREFERENCES_PATH, `${JSON.stringify(next, null, 2)}\n`);
 	} catch {}
 }
+/** Only the two booleans, and only when one is set; an empty object is no preference. */
+function readSidebar(value) {
+	if (!value || typeof value !== "object") return {};
+	const { compact, hidden } = value;
+	const sidebar = {
+		...typeof compact === "boolean" ? { compact } : {},
+		...typeof hidden === "boolean" ? { hidden } : {}
+	};
+	return Object.keys(sidebar).length === 0 ? {} : { sidebar };
+}
+/** The sidebar fields a request may set: a boolean or nothing, each. */
+function readSidebarPatch(value) {
+	const { sidebar } = readSidebar(value);
+	return sidebar ?? null;
+}
+/** Remember a change to the sidebar, merged into what is stored. Read-modify-write, like every other field here. */
+function writeSidebar(patch) {
+	try {
+		mkdirSync(STORE_ROOT, { recursive: true });
+		const current = readPreferences();
+		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
+			...current,
+			sidebar: {
+				...current.sidebar,
+				...patch
+			}
+		}, null, 2)}\n`);
+	} catch {}
+}
 /** Remember whether "Your prompts" is folded. Read-modify-write, like every other field here. */
 function writePromptsCollapsed(collapsed) {
 	try {
@@ -10872,7 +10965,7 @@ function renderSettingsPage(view) {
 	const { settings } = view;
 	const address = `http://localhost:${settings.port.configured}`;
 	return `<!doctype html>
-<html lang="en"${themeAttribute(settings.theme)}>
+<html lang="en"${themeAttribute(settings.theme)}${sidebarAttributes(view.sidebar)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -10891,7 +10984,8 @@ ${renderContextBar("Settings", renderBack())}
 <div class="prb-shell">
 ${renderWorkspaceRail({
 		current: "settings",
-		unread: view.unreadNotifications ?? 0
+		unread: view.unreadNotifications ?? 0,
+		sidebar: view.sidebar
 	})}
 <main class="prb-index prb-settings">
   <div class="prb-page-panel">
@@ -11392,8 +11486,10 @@ var TOPICS = [
       <li><code>prreviewbuddy review --pr &lt;url&gt;</code> for a pull request</li>
       <li><code>prreviewbuddy open</code> to list the reviews you have and open this workspace</li>
     </ul>
-    <p>Or press <kbd class="prb-kbd">⌘P</kbd> (<kbd class="prb-kbd">Ctrl+P</kbd> elsewhere) to add a
-    review from here. The panel follows the reviews it started while it stays open, and closing it
+    <p>Or click <b>New review</b> at the top of the sidebar, or press <kbd class="prb-kbd">⌘P</kbd>
+    (<kbd class="prb-kbd">Ctrl+P</kbd> on other platforms), to start a review from anywhere in the
+    workspace. With the sidebar collapsed, New review is the <b>+</b> at its top, and with the
+    sidebar hidden, the <b>+</b> in the top toolbar. The panel follows the reviews it started while it stays open, and closing it
     stops nothing. <code>prreviewbuddy help</code> lists every command, and
     <code>prreviewbuddy review --help</code> every option a review takes.</p>
     <p>There is no account and no API key. PR Review Buddy uses the coding agent installed on your
@@ -11491,18 +11587,13 @@ feature:                   E</pre>
 	{
 		id: "collections",
 		title: "Collections",
-		body: `<p>A collection groups reviews that belong together, such as the pull requests of one
-    refactor or one release. It says what belongs together; whether each review still needs your
-    attention is still Active, Waiting or Done, so adding or removing a review never changes its
-    state.</p>
-    <p>Make one with New collection on the Reviews page, or from Manage collections in a card's menu.
-    To add several reviews at once, choose Select, tick them, and choose Manage collections. A review
-    can be in more than one collection. Ticking a collection adds every selected review to it and
-    leaves their other collections alone.</p>
-    <p>Each collection lists its reviews that are not Done. Marking one Done moves it under Show
-    completed, in every collection it is in, and Move to active brings it back. A review in a
-    collection is listed there rather than under Active, Waiting or Done. A pinned review stays under
-    Pinned. The Collections filter shows chosen collections, or Uncollected for reviews in none.</p>
+		body: `<p>Collections help you organise related reviews, such as a refactor, release or feature.</p>
+    <p>Create a collection from the Reviews page, then add reviews individually or select several and
+    use <b>Manage collections</b>. A review can belong to multiple collections.</p>
+    <p>Collections show reviews that still need attention. When you mark a review <b>Done</b>, it moves
+    under <b>Show completed</b> within its collection, without losing its membership.</p>
+    <p>You can collapse collections to stay focused, and expand them when you're ready to continue.
+    Removing a review from all collections returns it to the normal review sections.</p>
     <p>Reviews in a collection are kept for ninety days, like pinned and Done reviews, counted from
     when you last opened or marked them. Adding one to a collection doesn't restart that count.
     Deleting a collection keeps its reviews.</p>`
@@ -11592,7 +11683,7 @@ function renderContents() {
 }
 function renderHelpPage(view) {
 	return `<!doctype html>
-<html lang="en"${themeAttribute(view.theme)}>
+<html lang="en"${themeAttribute(view.theme)}${sidebarAttributes(view.sidebar)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -11611,7 +11702,8 @@ ${renderContextBar("Help", "")}
 <div class="prb-shell">
 ${renderWorkspaceRail({
 		current: "help",
-		unread: view.unreadNotifications ?? 0
+		unread: view.unreadNotifications ?? 0,
+		sidebar: view.sidebar
 	})}
 <main class="prb-index prb-help">
   <div class="prb-page-panel">
@@ -11633,7 +11725,7 @@ ${renderWorkspaceRail({
 function renderNotificationsPage(view) {
 	const unread = view.notifications.filter((notification) => notification.readAt === void 0).length;
 	return `<!doctype html>
-<html lang="en"${themeAttribute(view.theme)}>
+<html lang="en"${themeAttribute(view.theme)}${sidebarAttributes(view.sidebar)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -11652,7 +11744,8 @@ ${renderContextBar("Activity", view.notifications.length === 0 ? "" : renderRead
 <div class="prb-shell">
 ${renderWorkspaceRail({
 		current: "notifications",
-		unread: view.unreadNotifications ?? 0
+		unread: view.unreadNotifications ?? 0,
+		sidebar: view.sidebar
 	})}
 <main class="prb-index prb-notifications-page">
   ${view.notifications.length === 0 ? `<p class="prb-index-empty" data-notifications-empty>Nothing yet. When a review finishes or fails, or a branch or pull request moves, it appears here.</p>` : `<ol class="prb-notifications" data-notifications>
@@ -13037,7 +13130,7 @@ function renderRunsPage(view) {
 	const nothing = inProgress.length === 0 && recent.length === 0;
 	const notResponding = recent.some((entry) => entry.status === "not-responding");
 	return `<!doctype html>
-<html lang="en"${themeAttribute(view.theme)}>
+<html lang="en"${themeAttribute(view.theme)}${sidebarAttributes(view.sidebar)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -13056,7 +13149,8 @@ ${renderContextBar("Runs", nothing ? "" : renderControls(notResponding))}
 <div class="prb-shell">
 ${renderWorkspaceRail({
 		current: "runs",
-		unread: view.unreadNotifications ?? 0
+		unread: view.unreadNotifications ?? 0,
+		sidebar: view.sidebar
 	})}
 <main class="prb-index prb-runs">
   <p class="prb-runs-lede">Monitor review executions and recent activity.</p>
@@ -14992,6 +15086,7 @@ async function handle(req, res, options) {
 	if (url.pathname === "/notifications/live") return handleLiveNotifications(req, res, url.searchParams, options.indexToken);
 	if (url.pathname === "/pin") return handlePin(req, res, options.indexToken);
 	if (url.pathname === "/sections") return handleSections(req, res, options.indexToken);
+	if (url.pathname === "/sidebar") return handleSidebar(req, res, options.indexToken);
 	if (url.pathname === "/queue") return handleQueue(req, res, options.indexToken);
 	if (url.pathname === "/collections") return handleCollections(req, res, options.indexToken);
 	if (url.pathname === "/collections/members") return handleCollectionMembers(req, res, options.indexToken);
@@ -15141,6 +15236,7 @@ function indexView() {
 		unreadNotifications: unreadCount$1(),
 		driving: isDriving(),
 		theme: readPreferences().theme,
+		sidebar: readPreferences().sidebar,
 		sectionsCollapsed: readPreferences().reviewSections
 	};
 }
@@ -15231,7 +15327,8 @@ async function handleSettings(req, res, params, indexToken) {
 	sendHtml(res, 200, renderSettingsPage({
 		settings: readSettings(),
 		unreadNotifications: unreadCount$1(),
-		driving: isDriving()
+		driving: isDriving(),
+		sidebar: readPreferences().sidebar
 	}), indexCookieHeader(indexToken));
 }
 /**
@@ -15243,7 +15340,8 @@ function handleHelp(req, res, params, indexToken) {
 	sendHtml(res, 200, renderHelpPage({
 		unreadNotifications: unreadCount$1(),
 		driving: isDriving(),
-		theme: readPreferences().theme
+		theme: readPreferences().theme,
+		sidebar: readPreferences().sidebar
 	}), indexCookieHeader(indexToken));
 }
 /**
@@ -15358,7 +15456,8 @@ async function handleNotifications(req, res, params, indexToken) {
 		gone: new Set(notifications.filter((entry) => !loadWorkspace(entry.reviewId)).map((entry) => entry.id)),
 		unreadNotifications: unreadCount$1(),
 		driving: isDriving(),
-		theme: readPreferences().theme
+		theme: readPreferences().theme,
+		sidebar: readPreferences().sidebar
 	}), indexCookieHeader(indexToken));
 }
 /**
@@ -15379,6 +15478,7 @@ function handleRuns(req, res, params, indexToken) {
 		unreadNotifications: unreadCount$1(),
 		driving: isDriving(),
 		theme: preferences.theme,
+		sidebar: preferences.sidebar,
 		sectionsCollapsed: preferences.runsSections
 	}), indexCookieHeader(indexToken));
 }
@@ -15792,6 +15892,18 @@ async function handleRunsSections(req, res, indexToken) {
 	writeRunsSectionCollapsed(body.section, body.collapsed);
 	send(res, 200, { ok: true });
 }
+/**
+* The sidebar, from a workspace page that is not a review, with the same credential as `/sections`.
+* A review page sends the same field to its own `/prefs`, since it may hold no index cookie.
+*/
+async function handleSidebar(req, res, indexToken) {
+	if (req.method !== "POST") return send(res, 405, { error: "Use POST." });
+	if (!indexToken || !secretMatches(indexToken, indexCookie(req))) return send(res, 403, { error: "Not authorised." });
+	const sidebar = readSidebarPatch((await readJson(req))?.sidebar);
+	if (!sidebar) return send(res, 400, { error: "Not a sidebar setting." });
+	writeSidebar(sidebar);
+	send(res, 200, { ok: true });
+}
 async function handleSections(req, res, indexToken) {
 	if (req.method !== "POST") return send(res, 405, { error: "Use POST." });
 	if (!indexToken || !secretMatches(indexToken, indexCookie(req))) return send(res, 403, { error: "Not authorised." });
@@ -16198,6 +16310,7 @@ async function sendWorkspacePage(res, workspace) {
 		repoPath: workspace.repoPath,
 		askWidth: readPreferences().askWidth,
 		theme: readPreferences().theme,
+		sidebar: readPreferences().sidebar,
 		dismissedNotices: readPreferences().dismissedNotices,
 		promptsCollapsed: readPreferences().promptsCollapsed === true,
 		reviewLayout: readPreferences().reviewLayout ?? {},
@@ -17001,6 +17114,8 @@ async function handlePrefs(req, res) {
 	if (typeof askWidth === "number") writeAskWidth(askWidth);
 	else if (askWidth === null) clearAskWidth();
 	if (typeof body?.promptsCollapsed === "boolean") writePromptsCollapsed(body.promptsCollapsed);
+	const sidebar = readSidebarPatch(body?.sidebar);
+	if (sidebar) writeSidebar(sidebar);
 	const layout = readReviewLayoutPatch(body?.reviewLayout);
 	if (layout) writeReviewLayout(layout);
 	if (body?.changeMapRegister === "simple" || body?.changeMapRegister === "technical") writeChangeMapRegister(body.changeMapRegister);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prreviewbuddy",
-  "version": "0.82.1",
+  "version": "0.83.3",
   "description": "Understand code changes before you approve them. Local branch and pull request reviews with Claude Code or Codex.",
   "license": "UNLICENSED",
   "homepage": "https://prreviewbuddy.com",

package/static/{chunk-CxFmUaBj.js → chunk-B2vcrGTe.js}

@@ -1 +1 @@
-import{dt as e}from"./chunk-CQNkZ9TG.js";var t=`button:not([disabled])`;function n(e){return e.querySelector(`[data-menu-panel]`)}function r(e){return e.querySelector(`[data-menu-trigger]`)}function i(e){let t=n(e);return t?[...t.querySelectorAll(`input:not([disabled]), textarea:not([disabled]), button:not([disabled])`)].filter(e=>!e.closest(`[hidden]`)):[]}function a(e){let r=n(e);return r?[...r.querySelectorAll(t)].filter(e=>!e.closest(`[hidden]`)):[]}var o=`data-menu-placement`;function s(e){let t=e.ownerDocument.defaultView,n=0,r=t?.innerHeight??1/0;for(let i=e.parentElement;i&&t;i=i.parentElement){let e=t.getComputedStyle(i).overflowY;if(e===`visible`||e===``)continue;let a=i.getBoundingClientRect();n=Math.max(n,a.top),r=Math.min(r,a.bottom)}return{top:n,bottom:r}}var c=8,l=120;function u(e){let t=n(e),i=r(e);if(!t||!i)return;t.removeAttribute(o),t.style.removeProperty(`max-height`),t.style.removeProperty(`overflow-y`),t.style.removeProperty(`box-sizing`),t.scrollTop=0;let a=t.getBoundingClientRect(),u=i.getBoundingClientRect(),d=s(e),f=Math.max(0,a.top-u.bottom),p=d.bottom-u.bottom-f-c,m=u.top-d.top-f-c,h=a.height,g=h<=p?`below`:h<=m||m>p?`above`:`below`;g===`above`&&t.setAttribute(o,`above`);let _=g===`above`?m:p;h<=_||(t.style.boxSizing=`border-box`,t.style.maxHeight=`${Math.max(Math.floor(_),l)}px`,t.style.overflowY=`auto`)}function d(e){return!e.metaKey&&!e.ctrlKey&&!e.altKey&&!e.shiftKey?e.key===`j`?`ArrowDown`:e.key===`k`?`ArrowUp`:e.key:e.key}function f(t,o={}){let s=null,c=null,l=o.within?`${o.within} [data-menu]`:`[data-menu]`,f=e=>e&&(!o.within||e.closest(o.within))?e:null,p=(e,t={})=>{let i=n(e);if(!i||i.hidden)return;i.hidden=!0;let a=r(e);a?.setAttribute(`aria-expanded`,`false`);let l=s===e?c:null;s===e&&(s=null,c=null),o.onClose?.(e),t.returnFocus&&(l??(()=>a?.focus()))()},m=()=>{for(let e of t.querySelectorAll(l))p(e)},h=(e,t,i=null)=>{let l=n(e);l&&(m(),c=i,o.onOpen?.(e),l.hidden=!1,u(e),r(e)?.setAttribute(`aria-expanded`,`true`),s=e,t&&a(e)[0]?.focus())};return t.addEventListener(`click`,e=>{let t=e.target,r=f(t?.closest?.(`[data-menu]`)??null);if(!r){m();return}if(t?.closest(`[data-menu-trigger]`)){let t=e.detail===0;n(r)?.hidden?h(r,t):p(r);return}s&&s!==r&&p(s)}),t.addEventListener(`keydown`,t=>{if(s&&!s.isConnected&&(s=null),!s)return;let n=s;if(t.key===`Escape`){t.preventDefault(),p(n,{returnFocus:!0});return}if(!n.contains(t.target))return;if(t.key===`Tab`){if(n.hasAttribute(`data-menu-form`)){let e=i(n),r=e.indexOf(t.target);if(e.length>0){t.preventDefault(),e[(r+(t.shiftKey?-1:1)+e.length)%e.length].focus();return}}p(n);return}if(e(t.target))return;let r=a(n);if(r.length===0)return;let o=r.indexOf(t.target),c=d(t),l=c===`ArrowDown`?r[(o+1)%r.length]:c===`ArrowUp`?r[(o-1+r.length)%r.length]:c===`Home`?r[0]:c===`End`?r[r.length-1]:null;l&&(t.preventDefault(),l.focus(),l.scrollIntoView({block:`nearest`}))}),{open:(e,t={})=>h(e,t.focusFirst===!0,t.returnFocus??null),close:p,closeAll:m}}function p(e){return e<3e4?1e3:e<12e4?3e3:e<6e5?8e3:15e3}export{f as n,p as t};
+import{dt as e}from"./chunk-DwdwtB9r.js";var t=`button:not([disabled])`;function n(e){return e.querySelector(`[data-menu-panel]`)}function r(e){return e.querySelector(`[data-menu-trigger]`)}function i(e){let t=n(e);return t?[...t.querySelectorAll(`input:not([disabled]), textarea:not([disabled]), button:not([disabled])`)].filter(e=>!e.closest(`[hidden]`)):[]}function a(e){let r=n(e);return r?[...r.querySelectorAll(t)].filter(e=>!e.closest(`[hidden]`)):[]}var o=`data-menu-placement`;function s(e){let t=e.ownerDocument.defaultView,n=0,r=t?.innerHeight??1/0;for(let i=e.parentElement;i&&t;i=i.parentElement){let e=t.getComputedStyle(i).overflowY;if(e===`visible`||e===``)continue;let a=i.getBoundingClientRect();n=Math.max(n,a.top),r=Math.min(r,a.bottom)}return{top:n,bottom:r}}var c=8,l=120;function u(e){let t=n(e),i=r(e);if(!t||!i)return;t.removeAttribute(o),t.style.removeProperty(`max-height`),t.style.removeProperty(`overflow-y`),t.style.removeProperty(`box-sizing`),t.scrollTop=0;let a=t.getBoundingClientRect(),u=i.getBoundingClientRect(),d=s(e),f=Math.max(0,a.top-u.bottom),p=d.bottom-u.bottom-f-c,m=u.top-d.top-f-c,h=a.height,g=h<=p?`below`:h<=m||m>p?`above`:`below`;g===`above`&&t.setAttribute(o,`above`);let _=g===`above`?m:p;h<=_||(t.style.boxSizing=`border-box`,t.style.maxHeight=`${Math.max(Math.floor(_),l)}px`,t.style.overflowY=`auto`)}function d(e){return!e.metaKey&&!e.ctrlKey&&!e.altKey&&!e.shiftKey?e.key===`j`?`ArrowDown`:e.key===`k`?`ArrowUp`:e.key:e.key}function f(t,o={}){let s=null,c=null,l=o.within?`${o.within} [data-menu]`:`[data-menu]`,f=e=>e&&(!o.within||e.closest(o.within))?e:null,p=(e,t={})=>{let i=n(e);if(!i||i.hidden)return;i.hidden=!0;let a=r(e);a?.setAttribute(`aria-expanded`,`false`);let l=s===e?c:null;s===e&&(s=null,c=null),o.onClose?.(e),t.returnFocus&&(l??(()=>a?.focus()))()},m=()=>{for(let e of t.querySelectorAll(l))p(e)},h=(e,t,i=null)=>{let l=n(e);l&&(m(),c=i,o.onOpen?.(e),l.hidden=!1,u(e),r(e)?.setAttribute(`aria-expanded`,`true`),s=e,t&&a(e)[0]?.focus())};return t.addEventListener(`click`,e=>{let t=e.target,r=f(t?.closest?.(`[data-menu]`)??null);if(!r){m();return}if(t?.closest(`[data-menu-trigger]`)){let t=e.detail===0;n(r)?.hidden?h(r,t):p(r);return}s&&s!==r&&p(s)}),t.addEventListener(`keydown`,t=>{if(s&&!s.isConnected&&(s=null),!s)return;let n=s;if(t.key===`Escape`){t.preventDefault(),p(n,{returnFocus:!0});return}if(!n.contains(t.target))return;if(t.key===`Tab`){if(n.hasAttribute(`data-menu-form`)){let e=i(n),r=e.indexOf(t.target);if(e.length>0){t.preventDefault(),e[(r+(t.shiftKey?-1:1)+e.length)%e.length].focus();return}}p(n);return}if(e(t.target))return;let r=a(n);if(r.length===0)return;let o=r.indexOf(t.target),c=d(t),l=c===`ArrowDown`?r[(o+1)%r.length]:c===`ArrowUp`?r[(o-1+r.length)%r.length]:c===`Home`?r[0]:c===`End`?r[r.length-1]:null;l&&(t.preventDefault(),l.focus(),l.scrollIntoView({block:`nearest`}))}),{open:(e,t={})=>h(e,t.focusFirst===!0,t.returnFocus??null),close:p,closeAll:m}}function p(e){return e<3e4?1e3:e<12e4?3e3:e<6e5?8e3:15e3}export{f as n,p as t};

package/static/{chunk-CWRKWlhO.js → chunk-BL6xJoZm.js}

@@ -1 +1 @@
-import{F as e}from"./chunk-CQNkZ9TG.js";function t(e=document){for(let t of e.querySelectorAll(`.prb-index-status`))t.addEventListener(`click`,e=>{let n=t.querySelector(`select`);if(!(!n||e.target instanceof Node&&n.contains(e.target))){e.preventDefault(),n.focus();try{n.showPicker()}catch{}}})}function n(t,n){r(t,t=>e(t.dataset.section)?t.dataset.section:void 0,n)}function r(e,t,n){let r=Promise.resolve();for(let i of e){let e=t(i);if(e===void 0)continue;let a=i.open;i.addEventListener(`toggle`,()=>{if(i.open===a)return;a=i.open;let t=!i.open;r=r.then(()=>n(e,t)).catch(()=>{})})}}export{n,t as r,r as t};
+import{F as e}from"./chunk-DwdwtB9r.js";function t(e=document){for(let t of e.querySelectorAll(`.prb-index-status`))t.addEventListener(`click`,e=>{let n=t.querySelector(`select`);if(!(!n||e.target instanceof Node&&n.contains(e.target))){e.preventDefault(),n.focus();try{n.showPicker()}catch{}}})}function n(t,n){r(t,t=>e(t.dataset.section)?t.dataset.section:void 0,n)}function r(e,t,n){let r=Promise.resolve();for(let i of e){let e=t(i);if(e===void 0)continue;let a=i.open;i.addEventListener(`toggle`,()=>{if(i.open===a)return;a=i.open;let t=!i.open;r=r.then(()=>n(e,t)).catch(()=>{})})}}export{n,t as r,r as t};

package/static/chunk-DWTt0jUK.js

@@ -0,0 +1 @@
+async function e(e,t){try{await navigator.clipboard.writeText(t);let n=e.textContent;e.textContent=`Copied`,window.setTimeout(()=>{e.textContent=n},1200)}catch{let n=document.createElement(`input`);n.type=`text`,n.readOnly=!0,n.value=t,n.className=`prb-menu-id`,n.setAttribute(`data-id-field`,``),e.after(n),n.select()}}function t(e,t){let n=t.toLowerCase().split(/\s+/).filter(Boolean);return n.length===0?[...e]:e.filter(e=>{let t=`${e.name}\n${e.prompt}`.toLowerCase();return n.every(e=>t.includes(e))})}function n(e){return e<0||e>=10?null:e===9?`0`:String(e+1)}function r(e){return/^[0-9]$/.test(e)?e===`0`?9:Number(e)-1:null}function i(e){return{title:`Choose a prompt`,mount(i,a){let o=i.ownerDocument,s=e.prompts(),c=o.createElement(`input`);c.type=`search`,c.className=`prb-palette-input`,c.placeholder=`Filter prompts`,c.setAttribute(`aria-label`,`Filter prompts`),c.spellcheck=!1;let l=o.createElement(`p`);l.className=`prb-palette-summary`,l.setAttribute(`aria-live`,`polite`);let u=o.createElement(`div`);u.className=`prb-palette-results`,u.setAttribute(`role`,`group`),u.setAttribute(`aria-label`,`Prompts`);let d=[],f=()=>[...u.querySelectorAll(`[data-prompt-choice]`)],p=()=>{let e=o.activeElement===c;for(let[t,n]of f().entries())n.toggleAttribute(`data-enter-target`,e&&t===0)},m=()=>{d=t(s,c.value),u.replaceChildren(...d.map((e,t)=>{let r=o.createElement(`button`);r.type=`button`,r.className=`prb-palette-result prb-palette-prompt`,r.dataset.promptChoice=e.id;let i=o.createElement(`kbd`);i.className=`prb-kbd prb-palette-num`;let a=n(t);a?i.textContent=a:i.setAttribute(`aria-hidden`,`true`);let s=o.createElement(`span`);s.className=`prb-palette-prompt-text`;let c=o.createElement(`span`);c.className=`prb-palette-result-title`,c.textContent=e.name;let l=o.createElement(`span`);return l.className=`prb-palette-prompt-said`,l.textContent=e.prompt,s.append(c,l),r.append(i,s),r})),l.textContent=s.length===0?`No saved prompts yet. Add one under Your prompts in Buddy AI.`:d.length===0?`No prompts match.`:d.length===s.length?`${s.length} ${s.length===1?`prompt`:`prompts`}`:`${d.length} of ${s.length} prompts`,p()},h=t=>{!t||!d.some(e=>e.id===t)||(a(),e.choose(t))},g=e=>{c.value=e===`Backspace`?c.value.slice(0,-1):c.value+e,c.focus(),c.dispatchEvent(new Event(`input`,{bubbles:!0}))};c.addEventListener(`input`,m),c.addEventListener(`focus`,p),c.addEventListener(`blur`,p),u.addEventListener(`click`,e=>{h(e.target.closest(`[data-prompt-choice]`)?.dataset.promptChoice)}),i.addEventListener(`keydown`,e=>{if(e.isComposing||e.metaKey||e.ctrlKey||e.altKey)return;let t=f(),n=e.target===c,i=t.indexOf(e.target),a=r(e.key);if(a!==null)return e.preventDefault(),h(t[a]?.dataset.promptChoice);if(e.key===`Enter`)return e.preventDefault(),h((n?t[0]:t[i])?.dataset.promptChoice);let o=e.key===`ArrowDown`||!n&&e.key===`j`,s=e.key===`ArrowUp`||!n&&e.key===`k`;if(o||s){let n=o?i<0?t[0]:t[i+1]:i<=0?c:t[i-1];e.preventDefault(),n?.focus();return}!n&&(e.key.length===1||e.key===`Backspace`)&&(e.preventDefault(),g(e.key))}),i.append(c,l,u),m(),c.focus()}}}var a=`Fetch selected`;function o(e){return`${e} ${e===1?`repository`:`repositories`}`}function s(e){return`Fetching ${o(e)}…`}function c(e){return e.fetched===0?`Could not fetch ${o(e.repositories)}`:e.fetched===e.repositories?`Fetched ${o(e.repositories)}`:`Fetched ${e.fetched} of ${o(e.repositories)}`}function l(e,t){let n=new Set,r=new Set;for(let i of e)for(let e of i.lineage){let i=t[e];i!==void 0&&(n.add(e),r.add(i))}return{ids:[...n],repositories:r.size}}function u(e,t){return l(t,e).repositories>0}export{s as a,n as c,c as i,e as l,u as n,i as o,l as r,r as s,a as t};