feat: Compile `.wasm.cs` and `.browserwasm.cs` files for WebAssembly by MartinZikmund · Pull Request #22690 · unoplatform/uno

MartinZikmund · 2026-02-19T17:06:34Z

GitHub Issue: closes #22689

PR Type:

What is the current behavior? 🤔

What is the new behavior? 🚀

PR Checklist ✅

Please check if your PR fulfills the following requirements:

📝 Commits must be following the Conventional Commits specification.
🧪 Added Runtime tests, UI tests, or a manual test sample for the changes have been added (for bug fixes / features) (if applicable)
📚 Docs have been added/updated which fit documentation template (for bug fixes / features)
🖼️ Validated PR Screenshots Compare Test Run results.
❗ Contains NO breaking changes

Other information ℹ️

MartinZikmund · 2026-02-19T17:08:12Z

@copilot Document the newly added support for .browserwasm.cs as alternative to .wasm.cs

Copilot · 2026-02-19T17:08:22Z

@MartinZikmund I've opened a new pull request, #22691, to work on those changes. Once the pull request is ready, I'll request review from you.

…# guide Co-authored-by: MartinZikmund <1075116+MartinZikmund@users.noreply.github.com>

Copilot

Pull request overview

This PR updates Uno’s cross-targeting MSBuild item rules so platform-suffixed C# files are compiled appropriately when targeting the newer browserwasm platform (in addition to the existing WebAssembly runtime identifier flow).

Changes:

Adjust .wasm.cs inclusion/removal conditions to allow compilation when IsBrowserWasm=true.
Add new default item handling for .browserwasm.cs files so they compile only for BrowserWasm targets.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/Uno.Sdk/targets/Uno.CrossTargeting.targets	Updates cross-targeting item conditions for `.wasm.cs` and introduces `.browserwasm.cs` gating for BrowserWasm builds.
src/Uno.CrossTargetting.targets	Mirrors the same item condition updates for solution-wide cross-targeting behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-19T17:10:24Z

src/Uno.Sdk/targets/Uno.CrossTargeting.targets


-		<None Include="**\*.wasm.cs" Exclude="bin\**\*.wasm.cs;obj\**\*.wasm.cs" Condition="'$(UnoRuntimeIdentifier)'!='WebAssembly'" />
-		<Compile Remove="**\*.wasm.cs" Condition="'$(UnoRuntimeIdentifier)'!='WebAssembly'" />
+		<None Include="**\*.wasm.cs" Exclude="bin\**\*.wasm.cs;obj\**\*.wasm.cs" Condition="'$(UnoRuntimeIdentifier)'!='WebAssembly' AND '$(IsBrowserWasm)'!='true'" />


PR description currently says "closes #" but does not reference an actual GitHub issue. Please update the PR description to include a fully-qualified issue link in a Fixes/Closes/Related footer (e.g., "Closes https://github.com/unoplatform//issues/<ISSUE_NUMBER>") or add the approved "No related issue" note if applicable.

unodevops · 2026-02-19T17:26:04Z

🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-22690/wasm-skia-net9/index.html

MartinZikmund · 2026-02-19T23:20:16Z

@copilot adjust the condition to make both suffixes apply in both "isbrowserwasm" and "unoruntime=webassembly" cases

Copilot · 2026-02-19T23:20:25Z

@MartinZikmund I've opened a new pull request, #22698, to work on those changes. Once the pull request is ready, I'll request review from you.

…rWasm and UnoRuntime=WebAssembly Co-authored-by: MartinZikmund <1075116+MartinZikmund@users.noreply.github.com>

feat: Compile `.wasm.cs` and `.browserwasm.cs` files for WebAssembly

CLAassistant · 2026-02-19T23:41:20Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ MartinZikmund
❌ Copilot
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

Co-authored-by: MartinZikmund <1075116+MartinZikmund@users.noreply.github.com>

…wasm.cs Co-authored-by: MartinZikmund <1075116+MartinZikmund@users.noreply.github.com>

Clarified file naming conventions

feat: Compile `.wasm.cs` and `.browserwasm.cs` files for WebAssembly

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-19T23:53:56Z

doc/articles/platform-specific-csharp.md

 Starting from Uno Platform 5.2, in project or class libraries using the `Uno.Sdk`, a set of implicit file name conventions can be used to target specific platforms:

 * `*.wasm.cs` is built only for `net10.0-browserwasm`
+* `*.browserwasm.cs` is built only for `net10.0-browserwasm`


The list now includes both *.wasm.cs and *.browserwasm.cs with the same description ("built only for net9.0-browserwasm") which is ambiguous for readers. Please clarify the intended difference between these suffixes (e.g., native WebAssembly vs Skia browserwasm), and adjust the "Starting from Uno Platform 5.2" wording if this new suffix is introduced in a later version. Also consider updating the Uno.Sdk cross-targeting docs elsewhere (e.g., doc/articles/features/using-the-uno-sdk.md) to include *.browserwasm.cs for consistency.

