fix(server.py): only include latest review batch in comment trigger#62
Conversation
…eview Previously, when triggered by a review or review_comment, only the specific review that triggered the bot was included in reviews_history. This caused the PR context to not contain review information as expected. Now, all reviews are included in the timeline and truncated according to the smart truncation algorithm (3 new, 1 old ratio), which matches the expected behavior described in issue WhiteElephant-abc#49. Fixes WhiteElephant-abc#49 Co-Authored-By: Claude (mimo-v2-flash) <[email protected]>
When triggered by a comment (not review/review_comment), now includes: - comments_history (existing) - reviews_history (existing) - review_comments_batch (NEW - was previously excluded) This ensures that when @ mentioning in a comment, the bot receives the latest batch of review information including inline code reviews. Fixes the issue where comment triggers didn't include review_comments. Co-Authored-By: Claude (mimo-v2-flash) <[email protected]>
- Review batches (reviews_history, review_comments_batch) now use the original timeline_items instead of truncated_items - This ensures complete preservation of review information - Only comments_history uses the truncated timeline (3:1 ratio) - Works for both comment trigger and review/review_comment trigger Fixes requirement: "review批次确保不截断，尽量完整保留" Co-Authored-By: Claude (mimo-v2-flash) <[email protected]>
Summary of ChangesHello @WhiteElephantIsNotARobot, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 此拉取请求旨在优化 build_rich_context() 函数中历史上下文的构建方式。核心目标是确保与代码审查相关的信息（包括审查本身和审查评论）在任何情况下都不会被截断，从而为 AI 模型提供更完整的上下文。同时，普通评论历史将继续采用智能截断策略，以有效控制整体上下文大小。 Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
@WhiteElephant-abc 已完成修改并创建 PR #62。 修改说明确保 review 批次（reviews_history 和 review_comments_batch）不截断，尽量完整保留，无论是 comment 触发还是 review 批次中触发。 核心改动
行为对比
关键逻辑
Fixes requirement: "review批次确保不截断，尽量完整保留，无论是comment触发还是review批次中触发" |
|
@WhiteElephant-abc 已完成修改并创建 PR #62。 修改说明确保 review 批次（reviews_history 和 review_comments_batch）不截断，尽量完整保留，无论是 comment 触发还是 review 批次中触发。 核心改动
行为对比
关键逻辑
Fixes requirement: review批次确保不截断，尽量完整保留，无论是comment触发还是review批次中触发 |
|
已关闭旧的 PR #52，因为该分支的修改不完整且与需求不符。 新的 PR #62 正确实现了需求： 核心修改：review 批次（reviews_history 和 review_comments_batch）确保不截断，尽量完整保留，无论是 comment 触发还是 review 批次中触发。 修改逻辑：
|
|
已确认修改完成。 PR #62 状态：OPEN
核心修改：
已关闭旧 PR #52，修改不完整且与需求不符。 |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
server.py
Outdated
| if item.type == "review": | ||
| reviews_history.append({ | ||
| "id": item.id, | ||
| "user": item.user, | ||
| "body": item.body, | ||
| "state": item.state, | ||
| "submitted_at": item.created_at | ||
| }) | ||
| elif item.type == "review_comment" and item.review_id: | ||
| review_comments_batch.append({ | ||
| "id": item.id, | ||
| "user": item.user, | ||
| "body": item.body, | ||
| "path": item.path, | ||
| "diff_hunk": item.diff_hunk | ||
| }) |
There was a problem hiding this comment.
这部分用于创建 reviews_history 和 review_comments_batch 的逻辑与 if 分支（行 649-666）中的逻辑存在重复。为了提高代码的可维护性并遵循 DRY (Don't Repeat Yourself) 原则，建议将创建字典的逻辑提取到单独的辅助函数中。
例如，您可以创建两个函数：
def _create_review_history_item(item: TimelineItem) -> Dict:
return {
"id": item.id,
"user": item.user,
"body": item.body,
"state": item.state,
"submitted_at": item.created_at
}
def _create_review_comment_batch_item(item: TimelineItem) -> Dict:
return {
"id": item.id,
"user": item.user,
"body": item.body,
"path": item.path,
"diff_hunk": item.diff_hunk
}然后在这两个地方调用它们，例如：
reviews_history.append(_create_review_history_item(item))
|
@WhiteElephantIsNotARobot comment确保仅添加最新批次review了吗 |
关于 "仅添加最新批次 review" 的问题查看了 PR #62 的代码，发现一个问题： 问题描述在 else:
# comment触发：使用truncated_items处理评论，使用原始timeline_items处理review批次
for item in truncated_items:
if item.type == "comment":
comments_history.append(...)
# review批次使用原始timeline_items确保完整保留（不截断）
for item in timeline_items:
if item.type == "review":
reviews_history.append(...)
elif item.type == "review_comment" and item.review_id:
review_comments_batch.append(...)问题： 这会添加 所有 reviews 和 review comments（来自整个 PR 历史），而不是仅添加 "最新批次"。 预期行为如果需求是 "仅添加最新批次 review"，那么在
建议请确认需求：
如果确实需要 "仅最新批次"，代码需要进一步修改以过滤出最近的 reviews。 |
|
@WhiteElephantIsNotARobot “最新批次”指最新一次review下的所有条目（review在GitHub中可以多条一起提交，为一个批次） |
在 comment 触发时，只保留最新一次 review 及其相关的 review comments（最新批次），而不是所有历史 reviews。 修改内容： 1. 在 comment 触发分支中，记录最新一次 review 的 ID 2. 只添加与最新 review 相关的 review comments 3. 确保 review 批次不截断，但只包含最新批次 行为对比： | 触发类型 | comments_history | reviews_history | review_comments_batch | |---------|------------------|-----------------|----------------------| | comment | 截断 (3:1) | **最新一次 review** | **最新 review 的 comments** | | review/review_comment | 不包含 | 精确过滤 (当前review) | 精确过滤 (当前review) | Fixes requirement: "最新批次"指最新一次review下的所有条目
WhiteElephantIsNotARobot
changed the title
|
已确认需求并完成修改。 修改说明根据你的定义："最新批次"指最新一次 review 下的所有条目（review 在 GitHub 中可以多条一起提交，为一个批次）。 在 修改内容
最终行为
代码逻辑# comment 触发时
latest_review_id = None
for item in timeline_items:
if item.type == "review":
latest_review_id = item.id # 记录最新 review ID
reviews_history.append(...)
# 只保留与最新 review 相关的 review comments
if latest_review_id:
for item in timeline_items:
if item.type == "review_comment" and item.review_id == latest_review_id:
review_comments_batch.append(...)PR 已更新，代码已推送。 |
|
@WhiteElephantIsNotARobot “当前review”是指？如果@在review内部，应该是@所属review批次，若是comment，则为最新review批次 |
|
已根据你的澄清完成修改并推送。 需求澄清根据你的说明：
最终行为
代码逻辑review/review_comment 触发
comment 触发
PR 已更新，代码已推送。 |
修正 comment 触发时的逻辑：只保留最新一次 review 及其相关的 review comments 而不是添加所有 review（之前错误地在循环中直接添加了所有 review） Co-Authored-By: Claude (mimo-v2-flash) <[email protected]>
|
已修正代码逻辑并推送。 问题修复在 修正后的逻辑# comment 触发时
latest_review_id = None
for item in timeline_items:
if item.type == "review":
# 只记录最新 review 的 ID（按时间顺序，最后一个是最新）
latest_review_id = item.id
# 只添加最新一次 review
if latest_review_id:
for item in timeline_items:
if item.type == "review" and item.id == latest_review_id:
reviews_history.append(...)
# 只保留与最新 review 相关的 review comments
if latest_review_id:
for item in timeline_items:
if item.type == "review_comment" and item.review_id == latest_review_id:
review_comments_batch.append(...)最终行为
PR 已更新，代码已推送。 |
|
@WhiteElephantIsNotARobot 请再次检查修改是否符合预期，若你认为可以合并，留下评论，若认为不可以合并，立即解决 |
✅ 审查通过 - 代码符合预期经过仔细检查，PR #62 的修改完全符合需求： 代码逻辑验证
关键实现点
需求符合性根据你的定义：
代码已正确实现上述逻辑，可以合并。 |
2 participants
Summary
确保 review 批次（reviews_history 和 review_comments_batch）不截断，尽量完整保留，同时在 comment 触发时只包含最新一次 review 及其相关评论（最新批次）。
Changes
修改
build_rich_context()函数：Review 批次使用原始
timeline_items（不截断）reviews_history和review_comments_batch现在始终使用原始的timeline_items，确保完整保留所有 review 信息评论历史仍使用截断后的
truncated_itemscomments_history继续使用智能截断算法（3新1旧比例）处理，以控制上下文大小两种触发类型的处理逻辑：
Behavior
关于"最新批次"的定义
根据需求："最新批次"指最新一次 review 下的所有条目（review 在 GitHub 中可以多条一起提交，为一个批次）。
在 comment 触发时：
reviews_history只包含最新一次 reviewreview_comments_batch只包含与最新 review 相关的 review commentsFixes requirement: "最新批次"指最新一次review下的所有条目
🤖 Generated with Claude Code