microlinkhq / html-get / 31159685158

98%

push

github

fix: fall back when mutool fails instead of returning empty HTML (#272)

* fix: fall back when mutool fails instead of returning empty HTML

PDF conversion had no try/catch (unlike pandoc after #265). A mutool
throw after a successful fetch escaped into fetch's outer catch, which
returned html:'' with no statusCode/headers — a blank success shell.

Catch mutool failures (and empty output), log them, and fall through to
the raw body path so the successful response is preserved.

Co-authored-by: kikohumanbeatbox <kikohumanbeatbox@gmail.com>

* perf(pdf): skip decoding PDF payload when mutool fails

The fallback path decoded the entire PDF buffer through toEncode, but
addBody drops non-media, non-JSON bodies, so the result was discarded.
A 20MB PDF with a failing mutool spent 7.5s producing a 190-char document.

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

* refactor(pdf): collapse duplicated mutool empty handling

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

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: kikohumanbeatbox <kikohumanbeatbox@gmail.com>
Co-authored-by: Kiko Beats <josefrancisco.verdu@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>

205 of 218 branches covered (94.04%)

Branch coverage included in aggregate %.

16 of 17 new or added lines in 1 file covered. (94.12%)

807 of 819 relevant lines covered (98.53%)

30.45 hits per line