Skip to content

[feature] Add skipped inventory aggregates for issue 267#279

Merged
paul-fresquet merged 10 commits intomasterfrom
feature/267-skipped-aggregates
Feb 18, 2026
Merged

[feature] Add skipped inventory aggregates for issue 267#279
paul-fresquet merged 10 commits intomasterfrom
feature/267-skipped-aggregates

Conversation

@paul-fresquet
Copy link
Copy Markdown
Contributor

@paul-fresquet paul-fresquet commented Feb 12, 2026

Summary:

  • Add skipped-entry aggregates to inventory models to support reliable cross-machine reporting.
  • Keep runtime skipped tracking while introducing persisted per-part skipped counts.

Key changes:

  • Added persisted skip aggregates in InventoryPart (SkippedCountsByReason, SkippedCount, GetSkippedCountByReason, RecordSkippedEntry).
  • Extended InventoryProcessData with runtime aggregate counters (SkippedCount + per-reason counts) and reset handling.
  • Updated InventoryBuilder.RecordSkippedEntry to write through a single path: InventoryPart aggregate + runtime skipped entry/counters.
  • Added/updated unit tests for InventoryPart aggregates, process data counters, builder propagation, and loader persistence.

Notes/risks:

Closes #267

Loading
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.

[feature] Track skipped entries in InventoryPart

2 participants