fix(transforms)!: run the browser server-exports strip before compile

.gitignore

@@ -35,6 +35,7 @@ __test_project__/
 
 # Dependencies
 node_modules/
+node_modules
 
 # NPM config
 # Local npm configs may contain credentials. Track only the root policy file.

docs/guides/data-fetching.md

@@ -45,6 +45,29 @@ entirely, including their top-level side effects. Put client initialization in a
 separate client-referenced module or a bare side-effect import that is not only
 used by a server data hook.
 
+Declare each data hook in the page module itself. Re-exporting one from another
+file fails the browser build, with or without an alias:
+
+```tsx
+// Not supported. Both forms stop the build.
+export { getServerData } from "../server/loaders.ts";
+export { loadDashboard as getServerData } from "../server/loaders.ts";
+```
+
+A re-export binds no local declaration, so Veryfront has nothing to strip and
+the loader module, its imports, and anything it closes over would reach the
+browser. Declare the hook in the page and import the helpers it needs:
+
+```tsx
+// app/dashboard/page.tsx
+import type { DataContext } from "veryfront";
+import { loadDashboard } from "../server/loaders.ts";
+
+export async function getServerData(ctx: DataContext) {
+  return { props: await loadDashboard(ctx) };
+}
+```
+
 The `props` you return are passed to the page component. To read the same props
 data from a layout or nested component without prop-drilling, use
 `usePageContext().data` (see

src/transforms/pipeline/index.ts

@@ -68,9 +68,9 @@ const SSR_PIPELINE: TransformPlugin[] = [
 
 const BROWSER_PIPELINE: TransformPlugin[] = [
   parsePlugin,
+  browserServerExportsStripPlugin, // Drop server-only hooks + their now-unused imports
   compilePlugin,
   cssStripPlugin, // Strip CSS imports before they hit import resolution
-  browserServerExportsStripPlugin, // Drop server-only hooks + their now-unused imports
   browserNodeBuiltinImportsPlugin, // node:* named imports -> namespace + destructure
   resolveImportsPlugin, // Unified import resolution
   finalizePlugin,

src/transforms/pipeline/stages/compile.ts

@@ -15,18 +15,6 @@ const ReflectApply = Reflect.apply;
 const ReflectGetOwnPropertyDescriptor = Reflect.getOwnPropertyDescriptor;
 const SOURCE_MAP_SUFFIX =
   /(^|\r?\n)[\t ]*\/\/[#@][\t ]*sourceMappingURL=[^"'`\s]+[\t ]*(?:\r?\n)?$/;
-export const COMPILE_SOURCE_MAP_DIRECTIVE_METADATA = "compileSourceMapDirective";
-export const COMPILE_SOURCE_MAP_INPUT_METADATA = "compileSourceMapInput";
-
-function trailingSourceMapDirective(code: string): string | undefined {
-  const match = SOURCE_MAP_SUFFIX.exec(code);
-  return match?.[0].slice((match[1] ?? "").length).trimEnd();
-}
-
-function dropTrailingSourceMapDirective(code: string): string {
-  return code.replace(SOURCE_MAP_SUFFIX, "$1");
-}
-
 function appendBeforeSourceMap(code: string, addition: string): string {
   const match = SOURCE_MAP_SUFFIX.exec(code);
   if (match?.index === undefined) return code + addition;
@@ -108,27 +96,10 @@ export const compilePlugin: TransformPlugin = {
         /\bconst\s+MDXLayout\b/.test(code) &&
         !/export\s+\{[^}]*MDXLayout/.test(code)
       ) {
-        // Keep esbuild's directive last. Browser server-hook stripping removes
-        // a stale compile map after rewriting, and a framework export appended
-        // after the directive would otherwise hide that suffix.
+        // Keep esbuild's directive last.
         code = appendBeforeSourceMap(code, "\nexport { MDXLayout };\n");
       }
 
-      const sourceMapDirective = trailingSourceMapDirective(code);
-      if (ctx.target === "browser" && sourceMapDirective) {
-        // Keep the compiler map out of intermediate browser plugins. The
-        // server-export strip stage restores it only when the map-free compile
-        // output is unchanged and no hook is rewritten. This makes the real
-        // comment unambiguous even if a plugin copies its text into a string,
-        // appends code, or removes the hook itself.
-        ctx.metadata.set(COMPILE_SOURCE_MAP_DIRECTIVE_METADATA, sourceMapDirective);
-        code = dropTrailingSourceMapDirective(code);
-        ctx.metadata.set(COMPILE_SOURCE_MAP_INPUT_METADATA, code);
-      } else {
-        ctx.metadata.delete(COMPILE_SOURCE_MAP_DIRECTIVE_METADATA);
-        ctx.metadata.delete(COMPILE_SOURCE_MAP_INPUT_METADATA);
-      }
-
       return code;
     } catch (err) {
       const errorMsg = err instanceof Error ? err.message : String(err);