Skip to content

新增工具覆盖能力 - #383

Closed
ykf173 wants to merge 1 commit into
mainfrom
feature/kaifeng.yan/mcp_cap_tooloverlap
Closed

新增工具覆盖能力#383
ykf173 wants to merge 1 commit into
mainfrom
feature/kaifeng.yan/mcp_cap_tooloverlap

Conversation

@ykf173

@ykf173 ykf173 commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator
  1. 工具描述复写
  2. 参数描述复写
  3. 工具控制
  4. 工具名称复写

@Million-mo

Copy link
Copy Markdown
Collaborator

与已合入 PR #384 的功能重复

本 PR 的功能(工具描述复写、参数描述复写、工具控制、工具名称复写)已在 PR #384 中实现并合入 main:

  • src/wolfharness/capabilities/tool_schema_overlap_capability.py
  • src/wolfharness/capabilities/tool_schema_overlap_config.py
  • docs/explanation/tool-schema-overlap-capability.md

两者在 mcp_server_cap.pytests/capabilities/test_mcp_server_cap.py 上有文件重叠。

PR 质量问题

  1. 标题不符合 conventional commit 规范 — 应为 feat(capabilities): ... 或类似格式
  2. 描述过于简略 — 仅列出 4 个 bullet,缺少问题背景、设计方案、测试说明
  3. 单 commit 包含 23 个文件 — 无法增量 review
  4. 缺少文档和 RFC 关联
  5. base 已过期 — 停在 v1 mcp tools capbility overlap #384 合入前的 commit,rebase 后会有大量冲突

建议

如果本 PR 有 #384 未覆盖的额外功能(如 resource 相关的改动),请:

  1. Rebase 到最新 main
  2. 只保留 v1 mcp tools capbility overlap #384 未覆盖的增量改动
  3. 补充标题、描述和文档

否则建议关闭。

Related: #384 #386

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.

2 participants