FritzAndFriends
diff --git a/‎.ai-team/agents/beast/history.md‎
Lines changed: 6 additions & 0 deletions b/‎.ai-team/agents/beast/history.md‎
Lines changed: 6 additions & 0 deletions
diff --git a/‎.ai-team/agents/cyclops/history.md‎
Lines changed: 21 additions & 0 deletions b/‎.ai-team/agents/cyclops/history.md‎
Lines changed: 21 additions & 0 deletions
diff --git a/‎.ai-team/agents/forge/history.md‎
Lines changed: 38 additions & 18 deletions b/‎.ai-team/agents/forge/history.md‎
Lines changed: 38 additions & 18 deletions
diff --git a/‎.ai-team/agents/jubilee/history.md‎
Lines changed: 12 additions & 0 deletions b/‎.ai-team/agents/jubilee/history.md‎
Lines changed: 12 additions & 0 deletions
diff --git a/‎.ai-team/agents/rogue/history.md‎
Lines changed: 12 additions & 0 deletions b/‎.ai-team/agents/rogue/history.md‎
Lines changed: 12 additions & 0 deletions
diff --git a/‎.ai-team/decisions.md‎
Lines changed: 24 additions & 0 deletions b/‎.ai-team/decisions.md‎
Lines changed: 24 additions & 0 deletions
@@ -88,3 +88,9 @@
 
  Team update (2026-03-02): ModelErrorMessage component spec consolidated  29/29 WingtipToys coverage, BaseStyledComponent, EditContext pattern  decided by Forge
 
+- **Executive Migration Report (`planning-docs/WINGTIPTOYS-MIGRATION-EXECUTIVE-REPORT.md`):**
+  - Created executive migration report for manager audience
+  - Key metrics: 96.6% control coverage, 55-70% time savings, 18-26 hour migration estimate
+  - Structured for non-technical readers: bottom-line callout, business value lead, tables for data, minimal jargon
+  - Sections: What is BWFC, Migration Scope, Component Coverage, Three-Layer Pipeline, Time & Cost Impact, Layer 1 Results, Page Readiness, Risk Reduction, What's Next
+
@@ -59,6 +59,27 @@ Team update (2026-02-27): UpdatePanel Triggers deliberately omitted  decided by
 Team update (2026-02-28): GetCssClassOrNull() uses IsNullOrEmpty not IsNullOrWhiteSpace  low priority  noted by Rogue
  Team update (2026-03-01): Skins & Themes has dual docs  SkinsAndThemes.md (practical guide, update first) and ThemesAndSkins.md (architecture). Update SkinsAndThemes.md first for API changes  decided by Beast
 
+<!--  Summarized 2026-03-03 by Scribe  covers M20 theming through CSS fixes -->
+
+### M20 Theming, Release & WingtipToys Context (2026-03-01 through 2026-03-03)
+
+**Issue #366 theme wiring:** CascadingParameter ThemeConfiguration moved to BaseWebFormsComponent (named CascadedTheme). ApplySkin>ApplyThemeSkin (virtual override chain). ThemeProvider uses @inherits ComponentBase. WebFormsPage cascades Theme ?? CascadedTheme. Lesson: _Imports.razor @inherits affects ALL .razor files.
+
+**FontInfo auto-sync:** Name/Names backing-field properties with bidirectional sync. ApplyThemeSkin guards both Font.Name AND Font.Names. Lesson: paired Web Forms properties must replicate sync behavior.
+
+**Unified release.yml:** Single workflow on release:published. Version from tag_name. NuGet override: -p:PackageVersion + -p:Version. version.json 3-segment SemVer. deploy-server-side.yml/nuget.yml to workflow_dispatch-only.
+
+**Issue #406 ListView EditItemTemplate:** (1) Closure bug: moved template selection before CascadingValue. (2) Diffing bug: added @key. Lessons: @key in loops with template switching; never reference loop-external mutable vars in ChildContent.
+
+**FormView RenderOuterTable:** [Parameter] bool RenderOuterTable=true. When false, suppresses table wrapper/header/footer/pager. Driven by WingtipToys ProductDetails.aspx.
+
+**Original WingtipToys build:** Connection strings v11.0 to MSSQLLocalDB. Empty Directory.Build.props blocks NBGV. `nuget install` for packages.config. IIS Express port 5200.
+
+**CSS fixes screenshot refresh (2026-03-03):** 7 visual fixes applied (Cerulean theme, 4-column grid, BoundField currency, Trucks category, Site.css, category IDs). 6 screenshots updated. Lessons: Playwright blocks file:// -- serve via HTTP; verify visual requirements before capturing; detached dotnet run needs Get-NetTCPConnection for PID cleanup.
+
+📌 Team update (2026-03-02): Skins & Themes roadmap — 3 waves, 15 WIs — decided by Forge
+📌 Team updates (2026-03-02): M22 planned (Forge), project reframed as migration system (Jeff), FormView RenderOuterTable resolved (Cyclops), ModelErrorMessage 29/29 coverage (Forge), WingtipToys pipeline validated — 28/29 controls covered.
+📌 Team update (2026-03-03): WingtipToys CSS fidelity — 7 visual differences identified requiring fixes (Cerulean theme, 4-column grid, BoundField bug, Trucks category, Site.css, category IDs) — decided by Forge
 <!-- Summarized 2026-03-02 by Scribe -- covers M20 theming + release process -->
 
 ### M20 Theming & Release Process Summary (2026-03-01 through 2026-03-02)
 
