Skip to content

Commit 176b496

Browse files
authored
fix: Remove hacky desktop connection, stand alone mcp install (#408)
1 parent 5f11da4 commit 176b496

3 files changed

Lines changed: 6 additions & 22 deletions

File tree

src/steps/add-mcp-server-to-clients/index.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import { analytics } from '../../utils/analytics';
55
import { getUI } from '../../ui';
66
import { MCPClient } from './MCPClient';
77
import { CursorMCPClient } from './clients/cursor';
8-
import { ClaudeMCPClient } from './clients/claude';
98
import { ClaudeCodeMCPClient } from './clients/claude-code';
109
import { VisualStudioCodeClient } from './clients/visual-studio-code';
1110
import { ZedClient } from './clients/zed';
@@ -16,7 +15,6 @@ import { debug } from '../../utils/debug';
1615
export const getSupportedClients = async (): Promise<MCPClient[]> => {
1716
const allClients = [
1817
new CursorMCPClient(),
19-
new ClaudeMCPClient(),
2018
new ClaudeCodeMCPClient(),
2119
new VisualStudioCodeClient(),
2220
new ZedClient(),

src/ui/tui/screen-registry.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,14 +67,13 @@ export function createScreens(
6767

6868
// Standalone MCP flows
6969
[Screen.McpAdd]: (
70-
<McpScreen store={store} installer={services.mcpInstaller} standalone />
70+
<McpScreen store={store} installer={services.mcpInstaller} />
7171
),
7272
[Screen.McpRemove]: (
7373
<McpScreen
7474
store={store}
7575
installer={services.mcpInstaller}
7676
mode="remove"
77-
standalone
7877
/>
7978
),
8079
};

src/ui/tui/screens/McpScreen.tsx

Lines changed: 5 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,6 @@ interface McpScreenProps {
3333
store: WizardStore;
3434
installer: McpInstaller;
3535
mode?: McpMode;
36-
/** When true, exit the process after completion instead of routing to outro. */
37-
standalone?: boolean;
3836
}
3937

4038
enum Phase {
@@ -51,19 +49,14 @@ const markDone = (
5149
store: WizardStore,
5250
outcome: McpOutcome,
5351
clients: string[] = [],
54-
standalone = false,
5552
) => {
5653
store.setMcpComplete(outcome, clients);
57-
if (standalone) {
58-
process.exit(0);
59-
}
6054
};
6155

6256
export const McpScreen = ({
6357
store,
6458
installer,
6559
mode = 'install',
66-
standalone = false,
6760
}: McpScreenProps) => {
6861
useSyncExternalStore(
6962
(cb) => store.subscribe(cb),
@@ -87,20 +80,14 @@ export const McpScreen = ({
8780
const detected = await installer.detectClients();
8881
if (detected.length === 0) {
8982
setPhase(Phase.None);
90-
setTimeout(
91-
() => markDone(store, McpOutcome.NoClients, [], standalone),
92-
1500,
93-
);
83+
setTimeout(() => markDone(store, McpOutcome.NoClients), 1500);
9484
} else {
9585
setClients(detected);
9686
setPhase(Phase.Ask);
9787
}
9888
} catch {
9989
setPhase(Phase.None);
100-
setTimeout(
101-
() => markDone(store, McpOutcome.Failed, [], standalone),
102-
1500,
103-
);
90+
setTimeout(() => markDone(store, McpOutcome.Failed), 1500);
10491
}
10592
})();
10693
}, [installer]); // eslint-disable-line
@@ -126,7 +113,7 @@ export const McpScreen = ({
126113
};
127114

128115
const handleSkip = () => {
129-
markDone(store, McpOutcome.Skipped, [], standalone);
116+
markDone(store, McpOutcome.Skipped);
130117
};
131118

132119
const doInstall = async (names: string[], features?: string[]) => {
@@ -141,7 +128,7 @@ export const McpScreen = ({
141128
setPhase(Phase.Done);
142129
const outcome =
143130
result.length > 0 ? McpOutcome.Installed : McpOutcome.Failed;
144-
setTimeout(() => markDone(store, outcome, result, standalone), 2000);
131+
setTimeout(() => markDone(store, outcome, result), 2000);
145132
};
146133

147134
const doRemove = async () => {
@@ -156,7 +143,7 @@ export const McpScreen = ({
156143
setPhase(Phase.Done);
157144
const outcome =
158145
result.length > 0 ? McpOutcome.Installed : McpOutcome.Failed;
159-
setTimeout(() => markDone(store, outcome, result, standalone), 2000);
146+
setTimeout(() => markDone(store, outcome, result), 2000);
160147
};
161148

162149
return (

0 commit comments

Comments
 (0)