Skip to content

Please upgrade source-map to 0.7.2 to get speed improvements #206

Open

Description

@andreicristianpetcu

Activity

  1. LinusU commented on Mar 4, 2018

    @LinusU
    Collaborator

    This is awesome, we need to figure out what breaking changes (if any) occurred in 0.7.0, then we are good to go!

    PRs welcome!

  2. LinusU commented on Mar 4, 2018

    @LinusU
    Collaborator

    Breaking change: Drop support for Node < 8. If you want to support older versions of node, please use v0.6 or below.

    Hmmmm, this might be a big problem for us... 🤔

  3. loganfsmyth commented on Mar 4, 2018

    @loganfsmyth

    Is this possible? SourceMapConsumer now returns a Promise, but catching and mapping thrown exceptions needs to be synchronous.

  4. SimenB commented on Mar 18, 2018

    @SimenB
    Contributor

    Jest is also unable to upgrade due to the lack of a sync API, FWIW

  5. niieani commented on Apr 28, 2018

    @niieani
    Contributor

    @SimenB Could deasync help here? It hooks into Node's event loop and essentially makes any async operation a sync one.

  6. devsnek commented on Jun 20, 2019

    @devsnek

    I published https://www.npmjs.com/package/@snek/source-map-support which works nicely in node and may work in browsers that allow you to load reasonable sized wasms synchronously. It wraps the 0.8.0-beta.0 release of source-map.

  7. pkit commented on May 6, 2021

    @pkit

    @andreicristianpetcu
    @devsnek there is no case for async source map consumer other than problems of specific ill-thought implementation by Mozilla.
    Like: await in stack trace handler? Are they nuts?

  8. cspotcode commented on Aug 30, 2021

    @cspotcode

    @cspotcode/source-map-support uses @cspotcode/source-map-consumer which is a fork of source-map that only has the consumer. @cspotcode/source-map includes the generator, too.

    These are based on the latest source-map version. They use WASM and have a sync API.

    You might have luck giving them a try. It's all open-source, so of course the changes can be ported over.

  9. jordanbtucker commented on Feb 24, 2022

    @jordanbtucker

    This will also fix an annoying issue in VS Code.

    source-map@<0.7.1 has an invalid typings field in its package.json which is causing VS Code to report an error that it can't find the type declarations. This is fixed in source-map@>=0.7.1.

  10. SimenB commented on Apr 20, 2022

    @SimenB
    Contributor

    Babel has migrated to https://npmjs.com/package/@jridgewell/trace-mapping, should this module as well? Code change seems fairly trivial (no idea about browser support, but probably?)

  11. cspotcode commented on Apr 20, 2022

    @cspotcode
  12. SimenB commented on Apr 20, 2022

    @SimenB
    Contributor

    Jest and C8 (via v8-to-istanbul) has also migrated, fwiw

  13. cspotcode commented on Apr 20, 2022

    @cspotcode

    I can migrate @cspotcode/source-map-support as well.

    Would be nice to consolidate the ecosystem a bit more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions