@pellux/goodvibes-tui 0.19.28 → 0.19.30

package/src/input/onboarding/onboarding-wizard-steps.ts

@@ -1,8 +1,14 @@
 import { NETWORK_MODE_OPTIONS, REASONING_OPTIONS, HITL_MODE_OPTIONS, GUIDANCE_MODE_OPTIONS, PERMISSION_MODE_OPTIONS, SECRET_POLICY_OPTIONS } from './onboarding-wizard-constants.ts';
-import { EXTERNAL_SURFACE_SPECS, type ExternalSurfaceSpec } from './onboarding-wizard-external-surfaces.ts';
+import {
+  EXTERNAL_SURFACE_SPECS,
+  getExternalSurfaceAutoStartDefaultValue,
+  getExternalSurfaceAutoStartFieldId,
+  isExternalSurfaceSelectedByDefault,
+  type ExternalSurfaceSpec,
+} from './onboarding-wizard-external-surfaces.ts';
 import { countSelected, modelSelectionLabel, normalizeText } from './onboarding-wizard-helpers.ts';
 import type { OnboardingWizardController } from './onboarding-wizard.ts';
-import type { OnboardingWizardAcknowledgementFieldDefinition, OnboardingWizardChecklistFieldDefinition, OnboardingWizardExternalSurfaceStepId, OnboardingWizardFieldDefinition, OnboardingWizardModelPickerFieldDefinition, OnboardingWizardRadioFieldDefinition, OnboardingWizardStepDefinition } from './onboarding-wizard-types.ts';
+import type { OnboardingWizardAcknowledgementFieldDefinition, OnboardingWizardChecklistFieldDefinition, OnboardingWizardExternalSurfaceStepId, OnboardingWizardFieldDefinition, OnboardingWizardModelPickerFieldDefinition, OnboardingWizardRadioFieldDefinition, OnboardingWizardRadioOption, OnboardingWizardStepDefinition } from './onboarding-wizard-types.ts';
 
 export function buildOnboardingWizardSteps(controller: OnboardingWizardController): readonly OnboardingWizardStepDefinition[] {
   if (controller.hydrationPending || controller.hydrationError !== null) return [buildLoadingStep(controller)];
@@ -86,7 +92,7 @@ export function buildCapabilitiesStep(controller: OnboardingWizardController): O
         id: 'capabilities.select-all',
         action: 'select-all-capabilities',
         label: 'Select all server-backed capabilities',
-        hint: 'Enable browser access, other-device LAN access, webhooks/events, and external integrations. Local TUI Only is turned off.',
+        hint: 'Enable browser access, LAN reachability, webhooks/events, and external app surfaces. Local TUI Only is turned off.',
         defaultValue: 'Action',
       },
       {
@@ -103,7 +109,7 @@ export function buildCapabilitiesStep(controller: OnboardingWizardController): O
       id: 'capabilities',
       title: 'Choose GoodVibes capabilities',
       shortLabel: 'Capabilities',
-      description: 'Select one or more capabilities. Local TUI Only means no browser, daemon, listener, or network setup; any other choice turns on service mode and autostart.',
+      description: 'Choose what GoodVibes should be able to do. Local TUI Only avoids servers; any other choice enables service mode and autostart.',
       summaryTitle: 'Selected capabilities',
       summaryLines: [
         `${selectedCount}/${capabilities.length} option(s) selected`,
@@ -266,8 +272,11 @@ export function buildDefaultModelStep(controller: OnboardingWizardController): O
   }
 
 export function buildExternalServicesStep(controller: OnboardingWizardController): OnboardingWizardStepDefinition {
-    const enabledCount = EXTERNAL_SURFACE_SPECS
-      .filter((surface) => controller.getBooleanFieldValue(surface.enabledFieldId, surface.defaultEnabled(controller.runtimeSnapshot)))
+    const selectedCount = EXTERNAL_SURFACE_SPECS
+      .filter((surface) => controller.getBooleanFieldValue(
+        surface.enabledFieldId,
+        isExternalSurfaceSelectedByDefault(surface, controller.runtimeSnapshot),
+      ))
       .length;
     const fields: OnboardingWizardFieldDefinition[] = [];
 
@@ -276,8 +285,8 @@ export function buildExternalServicesStep(controller: OnboardingWizardController
         kind: 'checklist',
         id: surface.enabledFieldId,
         label: surface.label,
-        hint: surface.hint,
-        defaultValue: surface.defaultEnabled(controller.runtimeSnapshot),
+        hint: `${surface.hint} Selecting this opens a dedicated setup screen; auto-start is chosen on that screen.`,
+        defaultValue: isExternalSurfaceSelectedByDefault(surface, controller.runtimeSnapshot),
       });
     }
 
