Skip to content

Add security policy for AI endpoints#2111

Merged
MarceloRGonc merged 3 commits intomainfrom
mg/security-ai
Mar 11, 2026
Merged

Add security policy for AI endpoints#2111
MarceloRGonc merged 3 commits intomainfrom
mg/security-ai

Conversation

@MarceloRGonc
Copy link
Contributor

@MarceloRGonc MarceloRGonc commented Mar 11, 2026

Fixes OPS-3894.

Copilot AI review requested due to automatic review settings March 11, 2026 14:54
@linear
Copy link

linear bot commented Mar 11, 2026

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds project-scoped route security policies to AI-related API endpoints so they are consistently tagged with the same authorization policy metadata used across the rest of the API.

Changes:

  • Add getProjectScopedRoutePolicy(...) to AI config routes.
  • Add getProjectScopedRoutePolicy(...) to MCP config routes.
  • Add getProjectScopedRoutePolicy(...) to MCP chat and related AI chat routes.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
packages/server/api/src/app/mcp/config/mcp-config.controller.ts Applies a project-scoped route security policy to MCP config CRUD routes.
packages/server/api/src/app/ai/config/ai-config.controller.ts Applies a project-scoped route security policy to AI config routes (save/list/get/delete).
packages/server/api/src/app/ai/chat/ai-mcp-chat.controller.ts Applies a project-scoped route security policy to AI MCP chat endpoints (open/send/rename/delete/list/update model/code generation).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sonarqubecloud
Copy link

@MarceloRGonc MarceloRGonc merged commit 9eb2644 into main Mar 11, 2026
21 checks passed
@MarceloRGonc MarceloRGonc deleted the mg/security-ai branch March 11, 2026 16:34
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.

3 participants