Skip to content

fix(VirtualUrlPlugin): set resourceData.context to avoid invalid fall…#20390

Merged
alexander-akait merged 1 commit intomainfrom
virtual
Jan 30, 2026
Merged

fix(VirtualUrlPlugin): set resourceData.context to avoid invalid fall…#20390
alexander-akait merged 1 commit intomainfrom
virtual

Conversation

@xiaoxiaojx
Copy link
Member

Summary
Fixes #20388

What kind of change does this PR introduce?
Fix VirtualUrlPlugin to set resourceData.context for proper module resolution. Previously, when context was not set, it would fallback to the virtual scheme path (e.g., virtual:routes), which is not a valid filesystem path, causing subsequent resolve operations to fail.

Did you add tests for your changes?
Yes

Does this PR introduce a breaking change?
No

If relevant, what needs to be documented once your changes are merged or what have you already documented?
Nothing

@changeset-bot
Copy link

changeset-bot bot commented Jan 30, 2026

🦋 Changeset detected

Latest commit: 622a187

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
webpack Patch

Not sure what this means? Click here to learn what changesets are.

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

@github-actions
Copy link
Contributor

github-actions bot commented Jan 30, 2026

This PR is packaged and the instant preview is available (de107f8).

Install it locally:

  • npm
npm i -D webpack@https://pkg.pr.new/webpack@de107f8
  • yarn
yarn add -D webpack@https://pkg.pr.new/webpack@de107f8
  • pnpm
pnpm add -D webpack@https://pkg.pr.new/webpack@de107f8

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 30, 2026

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing virtual (622a187) with main (a656ab1)

Summary

✅ 72 untouched benchmarks

@alexander-akait alexander-akait merged commit de107f8 into main Jan 30, 2026
50 of 52 checks passed
@alexander-akait alexander-akait deleted the virtual branch January 30, 2026 18:23
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.

Virtual module with shared ModuleFederation breaks build

2 participants