Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unify API for @babel/register, webpack, Jest etc. include/ignore and cache settings #621

Open
3 of 11 tasks
Tracked by #615
Domiii opened this issue Dec 3, 2021 · 0 comments
Open
3 of 11 tasks
Tracked by #615
Assignees
Labels
enhancement New feature or request performance Performance of runtime analysis tools can be quite a nuisance.

Comments

@Domiii
Copy link
Owner

Domiii commented Dec 3, 2021

Usually, we use @babel/register for instrumentation (and thus caching). However, we cannot use @babel/register with Jest, so we need to add the corresponding settings to Jest's config instead.

  • add proper include support for Jest
    • add to Jest's own transformer
    • also wrap Jest CLI call in @babel/register
    • allow the two to be configured once, without copy+pasting?
      • probably want to use ENV
    • make @babel/register/lib/node.js's configFile: false configurable
      • -> or, in case of Jest, change babel.config.js to only add @dbux/babel-plugin if not already added
        • -> or change @dbux/cli/src/buildBabelOptions to not add it instead
      • also, change Jest's babel.config.js, s.t. @dbux/babel-plugin is not added if DBUX is disabled
      • because else, the config file would add @dbux/babel-plugin a second time when read from register
    • add more finely grained config options for module path include/ignore to moduleFilter
      • because we want to disable (not just packages, but) specific files, such as jest-runner/build/requireOrImportModule.js etc.
  • add cache support for Jest
  • generalize cache management
  • test w/ javascript-algorithms
@Domiii Domiii added enhancement New feature or request priority performance Performance of runtime analysis tools can be quite a nuisance. labels Dec 3, 2021
@Domiii Domiii self-assigned this Dec 3, 2021
@Domiii Domiii changed the title Fix Project Cache for non @babel/register scenarios unify API for @babel/register, webpack, Jest etc. include/ignore and cache settings Dec 3, 2021
@Domiii Domiii removed the priority label Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request performance Performance of runtime analysis tools can be quite a nuisance.
Projects
None yet
Development

No branches or pull requests

1 participant