Skip to content

refactor: split homepage component boundaries - #1356

Closed
jderochervlk wants to merge 2 commits into
test/homepage-performance-guardrailsfrom
refactor/homepage-component-boundaries
Closed

jderochervlk wants to merge 2 commits into
test/homepage-performance-guardrailsfrom
refactor/homepage-component-boundaries

Conversation

@jderochervlk

@jderochervlk jderochervlk commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

Context

Make the homepage easier to optimize incrementally by giving each section a focused module and removing eager module work. This is the second PR in #1357 and depends on #1355.

Changes

  • split the seven homepage sections into private ReScript component modules
  • remove the unused MarkdownComponents prop and its homepage dependency edge
  • replace static useState and module-scope JSX with immutable data and render-time elements
  • ratchet the measured homepage budgets after the refactor

Measured impact

  • initial JavaScript requests: 21 to 20
  • initial JavaScript gzip: 284,515 to 279,510 bytes
  • initial CSS: 80,247 to 80,205 raw bytes
  • DOM and media counts unchanged

Raw JavaScript increases by 12,287 bytes because React Router expands its highly compressible generated manifest for the changed chunk graph; transferred gzip size falls by 5,005 bytes overall.

Relates to #1354.


Stack created with GitHub Stacks CLI

@jderochervlk
jderochervlk added this pull request to stack #1357 September 18, 2026 11:53
@jderochervlk jderochervlk changed the title refactor/homepage component boundaries refactor: split homepage component boundaries Sep 18, 2026
Extract each homepage section into a focused module, remove the unused MarkdownComponents dependency, and avoid eagerly creating static JSX at module initialization.
Record the component-boundary layer's lower initial request count, gzip payload, and CSS size while preserving the observed raw React Router manifest tradeoff.
@jderochervlk
jderochervlk force-pushed the refactor/homepage-component-boundaries branch from 6568387 to de5d1c9 Compare September 18, 2026 11:55
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 18, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
guide de5d1c9 Commit Preview URL

Branch Preview URL
Sep 18 2026, 12:05 PM

@github-actions

Copy link
Copy Markdown
Contributor

Cloudflare deployment

Deployement ID: cdd2cfbb-f324-44d0-bdd8-cd57353b4fe7
Deployment Environment: preview

⛅️ wrangler 4.130.0
────────────────────
✨ Compiled Worker successfully
Uploading... (8880/8882)
Uploading... (8881/8882)
Uploading... (8882/8882)
✨ Success! Uploaded 2 files (8880 already uploaded) (1.68 sec)

✨ Uploading _redirects
✨ Uploading Functions bundle
🌎 Deploying...
✨ Deployment complete! Take a peek over at https://cdd2cfbb.rescript-lang.pages.dev
✨ Deployment alias URL: https://refactor-homepage-component.rescript-lang.pages.dev

@jderochervlk
jderochervlk removed this pull request from stack #1357 September 18, 2026 12:03
@jderochervlk

Copy link
Copy Markdown
Collaborator Author

Superseded by #1358 after shortening the branch name so Cloudflare can generate a valid preview alias.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant