anc95 / ChatGPT-CodeReview

🐥 A code review bot powered by ChatGPT
https://github.com/apps/cr-gpt
ISC License
3.77k stars 361 forks source link

在批量提交多个commit时,似乎可能只评审其中的最后一个commit内容 #83

Open yimelia opened 1 year ago

yimelia commented 1 year ago

在同一个PR的第二次提交中,批量提交了多个commit,但bot没有产生任何回复,只有最后一个commit上面显示了一个√。 然后再次提交一个commit(修改的文件在之前的commit已经出现过),bot对该文件之前提交的内容进行了review,它原本应该出现在上一轮的提交中才对。

anc95 commented 1 year ago

确实,当同步新的 commit 时,Github的API里找不到新增的commits,因此默认只对最后一个commit所修改的文件进行review。

yimelia commented 1 year ago

https://github.com/AirtestProject/Airtest/pull/1132#pullrequestreview-1424046941

这里只提交了一个commit也只review了其中的短文件,是否文件过长不会review呢?

anc95 commented 1 year ago

@yimelia 是的 diff 过长不会 review,1是节约 token,2是openai本身也有token限制

yimelia commented 1 year ago

当长度到达多少时,不会进行review呢?或者是否可以提供配置项,以及是否可以将diff进行拆分提交来review?

anc95 commented 1 year ago

默认最大token是 4096,如果更高的话可能需要 GPT4,不是很建议,比较贵

将diff进行拆分提交来review 拆分会丢失一些上下文,可能效果不太好