Tighten the public API surface: root exports, listen types, dead-code removal - #3203
Open
maxisbey wants to merge 1 commit into
Open
Tighten the public API surface: root exports, listen types, dead-code removal#3203maxisbey wants to merge 1 commit into
maxisbey wants to merge 1 commit into
Claude / Claude Code Review
completed
Jul 28, 2026 in 13m 11s
Code review found 1 important issue
Found 2 candidates, confirmed 1. See review comments for details.
Details
| Severity | Count |
|---|---|
| 🔴 Important | 1 |
| 🟡 Nit | 0 |
| 🟣 Pre-existing | 0 |
| Severity | File:Line | Issue |
|---|---|---|
| 🔴 Important | docs/migration.md:2780-2783 |
Context privatization breaking change missing from docs/migration.md |
Annotations
Check failure on line 2783 in docs/migration.md
claude / Claude Code Review
Context privatization breaking change missing from docs/migration.md
The breaking change moving the lowlevel `Context` class from the public `mcp.server.context` module to the private `mcp.server._context` module is not documented in `docs/migration.md`, even though the PR description states it is. AGENTS.md requires all breaking changes to be documented there — please add a short migration entry (grouped with the existing lowlevel-server/context sections) noting the class is now private and that `from mcp.server.context import Context` raises `ImportError`.
Loading