fix(client): abort-on-close watcher busy-spins while events are pending (#101)

The watcher added in #98 waited with `wait(stream_to); yield()` in a loop.
`wait(::Channel)` returns as soon as data is *available*, so whenever an
event sits in the channel before the consumer takes it, the loop spins hot
— measured at 100% of a core in system time on a k8s watch stream. Poll
`isopen` with a 250ms sleep instead; the watcher only exists to abort the
read when the consumer walks away, so the added abort latency is irrelevant.

Bumps to 0.2.6.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

Author: krynju

Project.toml

@@ -4,7 +4,7 @@ keywords = ["Swagger", "OpenAPI", "REST"]
 license = "MIT"
 desc = "OpenAPI server and client helper for Julia"
 authors = ["JuliaHub Inc."]
-version = "0.2.5"
+version = "0.2.6"
 
 [deps]
 Base64 = "2a0f44e3-6c83-55bd-87e4-b1978d98bd5f"

src/client/httplibs/juliaweb_http.jl

@@ -373,9 +373,14 @@ function _http_streaming_request(ctx, method, url, headers, body, timeout, bytes
             # read-idle timeout. Mirrors the `:downloads` backend, which interrupts
             # its download task on channel close.
             try
+                # Block until the consumer closes the channel. Do NOT use
+                # `wait(stream_to)`: it returns as soon as data is AVAILABLE, so
+                # while an event sits in the channel not yet consumed, a
+                # wait+yield loop degenerates into a hot spin that burns a full
+                # core in scheduler/syscall overhead. Poll `isopen` instead;
+                # 250ms of extra abort latency is irrelevant here.
                 while isopen(stream_to)
-                    wait(stream_to)
-                    yield()
+                    sleep(0.25)
                 end
             catch ex
                 isa(ex, InvalidStateException) || rethrow(ex)