Reduce more setTimeout delays in app-event-watcher, file-watcher, ui, Tasks, AutocompletePrompt, http-reverse-proxy — 53.93s, tests_s 50.66s\n\nResult: {"status":"keep","wall_clock_s":53.93,"collect_s":426.8,"tests_s":50.66,"transform_s":17.95,"setup_s":3.5,"prepare_s":24.11,"passed":3794,"files_passed":393}

Author: gonzaloriestra

autoresearch.jsonl

@@ -19,3 +19,4 @@
 {"run":18,"commit":"f3f3069","metric":68.33,"metrics":{"collect_s":539.7,"tests_s":73.49,"transform_s":21.33,"setup_s":6.69,"prepare_s":28.33,"passed":3794,"files_passed":393},"status":"discard","description":"Remove coverage config — no improvement, coverage config is inert without --coverage flag","timestamp":1773672778674,"segment":0}
 {"run":19,"commit":"f3f3069","metric":66.11,"metrics":{"collect_s":521.27,"tests_s":71.55,"transform_s":16.87,"setup_s":5.4,"prepare_s":27.09,"passed":3790,"files_passed":391},"status":"discard","description":"unplugin-swc — transform 16.87s (−20%) but 4 test failures and no wall clock improvement","timestamp":1773672962547,"segment":0}
 {"run":20,"commit":"f3f3069","metric":68.27,"metrics":{"collect_s":540.74,"tests_s":71.54,"transform_s":19.85,"setup_s":3.65,"prepare_s":27.89,"passed":3794,"files_passed":393},"status":"discard","description":"Lazy ZodError check in setup — setup_s dropped to 3.65s but no wall clock improvement","timestamp":1773673284806,"segment":0}
+{"run":21,"commit":"c60c268","metric":54.72,"metrics":{"collect_s":433.03,"tests_s":52.61,"transform_s":18.18,"setup_s":3.25,"prepare_s":23.74,"passed":3794,"files_passed":393},"status":"keep","description":"Reduce test delays: waitForInputsToBeReady 100→10ms, sendInputAndWait 100→10ms, fix tcp.test.ts sleep, use waitForContent for polling test — 54.72s (−19%), tests_s 52.6s (−28%)","timestamp":1773673656502,"segment":0}

packages/app/src/cli/services/dev/app-events/app-event-watcher.test.ts

