maraphon-app

alexei.dolgolyov/maraphon-app

Fork 0

Commit Graph

Author	SHA1	Message	Date
alexei.dolgolyov	85bc99cac5	fix(host): wire DB migration init + Plotly CDN + attribute fix on hand-written migration Three fixes surfaced when launching the WPF host for the first time: 1. App.xaml.cs — call MarathonDbContextInitializer.InitializeAsync() between Host.Build() and Host.Start() so EF migrations + WAL pragma are applied BEFORE BackgroundServices race to query the DB. Without this, all pollers crashed on 'no such table: Events'. 2. wwwroot/index.html — added <script src='https://cdn.plot.ly/plotly-2.35.2.min.js'> before blazor.webview.js. Phase 6 reviewer flagged this for Phase 9, but charts are unrenderable without it; better to ship now. 3. Migrations/20260505000000_InitialCreate.cs — added [DbContext] and [Migration('20260505000000_InitialCreate')] attributes. Phase 2's hand-written migration was missing both, so EF saw 'no migrations to apply' even on a fresh DB. With the attributes, the migration runs on first launch and creates all tables (Events, Snapshots, Bets, EventResults, Anomalies, Sports, Leagues). Verified: clean DB → migration applied → all 7 tables created → pollers run with empty results (no data yet — UpcomingEventsPoller fires every 6h by default; first scrape will populate the DB).	2026-05-05 13:55:59 +03:00
alexei.dolgolyov	e4d8476782	WIP(initial-implementation): parallel batch P2/P3/P5 — code complete, unreviewed Snapshot of the parallel batch (Phases 2 + 3 + 5) at session pause. Solution does NOT build cleanly yet — known cross-phase compile issues remain to be resolved before review. See plans/initial-implementation/PLAN.md "Resume Notes" section for the exact tomorrow-morning action list. Phase 2 (Storage): - Repository interfaces in Marathon.Application/Abstractions - DateRange, ExportKind, StorageOptions in Marathon.Application/Storage - EF Core 8 + SQLite (WAL) persistence: 7 entities + configurations + 4 repos - Hand-written InitialCreate migration (dotnet ef blocked by parallel work) - ClosedXML ExcelExporter with exact customer-spec wide columns - PersistenceModule.AddMarathonPersistence DI extension - Round-trip + export tests (cannot run yet — see cross-phase issues) Phase 3 (Scraping): - IOddsScraper, IBetPlacer in Marathon.Application/Abstractions - ScrapingOptions in Marathon.Infrastructure/Configuration - MarathonbetScraper with 4 parsers (Upcoming, Live, EventOdds, Results) - Helpers: ServerTimeProvider, PeriodScopeMapper, OutcomeCodeMapper, MoscowDateParser - UserAgentRotatorHandler + Polly v8 resilience pipeline - ScrapingModule.AddMarathonScraping DI extension - GlobalUsings.cs aliases for EventId / Configuration disambiguation - Parser tests with trimmed HTML fixtures - ScrapeResultsAsync interim no-op (Phase 8 will replace via watch-list polling) Phase 5 (UI shell — killed mid-final-verify, assumed ~95%): - Marathon.UI populated: MainLayout, App.razor, Pages (Home, Settings), Components, Theme (MarathonTheme.cs + Tokens.cs + app.css), Resources (SharedResource.{cs,ru.resx,en.resx}), Services (ISettingsWriter), wwwroot - WPF host: App.xaml(.cs), MainWindow.xaml(.cs), Marathon.Hosts.WpfBlazor.csproj with Microsoft.AspNetCore.Components.WebView.Wpf + MudBlazor + Serilog - appsettings.json + appsettings.Development.json with all sections wired - bUnit tests: MainLayoutTests, LocaleSwitcherTests, ThemeToggleTests, JsonSettingsWriterTests + Support helpers Cross-phase issues to resolve at next session: 1. Phase 2 repository classes are 'internal' — Phase 3's tests can't reference them. Fix: add InternalsVisibleTo to Marathon.Infrastructure.csproj. 2. Phase 5: LocalizationOptions namespace ambiguity (AspNetCore vs Extensions). 3. Phase 5: WpfBlazor Serilog API mismatch. Reviewer has NOT run on this batch. Move to Phase 4 only after build is green and a combined parallel-batch reviewer passes.	2026-05-05 01:56:53 +03:00

Author

SHA1

Message

Date