@@ -287,7 +296,7 @@ export function buildExternalServicesStep(controller: OnboardingWizardController
         id: 'external-services.select-all',
         action: 'select-all-external-surfaces',
         label: 'Select all external surfaces',
-        hint: 'Enable every supported external surface so each one gets a setup screen.',
+        hint: 'Show setup screens for every supported external surface. Auto-start stays controlled per surface.',
         defaultValue: 'Action',
       },
       {
@@ -295,7 +304,7 @@ export function buildExternalServicesStep(controller: OnboardingWizardController
         id: 'external-services.clear',
         action: 'clear-external-surfaces',
         label: 'Clear all external surfaces',
-        hint: 'Disable all external surfaces. The HTTP listener can still be enabled separately by webhook/event capabilities.',
+        hint: 'Hide all external surface setup screens. The HTTP listener can still be enabled separately by webhook/event capabilities.',
         defaultValue: 'Action',
       },
       {
@@ -312,12 +321,12 @@ export function buildExternalServicesStep(controller: OnboardingWizardController
       id: 'external-services',
       title: 'Choose external surfaces',
       shortLabel: 'Services',
-      description: 'Select the apps and integration surfaces GoodVibes should prepare. Setup fields are not shown here; each selected surface opens as its own screen.',
+      description: 'Select the apps and integration surfaces GoodVibes should prepare. Each selected surface gets its own setup screen and its own auto-start choice.',
       summaryTitle: 'External surfaces',
       summaryLines: [
-        `${enabledCount} external surface(s) selected`,
+        `${selectedCount} external surface(s) selected for setup`,
         `Secret policy: ${controller.getStringFieldValue('external-services.secret-policy', controller.runtimeSnapshot?.runtimeDefaults.secretStoragePolicy ?? 'preferred_secure')}`,
-        enabledCount > 0 ? 'Selected surfaces appear as separate setup screens.' : 'No external surfaces selected.',
+        selectedCount > 0 ? 'Selected surfaces appear as separate setup screens.' : 'No external surfaces selected.',
       ],
       fields,
     };
@@ -325,28 +334,47 @@ export function buildExternalServicesStep(controller: OnboardingWizardController
 
 function getSelectedExternalSurfaceSpecs(controller: OnboardingWizardController): readonly ExternalSurfaceSpec[] {
     return EXTERNAL_SURFACE_SPECS.filter((surface) => (
-      controller.getBooleanFieldValue(surface.enabledFieldId, surface.defaultEnabled(controller.runtimeSnapshot))
+      controller.getBooleanFieldValue(
+        surface.enabledFieldId,
+        isExternalSurfaceSelectedByDefault(surface, controller.runtimeSnapshot),
+      )
     ));
   }
 
+const SURFACE_AUTO_START_OPTIONS: readonly OnboardingWizardRadioOption[] = [
+  {
+    id: 'yes',
+    label: 'Yes',
+    hint: 'Start this surface automatically when the GoodVibes service starts.',
+  },
+  {
+    id: 'no',
+    label: 'No',
+    hint: 'Save these settings but leave the surface idle until it is enabled from Settings > Surfaces.',
+  },
+];
+
 function buildExternalSurfaceStep(
   controller: OnboardingWizardController,
   surface: ExternalSurfaceSpec,
 ): OnboardingWizardStepDefinition {
-    let requiredCount = 0;
-    let requiredCompleteCount = 0;
+    let setupCount = 0;
+    let setupCompleteCount = 0;
+    const autoStartFieldId = getExternalSurfaceAutoStartFieldId(surface);
+    const autoStartDefault = getExternalSurfaceAutoStartDefaultValue(surface, controller.runtimeSnapshot);
+    const autoStartValue = controller.getStringFieldValue(autoStartFieldId, autoStartDefault);
     const setupFields = surface.fields.map((setupField): OnboardingWizardFieldDefinition => {
-      const required = controller.isRequiredExternalSetupField(setupField.id);
-      if (required) {
-        requiredCount += 1;
+      const suggested = controller.isRequiredExternalSetupField(setupField.id);
+      if (suggested) {
+        setupCount += 1;
         if (normalizeText(setupField.defaultValue(controller.runtimeSnapshot)).length > 0
           || normalizeText(controller.getStringFieldValue(setupField.id, '')).length > 0) {
-          requiredCompleteCount += 1;
+          setupCompleteCount += 1;
         }
       }
 
-      const hint = required
-        ? `${setupField.hint} Required because ${surface.label} is selected.`
+      const hint = suggested
+        ? `${setupField.hint} Recommended because ${surface.label} is selected, but it will not block saving.`
         : setupField.hint;
 
       if (setupField.kind === 'radio') {
@@ -367,26 +395,46 @@ function buildExternalSurfaceStep(
         hint,
         placeholder: setupField.placeholder,
         defaultValue: setupField.defaultValue(controller.runtimeSnapshot),
-        required,
       };
     });
+    const ntfyTopicSummary = surface.id === 'ntfy'
+      ? [
+          `Chat topic: ${controller.getStringFieldValue('external-services.ntfy.chat-topic', 'goodvibes-chat')}`,
+          `Agent topic: ${controller.getStringFieldValue('external-services.ntfy.agent-topic', 'goodvibes-agent')}`,
+          `Daemon-only remote topic: ${controller.getStringFieldValue('external-services.ntfy.remote-topic', 'goodvibes-ntfy')}`,
+        ]
+      : [];
     const title = `${surface.label.replace(/ surface$/i, '')} setup`;
-    const requiredSummary = requiredCount === 0
-      ? 'Required setup: none'
-      : `Required setup: ${requiredCompleteCount}/${requiredCount}`;
+    const setupSummary = setupCount === 0
+      ? 'Suggested setup: none'
+      : `Suggested setup entered: ${setupCompleteCount}/${setupCount}`;
 
     return {
       id: `external-surface:${surface.id}` as OnboardingWizardExternalSurfaceStepId,
       title,
       shortLabel: surface.label.replace(/ surface$/i, ''),
-      description: `Configure ${surface.label}. This screen only appears because that surface is selected on the previous screen.`,
+      description: `Configure ${surface.label}. Settings are saved either way; auto-start controls whether this surface starts with the background service.`,
       summaryTitle: `${surface.label} setup`,
       summaryLines: [
-        requiredSummary,
+        `Auto-start: ${autoStartValue === 'yes' ? 'yes' : 'no'}`,
+        ...ntfyTopicSummary,
+        setupSummary,
         `Secret policy: ${controller.getStringFieldValue('external-services.secret-policy', controller.runtimeSnapshot?.runtimeDefaults.secretStoragePolicy ?? 'preferred_secure')}`,
-        'Leave optional fields blank to keep defaults or existing values.',
+        autoStartValue === 'yes'
+          ? 'The surface will be enabled after apply.'
+          : 'Start it later from Settings > Surfaces by turning Enabled on.',
+      ],
+      fields: [
+        {
+          kind: 'radio',
+          id: autoStartFieldId,
+          label: 'Auto-start this surface',
+          hint: `Yes turns on ${surface.enabledConfigKey}. No saves setup values but keeps the surface off until Settings > Surfaces enables it.`,
+          options: SURFACE_AUTO_START_OPTIONS,
+          defaultValue: autoStartDefault,
+        },
+        ...setupFields,
       ],
-      fields: setupFields,
     };
   }
 
@@ -574,32 +622,34 @@ export function buildAccountsStep(controller: OnboardingWizardController): Onboa
       && !needsAuthBootstrap
       && controller.hasLocalAuthUser();
     const fields: OnboardingWizardFieldDefinition[] = [];
+    const defaultAdminUsername = controller.getDefaultAdminUsername();
 
-    if (needsAuthBootstrap) {
-      const defaultAdminUsername = controller.getDefaultAdminUsername();
-      fields.push(
-        {
-          kind: 'text',
-          id: 'accounts.admin-username',
-          label: 'Local auth admin username',
-          hint: 'Required before any background service, browser surface, or listener is exposed.',
-          placeholder: defaultAdminUsername,
-          defaultValue: defaultAdminUsername,
-          required: true,
-        },
-        {
-          kind: 'masked',
-          id: 'accounts.admin-password',
-          label: 'Local auth admin password',
-          hint: controller.hasBootstrapCredentialPresent()
-            ? 'Creates a new local admin, removes the bootstrap credential file, and retires the bootstrap admin before LAN/server settings are applied.'
-            : 'Creates the first local admin user and an initial session before LAN/server settings are applied.',
-          placeholder: 'password required',
-          defaultValue: '',
-          required: true,
-        },
-      );
-    }
+    fields.push(
+      {
+        kind: 'text',
+        id: 'accounts.admin-username',
+        label: 'Local auth admin username',
+        hint: needsAuthBootstrap
+          ? 'Required before any background service, browser surface, or listener is exposed.'
+          : 'Optional. Enter an existing admin username to rotate its password, or a new username to create another admin.',
+        placeholder: defaultAdminUsername,
+        defaultValue: defaultAdminUsername,
+        required: needsAuthBootstrap,
+      },
+      {
+        kind: 'masked',
+        id: 'accounts.admin-password',
+        label: 'Local auth admin password',
+        hint: needsAuthBootstrap
+          ? controller.hasBootstrapCredentialPresent()
+            ? 'Creates or updates the named local admin, removes the bootstrap credential file, and retires the bootstrap admin when it is a different user.'
+            : 'Creates the first local admin user and an initial session before LAN/server settings are applied.'
+          : 'Optional. Leave blank to keep existing local auth unchanged; enter a password to create or rotate the named admin user.',
+        placeholder: needsAuthBootstrap ? 'password required' : 'leave blank to keep unchanged',
+        defaultValue: '',
+        required: needsAuthBootstrap,
+      },
+    );
 
     fields.push(
       {
@@ -668,6 +718,29 @@ export function buildAccountsStep(controller: OnboardingWizardController): Onboa
   }
 
 export function buildReviewStep(controller: OnboardingWizardController): OnboardingWizardStepDefinition {
+    const feedback = controller.applyFeedback;
+    const feedbackFields: OnboardingWizardFieldDefinition[] = feedback
+      ? [
+          {
+            kind: 'status',
+            id: 'review.feedback',
+            label: feedback.title,
+            hint: feedback.summary,
+            defaultValue: feedback.severity === 'error' ? 'Needs attention' : feedback.severity === 'warning' ? 'Warning' : 'Info',
+          },
+          ...feedback.messages.slice(0, 8).map((message, index): OnboardingWizardFieldDefinition => ({
+            kind: 'status',
+            id: `review.feedback.${index}`,
+            label: message,
+            hint: message,
+            defaultValue: feedback.severity === 'error' ? 'Error' : feedback.severity === 'warning' ? 'Warning' : 'Info',
+          })),
+        ]
+      : [];
+    const unsavedLabel = controller.dirtyStepCount === 1
+      ? '1 screen has unapplied changes'
+      : `${controller.dirtyStepCount} screens have unapplied changes`;
+
     return {
       id: 'review',
       title: 'Review and apply',
@@ -675,12 +748,13 @@ export function buildReviewStep(controller: OnboardingWizardController): Onboard
       description: 'Review the selected settings and apply them directly from the wizard.',
       summaryTitle: 'Review posture',
       summaryLines: [
-        `${controller.dirtyStepCount} dirty step(s)`,
-        `${controller.buildApplyRequest().operations.length} operation(s) ready to apply`,
-        `Pending picker: ${controller.pendingModelPickerTarget ?? 'none'}`,
-        controller.isEditingTextField() ? `Editing: ${controller.editingFieldId}` : 'Ready for apply/verify',
+        unsavedLabel,
+        `${controller.buildApplyRequest().operations.length} settings change(s) ready to apply`,
+        feedback ? `Last apply: ${feedback.title}` : 'No apply errors reported',
+        controller.isEditingTextField() ? `Editing: ${controller.editingFieldId}` : 'Ready to apply',
       ],
       fields: [
+        ...feedbackFields,
         {
           kind: 'status',
           id: 'review.global-marker',

package/src/input/onboarding/onboarding-wizard-types.ts

@@ -49,6 +49,15 @@ export type OnboardingWizardAction =
   | 'start-openai-subscription'
   | 'finish-openai-subscription';
 
+export type OnboardingWizardApplyFeedbackSeverity = 'info' | 'warning' | 'error';
+
+export interface OnboardingWizardApplyFeedback {
+  readonly severity: OnboardingWizardApplyFeedbackSeverity;
+  readonly title: string;
+  readonly summary: string;
+  readonly messages: readonly string[];
+}
+
 export interface OnboardingWizardRadioOption {
   readonly id: string;
   readonly label: string;
@@ -164,6 +173,7 @@ export interface OnboardingWizardSnapshot {
   readonly editBuffer: string;
   readonly hydrationPending: boolean;
   readonly hydrationError: string | null;
+  readonly applyFeedback: OnboardingWizardApplyFeedback | null;
   readonly hydration: OnboardingWizardRuntimeHydration;
 }
 

package/src/input/onboarding/onboarding-wizard.ts

@@ -34,9 +34,9 @@ import {
   toggleCapability as toggleCapabilityForController,
 } from './onboarding-wizard-rules.ts';
 import { ensureSelectionVisible as ensureSelectionVisibleForController, getBlockingFieldLabels as getBlockingFieldLabelsForController, getCapabilitySelectionState as getCapabilitySelectionStateForController, getCompletedFieldCount as getCompletedFieldCountForController, getCompletedToggleCount as getCompletedToggleCountForController, getCurrentCapabilities as getCurrentCapabilitiesForController, getFieldById as getFieldByIdForController, getFieldValidationError as getFieldValidationErrorForController, getStepFieldCount as getStepFieldCountForController, getToggleFieldCount as getToggleFieldCountForController, hasExistingAccessState as hasExistingAccessStateForController, isFieldDirty as isFieldDirtyForController, isFieldDirtyByDefinition as isFieldDirtyByDefinitionForController, isFieldSatisfied as isFieldSatisfiedForController, isStepDirty as isStepDirtyForController, recalculateDirtyState as recalculateDirtyStateForController, reconcileStateWithCurrentDefinitions as reconcileStateWithCurrentDefinitionsForController, reconcileStepCursor as reconcileStepCursorForController, resetValuesFromCurrentDefinitions as resetValuesFromCurrentDefinitionsForController } from './onboarding-wizard-state.ts';
-import type { MutableModelSelectionMap, OnboardingWizardAction, OnboardingWizardFieldDefinition, OnboardingWizardFieldWindow, OnboardingWizardMode, OnboardingWizardModelSelection, OnboardingWizardRuntimeHydration, OnboardingWizardSnapshot, OnboardingWizardStepDefinition, OnboardingWizardStepId } from './onboarding-wizard-types.ts';
+import type { MutableModelSelectionMap, OnboardingWizardAction, OnboardingWizardApplyFeedback, OnboardingWizardFieldDefinition, OnboardingWizardFieldWindow, OnboardingWizardMode, OnboardingWizardModelSelection, OnboardingWizardRuntimeHydration, OnboardingWizardSnapshot, OnboardingWizardStepDefinition, OnboardingWizardStepId } from './onboarding-wizard-types.ts';
 
-export type { OnboardingWizardAcknowledgementFieldDefinition, OnboardingWizardAction, OnboardingWizardActionFieldDefinition, OnboardingWizardChecklistFieldDefinition, OnboardingWizardExternalSurfaceStepId, OnboardingWizardFieldDefinition, OnboardingWizardFieldKind, OnboardingWizardFieldWindow, OnboardingWizardMaskedFieldDefinition, OnboardingWizardMode, OnboardingWizardModelPickerFieldDefinition, OnboardingWizardModelSelection, OnboardingWizardRadioFieldDefinition, OnboardingWizardRadioOption, OnboardingWizardRuntimeHydration, OnboardingWizardSnapshot, OnboardingWizardStatusFieldDefinition, OnboardingWizardStepDefinition, OnboardingWizardStepId, OnboardingWizardTextFieldDefinition } from './onboarding-wizard-types.ts';
+export type { OnboardingWizardAcknowledgementFieldDefinition, OnboardingWizardAction, OnboardingWizardActionFieldDefinition, OnboardingWizardApplyFeedback, OnboardingWizardApplyFeedbackSeverity, OnboardingWizardChecklistFieldDefinition, OnboardingWizardExternalSurfaceStepId, OnboardingWizardFieldDefinition, OnboardingWizardFieldKind, OnboardingWizardFieldWindow, OnboardingWizardMaskedFieldDefinition, OnboardingWizardMode, OnboardingWizardModelPickerFieldDefinition, OnboardingWizardModelSelection, OnboardingWizardRadioFieldDefinition, OnboardingWizardRadioOption, OnboardingWizardRuntimeHydration, OnboardingWizardSnapshot, OnboardingWizardStatusFieldDefinition, OnboardingWizardStepDefinition, OnboardingWizardStepId, OnboardingWizardTextFieldDefinition } from './onboarding-wizard-types.ts';
 export { getOnboardingWizardBodyRows, getOnboardingWizardVisibleFieldCount } from './onboarding-wizard-helpers.ts';
 
 export class OnboardingWizardController {
@@ -58,6 +58,7 @@ export class OnboardingWizardController {
   public editingFieldId: string | null = null;
   public editBuffer = '';
   public hydrationError: string | null = null;
+  public applyFeedback: OnboardingWizardApplyFeedback | null = null;
 
   public readonly baselineToggleState = new Map<string, boolean>();
   public readonly baselineRadioState = new Map<string, string>();
@@ -102,6 +103,7 @@ export class OnboardingWizardController {
     this.stepIndex = 0;
     this.pendingModelPickerTarget = null;
     this.pendingAction = null;
+    this.applyFeedback = null;
     this.editingFieldId = null;
     this.editBuffer = '';
     this.scrollOffsets.fill(0);
@@ -115,6 +117,7 @@ export class OnboardingWizardController {
     this.hydrationError = null;
     this.pendingModelPickerTarget = null;
     this.pendingAction = null;
+    this.applyFeedback = null;
     this.cancelEdit();
   }
 
@@ -178,6 +181,7 @@ export class OnboardingWizardController {
       editBuffer: this.editBuffer,
       hydrationPending: this.hydrationPending,
       hydrationError: this.hydrationError,
+      applyFeedback: this.applyFeedback,
       hydration: this.captureHydratedState(),
     };
   }
@@ -191,6 +195,7 @@ export class OnboardingWizardController {
     this.stepIndex = clamp(snapshot.stepIndex, 0, Math.max(0, this.steps.length - 1));
     this.pendingModelPickerTarget = snapshot.pendingModelPickerTarget;
     this.pendingAction = snapshot.pendingAction;
+    this.applyFeedback = snapshot.applyFeedback;
     this.dirtyStepIds.clear();
     for (const stepId of snapshot.dirtyStepIds) this.dirtyStepIds.add(stepId);
 
@@ -223,6 +228,7 @@ export class OnboardingWizardController {
   public beginRuntimeHydration(): void {
     this.hydrationPending = true;
     this.hydrationError = null;
+    this.applyFeedback = null;
     this.stepIndex = 0;
     this.pendingModelPickerTarget = null;
     this.pendingAction = null;
@@ -232,6 +238,7 @@ export class OnboardingWizardController {
   public finishRuntimeHydration(): void {
     this.hydrationPending = false;
     this.hydrationError = null;
+    this.applyFeedback = null;
     this.stepIndex = clamp(this.stepIndex, 0, Math.max(0, this.steps.length - 1));
     this.reconcileStepCursor(this.stepIndex);
   }
@@ -239,6 +246,12 @@ export class OnboardingWizardController {
   public failRuntimeHydration(message: string): void {
     this.hydrationPending = false;
     this.hydrationError = message;
+    this.applyFeedback = {
+      severity: 'error',
+      title: 'Current settings could not load',
+      summary: message,
+      messages: [message],
+    };
     this.stepIndex = 0;
     this.pendingModelPickerTarget = null;
     this.pendingAction = null;
@@ -344,6 +357,14 @@ export class OnboardingWizardController {
     this.pendingAction = null;
   }
 
+  public setApplyFeedback(feedback: OnboardingWizardApplyFeedback): void {
+    this.applyFeedback = feedback;
+  }
+
+  public clearApplyFeedback(): void {
+    this.applyFeedback = null;
+  }
+
   public consumePendingAction(): OnboardingWizardAction | null {
     const action = this.pendingAction;
     this.pendingAction = null;
@@ -360,6 +381,7 @@ export class OnboardingWizardController {
       this.toggleCapability(field.capabilityId);
       this.pendingModelPickerTarget = null;
       this.pendingAction = null;
+      this.applyFeedback = null;
       this.recalculateDirtyState();
       return;
     }
@@ -369,6 +391,7 @@ export class OnboardingWizardController {
       this.toggleState.set(field.id, !current);
       this.pendingModelPickerTarget = null;
       this.pendingAction = null;
+      this.applyFeedback = null;
       this.recalculateDirtyState();
       return;
     }
@@ -382,6 +405,7 @@ export class OnboardingWizardController {
       this.radioState.set(field.id, next.id);
       this.pendingModelPickerTarget = null;
       this.pendingAction = null;
+      this.applyFeedback = null;
       this.recalculateDirtyState();
       return;
     }
@@ -397,6 +421,7 @@ export class OnboardingWizardController {
         this.selectAllServerCapabilities();
         this.pendingAction = null;
         this.pendingModelPickerTarget = null;
+        this.applyFeedback = null;
         this.recalculateDirtyState();
         return;
       }
@@ -405,6 +430,7 @@ export class OnboardingWizardController {
         this.selectLocalTuiOnly();
         this.pendingAction = null;
         this.pendingModelPickerTarget = null;
+        this.applyFeedback = null;
         this.recalculateDirtyState();
         return;
       }
@@ -413,6 +439,7 @@ export class OnboardingWizardController {
         this.selectAllExternalSurfaces();
         this.pendingAction = null;
         this.pendingModelPickerTarget = null;
+        this.applyFeedback = null;
         this.recalculateDirtyState();
         return;
       }
@@ -421,6 +448,7 @@ export class OnboardingWizardController {
         this.clearExternalSurfaces();
         this.pendingAction = null;
         this.pendingModelPickerTarget = null;
+        this.applyFeedback = null;
         this.recalculateDirtyState();
         return;
       }
@@ -433,6 +461,7 @@ export class OnboardingWizardController {
     this.pendingModelPickerTarget = field.target;
     this.pendingAction = null;
     this.touchedActionFields.add(field.id);
+    this.applyFeedback = null;
   }
 
   public beginEdit(fieldId: string): void {
@@ -458,6 +487,7 @@ export class OnboardingWizardController {
     const field = this.getFieldById(fieldId);
     if (field && (field.kind === 'text' || field.kind === 'masked')) {
       this.textState.set(fieldId, this.editBuffer);
+      this.applyFeedback = null;
       this.recalculateDirtyState();
     }
     this.editingFieldId = null;
@@ -479,6 +509,21 @@ export class OnboardingWizardController {
     this.editBuffer = this.editBuffer.slice(0, -1);
   }
 
+  public clearEditingValue(): void {
+    if (this.editingFieldId === null) return;
+    this.editBuffer = '';
+  }
+
+  public clearSelectedTextField(): boolean {
+    const field = this.getSelectedField();
+    if (!field || (field.kind !== 'text' && field.kind !== 'masked')) return false;
+    this.textState.set(field.id, '');
+    if (this.editingFieldId === field.id) this.editBuffer = '';
+    this.applyFeedback = null;
+    this.recalculateDirtyState();
+    return true;
+  }
+
   public setFieldValue(fieldId: string, value: boolean | string): void {
     const field = this.getFieldById(fieldId);
     if (!field) return;
@@ -487,6 +532,7 @@ export class OnboardingWizardController {
       if (typeof value === 'boolean') {
         if (field.kind === 'checklist' && field.capabilityId) this.setCapabilityValue(field.capabilityId, value);
         else this.toggleState.set(fieldId, value);
+        this.applyFeedback = null;
         this.recalculateDirtyState();
       }
       return;
@@ -495,6 +541,7 @@ export class OnboardingWizardController {
     if (field.kind === 'radio') {
       if (typeof value === 'string' && field.options.some((option) => option.id === value)) {
         this.radioState.set(fieldId, value);
+        this.applyFeedback = null;
         this.recalculateDirtyState();
       }
       return;
@@ -504,6 +551,7 @@ export class OnboardingWizardController {
       if (typeof value === 'string') {
         this.textState.set(fieldId, value);
         if (this.editingFieldId === fieldId) this.editBuffer = value;
+        this.applyFeedback = null;
         this.recalculateDirtyState();
       }
     }
@@ -559,13 +607,15 @@ export class OnboardingWizardController {
 
     if (field.kind === 'text') {
       const value = normalizeText(this.getFieldValue(field) as string);
-      if (value.length === 0 && (field.required === true || this.isRequiredExternalSetupField(field.id))) return 'Missing';
+      if (value.length === 0 && field.required === true) return 'Missing';
+      if (value.length === 0 && this.isRequiredExternalSetupField(field.id)) return 'Not set';
       return value.length > 0 ? value : field.placeholder;
     }
 
     if (field.kind === 'masked') {
       const value = normalizeText(this.getFieldValue(field) as string);
-      if (value.length === 0 && (field.required === true || this.isRequiredExternalSetupField(field.id))) return 'Missing';
+      if (value.length === 0 && field.required === true) return 'Missing';
+      if (value.length === 0 && this.isRequiredExternalSetupField(field.id)) return 'Not set';
       return value.length > 0 ? maskValue(value) : field.placeholder;
     }
 
@@ -584,6 +634,7 @@ export class OnboardingWizardController {
       enabled: selection.enabled ?? true,
     });
     this.pendingModelPickerTarget = null;
+    this.applyFeedback = null;
     this.recalculateDirtyState();
   }
 
@@ -601,6 +652,7 @@ export class OnboardingWizardController {
     this.baselineModelSelectionState.clear();
     for (const [key, value] of this.modelSelectionState) this.baselineModelSelectionState.set(key, cloneSelection(value));
     this.dirtyStepIds.clear();
+    this.applyFeedback = null;
   }
 
   public getSharedIpDefault(enabled: { readonly controlPlane: boolean; readonly httpListener: boolean; readonly web: boolean }): boolean {

package/src/input/settings-modal-types.ts

@@ -2,7 +2,7 @@ import type { ConfigSetting } from '@pellux/goodvibes-sdk/platform/config/schema
 import type { ProviderAuthFreshness, ProviderAuthRoute } from '@pellux/goodvibes-sdk/platform/runtime/provider-accounts/registry';
 import type { FeatureFlag, FlagState } from '@pellux/goodvibes-sdk/platform/runtime/feature-flags/types';
 
-export type SettingsCategory = 'display' | 'ui' | 'provider' | 'subscriptions' | 'behavior' | 'storage' | 'permissions' | 'mcp' | 'sandbox' | 'danger' | 'tools' | 'flags' | 'network';
+export type SettingsCategory = 'display' | 'ui' | 'provider' | 'subscriptions' | 'behavior' | 'storage' | 'permissions' | 'mcp' | 'sandbox' | 'surfaces' | 'danger' | 'tools' | 'flags' | 'network';
 
 export const SETTINGS_CATEGORIES: SettingsCategory[] = [
   'display',
@@ -14,6 +14,7 @@ export const SETTINGS_CATEGORIES: SettingsCategory[] = [
   'permissions',
   'mcp',
   'sandbox',
+  'surfaces',
   'danger',
   'tools',
   'flags',

package/src/input/settings-modal.ts

@@ -519,6 +519,8 @@ export class SettingsModal {
         cat = 'tools';
       } else if (rawCat === 'controlPlane' || rawCat === 'httpListener' || rawCat === 'web') {
         cat = 'network';
+      } else if (rawCat === 'surfaces') {
+        cat = 'surfaces';
       } else {
         cat = rawCat as SettingsCategory;
       }
@@ -746,6 +748,8 @@ export class SettingsModal {
         if (isRestartKey && previousValue !== value) {
           this.lastSaveTriggeredRestart = 'web';
         }
+      } else if (rawCat === 'surfaces') {
+        cat = 'surfaces';
       } else {
         cat = rawCat as SettingsCategory;
       }