Skip to content

[Flight] Fix the react-server-dom-rspack package contract - #1

Draft
BleedingDev wants to merge 1 commit into
agent/rsdr-flight-reply-parityfrom
agent/rsdr-package-contract
Draft

[Flight] Fix the react-server-dom-rspack package contract#1
BleedingDev wants to merge 1 commit into
agent/rsdr-flight-reply-parityfrom
agent/rsdr-package-contract

Conversation

@BleedingDev

Copy link
Copy Markdown
Owner

Summary

This PR is intentionally stacked within BleedingDev/react on SyMind#1. It should be reviewed after that PR and can be retargeted to SyMind/react:react-server-dom-rspack once it lands.

It addresses only the assembled react-server-dom-rspack package contract.

The package advertises ./plugin, but its wrapper requires a CJS artifact that the package does not build. Rspack’s compiler integration instead lives in @rspack/core through experiments.rsc.createPlugins().

This PR:

  • removes the unresolved ./plugin export and wrapper;
  • removes stale node-register, unbundled, and ESM template residue;
  • restores loadServerAction on the browser server entrypoint;
  • narrows Rspack host configuration to Rspack-owned modules;
  • keeps the supported client, server, and static runtime entries;
  • verifies package-root and non-react-server environment guards;
  • verifies removed subpaths are private and unsupported files are absent from the npm inventory; and
  • verifies conditional-export targets for workerd, deno, worker, node, edge-light, browser, and default conditions.

No working Rspack compiler or runtime implementation is removed. If maintainers prefer an intentionally implemented compatibility plugin entry, that decision can be handled here without blocking SyMind#1.

How did you test this change?

After an exact experimental release rebuild:

  • Assembled-package suite, development: 8/8 passed.
  • Assembled-package suite, production: 8/8 passed.
  • Frozen-lockfile install passed.
  • Targeted lint passed.
  • Prettier check-changed passed.
  • git diff --check passed.

Remove the advertised plugin wrapper that points at an artifact this package does not build, along with stale node-register, unbundled, and ESM template residue. Rspack compiler integration remains owned by @rspack/core experiments.rsc.
Keep the supported client, server, and static runtimes; restore the browser server loadServerAction export; narrow host entrypoints to Rspack-owned modules; and validate the assembled release through a real consumer, export-condition resolution, environment guards, and npm pack inventory.
@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz CodeflowRun & review this pull request in StackBlitz Codeflow.

Sign up for freeto 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

@BleedingDev