Skip to content

Commit f83fee2

Browse files
BrendonovichBrendonovich
committed
cleanup virtual modules and manifests
1 parent facc538 commit f83fee2

15 files changed

Lines changed: 81 additions & 173 deletions

‎packages/start/src/config/index.ts‎

Lines changed: 25 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,24 @@
1-
import { createTanStackServerFnPlugin } from "@tanstack/server-functions-plugin";
2-
import { defu } from "defu";
31
import { existsSync } from "node:fs";
42
import path, { isAbsolute, join, normalize } from "node:path";
53
import { fileURLToPath } from "node:url";
6-
import type { StartServerManifest } from "solid-start:server-manifest";
4+
import { createTanStackServerFnPlugin } from "@tanstack/server-functions-plugin";
5+
import { defu } from "defu";
76
import { normalizePath, type PluginOption, type Rollup, type ViteDevServer } from "vite";
87
import solid, { type Options as SolidOptions } from "vite-plugin-solid";
98

109
import { isCssModulesFile } from "../server/collect-styles.js";
11-
import { getSsrDevManifest } from "../server/manifest/dev-server-manifest.js";
10+
import { getSsrDevManifest } from "../server/manifest/dev-ssr-manifest.js";
1211
import { SolidStartClientFileRouter, SolidStartServerFileRouter } from "./fs-router.js";
1312
import { fsRoutes } from "./fs-routes/index.js";
13+
import type { BaseFileSystemRouter } from "./fs-routes/router.js";
1414
import {
1515
clientDistDir,
1616
nitroPlugin,
1717
serverDistDir,
1818
ssrEntryFile,
1919
type UserNitroConfig
2020
} from "./nitroPlugin.js";
21-
import { BaseFileSystemRouter } from "./fs-routes/router.js";
21+
import { CLIENT_BASE_PATH } from "../constants.js";
2222

2323
const DEFAULT_EXTENSIONS = ["js", "jsx", "ts", "tsx"];
2424

@@ -70,15 +70,11 @@ const absolute = (path: string, root: string) =>
7070
let ssrBundle: Rollup.OutputBundle;
7171

7272
const VIRTUAL_MODULES = {
73-
serverManifest: "solid-start:server-manifest",
73+
clientViteManifest: "solid-start:client-vite-manifest",
7474
getClientManifest: "solid-start:get-client-manifest",
75-
getSsrManifest: "solid-start:get-ssr-manifest",
76-
getManifest: "solid-start:get-manifest",
7775
middleware: "solid-start:middleware"
7876
} as const;
7977

