[Workers] Fix typos in React Router guide

[edmundhung]

Summary

Just notice a few typos and a legacy flag mentioned in the React Router guide.

Screenshots (optional)

Documentation checklist

• Is there a changelog entry (guidelines)? If you don't add one for something awesome and new (however small) — how will our customers find out? Changelogs are automatically posted to RSS feeds, the Discord, and X.
• The change adheres to the documentation style guide.
• If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
• Files which have changed name or location have been allocated redirects.

[edmundhung]

This is stabilized in v7.10.0

[github-actions]

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern	Owners
/src/content/docs/workers/	@cloudflare/workers-docs, @GregBrimble, @irvinebroque, @mikenomitch, @korinne, @WalshyDev, @cloudflare/deploy-config, @cloudflare/pcx-technical-writing, @cloudflare/wrangler, @mattietk

[github-actions]

Preview URL: https://304527a3.preview.developers.cloudflare.com
Preview Branch URL: https://edmundhung-react-router-guide-update.preview.developers.cloudflare.com

Files with changes (up to 15)

Original Link	Updated Link
https://developers.cloudflare.com/workers/framework-guides/web-apps/react-router/	https://edmundhung-react-router-guide-update.preview.developers.cloudflare.com/workers/framework-guides/web-apps/react-router/

[elithrar]

/bonk review this PR

[ask-bonk]

I can see the changes have already been applied to the base branch file (the diff shows the before/after). Let me review each change carefully.

PR Review: [Workers] Fix typos in React Router guide

This is a clean, well-scoped PR with 5 corrections across 1 file. All changes are legitimate fixes. Here's the breakdown:

Change 1: future.unstable_viteEnvironmentApi -> future.v8_viteEnvironmentApi (line 75)

Correct. The author notes this was stabilized in React Router v7.10.0. The unstable_ prefix was dropped and renamed to v8_viteEnvironmentApi. The docs should reflect the stable API name.

Change 2: WorkflowEvent<Params> -> WorkflowEvent<{ hello: string }> (line 147)

Correct. The Params type was never defined anywhere in the code example. Using { hello: string } is consistent with the create({ params: { "hello": "world" } }) call later in the example, making the code snippet self-contained and actually compilable.

Change 3: Route.LoaderArgs -> Route.ActionArgs (line 197)

Correct. The function is named action, not loader. An action function should use Route.ActionArgs, not Route.LoaderArgs. This was a clear copy-paste typo.

Change 4: Missing closing brace }) -> } ) (line 199)

Correct. The original had { "hello": "world" }) which is missing the closing } for the params object literal. The fix adds the missing brace: { "hello": "world" } ).

Change 5: instance.status() -> await instance.status() (line 200)

Correct. instance.status() returns a Promise, so it needs to be awaited to get the actual status value. Without await, the returned object would contain a pending Promise rather than the resolved status.

Verdict

All 5 fixes are valid and improve the correctness of the documentation. No issues found. This PR is ready to merge.

github run

[github-actions]

Hey there, we've marked this pull request as stale because there's no recent activity on it. This label is helps us identify PRs that might need updates (or to be closed out by our team if no longer relevant).