devpro · devpro · Jul 26, 2026 · Jul 26, 2026 · Jul 27, 2026 · Jul 27, 2026
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -67,10 +67,12 @@ jobs:
       dotnet-test-args: "--report-xunit-trx --coverage --coverage-output-format cobertura"
       extra-vars: |
         AllowedOrigins__0=5207
+        AllowedOrigins__1=7042
         Features__IsScalarEnabled=true
         Features__IsHttpsRedirectionEnabled=false
         Infrastructure__MongoDB__ConnectionString=mongodb://localhost:27017
         Infrastructure__MongoDB__DatabaseName=keeptrack_ci
+        Logging__LogLevel__Keeptrack=Debug
       fossa-enabled: true
       sonar-enabled: true
       sonar-cpd-exclusions: "**/*.g.cs,**/*.generated.cs,**/src/WebApi.Contracts/Dto/*.cs,**/src/Domain/Models/*.cs,**/src/Infrastructure.MongoDb/Entities/*.cs"
@@ -89,6 +91,10 @@ jobs:
         Tmdb__ApiKey=${{ secrets.TMDB_APIKEY }}
         Rawg__ApiKey=${{ secrets.RAWG_APIKEY }}
         Discogs__Token=${{ secrets.DISCOGS_TOKEN }}
+        GoogleBooks__ApiKey=${{ secrets.GOOGLEBOOKS_APIKEY }}
+        Omdb__ApiKey=${{ secrets.OMDB_APIKEY }}
+        Igdb__ClientId=${{ secrets.IGDB_CLIENTID }}
+        Igdb__ClientSecret=${{ secrets.IGDB_CLIENTSECRET }}
         FIREBASE_APIKEY=${{ secrets.FIREBASE_APIKEY }}
         FIREBASE_USERNAME=${{ secrets.FIREBASE_TESTUSERNAME }}
         FIREBASE_PASSWORD=${{ secrets.FIREBASE_TESTPASSWORD }}

diff --git a/AGENTS.md b/AGENTS.md
diff --git a/CLAUDE.md b/CLAUDE.md
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
diff --git a/Directory.Build.props b/Directory.Build.props
@@ -5,7 +5,7 @@
 
   <PropertyGroup>
     <!-- edit this value to change the current MAJOR.MINOR.PATCH version -->
-    <VersionPrefix>2.4.0</VersionPrefix>
+    <VersionPrefix>2.4.1</VersionPrefix>
   </PropertyGroup>
 
   <Choose>

diff --git a/Keeptrack.slnx b/Keeptrack.slnx
@@ -4,21 +4,38 @@
     <File Path=".gitignore" />
     <File Path=".markdownlint-cli2.yaml" />
     <File Path=".yamllint.yaml" />
-    <File Path="CLAUDE.md" />
+    <File Path="AGENTS.md" />
     <File Path="CONTRIBUTING.md" />
     <File Path="Directory.Build.props" />
     <File Path="Directory.Packages.props" />
     <File Path="Local.runsettings" />
     <File Path="README.md" />
   </Folder>
   <Folder Name="/Solution Items/docs/">
-    <File Path="docs/automapper-removal-plan.md" />
     <File Path="docs/backlog.md" />
-    <File Path="docs/code-quality-findings.md" />
+    <File Path="docs/igdb-api-notes.md" />
     <File Path="docs/operations.md" />
-    <File Path="docs/plan-quick-add.md" />
-    <File Path="docs/playwright-e2e-tests-plan.md" />
-    <File Path="docs/prerender-flash-fix.md" />
+  </Folder>
+  <Folder Name="/Solution Items/docs/archived/">
+    <File Path="docs/archived/automapper-removal-plan.md" />
+    <File Path="docs/archived/plan-quick-add.md" />
+    <File Path="docs/archived/playwright-e2e-tests-plan.md" />
+    <File Path="docs/archived/prerender-flash-fix.md" />
+    <File Path="docs/archived/README.md" />
+    <File Path="docs/archived/reference-ratings-plan.md" />
+    <File Path="docs/archived/share-collections-plan.md" />
+    <File Path="docs/archived/testing-assessment.md" />
+  </Folder>
+  <Folder Name="/Solution Items/docs/findings/">
+    <File Path="docs/findings/blazor-ui.md" />
+    <File Path="docs/findings/by-design-and-gaps.md" />
+    <File Path="docs/findings/persistence-and-mapping.md" />
+    <File Path="docs/findings/providers.md" />
+    <File Path="docs/findings/README.md" />
+    <File Path="docs/findings/reference-matching.md" />
+    <File Path="docs/findings/sonar.md" />
+    <File Path="docs/findings/sync-explore-and-import.md" />
+    <File Path="docs/findings/video-game-matching.md" />
   </Folder>
   <Folder Name="/Solution Items/pipelines/" />
   <Folder Name="/Solution Items/pipelines/azure-devops/">
