Fix loadHtml failing on HTML with JSON-escape-required characters

[just-be-dev]

Summary

Fixes #203 and refactors the stdin JSON parser to eliminate the bug class entirely.

The bug: loadHtml silently failed whenever the HTML payload contained characters that require JSON escaping (" in attribute values, newlines, backslashes, control chars, etc.). Root cause was in process_input's streaming JSON reconstruction — string values and field names were re-emitted wrapped in raw " without re-escaping, producing invalid JSON that serde_json::from_str then rejected. Only loadHtml was affected because the initial html is passed via CLI argv and bypasses this code path.

The fix and follow-on refactor:

1. 9b38cb0 — Minimal fix: use serde_json::to_string() to properly escape field names and string values during reconstruction. Adds 7 tests covering quotes/newlines, backslashes, control chars, unicode, braces in strings, ~64 KB payloads, and special chars in loadUrl headers.

2. a336acc — Replace the actson-based stream-parse-then-reconstruct-then-reparse approach with serde_json::Deserializer::from_reader(...).into_iter::<Request>(). The actson streaming property wasn't actually being used (each Request is consumed as a complete unit anyway), and the reconstruction step was the root source of The loadHtml method does not work. #203. Drops the actson dependency. ~80 lines deleted.

3. e6e029e — Add 6 tests for behaviors specific to the new implementation: whitespace-separated values, stream termination on malformed JSON, stream termination on schema mismatch, empty input, byte-by-byte chunked reads (verifies real streaming), and graceful shutdown when the receiver is dropped.

Test plan

• cargo test --lib — all 18 tests pass (5 pre-existing + 13 new)
• cargo build — binary builds cleanly with actson removed
• Original repro test (test_process_input_load_html_with_quotes_and_newlines) fails on main, passes on this branch
• Manual sanity check: run examples/load-html.ts with a real-world HTML payload containing quoted attributes

Behavior changes worth noting

• On malformed JSON or schema mismatch, the stream now terminates rather than logging and continuing. serde_json::StreamDeserializer cannot resync after a parse error, and silently continuing past garbage would risk desync rather than recovery. Behavior is covered by test_process_input_malformed_json_terminates_stream and test_process_input_wrong_schema_terminates_stream.
• Worker thread no longer panics if the receiver is dropped (previously sender.send(...).unwrap()).

🤖 Generated with Claude Code