Fix module watcher errors
This commit is contained in:
@@ -14,7 +14,11 @@ export default async function grabPageBundledReactComponent({ file_path, return_
|
||||
// Loader.registry.delete(abs);
|
||||
const mod = await import(`${abs}?t=${Date.now()}`);
|
||||
const Main = mod.default;
|
||||
return { component: Main };
|
||||
return {
|
||||
component: Main,
|
||||
page_module: mod.__bunext_page,
|
||||
root_module: mod.__bunext_root,
|
||||
};
|
||||
}
|
||||
const { root_file_path } = grabRootFilePath();
|
||||
let tsx = grabPageReactComponentString({
|
||||
@@ -35,6 +39,8 @@ export default async function grabPageBundledReactComponent({ file_path, return_
|
||||
return {
|
||||
component: Main,
|
||||
tsx,
|
||||
page_module: mod.__bunext_page,
|
||||
root_module: mod.__bunext_root,
|
||||
};
|
||||
}
|
||||
catch (error) {
|
||||
|
||||
+14
-9
@@ -19,20 +19,25 @@ export default async function grabPageModules({ file_path, debug, url, query, ro
|
||||
? 301
|
||||
: serverRes.redirect.status_code || 302);
|
||||
}
|
||||
const { root_file_path } = grabRootFilePath();
|
||||
const root_module = root_file_path
|
||||
? await import(`${root_file_path}?t=${now}`)
|
||||
: undefined;
|
||||
const module = await import(`${file_path}?t=${now}`);
|
||||
if (debug) {
|
||||
log.info(`module:`, module);
|
||||
}
|
||||
const { component } = (await grabPageBundledReactComponent({
|
||||
const { component, page_module, root_module: bundled_root_module, } = (await grabPageBundledReactComponent({
|
||||
file_path,
|
||||
})) || {};
|
||||
if (!component) {
|
||||
throw new Error(`Couldn't grab page component`);
|
||||
}
|
||||
// Prefer the page/root exports re-exported by the SSR bundle. Importing
|
||||
// the raw source with Bun's loader only cache-busts the page file itself:
|
||||
// its transitive imports stay in Bun's module registry, so a component
|
||||
// that failed to resolve once (e.g. imported before it was created)
|
||||
// keeps failing until the server restarts.
|
||||
const { root_file_path } = grabRootFilePath();
|
||||
const root_module = !root_file_path
|
||||
? undefined
|
||||
: bundled_root_module || (await import(`${root_file_path}?t=${now}`));
|
||||
const module = page_module || (await import(`${file_path}?t=${now}`));
|
||||
if (debug) {
|
||||
log.info(`module:`, module);
|
||||
}
|
||||
if (debug) {
|
||||
log.info(`component:`, component);
|
||||
}
|
||||
|
||||
@@ -16,6 +16,12 @@ export default function grabPageReactComponentString({ file_path, root_file_path
|
||||
}
|
||||
tsx += ` )\n`;
|
||||
tsx += `}\n`;
|
||||
// Expose the page/root's own exports (meta, Head, config, ...) through
|
||||
// the bundle so the server never has to import the raw source files.
|
||||
tsx += `export * as __bunext_page from "${file_path}"\n`;
|
||||
if (root_file_path) {
|
||||
tsx += `export * as __bunext_root from "${root_file_path}"\n`;
|
||||
}
|
||||
return tsx;
|
||||
}
|
||||
catch (error) {
|
||||
|
||||
Vendored
+6
@@ -318,6 +318,12 @@ export type GrabPageReactBundledComponentRes = {
|
||||
component?: FC;
|
||||
server_res?: BunextPageModuleServerReturn;
|
||||
tsx?: string;
|
||||
/**
|
||||
* The page's own exports (`meta`, `Head`, `config`, ...), re-exported
|
||||
* from the SSR bundle so they share its fresh module graph.
|
||||
*/
|
||||
page_module?: BunextPageModule;
|
||||
root_module?: BunextPageModule;
|
||||
};
|
||||
export type PageFiles = {
|
||||
local_path: string;
|
||||
|
||||
+1
-1
@@ -1,6 +1,6 @@
|
||||
{
|
||||
"name": "@moduletrace/bunext",
|
||||
"version": "1.1.18",
|
||||
"version": "1.1.19",
|
||||
"main": "dist/index.js",
|
||||
"module": "index.ts",
|
||||
"dependencies": {
|
||||
|
||||
@@ -32,7 +32,11 @@ export default async function grabPageBundledReactComponent({
|
||||
|
||||
const Main = mod.default as FC;
|
||||
|
||||
return { component: Main };
|
||||
return {
|
||||
component: Main,
|
||||
page_module: mod.__bunext_page,
|
||||
root_module: mod.__bunext_root,
|
||||
};
|
||||
}
|
||||
|
||||
const { root_file_path } = grabRootFilePath();
|
||||
@@ -60,6 +64,8 @@ export default async function grabPageBundledReactComponent({
|
||||
return {
|
||||
component: Main,
|
||||
tsx,
|
||||
page_module: mod.__bunext_page,
|
||||
root_module: mod.__bunext_root,
|
||||
};
|
||||
} catch (error: any) {
|
||||
log.error(
|
||||
|
||||
@@ -56,18 +56,11 @@ export default async function grabPageModules({
|
||||
);
|
||||
}
|
||||
|
||||
const { root_file_path } = grabRootFilePath();
|
||||
const root_module: BunextRootModule | undefined = root_file_path
|
||||
? await import(`${root_file_path}?t=${now}`)
|
||||
: undefined;
|
||||
|
||||
const module: BunextPageModule = await import(`${file_path}?t=${now}`);
|
||||
|
||||
if (debug) {
|
||||
log.info(`module:`, module);
|
||||
}
|
||||
|
||||
const { component } =
|
||||
const {
|
||||
component,
|
||||
page_module,
|
||||
root_module: bundled_root_module,
|
||||
} =
|
||||
(await grabPageBundledReactComponent({
|
||||
file_path,
|
||||
})) || {};
|
||||
@@ -76,6 +69,23 @@ export default async function grabPageModules({
|
||||
throw new Error(`Couldn't grab page component`);
|
||||
}
|
||||
|
||||
// Prefer the page/root exports re-exported by the SSR bundle. Importing
|
||||
// the raw source with Bun's loader only cache-busts the page file itself:
|
||||
// its transitive imports stay in Bun's module registry, so a component
|
||||
// that failed to resolve once (e.g. imported before it was created)
|
||||
// keeps failing until the server restarts.
|
||||
const { root_file_path } = grabRootFilePath();
|
||||
const root_module: BunextRootModule | undefined = !root_file_path
|
||||
? undefined
|
||||
: bundled_root_module || (await import(`${root_file_path}?t=${now}`));
|
||||
|
||||
const module: BunextPageModule =
|
||||
page_module || (await import(`${file_path}?t=${now}`));
|
||||
|
||||
if (debug) {
|
||||
log.info(`module:`, module);
|
||||
}
|
||||
|
||||
if (debug) {
|
||||
log.info(`component:`, component);
|
||||
}
|
||||
|
||||
@@ -27,6 +27,13 @@ export default function grabPageReactComponentString({
|
||||
tsx += ` )\n`;
|
||||
tsx += `}\n`;
|
||||
|
||||
// Expose the page/root's own exports (meta, Head, config, ...) through
|
||||
// the bundle so the server never has to import the raw source files.
|
||||
tsx += `export * as __bunext_page from "${file_path}"\n`;
|
||||
if (root_file_path) {
|
||||
tsx += `export * as __bunext_root from "${root_file_path}"\n`;
|
||||
}
|
||||
|
||||
return tsx;
|
||||
} catch (error: any) {
|
||||
log.error(`grabPageReactComponentString Error: ${error.message}`);
|
||||
|
||||
@@ -354,6 +354,12 @@ export type GrabPageReactBundledComponentRes = {
|
||||
component?: FC;
|
||||
server_res?: BunextPageModuleServerReturn;
|
||||
tsx?: string;
|
||||
/**
|
||||
* The page's own exports (`meta`, `Head`, `config`, ...), re-exported
|
||||
* from the SSR bundle so they share its fresh module graph.
|
||||
*/
|
||||
page_module?: BunextPageModule;
|
||||
root_module?: BunextPageModule;
|
||||
};
|
||||
|
||||
export type PageFiles = {
|
||||
|
||||
Reference in New Issue
Block a user