fix(tests): recheck abort after lazy handler load; list script tests by file

Author: waleedlatif1

apps/sim/lib/copilot/request/tools/executor.ts

@@ -505,6 +505,8 @@ async function executeToolAndReportInner(
     })
   }
 
+  // Loads the handler map on first use; the abort check below covers that wait.
+  await ensureHandlersRegistered()
   if (abortRequested(context, execContext, options)) {
     markToolCallCancelled('Request aborted before tool execution')
     markToolResultSeen(toolCall.id)
@@ -608,7 +610,6 @@ async function executeToolAndReportInner(
   }
 
   try {
-    await ensureHandlersRegistered()
     let result = await executeToolWithWatchdog(toolCall, toolExecutionContext)
     if (toolCall.endTime || isTerminalToolCallStatus(toolCall.status)) {
       endToolSpanFromTerminalState()

scripts/check-script-test-coverage.ts

@@ -42,7 +42,7 @@ if (!reachableScripts('test').has('test:scripts')) {
 }
 
 const listed = Bun.spawnSync(
-  ['bunx', 'vitest', 'list', '--json', '--config', 'vitest.scripts.config.ts'],
+  ['bunx', 'vitest', 'list', '--json', '--filesOnly', '--config', 'vitest.scripts.config.ts'],
   {
     cwd: ROOT,
   }
@@ -53,7 +53,7 @@ if (listed.exitCode !== 0) {
 }
 const collected = new Set(
   (JSON.parse(listed.stdout.toString()) as Array<{ file: string }>).map((entry) =>
-    path.relative(ROOT, entry.file)
+    path.relative(ROOT, entry.file).split(path.sep).join('/')
   )
 )