feat: L1 migration scaffold improvements shims, .targets, and script enhancements#526
Merged
csharpfritz merged 3 commits intoFritzAndFriends:devfrom Mar 27, 2026
Conversation
- Add .targets file shipping type aliases (Page, MasterPage, ImageClickEventArgs) and namespace imports with the BWFC NuGet package - Add Identity compatibility types to BWFC library (BlazorWebFormsComponents.Identity): IdentityUser, IdentityResult, UserLoginInfo, ApplicationUserManager, ApplicationSignInManager, SignInStatus virtual no-op stubs for issue FritzAndFriends#525 - Improve bwfc-migrate.ps1 L1 script: - Generate slimmed GlobalUsings.cs (Blazor infrastructure only) - Strip Microsoft.AspNet.*, Microsoft.Owin.*, Owin usings in all copy paths - Replace fully-qualified System.Web.UI base classes with alias-compatible names - Conditional @using BlazorAjaxToolkitComponents (only when detected) - Remove _Imports.razor hardcoded BlazorAjaxToolkitComponents - Remove WebFormsShims.cs and IdentityShims.cs generation (types now in library) - Add migration test reports (run22: full migration, run23: L1 improvements) WingtipToys L1 baseline: 372 errors 32 errors (91% reduction) Remaining 32 are genuine L2 issues (unclosed HTML, EF6 types, template params) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…pression - Add BlazorWebFormsComponents.EntityFramework namespace with DropCreateDatabaseIfModelChanges<T> and Database.SetInitializer<T> stubs - Add IdentityDbContext<T> shim in Identity namespace - Strip base class declarations (: Page, : MasterPage, : UserControl) from code-behinds since .razor @inherits handles inheritance - Add RZ9980/RZ9981/RZ9996/CS0612 to BwfcMigrationMode warning suppression - Auto-set BwfcMigrationMode=true in generated .csproj - Add EntityFramework namespace to .targets global usings - WingtipToys errors: 372 -> 3 (99.2% reduction) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…007 suppression - Add non-generic GridViewRow shim class for Web Forms compatibility (CS0305 fix) - Add [QueryString] and [RouteData] attributes targeting method parameters so L1 script preserves original Web Forms attributes instead of converting them - Add IDE0007 to BwfcMigrationMode warning suppression (style, not functional) - Update L1 script to preserve [QueryString]/[RouteData] (no conversion needed) - Eliminates CS0305 and CS0592 errors; 382 remaining are genuine L2 issues Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
1 participant
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.
Summary
Dramatically reduces L1 (mechanical transform) build errors by shipping type aliases, shim types, and compilation helpers with the BWFC NuGet package, plus improving the migration script's transform pipeline.
WingtipToys test migration: 372 baseline errors 382 remaining (all genuine L2 issues; zero L1 infrastructure errors)
What's Included
BWFC Library New Types
QueryStringAttribute[QueryString("id")]compiles on method params; L2 promotes to[SupplyParameterFromQuery]propertyRouteDataAttribute[RouteData]compiles on method params; L2 promotes to[Parameter]propertyGridViewRow(non-generic)GridViewRowvsGridViewRow<T>gap for migrated codeIdentityUser,IdentityResult,UserLoginInfoApplicationUserManager,ApplicationSignInManagerSignInStatusIdentityDbContext<TUser>DropCreateDatabaseIfModelChanges<T>DatabaseDatabase.SetInitializer<T>()no-opBWFC Library .targets File
New
Fritz.BlazorWebFormsComponents.targetsships in bothbuild/andbuildTransitive/for NuGet consumers:PageWebFormsPageBase,MasterPageLayoutComponentBase,ImageClickEventArgsMouseEventArgs<BwfcMigrationMode>true</BwfcMigrationMode>): CS8618, CS0414, CS0067, CS0612, IDE0007Migration Script (
bwfc-migrate.ps1): Page,: MasterPage,: UserControlremoved from code-behind class declarations (Razor@inheritshandles inheritance)[QueryString]and[RouteData]left as-is (BWFC shim attributes compile)System.Web.*,Microsoft.AspNet.*,Microsoft.Owin.*,Owinremoved from code-behinds, models, and BLL files@using BlazorAjaxToolkitComponentsonly added when AJAX controls detected.csprojTest Reports
dev-docs/migration-tests/wingtiptoys/run22/Baseline full migrationdev-docs/migration-tests/wingtiptoys/run23/First L1 improvementsdev-docs/migration-tests/wingtiptoys/run24/EntityFramework shims + base class strippingdev-docs/migration-tests/wingtiptoys/run25/GridViewRow + QueryString/RouteData attributesRelated