fix: support connectionless remote modules

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: heswell

docs/rfc/vuu_table_browser.md

@@ -233,6 +233,9 @@ The browser's registry descriptor is expected to be equivalent to:
 ```
 
 The descriptor intentionally has no `vuu` property.
+The host `RemoteModule` MUST therefore render the browser without installing a
+connection-scoped `AuthenticationProvider`; remotes that require Vuu data MUST
+provide explicit `vuu` metadata.
 
 ### 2. Navigation
 

vuu-ui/packages/core/src/remote-module/RemoteModule.tsx

@@ -29,8 +29,6 @@ export interface RemoteModuleProps<
   width?: number;
 }
 
-const portalConnection = { connectionId: "portal" };
-
 const getLazyComponent = (
   scope: string,
   component: string,
@@ -95,6 +93,9 @@ function RawRemoteModule<ComponentProps extends object | undefined>({
   ...remoteProps
 }: RemoteModuleProps<ComponentProps>) {
   const RemoteComponent = getRemoteComponent(mfUrl, mfScope, mfComponent);
+  const remoteComponent = (
+    <RemoteComponent {...remoteProps} {...componentProps} />
+  );
 
   return (
     <RemoteModuleErrorBoundary
@@ -106,12 +107,13 @@ function RawRemoteModule<ComponentProps extends object | undefined>({
         onError?.(error);
       }}
     >
-      <AuthenticationProvider
-        mode="vuu-connection"
-        connection={vuu ?? portalConnection}
-      >
-        <RemoteComponent {...remoteProps} {...componentProps} />
-      </AuthenticationProvider>
+      {vuu ? (
+        <AuthenticationProvider mode="vuu-connection" connection={vuu}>
+          {remoteComponent}
+        </AuthenticationProvider>
+      ) : (
+        remoteComponent
+      )}
     </RemoteModuleErrorBoundary>
   );
 }

vuu-ui/packages/core/test/remote-module/RemoteModule.test.tsx

@@ -0,0 +1,47 @@
+import { act, Suspense } from "react";
+import { createRoot, type Root } from "react-dom/client";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+
+vi.mock("@module-federation/enhanced/runtime", () => ({
+  loadRemote: vi.fn().mockResolvedValue({
+    default: () => <div>Connectionless remote loaded</div>,
+  }),
+  registerRemotes: vi.fn(),
+}));
+
+import { RemoteModule } from "../../src/remote-module/RemoteModule";
+
+describe("RemoteModule", () => {
+  let container: HTMLDivElement;
+  let root: Root;
+
+  beforeEach(() => {
+    globalThis.IS_REACT_ACT_ENVIRONMENT = true;
+    container = document.createElement("div");
+    document.body.append(container);
+    root = createRoot(container);
+    vi.spyOn(console, "error").mockImplementation(() => {});
+  });
+
+  afterEach(async () => {
+    await act(async () => root.unmount());
+    container.remove();
+    vi.restoreAllMocks();
+  });
+
+  it("loads a remote without a Vuu connection when metadata is absent", async () => {
+    await act(async () => {
+      root.render(
+        <Suspense fallback="Loading">
+          <RemoteModule
+            mfComponent="ConnectionlessRemote"
+            mfScope="connectionless"
+            mfUrl="http://localhost:5000"
+          />
+        </Suspense>,
+      );
+    });
+
+    expect(container.textContent).toBe("Connectionless remote loaded");
+  });
+});