80-
export const CLIENT_BASE_PATH = "_build";
81-
8278
function solidStartVitePlugin(options?: SolidStartOptions): Array<PluginOption> {
8379
const start = defu(options ?? {}, {
8480
appRoot: "./src",
@@ -127,7 +123,7 @@ function solidStartVitePlugin(options?: SolidStartOptions): Array<PluginOption>
127123
};
128124
},
129125
async config(_, env) {
130-
let clientInput = [handlers.client];
126+
const clientInput = [handlers.client];
131127

132128
if (env.command === "build") {
133129
const clientRouter: BaseFileSystemRouter = (globalThis as any).ROUTERS.client
@@ -168,8 +164,6 @@ function solidStartVitePlugin(options?: SolidStartOptions): Array<PluginOption>
168164
consumer: "server",
169165
build: {
170166
ssr: true,
171-
// we don't write to the file system as the below 'capture-output' plugin will
172-
// capture the output and write it to the virtual file system
173167
write: true,
174168
manifest: true,
175169
copyPublicDir: false,
@@ -244,46 +238,36 @@ function solidStartVitePlugin(options?: SolidStartOptions): Array<PluginOption>
244238
name: "solid-start:manifest-plugin",
245239
enforce: "pre",
246240
async resolveId(id) {
247-
if (id === VIRTUAL_MODULES.serverManifest) return `\0${VIRTUAL_MODULES.serverManifest}`;
241+
if (id === VIRTUAL_MODULES.clientViteManifest) return `\0${VIRTUAL_MODULES.clientViteManifest}`;
248242
if (id === VIRTUAL_MODULES.getClientManifest)
249243
return new URL("../server/manifest/client-manifest.js", import.meta.url).pathname;
250-
if (id === VIRTUAL_MODULES.getSsrManifest)
251-
return new URL("../server/manifest/ssr-manifest.js", import.meta.url).pathname;
252-
if (id === VIRTUAL_MODULES.getManifest)
253-
return this.environment.config.consumer === "server"
254-
? new URL("../server/manifest/ssr-manifest.js", import.meta.url).pathname
255-
: new URL("../server/manifest/client-manifest.js", import.meta.url).pathname;
256244
if (id === VIRTUAL_MODULES.middleware) {
257245
if (start.middleware) return await this.resolve(start.middleware);
258246

259247
return `\0${VIRTUAL_MODULES.middleware}`;
260248
}
261249
},
262250
async load(id) {
263-
if (id === `\0${VIRTUAL_MODULES.serverManifest}`) {
251+
if (id === `\0${VIRTUAL_MODULES.clientViteManifest}`) {
252+
let clientViteManifest: Record<string, Record<string, any>>;
253+
264254
if (this.environment.config.command === "serve") {
265-
const manifest: StartServerManifest = {
266-
clientViteManifest: {},
267-
};
255+
clientViteManifest = {};
256+
} else {
257+
const entry = Object.values(globalThis.START_CLIENT_BUNDLE).find(
258+
v => "isEntry" in v && v.isEntry
259+
);
260+
if (!entry) throw new Error("No client entry found");
268261

269-
return `export const manifest = ${JSON.stringify(manifest)}`;
262+
clientViteManifest = JSON.parse(
263+
(globalThis.START_CLIENT_BUNDLE[".vite/manifest.json"] as any).source
264+
);
270265
}
271266

272-
const entry = Object.values(globalThis.START_CLIENT_BUNDLE).find(
273-
v => "isEntry" in v && v.isEntry
274-
);
275-
if (!entry) throw new Error("No client entry found");
276-
277-
const clientManifest: Record<string, Record<string, any>> = JSON.parse(
278-
(globalThis.START_CLIENT_BUNDLE[".vite/manifest.json"] as any).source
279-
);
280-
281-
const manifest: StartServerManifest = {
282-
clientViteManifest: clientManifest as any,
283-
};
284-
285-
return `export const manifest = ${JSON.stringify(manifest)};`;
286-
} else if (id.startsWith("/@manifest")) {
267+
return `export const clientViteManifest = ${JSON.stringify(clientViteManifest)};`;
268+
}
269+
else if (id === `\0${VIRTUAL_MODULES.middleware}`) return "export default {};"
270+
else if (id.startsWith("/@manifest")) {
287271
const [path, query] = id.split("?");
288272
const params = new URLSearchParams(query);
289273
if (!path || !query) return;
@@ -296,7 +280,7 @@ function solidStartVitePlugin(options?: SolidStartOptions): Array<PluginOption>
296280
await getSsrDevManifest("server").getAssets(id)
297281
)}`;
298282
}
299-
} else if (id === `\0${VIRTUAL_MODULES.middleware}`) return "export default {};"
283+
}
300284
}
301285
},
302286
nitroPlugin({ root: process.cwd() }, () => ssrBundle, start.server),

‎packages/start/src/constants.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export const CLIENT_BASE_PATH = "_build";

‎packages/start/src/router.tsx‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
import { getRequestEvent, isServer } from "solid-js/web";
2-
import { getManifest } from "solid-start:get-manifest";
32

43
import lazyRoute from "./server/lazyRoute.jsx";
54
import type { PageEvent } from "./server/types.js";
65
import { pageRoutes as routeConfigs } from "./server/routes.js";
6+
import { getSsrManifest } from "./server/manifest/ssr-manifest.js";
77

88
export function createRoutes() {
99
function createRoute(route: any) {
@@ -16,7 +16,11 @@ export function createRoutes() {
1616
},
1717
component:
1818
route.$component &&
19-
lazyRoute(route.$component, getManifest(import.meta.env.START_ISLANDS), getManifest(true)),
19+
lazyRoute(
20+
route.$component,
21+
getSsrManifest(import.meta.env.START_ISLANDS ? "server" : "client"),
22+
getSsrManifest("server")
23+
),
2024
children: route.children ? route.children.map(createRoute) : undefined
2125
};
2226
}

‎packages/start/src/server/StartServer.tsx‎

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,11 @@ import {
99
ssr,
1010
useAssets
1111
} from "solid-js/web";
12-
import { getManifest } from "solid-start:get-manifest";
13-
import { manifest } from "solid-start:server-manifest";
1412

1513
import { ErrorBoundary, TopErrorBoundary } from "../shared/ErrorBoundary.jsx";
1614
import { renderAsset } from "./renderAsset.jsx";
17-
import { getClientEntryPath } from "./server-manifest.js";
1815
import type { Asset, DocumentComponentProps, PageEvent } from "./types.js";
16+
import { getSsrManifest } from "./manifest/ssr-manifest.js";
1917

2018
const docType = ssr("<!DOCTYPE html>");
2119

@@ -46,7 +44,7 @@ export function StartServer(props: { document: Component<DocumentComponentProps>
4644
let assets: Asset[] = [];
4745
Promise.resolve()
4846
.then(async () => {
49-
const manifest = getManifest(import.meta.env.START_ISLANDS);
47+
const manifest = getSsrManifest(import.meta.env.START_ISLANDS);
5048

5149
let assetPromises: Promise<Asset[]>[] = [];
5250
// @ts-ignore
@@ -93,7 +91,12 @@ export function StartServer(props: { document: Component<DocumentComponentProps>
9391
nonce={nonce}
9492
innerHTML={`window.manifest = ${JSON.stringify(context.manifest)}`}
9593
/>
96-
<script type="module" nonce={nonce} async src={getClientEntryPath()} />
94+
<script
95+
type="module"
96+
nonce={nonce}
97+
async
98+
src={getSsrManifest("client").path(import.meta.env.START_CLIENT_ENTRY)}
99+
/>
97100
</>
98101
}
99102
>

‎packages/start/src/server/collect-styles.ts‎

Lines changed: 1 addition & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
import path from "node:path";
22
import { join, resolve } from "pathe";
33
import type { ModuleNode, ViteDevServer } from "vite";
4-
5-
import { CLIENT_BASE_PATH } from "../config/index.js";
4+
import { CLIENT_BASE_PATH } from "../constants";
65

76
async function getViteModuleNode(vite: ViteDevServer, file: string, ssr = false) {
87
let nodePath = file;
@@ -162,34 +161,3 @@ export async function findStylesInModuleGraph(vite: ViteDevServer, id: string, s
162161

163162
return styles
164163
}
165-
166-
export async function getManifestEntryCssTags(id: string) {
167-
if (import.meta.env.DEV) {
168-
const styles = await findStylesInModuleGraph((globalThis as any).VITE_DEV_SERVER, id, false);
169-
170-
return Object.entries(
171-
styles
172-
).map(([key, value]) => ({
173-
tag: "style",
174-
attrs: {
175-
type: "text/css",
176-
key,
177-
"data-vite-dev-id": key,
178-
"data-vite-ref": "0",
179-
},
180-
children: value,
181-
}))
182-
} else {
183-
const { manifest } = await import("solid-start:server-manifest");
184-
185-
const entry = manifest.clientViteManifest[id];
186-
if (!entry) throw new Error(`No entry '${id}' found in vite manifest`);
187-
188-
return (
189-
entry.css?.map(css => ({
190-
tag: "link",
191-
attrs: { href: `/${"_build"}/${css}`, rel: "stylesheet" }
192-
})) ?? []
193-
);
194-
}
195-
}

‎packages/start/src/server/handler.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import { createRoutes } from "../router.jsx";
1010
import { getFetchEvent } from "./fetchEvent.js";
1111
import { matchAPIRoute } from "./routes.js";
1212
import { handleServerFunction } from "./server-functions-handler.js";
13-
import { getClientEntryCssTags } from "./server-manifest.js";
1413
import type { APIEvent, FetchEvent, HandlerOptions, PageEvent } from "./types.js";
1514
import { getSsrManifest } from "./manifest/ssr-manifest.js";
1615

‎packages/start/src/server/manifest/client-manifest.ts‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,3 @@ import { getClientProdManifest } from "./prod-client-manifest.js";
44
export function getClientManifest() {
55
return import.meta.env.DEV ? getClientDevManifest() : getClientProdManifest()
66
}
7-
8-
export { getClientManifest as getManifest }

‎packages/start/src/server/manifest/dev-client-manifest.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,5 +14,5 @@ export function getClientDevManifest() {
1414

1515
return (await import(/* @vite-ignore */ assetsPath)).default;
1616
},
17-
} satisfies StartManifest;
17+
} satisfies StartManifest & { import(id: string): Promise<any> };
1818
}

packages/start/src/server/manifest/dev-server-manifest.ts renamed to packages/start/src/server/manifest/dev-ssr-manifest.ts

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,13 @@
1-
import { isAbsolute, join } from "pathe";
2-
import type { ViteDevServer } from "vite";
1+
import { join, resolve } from "pathe";
2+
import { normalizePath, type ViteDevServer } from "vite";
33

44
import { findStylesInModuleGraph } from "../collect-styles.js";
55

66
export function getSsrDevManifest(target: "client" | "server") {
77
const vite: ViteDevServer = (globalThis as any).VITE_DEV_SERVER;
88

99
return {
10-
import(id: string) {
11-
const absolutePath = isAbsolute(id) ? id : join(process.cwd(), id);
12-
return vite.ssrLoadModule(join(absolutePath));
13-
},
10+
path: (id: string) => normalizePath(join("/@fs", resolve(process.cwd(), id))),
1411
async getAssets(id: string) {
1512
const styles = await findStylesInModuleGraph(vite, id, target === "server");
1613

@@ -25,7 +22,11 @@ export function getSsrDevManifest(target: "client" | "server") {
2522
children: value,
2623
}));
2724
},
28-
} satisfies StartManifest;
25+
} satisfies StartManifest & {
26+
path(
27+
id: string,
28+
): string
29+
}
2930
}
3031

3132
export { getSsrDevManifest as getSsrManifest }

‎packages/start/src/server/manifest/prod-client-manifest.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,5 @@ export function getClientProdManifest() {
1212
// @ts-ignore
1313
return window.manifest
1414
},
15-
} satisfies StartManifest & { json(): Promise<Record<string, any>> }
15+
} satisfies StartManifest & { json(): Promise<Record<string, any>>, import(id: string): Promise<any>; }
1616
}

0 commit comments

Comments
 (0)