commit 2963741f3938ba6bfef02c615be91d3e58b7a0b3
author Patrizio <patrizio.gelosi@amarulasolutions.com> Mon Oct 14 15:36:58 2024 +0200
committer Patrizio <patrizio.gelosi@amarulasolutions.com> Wed Oct 16 16:42:09 2024 +0200
tree 98faa72b8d62c0b927aa9fe2f972dbe2f63a0901
parent d940d1f8945c6664ec8d376175904960da2e5095

Fix missing `reply` field in ON_DEMAND code context

Fixed issue where ChatGPT occasionally sends response records without a
`reply` field in ON_DEMAND code context, ensuring proper handling in
line with the plugin’s expected behavior rather than causing an
exception.

Change-Id: I277cdb3f7de93309424bea02e176053714234e4f
Signed-off-by: Patrizio <patrizio.gelosi@amarulasolutions.com>

src/main/java/com/googlesource/gerrit/plugins/chatgpt/PatchSetReviewer.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/PatchSetReviewer.java b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/PatchSetReviewer.java
index f4e1d8c..dda6edb 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/PatchSetReviewer.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/PatchSetReviewer.java
@@ -138,7 +138,7 @@
                 log.debug("Score added: {}", score);
                 reviewScores.add(score);
             }
-            if (!change.getIsCommentEvent() && changeSetData.getReplyFilterEnabled() && isHidden) {
+            if (reply == null || !change.getIsCommentEvent() && changeSetData.getReplyFilterEnabled() && isHidden) {
                 continue;
             }
             if (changeSetData.getDebugReviewMode()) {