Skip to content

Conversation

@lightninglu10
Copy link
Contributor

No description provided.

@lightninglu10 lightninglu10 self-assigned this Dec 19, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review Summary

This PR threads existing human review comments into the autonomous agent's context.

Key changes

  • Adds an section to the system prompt so the agent understands how to handle prior comments and emoji reactions.
  • Introduces an ExistingReviewComment type and formatting helper to include prior review comments in the initial agent message.
  • Updates the GitHub Action to fetch non-bot review comments into pr-comments.json.
  • Updates ReviewService to load pr-comments.json and pass existing comments into reviewFullDiff.
  • Extends tests to cover the new reviewFullDiff signature and default empty existingComments when there is no pr-comments.json.

Review notes

  • The new ExistingReviewComment type is consistently used in src/index.ts, src/review-service.ts, src/agent/index.ts, and tests.
  • Filtering out bot comments in src/index.ts looks correct and should avoid the agent reacting to its own output.
  • The formatted block appears well-structured and matches the new policy in the system prompt.
  • Tests for ReviewService correctly assert the additional existingComments argument.

No blocking issues found; implementation is cohesive and consistent with existing patterns.

@lightninglu10 lightninglu10 merged commit c20f048 into main Dec 19, 2025
4 checks passed
@lightninglu10 lightninglu10 deleted the existing-comments branch December 19, 2025 06:59
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.

2 participants