docs: 增加 cad_mcp_frame 的移植说明与代码审查 #108
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: gjm/envi-code#108
Reference in new issue
Block a user
移植已完成（R230 ARX + Z2026 ZRX 均编译通过并部署），本文记录：
P0 收包只 recv 一次、缓冲固定 2048：大报文截断、TCP 分包即失败，且没有长度前缀
P0 bytesReceived<=0 时 socket 泄漏
P1 socket/bind/listen 返回值全未检查，端口占用时静默失败但 g_bRunning 仍为 true
P1 无鉴权：本机任意进程都能调用会改图的工具
P1 端口 8080 硬编码；代码找 mcp_config.json 而仓库里是 cad-mcp-frame.json
P2 卸载时挂起的 PostMessage 任务不释放；LISP 工具是异步 fire-and-forget 无结果回传；
加载期弹 AfxMessageBox 会卡住 CAD 启动；g_hMsgWnd 无同步；
.def 的 LIBRARY 名与实际输出不一致；/Zc:forScope 语义与 code2026 全局设置相反
移植已完成（R230 ARX + Z2026 ZRX 均编译通过并部署），本文记录： - 移植做了哪些改动（cadsdk.h->KTCad.h、两个单头依赖、SUB_ARX、中望 .def 与版本号函数） - C++11+ 限制：R180/R190 是 C++98，无法编译 nlohmann/json，故按 preset 条件跳过 - 架构：后台线程收包 -> PostMessage -> 主线程执行，工具调用全在主线程，避开了跨线程访问数据库 - 问题清单（按严重度）： P0 收包只 recv 一次、缓冲固定 2048：大报文截断、TCP 分包即失败，且没有长度前缀 P0 bytesReceived<=0 时 socket 泄漏 P1 socket/bind/listen 返回值全未检查，端口占用时静默失败但 g_bRunning 仍为 true P1 无鉴权：本机任意进程都能调用会改图的工具 P1 端口 8080 硬编码；代码找 mcp_config.json 而仓库里是 cad-mcp-frame.json P2 卸载时挂起的 PostMessage 任务不释放；LISP 工具是异步 fire-and-forget 无结果回传； 加载期弹 AfxMessageBox 会卡住 CAD 启动；g_hMsgWnd 无同步； .def 的 LIBRARY 名与实际输出不一致；/Zc:forScope 语义与 code2026 全局设置相反 - 移植后仍需处理：统一配置文件名、移植 cad_mcp_plugins、带上 lisp 目录、在 CAD 里实测P0 收包协议（大请求被截断 + 分包即失败） 原来只 recv 一次、缓冲固定 2048 字节。改为： - IsJsonComplete()：按 {} 与 [] 配平判断报文完整性（跳过字符串内与转义字符） - RecvOneRequest()：select + 循环 recv 追加，单次等 5s、整请求 30s、上限 4MB - SendAll()：send 循环到发完（tools/list 应答可能超一个包） 不改客户端协议：客户端仍是 sendall 后等响应断开。 P0 socket 泄漏 recv<=0 的分支原来什么都不做。现在收包失败/超时回一个 JSON-RPC 错误帧并 closesocket； accept 失败直接 continue（原代码对 INVALID_SOCKET 也调 closesocket）。 P1 监听失败不再静默 socket/WSASocket/bind/listen/CreateWindowEx/std::thread 全部检查返回值， 监听 socket 改在主线程创建，失败时 init_mcp_server() 返回 false 并打印 WSAGetLastError， 端口被占用时能看到 "Fatal: bind 127.0.0.1:8080 failed, err=10048"。 P1 端口与配置文件名可配 - server.port 可配（默认 8080） - 配置先找 mcp_config.json，再找 cad-mcp-frame.json（此前代码只找前者， 而仓库里带的是后者，导致加载即自关） - 启动日志打印实际端口与 token 是否启用 P1 可选 token 鉴权 配置写了 server.token 就要求请求根级带 token，不匹配返回 -32001 且不执行工具； 没配 token 保持原行为，不破坏现有客户端。 顺带 P2：close_mcp_server() 先排空挂起的 WM_MCP_EXECUTE_TASK 并 delete TaskData （原来会漏 socket + 内存），再销毁窗口。 验证：R230 与 Z2026 Debug 均编译通过（818->826 KB，已部署）。 未做：没有在 CAD 里实际跑 tools/list 与 tools/call 验证协议改动。b14ed8c5b8tob9c4ce56f9端到端实测（/b 脚本按 AecRxBase -> CoreCommand -> cad_mcp_frame 顺序加载） 暴露两个真问题： 1) 版本校验拿到垃圾值，15/16 个工具被误踢 原实现 GetProcAddress(hMod,"acrxGetApiVersion") 转函数指针调用， 和基座自己的 acrxGetApiVersion() 比数值。实测两边返回值是乱的 （1507328 / 2052390912 / 0 交替，其中 1507328=0x170000 才是正确的 23.0）， 结果只有第 3 个工具加载成功，其余全被判 SDK 版本不匹配。 改为插件导出数据符号 g_mcpPluginApiVersion=(ARX<<16)|SUB_ARX （mcp_plugin_api.h 新增 EXPORT_MCP_PLUGIN_VERSION()，每个插件 DLL 调用一次）， 基座 GetProcAddress 读内存比对，不涉及调用约定；旧插件没有该符号时只告警按兼容处理。 2) 加载期 AfxMessageBox 会卡住主线程 mcp_ConfigParser 里的几处弹窗全部换成 McpLog： 同时写 %TEMP%\mcp_plugin_load.log 和命令行，并补上每个工具的 [try]/[skip] 明细（dll 是否存在、版本、create/destroy 指针）。 这也是"看起来只加载 3 个工具"的原因——弹窗把主线程挡住了。 实测结果（R230，自动化烟测）： 端口 127.0.0.1:8080 6 s 内监听 tools/list 返回 18 个工具（2 lisp_inline + 16 c++_dll） 5 KB 大请求 -32601，证明整包被读完（旧实现会截断） tools/call get_layers 真正执行插件，返回 {"count":1,"items":[{"name":"0",...}]} 加载日志 16 个 c++_dll 全部 [try] 成功，0 个 [skip] 另外把配置文件加入构建部署：copyarx.exe 能拷任意文件，照 set_debug_post_build 的写法 再加一条 POST_BUILD，把 cad-mcp-frame.json 也拷到部署目录（ZWCAD 走 zw_output_path.json）。 不部署的话 frame 起来后 LoadAndRegister 找不到配置会直接自关。 验证：R230 与 Z2026 的 frame + plugins 均编译通过并部署；R230 已做上述端到端实测。