fix(core): Fix issue in .d.ts typing for TextEncoder

[danielkatz]

Fixes #7247 without loosing the typing

This pull request makes a small but important change to the BytesOutputParser class in the langchain-core package. The change involves updating the type of the textEncoder property to use a more specific type.

globalThis.TextEncoder (when not imported from node:util) is a value, not a type. In this case, the type is actually InstanceType<typeof TextEncoder>.

• langchain-core/src/output_parsers/bytes.ts: Updated the textEncoder property to use InstanceType<typeof TextEncoder> instead of any for better type safety.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
langchainjs-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 19, 2025 4:10pm

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
langchainjs-api-refs	⬜️ Ignored (Inspect)			Feb 19, 2025 4:10pm

[jacoblee93]

Nice! Makes sense

Any chance this would miss some other edge case? Given it's internal facing seems like some risk for not much reward

[danielkatz]

Thanks for the feedback.

I encountered the same issue and, after researching, found the #7247 issue without a real solution.
Once I figured it out for myself, I decided to submit this PR to help others. I’ve been using this solution in production without encountering any related problems.

[jacoblee93]

Ok thank you! Appreciate you digging in