Dismiss stale action reviews reliably with review marker

Author: matej

scripts/comment-pr-findings.js

@@ -6,6 +6,7 @@
 
 const fs = require('fs');
 const { spawnSync } = require('child_process');
+const REVIEW_MARKER = '<!-- nutrient-code-review-action -->';
 
 // Parse GitHub context from environment
 const context = {
@@ -91,6 +92,7 @@ function addReactionsToReview(reviewId) {
 // Check if a review was posted by this action
 function isOwnReview(review) {
   if (!review.body) return false;
+  if (review.body.includes(REVIEW_MARKER)) return true;
 
   // Check for our review summary patterns
   const ownPatterns = [
@@ -124,10 +126,13 @@ function dismissStaleReviews() {
     let dismissedCount = 0;
     for (const review of reviews) {
       const isDismissible = review.state === 'APPROVED' || review.state === 'CHANGES_REQUESTED';
-      const isBot = review.user && review.user.type === 'Bot';
       const isOwn = isOwnReview(review);
+      const hasMarker = Boolean(review.body && review.body.includes(REVIEW_MARKER));
+      const isBot = review.user && review.user.type === 'Bot';
+      const canDismissLegacyReview = isBot && isOwn;
+      const canDismissMarkedReview = hasMarker;
 
-      if (isBot && isDismissible && isOwn) {
+      if (isDismissible && (canDismissMarkedReview || canDismissLegacyReview)) {
         try {
           ghApi(
             `/repos/${context.repo.owner}/${context.repo.repo}/pulls/${context.issue.number}/reviews/${review.id}/dismissals`,
@@ -231,7 +236,7 @@ async function run() {
 
     const { high: highSeverityCount } = countSeverities(newFindings);
     const reviewEvent = highSeverityCount > 0 ? 'REQUEST_CHANGES' : 'APPROVE';
-    const reviewBody = buildReviewSummary(newFindings);
+    const reviewBody = `${buildReviewSummary(newFindings)}\n\n${REVIEW_MARKER}`;
 
     // Prepare review comments
     const reviewComments = [];