Skip to content

Move MCP into Deckle.API, available at /mcp#71

Merged
jordanwallwork merged 1 commit into
masterfrom
claude/move-mcp-to-api-yCs6I
May 10, 2026
Merged

Move MCP into Deckle.API, available at /mcp#71
jordanwallwork merged 1 commit into
masterfrom
claude/move-mcp-to-api-yCs6I

Conversation

@jordanwallwork

Copy link
Copy Markdown
Owner

Eliminates the standalone Deckle.MCP service and its separate deployment
slot. MCP tools (ProjectTools, ComponentTools, DataSourceTools, FileTools)
now live in Deckle.API/McpTools/ and are served at /mcp with API key auth.

  • Add ModelContextProtocol.AspNetCore to Deckle.API
  • Add ApiKey auth scheme alongside existing cookie/Google OAuth
  • Register "ApiKey" policy so /mcp uses API key auth exclusively,
    leaving all other endpoints on the cookie path unchanged
  • Wire up AddDeckleMcpTools() in Program.cs
  • Map app.MapMcp("/mcp").RequireAuthorization("ApiKey")
  • Remove Deckle.MCP and Deckle.MCP.Tests projects from solution
  • Migrate all MCP tests into Deckle.API.Tests (Auth/ and McpTools/)
  • Add InternalsVisibleTo(Deckle.API.Tests) for McpErrors access
  • Remove mcp service from AppHost, docker-compose, and CI pipeline

https://claude.ai/code/session_014GTFHNaGGbSEWU8RUbP5nk

Eliminates the standalone Deckle.MCP service and its separate deployment
slot. MCP tools (ProjectTools, ComponentTools, DataSourceTools, FileTools)
now live in Deckle.API/McpTools/ and are served at /mcp with API key auth.

- Add ModelContextProtocol.AspNetCore to Deckle.API
- Add ApiKey auth scheme alongside existing cookie/Google OAuth
- Register "ApiKey" policy so /mcp uses API key auth exclusively,
  leaving all other endpoints on the cookie path unchanged
- Wire up AddDeckleMcpTools() in Program.cs
- Map app.MapMcp("/mcp").RequireAuthorization("ApiKey")
- Remove Deckle.MCP and Deckle.MCP.Tests projects from solution
- Migrate all MCP tests into Deckle.API.Tests (Auth/ and McpTools/)
- Add InternalsVisibleTo(Deckle.API.Tests) for McpErrors access
- Remove mcp service from AppHost, docker-compose, and CI pipeline

https://claude.ai/code/session_014GTFHNaGGbSEWU8RUbP5nk
@sonarqubecloud

Copy link
Copy Markdown

@jordanwallwork jordanwallwork merged commit f040ecf into master May 10, 2026
3 checks passed
@jordanwallwork jordanwallwork deleted the claude/move-mcp-to-api-yCs6I branch May 10, 2026 20:20
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.

2 participants