Add MCP setup to app menu (#76)

Author: pranc1ngpegasus

README.md

@@ -80,7 +80,7 @@ Set `WISP_OUTPUT_DIR` to override where recordings are written. When unset, Wisp
 
 ### Local MCP bridge
 
-Enable **Local MCP Bridge** from the Wisp library screen to expose the visible transcript through a local IPC endpoint. By default it binds to `http://127.0.0.1:8765/conversation`; set `WISP_IPC_ADDR=127.0.0.1:9001` to override the address while developing. Set `WISP_IPC_TOKEN` to require `Authorization: Bearer <token>` on IPC requests.
+Choose **Wisp → MCP Setup…** (or press <kbd>⌘,</kbd>) to open the guided setup window. From there you can enable the Local MCP Bridge and copy the bundled server path or ready-to-paste JSON for Claude and OpenCode. The bridge exposes the visible transcript through a local IPC endpoint. By default it binds to `http://127.0.0.1:8765/conversation`; set `WISP_IPC_ADDR=127.0.0.1:9001` to override the address while developing. Set `WISP_IPC_TOKEN` to require `Authorization: Bearer <token>` on IPC requests; copied client configs include the current address and, when set, the token.
 
 MCP hosts should run the bundled `wisp-mcp` binary over stdio, for example `/Applications/Wisp.app/Contents/MacOS/wisp-mcp`. The bridge provides the `ask_current_conversation` tool, fetches the current Wisp transcript from the IPC endpoint, and returns context for the host LLM to answer questions such as `いまの話ってどういうこと?`.
 

apps/wisp-desktop/src/app_menu.rs

@@ -1,7 +1,9 @@
-//! macOS menu bar: application menu (About, Start/Stop Recording, Quit)
-//! plus the Cmd+Q and Cmd+R shortcuts.
+//! macOS menu bar: application menu (About, MCP Setup, recording, export,
+//! and Quit) plus their keyboard shortcuts.
 
+use std::cell::RefCell;
 use std::path::PathBuf;
+use std::rc::Rc;
 use std::sync::Arc;
 use std::time::Duration;
 
@@ -10,6 +12,7 @@ use gpui::{App, Entity, KeyBinding, Menu, MenuItem, actions};
 use crate::about_view;
 use crate::app::{AppModel, SessionState, View};
 use crate::library::SharedStorage;
+use crate::mcp_setup_view;
 use crate::session_runner::SessionRunner;
 use crate::session_updates::apply_update;
 use crate::transcript_export::{self, suggested_export_name};
@@ -19,6 +22,7 @@ actions!(
     [
         Quit,
         About,
+        OpenMcpSetup,
         ToggleRecording,
         CopyTranscript,
         ExportTranscript
@@ -31,6 +35,7 @@ pub fn configure(
     runner: Arc<SessionRunner>,
     storage: SharedStorage,
     model: Entity<AppModel>,
+    on_set_local_mcp_enabled: Arc<dyn Fn(bool, &mut App)>,
     data_dir: PathBuf,
     recordings_dir: PathBuf,
 ) {
@@ -47,6 +52,31 @@ pub fn configure(
         about_view::open(cx);
     });
 
+    let model_for_mcp_setup = model.clone();
+    let mcp_setup_window: Rc<RefCell<Option<gpui::WindowHandle<mcp_setup_view::McpSetupView>>>> =
+        Rc::new(RefCell::new(None));
+    cx.on_action(move |_: &OpenMcpSetup, cx| {
+        let existing = *mcp_setup_window.borrow();
+        if let Some(handle) = existing {
+            if cx.active_window() == Some(handle.into()) {
+                return;
+            }
+            if handle
+                .update(cx, |_, window, _| window.activate_window())
+                .is_ok()
+            {
+                return;
+            }
+        }
+
+        let window = mcp_setup_view::open(
+            cx,
+            model_for_mcp_setup.clone(),
+            on_set_local_mcp_enabled.clone(),
+        );
+        *mcp_setup_window.borrow_mut() = Some(window);
+    });
+
     // Start/stop recording straight from the menu bar (and Cmd+R), so the
     // user doesn't have to reach for the in-window Record button. Reuses the
     // same state machine as that button via `toggle_recording`.
@@ -85,6 +115,7 @@ pub fn configure(
 
     cx.bind_keys([
         KeyBinding::new("cmd-q", Quit, None),
+        KeyBinding::new("cmd-,", OpenMcpSetup, None),
         KeyBinding::new("cmd-r", ToggleRecording, None),
         KeyBinding::new("cmd-shift-c", CopyTranscript, None),
         KeyBinding::new("cmd-shift-e", ExportTranscript, None),
@@ -136,6 +167,8 @@ fn build_menus(record_label: &'static str) -> Vec<Menu> {
         items: vec![
             MenuItem::action("About Wisp", About),
             MenuItem::separator(),
+            MenuItem::action("MCP Setup…", OpenMcpSetup),
+            MenuItem::separator(),
             MenuItem::action(record_label, ToggleRecording),
             MenuItem::separator(),
             MenuItem::action("Copy Transcript", CopyTranscript),
@@ -176,10 +209,37 @@ fn graceful_stop_session(
     }
 
     let updates = runner.wait_for_idle(Duration::from_secs(5));
-    let _ = model.update(cx, |m, cx| {
+    model.update(cx, |m, cx| {
         for update in updates {
             apply_update(update, m, storage);
         }
         cx.notify();
     });
 }
+
+#[cfg(test)]
+mod tests {
+    use gpui::MenuItem;
+
+    use super::{OpenMcpSetup, build_menus};
+
+    #[test]
+    fn menu_exposes_mcp_setup_action() {
+        let menus = build_menus("Start Recording");
+        let menu = menus.first().expect("Wisp menu");
+
+        let actions = menu
+            .items
+            .iter()
+            .filter_map(|item| match item {
+                MenuItem::Action { name, action, .. } if name.as_ref() == "MCP Setup…" => {
+                    Some(action.as_ref())
+                },
+                _ => None,
+            })
+            .collect::<Vec<_>>();
+
+        assert_eq!(actions.len(), 1);
+        assert!(actions[0].as_any().is::<OpenMcpSetup>());
+    }
+}

apps/wisp-desktop/src/main.rs

@@ -39,6 +39,7 @@ mod app;
 mod app_menu;
 mod ipc_server;
 mod library;
+mod mcp_setup_view;
 mod permissions;
 mod session_runner;
 mod session_updates;
@@ -108,15 +109,31 @@ fn main() {
             start_local_mcp_bridge(&ipc_handle, &ipc_snapshot, &model, cx);
         }
 
+        let on_set_local_mcp_enabled: Arc<dyn Fn(bool, &mut App)> = {
+            let ipc_handle = ipc_handle.clone();
+            let ipc_snapshot = ipc_snapshot.clone();
+            let model = model.clone();
+            let data_dir = data_dir.clone();
+            Arc::new(move |enabled, cx| {
+                set_local_mcp_bridge_enabled(
+                    enabled,
+                    &ipc_handle,
+                    &ipc_snapshot,
+                    &model,
+                    &data_dir,
+                    cx,
+                );
+            })
+        };
+
         let window = open_main_window(
             cx,
             window_options,
             MainWindowDeps {
                 runner: runner.clone(),
                 storage: storage.clone(),
                 model: model.clone(),
-                ipc_handle: ipc_handle.clone(),
-                ipc_snapshot: ipc_snapshot.clone(),
+                on_set_local_mcp_enabled: on_set_local_mcp_enabled.clone(),
                 data_dir: data_dir.clone(),
                 recordings_dir: recordings_dir.clone(),
             },
@@ -127,6 +144,7 @@ fn main() {
             runner.clone(),
             storage.clone(),
             model.clone(),
+            on_set_local_mcp_enabled,
             data_dir.clone(),
             recordings_dir,
         );
@@ -142,8 +160,7 @@ struct MainWindowDeps {
     runner: Arc<SessionRunner>,
     storage: SharedStorage,
     model: Entity<AppModel>,
-    ipc_handle: Arc<Mutex<Option<ipc_server::IpcServer>>>,
-    ipc_snapshot: ipc_server::SharedSnapshot,
+    on_set_local_mcp_enabled: Arc<dyn Fn(bool, &mut App)>,
     data_dir: PathBuf,
     recordings_dir: PathBuf,
 }
@@ -157,8 +174,7 @@ fn open_main_window(
         runner,
         storage,
         model,
-        ipc_handle,
-        ipc_snapshot,
+        on_set_local_mcp_enabled,
         data_dir,
         recordings_dir,
     } = deps;
@@ -175,11 +191,8 @@ fn open_main_window(
             let storage_for_open_history = storage.clone();
             let data_for_toggle = data_dir.clone();
             let data_for_download = data_dir.clone();
-            let data_for_local_mcp = data_dir.clone();
             let recordings_for_toggle = recordings_dir.clone();
             let runner_for_toggle = runner.clone();
-            let ipc_for_local_mcp = ipc_handle.clone();
-            let ipc_snapshot_for_local_mcp = ipc_snapshot.clone();
             let (transcript_list, follow_transcript) = new_transcript_list_state();
             let view = TranscriptView {
                 app: model.clone(),
@@ -238,13 +251,8 @@ fn open_main_window(
                     });
                 }),
                 on_toggle_local_mcp: Arc::new(move |_window, cx| {
-                    toggle_local_mcp_bridge(
-                        &ipc_for_local_mcp,
-                        &ipc_snapshot_for_local_mcp,
-                        &model_for_local_mcp,
-                        &data_for_local_mcp,
-                        cx,
-                    );
+                    let enabled = !model_for_local_mcp.read(cx).local_mcp.enabled;
+                    on_set_local_mcp_enabled(enabled, cx);
                 }),
             };
             // Re-render whenever the underlying model changes.
@@ -357,17 +365,21 @@ fn spawn_ipc_snapshot_sync(
     .detach();
 }
 
-fn toggle_local_mcp_bridge(
+fn set_local_mcp_bridge_enabled(
+    enabled: bool,
     ipc_handle: &Arc<Mutex<Option<ipc_server::IpcServer>>>,
     snapshot: &ipc_server::SharedSnapshot,
     model: &Entity<AppModel>,
     data_dir: &std::path::Path,
     cx: &mut App,
 ) {
-    if model.read(cx).local_mcp.enabled {
-        stop_local_mcp_bridge(ipc_handle, model, cx);
-    } else {
+    if enabled {
+        if model.read(cx).local_mcp.running {
+            return;
+        }
         start_local_mcp_bridge(ipc_handle, snapshot, model, cx);
+    } else {
+        stop_local_mcp_bridge(ipc_handle, model, cx);
     }
     save_local_mcp_settings(data_dir, model, cx);
 }