Nitpick: Assume -w when --build-watch-path has been set

[evanderkoogh]

This is an absolutely incredibly well done local testbed for Workers. I got it working on a Typescript Durable Object project with an install and a run command. Well done!

My only gripe so far has been that I had set only --build-watch-path, but I also had to set -w to get it to reload. This PR should fix that :)

[mrbbot]

👋 Thanks for raising this. This PR definitely fixes that for the CLI, however I think the best place for this to live would be in OptionsProcessor around here: https://github.com/mrbbot/miniflare/blob/84139a2b5b4d264288a607ec3887cda98f210612/src/options/processor.ts#L269 That way setting build watch path in the CLI, wrangler.toml or via the API would set the watch option too.

[mrbbot]

I've gone ahead and implemented this. It'll be in the next release. 👍

[mrbbot]

Thinking about this a bit more whilst updating the documentation, I think you're right that this should be CLI only 😄. Having this behaviour for wrangler.toml files and the API would lead to unexpected behaviour, especially since users would need to call dispose to clean up the watcher if Miniflare found a build command in wrangler.toml (since build watch path would default to src if this was the case).

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Pages

Latest commit:	9c13559
Status:	✅  Deploy successful!
Preview URL:	https://f1fe4fce.cf-miniflare.pages.dev

View logs

[mrbbot]

Better late than never 😅