Repository navigation
refactor: 收紧 Agent 运行时边界 - #57
Merged
Merged
Conversation
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.
变更\n\n- 在 run 接纳时快照排队消息、执行上下文与 signal 引用，避免跨异步边界被调用方修改\n- 拆分模型 attempt 与单次工具执行，统一 observer 通知和正整数校验\n- 在通知 observer 前复制并冻结模型流事件，防止观察者反向修改 Core 状态\n- 统一父取消与局部 deadline 的错误语义，拒绝模型或工具在超时后的迟返成功\n- 在持久化前校验、复制并冻结 ToolResult，拒绝错误 callId 与非法 JSON 载荷，并保证调用终态 observation 恰好一次\n\n## 验证\n\n- pnpm run check\n- Vitest: 31 passed / 1 skipped；352 passed / 10 skipped\n- 独立 Review Agent 已批准，无阻断项