Skip to content

PM-5437: Deduplicate development profile stats#1988

Merged
jmgasper merged 1 commit into
devfrom
PM-5437
Jun 25, 2026
Merged

PM-5437: Deduplicate development profile stats#1988
jmgasper merged 1 commit into
devfrom
PM-5437

Conversation

@jmgasper

Copy link
Copy Markdown
Collaborator

What was broken
Development totals on profiles added the AI Engineering and Challenge subtrack counters together, so the same Dev-track AI challenges could be counted twice.

Root cause
AI Engineering is displayed under Development from the DATA_SCIENCE rating path, while overlapping Dev Challenge history remains visible as a Development subtrack. Parent totals were simple subtrack sums and ignored shared challenge history.

What was changed
Added history-aware parent summary calculation for Development, keyed by challenge history identity, and used it in active track aggregation and the Development drilldown. Non-Development drilldowns keep the existing summing behavior.

Any added/updated tests
Added a profiles hook regression test covering AI Engineering and Challenge histories sharing the same Dev AI challenges.

Validation note: focused profiles tests, lint, and build pass. The full non-watch test suite currently fails in unrelated src/apps/work suites on the dev baseline.

What was broken
Development totals on profiles added the AI Engineering and Challenge subtrack counters together, so the same Dev-track AI challenges could be counted twice.

Root cause
AI Engineering is displayed under Development from the DATA_SCIENCE rating path, while overlapping Dev Challenge history remains visible as a Development subtrack. Parent totals were simple subtrack sums and ignored shared challenge history.

What was changed
Added history-aware parent summary calculation for Development, keyed by challenge history identity, and used it in active track aggregation and the Development drilldown. Non-Development drilldowns keep the existing summing behavior.

Any added/updated tests
Added a profiles hook regression test covering AI Engineering and Challenge histories sharing the same Dev AI challenges.
@jmgasper jmgasper requested a review from kkartunov as a code owner June 24, 2026 23:24
@jmgasper jmgasper merged commit dc2a57e into dev Jun 25, 2026
7 checks passed
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