fix: use CJS shims for stream

[morgante]

Explicitly importing the .mjs module breaks CommonJS compatibility, we should instead import the streaming path which will choose between ESM and CommonJS automatically.

[changeset-bot]

🦋 Changeset detected

Latest commit: d7e228a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@instructor-ai/instructor	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[morgante]

@roodboi Can you take a look?

[roodboi]

looks good, thanks!

[jeremypress]

whoops, opened here as well #180. closing mine

[jeremypress]

hey @morgante 👋🏻 @roodboi can land? I'd like to remove my pnpm patch so I can distribute my code

[morgante]

Yes I think @roodboi needs to merge.

[pakamienny]

hey has this been solved?
I am still running into an error:

Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/pakamienny/Documents/hackathons/batman/node_modules/openai/streaming.mjs not supported.
Instead change the require of /Users/pakamienny/Documents/hackathons/batman/node_modules/openai/streaming.mjs to a dynamic import() which is available in all CommonJS modules.
at Object. (/Users/pakamienny/Documents/hackathons/batman/node_modules/@instructor-ai/instructor/dist/index.cjs:1:806) {
code: 'ERR_REQUIRE_ESM'
}