fix: address PR #49 review comments

- Exclude setup/teardown hook files (setup.js, teardown.js,
  *.setup.js, *.teardown.js) from .js racer fallback in discoverRacers
- Add settled flag in runScript to prevent double-resolve/reject when
  timeout fires and close event both trigger

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: kertal

cli/config.js

@@ -32,7 +32,9 @@ export function discoverRacers(raceDir) {
   let racerFiles = allFiles.filter(f => f.endsWith('.spec.js')).sort();
 
   if (racerFiles.length < 2) {
-    const jsFiles = allFiles.filter(f => f.endsWith('.js')).sort();
+    // Exclude setup/teardown hook files from racer fallback
+    const hookPattern = /\.(setup|teardown)\.js$/;
+    const jsFiles = allFiles.filter(f => f.endsWith('.js') && !hookPattern.test(f) && f !== 'setup.js' && f !== 'teardown.js').sort();
     if (jsFiles.length >= 2) {
       console.error(`Warning: Found ${racerFiles.length} .spec.js files, using .js files instead`);
       racerFiles = jsFiles;

race.js

@@ -584,6 +584,7 @@ async function runScript(script, label) {
     let stdout = '';
     let stderr = '';
     let timedOut = false;
+    let settled = false;
 
     child.stdout.on('data', d => { stdout += d; });
     child.stderr.on('data', d => { stderr += d; });
@@ -597,8 +598,10 @@ async function runScript(script, label) {
       }, 5000);
     }, timeout);
 
-    child.on('close', code => {
+    child.on('close', (code) => {
       clearTimeout(timeoutId);
+      if (settled) return;
+      settled = true;
 
       if (timedOut) {
         progress.done(`${label} timed out after ${timeout}ms`);
@@ -655,6 +658,8 @@ async function runScript(script, label) {
 
     child.on('error', err => {
       clearTimeout(timeoutId);
+      if (settled) return;
+      settled = true;
       progress.done(`${label} error: ${err.message}`);
       reject(err);
     });