Repository navigation
fix(seed): give House 2 realistic demo history - #917
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
After evidence for head 29cc12f. Lay Rate frames show fourteen complete bars; viewport-top frames show House 2 remains unrecorded today. |
29cc12f to
4ee57c2
Compare
Review round — resolved at
|
| # | Finding | Fix |
|---|---|---|
| P2 | DemoDataSeeder.cs: House 2 was placed 20 weeks ago, so ~100 of its new 240 days predated placement and Reports excluded them from hen-day |
Placement moved to 40 weeks ago; new literal assertion in DemoSeedTests that no House 2 entry predates placement (red first: Expected: 0 / Actual: 100, then green) |
| P3 | tools/verify/features/dashboard.md and tools/simulation/ui/README.md still described seven hatched demo days / one house with 240 days |
Both sentences updated |
Coordinator finding during capture review (fixed before the PR opened): with 240 days the seeded mortality shrinks House 2 to ~320 birds while its egg totals stayed at 350–372, pushing the Dashboard hen-day figure to 108.2%; baseline lowered to 275 (now 98.1%) and the test pins every seeded total at or below the final bird count.
Verification at 4ee57c2 (autosquashed; tree identical to the pre-rebase tip): Application 532/532, Integration 1,851/1,851, ImagePin 2/2, dotnet build Cluckwork.sln 0 warnings. Frames captured at 8152610; the later commits change placement date, comments and docs only, and the visible 14-day strip and 7-day hen-day windows are unaffected. Seed duration ~10 s → ~23 s.
Owner decisions carried in the body: demo stock rises 107,583 → 173,458 eggs; the README dashboard image may be recaptured after merge. Product defects this round: 1 (placement date). No further Codex trigger unless CI raises something.
|
Addressed both Astra findings at head
No recapture was needed: this changes placement outside the displayed window. The visible 14-day strip and both 7-day hen-day windows are unchanged. |
|
CI at |












Summary
Closes #886
TDD evidence
The focused regression test failed before the seeder change:
After the fix, the focused test passed 1/1.
Runtime evidence
Before: PR #912's readme-farm Lay Rate frame at 406cc0b shows seven partial days in the fortnight strip. I used that existing frame because it captures the same demo seeder before this change.
After: frames are attached in a PR comment and name head
29cc12f10995a8a77c430361402f8366d056c0ef. The Lay Rate capture asserts 14 recorded days and no partial or unrecorded slots; the viewport-top frames show House 2 still unrecorded today. Frames were captured at8152610;29cc12fchanges only a seeder comment and a test variable name, so rendered bytes are unchanged.Demo seeding took about 10 seconds before and 23 seconds after on this host. The full final isolated reset took 83.68 seconds.
Available stock rises from 107,583 eggs in the before frame to 173,458 after because House 2 contributes 233 additional production days. This PR adds no stock-consuming facts; accepting that larger demo inventory is an owner decision.
docs/images/dashboard.pngis intentionally unchanged. It can be recaptured on the demo farm after merge as an owner decision.Verification
Cluckwork.Application.Tests: 532 passedCluckwork.Api.IntegrationTests: 1,851 passeddotnet build Cluckwork.sln: succeeded with 0 warnings and 0 errors