Avoid probing optional page renderer exports

Author: cpojer

packages/react/test/integration/react-pages-client.test.ts

@@ -694,6 +694,28 @@ describe('react-pages client (dev)', () => {
     expect(stderr).not.toMatch(/\(void_worker\).*new dependencies optimized: @void-repro/);
   });
 
+  it('dev: client-only shells retain the React preamble and mount loader props', async () => {
+    const response = await fetch(`${DEV_BASE}/client-only`);
+    const html = await response.text();
+    expect(response.status).toBe(200);
+    expect(html).toContain('data-void-ssr="false"');
+    expect(html).toContain('__vite_plugin_react_preamble_installed__ = true');
+    expect(html).not.toContain('client-only page component rendered on the server');
+
+    const context = await browser.newContext();
+    try {
+      const page = await context.newPage();
+      await page.goto(`${DEV_BASE}/client-only`);
+      await page.waitForSelector('#app[data-hydrated="true"]');
+      expect(await page.textContent('[data-testid="client-only-title"]')).toBe('Client-only Route');
+      expect(await page.textContent('[data-testid="client-only-marker"]')).toBe(
+        'loader-props-available',
+      );
+    } finally {
+      await context.close();
+    }
+  });
+
   it('dev: SSR pages, Inertia protocol, form handling, API coexistence', async () => {
     // 1. Initial page load returns SSR HTML with embedded page data
     const rootRes = await fetch(`${DEV_BASE}/`);

packages/void/src/router/compile.ts

@@ -997,7 +997,7 @@ function compilePageRoutes(
   options: CompileOptions,
 ): void {
   // Import protocol helpers and renderer
-  lines.push(`import * as __voidPagesRenderer from "virtual:void-pages-renderer";`);
+  lines.push(`import { renderPage } from "virtual:void-pages-renderer";`);
   if (options.dev) {
     lines.push(`import { getVoidPagesDevCss } from "virtual:void-pages-dev-assets";`);
   }
@@ -1008,12 +1008,6 @@ function compilePageRoutes(
     lines.push(`import { renderIslandPage } from "virtual:void-islands-renderer";`);
   }
   lines.push('');
-  lines.push(`const renderPage = __voidPagesRenderer.renderPage;`);
-  lines.push(`if (typeof __voidPagesRenderer.renderPreambleScript === "string") {`);
-  lines.push(`  renderPage.preambleScript = __voidPagesRenderer.renderPreambleScript;`);
-  lines.push(`}`);
-  lines.push('');
-
   // Emit head config constant from void.json
   lines.push(`const __voidHeadConfig = ${JSON.stringify(pages.headConfig ?? null)};`);
   lines.push('');

packages/void/test/integration/typed-routing-runtime.test.ts

@@ -76,6 +76,19 @@ describe('typed routing in generated Workers', () => {
           renderMode: 'server',
           shouldPrerender: false,
         },
+        {
+          pattern: '/client-only',
+          params: [],
+          catchAll: false,
+          componentPath: 'client-only.tsx',
+          componentId: 'client-only',
+          serverPath: null,
+          methods: [],
+          ssrMode: 'client',
+          prerenderMode: 'disabled',
+          renderMode: 'client',
+          shouldPrerender: false,
+        },
       ];
       const routeTable = join(directory, 'generated-routes.ts');
       const entry = join(directory, 'generated-worker.ts');
@@ -113,6 +126,10 @@ describe('typed routing in generated Workers', () => {
         ),
       );
       writeFileSync(entry, compileWorkerEntry({ runtimeEnvPath: source('env.ts') }));
+      const preambleScript = dev
+        ? '<script type="module">window.__fixturePreamble = true</script>'
+        : '';
+      const undefinedImports: Array<string> = [];
       await build({
         configFile: false,
         logLevel: 'silent',
@@ -134,7 +151,8 @@ describe('typed routing in generated Workers', () => {
             },
             load(id) {
               if (id === '\0virtual:void-pages-renderer') {
-                return `export async function renderPage(page) { return '<html><body>' + page.props.title + '</body></html>'; }`;
+                return `export async function renderPage(page) { return '<html><body>' + page.props.title + '</body></html>'; }
+renderPage.preambleScript = ${JSON.stringify(preambleScript)};`;
               }
               if (id === '\0virtual:void-pages-dev-assets') {
                 return 'export async function getVoidPagesDevCss() { return []; }';
@@ -150,11 +168,18 @@ describe('typed routing in generated Workers', () => {
           ssr: entry,
           target: 'esnext',
           rollupOptions: {
+            onwarn(warning, defaultHandler) {
+              if (warning.code === 'IMPORT_IS_UNDEFINED') {
+                undefinedImports.push(warning.message);
+              }
+              defaultHandler(warning);
+            },
             external: [/^cloudflare:/, /^node:/],
             output: { entryFileNames: 'worker.mjs', codeSplitting: false },
           },
         },
       });
+      expect(undefinedImports).toEqual([]);
       runtime = new Miniflare(
         convertV4MiniflareOptions({
           compatibilityDate: '2026-07-01',
@@ -201,6 +226,11 @@ describe('typed routing in generated Workers', () => {
       }
       const page = await runtime.dispatchFetch('https://app.test/');
       expect(await page.text()).toContain('real page loader');
+      const clientOnlyPage = await runtime.dispatchFetch('https://app.test/client-only');
+      const clientOnlyHtml = await clientOnlyPage.text();
+      expect(clientOnlyPage.status).toBe(200);
+      expect(clientOnlyHtml).toContain('data-void-ssr="false"');
+      expect(clientOnlyHtml.includes('window.__fixturePreamble = true')).toBe(dev);
       const protocol = await runtime.dispatchFetch('https://app.test/', {
         headers: { 'x-voidpages': 'true' },
       });

packages/void/test/unit/pages-compile.test.ts

@@ -77,7 +77,7 @@ describe('Pages metadata wiring', () => {
     );
   it('emits lazy imports and renderer/action metadata with typed execution calls', () => {
     const code = compile();
-    expect(code).toContain('import * as __voidPagesRenderer from "virtual:void-pages-renderer"');
+    expect(code).toContain('import { renderPage } from "virtual:void-pages-renderer"');
     expect(code).toContain(
       'const _pload_index = __createModuleLoader(() => import("/project/pages/index.server.ts"))',
     );