Merge pull request #1451 from datum-cloud/fix/seo-review-skip-no-changes-comment

fix: skip PR comment when seo-review has nothing to check

Author: AriaEdo

.github/workflows/seo-review.yml

@@ -109,6 +109,7 @@ jobs:
           } >> "$GITHUB_OUTPUT"
 
       - name: Run SEO review
+        id: seo-review
         env:
           ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
           SCAN_MODE: ${{ steps.mode.outputs.value }}
@@ -119,7 +120,7 @@ jobs:
         run: node scripts/seo-review.mjs
 
       - name: Post or update PR comment
-        if: github.event_name == 'pull_request'
+        if: github.event_name == 'pull_request' && steps.seo-review.outputs.mode != 'skipped-no-changes'
         uses: marocchino/sticky-pull-request-comment@v3
         with:
           header: seo-review-bot

scripts/seo-review.mjs

@@ -18,7 +18,7 @@
  *   SITE_URL            default: https://www.datum.net (host used to recognise internal links)
  */
 
-import { readFile, writeFile, readdir, stat, mkdir } from 'node:fs/promises';
+import { readFile, writeFile, appendFile, readdir, stat, mkdir } from 'node:fs/promises';
 import path from 'node:path';
 import { load } from 'cheerio';
 
@@ -804,6 +804,10 @@ async function main() {
   await writeFile(OUTPUT_FILE, body, 'utf8');
   console.log(`Wrote ${OUTPUT_FILE} (${body.length} bytes, ${picked.length} pages).`);
 
+  if (process.env.GITHUB_OUTPUT) {
+    await appendFile(process.env.GITHUB_OUTPUT, `mode=${mode}\n`, 'utf8');
+  }
+
   if (tokenTotals.calls > 0) {
     const totalIn = tokenTotals.input + tokenTotals.cacheRead + tokenTotals.cacheCreate;
     const totalOut = tokenTotals.output;