@@ -57,26 +57,46 @@ Line-by-line classification: DataList (110 lines), GridView (33 lines), ListView
 Team updates: M17 audit fixes resolved (PR #402), Skins dual docs (SkinsAndThemes.md + ThemesAndSkins.md), Normalizer pipeline codified (Issue #387).
 
 
-### Summary: Build/Version/Release Process Audit (2026-03-02)
 
-**By:** Forge
-**What:** Full audit of 7 CI/CD workflows, NBGV version management, and release coordination. Audit saved to `.ai-team/decisions/inbox/forge-version-release-audit.md`.
-
-**Critical findings:**
-1. `version.json` on main says `0.15` but latest tag is `v0.16` — NBGV computes `0.15.X`, not `0.16.0`. Every artifact built from main has wrong version prefix.
-2. NuGet (tag-triggered), Docker (main-push-triggered), docs (main-push or tag), and demos (main-push) all fire independently with no version coordination.
-3. `docs.yml` uses deprecated `::set-output` and has a release-detection regex that never matches this project's 2-segment tags (`v0.16`). Docs only deploy on main push, never on tag.
-4. No GitHub Release automation — releases are created manually, not all tags have releases.
-5. Docker image version comes from `nbgv get-version` on runner which reads stale `version.json`, not the tag.
-
-**Recommendation:**
-- Create unified `release.yml` triggered by GitHub Release `published` event. One trigger → NuGet + Docker + docs + demos, all with same version from tag.
-- Keep NBGV but use it correctly: version.json drives dev/CI versions, release workflow overrides with `-p:Version=${TAG}` for exact release version.
-- Standardize on 3-segment SemVer tags (`v0.17.0`).
-- Retire independent deployment triggers (nuget.yml tag trigger, deploy-server-side.yml main-push trigger).
-- Automate post-release version bump via PR to dev.
-- Fix docs.yml deprecated syntax and broken regex immediately.
+<!-- Summarized 2026-03-02 by Scribe -- covers Build/Release audit through WingtipToys pipeline validation -->
+
+### Build/Release & M22 Migration Summary (2026-03-02)
+
+**Build/Version/Release audit:** version.json on main said 0.15 vs latest tag v0.16. Unified release.yml implemented (PR #408)  single workflow on release:published coordinates NuGet + Docker + docs + demos. NBGV 3.9.50, PackageId Fritz.BlazorWebFormsComponents, Docker ghcr.io/fritzandfriends/blazorwebformscomponents/serversidesamples. version.json now 0.17.0 (3-segment SemVer). Old nuget.yml/deploy-server-side.yml refactored to workflow_dispatch-only.
+
+**M22 Copilot-Led Migration Showcase:** 12 work items, 4 waves. 57 controls (51 functional, 6 stubs). All 16 core demo controls ready. Use existing BeforeWebForms sample (6-8 pages). Separate .github/copilot-migration-instructions.md. Skins & AJAX Toolkit OUT. ListView EditItemTemplate bug (#406) IN.
+
+**WingtipToys initial analysis:** 15+ pages, 22 controls, 100% BWFC coverage (all 4 "missing" controls already existed). Only blocking gap was FormView RenderOuterTable (resolved). Architecture: LayoutComponentBase, @page directives, EF Core + IDbContextFactory, scoped DI services, scaffolded Identity UI, HttpClient for PayPal, InteractiveServer. 36 work items, 5 phases.
+
+**ASPX/ASCX tooling strategy:** 85+ syntax patterns from 33 WingtipToys files. Three-layer pipeline: Layer 1 (bwfc-migrate.ps1, ~40%, mechanical regex), Layer 2 (Copilot skill, ~45%, structural), Layer 3 (Copilot agent, ~15%, semantic). NOT building: standalone CLI, VS Code extension, Roslyn analyzer. SelectMethod->Items = #1 structural transform. Session->scoped DI = hardest semantic transform.
+
+**ModelErrorMessage spec:** BaseStyledComponent (not BaseValidator). CascadingParameter EditContext. ModelStateKey->Field(key). Renders <span>, nothing when no error. Strips \x1F metadata. Validations/ folder. 29/29 WingtipToys coverage.
+
+**WingtipToys pipeline validation (post-script):** Layer 1 ~70% markup: 147+ tag removals, 165+ runat removals, 35+ expression conversions. 18 data-binding expressions unconverted (<%#: inside GetRouteUrl/String.Format/Eval). 3 user-control prefixes survive. FormView RenderOuterTable working. ModelErrorMessage in 2 files. Actionable: (1) add <%#: regex, (2) strip uc:/friendlyUrls: prefixes, (3) SelectMethod->Items as #1 Layer 2 example, (4) Identity scaffold guidance. **18-26 hours total, 4-6 hours for demo subset.**
+
+Team updates (2026-03-02): Unified release (PR #408), project reframed as migration acceleration system (Jeff), ModelErrorMessage docs (52 components, Beast), WingtipToys pipeline validated (4 ready, 21 skill, 8 architecture).
+
+### WingtipToys CSS Fidelity Audit (2026-03-02)
+
+**7 visual differences found** between original WingtipToys (:5200) and migrated Blazor (:5201):
+
+1. **Navbar color (CRITICAL):** Original uses Bootswatch Cerulean v3.2.0 (`#033c73` dark blue navbar, `#ffffff` white link text). Migrated loads stock Bootstrap 3.4.1 from CDN (`#222222` dark gray navbar, `#999999` gray link text). Root cause: `App.razor` references CDN bootstrap instead of the custom `Content/bootstrap.css`. File: `samples/WingtipToys/WingtipToys/Content/bootstrap.css` line 1 = `bootswatch v3.2.0`, line 4177 = `background-color: #033c73`.
+
+2. **Heading colors:** Bootswatch Cerulean sets `h1-h6 { color: #317eac }` (line 995). Stock Bootstrap uses `#333333`. Visible on "Welcome.", "Shopping Cart", "Products" headings.
+
+3. **Link colors:** Cerulean: `a { color: #2fa4e7 }` (line 906). Stock Bootstrap 3.4.1: `#337ab7`. Affects category menu, product names, "Add To Cart" links.
+
+4. **Product grid layout (CRITICAL):** Original `ProductList.aspx` uses `GroupItemCount="4"` + `GroupTemplate` + `LayoutTemplate` for 4-column grid. Migrated `ProductList.razor` omits all three — products render in single column. BWFC ListView supports GroupItemCount (lines 80-134 of `ListView.razor`), so this is a migration omission.
+
+5. **Missing "Trucks" category:** Original has 5 categories (data-driven via ListView). `MainLayout.razor` hardcodes only 4 (Cars, Planes, Boats, Rockets — missing Trucks).
+
+6. **Site.css not loaded:** `Content/Site.css` exists in migrated project but is not referenced in `App.razor`. Original bundles it via `Bundle.config`. Missing `body { padding-top: 50px }` and `.body-content` padding rules.
+
+7. **BoundField DataFormatString bug:** `BoundField.razor.cs` line 48: `string.Format(DataFormatString, obj?.ToString())` converts obj to string BEFORE formatting, so `{0:c}` currency format is lost. Cart "Price (each)" shows "15.95" instead of "$15.95". Fix: use `obj` not `obj?.ToString()`.
+
+**Key files:** Original CSS: `samples/WingtipToys/WingtipToys/Content/bootstrap.css` (Bootswatch Cerulean). Migrated layout: `samples/AfterWingtipToys/Components/App.razor` (CDN bootstrap ref). Migrated content: `samples/AfterWingtipToys/Components/Layout/MainLayout.razor`. BWFC bug: `src/BlazorWebFormsComponents/BoundField.razor.cs:48`.
 
+ Team update (2026-03-03): Original WingtipToys build/run config documented (LocalDB, NBGV isolation, NuGet restore, IIS Express port 5200)  decided by Cyclops
 **Key infrastructure details learned:**
 - NBGV 3.9.50 in `Directory.Build.props`, applied to all projects
 - NuGet PackageId: `Fritz.BlazorWebFormsComponents`
 
@@ -69,5 +69,17 @@
  Team update (2026-03-02): ModelErrorMessage component spec consolidated  29/29 WingtipToys coverage, BaseStyledComponent, EditContext pattern  decided by Forge
 
 
+📌 Team update (2026-03-02): ModelErrorMessage documentation shipped — docs/ValidationControls/ModelErrorMessage.md, status.md updated to 52 components — decided by Beast
+
+### M22 Executive Screenshot Comparison Pages (2026-03-02)
+
+- **Created 3 HTML comparison pages** in `planning-docs/screenshots/` for Playwright screenshots at 1400×900:
+  - `comparison-productlist.html` — ListView before/after (Web Forms → Blazor+BWFC)
+  - `comparison-shoppingcart.html` — GridView, BoundField, TemplateField, TextBox, CheckBox, Label, Button
+  - `comparison-login.html` — PlaceHolder, Literal, Label, TextBox, RequiredFieldValidator, CheckBox, Button, HyperLink
+- **Used dark theme** (#1e1e1e background) with red (`#f48771`) highlighting for removed Web Forms artifacts and green (`#89d185`) for new Blazor syntax.
+- **Highlighted key migration changes:** `asp:` prefix removal, `runat="server"` removal, `ItemType` → `TItem`, server binding expressions → `@context`, `ViewStateMode`/`EnableViewState` removal.
+- **Stats bar** at bottom of each page shows controls migrated, attributes preserved, and lines changed.
+- **Source files read:** ProductList.aspx, ShoppingCart.aspx, Account/Login.aspx and their AfterWingtipToys .razor counterparts.
 � Team update (2026-03-02): ModelErrorMessage documentation shipped  docs/ValidationControls/ModelErrorMessage.md, status.md updated to 52 components  decided by Beast
 
@@ -76,6 +76,18 @@ Key patterns: Button=input, Label=span, Panel=div for theme tests. Missing SkinI
 
 Team updates: Unified release process (PR #408), Skins & Themes roadmap (3 waves, 15 WIs).
 
+
+### ListView EditItemTemplate Tests (Issue #406, 2026-03-02)
+
+6 bUnit tests in ListView/EditTemplateTests.razor: EditIndex matching/non-matching/negative, HandleCommand Edit/Cancel swap, null EditItemTemplate fallback. TDD: 2 pass (negative/null), 4 fail (the bug). CSS class selectors (span.display vs span.edit) for template identification. HandleCommand tests use cut.InvokeAsync() for Blazor dispatcher context.
+
+### FormView RenderOuterTable Tests (2026-03-02)
+
+8 bUnit tests in FormView/RenderOuterTable.razor: default/true/false rendering, template content, structural equivalence, empty data (text + template), edit mode. All pass. Compare element counts not raw Markup (Blazor generates unique event handler IDs per render). RenderOuterTable=false: no table/tr/td wrappers, empty data path also strips wrappers.
+
+📌 Test patterns: (1) CSS class selectors for template switching tests. (2) cut.InvokeAsync for HandleCommand. (3) FindAll().Count for structural comparison, not ShouldBe on Markup.
+
+📌 Team updates (2026-03-02): Skins roadmap (3 waves, 15 WIs, Forge), M22 planned (Forge), project reframed as migration system (Jeff), ModelErrorMessage 29/29 (Forge), WingtipToys pipeline validated — three-layer pipeline (Script→Skill→Agent), 18-26 hours total, scaffolded Identity UI for Account pages (Forge).
  Team update (2026-03-02): Full Skins & Themes roadmap defined  3 waves, 15 work items. Wave 1: Theme mode, sub-component styles (41 slots across 6 controls), EnableTheming propagation, runtime switching. See decisions.md for full roadmap and agent assignments  decided by Forge
 ### ListView EditItemTemplate Rendering Tests (Issue #406)
 
 
@@ -5470,3 +5470,27 @@ Nothing is rendered (the component returns `null` / empty fragment). This matche
 **By:** Colossus
 **What:** Added 1 smoke test `[InlineData]` and 3 interactive tests for the ModelErrorMessage sample page (`/ControlSamples/ModelErrorMessage`). Smoke test added to `ValidationControl_Loads_WithoutErrors` Theory group. Interactive tests cover: submit-shows-errors, valid-submit-no-errors, and clear-button-removes-errors.
 **Why:** Every sample page gets an integration test — no exceptions. The ModelErrorMessage component is a validation control that conditionally renders `<span class="text-danger">` elements, so tests verify both the error-present and error-absent states via DOM element counting. The Clear button test exercises the EditContext reset path, which is unique to this sample page.
+
+### 2026-03-02: PR target repository directive
+**By:** Jeffrey T. Fritz (via Copilot)
+**What:** PRs should always target the upstream repository (FritzAndFriends/BlazorWebFormsComponents), not the fork (csharpfritz/BlazorWebFormsComponents). Use cross-fork PR format: head = csharpfritz:{branch}, base = dev on FritzAndFriends.
+**Why:** User request  captured for team memory
+
+
+### 2026-03-02: WingtipToys Migration Analysis Results
+**By:** Forge
+**What:** Comprehensive page-by-page comparison of all 33 WingtipToys source files (.aspx/.ascx/.master) against their migrated Blazor equivalents (.razor). Layer 1 (bwfc-migrate.ps1) successfully handled ~70% of markup transforms: 147+ tag prefix removals, 165+ runat="server" removals, Content wrapper stripping, @page directive generation, ~35 expression conversions, ItemType→TItem conversion, comment syntax, and URL prefix conversion. 18 data-binding expressions remain unconverted (<%#: syntax), 8 SelectMethod attributes need Items/DataItem replacement, 3 GetRouteUrl calls need route interpolation, 3 user-control tag prefixes (uc:, friendlyUrls:) need stripping. BWFC has 100% control coverage for WingtipToys (28/29 controls exist; ContentPlaceHolder maps to @Body). 4 pages are fully ready, 21 need Layer 2 skill work, 8 need Layer 3 architecture (Identity, EF, Session, PayPal). Estimated total migration effort: 18-26 hours across all three layers.
+**Why:** Jeff needs to understand the effectiveness of the three-layer migration pipeline (Script → Skill → Agent) before the M22 Copilot-Led Migration Showcase. This analysis validates that Layer 1 handles high-volume mechanical transforms effectively, Layer 2 covers structural patterns via the Copilot Skill, and Layer 3 architectural decisions are limited to auth/data/session/integrations. The pipeline is proven: a developer using all three layers could migrate WingtipToys to a running Blazor app in under a day. Key actionable findings: (1) Layer 1 should add regex for <%#: Item.X %> → @context.X conversion, (2) Layer 1 should strip user-control tag prefixes during Register directive removal, (3) Account pages should use scaffolded Identity UI rather than migrating OWIN code-behind, (4) SelectMethod→Items is the #1 most common Layer 2 transform.
+
+
+### 2026-03-02: Original WingtipToys Build & Run Configuration
+**By:** Cyclops
+**Date:** 2026-03-02
+**What:** Documented how to build and run the original WingtipToys ASP.NET Web Forms app locally. Connection strings changed from `(LocalDb)\v11.0` to `(LocalDb)\MSSQLLocalDB`. Created empty `samples/WingtipToys/Directory.Build.props` to block NBGV inheritance. Use `nuget install` for packages.config restore. IIS Express on port 5200.
+**Why:** Machine-specific config needed for any dev running the original app for comparison screenshots. Changes confined to samples/WingtipToys/.
+
+### 2026-03-02: WingtipToys CSS Fidelity  7 Visual Differences Identified
+**By:** Forge
+**Date:** 2026-03-02
+**What:** Side-by-side comparison found 7 CSS/visual differences: (1) Wrong Bootstrap theme  stock BS3 instead of Bootswatch Cerulean, (2) Single-column product grid instead of 4-column, (3) Missing Trucks category, (4) Site.css not referenced, (5) BoundField DataFormatString bug  premature .ToString() loses numeric formatting, (6) bootstrap-theme.min.css adding unwanted gradients, (7) Cart prices missing dollar sign (symptom of #5). Fixes: replace CDN with local Cerulean CSS, add GroupItemCount/templates to ListView, add Trucks category, fix BoundField.razor.cs line 48.
+**Why:** Migration showcase screenshots must visually match the original. The BoundField bug is a library-level defect affecting all DataFormatString consumers.