Kikobeats / send-http / 30559414125

100%

push

github

Simplify sweep: two bug fixes and a faster response path (#21)

* refactor: simplify and speed up the stream path

Flatten the sniffer pipeline when the content-type is already known: the
nested pipeline the sniffer builds is pure overhead once there is nothing
to detect. ~13% throughput on a 64KB body.

The hop itself stays. Piping a request stream straight into a
ServerResponse copies its headers over, past the proxy allowlist.

Also: forward send's options to sendStream, swap proxy's finished() for a
close listener, hoist the pipeline callback, and unify the content-type
predicate.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011WbFt6CmzN7vNw1kAXAbtm

* refactor: one buffered tail, one pass over the payload

A string was measured, written and then encoded: three walks of the same
bytes. Encoding it once up front cuts ~35% off a 256KB body and leaves
small ones where they were.

That gives every buffered body the same shape, so the Buffer branch stops
ending the response behind create()'s back: the hook now sees every
buffered body, which is what the README always claimed.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011WbFt6CmzN7vNw1kAXAbtm

* fix: one rule for whether a response can still be answered

The README's own onError example calls send() from a stream error, and
past the headers that threw ERR_HTTP_HEADERS_SENT from a listener with no
catch above it: an upstream failing mid-body took the process with it.

The stream path already refused to touch a response that had gone; the
buffered path did not. Both now ask canAnswer(), which is also the
condition destroyUnanswered was spelling out by hand.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011WbFt6CmzN7vNw1kAXAbtm

* docs: say what the library actually does

It writes to a ServerRespo... (continued)

42 of 42 branches covered (100.0%)

Branch coverage included in aggregate %.

110 of 110 relevant lines covered (100.0%)

8.38 hits per line