Skip to content

docs(website): add OriginJS Vite migration guide - #4925

Open
dmchoi77 wants to merge 1 commit into
module-federation:mainfrom
dmchoi77:docs/originjs-vite-migration-guide
Open

docs(website): add OriginJS Vite migration guide#4925
dmchoi77 wants to merge 1 commit into
module-federation:mainfrom
dmchoi77:docs/originjs-vite-migration-guide

Conversation

@dmchoi77

Copy link
Copy Markdown
Contributor

Description

Adds an OriginJS to @module-federation/vite migration guide to the Vite integration documentation.

The guide covers supported environments, complete OriginJS and mf-vite remote/host configuration examples, dynamic remote migration, shared dependency and CSS considerations, and incremental deployment guidance.

Reference: module-federation/vite#956

Validation

  • Ran pnpm exec prettier --check apps/website-new/docs/en/integrations/build-tool/vite.mdx
  • Ran git diff --check

Related Issue

N/A — documentation-only migration guide.

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have updated the documentation.

@changeset-bot

changeset-bot Bot commented Jul 22, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 376039c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 376039cf8f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +267 to +268
const module = await loadRemote('catalog/Product');
const Product = module?.default ?? module;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Add a result type before reading default

For TypeScript hosts copying this example, loadRemote declares an uninferred generic result (packages/runtime/src/index.ts:56-58), so this call produces an unknown-like value and module?.default fails strict compilation with TS2339: Property 'default' does not exist on type '{}'. Supply a generic module shape (or otherwise narrow the result) before accessing its default export.

Useful? React with 👍 / 👎.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant