mirror of
https://github.com/andrewyng/openworker.git
synced 2026-09-11 14:50:14 +00:00
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.
This commit is contained in:
@@ -24,9 +24,9 @@ import {
|
|||||||
// kept so saved sessions and configs keep working. Auto-Approve ("auto-approve") is the
|
// 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
|
// 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.
|
// 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
|
// `caution` prefixes the label with a warning triangle — a picker-local extension of
|
||||||
// label with a warning triangle. Both are picker-local extensions of Dropdown's Option.
|
// Dropdown's Option.
|
||||||
type ModeOption = Option & { note?: string; caution?: boolean };
|
type ModeOption = Option & { caution?: boolean };
|
||||||
|
|
||||||
const PERMISSION_OPTIONS: ModeOption[] = [
|
const PERMISSION_OPTIONS: ModeOption[] = [
|
||||||
{ value: "discuss", label: "Discuss", description: "Chat and explore — no edits or commands" },
|
{ value: "discuss", label: "Discuss", description: "Chat and explore — no edits or commands" },
|
||||||
@@ -897,11 +897,6 @@ function ModeMenu({
|
|||||||
{o.value === mode && <span className="ml-1.5">✓</span>}
|
{o.value === mode && <span className="ml-1.5">✓</span>}
|
||||||
</span>
|
</span>
|
||||||
<span className="text-[11px] text-faint leading-snug">{o.description}</span>
|
<span className="text-[11px] text-faint leading-snug">{o.description}</span>
|
||||||
{o.note && (
|
|
||||||
<span className="text-[10.5px] text-faint/80 italic leading-snug mt-0.5">
|
|
||||||
{o.note}
|
|
||||||
</span>
|
|
||||||
)}
|
|
||||||
</button>
|
</button>
|
||||||
))}
|
))}
|
||||||
{onUnattendedChange && (
|
{onUnattendedChange && (
|
||||||
|
|||||||
Reference in New Issue
Block a user