Repository navigation
feat(core,client,server): ttl on list results (SEP-2549) - #2070
Closed
felixweinberger wants to merge 3 commits into
Closed
felixweinberger wants to merge 3 commits into
felixweinberger wants to merge 3 commits into
Claude / Claude Code Review
completed
May 12, 2026 in 17m 58s
Code review found 1 potential issue
Found 5 candidates, confirmed 1. See review comments for details.
Details
| Severity | Count |
|---|---|
| 🔴 Important | 0 |
| 🟡 Nit | 1 |
| 🟣 Pre-existing | 0 |
| Severity | File:Line | Issue |
|---|---|---|
| 🟡 Nit | packages/core/src/types/schemas.ts:616-622 |
Missing changeset for new public API across core/server/client |
Annotations
Check warning on line 622 in packages/core/src/types/schemas.ts
claude / Claude Code Review
Missing changeset for new public API across core/server/client
This PR adds public API to three published packages — `@modelcontextprotocol/core` (`PaginatedResult.ttl`), `@modelcontextprotocol/server` (`ServerOptions.listTtlSeconds`), and `@modelcontextprotocol/client` (`Client.isToolCacheStale`) — but ships no `.changeset/*.md` entry (changeset-bot confirms "No Changeset found" at b8536fd). Without one, `changesets/action` will not bump versions or write CHANGELOG entries for any of these packages, so SEP-2549 ships silently; add a minor-bump changeset fo
Loading