Refactor/simplify dsh commands - #157
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.
改动说明
将 DSH BrowserSkill 插件原有的 28 个细粒度工具合并为 6 个领域工具:
browser_sessionbrowser_pagebrowser_inspectbrowser_interactbrowser_tabsbrowser_assist原有 28 项浏览器操作作为内部 action handler 保留,浏览器小窗、会话管理、取消、截图附件等能力不变。
同时更新了:
browser_inspectToolView测试
原因:
之前按照第一版dsh插件的思路补齐了工具,命令太多 了,想了想应该简化一下,不过还是做了一下测试:

两个方案准确率没有变化。效率上28个工具更高,就是耗时更短。
但是成本上6个工具的成本更低。虽然输出token多了,但是输入token大幅减少,考虑成本,还是改成6个工具版本。