Traversal fails when ObjectExpression properties don't have the optional type property about estraverse HOT 4 CLOSED

I'm looking into solving this in a similar way.

edit: Actually, a pretty different way.

from estraverse.

@Constellation: I've implemented my fix for the traverse function in michaelficarra/estraverse@80d3e6b. Can you take a look at that and approve it before I do the same with the replace function?

from estraverse.

@michaelficarra:
Nice! This fix solves this issue perfectly.

from estraverse.

@Constellation: I've generalised the solution in michaelficarra/estraverse@1376b05 to make it easier to compensate with future problems that @jasvir pointed out at 80d3e6b#commitcomment-2404694.

Next up, I'll fix it for replace and open a pull request.

edit: I amended the commit, it's now michaelficarra/estraverse@7b9217d.

edit again: There's something wrong with me today. It's actually michaelficarra/estraverse@47c00b7.

from estraverse.