From 4462dbb861de77078a9a517ebc51a3be1d9abfc0 Mon Sep 17 00:00:00 2001
From: Christian Gunderman <gundermanc@google.com>
Date: Mon, 2 Mar 2026 15:22:53 -0800
Subject: [PATCH] refactor(ci): clean up check_evals_trigger.js

---
 scripts/check_evals_trigger.js | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/scripts/check_evals_trigger.js b/scripts/check_evals_trigger.js
index e791b64a87..919f78d678 100644
--- a/scripts/check_evals_trigger.js
+++ b/scripts/check_evals_trigger.js
@@ -5,6 +5,11 @@
  */
 import { execSync } from 'node:child_process';
 
+const EVALS_FILE_PREFIXES = [
+  'packages/core/src/prompts/',
+  'packages/core/src/tools/',
+];
+
 function main() {
   try {
     // Fetch main branch to compare against. Use || true to avoid failing if already up to date or no remote.
@@ -24,14 +29,12 @@ function main() {
       .split('\n')
       .filter(Boolean);
 
-    const shouldRun = changedFiles.some(
-      (file) =>
-        file.startsWith('packages/core/src/prompts/') ||
-        file.startsWith('packages/core/src/tools/'),
+    const shouldRun = changedFiles.some((file) =>
+      EVALS_FILE_PREFIXES.some((prefix) => file.startsWith(prefix)),
     );
 
     console.log(shouldRun ? 'true' : 'false');
-  } catch (_error) {
+  } catch {
     // If anything fails (e.g., no git history), run evals to be safe
     console.log('true');
   }