Return MergedList<T> from list_all_* and drop the _with_cache_hint variants

未关闭
#1,430 0 条评论 0 个 reaction 已指派 1 人 在 GitHub 查看

维护者通常 3 天内回复

@DaleSeo 已经在做这个了。

开始于 2026年11月3日。

评估

这个 Issue 还没有评估数据。

描述

enhancement P3 ready for work T-service v4

#1429 (for #1411) adds Peer::list_all_*_with_cache_hint() methods next to the existing list_all_* methods. They return a MergedList<T> { items, ttl_ms, cache_scope } so callers can cache a merged list. The new methods sit beside the old ones only because changing the return type of list_all_* is a breaking change.

In the next major release, list_all_* could return MergedList<T> directly, and the five _with_cache_hint methods would be removed. That leaves one method per list kind instead of two. Callers that only need the items would write .items.

This is a breaking change, so it should ship with the other changes in the next major release rather than in a major release of its own.

主要语言
Rust
星标
4.1k
派生
694
平均合并
4 天 16 小时
30 天内合并 PR
43

环境准备

在 Codespaces 中打开

在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

modelcontextprotocol/rust-sdk 的其他 Issue

查看 modelcontextprotocol/rust-sdk 的全部 Issue

相似的 Issue

更多 Rust Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。