From 36f27b32620cc207906de68a9affe7d5d32032d4 Mon Sep 17 00:00:00 2001
From: Hentry Martin <hentrymartin.p@gmail.com>
Date: Fri, 12 Dec 2025 21:20:29 +0100
Subject: [PATCH 1/2] fix: dont show edit manager comment to reviewer and
 filter reviews which are failed screening

---
 .../ChallengeDetailsContent.tsx                      |  1 +
 .../ChallengeDetailsContent/TabContentReview.tsx     |  6 ++++++
 .../ReviewManagerComment/ReviewManagerComment.tsx    |  2 +-
 .../src/lib/components/TableReview/TableReview.tsx   | 12 ++++++++++--
 4 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/src/apps/review/src/lib/components/ChallengeDetailsContent/ChallengeDetailsContent.tsx b/src/apps/review/src/lib/components/ChallengeDetailsContent/ChallengeDetailsContent.tsx
index 69033a337..baaa79adf 100644
--- a/src/apps/review/src/lib/components/ChallengeDetailsContent/ChallengeDetailsContent.tsx
+++ b/src/apps/review/src/lib/components/ChallengeDetailsContent/ChallengeDetailsContent.tsx
@@ -508,6 +508,7 @@ export const ChallengeDetailsContent: FC<Props> = (props: Props) => {
                 selectedTab={props.selectedTab}
                 reviews={reviewTabReviews}
                 submitterReviews={reviewTabSubmitterReviews}
+                screeningOutcome={screeningOutcome}
                 reviewMinimumPassingScore={props.reviewMinimumPassingScore}
                 isLoadingReview={props.isLoadingSubmission}
                 isDownloading={isDownloadingSubmission}
diff --git a/src/apps/review/src/lib/components/ChallengeDetailsContent/TabContentReview.tsx b/src/apps/review/src/lib/components/ChallengeDetailsContent/TabContentReview.tsx
index 8efcb4805..25721f38b 100644
--- a/src/apps/review/src/lib/components/ChallengeDetailsContent/TabContentReview.tsx
+++ b/src/apps/review/src/lib/components/ChallengeDetailsContent/TabContentReview.tsx
@@ -46,6 +46,10 @@ import { hasSubmitterPassedThreshold } from '../../utils/reviewScoring'
 interface Props {
     aiReviewers?: { aiWorkflowId: string }[]
     selectedTab: string
+    screeningOutcome: {
+        failingSubmissionIds: Set<string>;
+        passingSubmissionIds: Set<string>;
+    }
     reviews: SubmissionInfo[]
     submitterReviews: SubmissionInfo[]
     reviewMinimumPassingScore?: number | null
@@ -636,6 +640,7 @@ export const TabContentReview: FC<Props> = (props: Props) => {
         },
         [resolvedReviewsWithSubmitter],
     )
+
     const filteredSubmitterReviews = useMemo(
         () => {
             if (!resolvedSubmitterReviews.length) {
@@ -729,6 +734,7 @@ export const TabContentReview: FC<Props> = (props: Props) => {
         <TableReview
             aiReviewers={props.aiReviewers}
             datas={reviewRows}
+            screeningOutcome={props.screeningOutcome}
             isDownloading={props.isDownloading}
             downloadSubmission={props.downloadSubmission}
             mappingReviewAppeal={props.mappingReviewAppeal}
diff --git a/src/apps/review/src/lib/components/Scorecard/ScorecardViewer/ScorecardQuestion/ReviewResponse/ReviewManagerComment/ReviewManagerComment.tsx b/src/apps/review/src/lib/components/Scorecard/ScorecardViewer/ScorecardQuestion/ReviewResponse/ReviewManagerComment/ReviewManagerComment.tsx
index 82f26a1a8..5a63d077d 100644
--- a/src/apps/review/src/lib/components/Scorecard/ScorecardViewer/ScorecardQuestion/ReviewResponse/ReviewManagerComment/ReviewManagerComment.tsx
+++ b/src/apps/review/src/lib/components/Scorecard/ScorecardViewer/ScorecardQuestion/ReviewResponse/ReviewManagerComment/ReviewManagerComment.tsx
@@ -119,7 +119,7 @@ const ReviewManagerComment: FC<ReviewManagerCommentProps> = props => {
                     <div className={styles.content}>
                         <MarkdownReview value={comment} />
                     </div>
-                    {isManagerEdit && (
+                    {isManagerEdit && canAddManagerComment && (
                         <button
                             type='button'
                             onClick={handleShowCommentForm}
diff --git a/src/apps/review/src/lib/components/TableReview/TableReview.tsx b/src/apps/review/src/lib/components/TableReview/TableReview.tsx
index eb07bb1bb..dea1c6647 100644
--- a/src/apps/review/src/lib/components/TableReview/TableReview.tsx
+++ b/src/apps/review/src/lib/components/TableReview/TableReview.tsx
@@ -72,6 +72,10 @@ export interface TableReviewProps {
     aiReviewers?: { aiWorkflowId: string }[]
     className?: string
     datas: SubmissionInfo[]
+    screeningOutcome: {
+        failingSubmissionIds: Set<string>;
+        passingSubmissionIds: Set<string>;
+    }
     isDownloading: IsRemovingType
     downloadSubmission: (submissionId: string) => void
     mappingReviewAppeal: MappingReviewAppeal
@@ -230,9 +234,13 @@ export const TableReview: FC<TableReviewProps> = (props: TableReviewProps) => {
         [myReviewerResourceIds],
     )
 
+    const filterScreeningPassedReviews = useCallback((submissions: SubmissionInfo[]): SubmissionInfo[] => {
+        return submissions.filter(submission => props.screeningOutcome.passingSubmissionIds.has(submission.id ?? ''))
+    }, [props.screeningOutcome.passingSubmissionIds])
+
     const submissionsForAggregation = useMemo<SubmissionInfo[]>(
-        () => (restrictToLatest ? latestSubmissions : reviewPhaseDatas),
-        [latestSubmissions, restrictToLatest, reviewPhaseDatas],
+        () => (restrictToLatest ? filterScreeningPassedReviews(latestSubmissions) : filterScreeningPassedReviews(reviewPhaseDatas)),
+        [latestSubmissions, restrictToLatest, reviewPhaseDatas, filterScreeningPassedReviews],
     )
 
     const aggregatedSubmissionRows = useMemo<AggregatedSubmissionReviews[]>(() => (

From ff680e7d704b48bf1322609d0f8356c71db5c9fa Mon Sep 17 00:00:00 2001
From: Hentry Martin <hentrymartin.p@gmail.com>
Date: Mon, 15 Dec 2025 12:40:20 +0100
Subject: [PATCH 2/2] fix: lint

---
 .../lib/components/TableReview/TableReview.tsx    | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/src/apps/review/src/lib/components/TableReview/TableReview.tsx b/src/apps/review/src/lib/components/TableReview/TableReview.tsx
index dea1c6647..2635577bb 100644
--- a/src/apps/review/src/lib/components/TableReview/TableReview.tsx
+++ b/src/apps/review/src/lib/components/TableReview/TableReview.tsx
@@ -234,12 +234,19 @@ export const TableReview: FC<TableReviewProps> = (props: TableReviewProps) => {
         [myReviewerResourceIds],
     )
 
-    const filterScreeningPassedReviews = useCallback((submissions: SubmissionInfo[]): SubmissionInfo[] => {
-        return submissions.filter(submission => props.screeningOutcome.passingSubmissionIds.has(submission.id ?? ''))
-    }, [props.screeningOutcome.passingSubmissionIds])
+    const filterScreeningPassedReviews = useCallback(
+        (submissions: SubmissionInfo[]): SubmissionInfo[] => submissions.filter(
+            submission => props.screeningOutcome.passingSubmissionIds.has(submission.id ?? ''),
+        ),
+        [props.screeningOutcome.passingSubmissionIds],
+    )
 
     const submissionsForAggregation = useMemo<SubmissionInfo[]>(
-        () => (restrictToLatest ? filterScreeningPassedReviews(latestSubmissions) : filterScreeningPassedReviews(reviewPhaseDatas)),
+        () => (
+            restrictToLatest
+                ? filterScreeningPassedReviews(latestSubmissions)
+                : filterScreeningPassedReviews(reviewPhaseDatas)
+        ),
         [latestSubmissions, restrictToLatest, reviewPhaseDatas, filterScreeningPassedReviews],
     )