Drop MSIX scaffold cruft so VS stops demanding deployment#3
Closed
StuartMeeks wants to merge 1 commit into
Closed
Conversation
VS was failing debug with "The project needs to be deployed before we can debug. Please enable Deploy in the Configuration Manager." even though Deploy was checked. Root cause: the csproj still carried EnableMsixTooling=true and the matching Msix ProjectCapability / HasPackageAndPublishMenu blocks from the template, which made VS classify the project as needing MSIX deployment. WindowsPackageType=None says we ship unpackaged, so the deploy step finds nothing to deploy and the F5 path errors out. CLAUDE.md is explicit: unpackaged distribution, no MSIX assumptions. Removing the MSIX bits and the dead assets that go with them: - EnableMsixTooling, Msix ProjectCapability, HasPackageAndPublishMenu - Package.appxmanifest (only meaningful for MSIX-packaged builds) - Assets/*.png (Start tile / store / lock screen images for MSIX manifests) - <Deploy /> on Snipdeck.App in the slnx — F5 no longer needs a deploy step - Content Include entries pointing at the now-deleted assets The Win32 app.manifest stays (DPI awareness etc.) and is unrelated. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
F5 in Visual Studio failed with:
Deploy was already checked. The advice is for MSIX-packaged projects; unpackaged WinUI 3 apps shouldn't have Deploy checked at all, AND shouldn't have the MSIX tooling enabled in the csproj.
Root cause
The csproj still carried
<EnableMsixTooling>true</EnableMsixTooling>+ the conditional MsixProjectCapability/HasPackageAndPublishMenublocks from the template, which made VS classify the project as needing MSIX deployment.WindowsPackageType=Nonesays we ship unpackaged, so the deploy step finds nothing to deploy and the F5 path errors out.CLAUDE.md is explicit:
Changes
EnableMsixTooling, the conditionalMsixProjectCapability, and theHasPackageAndPublishMenublock fromSnipdeck.App.csproj<Deploy />fromSnipdeck.Appin the slnx so the build doesn't try to run a deploy stepPackage.appxmanifest(only meaningful for MSIX-packaged builds)Assets/*.png(the Start tile / store / lock screen images those manifests pointed at) and theContent Includeentries that referenced themapp.manifeststays (DPI awareness, etc.) and is unrelatedTest plan
dotnet test tests/Snipdeck.Core.Tests— still green🤖 Generated with Claude Code