Skip to content

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

Open
paul-fresquet wants to merge 3 commits intomasterfrom
feature/267-skipped-aggregates
Open

[feature] Add skipped inventory aggregates for issue 267#279
paul-fresquet wants to merge 3 commits intomasterfrom
feature/267-skipped-aggregates

Conversation

@paul-fresquet
Copy link
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

@sonarqubecloud
Copy link

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

1 participant