Repository navigation
fix(minimax-bot): 接力交接带上本轮各处的结论,不只是最后一句的尾巴 - #108
Open
Apageoflove wants to merge 1 commit into
Open
Apageoflove wants to merge 1 commit into
Apageoflove wants to merge 1 commit into
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.
群里一个 Bot 答完、轮到下一个时,交接块里带的是上一条回复的尾巴(最后 400 字)。一轮里跳了三四个的时候,下一位其实只看得见上一位说了什么,前面几位各自得出了什么结论,得自己回头翻聊天记录。
这一版改成带本轮到目前为止每一位的结论,每条前面标出是谁说的,超出预算就丢掉最旧的几条。真实跑了一遍三个 Bot 的接力:改之前第三位收到的块里只有第二位的尾巴,改之后第一位和第二位的结论都在里面。
只有一位说过的时候仍然是原来那句话,逐字节不变 —— 这一点专门做了对照,第一个接着说的那位收到的块和改之前完全一样。多于一人的时候换个说法,因为几句话不能算成一个人说的,所以块里不再写「上一位说」,改成「本轮到目前为止各位的结论」,并注明这是一份记录、不是已达成的共识。设置面板里那两行提示也跟着改了,免得用户看到的说明和实际行为对不上。
每条最多 240 字,整块最多 600 字,超了整条丢最旧的,不做半条拼接。
新加的一个测试文件里有八条用例:单跳逐字节不变;多跳用新的说法且不出现「上一位」;每条不超过 240 字、整块不超过 600 字、超了丢最旧的;群纪要开不开都不影响交接块;伪造的标记会被洗掉;真跑两轮群聊,第二轮的交接里不出现第一轮的话;第一跳没有交接块。插件自带套件 373 条全过,上游 npm run check 两半也过,这八条在未改的上游代码上有七条是红的。
这套只在本地把三个 Bot 拉起来跑过接力,别人的环境里没验过,这点未验证。一轮里人多的时候更早的结论会被丢掉,这是上面那个 240 字和 600 字决定的。
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.