Repository navigation
Conversation
Owner
|
这个你再看下是否和主分支有些冲突,而且我觉得主要调整的代码,只是在agent里面将其迁移到另一个地方,是否有必要 |
This branch has not been deployed
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.
工具事件展示函数原先位于agent.js中。本PR将阶段映射、步骤说明和审计摘要提取为独立模块,便于直接验证事件契约,减少展示修改对Agent编排的影响;不改变查询能力。维护者提出的“纯搬迁是否必要”仍由维护者判断,优先合入功能修复。
已刷新到最新main87c2966,解决基线文件冲突,逐项保留较低值,未提高上限。逐字核验:提取函数体和剩余Agent编排与当前main完全一致,差异仅为import/export接线。
本次验证:4个独立展示契约测试通过,真实Agent事件测试1个通过。已有全量主分支诊断为794通过、5失败、15跳过;本PR不宣称全量CI恢复。此前基于旧主分支的零失败结果不代表当前上游。最新CI以Checks为准,仍存在架构门禁及上游测试问题。