fix: server preset defined in app config not working

[edivados]

solidjs/solid-start#1208

on build the cli determines the preset without checking the app config.

Oh... just noticed that for dev the preset is also determined in the cli. Should it be moved back to the cli for build?
But then you would have to do it again in createBuild to cover this I guess. 🤔

vinxi/packages/vinxi/lib/app.js

Lines 176 to 179 in 16814e6

	async build() {
	const { createBuild } = await import("./build.js");
	await withLogger({}, () => createBuild(app, {}));
	},

[codesandbox]

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders

Open Preview

[changeset-bot]

🦋 Changeset detected

Latest commit: 8c87b24

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

This PR includes changesets to release 28 packages

Name	Type
vinxi	Patch
@vinxi/plugin-directives	Patch
@vinxi/react-server	Patch
@vinxi/router	Patch
@vinxi/server-components	Patch
@vinxi/server-functions	Patch
example-react-server	Patch
app	Patch
example-react-spa-basic	Patch
react-spa-mdx	Patch
react-spa-tanstack-router-app	Patch
example-react-spa-tanstack-router-app	Patch
react-ssr-basic-cloudflare	Patch
react-ssr-basic	Patch
react-ssr-tanstack-router-app	Patch
react-ssr-wouter	Patch
solid-spa-basic	Patch
solid-ssr-basic	Patch
example-ssr-solid-router	Patch
example-vanilla-empty	Patch
example-vanilla-partyroom	Patch
example-vanilla-spa	Patch
example-vanilla-stack	Patch
example-vanilla-trpc	Patch
test-react-rsc	Patch
test-react-srv-fn	Patch
react-ssr-test-fs	Patch
test-react	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

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
vinxi-website	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 4, 2024 7:44am