getPorts disabled on timeout

[d3nd3]

https://github.com/espruino/EspruinoTools/blob/4067a9c43842c67e946b275ef5541d87ef263dd4/core/serial_frame.js#L105

I am using EspruinoHub with the serial_frame mqtt-ble solution.
I use a raspberry pi 4 as my main pc, so its very slow, sometimes when i disconnect from the IDE, then reOpen the menu ports, it does not communicate fast enough causing the 'port' to be disabled in menu until i refresh the page. Its a minor annoyance.

I wonder if the timeout check can be made optional, I understand the code lives in EspruinoTools, so must be very generic. I think the WebIDE does not require the timeout because it checks the ports again on a 2 sec interval.

What are your thoughts , can anything be changed about this? If least perhaps an increase in the 100ms timeout.

[gfwilliams]

Just looking at this and I thought that the IDE called getPorts repeatedly - like every few 2 seconds as you say.

So if the first one failed, the ports should still appear 2 seconds later? Or is something else happening that is breaking this for some reason?

[d3nd3]

https://github.com/espruino/EspruinoTools/blob/4067a9c43842c67e946b275ef5541d87ef263dd4/core/serial_frame.js#L107

It disables this 'type of port' permanently I think.
Reason for believing this :
A race is setup, a Timeout 100ms is seeing if it fires after the callbacks.ports function. Which is same as saying , if time for callback to fire is not < 100ms, do something: What it does, if it fires at 100ms is:
`ERROR = "getPorts timeout, disabling";

This returns no ports for this portType and removes the callback for further events, but most importantly sets the ERROR variable, which then means :
https://github.com/espruino/EspruinoTools/blob/4067a9c43842c67e946b275ef5541d87ef263dd4/core/serial_frame.js#L98-L101
Then any future calls to getPorts will just return nothing. I think thats why?

[gfwilliams]

Ok, thanks for checking into this. I'd have thought we could easily bump this to 500ms, and could make the error non-permanent, which might fix it for you without risking breaking anything else?