@@ -369,7 +369,7 @@ describe('app-event-watcher', () => {
         await flushPromises()
 
         // Wait for event processing
-        await new Promise((resolve) => setTimeout(resolve, 100))
+        await new Promise((resolve) => setTimeout(resolve, 10))
 
         // Then
         expect(generateTypesSpy).toHaveBeenCalled()
@@ -405,7 +405,7 @@ describe('app-event-watcher', () => {
         await flushPromises()
 
         // Wait for event processing
-        await new Promise((resolve) => setTimeout(resolve, 100))
+        await new Promise((resolve) => setTimeout(resolve, 10))
 
         // Then - not called in watcher because it was already called during reloadApp
         expect(generateTypesSpy).not.toHaveBeenCalled()
@@ -441,7 +441,7 @@ describe('app-event-watcher', () => {
         await flushPromises()
 
         // Wait for event processing
-        await new Promise((resolve) => setTimeout(resolve, 100))
+        await new Promise((resolve) => setTimeout(resolve, 10))
 
         // Then - not called in watcher because it was already called during reloadApp
         expect(generateTypesSpy).not.toHaveBeenCalled()
@@ -474,7 +474,7 @@ describe('app-event-watcher', () => {
         await flushPromises()
 
         // Wait for event processing
-        await new Promise((resolve) => setTimeout(resolve, 100))
+        await new Promise((resolve) => setTimeout(resolve, 10))
 
         // Then - generateExtensionTypes should still be called when extensions are deleted
         // to clean up type definitions for the removed extension

packages/app/src/cli/services/dev/app-events/file-watcher.test.ts

@@ -707,7 +707,7 @@ describe('file-watcher events', () => {
         }
 
         // Wait for debounced events
-        await new Promise((resolve) => setTimeout(resolve, 300))
+        await new Promise((resolve) => setTimeout(resolve, 30))
 
         // Test passes if we reach here without hanging
         clearTimeout(timeout)

packages/app/src/cli/services/dev/ui.test.tsx

@@ -189,7 +189,7 @@ describe('ui', () => {
         devSessionStatusManager,
       })
 
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 10))
 
       expect(vi.mocked(Dev)).toHaveBeenCalled()
       expect(concurrentProcess.action).not.toHaveBeenCalled()
@@ -235,7 +235,7 @@ describe('ui', () => {
         devSessionStatusManager,
       })
 
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 10))
 
       expect(vi.mocked(DevSessionUI)).toHaveBeenCalledWith(
         expect.objectContaining({
@@ -286,7 +286,7 @@ describe('ui', () => {
         devSessionStatusManager,
       })
 
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 10))
 
       // Get the onAbort callback that was passed to DevSessionUI
       const onAbort = vi.mocked(DevSessionUI).mock.calls[0]?.[0]?.onAbort

packages/app/src/cli/utilities/app/http-reverse-proxy.test.ts

@@ -59,7 +59,7 @@ describe.sequential.each(each)('http-reverse-proxy for %s', (protocol) => {
       // eslint-disable-next-line no-catch-all/no-catch-all
     } catch (error) {
       // If the assertion fails, wait a bit and try again
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 10))
       await expect(fetch(`${protocol}://localhost:${ports.proxyPort}/path1`, {agent})).rejects.toThrow()
     }
   })

packages/cli-kit/src/private/node/ui/components/AutocompletePrompt.test.tsx

@@ -348,7 +348,7 @@ describe('AutocompletePrompt', async () => {
     await waitForInputsToBeReady()
     await sendInputAndWaitForContent(renderInstance, 'Loading...', 'a')
     // prompt doesn't change when enter is pressed
-    await new Promise((resolve) => setTimeout(resolve, 100))
+    await new Promise((resolve) => setTimeout(resolve, 10))
     await sendInputAndWait(renderInstance, 10, ENTER)
 
     expect(renderInstance.lastFrame()).toMatchInlineSnapshot(`

packages/cli-kit/src/private/node/ui/components/ConcurrentOutput.test.tsx

@@ -310,7 +310,7 @@ describe('ConcurrentOutput', () => {
       />,
     )
 
-    await new Promise((resolve) => setTimeout(resolve, 500))
+    await new Promise((resolve) => setTimeout(resolve, 50))
     expect(renderInstance.waitUntilExit().isRejected()).toBe(false)
   })
 
@@ -354,7 +354,7 @@ describe('ConcurrentOutput', () => {
       />,
     )
 
-    await new Promise((resolve) => setTimeout(resolve, 500))
+    await new Promise((resolve) => setTimeout(resolve, 50))
 
     expect(renderInstance.waitUntilExit().isFulfilled()).toBe(false)
   })

packages/cli-kit/src/private/node/ui/components/Tasks.test.tsx

@@ -77,7 +77,7 @@ describe('Tasks', () => {
   test('it supports subtasks', async () => {
     // Given
     const firstSubtaskFunction = vi.fn(async () => {
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 10))
     })
 
     const secondSubtaskFunction = vi.fn(async () => {})
@@ -115,11 +115,11 @@ describe('Tasks', () => {
   test('supports skipping', async () => {
     // Given
     const firstTaskFunction = vi.fn(async () => {
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 10))
     })
 
     const secondTaskFunction = vi.fn(async () => {
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 10))
     })
 
     const firstTask = {
@@ -145,11 +145,11 @@ describe('Tasks', () => {
   test('supports skipping a subtask', async () => {
     // Given
     const firstSubTaskFunction = vi.fn(async () => {
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 10))
     })
 
     const secondSubTaskFunction = vi.fn(async () => {
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 10))
     })
 
     const firstTask = {
@@ -416,5 +416,5 @@ describe('Tasks', () => {
 })
 
 async function taskHasRendered() {
-  await new Promise((resolve) => setTimeout(resolve, 100))
+  await new Promise((resolve) => setTimeout(resolve, 10))
 }