History

Merge remote-tracking branch 'origin/codex/effect-lint-mcp-connection' into codex/effect-lint-integration

Rhys Sullivan committed 6 months ago

Commit 2d588fc2 parents 4a99018 ea3d6821 file · +4 −8Browse files

Unified · Split

packages/plugins/mcp/src/sdk/connection.ts+4−8
@@ -87,12 +87,10 @@ const connectClient = (input: { · 86 unmodified lines
8787
8888 yield* Effect.tryPromise({
8989 try: () => client.connect(transportInstance),
90− catch: (cause) =>
90+ catch: () =>
9191 new McpConnectionError({
9292 transport: input.transport,
93− message: `Failed connecting via ${input.transport}: ${
94− cause instanceof Error ? cause.message : String(cause)
95− }`,
93+ message: `Failed connecting via ${input.transport}`,
9694 }),
9795 }).pipe(
9896 Effect.withSpan("plugin.mcp.connection.handshake", {
@@ -124,12 +122,10 @@ export const createMcpConnector = (input: ConnectorInput): McpConnector => { · 25 unmodified lines
124122 // `node:child_process`) is only loaded when stdio is actually used.
125123 const { createStdioTransport } = yield* Effect.tryPromise({
126124 try: () => import("./stdio-connector"),
127− catch: (cause) =>
125+ catch: () =>
128126 new McpConnectionError({
129127 transport: "stdio",
130− message: `Failed to load stdio transport module: ${
131− cause instanceof Error ? cause.message : String(cause)
132− }`,
128+ message: "Failed to load stdio transport module",
133129 }),
134130 });
135131