[Regression][Windows]Fix Exception thrown on .NET 10 Windows when calling Permissions.CheckStatusAsync<Permissions.Microphone>() by devanathan-vaithiyanathan · Pull Request #33179 · dotnet/maui

devanathan-vaithiyanathan · 2025-12-16T10:38:17Z

Issue Details

On .NET 10 Windows, calling
Permissions.CheckStatusAsync<Permissions.Microphone>() throws an exception when the app is running as an unpackaged app.

This is a regression introduced after Windows apps were changed to run as unpackaged by default. In unpackaged apps, AppxManifest.xml is not used, so microphone capabilities declared in the manifest are ignored. However, the current implementation always validates microphone capability declarations against AppxManifest.xml, which causes an exception for unpackaged apps.

Description of Change

Updated the Windows microphone permission logic to skip manifest capability checks for unpackaged apps. The microphone declaration is now required only for packaged apps, preventing exceptions and aligning the behavior with other Essentials APIs.

Issues Fixed

Fixes #32989

Tested the behavior in the following platforms.

Android
Windows
iOS
Mac

Before	After
Windows	Windows

Copilot

Pull request overview

This PR fixes a Windows-specific exception that occurs when calling Permissions.CheckStatusAsync<Permissions.Microphone>() on .NET 10 unpackaged apps. The issue was caused by the permission logic always validating against AppxManifest.xml, which doesn't exist in unpackaged apps (now the default for Windows). The fix adds conditional checks using AppInfoUtils.IsPackagedApp to skip manifest validation for unpackaged apps.

Key Changes:

Modified microphone permission logic to skip EnsureDeclared() for unpackaged apps
Refactored RequestAsync method to extract permission request logic into TryRequestPermissionAsync
Added UI test for Windows to verify the fix
Added new "Essentials" category to UI test infrastructure

Reviewed changes

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

Show a summary per file

File	Description
src/Essentials/src/Permissions/Permissions.windows.cs	Adds conditional `IsPackagedApp` checks before calling `EnsureDeclared()` in both `CheckStatusAsync` and `RequestAsync` methods, and extracts try-catch logic into a new `TryRequestPermissionAsync` method
src/Controls/tests/TestCases.Shared.Tests/UITestCategories.cs	Adds new "Essentials" category constant for UI test categorization
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue32989.cs	NUnit test implementation that taps a button and verifies no crash occurs
src/Controls/tests/TestCases.HostApp/Issues/Issue32989.cs	Test page with button to trigger microphone permission check
eng/pipelines/common/ui-tests.yml	Adds "Essentials" to the test category groups in the pipeline configuration

jfversluis

Can we do this with a device tests rather than with a UI test? I think all of the Essentials are device tests?

devanathan-vaithiyanathan · 2025-12-22T06:22:37Z

Can we do this with a device tests rather than with a UI test? I think all of the Essentials are device tests?

@jfversluis , This issue reproduces only in unpackaged apps, not in packaged apps. Since only the HostApp is an unpackaged app and the DeviceTest is packaged, I wrote a UI test.

## Description Remove the "Are you waiting for the changes in this PR to be merged?" note from all Copilot instruction files. This note was previously required at the top of every PR description so users could find instructions on how to test PR artifacts. We now have a **dogfooding comment bot** that automatically posts testing instructions under each PR, making this manual note redundant. Copilot agents were still prepending this note to every PR description because it was hardcoded in: - `.github/copilot-instructions.md` (main instructions) - `.github/skills/pr-finalize/SKILL.md` (PR finalization skill) - `.github/skills/pr-finalize/references/complete-example.md` (example PR) ### Issues Fixed N/A — instruction cleanup only. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

## Problem The `maui-pr-uitests` pipeline has 98+ test jobs that all publish artifacts using `PublishBuildArtifacts@1` with **no artifact name**, defaulting to `drop`. When multiple jobs upload the same file to the same container simultaneously, AzDO blob storage encounters write conflicts: ``` Blob is incomplete (missing block). Blob: 29adda685a1ff1119a49000d3a9183a5, Expected Offset: 0, Actual Offset: 8388608 ##[error]File upload failed even after retry. ``` ### Recent failures - [Build 1334980](https://dev.azure.com/dnceng-public/public/_build/results?buildId=1334980): 3 jobs failed (Controls CollectionView, Controls (vlatest), Controls (vlatest) CollectionView) - [Build 1334245](https://dev.azure.com/dnceng-public/public/_build/results?buildId=1334245): 1 job failed (Controls (vlatest)) The specific collision: both "Controls (vlatest)" and "Controls (vlatest) CollectionView" upload `drop/logs/appium_ios_Controls.TestCases.iOS.Tests-Release-ios-CollectionView.log` — same blob ID, concurrent writes. ## Fix Add a unique artifact name per job using `$(System.StageName)-$(System.JobName)-$(System.JobAttempt)` in `eng/pipelines/common/ui-tests-steps.yml`. This matches the pattern already used by snapshot diff artifacts in `ui-tests-collect-snapshot-diffs.yml`. Fixes dotnet#34477 Ref: dotnet/dnceng#1916 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