alexei.dolgolyov

85bc99cac5

fix(host): wire DB migration init + Plotly CDN + attribute fix on hand-written migration

Three fixes surfaced when launching the WPF host for the first time:

1. App.xaml.cs — call MarathonDbContextInitializer.InitializeAsync()
   between Host.Build() and Host.Start() so EF migrations + WAL pragma
   are applied BEFORE BackgroundServices race to query the DB. Without
   this, all pollers crashed on 'no such table: Events'.

2. wwwroot/index.html — added <script src='https://cdn.plot.ly/plotly-2.35.2.min.js'>
   before blazor.webview.js. Phase 6 reviewer flagged this for Phase 9,
   but charts are unrenderable without it; better to ship now.

3. Migrations/20260505000000_InitialCreate.cs — added [DbContext] and
   [Migration('20260505000000_InitialCreate')] attributes. Phase 2's
   hand-written migration was missing both, so EF saw 'no migrations to
   apply' even on a fresh DB. With the attributes, the migration runs
   on first launch and creates all tables (Events, Snapshots, Bets,
   EventResults, Anomalies, Sports, Leagues).

Verified: clean DB → migration applied → all 7 tables created → pollers
run with empty results (no data yet — UpcomingEventsPoller fires every 6h
by default; first scrape will populate the DB).

2026-05-05 13:55:59 +03:00

alexei.dolgolyov

e4d8476782

WIP(initial-implementation): parallel batch P2/P3/P5 — code complete, unreviewed

Snapshot of the parallel batch (Phases 2 + 3 + 5) at session pause. Solution does
NOT build cleanly yet — known cross-phase compile issues remain to be resolved
before review. See plans/initial-implementation/PLAN.md "Resume Notes" section
for the exact tomorrow-morning action list.

Phase 2 (Storage):
- Repository interfaces in Marathon.Application/Abstractions
- DateRange, ExportKind, StorageOptions in Marathon.Application/Storage
- EF Core 8 + SQLite (WAL) persistence: 7 entities + configurations + 4 repos
- Hand-written InitialCreate migration (dotnet ef blocked by parallel work)
- ClosedXML ExcelExporter with exact customer-spec wide columns
- PersistenceModule.AddMarathonPersistence DI extension
- Round-trip + export tests (cannot run yet — see cross-phase issues)

Phase 3 (Scraping):
- IOddsScraper, IBetPlacer in Marathon.Application/Abstractions
- ScrapingOptions in Marathon.Infrastructure/Configuration
- MarathonbetScraper with 4 parsers (Upcoming, Live, EventOdds, Results)
- Helpers: ServerTimeProvider, PeriodScopeMapper, OutcomeCodeMapper, MoscowDateParser
- UserAgentRotatorHandler + Polly v8 resilience pipeline
- ScrapingModule.AddMarathonScraping DI extension
- GlobalUsings.cs aliases for EventId / Configuration disambiguation
- Parser tests with trimmed HTML fixtures
- ScrapeResultsAsync interim no-op (Phase 8 will replace via watch-list polling)

Phase 5 (UI shell — killed mid-final-verify, assumed ~95%):
- Marathon.UI populated: MainLayout, App.razor, Pages (Home, Settings),
  Components, Theme (MarathonTheme.cs + Tokens.cs + app.css), Resources
  (SharedResource.{cs,ru.resx,en.resx}), Services (ISettingsWriter), wwwroot
- WPF host: App.xaml(.cs), MainWindow.xaml(.cs), Marathon.Hosts.WpfBlazor.csproj
  with Microsoft.AspNetCore.Components.WebView.Wpf + MudBlazor + Serilog
- appsettings.json + appsettings.Development.json with all sections wired
- bUnit tests: MainLayoutTests, LocaleSwitcherTests, ThemeToggleTests,
  JsonSettingsWriterTests + Support helpers

Cross-phase issues to resolve at next session:
1. Phase 2 repository classes are 'internal' — Phase 3's tests can't reference
   them. Fix: add InternalsVisibleTo to Marathon.Infrastructure.csproj.
2. Phase 5: LocalizationOptions namespace ambiguity (AspNetCore vs Extensions).
3. Phase 5: WpfBlazor Serilog API mismatch.

Reviewer has NOT run on this batch. Move to Phase 4 only after build is green
and a combined parallel-batch reviewer passes.

2026-05-05 01:56:53 +03:00

2 Commits