HicServices / RDMP / 26280548007 / 1

57%

develop: 57%

push

github

Cohort Builder: partial-refresh AggregateConfiguration renames/moves (#2336)

Adds AggregateConfiguration to the SelectiveRefresh type switch in
CatalogueChildProvider.SelectiveRefresh(IMapsDirectlyToDatabaseTable).
Previously this type fell through to '_ => false', forcing
ActivateItems.RefreshBus_RefreshObject down the full GetChildProvider()
rebuild path (~9-10s at HIC scale) on every aggregate rename, parameter
edit, or other property change.

Implements SelectiveRefresh(AggregateConfiguration ac) following the
established pattern from SelectiveRefresh(CohortAggregateContainer):

  - For cohort builder set members (parent CohortAggregateContainer):
    re-fetch aggregates + container hierarchy, refresh the parent
    container's subtree.

  - For graph aggregates hanging off a Catalogue: re-fetch aggregates,
    refresh the catalogue's subtree.

  - Falls back to false (i.e. full rebuild) if neither parent type is
    in the descendancy.

Pairs with the preceding 'inject joinable knowledge after partial-refresh
rebuild' commit on this branch — that fix is what makes the
BuildAggregateConfigurations() call here cheap rather than triggering
N database round-trips downstream.

Expected impact on HIC platform: aggregate rename freeze drops from
~10s to ~1-2s (matching the now-snappy container rename).

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

13834 of 42797 branches covered (32.32%)

Branch coverage included in aggregate %.

41110 of 108750 relevant lines covered (37.8%)

4631.55 hits per line