Arrow function with inner function fails to parse about parse-function HOT 7 OPEN

Nope, unfortunately not.

But in the mean time my requirements changed and I also needed to be able to parse full code snippets, not just functions, which means I am currently using babel to parse/transform the code.

So feel free to close this issue if you want and thanks again for all the feedback!

from parse-function.

Thanks for the report.

Interesting, I'll look on it.

from parse-function.

For some reason, wrapping it as string works.

import parseFunction from 'parse-function';

const res = parseFunction().parse(
  `(arg, bar) =>
    new Promise((resolve) => {
      function inner() {}

      inner();
    })`,
);

console.log(res);
/*
{ name: null,
  body:
   'new Promise((resolve) => {\n      function inner() {}\n\n      inner();\n    })',
  args: [ 'arg', 'bar' ],
  params: 'arg, bar' }
*/

from parse-function.

Actually, forget that. When I initially pasted it and executed it really it failed. But when it is formatted, your snippet works too.

import parseFunction from 'parse-function';

parseFunction().parse(
  (arg) =>
    new Promise((resolve) => {
      function inner() {}
      inner();
    }),
);

Does not fail. But you are kind of right, it's probably something with the wrapping. But it's done that way because we use .parseExpression.

This works too

parseFunction().parse(arg => {
  return new Promise(resolve => {
    function inner() {}
    inner();
  });
});

but this doesn't

parseFunction().parse(arg => new Promise(resolve => {
    function inner() {}
    inner();
  });
);

So, please use Eslint and/or Prettier.

from parse-function.

Thanks for the quick update. Actually I am using eslint already and it didnt report anytning because it has 'arrow-parens': [2, 'as-needed', { requireForBlockBody: true }], set and the rule doesnt think I need a block'ed body I guess.
-- edit --
hmm sorry, this rule is about () around the args ofc. I extend some other configs, will have a look at them what they do
-- /edit --

Really strange adding a linebreak works, from all the things I did to debug this thats one of the only things I didnt try ;)

Just wondering, if it is about the new Promise expression I would've expected the following to work too (but it doesnt):

parseFunction().parse(arg => (new Promise(resolve => {
    function inner() {}
    inner();
  }));
);

from parse-function.

@pimlie yea, I'm not sure why it is happening too. I have a pretty big eslint config, so I'm not sure if ESLint-only approach will help either. That's one of the things why Prettier is amazing.

It's kinda strange because it's perfectly valid javascript.

'use strict'

const foo = arg => new Promise(resolve => {
    function inner() {}
    inner()
})

foo(123)

from parse-function.

I would've expected the following to work too

Agree on that too.

Hey, @pimlie did you found something? I'm just looking around because finished moving it to tunnckoCore/opensource#62

from parse-function.