admin: prettier formatting for the toolbar-era files, missed before ci ever ran them

admin/src/features/activity/ActivityDetailPage.test.tsx

@@ -332,7 +332,11 @@ test("Related carries its four links and no control, blocked query or not", asyn
 
 	await screen.findByRole("heading", { name: "example.com" });
 	await waitFor(() => expect(within(related()).getByText("Diagnostics around this query")).toBeTruthy());
-	expect(within(related()).getAllByRole("link").map((link) => link.textContent)).toEqual([
+	expect(
+		within(related())
+			.getAllByRole("link")
+			.map((link) => link.textContent),
+	).toEqual([
 		"Test this domain against current policy",
 		"All activity for this domain",
 		"All activity from this client",

admin/src/features/activity/ActivityDetailPage.tsx

@@ -68,13 +68,7 @@ export default function ActivityDetailPage() {
 			<ProvenanceDetail
 				provenance={detail}
 				persistedId={detail.id}
-				relatedActions={
+				relatedActions={<RelatedActions domain={domain} client={client} ts={time} origin={origin} />}
-					<RelatedActions
-						domain={domain}
-						client={client}
-						ts={time}
-						origin={origin} />
-				}
 			/>
 		</section>
 	);

admin/src/features/activity/ActivityFilters.test.tsx

@@ -76,9 +76,11 @@ function openTimeMenu() {
 
 test("the segments commit to the applied state on the click, with no button in between", () => {
 	const { patches, state } = renderFilters();
-	expect(
+	expect(screen.getAllByRole("radio").map((radio) => radio.closest("label")?.textContent)).toEqual([
-		screen.getAllByRole("radio").map((radio) => radio.closest("label")?.textContent),
+		"Any",
-	).toEqual(["Any", "Blocked", "Allowed"]);
+		"Blocked",
+		"Allowed",
+	]);
 
 	fireEvent.click(screen.getByRole("radio", { name: "Blocked" }));
 	expect(patches).toEqual([{ blocked: true }]);
@@ -325,12 +327,8 @@ test("a window moved from outside re-opens the custom row over the bounds it arr
 	// that range — a Custom window with nothing to read is the label lying.
 	// jsdom's value sanitizer spells the milliseconds out, so the seeded text is
 	// the prefix rather than the whole of what the input holds.
-	expect((screen.getByLabelText("Since") as HTMLInputElement).value).toContain(
+	expect((screen.getByLabelText("Since") as HTMLInputElement).value).toContain(unixToDatetimeLocal(since));
-		unixToDatetimeLocal(since),
+	expect((screen.getByLabelText("Until") as HTMLInputElement).value).toContain(unixToDatetimeLocal(since + 600));
-	);
-	expect((screen.getByLabelText("Until") as HTMLInputElement).value).toContain(
-		unixToDatetimeLocal(since + 600),
-	);
 
 	vi.restoreAllMocks();
 });

admin/src/features/activity/ActivityFilters.tsx

@@ -425,11 +425,7 @@ export default function ActivityFilters({ applied, onApply, onClear }: Props) {
 		}
 		// The window is half-open — `ts >= since` and `ts < until` — so two equal
 		// bounds are as empty as an inverted pair, and neither is worth applying.
-		if (
+		if (sinceValue.value !== undefined && untilValue.value !== undefined && untilValue.value <= sinceValue.value) {
-			sinceValue.value !== undefined &&
-			untilValue.value !== undefined &&
-			untilValue.value <= sinceValue.value
-		) {
 			setError({
 				field: "until",
 				message: "Until must be after Since, or the range selects nothing.",
@@ -612,12 +608,7 @@ export default function ActivityFilters({ applied, onApply, onClear }: Props) {
 					onClick={clear}
 					tabIndex={active ? undefined : -1}
 					aria-hidden={active ? undefined : true}
-					{...stylex.props(
+					{...stylex.props(shared.button, styles.hitTarget, shared.focusRing, !active && styles.clearHidden)}
-						shared.button,
-						styles.hitTarget,
-						shared.focusRing,
-						!active && styles.clearHidden,
-					)}
 				>
 					Clear
 				</button>

admin/src/features/activity/ActivityPage.tsx

@@ -148,10 +148,7 @@ export default function ActivityPage() {
 			>
 				<div {...stylex.props(styles.header)}>
 					<h1 {...stylex.props(styles.heading)}>Activity</h1>
-					<TabList
+					<TabList aria-label="Activity mode" className={() => stylex.props(styles.switch).className ?? ""}>
-						aria-label="Activity mode"
-						className={() => stylex.props(styles.switch).className ?? ""}
-					>
 						{MODES.map((option) => (
 							<Tab
 								key={option.mode}

admin/src/shell/AppShell.tsx

@@ -337,11 +337,7 @@ export default function AppShell() {
 			{/* The column itself is the disclosure: the trigger sits in the header
 			    and the panel opens below the notices, so any wrapper around only
 			    the two would have to cut across the column's own flex children. */}
-			<Disclosure
+			<Disclosure isExpanded={drawerOpen} onExpandedChange={setDrawerOpen} {...stylex.props(styles.column)}>
-				isExpanded={drawerOpen}
-				onExpandedChange={setDrawerOpen}
-				{...stylex.props(styles.column)}
-			>
 				<header {...stylex.props(styles.header)}>
 					<Button
 						slot="trigger"