Skip to content

fix(cli): correct scaffolder project path and drop stale WindowsSdkPackageVersion#146

Merged
sundaramramaswamy merged 2 commits into
microsoft:mainfrom
sundaramramaswamy:fix/scaffolder-path-and-sdk
May 5, 2026
Merged

fix(cli): correct scaffolder project path and drop stale WindowsSdkPackageVersion#146
sundaramramaswamy merged 2 commits into
microsoft:mainfrom
sundaramramaswamy:fix/scaffolder-path-and-sdk

Conversation

@sundaramramaswamy
Copy link
Copy Markdown
Collaborator

@sundaramramaswamy sundaramramaswamy commented May 5, 2026

Problem

dotnet run --project src/Reactor.Cli -- --create MyApp generated a broken app:

  1. Wrong project path — emitted ..\Reactor\Reactor.csproj but the actual path is ..\src\Reactor\Reactor.csproj
  2. Stale WindowsSdkPackageVersion — pinned 10.0.22621.52 which ships CsWinRT 2.1, conflicting with Windows App SDK 2.0-preview2 (CsWinRT 2.2). The app builds with MSB3277 warnings and crashes at runtime with FileNotFoundException: WinRT.Runtime 2.2.0.0

Fix

  • Correct the ProjectReference path in the scaffolder template, generated .sln, and README example
  • Remove the hardcoded WindowsSdkPackageVersion — let the SDK resolve it automatically (matching what Reactor.csproj already does)

Verification

Regenerated MyApp, built, and launched successfully with .NET 9 SDK.

sundaramramaswamy and others added 2 commits May 5, 2026 11:39
The --create scaffolder and README Quick Start emitted
..\Reactor\Reactor.csproj but the actual path is
..\src\Reactor\Reactor.csproj.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The template pinned WindowsSdkPackageVersion to
10.0.22621.52 which ships CsWinRT 2.1, conflicting
with Windows App SDK 2.0-preview2 (CsWinRT 2.2).
Let the SDK resolve it automatically, matching what
Reactor.csproj already does.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@sundaramramaswamy sundaramramaswamy force-pushed the fix/scaffolder-path-and-sdk branch from 0e98624 to 3084dd0 Compare May 5, 2026 06:09
@sundaramramaswamy
Copy link
Copy Markdown
Collaborator Author

All 6 CI checks passed (Build, Unit Tests, Selftests, Pack, CLA, Detect changes). Local verification also confirmed: unit tests (6790 passed, 3 pre-existing failures), selftests (638 passed, 1 pre-existing failure). Ready for merge.

Copy link
Copy Markdown
Collaborator

@codemonkeychris codemonkeychris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sundaramramaswamy sundaramramaswamy merged commit 3e8c6ed into microsoft:main May 5, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants