Adding changes to support use monovsdbg to debug wasm apps. by thaystg · Pull Request #7220 · dotnet/vscode-csharp

thaystg · 2024-06-10T16:04:02Z

Adding changes to support debug wasm apps using monovsdbg.

Tested scenarios with the new setting enabled and this PR:

With Assets and C# DevKit Not Installed targetting .NET 8 (Blazor Wasm) - DISABLED
With Assets and C# DevKit Not Installed targetting .NET 8 (Blazor Server Wasm) - DISABLED
With Assets and C# DevKit Not Installed targetting .NET 9 (Blazor Wasm) - ENABLED
With Assets and C# DevKit Not Installed targetting .NET 9 (Blazor Server Wasm) - ENABLED
With Assets and C# DevKit Installed targetting .NET 8 (Blazor Wasm) - DISABLED
With Assets and C# DevKit Installed targetting .NET 8 (Blazor Server Wasm) - DISABLED
With Assets and C# DevKit Installed targetting .NET 9 (Blazor Wasm) - ENABLED
With Assets and C# DevKit Installed targetting .NET 9 (Blazor Server Wasm) - ENABLED
Without Assets and C# DevKit Installed targetting .NET 8 (Blazor Wasm) - DISABLED
Without Assets and C# DevKit Installed targetting .NET 8 (Blazor Server Wasm) - DISABLED
Without Assets and C# DevKit Installed targetting .NET 9 (Blazor Wasm) - ENABLED
Without Assets and C# DevKit Installed targetting .NET 9 (Blazor Server Wasm) - ENABLED

Tested scenarios only updating C# DevKit with the monovsdbg support:

Without Assets and C# DevKit Installed with C# extension without this PR targetting .NET 8 (Blazor Wasm) - DISABLED
Without Assets and C# DevKit Installed with C# extension without this PR targetting .NET 8 (Blazor Server Wasm) - DISABLED
Without Assets and C# DevKit Installed with C# extension without this PR targetting .NET 9 (Blazor Wasm) - DISABLED
Without Assets and C# DevKit Installed with C# extension without this PR targetting .NET 9 (Blazor Server Wasm) - DISABLED

Tested scenarios with this PR and C# DevKit without the monovsdbg support:

With Assets and C# DevKit Installed targetting .NET 8 (Blazor Wasm) - DISABLED
With Assets and C# DevKit Installed targetting .NET 8 (Blazor Server Wasm) - DISABLED
With Assets and C# DevKit Installed targetting .NET 9 (Blazor Wasm) - ENABLED
With Assets and C# DevKit Installed targetting .NET 9 (Blazor Server Wasm) - ENABLED
Without Assets and C# DevKit Installed targetting .NET 8 (Blazor Wasm) - DISABLED
Without Assets and C# DevKit Installed targetting .NET 8 (Blazor Server Wasm) - DISABLED
Without Assets and C# DevKit Installed targetting .NET 9 (Blazor Wasm) - DISABLED
Without Assets and C# DevKit Installed targetting .NET 9 (Blazor Server Wasm) - DISABLED

package.json

package.nls.json

src/razor/src/blazorDebug/blazorDebugConfigurationProvider.ts

src/shared/utils.ts

src/shared/assets.ts

…_ASSEMBLIES to enable hotreload.

…/vscode-csharp into dev/thays/support_wasm_monovsdbg

thaystg · 2024-06-28T05:44:04Z

@gregg-miskelly can you please review again?
I think I addressed all your comments.

src/razor/src/blazorDebug/blazorDebugConfigurationProvider.ts

src/shared/assets.ts

src/shared/utils.ts

thaystg · 2025-08-08T13:27:55Z

Can I get a review again here?

gregg-miskelly · 2025-08-08T16:53:50Z

package.json

      ]
+    },
+    {
+      "id": "VSWebAssemblyBridge",


id

Should this be added here so you can use the update dependencies script?

Did you push this fix? I don't see it

I forgot to push, it's there now.

gregg-miskelly · 2025-08-08T17:46:04Z

src/shared/assets.ts

+            }
+        });
+        assetsPath = assetsPath.slice(0, -1);
+        return [assetsPath, this.executableProjects[0].outputPath];


this.executableProjects[0].outputPath

I am not quite following what this is doing -- we are returning the assetsPath of all the web assembly projects, but then the output path of whatever random project happens to be the first executable project in the solution.

The program name is basically not used, is a requirement for concord, but we don't use it for wasm, that is why I'm getting the first one.
The assets path is used to load the metadata, this one is important.

Should we just use a sentinal program name? Ex: "Blazor WASM"?

We can't it's required that the program exists in the disk.

src/shared/assets.ts

src/shared/utils.ts

…/vscode-csharp into dev/thays/support_wasm_monovsdbg

macel94 · 2026-01-22T10:59:00Z

can we get some news here?
i am one of the 2 that actually use blazor wasm in wsl2 and would like to be able to debug :D

gregg-miskelly · 2026-04-02T21:33:17Z

src/razor/src/blazorDebug/blazorDebugConfigurationProvider.ts

