TUI: add inline /api-keys menu (#14472)

## Description
Replaces the TUI-only `/add-api-key` and `/clear-provider-api-key`
commands with a single inline `/api-keys` menu.

The menu supports filtering providers, setting and clearing provider
keys, connecting or disconnecting X Premium/SuperGrok, and toggling
Warp-credit fallback. It reuses the shared TUI editor through the
ownership model reviewed separately in downstack PR #14488.

This is a pretty large change, but the large majority of it is UI
changes. Reading the specs should be enough for this one (I'll also
attach an in depth demo of the change below).

## Testing
- [x] I have manually tested my changes locally with `./script/run`

## Screenshots/Video

https://www.loom.com/share/853ce5b6278840c89ced6bfde7b53820

## Agent Mode
- [x] Warp Agent Mode - This PR was created via Warp's AI Agent Mode

Co-Authored-By: Oz <oz-agent@warp.dev>

Co-authored-by: Oz <oz-agent@warp.dev>

Author: harryalbert

app/src/search/slash_command_menu/static_commands/commands.rs

@@ -1,7 +1,6 @@
 use std::collections::HashMap;
 use std::sync::LazyLock;
 
-use ai::LLMProvider;
 use serde::{Deserialize, Serialize};
 use uuid::Uuid;
 use warp_core::features::FeatureFlag;
@@ -118,31 +117,15 @@ pub const NATURAL_LANGUAGE_DETECTION: StaticCommand = StaticCommand {
     auto_enter_ai_mode: false,
     argument: None,
 };
-pub const ADD_API_KEY: StaticCommand = StaticCommand {
-    name: "/add-api-key",
-    description: "Securely store a model-provider API key",
-    kind: SlashCommandKind::AddApiKey,
-    supported_surfaces: SlashCommandSurfaces::TuiOnly,
-    availability: Availability::AI_ENABLED,
-    auto_enter_ai_mode: false,
-    argument: Some(Argument {
-        hint_text: Some(LLMProvider::API_KEY_PROVIDER_VALUE_NAME),
-        is_optional: false,
-        should_execute_on_selection: false,
-    }),
-};
-pub const CLEAR_API_KEY: StaticCommand = StaticCommand {
-    name: "/clear-provider-api-key",
-    description: "Remove a stored model-provider API key",
-    kind: SlashCommandKind::ClearApiKey,
+
+pub const API_KEYS: StaticCommand = StaticCommand {
+    name: "/api-keys",
+    description: "View and manage API keys",
+    kind: SlashCommandKind::ApiKeys,
     supported_surfaces: SlashCommandSurfaces::TuiOnly,
     availability: Availability::AI_ENABLED,
     auto_enter_ai_mode: false,
-    argument: Some(Argument {
-        hint_text: Some(LLMProvider::API_KEY_PROVIDER_VALUE_NAME),
-        is_optional: false,
-        should_execute_on_selection: false,
-    }),
+    argument: None,
 };
 pub const THEME: StaticCommand = StaticCommand {
     name: "/theme",
@@ -940,8 +923,6 @@ impl Registry {
 fn all_commands(settings_mode: settings::SettingsMode) -> Vec<StaticCommand> {
     let mut commands = vec![
         ADD_MCP,
-        ADD_API_KEY,
-        CLEAR_API_KEY,
         ADD_PROMPT.clone(),
         ADD_RULE,
         AUTO_APPROVE,
@@ -950,6 +931,7 @@ fn all_commands(settings_mode: settings::SettingsMode) -> Vec<StaticCommand> {
         FEEDBACK.clone(),
         INDEX,
         INIT,
+        API_KEYS,
         LOGOUT,
         MCP,
         OPEN_PROJECT_RULES,

app/src/search/slash_command_menu/static_commands/commands_tests.rs

@@ -97,64 +97,25 @@ fn view_logs_command_is_registered_only_for_tui_mode() {
 }
 
 #[test]
-fn add_api_key_command_is_tui_only_and_requires_a_provider() {
+fn api_keys_command_is_tui_only_and_has_no_arguments() {
     let command = all_commands(settings::SettingsMode::Tui)
         .into_iter()
-        .find(|command| command.kind == SlashCommandKind::AddApiKey)
-        .expect("expected /add-api-key to be registered in TUI mode");
-    assert_eq!(command, ADD_API_KEY);
+        .find(|command| command.kind == SlashCommandKind::ApiKeys)
+        .expect("expected /api-keys to be registered in TUI mode");
+    assert_eq!(command, API_KEYS);
     assert!(!command.auto_enter_ai_mode);
     assert_eq!(command.availability, Availability::AI_ENABLED);
-    let argument = command
-        .argument
-        .as_ref()
-        .expect("expected /add-api-key to require a provider");
-    assert!(!argument.is_optional);
-    assert!(!argument.should_execute_on_selection);
-    assert_eq!(
-        argument.hint_text,
-        Some(LLMProvider::API_KEY_PROVIDER_VALUE_NAME)
-    );
-    assert!(
-        argument
-            .hint_text
-            .is_some_and(|hint| hint.split('|').any(|provider| provider == "grok"))
-    );
+    assert!(command.argument.is_none());
+    assert_eq!(command.description, "View and manage API keys");
     assert!(
         all_commands(settings::SettingsMode::Gui)
             .iter()
-            .all(|command| command.kind != SlashCommandKind::AddApiKey)
-    );
-}
-
-#[test]
-fn clear_api_key_command_is_tui_only_and_requires_a_provider() {
-    let command = all_commands(settings::SettingsMode::Tui)
-        .into_iter()
-        .find(|command| command.kind == SlashCommandKind::ClearApiKey)
-        .expect("expected /clear-provider-api-key to be registered in TUI mode");
-    assert_eq!(command, CLEAR_API_KEY);
-    assert!(!command.auto_enter_ai_mode);
-    assert_eq!(command.availability, Availability::AI_ENABLED);
-    let argument = command
-        .argument
-        .as_ref()
-        .expect("expected /clear-provider-api-key to require a provider");
-    assert!(!argument.is_optional);
-    assert!(!argument.should_execute_on_selection);
-    assert_eq!(
-        argument.hint_text,
-        Some(LLMProvider::API_KEY_PROVIDER_VALUE_NAME)
+            .all(|command| command.kind != SlashCommandKind::ApiKeys)
     );
     assert!(
-        argument
-            .hint_text
-            .is_some_and(|hint| hint.split('|').any(|provider| provider == "grok"))
-    );
-    assert!(
-        all_commands(settings::SettingsMode::Gui)
+        all_commands(settings::SettingsMode::Tui)
             .iter()
-            .all(|command| command.kind != SlashCommandKind::ClearApiKey)
+            .all(|command| !matches!(command.name, "/add-api-key" | "/clear-provider-api-key"))
     );
 }
 #[test]

app/src/search/slash_command_menu/static_commands/mod.rs

@@ -54,8 +54,7 @@ pub enum SlashCommandKind {
     Agent,
     CloudAgent,
     AddMcp,
-    AddApiKey,
-    ClearApiKey,
+    ApiKeys,
     AutoApprove,
     Statusline,
     ResetStatusline,

app/src/settings/ai.rs

@@ -1788,7 +1788,7 @@ define_settings_group!(AISettings, settings: [
         default: false,
         supported_platforms: SupportedPlatforms::ALL,
         sync_to_cloud: SyncToCloud::Globally(RespectUserSyncSetting::Yes),
-        surface: settings::SettingSurfaces::GUI,
+        surface: settings::SettingSurfaces::ALL,
         private: false,
         storage_key: "CanUseWarpCreditsWithByok",
         toml_path: "cloud_platform.third_party_api_keys.can_use_warp_credits_with_byok",

app/src/terminal/input/slash_commands/mod.rs

@@ -1289,8 +1289,7 @@ impl Input {
             SlashCommandKind::AutoApprove
             | SlashCommandKind::Statusline
             | SlashCommandKind::ResetStatusline
-            | SlashCommandKind::AddApiKey
-            | SlashCommandKind::ClearApiKey
+            | SlashCommandKind::ApiKeys
             | SlashCommandKind::ViewLogs
             | SlashCommandKind::Voice
             | SlashCommandKind::NaturalLanguageDetection