fix(dev/config): update default RSC entries to support formState for progressive enhanced actions (#14248)

chore(rsc): update default RSC entries to support formState for progressive enahnced actions
chore (rsc): Remove rsc-parcel-framework tests. I'm not maintaining that and I don't want problems against it to slow down progress for library and (vite) framework modes.
fix(rsc): decode each time `getPayload()` is called to allow for "in-context" decoding and hoisting of contextual assets
chore(rsc): use `vite build` for rsc framework fixtures
chore(rsc): Skip flaky RSC Framework Mode HMR test expectation

---------

Co-authored-by: Mark Dalgleish <[email protected]>

Author: jacob-ebey

.changeset/shiny-hotels-guess.md

@@ -0,0 +1,5 @@
+---
+"react-router": patch
+---
+
+decode each time `getPayload()` is called to allow for "in-context" decoding and hoisting of contextual assets

integration/action-test.ts

@@ -11,7 +11,6 @@ import { type TemplateName, reactRouterConfig } from "./helpers/vite.js";
 
 const templateNames = [
   "vite-5-template",
-  "rsc-parcel-framework",
   "rsc-vite-framework",
 ] as const satisfies TemplateName[];
 

integration/catch-boundary-data-test.ts

@@ -11,7 +11,6 @@ import { reactRouterConfig, type TemplateName } from "./helpers/vite.js";
 
 const templateNames = [
   "vite-5-template",
-  "rsc-parcel-framework",
   "rsc-vite-framework",
 ] as const satisfies TemplateName[];
 

integration/helpers/create-fixture.ts

@@ -205,15 +205,6 @@ export async function createFixture(init: FixtureInit, mode?: ServerMode) {
         "Expected a default request handler function export in Vite RSC Framework Mode server build",
       );
     }
-  } else if (templateName === "rsc-parcel-framework") {
-    let serverBuild = await import(buildPath);
-    handler = (serverBuild?.requestHandler ??
-      serverBuild?.default?.requestHandler) as RequestHandler;
-    if (typeof handler !== "function") {
-      throw new Error(
-        "Expected a 'requestHandler' function export in Parcel RSC Framework server build",
-      );
-    }
   } else {
     build = (await import(buildPath)) as ServerBuild;
     handler = createRequestHandler(build, mode || ServerMode.Production);
@@ -489,7 +480,12 @@ export async function createFixtureProject(
   if (templateName.includes("parcel")) {
     parcelBuild(projectDir, init.buildStdio, mode);
   } else {
-    reactRouterBuild(projectDir, init.buildStdio, mode);
+    reactRouterBuild(
+      projectDir,
+      init.buildStdio,
+      mode,
+      templateName.includes("rsc"),
+    );
   }
 
   return projectDir;
@@ -535,6 +531,7 @@ function reactRouterBuild(
   projectDir: string,
   buildStdio?: Writable,
   mode?: ServerMode,
+  isRsc?: boolean,
 ) {
   // We have a "require" instead of a dynamic import in readConfig gated
   // behind mode === ServerMode.Test to make jest happy, but that doesn't
@@ -544,8 +541,9 @@ function reactRouterBuild(
   mode = mode === ServerMode.Test ? ServerMode.Production : mode;
 
   let reactRouterBin = "node_modules/@react-router/dev/dist/cli/index.js";
+  let viteBin = "node_modules/vite/dist/node/cli.js";
 
-  let buildArgs: string[] = [reactRouterBin, "build"];
+  let buildArgs: string[] = [isRsc ? viteBin : reactRouterBin, "build"];
 
   let buildSpawn = spawnSync("node", buildArgs, {
     cwd: projectDir,