Skip to content

refactor(seeding): boot-seed marker I/O moves into StationSettingsRepository (gh-#406 slice 4) - #511

Merged
genwave-radio merged 1 commit into
refactor/gh-406-s3-settings-repositoryfrom
refactor/gh-406-s4-seed-marker
Aug 14, 2026
Merged

refactor(seeding): boot-seed marker I/O moves into StationSettingsRepository (gh-#406 slice 4)#511
genwave-radio merged 1 commit into
refactor/gh-406-s3-settings-repositoryfrom
refactor/gh-406-s4-seed-marker

Conversation

@genwave-radio

Copy link
Copy Markdown
Collaborator

gh-#406 L2 burn-down, slice 4 of 5 — stacked on slice 3 (merge in order; merge commits, not squash).

⚖️ SafeLoopSeedMarkerStore ran its own NpgsqlConnection SQL against station.settings (F27.10). It now delegates: new StationSettingsRepository.ExistsAsync(key) single-key probe (SQL byte-identical), and MarkCompletedAsync reuses the existing WriteAsync — its upsert SQL was already byte-identical to the marker's, so no duplicate method. using Npgsql gone from Host seeding; the store never had an exception catch, so nothing to translate.

📉 Baseline row deleted (2 debt rows → 1).

✅ Build 0 warnings; Architecture 104/104; Host seed-filtered 37/37; repository db specs 12/12 at this tip (full suites re-run at the slice-5 tip above this).

@genwave-radio
genwave-radio merged commit 3a4f936 into refactor/gh-406-s3-settings-repository Aug 14, 2026
6 checks passed
@genwave-radio
genwave-radio deleted the refactor/gh-406-s4-seed-marker branch August 14, 2026 01:48
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant