Skip to content

Conversation

@ryanking13
Copy link
Contributor

This adds HTMLRewriter class in Python SDK, which does proxy lifetime handling internally.

We have an example that uses HTMLRewriter, but it uses JS API directly which is slightly annoying to use in Python because of the proxy handling.

@ryanking13 ryanking13 requested review from a team as code owners January 22, 2026 09:04
@ryanking13 ryanking13 marked this pull request as draft January 22, 2026 16:20
@ryanking13 ryanking13 force-pushed the gyeongjae/htmlrewriter-2 branch from d71bcaa to 41c35e0 Compare January 23, 2026 04:17
@codecov-commenter
Copy link

codecov-commenter commented Jan 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.35%. Comparing base (d2c9058) to head (7bfded9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5945      +/-   ##
==========================================
- Coverage   70.35%   70.35%   -0.01%     
==========================================
  Files         408      408              
  Lines      108651   108651              
  Branches    17991    17991              
==========================================
- Hits        76444    76441       -3     
- Misses      21409    21410       +1     
- Partials    10798    10800       +2     

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ryanking13 ryanking13 force-pushed the gyeongjae/htmlrewriter-2 branch from 41c35e0 to 4f0e824 Compare January 26, 2026 10:15
@ryanking13 ryanking13 force-pushed the gyeongjae/htmlrewriter-2 branch 2 times, most recently from 57df7e0 to d386d66 Compare February 6, 2026 08:40
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 6, 2026

Merging this PR will not alter performance

✅ 70 untouched benchmarks
⏩ 129 skipped benchmarks1


Comparing gyeongjae/htmlrewriter-2 (7bfded9) with main (d2c9058)

Open in CodSpeed

Footnotes

  1. 129 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@ryanking13 ryanking13 force-pushed the gyeongjae/htmlrewriter-2 branch from d386d66 to 0940d11 Compare February 10, 2026 06:26
@ryanking13 ryanking13 force-pushed the gyeongjae/htmlrewriter-2 branch from 0940d11 to 7bfded9 Compare February 10, 2026 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants