BlockStatement is not handled correctly about ast-parser

[daybrush]

@Xesenix

ast-parser 0.0.6 is released. Thank you for finding the typo. :)

[Xesenix]

@daybrush unfortunately this change is not enough to fix displaying function body, and I am not really sure where to look to fix that.

In general now when you run above code you will see something like this:

NODE: {
  callee: {
    typeAnnotation: {
      string: '',
      html: '',
      nodeType: '',
      typeId: [Function: getTypeId]
    },
    name: 'map',
    optional: '',
    id: 'map',
    template: '${id}$if{optional, "?"}$if{typeAnnotation, ": " + typeAnnotation}',
    nodeType: 'Identifier',
    string: 'map',
    html: 'map',
    typeId: [Function: getTypeId]
  },
  arguments: [
    {
      params: [Array],
      async: false,
      expression: '',
      generator: false,
      returnType: [Object],
      typeParameters: [Object],
      body: [Object],
      template: '$if{async, "async "}${typeParameters}(${params}) => ${body}',
      nodeType: 'ArrowFunctionExpression',
      string: '({defaultNoteFilter: defaultNoteFilter}: {defaultNoteFilter: NotesFiltersInterface}) => {}',
      html: '({defaultNoteFilter: defaultNoteFilter}: {defaultNoteFilter: NotesFiltersInterface}) => {}',
      typeId: [Function: getTypeId]
    }
  ],
  optional: '',
  typeArguments: { string: '', html: '', nodeType: '', typeId: [Function: getTypeId] },
  typeParameters: { string: '', html: '', nodeType: '', typeId: [Function: getTypeId] },
  template: '${callee}${typeParameters}(${arguments})',
  nodeType: 'CallExpression',
  string: 'map(({defaultNoteFilter: defaultNoteFilter}: {defaultNoteFilter: NotesFiltersInterface}) => {})',
  html: 'map(({defaultNoteFilter: defaultNoteFilter}: {defaultNoteFilter: NotesFiltersInterface}) => {})',
  typeId: [Function: getTypeId]
}
MISING FUNCTION BODY: map(({defaultNoteFilter: defaultNoteFilter}: {defaultNoteFilter: NotesFiltersInterface}) => {})

and I would expect the last line to contain something like this:

MISING FUNCTION BODY:  map(({defaultNoteFilter}: { defaultNoteFilter: NotesFiltersInterface}) => {
          let existingNoteFilter: NotesFiltersInterface | null = JSON.parse(localStorage.getItem(defaultNoteFilter.id));
          if (!existingNoteFilter) {
            return getNotesFiltersFromLocalStorageFail({defaultNoteFilter});
          } else {
            if (!existingNoteFilter.searchPhrases) {
              existingNoteFilter = {
                ...existingNoteFilter,
                searchPhrases: []
              };
            }
            return getNotesFiltersFromLocalStorageSuccess({existingNoteFilter});
          }
        })

You can download project from here: https://stackblitz.com/edit/ast-parser-issue and run:

npm i
npm run test

[daybrush]

@Xesenix

I'm sorry. You're right, but the reason I made ast-parser is to omit it. I thought the content in the blockstatement was unnecessary.

[Xesenix]

@daybrush Maybe it's worth adding some documentation about this behavior as I kind of expected full AST as result and after using this one I had to rewrite it to different parser.
I have managed to resolve my use case using ts-morph. It kind of felt a little bit harder to use but I've got the data I needed.