[gravitypdf-bot]

Fixes #95

There were seven copies of set_error_handler() / catch (Exception) / restore_error_handler() in DOM and DOMQuery. Because they caught only QueryPath\Exception, any other throwable (a TypeError, or an exception from a stream wrapper) left QueryPath's handler installed for the rest of the request. They now all go through DOM::withErrorHandler($exceptionClass, $callback, $types), which restores the handler in finally.

Each call site keeps its exception class and error levels. In particular writeHTML() still uses ParseException; #92 changes that, and whichever of the two merges second needs a one-line rebase. While moving the file loader into a closure, I worked out the HTML/XML choice before the call, which removed a duplicated load() branch.

Tests: testErrorHandlerRestoredAfterForeignThrowable uses a new ThrowingStreamWrapper fixture to throw a RuntimeException inside writeXML(), writeHTML(), writeHTML5() and a load with a stream context. It then checks that the previous handler is back. The first three cases fail on main; writeHTML5() already used finally after #89.

parseXMLString() and prepareInsert() have no such test, because I couldn't find a reliable way to make them throw a non-QueryPath exception.

🤖 Generated with Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.55%. Comparing base (b40b4c3) to head (ee074dc).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #103      +/-   ##
============================================
+ Coverage     90.38%   90.55%   +0.17%     
+ Complexity     1430     1425       -5     
============================================
  Files            25       25              
  Lines          3181     3166      -15     
============================================
- Hits           2875     2867       -8     
+ Misses          306      299       -7     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.