fix(rstest): apply review fixes to the federation plugin

- default off MF dts/manifest/dev machinery in test builds (explicit user
  values win) and warn when forcing asyncStartup, library.name, or
  experiments.optimization.target over explicit user config
- use api.context.callerName to warn outside rstest and default the target
  to browser under the rstest-browser runner (explicit target wins)
- export FEDERATION_PLUGIN_NAME as the stable public plugin name
- register modifyEnvironmentConfig with order 'post' so the externals
  bypass stays ahead of other plugins regardless of registration order
- sniff federation remotes from duck-typed plugins and cover scoped string
  remotes (@scope/remote@url) with a regression test

Author: ScriptedAlchemy

packages/rstest/src/externals-bypass.ts

@@ -2,6 +2,8 @@ import type { Rspack } from '@rsbuild/core';
 
 import { NODE_RUNTIME_PLUGIN } from './runtime-plugin';
 
+const DATA_JAVASCRIPT_REQUEST = /!=!data:text\/javascript(?:;|,)/i;
+
 const isFederationRemoteRequest = (
   request: string,
   remoteNames: Set<string>,
@@ -11,11 +13,16 @@ const isFederationRemoteRequest = (
   }
 
   for (const remoteName of remoteNames) {
-    if (
-      request === remoteName ||
-      request.startsWith(`${remoteName}/`) ||
-      request.startsWith(`${remoteName}@`)
-    ) {
+    if (request === remoteName) {
+      return true;
+    }
+
+    if (!request.startsWith(remoteName)) {
+      continue;
+    }
+
+    const separator = request[remoteName.length];
+    if (separator === '/' || separator === '@') {
       return true;
     }
   }
@@ -27,7 +34,7 @@ export const shouldKeepBundledForFederation = (
   request: string,
   remoteNames?: Set<string>,
 ): boolean => {
-  if (/!=!data:text\/javascript(?:;|,)/i.test(request)) {
+  if (DATA_JAVASCRIPT_REQUEST.test(request)) {
     return true;
   }