-
Notifications
You must be signed in to change notification settings - Fork 5
size: add utf16.browser impl #62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Adds a browser-targeted UTF-16 implementation and wires it into package resolution so bundlers can pick a smaller browser build.
Changes:
- Added
utf16.browser.jsimplementation that decodes viaTextDecoderand encodes via the existing JS fallback encoder. - Updated
package.jsonfiles,exports, andbrowsermappings to route browser consumers toutf16.browser.js.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| utf16.browser.js | New browser-specific UTF-16 encode/decode implementation (TextDecoder-based decode). |
| package.json | Exposes and aliases the new browser entrypoint via exports and the legacy browser field. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
77fe45c to
a93d6d7
Compare
7de2ebc to
a489cf9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2600ad6 to
a14b1c1
Compare
66b8e8f to
22d1b21
Compare
52c3453 to
29346e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
c2490a4 to
24a5011
Compare
24a5011 to
dbd9bf2
Compare
dbd9bf2 to
346a4fa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Under 1 KiB gzipped