docs(ai-team): Sprint 3 planning session

Session: 2026-02-11-sprint3-planning
Requested by: Jeffrey T. Fritz

Changes:
- Logged session to .ai-team/log/2026-02-11-sprint3-planning.md
- Merged 3 decisions from inbox into decisions.md
- Updated status.md to reflect 48/53 components complete
- Sprint 3 scope: DetailsView + PasswordRecovery
- Propagated cross-agent updates to all agent history files

Author: csharpfritz

.ai-team/agents/beast/history.md

@@ -21,3 +21,5 @@
 📌 Team update (2026-02-10): PRs #328 (ASCX CLI) and #309 (VS Snippets) shelved indefinitely — decided by Jeffrey T. Fritz
 📌 Team update (2026-02-10): Sprint 1 gate review — ImageMap (#337) APPROVED, PageService (#327) APPROVED, ready to merge — decided by Forge
 📌 Team update (2026-02-10): Sprint 2 complete — Localize, MultiView+View, ChangePassword, CreateUserWizard shipped with docs, samples, tests. 709 tests passing. 41/53 components done. — decided by Squad
+📌 Team update (2026-02-11): Sprint 3 scope: DetailsView + PasswordRecovery. Chart/Substitution/Xml deferred. 48/53 → target 50/53. — decided by Forge
+📌 Team update (2026-02-11): Colossus added as dedicated integration test engineer. Rogue retains bUnit unit tests. — decided by Jeffrey T. Fritz

.ai-team/agents/cyclops/history.md

@@ -31,3 +31,5 @@
 📌 Team update (2026-02-10): Lockout protocol — Cyclops locked out of Calendar and FileUpload revisions — decided by Jeffrey T. Fritz
 📌 Team update (2026-02-10): Close PR #333 without merging — all Calendar work already on dev, fixes committed directly to dev — decided by Rogue
 📌 Team update (2026-02-10): Sprint 2 complete — Localize, MultiView+View, ChangePassword, CreateUserWizard shipped with docs, samples, tests. 709 tests passing. 41/53 components done. — decided by Squad
+📌 Team update (2026-02-11): Sprint 3 scope: DetailsView + PasswordRecovery. Chart/Substitution/Xml deferred. 48/53 → target 50/53. — decided by Forge
+📌 Team update (2026-02-11): Colossus added as dedicated integration test engineer. Rogue retains bUnit unit tests. — decided by Jeffrey T. Fritz

.ai-team/agents/forge/history.md

@@ -49,3 +49,35 @@
 📌 Team update (2026-02-10): Lockout protocol — Cyclops locked out of Calendar and FileUpload revisions — decided by Jeffrey T. Fritz
 📌 Team update (2026-02-10): Close PR #333 without merging — all Calendar work already on dev, PR branch has 0 unique commits — decided by Rogue
 📌 Team update (2026-02-10): Sprint 2 complete — Localize, MultiView+View, ChangePassword, CreateUserWizard shipped with docs, samples, tests. 709 tests passing. 41/53 components done. — decided by Squad
+
+### 2026-02-10 — Sprint 3 Planning & Status Reconciliation
+
+**Status.md was significantly stale:**
+- Calendar was merged to dev via commit d33e156 and PR #339 but still marked 🔴 Not Started
+- FileUpload was merged via PRs #335 and #338 but still marked 🔴 Not Started
+- Summary table said 41/53 (Editor: 20/27) but actual count of ✅ entries in the detailed section was already 23/27 for Editors (now 25/27 with Calendar + FileUpload fixed)
+- The 27-count for Editor Controls groups MultiView and View as one logical component despite separate table rows
+- Corrected total: 48/53 components complete (91%), 5 remaining
+
+**Sprint 3 scope decision:**
+- DetailsView and PasswordRecovery are the two buildable components
+- Chart deferred: requires SVG/Canvas rendering engine, no Blazor primitive equivalent
+- Substitution deferred: Web Forms output caching has no Blazor architectural equivalent
+- Xml deferred: XSLT transforms are a dead-end pattern with near-zero migration demand
+- Post-Sprint 3 state: 50/53 (94%), library effectively feature-complete for practical migration
+
+**DetailsView design notes:**
+- Must inherit BaseStyledComponent (Web Forms DetailsView → CompositeDataBoundControl → WebControl)
+- Renders as `<table>` with one `<tr>` per field (vertical layout vs GridView's horizontal)
+- Can reuse existing BoundField, TemplateField, CommandField, HyperLinkField, ButtonField from GridView
+- Needs DetailsViewMode enum (ReadOnly=0, Edit=1, Insert=2)
+- Needs 8 EventArgs classes for mode changes, CRUD operations
+
+**PasswordRecovery design notes:**
+- Must inherit BaseStyledComponent
+- 3-step wizard flow: UserName → Question → Success (same pattern as CreateUserWizard's 2-step)
+- Can reuse existing LoginControls style sub-components (TitleTextStyle, TextBoxStyle, LabelStyle, etc.)
+- Table-based HTML output matching ChangePassword's render pattern
+
+📌 Team update (2026-02-10): Sprint 3 plan ratified — DetailsView + PasswordRecovery. Chart/Substitution/Xml deferred indefinitely with migration docs. 48/53 → target 50/53. — decided by Forge
+📌 Team update (2026-02-11): Colossus added as dedicated integration test engineer. Rogue retains bUnit unit tests. — decided by Jeffrey T. Fritz

.ai-team/agents/jubilee/history.md

@@ -31,3 +31,5 @@
 - **Assigned because:** Cyclops (original author) was locked out per reviewer rejection protocol after Forge's gate review flagged this issue.
 
 📌 Team update (2026-02-10): Sprint 2 complete — Localize, MultiView+View, ChangePassword, CreateUserWizard shipped with docs, samples, tests. 709 tests passing. 41/53 components done. — decided by Squad
+📌 Team update (2026-02-11): Sprint 3 scope: DetailsView + PasswordRecovery. Chart/Substitution/Xml deferred. 48/53 → target 50/53. — decided by Forge
+📌 Team update (2026-02-11): Colossus added as dedicated integration test engineer. Rogue retains bUnit unit tests. — decided by Jeffrey T. Fritz

.ai-team/agents/rogue/history.md

@@ -17,3 +17,5 @@
 📌 Team update (2026-02-10): Sprint 1 gate review — Calendar (#333) REJECTED (assigned Rogue for triage) — decided by Forge
 📌 Team update (2026-02-10): Close PR #333 without merging — all Calendar work already on dev, PR branch has 0 unique commits — decided by Rogue
 📌 Team update (2026-02-10): Sprint 2 complete — Localize, MultiView+View, ChangePassword, CreateUserWizard shipped with docs, samples, tests. 709 tests passing. 41/53 components done. — decided by Squad
+📌 Team update (2026-02-11): Sprint 3 scope: DetailsView + PasswordRecovery. Chart/Substitution/Xml deferred. 48/53 → target 50/53. — decided by Forge
+📌 Team update (2026-02-11): Colossus added as dedicated integration test engineer. Rogue retains bUnit unit tests. — decided by Jeffrey T. Fritz

.ai-team/decisions.md

@@ -93,3 +93,21 @@
 **By:** Squad (Forge, Cyclops, Beast, Jubilee, Rogue)
 **What:** Localize, MultiView+View, ChangePassword, and CreateUserWizard all shipped with full docs, sample pages, and tests. Build passes with 0 errors, 709 tests. status.md updated to 41/53 components (77%).
 **Why:** Sprint 2 milestone — all planned components delivered with docs and samples per team policy.
+
+### Integration test audit — full coverage achieved
+
+**By:** Colossus
+**What:** Audited all 74 sample page routes against existing smoke tests. Found 32 pages without smoke tests and added them all as `[InlineData]` entries in `ControlSampleTests.cs`. Added 4 new interaction tests in `InteractiveComponentTests.cs` for Sprint 2 components: MultiView (view switching), ChangePassword (form fields), CreateUserWizard (form fields), Localize (text rendering). Fixed pre-existing Calendar sample page CS1503 errors (bare enum values → fully qualified `CalendarSelectionMode.X`).
+**Why:** Every sample page is a promise to developers. The integration test matrix must cover every route to catch rendering regressions. The Calendar fix was required to unblock the build — all 4 errors were in the sample page, not the component.
+
+### 2026-02-10: Sprint 3 Scope and Plan
+
+**By:** Forge
+**What:** Sprint 3 scope finalized — DetailsView and PasswordRecovery are the two buildable components. Chart, Substitution, and Xml deferred indefinitely.
+**Why:** With 48/53 components complete (91%), we have exactly 5 remaining. Three of them (Chart, Substitution, Xml) are poor candidates: Chart requires an entire charting library, Substitution is a cache-control mechanism that has no Blazor equivalent, and Xml/XSLT transforms are a dead-end technology with near-zero migration demand. DetailsView and PasswordRecovery are the only two that provide real migration value and are feasible to build.
+
+### 2026-02-10: Colossus added — dedicated integration test engineer
+
+**By:** Jeffrey T. Fritz (via Squad)
+**What:** Added Colossus as a new team member responsible for Playwright integration tests. Colossus owns `samples/AfterBlazorServerSide.Tests/` and ensures every sample page has a corresponding integration test (smoke, render, and interaction). Rogue retains ownership of bUnit unit tests. Integration testing split from Rogue's QA role.
+**Why:** Sprint 2 audit revealed no integration tests existed for any newly shipped components. Having a dedicated agent ensures integration test coverage keeps pace with component development. Every sample page is a promise to developers — Colossus verifies that promise in a real browser.

.ai-team/decisions/inbox/colossus-integration-test-audit.md

@@ -0,0 +1,26 @@
+# SKILL: Status Reconciliation
+
+**Confidence:** low
+**Source:** earned
+
+## When to Use
+When planning a sprint or auditing project status, and the status tracking document may be stale relative to the actual codebase state.
+
+## Pattern
+
+1. **Read the status document** to understand what it claims.
+2. **Verify claims against the filesystem** — check if files marked "Not Started" actually exist on the current branch. Use `glob` to search for `.razor`, `.cs`, or other source files.
+3. **Verify claims against git history** — use `git log --oneline -- <path>` to confirm when files were added and which PRs merged them.
+4. **Check downstream artifacts** — docs, sample pages, tests, nav entries. A component isn't truly "Complete" without these per team policy.
+5. **Reconcile counts** — summary tables often drift from detailed breakdowns. Count the actual ✅ entries and compare to the summary. Fix both to match reality.
+6. **Watch for grouping mismatches** — components like MultiView/View may be listed as separate rows but counted as one logical component in totals.
+
+## Common Traps
+
+- **Merge-to-dev without status update:** PRs merge but nobody updates the tracking doc. Always check git log for recent merges.
+- **Summary vs detail drift:** Summary table says "20 complete" but counting ✅ rows gives 23. One gets updated, the other doesn't.
+- **Section headers vs tables:** A section header says "18/27" but the summary table says "20/27" and the actual count is "23/27". Three numbers, three different values.
+- **Stale estimation tables:** Effort estimation sections list items as remaining work that are already complete. These sections are informational but misleading if not updated.
+
+## Output
+A corrected status document with consistent counts across summary, detail, and estimation sections.

.ai-team/decisions/inbox/squad-colossus-added.md

@@ -2,18 +2,18 @@
 
 | Category | Completed | In Progress | Not Started | Total |
 |----------|-----------|-------------|-------------|-------|
-| Editor Controls | 20 | 0 | 7 | 27 |
+| Editor Controls | 25 | 0 | 2 | 27 |
 | Data Controls | 7 | 0 | 2 | 9 |
 | Validation Controls | 7 | 0 | 0 | 7 |
 | Navigation Controls | 3 | 0 | 0 | 3 |
 | Login Controls | 6 | 0 | 1 | 7 |
-| **TOTAL** | **41** | **0** | **12** | **53** |
+| **TOTAL** | **48** | **0** | **5** | **53** |
 
 ---
 
 ## Detailed Component Breakdown
 
-### 🟡 Editor Controls (18/27 - 67% Complete)
+### 🟢 Editor Controls (25/27 - 93% Complete)
 
 | Component | Status | Notes |
 |-----------|--------|-------|
@@ -31,9 +31,9 @@
 | RadioButton | ✅ Complete | Documented, tested, sample page exists |
 | TextBox | ✅ Complete | Documented, tested, sample page exists |
 | BulletedList | ✅ Complete | Documented, tested (41 tests), sample page exists |
-| Calendar | 🔴 Not Started | Complex date picker |
+| Calendar | ✅ Complete | Documented, tested, table-based rendering, CalendarSelectionMode enum |
 | CheckBoxList | ✅ Complete | Documented, tested (26 tests) |
-| FileUpload | 🔴 Not Started | Consider Blazor InputFile |
+| FileUpload | ✅ Complete | Documented, tested, uses Blazor InputFile internally |
 | ImageMap | ✅ Complete | Documented, tested (23 tests) |
 | ListBox | ✅ Complete | Documented, tested, supports single/multi-select |
 | Localize | ✅ Complete | Documented, tested, inherits from Literal |
@@ -81,7 +81,7 @@
 | SiteMapPath | ✅ Complete | Documented, tested (23 tests), sample page exists |
 | TreeView | ✅ Complete | Documented in TreeView.md |
 
-### 🟡 Login Controls (4/7 - 57% Complete)
+### 🟢 Login Controls (6/7 - 86% Complete)
 
 | Component | Status | Notes |
 |-----------|--------|-------|
@@ -131,25 +131,24 @@
 | ~~**Menu**~~ | ~~Medium-High~~ | ~~12-16~~ | ~~6-8~~ | ✅ Complete |
 | ~~**SiteMapPath**~~ | ~~Medium~~ | ~~8-10~~ | ~~4-5~~ | ✅ Complete |
 | ~~**DataPager**~~ | ~~Medium~~ | ~~8-12~~ | ~~4-6~~ | ✅ Complete |
-| **DetailsView** | High | 16-24 | 8-12 |
+| ~~**DetailsView**~~ | ~~High~~ | ~~16-24~~ | ~~8-12~~ | Sprint 3 |
 
 #### Login Controls
 | Component | Complexity | Est. Hours (Manual) | Est. Hours (with Copilot) |
 |-----------|------------|---------------------|---------------------------|
-| **ChangePassword** | High | 16-24 | 8-12 |
-| **PasswordRecovery** | High | 16-24 | 8-12 |
-| **CreateUserWizard** | Very High | 24-32 | 12-16 |
+| ~~**ChangePassword**~~ | ~~High~~ | ~~16-24~~ | ~~8-12~~ | ✅ Complete |
+| **PasswordRecovery** | High | 16-24 | 8-12 | Sprint 3 |
+| ~~**CreateUserWizard**~~ | ~~Very High~~ | ~~24-32~~ | ~~12-16~~ | ✅ Complete |
 
 #### Lower Priority / Consider Deferring
 | Component | Complexity | Notes |
 |-----------|------------|-------|
-| ~~**BulletedList**~~ | ~~Low~~ | ~~Simple HTML list~~ | ✅ Complete |
-| **Calendar** | High | Complex date picker |
-| **FileUpload** | Medium | Blazor has InputFile |
+| ~~**Calendar**~~ | ~~High~~ | ~~Complex date picker~~ | ✅ Complete |
+| ~~**FileUpload**~~ | ~~Medium~~ | ~~Blazor has InputFile~~ | ✅ Complete |
 | ~~**ImageMap**~~ | ~~Medium~~ | ~~Clickable regions~~ | ✅ Complete |
-| **MultiView/View** | Medium | Tab-like container |
+| ~~**MultiView/View**~~ | ~~Medium~~ | ~~Tab-like container~~ | ✅ Complete |
 | ~~**Table**~~ | ~~Low~~ | ~~HTML table wrapper~~ | ✅ Complete |
-| **Localize** | Low | Localization |
+| ~~**Localize**~~ | ~~Low~~ | ~~Localization~~ | ✅ Complete |
 | **Xml** | Medium | XML transform |
 | **Substitution** | N/A | Cache-related, may not apply |
 | **Chart** | Very High | Consider external library |
@@ -161,10 +160,10 @@
 |--------|-------------------|------------------------|
 | ~~**High Priority (4)**~~ | ~~20-30 hours~~ | ~~10-15 hours~~ | ✅ Complete |
 | **Medium Priority (3 remaining)** | ~12-20 hours | ~6-10 hours |
-| **Nav & Data (3 remaining)** | ~32-46 hours | ~16-23 hours |
-| **Login (3)** | ~56-80 hours | ~28-40 hours |
-| **Lower Priority (10)** | Variable | Variable |
-| **Total Remaining** | 18 components | 18 components |
+| **Nav & Data (1 remaining)** | ~16-24 hours | ~8-12 hours |
+| **Login (1 remaining)** | ~16-24 hours | ~8-12 hours |
+| **Lower Priority (3 remaining)** | Variable | Variable |
+| **Total Remaining** | 5 components | 5 components |
 
 ---
 
@@ -196,27 +195,25 @@
 3. ~~**RadioButton**~~ - ✅ Complete
 4. ~~**DropDownList**~~ - ✅ Complete
 
-### Phase 2: List & Container Controls (Next Priority)
+### Phase 2: List & Container Controls ✅ COMPLETE
 5. ~~**Panel**~~ - ✅ Complete
 6. ~~**PlaceHolder**~~ - ✅ Complete
 7. ~~**CheckBoxList**~~ - ✅ Complete (Multi-select)
 8. ~~**RadioButtonList**~~ - ✅ Complete (Single-select group, 30 tests)
 9. ~~**ListBox**~~ - ✅ Complete
 
-### Phase 3: Navigation & Data
+### Phase 3: Navigation & Data ✅ MOSTLY COMPLETE
 10. ~~**Menu**~~ - ✅ Complete
 11. ~~**SiteMapPath**~~ - ✅ Complete (Breadcrumb navigation, 23 tests)
 12. ~~**DataPager**~~ - ✅ Complete (Paging for ListView)
-13. **DetailsView** - Single-record display
+13. **DetailsView** - Single-record display (Sprint 3)
 
-### Phase 4: Login Controls
-14. **ChangePassword** - Password change UI
-15. **PasswordRecovery** - Password reset flow
-16. **CreateUserWizard** - User registration
+### Phase 4: Login Controls ✅ MOSTLY COMPLETE
+14. ~~**ChangePassword**~~ - ✅ Complete
+15. **PasswordRecovery** - Password reset flow (Sprint 3)
+16. ~~**CreateUserWizard**~~ - ✅ Complete
 
 ### Consider Deferring
 - **Chart** - Very high complexity, consider Blazor charting libraries
-- **Calendar** - Complex, many alternatives exist
-- **DataGrid** - Legacy, use GridView instead
 - **Substitution** - Cache-related, may not apply to Blazor
-- **FileUpload** - Blazor has built-in InputFile component
+- **Xml** - Low migration demand, XSLT is rarely used in modern apps