Suggested change

Starting from Uno Platform 5.2, in project or class libraries using the `Uno.Sdk`, a set of implicit file name conventions can be used to target specific platforms:

* `*.wasm.cs` is built only for `net10.0-browserwasm`

* `*.browserwasm.cs` is built only for `net10.0-browserwasm`

In projects or class libraries using the `Uno.Sdk`, a set of implicit file name conventions can be used to target specific platforms:

* `*.browserwasm.cs` is built only for `net10.0-browserwasm` (recommended suffix for Browser WebAssembly)

* `*.wasm.cs` is also built only for `net10.0-browserwasm` and is kept for backward compatibility with older Uno projects

unodevops · 2026-02-20T00:12:41Z

🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-22690/wasm-skia-net9/index.html

unodevops · 2026-02-20T00:15:45Z

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-22690/docs/index.html

unodevops · 2026-02-20T02:15:29Z

⚠️⚠️ The build 197279 has failed on Uno.UI - CI.

feat: Compile .wasm.cs and .browserwasm.cs files for WebAssembly

ccf4980

Copilot AI review requested due to automatic review settings February 19, 2026 17:06

github-actions bot assigned MartinZikmund Feb 19, 2026

Copilot started reviewing on behalf of MartinZikmund February 19, 2026 17:07 View session

MartinZikmund requested a review from kazo0 February 19, 2026 17:07

Initial plan

6761aa7

Copilot AI mentioned this pull request Feb 19, 2026

feat: Compile .wasm.cs and .browserwasm.cs files for WebAssembly #22691

Merged

5 tasks

docs: document .browserwasm.cs file convention in platform-specific C…

2f29f80

…# guide Co-authored-by: MartinZikmund <1075116+MartinZikmund@users.noreply.github.com>

Copilot AI reviewed Feb 19, 2026

View reviewed changes

kazo0 approved these changes Feb 19, 2026

View reviewed changes

ajpinedam approved these changes Feb 19, 2026

View reviewed changes

Initial plan

2b0e63d

Copilot AI mentioned this pull request Feb 19, 2026

feat: Compile .wasm.cs and .browserwasm.cs files for WebAssembly #22698

Merged

5 tasks

Copilot AI and others added 2 commits February 19, 2026 23:21

fix: make both .wasm.cs and .browserwasm.cs compile for both IsBrowse…

7a3f022

…rWasm and UnoRuntime=WebAssembly Co-authored-by: MartinZikmund <1075116+MartinZikmund@users.noreply.github.com>

Merge pull request #22698 from unoplatform/copilot/sub-pr-22690-again

db0c0ab

feat: Compile `.wasm.cs` and `.browserwasm.cs` files for WebAssembly

Copilot AI and others added 4 commits February 19, 2026 23:43

docs: update .browserwasm.cs to reflect equivalence with .wasm.cs

9f39e70

Co-authored-by: MartinZikmund <1075116+MartinZikmund@users.noreply.github.com>

docs: use WebAssembly native/Skia terminology for wasm.cs and browser…

756ec10

…wasm.cs Co-authored-by: MartinZikmund <1075116+MartinZikmund@users.noreply.github.com>

chore: Update platform-specific file naming conventions

7f4c404

Clarified file naming conventions

Merge pull request #22691 from unoplatform/copilot/sub-pr-22690

e2cee77

feat: Compile `.wasm.cs` and `.browserwasm.cs` files for WebAssembly

Copilot AI review requested due to automatic review settings February 19, 2026 23:49

github-actions bot added the kind/documentation label Feb 19, 2026

Copilot started reviewing on behalf of MartinZikmund February 19, 2026 23:49 View session

Merge branch 'master' into dev/mazi/wasmcs

3368621

MartinZikmund enabled auto-merge February 19, 2026 23:52

Copilot AI reviewed Feb 19, 2026

View reviewed changes

Conversation

MartinZikmund commented Feb 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Type:

What is the current behavior? 🤔

What is the new behavior? 🚀

PR Checklist ✅

Other information ℹ️

Uh oh!

MartinZikmund commented Feb 19, 2026

Uh oh!

Copilot AI commented Feb 19, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

Uh oh!

unodevops commented Feb 19, 2026

Uh oh!

MartinZikmund commented Feb 19, 2026

Uh oh!

Copilot AI commented Feb 19, 2026

Uh oh!

CLAassistant commented Feb 19, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

Uh oh!

unodevops commented Feb 20, 2026

Uh oh!

unodevops commented Feb 20, 2026

Uh oh!

unodevops commented Feb 20, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Comments

MartinZikmund commented Feb 19, 2026 •

edited

Loading