Would be great.

Also, while we are at it, some other things could make using parse-function much easier: TypeScript definitions.

Currently it's difficult to use parse-function in TypeScript project because of the lack of definition. We currently have to import parse-function using require instead of import ... from ... because of the lack of definition and it makes the compiled code userequire too. Why is this a problem? Because of this:

webpack/webpack#8171

Webpack will resolve the require to dist/index.es.js and the code will fail since dist/index.es.js export a default property that holds the parser constructor while dist/index.js export the parser constructor.

By having a definition, we would be able to use the import ... from ... in our TypeScript code and have the import ... from ... also used in the compiled code.

See that issue to see the impact of that on a real project:

https://github.com/ericmorand/twing/issues/284

I'm not so in TypeScript, so definitions are always welcome as PR.
I may add them by myself but i'm not sure.

I can take care of that. Since your project is not written in TS, it's better to host then in DefinitelyTyped repository. If you don't mind, I'll add them there.

Great, yea! :)

@ericmorand

I'll handle that. I already moved the source code to new home at my monorepo https://github.com/tunnckoCore/opensource/tree/master/packages/parse-function, so it will eventually get written in typescript.

GitHub

tunnckoCore/opensource

Delivering delightful digital solutions. JavaScript Open Source Monorepo, semantically versioned following @conventional-commits. Fully powered by Jest, @babel TypeScript, @airbnb @eslint + @pretti...

Final API for v6, defaults to use @babel/parser if not options.parse given.

import { parse as acornParse } from 'acorn';
import { parseFunction } from '.';

// `node` is an AST Node
function bobbyPlugin(node, result) {
  const bobby = 'bobby';

  return { ...result, bobby };
}

function barryPlugin(node, result) {
  return { ...result, barry: 'barry barry' };
}

const result = parseFunction('function foo(bar, qux) {}', {
  parse: acornParse,
  plugins: [bobbyPlugin, barryPlugin], // supports array of plugins
  parserOptions: {},
});

console.log(result);

It also has some TypeScript support. For more, PRs welcome.