refactor(rsc-mf): normalize raw action ids before bridge prefix

Author: cursoragent

tests/integration/rsc-mf/remote/src/runtime/registerServerCallback.ts

@@ -7,13 +7,23 @@ import {
 
 let registeredCallbackKey = '';
 const ALIAS_TOKEN_PATTERN = /^[A-Za-z0-9_.-]+$/;
+const getNormalizedRawActionId = (rawActionId: string) => {
+  const normalizedRawActionId = rawActionId.trim();
+  if (!normalizedRawActionId || /\s/.test(normalizedRawActionId)) {
+    throw new Error(
+      `Remote action id must be a non-empty token without whitespace. Received: ${rawActionId}`,
+    );
+  }
+  return normalizedRawActionId;
+};
 const getHostActionId = (rawActionId: string, remoteAlias: string) => {
-  if (rawActionId.startsWith('remote:')) {
-    return rawActionId;
+  const normalizedRawActionId = getNormalizedRawActionId(rawActionId);
+  if (normalizedRawActionId.startsWith('remote:')) {
+    return normalizedRawActionId;
   }
 
   // Align with RSC bridge action-id format expected by host runtime plugin.
-  return `remote:${remoteAlias}:${rawActionId}`;
+  return `remote:${remoteAlias}:${normalizedRawActionId}`;
 };
 const getNormalizedRemoteActionUrl = (remoteOrigin: string) => {
   const url = new URL(remoteOrigin);

tests/integration/rsc-mf/tests/index.test.ts

@@ -618,27 +618,11 @@ function runTests({ mode }: TestConfig) {
       expect(runtimeRegisterSource).not.toContain('127.0.0.1:');
       expect(runtimeRegisterSource).not.toContain('window.location');
       expect(runtimeRegisterSource).toContain("remoteAlias = 'rscRemote'");
-      expect(runtimeRegisterSource).toContain(
-        'const ALIAS_TOKEN_PATTERN = /^[A-Za-z0-9_.-]+$/',
-      );
-      expect(runtimeRegisterSource).toContain(
-        'const normalizedRemoteAlias = remoteAlias.trim()',
-      );
-      expect(runtimeRegisterSource).toContain(
-        "normalizedRemoteAlias.includes(':')",
-      );
-      expect(runtimeRegisterSource).toContain(
-        '!ALIAS_TOKEN_PATTERN.test(normalizedRemoteAlias)',
-      );
       expect(runtimeRegisterSource).toContain(
         'Remote alias must be a non-empty token (letters, numbers, "-", "_", ".") without ":" delimiters',
       );
-      expect(runtimeRegisterSource).toContain(
-        "if (rawActionId.startsWith('remote:'))",
-      );
-      expect(runtimeRegisterSource).toContain(
-        'return `remote:${remoteAlias}:${rawActionId}`',
-      );
+      expect(runtimeRegisterSource).toContain("startsWith('remote:')");
+      expect(runtimeRegisterSource).toContain('return `remote:${remoteAlias}:');
       expect(runtimeRegisterSource).toContain("'x-rsc-action': hostActionId");
       expect(runtimeRegisterSource).toContain("method: 'POST'");
       expect(runtimeRegisterSource).toContain("Accept: 'text/x-component'");