Skip to content

Centralize report and evidence period aggregation - #797

Merged
itsDNNS merged 1 commit into
mainfrom
refactor/deterministic-period-aggregation
Aug 16, 2026
Merged

Centralize report and evidence period aggregation#797
itsDNNS merged 1 commit into
mainfrom
refactor/deterministic-period-aggregation

Conversation

@itsDNNS

@itsDNNS itsDNNS commented Aug 16, 2026

Copy link
Copy Markdown
Owner

What changed

  • add one pure, deterministic period aggregation contract for stored DOCSIS snapshots
  • preserve duplicate multiplicity and distinguish numeric samples, nulls, unsupported counters, invalid values, and missing samples
  • retain stored analyzer and threshold provenance while keeping legacy fallback explicit
  • migrate PDF reports, incident reports, complaint text, comparison, and Evidence to the shared aggregate
  • load Evidence snapshots once and keep BQM and Connection Monitor as explicit source adapters
  • document the dependency boundary and add architecture, determinism, provenance, coverage, and compatibility guards

Compatibility

Existing report, complaint, comparison, and Evidence response shapes remain unchanged. A representative mixed SC-QAM and OFDM fixture produced identical complaint text and normalized PDF text before and after the refactor. There are no database, snapshot, authentication, threshold, locale, or UI migrations.

Validation

  • 3818 passed, 2 skipped in the full non-E2E Python suite
  • 531 passed in the complete Playwright E2E suite under UTC
  • 191 passed in the final focused report, Evidence, comparison, aggregation, and browser gate
  • 9 passed in the JavaScript suite
  • all 69 locale files validated
  • workflow YAML, Python compilation, diff checks, and production dependency audit passed
  • deterministic fault proof failed with 8 distinct digests when the same-timestamp content tiebreak was removed, then passed after restoration
  • all three representative PDF pages were rasterized and checked for clipping, overlap, overflow, ordering, and null rendering

@itsDNNS
itsDNNS merged commit 16a2960 into main Aug 16, 2026
19 checks passed
@itsDNNS
itsDNNS deleted the refactor/deterministic-period-aggregation branch August 16, 2026 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant