From dd2090be5c0f58ac42debf72e96a5a090cd3d750 Mon Sep 17 00:00:00 2001 From: Devika Verma Date: Wed, 12 Aug 2026 14:03:29 -0700 Subject: [PATCH] Mode picker: drop the note machinery - Auto-Approve copy is two lines (A) Owner call after seeing it rendered: the three-line entry read as a paragraph in a list of two-liners. Decision A: fold the who-judges fact into the description itself - Auto-Approve Your session model clears routine actions; doubtful ones still ask - and let per-check cost surface in the 1.7 metering badge where it actually accrues, instead of as picker text. This supersedes the copy recorded in the previous commit. The `note` field and its render block are removed as dead code; `caution` (the Bypass warning triangle) stays. tsc clean. --- surfaces/gui/src/components/Composer.tsx | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/surfaces/gui/src/components/Composer.tsx b/surfaces/gui/src/components/Composer.tsx index a0fa586a..a7d115b0 100644 --- a/surfaces/gui/src/components/Composer.tsx +++ b/surfaces/gui/src/components/Composer.tsx @@ -24,9 +24,9 @@ import { // kept so saved sessions and configs keep working. Auto-Approve ("auto-approve") is the // reviewer mode (spec: reviewed-auto-mode.md); it appears only when the server says the // feature flag is on, wired in the settings pass — until then the picker omits it. -// `note` renders as a second, dimmer line under the description; `caution` prefixes the -// label with a warning triangle. Both are picker-local extensions of Dropdown's Option. -type ModeOption = Option & { note?: string; caution?: boolean }; +// `caution` prefixes the label with a warning triangle — a picker-local extension of +// Dropdown's Option. +type ModeOption = Option & { caution?: boolean }; const PERMISSION_OPTIONS: ModeOption[] = [ { value: "discuss", label: "Discuss", description: "Chat and explore — no edits or commands" }, @@ -897,11 +897,6 @@ function ModeMenu({ {o.value === mode && } {o.description} - {o.note && ( - - {o.note} - - )} ))} {onUnattendedChange && (