+        try {
+            await this.vscodeType.debug.startDebugging(folder, app);
+            const terminate = this.vscodeType.debug.onDidTerminateDebugSession(async (event) => {
+                if (process.platform !== 'win32') {


if (

Do you need a check that the terminating session is the one you are interested in?

src/omnisharp/protocol.ts

gregg-miskelly · 2026-04-02T21:40:01Z

src/razor/src/blazorDebug/blazorDebugConfigurationProvider.ts

+    }
+
+    private static async launchVsWebAssemblyBridge(urlStr: string): Promise<[string, number, number]> {
+        const dotnetPath = process.platform === 'win32' ? 'dotnet.exe' : 'dotnet';


const dotnetPath = process.platform === 'win32' ? 'dotnet.exe' : 'dotnet';

This will use the version of dotnet which is globally installed. Is that the one you want? Or do you want the version that comes with the C# extension?

I fixed it, now it's using the one that comes with the C# extension.

gregg-miskelly · 2026-04-02T21:41:47Z

src/razor/src/blazorDebug/blazorDebugConfigurationProvider.ts

+        let proxyICorDebugPort = -1;
+        let proxyBrowserPort = -1;
+        let newUri = '';
+        const spawnedProxy = cp.spawn(dotnetPath, spawnedProxyArgs, cpOptions);


spawnedProxy

Do you need to handle the process exited before it says it is ready?

gregg-miskelly

Otherwise LGTM

Adding changes to support use monovsdbg to debug wasm apps.

1f4d94e

gregg-miskelly reviewed Jun 10, 2024

View reviewed changes

package.json Outdated Show resolved Hide resolved

gregg-miskelly reviewed Jun 10, 2024

View reviewed changes

package.nls.json Outdated Show resolved Hide resolved

gregg-miskelly reviewed Jun 10, 2024

View reviewed changes

src/razor/src/blazorDebug/blazorDebugConfigurationProvider.ts Outdated Show resolved Hide resolved

gregg-miskelly reviewed Jun 10, 2024

View reviewed changes

src/shared/utils.ts Outdated Show resolved Hide resolved

gregg-miskelly reviewed Jun 10, 2024

View reviewed changes

src/shared/assets.ts Show resolved Hide resolved

thaystg added 11 commits June 13, 2024 12:11

Adding VSWebAssemblyPackage.

d7f8606

Addressing Gregg's comments

cfb2d67

Fixing prettier

d59699b

Fixing package.json

981c74b

Merge branch 'main' into dev/thays/support_wasm_monovsdbg

0bc4ca2

Adding vswebassembly to gitignore

c4d88dc

Test if the vswebassembly path exists and also pass DOTNET_MODIFIABLE…

d3b8441

…_ASSEMBLIES to enable hotreload.

Fix prettier

b094acf

Bump VSWebAssemblyBridge.

4796772

Check targetFramework before enable it.

83de7bb

Merge branch 'dev/thays/support_wasm_monovsdbg' of github.com:thaystg…

e2802c1

…/vscode-csharp into dev/thays/support_wasm_monovsdbg

thaystg marked this pull request as ready for review June 22, 2024 00:38

thaystg requested review from a team as code owners June 22, 2024 00:38

thaystg requested a review from gregg-miskelly June 22, 2024 00:52

gregg-miskelly requested changes Jun 28, 2024

View reviewed changes

thaystg added 3 commits July 1, 2024 20:03

Addressing Gregg's comments.

d6fd0b6

Renaming function.

04b4c77

Addressing Gregg's comments.

a5452a8

thaystg requested a review from gregg-miskelly July 2, 2024 15:28

thaystg added 3 commits July 2, 2024 13:48

Merge branch 'main' into dev/thays/support_wasm_monovsdbg

2b1ee0d

Addressing Gregg's comments.

e06f914

Merge branch 'main' into dev/thays/support_wasm_monovsdbg

4e70917

thaystg added 6 commits July 29, 2025 14:07

Merge branch 'main' into dev/thays/support_wasm_monovsdbg

7e49904

Fixing merge

4163249

fix compilation

4f86d19

Merge branch 'main' into dev/thays/support_wasm_monovsdbg

bcb7bdd

Last adjust to support wasm debugger with icordebug on vscode-csharp

d9176bd

fix eslint errors and adding .gitignore.

8fd232c

thaystg mentioned this pull request Aug 5, 2025

Debugging Blazor Webassembly running in a dev container dotnet/runtime#112860

Closed

Bump vswebassemblybridge

f4c236e

thaystg marked this pull request as ready for review August 8, 2025 13:27

gregg-miskelly reviewed Aug 8, 2025

View reviewed changes

src/shared/assets.ts Show resolved Hide resolved

gregg-miskelly reviewed Aug 8, 2025

View reviewed changes

src/shared/utils.ts Show resolved Hide resolved

thaystg added 3 commits October 14, 2025 11:13

Merge branch 'main' into dev/thays/support_wasm_monovsdbg

e188135

Merge branch 'dev/thays/support_wasm_monovsdbg' of github.com:thaystg…

0655291

…/vscode-csharp into dev/thays/support_wasm_monovsdbg

Fix merge

b7125a3

thaystg added 4 commits April 1, 2026 16:29

Merge branch 'main' into dev/thays/support_wasm_monovsdbg

b7a5367

Fix compilation failures

8cbc93c

Addressing Gregg's comments and bumping vswebassemblybridge.

4c47709

Addressing copilot review changes.

55dfbe8

gregg-miskelly reviewed Apr 2, 2026

View reviewed changes

src/omnisharp/protocol.ts Show resolved Hide resolved

gregg-miskelly reviewed Apr 2, 2026

View reviewed changes

gregg-miskelly requested changes Apr 2, 2026

View reviewed changes

thaystg added 2 commits April 6, 2026 13:38

Addressing Gregg's comments.

becc63a

Terminating all the debugging sessions when one of them is terminated.

4e8b239

thaystg requested a review from gregg-miskelly April 6, 2026 20:06

Conversation

thaystg commented Jun 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

thaystg commented Jun 28, 2024

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

thaystg commented Aug 8, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

macel94 commented Jan 22, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

gregg-miskelly left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

thaystg commented Jun 10, 2024 •

edited

Loading