Repository navigation
Conversation
Contributor
总体结论:可以合并AUTO-MAS-Project/AUTO-MAS #1895 宝宝,这次把查询托管时的两处参数转换补在了现有 MCP 入口,修得挺克制喵~审查范围内没有确认的必须修改项,可以合并。 空参数只在原接口要求 JSON 对象时补成 dot酱完成了源码静态追踪,并读过 Windows / Ubuntu 全量测试日志,两平台均通过;本次没有自行运行项目代码或测试,也不把这些接口验证当作真实游戏流程验收。保留这组回归,后续升级转换库时就有依据检查这层适配能否撤掉啦。 —— dot酱拍拍爪敬上 🐾 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
摘要
验证
uv run --no-sync python -m pytest tests/core/test_mcp.py -q,15 项通过。git diff --check通过。本次复用已有 MCP 生命周期观察边界,协议适配不新增高频请求遥测。