[browser] Fix debug URL in lauchSettings.json in wasmbrowser template

[maraf]

Regression from wasmbrowser template migration to WebAssembly SDK
Tested manually be creating a project from updated template and launched from Visual studio using F5
Fixes #95391

[ghost]

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

Issue Details

---

Regression from wasmbrowser template migration to WebAssembly SDK
Tested manually be creating a project from updated template and launched from Visual studio using F5
Fixes #95391

Author:	maraf
Assignees:	maraf
Labels:	arch-wasm, area-Build-mono, os-browser
Milestone:	9.0.0

[maraf]

@radical @thaystg Is F5 suppose to work for console template? I'm getting

[thaystg]

If the implementation is not changed the URL was correct:
https://github.com/dotnet/runtime/blob/f66c1c15409ff505ba55f8fdf86bc5c041bf47a3/src/mono/wasm/host/WebServerStartup.cs#L132C1-L132C1

_framework is not mapped in the wasmbrowser template.

[thaystg]

After a long chat with @maraf , approving! :)
Thanks for the explanation.

[maraf]

/backport to release/8.0-staging

[github-actions]

Started backporting to release/8.0-staging: https://github.com/dotnet/runtime/actions/runs/7036833502

[radical]

After a long chat with @maraf , approving! :) Thanks for the explanation.

I would love to read the explanation too.

[radical]

@thaystg Does CTI test for this scenario?

[thaystg]

Probably no.
Not sure who should help us to ask them to test. Maybe @mkArtakMSFT ?

[thaystg]

After a long chat with @maraf , approving! :) Thanks for the explanation.

I would love to read the explanation too.

@maraf: New browser template based on WebAssembly SDK has a copy of blazor's dev server in wasmapphost.

So we should use this URL with _framework/debug.

We are not using this code anymore:
https://github.com/dotnet/runtime/blob/f66c1c15409ff505ba55f8fdf86bc5c041bf47a3/src/mono/wasm/host/WebServerStartup.cs#L132C1-L132C1

[radical]

ok, thanks for sharing that. We should add at least one test for debugging wasmbrowser the way an IDE might connect to it.

Maybe for blazor after that?

[thaystg]

Do you mean an automatic test?

[radical]

Do you mean an automatic test?

yeah, in DebuggerTestSuite. Just a simple one that validates that we can connect, and step or something.

[pavelsavara]

We are not using this code anymore: https://github.com/dotnet/runtime/blob/f66c1c15409ff505ba55f8fdf86bc5c041bf47a3/src/mono/wasm/host/WebServerStartup.cs#L132C1-L132C1

Should we delete the dead code ?

[maraf]

Should we delete the dead code ?

We can do it once we drop support for older template. I hope to unite browser & console template & runtime tests on Wasm SDK in 9 and remove the former bundler