Refactor server url logs

[bluwy]

Changes

Close #4315

Vite 3 has an experimental server.resolvedUrls property for the URLs Vite is serving from. It will be stabilize in 3.1 without any API change. vitejs/vite#9866

This PR uses that for the dev server so we closely match the Vite dev server behaviour.

Before:

> astro dev

  🚀  astro  v1.1.1 started in 84ms
  
  ┃ Local    http://localhost:3000/
  ┃ Network  use --host to expose

> astro dev "--host"

  🚀  astro  v1.1.1 started in 78ms
  
  ┃ Local    http://localhost:3000/
  ┃ Network  http://192.168.0.165:3000/

> astro preview

  🚀  astro  v1.1.1 started in 2ms
  
  ┃ Local    http://localhost:3000/
  ┃ Network  use --host to expose

After:

> astro dev

  🚀  astro  v1.1.1 started in 82ms
  
  ┃ Local    http://127.0.0.1:3000/
  ┃ Network  use --host to expose

> astro dev "--host"

  🚀  astro  v1.1.1 started in 106ms
  
  ┃ Local    http://localhost:3000/
  ┃ Network  http://192.168.0.165:3000/

> astro preview

  🚀  astro  v1.1.1 started in 2ms
  
  ┃ Local    http://localhost:3000/
  ┃ Network  use --host to expose

NOTE that the http://127.0.0.1:3000/ change in astro dev after, without the --host option Vite (or Node specifically) would only serve under 127.0.0.1 (ipv4), [::1] (ipv6) isn't served, which could be covered by localhost.

Testing

Tested with the local examples.

Docs

N/A

[changeset-bot]

🦋 Changeset detected

Latest commit: d9b3ab4

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

This PR includes changesets to release 16 packages

Name	Type
astro	Patch
@e2e/astro-component	Patch
@e2e/error-cyclic	Patch
@e2e/error-react-spectrum	Patch
@e2e/error-sass	Patch
@e2e/errors	Patch
@e2e/hydration-race	Patch
@e2e/lit-component	Patch
@e2e/preact-component	Patch
@e2e/react-component	Patch
@e2e/solid-component	Patch
@e2e/solid-recurse	Patch
@e2e/svelte-component	Patch
@e2e/e2e-tailwindcss	Patch
@e2e/ts-resolution	Patch
@e2e/third-party-astro	Patch

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

[bholmesdev]

Love aligning with Vite on this behavior! It was a lot to untangle and re-implement when redesigning our own server 😅 Should be good-to-go once tests are passing.

[bluwy]

Ah my bad I didn't realize the tests were failing! That seems like tests errorrs I still have to fix. I'll put this to draft at the meantime, thanks for reviewing 😬

[bluwy]

The tests should be fixed now. I made the dev server test a bit more relax as there's a difference in output between node <17 and node >=17. The preview server test is unchanged.

[matthewp]

@bluwy can you update to resolve the conflicts?

[bluwy]

Resolved!