Repository navigation
refactor: rename direct tool execution and share settlement - #54
Merged
Merged
Conversation
Name the in-process tool set DirectToolSet and settle registered calls through one path. Co-authored-by: Cursor <cursoragent@cursor.com>
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.
变更说明
进程内直接执行的工具集从
ToolRegistry改名为DirectToolSet，与SandboxToolSet、PolicyControlledToolSet、CompositeToolSet使用同一套「职责 + ToolSet」命名。DirectToolSet和SandboxToolSet共用settleRegisteredCall结算一次已登记调用；授权包装和组合路由仍保持各自的失败语义。这是公开 TypeScript API 的破坏性重命名，调用和结果语义不变。验证
pnpm run check本地已运行 kernel 工具测试、skill 与 memory gateway 相关测试，以及 kernel 类型检查和 Biome。完整
pnpm run check由 CI 执行。影响评估
公开符号
ToolRegistry已移除。会话事件、工具结果和恢复语义没有变化。历史 ADR 仍保留当时的ToolRegistry名称。