@@ -35,11 +52,14 @@
   <Folder Name="/Solution Items/scripts/">
     <File Path="scripts/dedupe-matched-aliases.js" />
     <File Path="scripts/firebase-user-role.js" />
+    <File Path="scripts/migrate-car-history-station.js" />
+    <File Path="scripts/migrate-house-schema.js" />
     <File Path="scripts/migrate-is-owned-to-owned-versions.js" />
     <File Path="scripts/migrate-poster-url-to-image-url.js" />
     <File Path="scripts/migrate-videogame-to-embedded-platforms.js" />
     <File Path="scripts/mongodb-create-index.js" />
     <File Path="scripts/mongodb-install.sh" />
+    <File Path="scripts/unset-tvshow-want-to-watch.js" />
   </Folder>
   <Project Path="src/BlazorApp/BlazorApp.csproj" />
   <Project Path="src/Common.System/Common.System.csproj" />

diff --git a/docs/archived/README.md b/docs/archived/README.md
@@ -0,0 +1,15 @@
+# Archived documents
+
+Finished work kept for provenance only.
+Nothing here describes how the application currently behaves: `AGENTS.md` is the authority for that, and `docs/findings/` tracks open findings.
+A document lands here once its plan has shipped, its migration has run, or its snapshot has been overtaken by the code.
+
+Document                         | What it was                                                                      | Why it is archived
+---------------------------------|----------------------------------------------------------------------------------|-------------------
+`automapper-removal-plan.md`     | Plan to replace AutoMapper with Riok.Mapperly, and the licensing reason for it.   | The migration shipped, Mapperly is the only mapper left.
+`plan-quick-add.md`              | Design for the one-tap Quick Add capture form.                                    | Shipped as `BlazorApp/Components/QuickAdd`.
+`playwright-e2e-tests-plan.md`   | Original design of the Playwright e2e suite and its environment variables.        | The suite exists, `CONTRIBUTING.md` documents the current `E2E_*` surface.
+`prerender-flash-fix.md`         | Investigation of the .NET 10 prerender double-render flash.                       | Fixed, the remaining code comments point back here for the reasoning.
+`reference-ratings-plan.md`      | Design of provider reference ratings, written when RAWG was the game provider.    | Shipped, and the video game half was superseded by IGDB.
+`share-collections-plan.md`      | Design for sharing whole categories with other Keeptrack accounts.                | Shipped as `ShareController`/`SharedWithMeController`.
+`testing-assessment.md`          | Snapshot of the test suite as of 2026-07-27.                                      | A dated inventory, overtaken by the suite itself.
diff --git a/docs/automapper-removal-plan.md → docs/archived/automapper-removal-plan.md b/docs/automapper-removal-plan.md → docs/archived/automapper-removal-plan.md
@@ -7,7 +7,7 @@ RPL-1.5 is an aggressive copyleft license: using AutoMapper under it obligates K
 Until this removal ships, Keeptrack relies on AutoMapper's commercial "Community" tier (free below $5M revenue), which keeps the project bound to a third party's licensing terms and enforcement mechanism (license keys).
 AutoMapper 16.2.0 is currently the only non-permissive dependency in `Directory.Packages.props`; every other package is MIT, Apache-2.0, BSD-3-Clause or MS-PL.
 Removing it therefore ends the last third-party licensing constraint on Keeptrack.
-It is also a quality win: the `AllowNullDestinationValues = false` behavior has caused at least four documented production bugs (see CLAUDE.md and `docs/code-quality-findings.md`).
+It is also a quality win: the `AllowNullDestinationValues = false` behavior has caused at least four documented production bugs (see CLAUDE.md and `docs/findings/`).
 
 ## Replacement choice
 
