Address 3 issues: baud rate change, flash reading, instantiation with port

[g3gg0]

This PR adds 3 changes that made the code usable in TeddyCloud, a toniebox server implementation.

Reading flash memory

the function read_flash() allows reading back the flash

Baud rate fix

changing the baud rate did not resynchronize properly. added sync and retries.
this way 921600 baud connection works always. before it worked 1 out of 30 connection attempts.

Instantiation using WebSerial port as well as Transport

simplify caller's code by directly using WebSerial port object if specified.

esploader = new ESPLoader({
  port: port,
  baudrate: 921600,
  romBaudrate: 115200
});

[balloob]

@igrr this change looks good. It might fix an issue we’re seeing with ESP Web Tools. Can we get this merged and released?

[g3gg0]

interesting failures...
ah i see. it is complaing due to formatting issues?
got it.

[g3gg0]

did update the code, but didn't run it yet in our project (https://github.com/toniebox-reverse-engineering/teddycloud)
will do the functional checks in 1-2 days

[brianignacio5]

Thanks for the PR @g3gg0 looks great !

I've tested locally with esp32 wrover kit and works good. You can test the lint locally with npm run lint and after this will merge the PR.

[brianignacio5]

LGTM with minor changes

[brianignacio5]

Remove or handle packet not received

[brianignacio5]

can use this.debug(msg) which will only log when debugLogging is enabled.

[brianignacio5]

Closing this PR since it is merged with fixes in 04e337d

[balloob]

Thanks. Don't forget to tag the release on GitHub too.

[balloob]

I found a small bug in the type which I reported here 04e337d#r130045791