8
79%
main: 79%

Ran 25 Jun 2026 02:43AM UTC

Files 59

Run time 1s

Badge

Embed ▾

Committed 25 Jun 2026 02:06AM UTC coverage: 76.933% (-0.01%) from 76.946%

Job # 28143035874.8

Build Type

Pull #601

github

Committed by

mxschmitt

Commit Message

fix: prevent jsonPipe deadlock that hung remote connections

The remote-connection transport (jsonPipe) delivered incoming messages into
a bounded channel (buffer 10) from the outer connection's dispatch goroutine.
When more than 10 messages queued before the inner connection drained them,
that blocking send stalled the dispatch goroutine — which is also the only
goroutine that delivers replies. Any in-flight Send() awaiting a reply then
blocked forever.

This surfaced as a flaky hang in TestShouldUploadAFolderRemote: streaming
files to a remote server produces a burst of write/close responses plus async
page events, occasionally exceeding the buffer and deadlocking until the test
timeout. Reproduced deterministically by shrinking the buffer to 1; goroutine
dump showed the dispatch goroutine blocked on 'chan send' at jsonPipe while the
test goroutine blocked on 'chan receive' in jsonPipe.Send.

Replace the bounded channel with an unbounded, order-preserving queue drained
by Poll(), so the dispatch goroutine never blocks. Add unit tests covering the
non-blocking guarantee, ordering, blocking Poll, close, and drain-before-close.

Pull Request Pull Request #601: fix: eliminate the two remaining connection-layer deadlocks (event-handler + jsonPipe)

Coverage Stats

7821 of 10166 relevant lines covered (76.93%)

777.71 hits per line

playwright-community / playwright-go / 28143035874 / 8
79%
main: 79%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job ubuntu-latest-firefox - 28143035874.8

playwright-community / playwright-go / 28143035874 / 8 79% main: 79%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job ubuntu-latest-firefox - 28143035874.8

playwright-community / playwright-go / 28143035874 / 8
79%
main: 79%

README BADGES
x