Unify ReactOnServer.node and .full

Author: alexeyr

CHANGELOG.md

@@ -18,13 +18,9 @@ Please follow the recommendations outlined at [keepachangelog.com](http://keepac
 ### [Unreleased]
 Changes since the last non-beta release.
 
-#### Breaking
-- Reduced bundle size [PR 1697](https://github.com/shakacode/react_on_rails/pull/1697) by [Romex91](https://github.com/Romex91)
-  - Migrated from CJS to ESM for more compact modules (~1KB improvement). **Breaking change:** Dropped CJS support. All projects running `require('react-on-rails')` will need to update to ESM `import ReactOnRails from 'react-on-rails'`.
-  - Add export option 'react-on-rails/client' to avoid shipping server-rendering code to browsers (~14KB improvement).
-
 #### Fixed
 - Fix obscure errors by introducing FULL_TEXT_ERRORS [PR 1695](https://github.com/shakacode/react_on_rails/pull/1695) by [Romex91](https://github.com/Romex91).
+- Remove server-side-only functions from client bundles [PR 1697](https://github.com/shakacode/react_on_rails/pull/1697) by [Romex91](https://github.com/Romex91).
 
 ### [14.1.1] - 2025-01-15
 

knip.ts

@@ -4,7 +4,7 @@ const config: KnipConfig = {
   // ! at the end means files are used in production
   workspaces: {
     '.': {
-      entry: ['node_package/src/ReactOnRails.ts!', 'node_package/src/ReactOnRails.node.ts!'],
+      entry: ['node_package/src/ReactOnRails.server.ts!'],
       project: ['node_package/src/**/*.[jt]s!', 'node_package/tests/**/*.[jt]s'],
       babel: {
         config: ['node_package/babel.config.js'],

node_package/src/ReactOnRails.client.ts

@@ -240,7 +240,7 @@ ctx.ReactOnRails = {
    * @param options
    */
   serverRenderReactComponent(): null | string | Promise<RenderResult> {
-    throw new Error('serverRenderReactComponent is not available in "react-on-rails/client". Import "react-on-rails" server-side.');
+    throw new Error('serverRenderReactComponent is not available in "react-on-rails.client". Import "react-on-rails" server-side.');
   },
 
   /**
@@ -256,7 +256,7 @@ ctx.ReactOnRails = {
    * @param options
    */
   handleError(): string | undefined {
-    throw new Error('handleError is not available in "react-on-rails/client". Import "react-on-rails" server-side.');
+    throw new Error('handleError is not available in "react-on-rails.client". Import "react-on-rails" server-side.');
   },
 
   /**

node_package/src/ReactOnRails.full.ts

@@ -0,0 +1,18 @@
+import handleError from './handleError';
+import serverRenderReactComponent from './serverRenderReactComponent';
+import streamServerRenderedReactComponent from './streamServerRenderedReactComponent';
+
+import ReactOnRails from './ReactOnRails.client';
+
+if (typeof window !== 'undefined') {
+  console.log("This file shouldn't be loaded in the browser, your configuration may be wrong");
+}
+
+ReactOnRails.handleError = handleError;
+
+ReactOnRails.serverRenderReactComponent = serverRenderReactComponent;
+
+ReactOnRails.streamServerRenderedReactComponent = streamServerRenderedReactComponent;
+
+export * from "./types";
+export default ReactOnRails;

node_package/src/ReactOnRails.node.ts

@@ -47,23 +47,23 @@ type RenderFunctionResult = ReactComponent | ServerRenderResult | Promise<string
 /**
  * Render functions are used to create dynamic React components or server-rendered HTML with side effects.
  * They receive two arguments: props and railsContext.
- * 
+ *
  * @param props - The component props passed to the render function
  * @param railsContext - The Rails context object containing environment information
  * @returns A string, React component, React element, or a Promise resolving to a string
- * 
+ *
  * @remarks
  * To distinguish a render function from a React Function Component:
  * 1. Ensure it accepts two parameters (props and railsContext), even if railsContext is unused, or
  * 2. Set the `renderFunction` property to `true` on the function object.
- * 
+ *
  * If neither condition is met, it will be treated as a React Function Component,
  * and ReactDOMServer will attempt to render it.
- * 
+ *
  * @example
  * // Option 1: Two-parameter function
  * const renderFunction = (props, railsContext) => { ... };
- * 
+ *
  * // Option 2: Using renderFunction property
  * const anotherRenderFunction = (props) => { ... };
  * anotherRenderFunction.renderFunction = true;
@@ -82,7 +82,6 @@ export type { // eslint-disable-line import/prefer-default-export
   ReactComponent,
   AuthenticityHeaders,
   RenderFunction,
-  RenderFunctionResult,
   Store,
   StoreGenerator,
   CreateReactOutputResult,
@@ -131,7 +130,7 @@ export interface ErrorOptions {
   serverSide: boolean;
 }
 
-export type RenderingError = Pick<Error, 'message' | 'stack'>;
+type RenderingError = Pick<Error, 'message' | 'stack'>;
 
 export interface RenderResult {
   html: string | null;

node_package/src/ReactOnRails.server.ts

@@ -25,9 +25,9 @@ scripts:
       # 3. Check if the project is built now;
       # 4. If it failed, print an error message (still follow https://docs.npmjs.com/cli/v8/using-npm/scripts#best-practices).
       script: >
-        [ -f node_package/lib/ReactOnRails.full.js ] ||
+        [ -f node_package/lib/ReactOnRails.server.js ] ||
           (npm run build >/dev/null 2>&1 || true) &&
-          [ -f node_package/lib/ReactOnRails.full.js ] ||
+          [ -f node_package/lib/ReactOnRails.server.js ] ||
           { echo 'Building react-on-rails seems to have failed!'; }
 
   format:

node_package/src/types/index.ts

@@ -4,10 +4,10 @@
   "description": "react-on-rails JavaScript for react_on_rails Ruby gem",
   "exports": {
     ".": {
-      "node": "./node_package/lib/ReactOnRails.node.js",
-      "default": "./node_package/lib/ReactOnRails.full.js"
-    },
-    "./client": "./node_package/lib/ReactOnRails.client.js"
+      "react-server": "./node_package/lib/ReactOnRails.server.js",
+      "browser": "./node_package/lib/ReactOnRails.client.js",
+      "default": "./node_package/lib/ReactOnRails.server.js"
+    }
   },
   "directories": {
     "doc": "docs"