Skip to content

[codex] Remove OAuth and MCP support - #42

Merged
hhhjin merged 1 commit into
mainfrom
codex/remove-oauth-mcp-support
Apr 11, 2026
Merged

[codex] Remove OAuth and MCP support#42
hhhjin merged 1 commit into
mainfrom
codex/remove-oauth-mcp-support

Conversation

@hhhjin

@hhhjin hhhjin commented Apr 11, 2026

Copy link
Copy Markdown
Member

Summary

  • remove the Better Auth OAuth provider and all MCP server routes, handlers, and supporting client pages
  • drop the OAuth/JWKS tables from the auth schema and add a migration that removes them from existing databases
  • remove the OAuth/MCP-specific tests and docs references that no longer apply

Why

This change removes the OAuth-backed MCP integration surface from mdto so the codebase, schema, and documentation match the currently supported product surface.

Impact

The app continues to support the existing Better Auth sign-in flow and API key functionality, but no longer exposes OAuth2, JWKS, or MCP endpoints.

Validation

  • pnpm lint:fix
  • pnpm test
  • pnpm build (client/server builds completed, but prerender failed to start the Vite preview server in the local environment)

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the Model Context Protocol (MCP) server integration and its associated OAuth provider functionality, including UI routes, database schemas, and related dependencies. The review suggests improving the authentication configuration to support dynamic origins for local development and ensuring the migration script is idempotent by using DROP TABLE IF EXISTS statements.

Comment thread server/lib/auth.ts
Comment thread migrations/0005_tranquil_imperial_guard.sql
@hhhjin
hhhjin marked this pull request as ready for review April 11, 2026 06:43
@hhhjin
hhhjin merged commit f2472e2 into main Apr 11, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant