typhonjs-node-escomplex/typhonjs-escomplex

Dependency update

aberaud opened this issue ยท 3 comments

Hi!

It would be awesome if the project was updated for Babel 7 :)
Many thanks

Hi @aberaud. Thanks for the reminder as the current arrangement is a bit long in the tooth. I have a lot of experience w/ modern ESM & Node packages, so quite likely I'll rework this package as straight ESM w/ no Babel / CJS build; Node 12+ at that point. Quite possibly I'll create a new mono-repo and build process instead of the many-repo arrangement currently.

As far as continuing to work on improving the analysis unfortunately I can't really justify the development cost being open source with no sponsors for this effort and it is not a project that I am actively using at least at this time. Hopefully arranging things as ESM and in a mono-repo format will make it easier to coordinate with anyone brave / has time enough to continue working on the metrics side of things.

Hopefully I can schedule the ESM conversion / repo maintenance aspects in the first quarter of next year.

That would be even better. I thought babel was used for code parsing/analysis, not just for packaging.

Oh it is... I'll switch to @babel/parser which is the new AST parser as well; oomph I'm not looking forward to debugging any potential AST changes. It has been a minute since I have examined this project. I did a minor bump ~3 years ago to the build process. I think I removed the dependence on babel-runtime at that time as that will cause issues for folks using a more recent version of Babel, so perhaps that sticking point is what I'm vaguely recalling as an issue when the Babel version is older. Nonetheless switching to a native ESM approach for the package will be best.

I'm expecting this to take ~1-2 weeks of effort, but hopefully it goes smoother.