Repository navigation
Implement #185: [MCP Sink] 支持从零开始发现 InKCre 类型、资源与数据 - #189
Conversation
已实现 #185,候选提交
验证:
影响发布项目:core-py。无外部凭据/生产部署变更。风险:普通事务内索引构建可能阻塞 Block 写入,大库需维护窗口。回滚可撤回应用 Tool,保留或独立删除新增索引,不删除业务数据。 请按仓库 required checks 完成 PR 审阅/合并;未执行真实 ChatGPT/tunnel 或生产环境验收。当前 Braid |
已推送修复
在 disposable PostgreSQL 中复现 CI 的角色分离后, 新提交的 GitHub required checks 待完成;原有索引维护窗口风险与回滚方案不变。请维护者在 checks 通过后审阅/验收,尚未合并或部署。 |
xiaoland
left a comment
There was a problem hiding this comment.
Please address the inline finding: the current Resolver catalog must come from the peer-local registry, not from persisted Block values.
实现范围已按 #185 的四条新 Human 评论更新;当前验收基线见 #185 (comment) 。请在本 PR 继续实现,不另建 PR:
这是对现有实现的后续修订任务;旧 CI/手动探测尚未证明新验收完成。Issue 工作树本轮仅做设计/交接,未修改产品代码。 |
|
@braid 请继续在本 PR 实现 #185 更新后的验收基线,按 Issue 评论 #185 (comment) 和上方 Braid 交接评论执行。完成代码、必要测试、文档与 release fragment 后推送到当前 PR,并在这里报告验证结果及剩余风险。 |
已按 #185 当前验收基线完成修订并推送
验证:
剩余风险:Client 需刷新 tools;事务内索引构建可能阻塞写入,大库需维护窗口;分页非快照、AND 成本取决于选择性。回滚应用可恢复旧工具面,索引可保留或单独 downgrade,不删业务数据。未执行真实 ChatGPT/tunnel 或生产验收,未合并/部署;新 HEAD 的 required checks 待完成。 |
xiaoland
left a comment
There was a problem hiding this comment.
Two follow-up items to address before merge: align the internal query-mode terminology, and make the MCP Sink lifecycle E2E verify public transport behavior rather than private implementation state.
已处理本轮两条评审意见,推送
已复核远端 HEAD 并重跑验证:disposable PostgreSQL 上 无新增迁移或运行时权限变化;既有索引维护窗口、非快照分页及 ChatGPT/tunnel 手动验收风险不变。未合并或生产部署。 |
合并收尾:已核对 #189 已合并,#185 已完成关闭。最终实现与验证记录见 #189 (comment) 。发布时仍需关注索引迁移的写入阻塞风险,并刷新 MCP Client 的 tools;真实 ChatGPT/tunnel 与生产验收尚未执行。本轮仅核对收尾状态,未追加代码或重跑测试。 |
Closes #185
Implementation request: #185 (comment)