Tim Fish

Results 419 comments of Tim Fish

It looks like this is caused by `import-in-the-middle`. It uses a `Map` so when it encounters duplicate exports it **should** just result in one of them being exposed but for...

There are outstanding PRs for `import-in-the-middle` that hopefully fix a wide range of issues. There is a patch available [here](https://github.com/getsentry/sentry-javascript/issues/12242#issuecomment-2133993135) that can be used with `patch-package` that should fix all...

> https://github.com/mitsuhiko/might-be-minified That regex is wild! Can you not get a reasonably reliable heuristic by just looking at the line lengths? Is there any point including context lines if they're...

These benchmark results make sense to me because I wouldn't have expected the readline approach to be considerably faster, especially when the errors are towards the end of the file...

> the goal here is to reduce peak memory usage while not impact CPU time. This PR has already achieved that by using `createReadStream` which reads in chunks rather than...

Here is a patch for `import-in-the-middle` that can be used with [`patch-package`](https://www.npmjs.com/package/patch-package) which includes all the pending PR's. [import-in-the-middle+1.7.4.patch](https://github.com/getsentry/sentry-javascript/files/15461352/import-in-the-middle%2B1.7.4.patch) If you encounter issues with this patch applied, please open a...

`[email protected]` has been released and I opened an [otel PR](https://github.com/open-telemetry/opentelemetry-js/pull/4745) to update it!

@danilofuchs thank you so much for the minimal reproduction! I don't get the same error but it's similar enough: ``` ReferenceError: Cannot access 'BatchesPage' before initialization at file:///Users/tim/Documents/Repositories/sentry-esm-openai-repro/node_modules/openai/resources/batches.mjs:35:38 at file:///Users/tim/Documents/Repositories/sentry-esm-openai-repro/node_modules/openai/resources/batches.mjs:36:3...

Here is the `import-in-the-middle` issue: https://github.com/DataDog/import-in-the-middle/issues/82

I opened another PR for `import-in-the-middle` that should fix this. There is patch available [here](https://github.com/getsentry/sentry-javascript/issues/12242#issuecomment-2133993135) that combines all the outstanding open PRs.