Redirect all frontend routes to start-instance until configured

[petebacondarwin]

No description provided.

[xtuc]

I intentially didn't do it to avoid adding a D1 query on all requests.

[petebacondarwin]

But don't we hit D1 on every frontend UI request anyway? Especially now that we are reading the thumbnail etc from the DB?
In the data loader for access the instance-config, we could trigger a redirect if no data is returned.

[xtuc]

We would need to query D1 to detect that the instance hasn't been configured.

[petebacondarwin]

This code runs on every frontend request:

wildebeest/frontend/src/routes/layout.tsx

Lines 18 to 25 in 44898ac

	export const instanceLoader = loader$<{ DATABASE: D1Database; domain: string }, Promise<InstanceConfig>>(
	async ({ platform }) => {
	const response = await instance.handleRequest('', platform.DATABASE)
	const results = await response.text()
	const json = JSON.parse(results) as InstanceConfig
	return json
	}
	)

In this function, we can check whether the config exists (rather than just naively casting it) and throw a response.redirect() to push the user to the start-instance page instead.

[petebacondarwin]

And if this is problematic (from a performance point of view) we need to implement some caching of the instance data in the system somewhere (outside of D1) perhaps in a KV?