Skip to content

Conversation

@anyuj
Copy link

@anyuj anyuj commented Oct 30, 2025

  • Why
    As noted on Discord: “right now prerendering does not run using wrangler and hence fails when you reference cloudflare env during prerendering”.
  • What changed
    Documentation update: added a note in the Static Prerendering section.
  • Scope/Impact
    Docs only.

Summary by CodeRabbit

  • Documentation
    • Updated static-prerendering guides for React and Solid frameworks with informational notes clarifying that prerendering on Cloudflare Workers is not yet supported and currently under development. Notes have been added to inform developers of current limitations.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 30, 2025

Walkthrough

Documentation updates to React and Solid static prerendering guides add informational notes indicating that prerendering on Cloudflare Workers is not yet supported and is under development. No logic, configuration, or control flow changes applied.

Changes

Cohort / File(s) Summary
Documentation: Prerendering Status Notes
docs/start/framework/react/guide/static-prerendering.md, docs/start/framework/solid/guide/static-prerendering.md
Added informational notes after code examples documenting that Cloudflare Workers prerendering support is not yet available and remains under development.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Documentation-only changes with no functional impact
  • Simple, repetitive pattern applied consistently across two files
  • No configuration, logic, or API modifications

Poem

🐰 A note for wandering developers true,
"On Workers, prerendering's not quite through!"
This message now guides the path they take,
Future features the docs will awake.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The PR title "docs: add note that prerendering is not yet supported on cf workers" directly and accurately reflects the main changes in the pull request. Both modified documentation files add informational notes about the limitation of prerendering on Cloudflare Workers, which is exactly what the title conveys. The title is concise, uses standard abbreviation conventions (cf for Cloudflare), and avoids generic language or unnecessary details. A teammate scanning the git history would clearly understand that this PR adds documentation about a prerendering limitation on Cloudflare Workers, making it effective for navigation and understanding the changeset's purpose.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a282b97 and 1035eca.

📒 Files selected for processing (2)
  • docs/start/framework/react/guide/static-prerendering.md (1 hunks)
  • docs/start/framework/solid/guide/static-prerendering.md (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (AGENTS.md)

Use internal docs links relative to the docs/ folder (e.g., ./guide/data-loading)

Files:

  • docs/start/framework/react/guide/static-prerendering.md
  • docs/start/framework/solid/guide/static-prerendering.md
docs/{router,start}/**

📄 CodeRabbit inference engine (AGENTS.md)

Place router docs under docs/router/ and start framework docs under docs/start/

Files:

  • docs/start/framework/react/guide/static-prerendering.md
  • docs/start/framework/solid/guide/static-prerendering.md
🔇 Additional comments (2)
docs/start/framework/react/guide/static-prerendering.md (1)

71-71: Clear and appropriately positioned note on Cloudflare Workers limitation.

The note is well-placed immediately after the configuration section and uses consistent formatting with other notes in the document. The wording appropriately signals this is a known limitation under active development.

docs/start/framework/solid/guide/static-prerendering.md (1)

71-71: Consistent cross-framework documentation update.

The note matches the React file addition exactly, ensuring users of both frameworks receive identical information about this limitation. Placement and formatting are consistent.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot added the documentation Everything documentation related label Oct 30, 2025
@daveycodez
Copy link

daveycodez commented Oct 30, 2025

There is a workaround. Simply remove the Cloudflare plugin and use this wrangler -

{
    "$schema": "node_modules/wrangler/config-schema.json",
    "name": "start-intl-example",
    "compatibility_date": "2025-09-02",
    "compatibility_flags": ["nodejs_compat"],
    "main": "./dist/server/server.js",
    "assets": {
        "directory": "dist/client",
        "html_handling": "drop-trailing-slash"
    }
}

Bindings won't work, but Environment variables do. Which suffices for most use cases.

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

Labels

documentation Everything documentation related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants