From 05df51ff8453bd877b8b617335f2e295d222b5d4 Mon Sep 17 00:00:00 2001
From: robobun <robobun@oven.sh>
Date: Wed, 14 Jan 2026 16:28:29 -0800
Subject: [PATCH] fix(runner): filter out non-JS files from node tests (#26092)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

## Summary
- `isNodeTest()` was only checking if the path included the node test
directories but not verifying the file was actually a JavaScript file
- This caused `test/js/node/test/parallel/CLAUDE.md` to be incorrectly
treated as a test file
- Added `isJavaScript(path)` check to filter out non-JS files

## Test plan
- [x] Verify CLAUDE.md is no longer picked up as a test file

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Bot <claude-bot@bun.sh>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
---
 scripts/runner.node.mjs | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/scripts/runner.node.mjs b/scripts/runner.node.mjs
index b680f87294..f694a3aa29 100755
--- a/scripts/runner.node.mjs
+++ b/scripts/runner.node.mjs
@@ -1585,6 +1585,9 @@ function isNodeTest(path) {
   if (isCI && isMacOS && isX64) {
     return false;
   }
+  if (!isJavaScript(path)) {
+    return false;
+  }
   const unixPath = path.replaceAll(sep, "/");
   return (
     unixPath.includes("js/node/test/parallel/") ||