Skip to content

Commit 2de59cf

Browse files
committed
refactor: remove pre-rendering complexity (Puppeteer, serve-handler)
- Remove puppeteer and serve-handler from devDependencies - Delete scripts/prerender.js - Remove build:prerender script from package.json - Remove pre-rendering section from README.md - Remove PUPPETEER_SKIP_DOWNLOAD from CI workflow Rationale: Over-engineered solution. Modern search engines execute JavaScript and index React apps. SEO meta tags are sufficient.
1 parent 56bf66e commit 2de59cf

7 files changed

Lines changed: 4 additions & 1272 deletions

File tree

.github/workflows/deploy.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,6 @@ jobs:
2929

3030
- name: Install dependencies
3131
run: npm ci
32-
env:
33-
PUPPETEER_SKIP_DOWNLOAD: true
3432

3533
- name: Detect base path
3634
id: base
243 KB
Loading
240 KB
Loading

README.md

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -21,20 +21,6 @@ npm run build # Production build
2121
npm run preview # Preview build locally
2222
```
2323

24-
## Pre-rendering for SEO
25-
26-
The app includes a Puppeteer-based pre-rendering script for better SEO.
27-
28-
**Local pre-rendering (before release):**
29-
30-
```bash
31-
npm run build:prerender
32-
```
33-
34-
This renders the full React app to static HTML in `dist/index.html`.
35-
36-
**Note:** Pre-rendering is not run in CI due to Puppeteer/Chrome setup complexity. The deployed version is a standard SPA (search engines with JS support will still index it, just slower).
37-
3824
## Documentation
3925

4026
- Full docs: `npm run docs``dist/docs/`

0 commit comments

Comments
 (0)