Append base to hot file

[timacdonald]

fixes #288

Fixes an issue where the base path is not included in the /public/hot file. This issue only impacts the npm run dev command. When building, assets are appended with the configured base.

base description

What

If we were to set the base path to /frontend/:

import { defineConfig } from 'vite';
import laravel from 'laravel-vite-plugin';

export default defineConfig({
    base: '/frontend/',
    plugins: [
        laravel({
            input: ['resources/css/app.css', 'resources/js/app.js'],
            refresh: true,
        }),
    ],
});

Then the dev server will be appended with /frontend/:

However, the hot file we write will not contain the /frontend base path:

So we do not create the correct URLs in Laravel. With this change in place, the hot file now contains the correct base.

Implementation notes

Vite normalizes the base value before we get access to it. The following table shows how Vite normalizes different values and what we end up appending to the hot file.

The only valid values for this option are undefined, an empty string, ./, or an absolute URL, i.e., https://example.com/... or /...

User config	Normalized	Hot file
undefined	"/"	""
""	"/"	""
"/"	"/"	""
"./"	"/"	""
"/frontend"	"/frontend/"	"/frontend"
"/frontend/"	"/frontend/"	"/frontend"
"https://example.com"	"/"	""
"https://example.com/"	"/"	""
"https://example.com/frontend"	"/frontend/"	"/frontend"
"https://example.com/frontend/"	"/frontend/"	"/frontend"

[github-actions]

Thanks for submitting a PR!

Note that draft PR's are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

[timacdonald]

Context: In Vite 4, the dev server works with and without the /frontend prefix. In Vite 5 is breaks.

[jessarcher]

Looks like this might also help with #197, as it's a similar implementation to #207.

[gazben]

@driesvints Can you make new release with this fix please?

[driesvints]

@gazben done