CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

QA: this might break the watch-and-build flow on error, need to check

Oh, it does break the watch flow though. I can see that being undesirable 🤔 But... I'd rather it crash loudly and break the flow, than fail silently I think? Maybe we can make watch behave more sensibly in a future PR.

if you accidentally save with a syntax error it breaks the reload on change flow, so do want to fix that. just needs a try/catch around the bundle on change. ill get to it eventually

sounds like i made an assumption that it would break and your experimentation showed it to be wrong

When I tested this with yarn start, it would crash the process completely if a build failure occurred on the initial build, stopping the file watcher. It would also cleanly print the error, making it easy to see what went wrong.

But if I introduce a build error after the initial build while yarn start is running, it'll print some incomprehensible "UnhandledRejection" warnings without telling me what went wrong, but the process will continue unimpeded. The watcher continues running, and will build successfully when the error is corrected.

Ideally we'd keep the process running in the first place, and print the real error in the second, but at least the syntax error scenario doesn't break the watcher.

@Gudahtt addressed both issues 👍
this should be good2go