MauiBot · 2026-03-21T00:37:45Z

⚠️ Merge Conflict Detected — This PR has merge conflicts with its target branch. Please rebase onto the target branch and resolve the conflicts.

github-actions · 2026-03-21T13:41:08Z

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 33179

Or

Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 33179"

kubaflo · 2026-03-21T13:41:33Z

/azp run maui-pr-devicetests

azure-pipelines · 2026-03-21T13:41:45Z

Azure Pipelines successfully started running 1 pipeline(s).

…ling Permissions.CheckStatusAsync<Permissions.Microphone>() (#33179)  ### Issue Details On .NET 10 Windows, calling Permissions.CheckStatusAsync<Permissions.Microphone>() throws an exception when the app is running as an unpackaged app. This is a regression introduced after Windows apps were changed to run as unpackaged by default. In unpackaged apps, AppxManifest.xml is not used, so microphone capabilities declared in the manifest are ignored. However, the current implementation always validates microphone capability declarations against AppxManifest.xml, which causes an exception for unpackaged apps. ### Description of Change  Updated the Windows microphone permission logic to skip manifest capability checks for unpackaged apps. The microphone declaration is now required only for packaged apps, preventing exceptions and aligning the behavior with other Essentials APIs. ### Issues Fixed  Fixes #32989  **Tested the behavior in the following platforms.** - [ ] Android - [x] Windows - [ ] iOS - [ ] Mac | Before | After | |---------|--------| | **Windows** <img src="https://github.com/user-attachments/assets/7870d1a7-eeab-4e85-be4d-b951f0bd3d19" width="600" height="300"> | **Windows** <img src="https://github.com/user-attachments/assets/bd43f5b8-85b7-4e3c-8404-d01c165fefa4" width="600" height="300"> | ---------

…ling Permissions.CheckStatusAsync<Permissions.Microphone>() (dotnet#33179)  ### Issue Details On .NET 10 Windows, calling Permissions.CheckStatusAsync<Permissions.Microphone>() throws an exception when the app is running as an unpackaged app. This is a regression introduced after Windows apps were changed to run as unpackaged by default. In unpackaged apps, AppxManifest.xml is not used, so microphone capabilities declared in the manifest are ignored. However, the current implementation always validates microphone capability declarations against AppxManifest.xml, which causes an exception for unpackaged apps. ### Description of Change  Updated the Windows microphone permission logic to skip manifest capability checks for unpackaged apps. The microphone declaration is now required only for packaged apps, preventing exceptions and aligning the behavior with other Essentials APIs. ### Issues Fixed  Fixes dotnet#32989  **Tested the behavior in the following platforms.** - [ ] Android - [x] Windows - [ ] iOS - [ ] Mac | Before | After | |---------|--------| | **Windows** <img src="https://github.com/user-attachments/assets/7870d1a7-eeab-4e85-be4d-b951f0bd3d19" width="600" height="300"> | **Windows** <img src="https://github.com/user-attachments/assets/bd43f5b8-85b7-4e3c-8404-d01c165fefa4" width="600" height="300"> | ---------

…ling Permissions.CheckStatusAsync<Permissions.Microphone>() (#33179)  ### Issue Details On .NET 10 Windows, calling Permissions.CheckStatusAsync<Permissions.Microphone>() throws an exception when the app is running as an unpackaged app. This is a regression introduced after Windows apps were changed to run as unpackaged by default. In unpackaged apps, AppxManifest.xml is not used, so microphone capabilities declared in the manifest are ignored. However, the current implementation always validates microphone capability declarations against AppxManifest.xml, which causes an exception for unpackaged apps. ### Description of Change  Updated the Windows microphone permission logic to skip manifest capability checks for unpackaged apps. The microphone declaration is now required only for packaged apps, preventing exceptions and aligning the behavior with other Essentials APIs. ### Issues Fixed  Fixes #32989  **Tested the behavior in the following platforms.** - [ ] Android - [x] Windows - [ ] iOS - [ ] Mac | Before | After | |---------|--------| | **Windows** <img src="https://github.com/user-attachments/assets/7870d1a7-eeab-4e85-be4d-b951f0bd3d19" width="600" height="300"> | **Windows** <img src="https://github.com/user-attachments/assets/bd43f5b8-85b7-4e3c-8404-d01c165fefa4" width="600" height="300"> | ---------

devanathan-vaithiyanathan added 7 commits December 11, 2025 19:03

fix added

d4d5dd1

Update Permissions.windows.cs

b8a401d

alignment added

dbcecb8

test added

0b2fadd

Test updated

2d50f7f

Update Issue32989.cs

f01c27d

category changes added

fbdeb83

dotnet-policy-service bot added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Dec 16, 2025

build-analysis bot mentioned this pull request Dec 16, 2025

"We stopped hearing from agent Azure Pipelines 32. Verify the agent machine is running and has a healthy network connection" dotnet/dnceng#1886

Open

3 tasks

sheiksyedm added platform/windows area-essentials Essentials: Device, Display, Connectivity, Secure Storage, Sensors, App Info labels Dec 16, 2025

sheiksyedm marked this pull request as ready for review December 16, 2025 14:06

Copilot AI review requested due to automatic review settings December 16, 2025 14:06

Copilot started reviewing on behalf of sheiksyedm December 16, 2025 14:07 View session

Copilot AI reviewed Dec 16, 2025

View reviewed changes

jfversluis requested changes Dec 19, 2025

View reviewed changes

jfversluis added this to the .NET 10.0 SR3 milestone Dec 19, 2025

PureWeen added this to MAUI SDK Ongoing Dec 30, 2025

github-project-automation bot moved this from Todo to Changes Requested in MAUI SDK Ongoing Dec 30, 2025

github-project-automation bot moved this to Todo in MAUI SDK Ongoing Dec 30, 2025

karthikraja-arumugam added the community ✨ Community Contribution label Jan 9, 2026

PureWeen modified the milestones: .NET 10.0 SR3, .NET 10.0 SR4 Jan 21, 2026

sheiksyedm modified the milestones: .NET 10.0 SR4, .NET 10 SR5 Feb 12, 2026

PureWeen modified the milestones: .NET 10 SR5, .NET 10 SR6 Mar 3, 2026

Merge branch 'main' into fix-32989

7f0c9e5

kubaflo approved these changes Mar 21, 2026

View reviewed changes

kubaflo changed the base branch from main to inflight/current March 21, 2026 18:12

kubaflo merged commit ef3ba62 into dotnet:inflight/current Mar 21, 2026
37 of 40 checks passed

github-project-automation bot moved this from Changes Requested to Done in MAUI SDK Ongoing Mar 21, 2026

This was referenced Mar 21, 2026

The Operation will be canceled. The next steps may not contain expected logs. dotnet/dnceng#3008

Open

[android-arm64] The Operation will be canceled. The next steps may not contain expected logs. dotnet/dnceng#6408

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Regression][Windows]Fix Exception thrown on .NET 10 Windows when calling Permissions.CheckStatusAsync<Permissions.Microphone>()#33179

[Regression][Windows]Fix Exception thrown on .NET 10 Windows when calling Permissions.CheckStatusAsync<Permissions.Microphone>()#33179
kubaflo merged 10 commits intodotnet:inflight/currentfrom
devanathan-vaithiyanathan:fix-32989

devanathan-vaithiyanathan commented Dec 16, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

jfversluis left a comment

Uh oh!

devanathan-vaithiyanathan commented Dec 22, 2025

Uh oh!

MauiBot commented Mar 21, 2026

Uh oh!

github-actions bot commented Mar 21, 2026

Uh oh!

kubaflo commented Mar 21, 2026

Uh oh!

azure-pipelines bot commented Mar 21, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

Conversation

devanathan-vaithiyanathan commented Dec 16, 2025

Issue Details

Description of Change

Issues Fixed

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

jfversluis left a comment

Choose a reason for hiding this comment

Uh oh!

devanathan-vaithiyanathan commented Dec 22, 2025

Uh oh!

MauiBot commented Mar 21, 2026

Uh oh!

github-actions bot commented Mar 21, 2026

Uh oh!

kubaflo commented Mar 21, 2026

Uh oh!

azure-pipelines bot commented Mar 21, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants