Skip to content

Fix: Start UI event listener before spawning workers [Vibecoded]#1563

Merged
elopez merged 1 commit into
crytic:masterfrom
rappie:rappie/fix/start-listener-before-workers
Apr 30, 2026
Merged

Fix: Start UI event listener before spawning workers [Vibecoded]#1563
elopez merged 1 commit into
crytic:masterfrom
rappie:rappie/fix/start-listener-before-workers

Conversation

@rappie
Copy link
Copy Markdown
Contributor

@rappie rappie commented Apr 28, 2026

WARNING: this is 100% vibecoded

Successfully tested locally.

Summary

Start the UI/log event listener before spawning campaign workers.

spawnListener uses dupChan, so it only receives events written after the listener is attached.
Previously, workers were spawned first, which meant early worker events could be missed if they were emitted before the UI/log listener started.

Why

Worker startup and early replay events are emitted immediately after worker threads start. If the listener subscribes too late, text output can miss early events such as Starting FuzzerAgent, making logs incomplete and downstream tooling harder to reason about.

Changes

  • Delay worker spawning behind a local spawnWorkers action.
  • Start uiEventsForwarderStopVar <- spawnListener forwardEvent before spawning workers.
  • Preserve existing interactive and non-interactive behavior otherwise.

@elopez elopez merged commit 55e1303 into crytic:master Apr 30, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants