Have admin access be priority over resource role on challenge, and fix visibility of checkpoint submissions that fail screening

Author: jmgasper

src/apps/review/src/lib/hooks/useFetchScreeningReview.ts

@@ -1502,6 +1502,25 @@ export function useFetchScreeningReview(): useFetchScreeningReviewProps {
                 reviewEntries: reviewsToRender,
             })
 
+            const hasCheckpointReview = rowsForSubmission.some(row => Boolean(row.reviewId))
+            const submissionStatus = (item.status ?? '')
+                .toString()
+                .trim()
+                .toUpperCase()
+            const failedCheckpointScreening = submissionStatus === 'FAILED_CHECKPOINT_SCREENING'
+
+            if (failedCheckpointScreening && !hasCheckpointReview) {
+                if (debugCheckpointPhases) {
+                    debugLog('checkpointReview.skipSubmission', {
+                        reason: 'failedCheckpointScreeningWithoutReview',
+                        submissionId: item.id,
+                        submissionStatus,
+                    })
+                }
+
+                return rows
+            }
+
             rows.push(...rowsForSubmission)
 
             return rows

src/apps/review/src/lib/hooks/useRole.ts

@@ -69,7 +69,10 @@ const useRole = (): useRoleProps => {
             return ''
         }
 
-        const normalizedRoles = myRoles.map(role => role.toLowerCase())
+        const normalizedRoles = [
+            ...myRoles.map(role => role.toLowerCase()),
+            ...(isTopcoderAdmin ? ['admin'] : []),
+        ]
         const rolePriority: ChallengeRole[] = [
             'Admin',
             'Manager',