From b1b8d1b7afd2493942ceb39a19dc30855d108a9f Mon Sep 17 00:00:00 2001 From: Aidan Daly Date: Wed, 24 Jun 2026 18:58:07 +0000 Subject: [PATCH] fix(tui): use "No memory" instead of "Disabled" in harness memory picker The harness memory-mode picker and confirm summary labeled the opt-out option "Disabled". Per Amazon's inclusive-language guidance, replace the displayed label with "No memory" (matching the agent/generate wizard, which already uses "None" / "No memory"). Display-only change. The option id, config value { mode: 'disabled' }, the --memory-mode disabled flag, and the CFN Memory: { Disabled: {} } mapping are wire/API contracts and are deliberately left unchanged. Closes #1634 --- src/cli/tui/screens/harness/AddHarnessScreen.tsx | 2 +- src/cli/tui/screens/harness/types.ts | 7 ++++--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/cli/tui/screens/harness/AddHarnessScreen.tsx b/src/cli/tui/screens/harness/AddHarnessScreen.tsx index 4a45f0707..96d9954cd 100644 --- a/src/cli/tui/screens/harness/AddHarnessScreen.tsx +++ b/src/cli/tui/screens/harness/AddHarnessScreen.tsx @@ -455,7 +455,7 @@ export function AddHarnessScreen({ fields.push({ label: 'Memory Relevance Score', value: String(mem.relevanceScore) }); } } else { - fields.push({ label: 'Memory', value: 'Disabled' }); + fields.push({ label: 'Memory', value: 'No memory' }); } } diff --git a/src/cli/tui/screens/harness/types.ts b/src/cli/tui/screens/harness/types.ts index 2b6e9674f..6c6906d25 100644 --- a/src/cli/tui/screens/harness/types.ts +++ b/src/cli/tui/screens/harness/types.ts @@ -299,9 +299,10 @@ export type AdvancedSetting = (typeof ADVANCED_SETTING_OPTIONS)[number]['id']; /** Mode-first memory options. Mirrors the schema's 3-mode union. */ export const MEMORY_MODE_OPTIONS = [ - // Disabled is first so it is the highlighted default (the picker selects index 0). Memory is - // opt-in: a harness has no memory unless the user picks Managed or Existing here. - { id: 'disabled' as const, title: 'Disabled', description: 'No memory (default)' }, + // "No memory" is first so it is the highlighted default (the picker selects index 0). Memory is + // opt-in: a harness has no memory unless the user picks Managed or Existing here. The id stays + // 'disabled' — it is the schema/CFN contract token; only the displayed title is reworded. + { id: 'disabled' as const, title: 'No memory', description: 'Default' }, { id: 'managed' as const, title: 'Managed',