perf(Node.js): Major performance bottleneck in Node.js due to fs.statSync about ts-morph HOT 5 CLOSED

Actually, I'm going to consolidate the deno code to use the node code and in the meantime fix this perf issue.

from ts-morph.

Less critical, but out of 40s, 2s was spent in this function, thus if possible it should be optimized better too:

The fact that this function creates another function inside of it is a possible culprit

from ts-morph.

For reference, this is how I captured and inspected the performance:

# I am running Node.js like this:
node --cpu-prof --cpu-prof-dir ~/Downloads/node --no-warnings=ExperimentalWarning --loader @swc-node/register/esm ./myScript.ts
# But if you are dealing with .js file, this would work too:
node --cpu-prof --cpu-prof-dir ~/Downloads/node ./myScript.js
# (The performance issue is independent of using a TypeScript loader)

That creates several .cpuprofile files in the ~/Downloads/node directory. Pick the largest among those, and open it in https://www.speedscope.app/. In that tool, use the "Sanditch" view, and sort by "Self" time to see functions which are good targets for further optimization.

from ts-morph.

@maxxxxxdlp thanks for benchmarking it! Feel free to submit a PR. I don't have time to spend on performance optimizations unfortunately (though I'll probably get to this eventually).

from ts-morph.

Can confirm the performance issue with statSync/captureLargerStackTrace is resolved in the most recent release. Thanks for the quick fix!

from ts-morph.