Skip to content

PM-5392: Route AI Engineering ratings to develop#134

Draft
jmgasper wants to merge 1 commit into
developfrom
PM-5392
Draft

PM-5392: Route AI Engineering ratings to develop#134
jmgasper wants to merge 1 commit into
developfrom
PM-5392

Conversation

@jmgasper

Copy link
Copy Markdown
Contributor

What was broken
AI-tagged development challenges were configured to use the Data Science stats track, so member profile stats exposed AI Engineering ratings under DATA_SCIENCE.

Root cause
The default AI Engineering rating path in member-api-v6 was seeded with track DATA_SCIENCE even though the tagged challenges are development challenges.

What was changed
Changed the default AI Engineering rating path destination to DEVELOP so rerates and profile stats place those ratings under the development track.

Any added/updated tests
Updated stats response and rerate summary unit tests to assert AI Engineering rating path rows are exposed under DEVELOP.

Validation run:

  • pnpm lint passed.
  • pnpm exec mocha -t 20000 test/unit/StatsDimensionHelper.test.js test/unit/StatisticsService.test.js test/unit/MmRatingEngine.test.js --exit passed with 70 passing tests.
  • pnpm test was attempted; it reached unrelated legacy DB-backed tests and failed on local fixture/environment issues outside PM-5392, including missing standardized skills tables, bus/Auth0 test configuration, and an existing communication secure-fields assertion.
  • pnpm build was attempted; this package has no build script, so pnpm reported Command "build" not found.

What was broken
AI-tagged development challenges were configured to use the Data Science stats track, so member profile stats exposed AI Engineering ratings under DATA_SCIENCE.

Root cause
The default AI Engineering rating path in member-api-v6 was seeded with track DATA_SCIENCE even though the tagged challenges are development challenges.

What was changed
Changed the default AI Engineering rating path destination to DEVELOP so rerates and profile stats place those ratings under the development track.

Any added/updated tests
Updated stats response and rerate summary unit tests to assert AI Engineering rating path rows are exposed under DEVELOP.
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