From cb680737ef87b924503de4bf64abd24b28ad027f Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Sat, 31 Jan 2026 23:14:44 +0000 Subject: [PATCH] feat: improve automated PR review script reporting - Parse `npm run test:unit` output to identify and list failing test files in the review comment. - Add a suggestion to run `npm run lint:css -- --fix` if stylelint errors are detected. - Maintain existing guardrails and micro-fix logic. Co-authored-by: PR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com> --- scripts/automated-pr-review.mjs | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/scripts/automated-pr-review.mjs b/scripts/automated-pr-review.mjs index a767c60d..9ce11a9f 100644 --- a/scripts/automated-pr-review.mjs +++ b/scripts/automated-pr-review.mjs @@ -129,6 +129,10 @@ async function reviewPR(prNumber, baseRef) { hasFailures = true; commentBody += '### ⚠️ Lint Warnings/Errors\n\n'; commentBody += '```\n' + lintRes.stdout + '\n' + lintRes.stderr + '\n```\n\n'; + + if (lintRes.stdout.includes('stylelint') || lintRes.stderr.includes('stylelint')) { + commentBody += '> **💡 Tip:** Try running `npm run lint:css -- --fix` to automatically resolve some style issues.\n\n'; + } } // 4. Unit Tests @@ -138,6 +142,18 @@ async function reviewPR(prNumber, baseRef) { hasFailures = true; commentBody += '### ❌ Test Failures\n\n'; commentBody += '```\n' + testRes.stdout + '\n' + testRes.stderr + '\n```\n\n'; + + // Extract failing test files + const failingTests = []; + const failureRegex = /✖\s+(tests\/[\w\-\.\/]+)\s+failed with exit code 1/g; + let match; + while ((match = failureRegex.exec(testRes.stdout + '\n' + testRes.stderr)) !== null) { + failingTests.push(match[1]); + } + + if (failingTests.length > 0) { + commentBody += `> **Suggested areas to inspect:** ${failingTests.map(f => `\`${f}\``).join(', ')}\n`; + } commentBody += '> **Suggestion:** Inspect the stack trace above. Run `npm run test:unit` locally to reproduce.\n\n'; } else { commentBody += '### ✅ Tests Passed\n\nAll unit tests passed.\n\n';