Skip to content

Conversation

@botengyao
Copy link
Member

@botengyao botengyao commented Jan 6, 2026

#39174

This allows a group policy enforcement rather than apply on each method.

Added method group classification to the MCP filter. Methods are classified into built-in groups (lifecycle, tool,
resource, prompt, notification, logging, sampling, completion, unknown) and the group is added to dynamic metadata

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Boteng Yao <boteng@google.com>
Signed-off-by: Boteng Yao <boteng@google.com>
Signed-off-by: Boteng Yao <boteng@google.com>
@repokitteh-read-only
Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #42877 was opened by botengyao.

see: more, trace.

@botengyao botengyao changed the title mcp_filter: ad mcp methods group annotations mcp_filter: add mcp methods group annotations Jan 6, 2026
@repokitteh-read-only
Copy link

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @wbpcode
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).

🐱

Caused by: #42877 was opened by botengyao.

see: more, trace.

Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao botengyao marked this pull request as ready for review January 7, 2026 15:01
@botengyao botengyao requested a review from yanavlasov as a code owner January 7, 2026 15:02
@wbpcode
Copy link
Member

wbpcode commented Jan 8, 2026

/lgtm api

@repokitteh-read-only repokitteh-read-only bot removed the api label Jan 8, 2026
@yanavlasov yanavlasov merged commit d1625ac into envoyproxy:main Jan 8, 2026
26 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.

3 participants