refactor(function): compress via CompressionStream - #35
Conversation
Prefer native brotli and fall back to gzip so Chrome works without shipping lz-ts. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Warning Review limit reachedNext included review available in 30 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (4)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 7d3a227. Configure here.
Uint8Array#toBase64 is not on Node 24, which CI and engines allow. Co-authored-by: Cursor <cursoragent@cursor.com>
Coverage Report for CI Build 34240008394Warning No base build found for commit Coverage: 77.924%Details
Uncovered Changes
Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
2 participants
Summary
CompressionStream:br#when brotli is available,gz#in Chrome.lz-ts; the API already accepts both prefixes.Test plan
packages/functioncompress tests (brotli happy path + gzip fallback)pnpm --filter @microlink/function build)Made with Cursor