Memory repository drops excludeFromWholeHome on circuit assignments #35

New issue

Open

opened 2026-07-17 03:07:57 +00:00 by claude-bot · 0 comments

claude-bot commented

2026-07-17 03:07:57 +00:00

Contributor

repository.memory.js:454-464 normalizes circuitAssignments without excludeFromWholeHome, while migration 003_circuit_whole_home_exclusion.sql and repository.pg.js:732-757 persist it. Since the whole test suite runs on the memory repo, the field's round-trip and its downstream effect on outage analysis are untested, and the memory backend silently loses user data written through it. Add the field to the memory normalizer and a test asserting parity of the normalized shape between both repositories for circuit assignments.

Acceptance criteria

Field round-trips through the memory repo
Shape-parity test between memory and pg normalization exists

Relations: relates to #7

(Audit finding F-14, 2026-07-16 — Severity: Medium, Confidence: High)

`repository.memory.js:454-464` normalizes circuitAssignments without `excludeFromWholeHome`, while migration `003_circuit_whole_home_exclusion.sql` and `repository.pg.js:732-757` persist it. Since the whole test suite runs on the memory repo, the field's round-trip and its downstream effect on outage analysis are untested, and the memory backend silently loses user data written through it. Add the field to the memory normalizer and a test asserting parity of the normalized shape between both repositories for circuit assignments. **Acceptance criteria** - [ ] Field round-trips through the memory repo - [ ] Shape-parity test between memory and pg normalization exists **Relations:** relates to #7 *(Audit finding F-14, 2026-07-16 — Severity: Medium, Confidence: High)*

claude-bot added this to the v1.1.0 - Test & Ops Hardening milestone

2026-07-17 03:10:51 +00:00

claude-bot added the

bug

testing

labels

2026-07-17 03:11:50 +00:00