Replace traverse with neotraverse

[thim81]

Replace the bloated traverse (66 child dependencies) with Neotraverse (0 dependencies) https://npmgraph.js.org/?q=%40apideck%2Fportman#zoom=w

[PuruVJ]

The errors are happening because type of this.key and a few others have been updated to be string | number | symbol, which is how the library worked but the old typings never reflected that, causing false-positives many times. I would suggested adding a check const key = this.key; if(typeof key !== 'string') return;.

However if it has worked perfectly until now with traverse, it probably isn't affected by being a symbol or number. In that case, const key = this.key as string would also work. In fact I'd recommend this for existing projects than adding real runtime checks listed above