ulixee / hero

The web browser built for scraping
MIT License
698 stars 33 forks source link

feat(client): add serialized callstack #54

Closed blakebyrnes closed 2 years ago

blakebyrnes commented 2 years ago

Use a common standard for the stacktraces we use during development.

This PR moves Hero off of using source-map-support in favor of directly using the source-map library from Mozilla. Unfortunately, they've moved to an async model that makes it difficult to support stack traces (which are synchronous). I used a fork of that library that is porting some of their performance improvements (minus wasm) (source-map-js). In a recent ticket, the mozilla lib is considering taking a PR, so we can possibly move back to that once it's up (https://github.com/mozilla/source-map/issues/370#issuecomment-823403894)