Skip to content

Fix source map URLs for .NET 10 VMR builds #121923

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
akoeplinger merged 2 commits into from
Nov 24, 2025
Merged

Fix source map URLs for .NET 10 VMR builds #121923

akoeplinger merged 2 commits into from
Nov 24, 2025

Conversation

Copy link
Contributor

Copilot AI commented Nov 24, 2025 •
edited
Loading

.NET 10 builds from the VMR (dotnet/dotnet) instead of dotnet/runtime. Source maps generated during CI builds were pointing to the old repository, causing 404 errors when browser debuggers attempted to load TypeScript sources.

Changes

Updated sourcemapPathTransform in rollup configurations to generate VMR-relative URLs:

  • src/mono/browser/runtime/rollup.config.js
  • src/native/rollup.config.plugins.js
// Before
res = `https://raw.githubusercontent.com/dotnet/runtime/${gitHash}/${relativeSourcePath}`;

// After  
res = `https://raw.githubusercontent.com/dotnet/dotnet/${gitHash}/src/runtime/${relativeSourcePath}`;

Fixes #121672

Original prompt

This section details on the original issue you should resolve

<issue_title>Could not load content for githubusercontent</issue_title>
<issue_description>### Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

Since upgrading to Visual Studio 2026 and .NET 10 I have started to receive a number of "Could not load content..." errors in the browser.

For example:

Could not load content for https://raw.githubusercontent.com/dotnet/runtime/b0f34d51fccc69fd334253924abd8d6853fad7aa/src/mono/browser/runtime/debug.ts (HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE)

The Sources tab in the browser shows:

Image

I have tried cleareing the cache and done a hard refresh, updating the .NET workloads, and even installed the .NET 10.0 WebAssembly Build Tools but I still receive the errors.

Is it possible I have missed a step in the upgrade?

Expected Behavior

No response

Steps To Reproduce

No response

Exceptions (if any)

No response

.NET Version

No response

Anything else?

No response</issue_description>

Comments on the Issue (you are @copilot in this section)

@akoeplinger This is because we switched to building .NET inside of the VMR repo (https://github.com/dotnet/dotnet) in .NET 10 and so this URL for the source map needs to be updated: https://github.com/dotnet/runtime/blob/928eb5fd02d066bfeada8d58547cac48baebe78a/src/mono/browser/runtime/rollup.config.js#L134 @pavelsavara And this one too

runtime/src/native/rollup.config.plugins.js

Line 148 in 928eb5f

res = `https://raw.githubusercontent.com/dotnet/runtime/${gitHash}/${relativeSourcePath}`;
</comment_new>

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Nov 24, 2025
@akoeplinger
Update source map URLs to point to VMR (dotnet/dotnet) repository
33065f2 
Co-authored-by: akoeplinger <1376924+akoeplinger@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix loading issue for githubusercontent in browser Fix source map URLs for .NET 10 VMR builds Nov 24, 2025
Copilot AI requested a review from akoeplinger November 24, 2025 15:21
@akoeplinger akoeplinger marked this pull request as ready for review November 24, 2025 15:21
Copilot AI review requested due to automatic review settings November 24, 2025 15:22
@akoeplinger akoeplinger added arch-wasm WebAssembly architecture area-Build-mono and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Nov 24, 2025
@dotnet-policy-service
Copy link
Contributor

dotnet-policy-service bot commented Nov 24, 2025

Tagging subscribers to 'arch-wasm': @lewing, @pavelsavara
See info in area-owners.md if you want to be subscribed.

Copilot AI reviewed Nov 24, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes source map URLs for .NET 10 builds to work with the new VMR (Virtual Monolithic Repository) structure. Since .NET 10 now builds from dotnet/dotnet instead of dotnet/runtime, the source map URLs need to be updated to point to the correct repository and include the src/runtime/ path prefix.

  • Updated source map URL generation in two rollup configuration files
  • Changed repository reference from dotnet/runtime to dotnet/dotnet with src/runtime/ path prefix

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/native/rollup.config.plugins.js Updated sourcemapPathTransform function to generate VMR-relative GitHub URLs with /src/runtime/ path prefix for CI builds
src/mono/browser/runtime/rollup.config.js Updated sourcemapPathTransform function to generate VMR-relative GitHub URLs with /src/runtime/ path prefix for CI builds

@akoeplinger
Copy link
Member

akoeplinger commented Nov 24, 2025

/backport to release/10.0

@github-actions

This comment was marked as outdated.

Sign in to view
@github-actions

This comment was marked as outdated.

Sign in to view
@akoeplinger akoeplinger mentioned this pull request Nov 24, 2025
Merged
4 tasks
This was referenced Nov 24, 2025
Open
Open
Open
Open
@build-analysis build-analysis bot mentioned this pull request Nov 24, 2025
Open
@akoeplinger akoeplinger enabled auto-merge (squash) November 24, 2025 21:32
@akoeplinger akoeplinger merged commit daced03 into main Nov 24, 2025
169 of 171 checks passed
@akoeplinger akoeplinger deleted the copilot/fix-githubusercontent-loading-issue branch November 24, 2025 21:38
@akoeplinger akoeplinger mentioned this pull request Nov 24, 2025
Merged
@build-analysis build-analysis bot mentioned this pull request Nov 24, 2025
Open
@dotnet-maestro dotnet-maestro bot mentioned this pull request Nov 25, 2025
Merged
lewing pushed a commit that referenced this pull request Nov 26, 2025
@akoeplinger
[release/10.0] Fix source map URLs for .NET 10 VMR builds (#121926)
baf4eb1 
Backport of #121923

## Customer Impact

- [x] Customer reported: #121804
- [ ] Found internally

.NET 10 builds from the VMR (dotnet/dotnet) instead of dotnet/runtime.
Source maps generated during CI builds were pointing to the old
repository, causing 404 errors when browser debuggers attempted to load
TypeScript sources.

## Regression

- [X] Yes
- [ ] No

This regressed when the official runtime build moved to happen in the
VMR.

## Testing

Local testing to make sure the correct URL is produced.

## Risk

Low. This only affects debugging scenarios.
@github-actions github-actions bot locked and limited conversation to collaborators Dec 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

Copilot code review Copilot Copilot left review comments

@pavelsavara pavelsavara pavelsavara approved these changes

@lewing lewing Awaiting requested review from lewing lewing is a code owner

+1 more reviewer

@akoeplinger akoeplinger akoeplinger approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@akoeplinger akoeplinger

Copilot code review Copilot

Labels

arch-wasm WebAssembly architecture area-Build-mono

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Could not load content for githubusercontent

3 participants

@akoeplinger @pavelsavara