Skip to content

Commit 0e047ee

Browse files
authored
docs: fix SSR section framing in React integration guide (#1688)
## Summary - Reframes the SSR section to clarify that the benefit of `dynamic(..., { ssr: false })` is keeping HyperFormula out of the **initial JS bundle sent to the browser**, not skipping the server-side render itself (which is already a non-issue since `useEffect` never runs on the server). ## Test Plan - [ ] Review the updated wording in `docs/guide/integration-with-react.md` <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Documentation-only wording change with no runtime or API impact. > > **Overview** > Updates one sentence in the React integration guide’s SSR section so it no longer suggests `dynamic(..., { ssr: false })` is mainly about avoiding server render. The text now states the goal is keeping **HyperFormula out of the initial JS bundle sent to the browser** (still noting the library is a few hundred kB), which matches the earlier point that `useEffect` already makes the pattern SSR-safe. > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit 46d1d61. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup> <!-- /CURSOR_SUMMARY -->
1 parent 00e5c73 commit 0e047ee

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

docs/guide/integration-with-react.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ In development, React runs effects twice (mount → unmount → mount) to surfac
7575

7676
## Server-side rendering (Next.js App Router)
7777

78-
The component above is already SSR-safe — the engine is constructed in `useEffect`, which never runs on the server. If you still want to skip the initial bundle on the server (it is a few hundred kB), wrap it in a client-only dynamic import.
78+
The component above is already SSR-safe — the engine is constructed in `useEffect`, which never runs on the server. If you still want to keep HyperFormula out of the initial JS bundle sent to the browser (it is a few hundred kB), wrap it in a client-only dynamic import.
7979

8080
In the App Router, `dynamic(..., { ssr: false })` is only allowed inside a client component. Put the dynamic call in a `'use client'` wrapper and import the wrapper from your server page:
8181

0 commit comments

Comments
 (0)