diff options
| -rw-r--r-- | packages/opencode/src/mcp/index.ts | 3 | ||||
| -rw-r--r-- | packages/opencode/test/mcp/oauth-auto-connect.test.ts | 12 |
2 files changed, 9 insertions, 6 deletions
diff --git a/packages/opencode/src/mcp/index.ts b/packages/opencode/src/mcp/index.ts index 2c0fc1d37..e48a42a8b 100644 --- a/packages/opencode/src/mcp/index.ts +++ b/packages/opencode/src/mcp/index.ts @@ -401,7 +401,8 @@ export namespace MCP { // but may also throw plain Errors when auth() fails internally // (e.g. during discovery, registration, or state generation). // When an authProvider is attached, treat both cases as auth-related. - const isAuthError = error instanceof UnauthorizedError || (authProvider && lastError.message.includes("OAuth")) + const isAuthError = + error instanceof UnauthorizedError || (authProvider && lastError.message.includes("OAuth")) if (isAuthError) { log.info("mcp server requires authentication", { key, transport: name }) diff --git a/packages/opencode/test/mcp/oauth-auto-connect.test.ts b/packages/opencode/test/mcp/oauth-auto-connect.test.ts index 0cd5c36e5..76f825247 100644 --- a/packages/opencode/test/mcp/oauth-auto-connect.test.ts +++ b/packages/opencode/test/mcp/oauth-auto-connect.test.ts @@ -22,11 +22,13 @@ let simulateAuthFlow = true // Mock the transport constructors to simulate OAuth auto-auth on 401 mock.module("@modelcontextprotocol/sdk/client/streamableHttp.js", () => ({ StreamableHTTPClientTransport: class MockStreamableHTTP { - authProvider: { - state?: () => Promise<string> - redirectToAuthorization?: (url: URL) => Promise<void> - saveCodeVerifier?: (v: string) => Promise<void> - } | undefined + authProvider: + | { + state?: () => Promise<string> + redirectToAuthorization?: (url: URL) => Promise<void> + saveCodeVerifier?: (v: string) => Promise<void> + } + | undefined constructor(url: URL, options?: { authProvider?: unknown }) { this.authProvider = options?.authProvider as typeof this.authProvider transportCalls.push({ |