From a7aa630707719ac6e6036c8ca7724fa0dc6a95ea Mon Sep 17 00:00:00 2001
From: "mux-bot[bot]" <264182336+mux-bot[bot]@users.noreply.github.com>
Date: Mon, 15 Jun 2026 10:10:54 +0000
Subject: [PATCH 1/2] refactor: extract DisclosureChevron helper in
 MemoryBrowser

Dedupe the identical 'isOpen ? ChevronDown : ChevronRight' disclosure
toggle that ScopeSection and DirRow each rendered inline. Both now use a
local DisclosureChevron helper; rendered className output is unchanged
(DirRow keeps its extra shrink-0).
---
 src/browser/features/Memory/MemoryBrowser.tsx | 25 +++++++++++--------
 1 file changed, 15 insertions(+), 10 deletions(-)

diff --git a/src/browser/features/Memory/MemoryBrowser.tsx b/src/browser/features/Memory/MemoryBrowser.tsx
index e85aa56bf3..dba13f9bdc 100644
--- a/src/browser/features/Memory/MemoryBrowser.tsx
+++ b/src/browser/features/Memory/MemoryBrowser.tsx
@@ -319,6 +319,19 @@ function buildTree(files: MemoryFileInfo[]): TreeDir {
   return root;
 }
 
+/**
+ * Disclosure caret shared by the scope and directory rows: points down when
+ * expanded, right when collapsed. `className` is appended to the shared sizing
+ * classes so each row can add layout tweaks (e.g. `shrink-0`).
+ */
+function DisclosureChevron(props: { open: boolean; className?: string }) {
+  const Icon = props.open ? ChevronDown : ChevronRight;
+  const className = props.className
+    ? `text-muted h-3 w-3 ${props.className}`
+    : "text-muted h-3 w-3";
+  return <Icon className={className} aria-hidden="true" />;
+}
+
 /** Collapsible scope section, mirroring the GoalBoardSections shell. */
 function ScopeSection(props: ScopeSectionProps) {
   const [isOpen, setIsOpen] = useState(true);
@@ -330,11 +343,7 @@ function ScopeSection(props: ScopeSectionProps) {
         aria-expanded={isOpen}
         onClick={() => setIsOpen((prev) => !prev)}
       >
-        {isOpen ? (
-          <ChevronDown className="text-muted h-3 w-3" aria-hidden="true" />
-        ) : (
-          <ChevronRight className="text-muted h-3 w-3" aria-hidden="true" />
-        )}
+        <DisclosureChevron open={isOpen} />
         <span className="text-foreground font-medium">{props.title}</span>
         <span className="text-muted lowercase">({props.files.length})</span>
       </button>
@@ -393,11 +402,7 @@ function DirRow(props: TreeNodesProps) {
         aria-expanded={isOpen}
         onClick={() => setIsOpen((prev) => !prev)}
       >
-        {isOpen ? (
-          <ChevronDown className="text-muted h-3 w-3 shrink-0" aria-hidden="true" />
-        ) : (
-          <ChevronRight className="text-muted h-3 w-3 shrink-0" aria-hidden="true" />
-        )}
+        <DisclosureChevron open={isOpen} className="shrink-0" />
         {isOpen ? (
           <FolderOpen className="text-muted h-3.5 w-3.5 shrink-0" aria-hidden="true" />
         ) : (

From 5284d7f5c022fb4685b8285c9b0a11d69a308a9a Mon Sep 17 00:00:00 2001
From: "mux-bot[bot]" <264182336+mux-bot[bot]@users.noreply.github.com>
Date: Mon, 15 Jun 2026 10:21:11 +0000
Subject: [PATCH 2/2] ci: re-trigger checks (clear unrelated flaky
 workflow-lease test)

The prior run failed only on
'built-in deep-review-workflow > auto-fix checkpoint retry preserves
completed patch after HEAD advances' with 'Workflow run is already
active' (a test-isolation lease collision). It passes locally on this
branch and is unrelated to the MemoryBrowser-only cleanup in this PR.