From 1e007825c7ed4af31eb8e1cfbedc06c0f0a9c156 Mon Sep 17 00:00:00 2001
From: Justin Gasper <jmgasper@gmail.com>
Date: Wed, 5 Nov 2025 08:57:37 +1100
Subject: [PATCH] Fix an issue where we were making bad requests to load
 scorecards for the reviewers section

---
 .local-ssl-proxy.pid                          |  1 -
 .../ChallengeReviewer-Field/index.js          | 55 ++++++++++++++++---
 2 files changed, 47 insertions(+), 9 deletions(-)
 delete mode 100644 .local-ssl-proxy.pid

diff --git a/.local-ssl-proxy.pid b/.local-ssl-proxy.pid
deleted file mode 100644
index 64b146f3..00000000
--- a/.local-ssl-proxy.pid
+++ /dev/null
@@ -1 +0,0 @@
-321029
diff --git a/src/components/ChallengeEditor/ChallengeReviewer-Field/index.js b/src/components/ChallengeEditor/ChallengeReviewer-Field/index.js
index 56b070db..523d3588 100644
--- a/src/components/ChallengeEditor/ChallengeReviewer-Field/index.js
+++ b/src/components/ChallengeEditor/ChallengeReviewer-Field/index.js
@@ -20,6 +20,47 @@ const ResourceToPhaseNameMap = {
   'Checkpoint Screener': 'Checkpoint Screening'
 }
 
+// Keep track filters aligned with the scorecards API regardless of legacy values
+const SCORECARD_TRACK_ALIASES = {
+  DEVELOP: 'DEVELOPMENT',
+  DEV: 'DEVELOPMENT',
+  QA: 'QUALITY_ASSURANCE',
+  QUALITY_ASSURANCE: 'QUALITY_ASSURANCE',
+  QUALITYASSURANCE: 'QUALITY_ASSURANCE',
+  DES: 'DESIGN',
+  DESIGN: 'DESIGN',
+  DS: 'DATA_SCIENCE',
+  DATA_SCIENCE: 'DATA_SCIENCE',
+  DATASCIENCE: 'DATA_SCIENCE'
+}
+
+const normalizeTrackForScorecards = (challenge, metadata) => {
+  const normalize = (value) => {
+    if (!value) return null
+    const normalized = value.toString().trim().toUpperCase().replace(/\s+/g, '_')
+    return SCORECARD_TRACK_ALIASES[normalized] || normalized
+  }
+
+  if (!challenge) return null
+
+  if (challenge.trackId && metadata && Array.isArray(metadata.challengeTracks)) {
+    const trackFromMetadata = metadata.challengeTracks.find(t => t.id === challenge.trackId)
+    if (trackFromMetadata) {
+      return normalize(trackFromMetadata.track || trackFromMetadata.name || trackFromMetadata.abbreviation)
+    }
+  }
+
+  const { track } = challenge
+  if (typeof track === 'string') {
+    return normalize(track)
+  }
+  if (track && typeof track === 'object') {
+    return normalize(track.track || track.name || track.abbreviation)
+  }
+
+  return null
+}
+
 class ChallengeReviewerField extends Component {
   constructor (props) {
     super(props)
@@ -327,19 +368,16 @@ class ChallengeReviewerField extends Component {
   }
 
   loadScorecards () {
-    const { challenge, loadScorecards } = this.props
+    const { challenge, loadScorecards, metadata } = this.props
 
     const filters = {
       status: 'ACTIVE'
     }
 
     // Add challenge track if available
-    if (challenge.track) {
-      if (typeof challenge.track === 'string') {
-        filters.challengeTrack = challenge.track.toUpperCase().replace(' ', '_')
-      } else if (challenge.track.track) {
-        filters.challengeTrack = challenge.track.track
-      }
+    const normalizedTrack = normalizeTrackForScorecards(challenge, metadata)
+    if (normalizedTrack) {
+      filters.challengeTrack = normalizedTrack
     }
 
     // Add challenge type if available
@@ -1013,7 +1051,8 @@ ChallengeReviewerField.propTypes = {
     scorecards: PropTypes.array,
     defaultReviewers: PropTypes.array,
     workflows: PropTypes.array,
-    resourceRoles: PropTypes.array
+    resourceRoles: PropTypes.array,
+    challengeTracks: PropTypes.array
   }),
   isLoading: PropTypes.bool,
   readOnly: PropTypes.bool,