ts-node icon indicating copy to clipboard operation
ts-node copied to clipboard

Ability to turn off sourceMap

Open staeke opened this issue 6 years ago • 6 comments
trafficstars

There is no way to turn off source maps as of now. See e.g. https://github.com/TypeStrong/ts-node/issues/27

Source maps are not always desired, and can make debugging a lot harder - see e.g. https://github.com/facebook/create-react-app/issues/1557

Feature request: allow turning off. Suggested way by respecting tsconfig.json

staeke avatar Mar 06 '19 18:03 staeke

What's the use case or issue here? You linked to unrelated things.

blakeembrey avatar Mar 06 '19 18:03 blakeembrey

@blakeembrey they're related....let me clarify:

#27 is about how default settings for source maps override any user settings. The person creating the other ticket considered it a bug, but someone stated that sourceMaps should always override user settings anyhow. I think that's bad, but understand that changes elsewhere in code is also needed.

The create-react-app link is about why people would want to turn off source maps

staeke avatar Mar 06 '19 19:03 staeke

@staeke You're always welcome to submit a PR that'll add a flag to disable this behavior.

My question is because it sounds like this is a feature request, but there hasn't been anyone with an issue that warrants this feature. But again, if you'd like to add it, I won't block on that.

blakeembrey avatar Mar 06 '19 19:03 blakeembrey

Looking forward to this feature. When running ts-node inside electron, source-map-support blows up every time any module attempts to read the stack of an exception, with the following message:

Uncaught RangeError: WebAssembly.Compile is disallowed on the main thread, if the buffer size is larger than 4KB. Use WebAssembly.compile, or compile on a worker thread.

rubenlg avatar Dec 12 '21 16:12 rubenlg

I recommend checking out #1533 and submitting a pull request. It should solve your problem. #1533 has a low level of effort required for implementation, and I'm happy to answer any questions in the comments for #1533.

cspotcode avatar Dec 13 '21 02:12 cspotcode

I dropped the require that you pointed out in #1533 altogether, and that fixes the issue. I suspect the right solution in my case would be to fix the interoperability between source-map-support and electron in the source-map-support module instead, but as a quick workaround, patching ts-node did the trick.

rubenlg avatar Dec 13 '21 08:12 rubenlg