[wrangler] Notify user on local dev server reload

[jacoblearned]

Overview

Currently, wrangler dev does not notify the user that reloads have completed for local servers. Users only see "Reloading local server...", which might cause confusion for users expecting a ready message.

 ⛅️ wrangler 4.39.0
───────────────────
⎔ Starting local server...
[wrangler:info] Ready on http://localhost:8787
[wrangler:info] GET / 200 OK (3ms)
⎔ Reloading local server...
[wrangler:info] GET / 200 OK (2ms)
⎔ Shutting down local server...

This change adds a log message to inform users the local server is ready after reload.

 ⛅️ wrangler 4.39.1
───────────────────
⎔ Starting local server...
[wrangler:info] Ready on http://localhost:8787
[wrangler:info] GET / 200 OK (3ms)
⎔ Reloading local server...
⎔ Local server updated and ready
[wrangler:info] GET / 200 OK (2ms)
⎔ Shutting down local server...

Implementation Details

Because we set logRequests: false in the miniflare options, miniflare's "Updated and ready" message is suppressed for all local server reloads.

This is done to avoid duplicate log entries from the proxy and user workers, so it seems simplest for wrangler to provide its own reload complete message after awaiting the miniflare options change.

This also changes the waitForReload e2e helper function to check for the reload success message rather than the
"Updating local server..." message for a stronger guarantee that reloads have completed.

To accommodate the stronger wait condition, we set the default timeout of waitForReload to 15 seconds to match the waitForReady timeout.

---

• Tests
  • Tests included
  • Tests not necessary because:
• Public documentation
  • Cloudflare docs PR(s):
  • Documentation not necessary because: Does not affect user-facing docs
• Wrangler V3 Backport
  • Wrangler PR: TBD - I think this does need to be backported but we'll have to create it manually since the CI won't automatically for PRs from forks
  • Not necessary because:

[changeset-bot]

🦋 Changeset detected

Latest commit: 9bf2824

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@10750

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@10750

miniflare

npm i https://pkg.pr.new/miniflare@10750

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@10750

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@10750

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@10750

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@10750

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@10750

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@10750

wrangler

npm i https://pkg.pr.new/wrangler@10750

commit: 9bf2824

[petebacondarwin]

LGTM if the CI agrees