Skip to content

Conversation

@WhiteElephantIsNotARobot
Copy link
Owner

Summary

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.

Fix

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:

  • Should contain the latest batch of reviews
  • Or contain all reviews but truncate them according to the timeline

Changes

  • Modified build_rich_context() function in server.py
  • Removed the filter that only included the specific triggered review
  • Now all reviews in the truncated timeline are included when triggered by review/review_comment

Fixes WhiteElephant-abc#49

🤖 Generated with Claude Code

…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) <noreply@anthropic.com>
@WhiteElephantIsNotARobot
Copy link
Owner Author

Fixes WhiteElephant-abc#49 - 已修复 server.py 中 review 上下文缺失的问题。现在当被 review 或 review_comment 触发时,所有 review 都会包含在上下文中(按时间线截断)。PR: #2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pr上下文不包含review

1 participant