@@ -119,7 +119,7 @@ Each phase builds, passes the full test suite (including integration tests again
 4. Delete `AutoMapperConfigurationTest` - its job is now done at compile time.
 5. Escalate Mapperly's unmapped-member diagnostics (`RMG012`, `RMG020`) to errors in `.editorconfig`, so configuration drift fails the build the way the deleted test used to fail the run.
 6. Update CLAUDE.md: rewrite the `AllowNullDestinationValues` gotchas for the new null-preserving semantics, and update the "Adding a new trackable item type" checklist (step 6 becomes "add a storage mapper and a DTO mapper").
-7. Update `CONTRIBUTING.md` and `docs/code-quality-findings.md` where they reference AutoMapper.
+7. Update `CONTRIBUTING.md` and `docs/findings/` where they reference AutoMapper.
 
 ## Out of scope
 

diff --git a/docs/plan-quick-add.md → docs/archived/plan-quick-add.md b/docs/plan-quick-add.md → docs/archived/plan-quick-add.md
@@ -62,7 +62,8 @@ Three extraction refactors come first — they remove exactly the duplication **
 4. `IOwnedCopyDto` + `OwnedVersionFields`
 
     - New `src/WebApi.Contracts/Dto/IOwnedCopyDto.cs`:
-      CopyType, Price, AcquiredAt, Vendor, Reference. Implemented by OwnedVersionDto and VideoGamePlatformDto (identical existing members; precedent: IReferenceLinkedDto; no Mapperly impact — mappers map members, not interfaces).
+      CopyType, Price, AcquiredAt, Vendor, Reference.
+      Implemented by OwnedVersionDto and VideoGamePlatformDto (identical existing members; precedent: IReferenceLinkedDto; no Mapperly impact — mappers map members, not interfaces).
     - New `src/BlazorApp/Components/Inventory/Shared/OwnedVersionFields.razor`:
       params required IOwnedCopyDto Copy, optional EventCallback OnChanged (default no-op for Quick Add's nothing-persists-until-Save flow).
       Body = Physical/Digital button pair + the four fields (invariant-culture decimal parsing included), keeping data-testid="version-*-input".
@@ -79,7 +80,9 @@ Three extraction refactors come first — they remove exactly the duplication **
       Movie + TV show always visible; the other six tiles AND their forms inside `<AuthorizeView Policy="MemberOnly" Context="memberContext">` with the preview-account note in `<NotAuthorized>` (hiding is UX; the API enforces).
     - Media forms (per-type markup local to the page, same convention as the list FormTemplates):
       - Movie: Title, Year, "Watched on" (FirstSeenAt, default today for the "just saw it" scenario — visible and clearable, since prefilling marks it Seen), Rating.
-      - TV show: Title, Year. Book: Title, Author, Year, FirstReadAt default today. Album: Title, Artist, Year (Author/Artist feed Open Library/Discogs auto-resolution).
+      - TV show: Title, Year.
+        Book: Title, Author, Year, FirstReadAt default today.
+        Album: Title, Artist, Year (Author/Artist feed Open Library/Discogs auto-resolution).
       - Video game: Title, Year, Platform `<select>` over VideoGames.VideoGamePlatforms; choosing a platform adds a VideoGamePlatformDto draft rendered via `<OwnedVersionFields Copy="...">` — no platform = unowned game.
       - Movie/TvShow/Book/Album: "I own a copy" toggle revealing `<OwnedVersionFields>`; on save if toggled, `dto.OwnedVersions = [_ownedDraft]` (a bare Physical copy is legitimate).
         Single POST carries the copy — no follow-up PUT.
@@ -88,28 +91,30 @@ Three extraction refactors come first — they remove exactly the duplication **
 
 6. Quick Add page — record types
 
-- Parent fetch on type selection via `CarApiClient/HouseApiClient/HealthProfileApiClient.GetAsync("", 1, 100)`, guarded by a per-type loaded flag.
-- 0 parents → empty-state note linking to /cars / /houses / /health to create one; 1 → preselected silently (the common "my car" case);N → segmented button row.
-- Below it: `<CarHistoryForm Entry=... ShowFuel/ShowElectric from the selected car's EnergyType/>`, `<HouseHistoryForm/>`, `<HealthRecordForm/>`. Selecting a parent stamps the Entry's parent id; Save disabled until a parent is selected.
-- SaveRecordAsync → child client AddAsync → navigate /cars/{carId} / /houses/{houseId} / /health/{profileId}.
+    - Parent fetch on type selection via `CarApiClient/HouseApiClient/HealthProfileApiClient.GetAsync("", 1, 100)`, guarded by a per-type loaded flag.
+    - 0 parents → empty-state note linking to /cars / /houses / /health to create one; 1 → preselected silently (the common "my car" case);N → segmented button row.
+    - Below it: `<CarHistoryForm Entry=... ShowFuel/ShowElectric from the selected car's EnergyType/>`, `<HouseHistoryForm/>`, `<HealthRecordForm/>`.
+      Selecting a parent stamps the Entry's parent id; Save disabled until a parent is selected.
+    - SaveRecordAsync → child client AddAsync → navigate /cars/{carId} / /houses/{houseId} / /health/{profileId}.
 
-1. Entry points + CSS
+7. Entry points + CSS
 
-- NavMenu.razor: first item inside `<AuthorizeView><Authorized>` (directly below Home): `<NavLink class="nav-link" href="add"><span class="nav-icon">＋</span> Quick add</NavLink>`.
-  Verify ＋ (U+FF0B) has default text presentation per the emoji gotcha; plain ASCII + is the safe fallback.
-- Home.razor: stats variant gets a kt-home-cta "＋ Quick add" primary button above the stat grid; the empty-state variant makes Quick Add the primary CTA, "Go to my movies" secondary.
-- CSS: mostly reuse (.kt-stat-grid/.kt-stat-tile, .kt-form-card, row g-3 with col-6/col-12 mobile splits, segmented buttons).
-  New app.css is minimal: a .kt-quickadd block forcing the picker to 2 columns under 767px and a full-width Save button on mobile. Forms live in .kt-form-card on a page, not a modal.
-  No sticky save bar in v1 (forms are short) — note as follow-up.
+    - NavMenu.razor: first item inside `<AuthorizeView><Authorized>` (directly below Home): `<NavLink class="nav-link" href="add"><span class="nav-icon">＋</span> Quick add</NavLink>`.
+      Verify ＋ (U+FF0B) has default text presentation per the emoji gotcha; plain ASCII + is the safe fallback.
+    - Home.razor: stats variant gets a kt-home-cta "＋ Quick add" primary button above the stat grid; the empty-state variant makes Quick Add the primary CTA, "Go to my movies" secondary.
+    - CSS: mostly reuse (.kt-stat-grid/.kt-stat-tile, .kt-form-card, row g-3 with col-6/col-12 mobile splits, segmented buttons).
+      New app.css is minimal: a .kt-quickadd block forcing the picker to 2 columns under 767px and a full-width Save button on mobile.
+      Forms live in .kt-form-card on a page, not a modal.
+      No sticky save bar in v1 (forms are short) — note as follow-up.
 
-1. Playwright coverage
+8. Playwright coverage
 
-- Pages/QuickAddPage.cs page object + OpenQuickAddAsync() on Pages/PageBase.cs.
-- QuickAddSmokeTest (E2E gate, cleanup via E2eFixture.ApiHttpClient):
-  a. Movie with owned copy — GUID-suffixed title, toggle copy + price, Save, assert landing on /movies/{id} with the copy in the Ownership section.
-  b. Car refuel — seed a car via API, single car auto-preselected, mileage/fuel/cost, Save, assert landing on /cars/{id} with the refuel row.
-- MobileScreenshotTest: add /add (picker) and /add?type=movie (form) to the 390×844 capture list — the mobile-first acceptance check.
-- Regression safety for the extractions: the existing Car/House/Health/Ownership/VideoGamePlatform smoke tests already cover the refactored surfaces — run them after each extraction step.
+    - Pages/QuickAddPage.cs page object + OpenQuickAddAsync() on Pages/PageBase.cs.
+    - QuickAddSmokeTest (E2E gate, cleanup via E2eFixture.ApiHttpClient):
+      a. Movie with owned copy — GUID-suffixed title, toggle copy + price, Save, assert landing on /movies/{id} with the copy in the Ownership section.
+      b. Car refuel — seed a car via API, single car auto-preselected, mileage/fuel/cost, Save, assert landing on /cars/{id} with the refuel row.
+    - MobileScreenshotTest: add /add (picker) and /add?type=movie (form) to the 390×844 capture list — the mobile-first acceptance check.
+    - Regression safety for the extractions: the existing Car/House/Health/Ownership/VideoGamePlatform smoke tests already cover the refactored surfaces — run them after each extraction step.
 
 ## Gotchas to respect
 
@@ -122,7 +127,7 @@ Three extraction refactors come first — they remove exactly the duplication **
 
 - `dotnet build`; `dotnet test` (needs local MongoDB + `Local.runsettings`; Playwright self-skips without `E2E_ENABLED`).
   E2E: `E2E_ENABLED=true dotnet test test/BlazorApp.PlaywrightTests/BlazorApp.PlaywrightTests.csproj` per the e2e-local-run-recipe memory / `CONTRIBUTING.md`.
-  Mobile review: `E2E_SCREENSHOTS=true` + `E2E_SHOTS_DIR` screenshot run, plus a manual pass of <https://localhost:5021/add> at 390px width.
+  Mobile review: `E2E_MOBILE_CHECK=true` + `E2E_MOBILE_DIR` screenshot run, plus a manual pass of <https://localhost:5021/add> at 390px width.
   Manual regression: the three detail pages' add/edit modals and the game platform cards must behave identically after the extractions (same fields, date/time proxies, balance preview).
 
 ## Ready-to-use implementation prompt