From b13e758db15183cec0175fb8488a740d8688d01b Mon Sep 17 00:00:00 2001
From: wren <porlong@qq.com>
Date: Fri, 20 Mar 2026 19:08:33 +0800
Subject: [PATCH] fix(reviews): map failed_count to warning, always show
 not-applicable badge
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- failed_count should be 'warning' not 'error' for scored eval
- Show '未涉及' badge always, not only when count > 0

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 app/components/reviews/ReviewPointsList.tsx | 20 ++++++++------------
 app/routes/reviews.tsx                      |  4 ++--
 2 files changed, 10 insertions(+), 14 deletions(-)

diff --git a/app/components/reviews/ReviewPointsList.tsx b/app/components/reviews/ReviewPointsList.tsx
index def63d9..7871ea4 100644
--- a/app/components/reviews/ReviewPointsList.tsx
+++ b/app/components/reviews/ReviewPointsList.tsx
@@ -746,18 +746,14 @@ export function ReviewPointsList({
               <span className="text-xs text-gray-500 ml-2">错误</span>
             </button>
           </div>
-          {/* 未涉及数量（仅在有未涉及评查点时显示，仅统计展示不支持过滤） */}
-          {notApplicableToShow > 0 && (
-            <>
-              <div className="h-8 border-r border-gray-200"></div>
-              <div className="flex items-center">
-                <div className="px-3 h-7 bg-blue-50 rounded-md flex items-center justify-center">
-                  <span className="text-sm font-semibold text-blue-500">{notApplicableToShow}</span>
-                  <span className="text-xs text-gray-500 ml-2">未涉及</span>
-                </div>
-              </div>
-            </>
-          )}
+          {/* 未涉及数量 */}
+          <div className="h-8 border-r border-gray-200"></div>
+          <div className="flex items-center">
+            <div className="px-3 h-7 bg-blue-50 rounded-md flex items-center justify-center">
+              <span className="text-sm font-semibold text-blue-500">{notApplicableToShow}</span>
+              <span className="text-xs text-gray-500 ml-2">未涉及</span>
+            </div>
+          </div>
         </div>
       </div>
     );
diff --git a/app/routes/reviews.tsx b/app/routes/reviews.tsx
index 8f1ffc0..1e50eda 100644
--- a/app/routes/reviews.tsx
+++ b/app/routes/reviews.tsx
@@ -238,8 +238,8 @@ export async function loader({ request }: LoaderFunctionArgs) {
         statistics: {
           total: unifiedData.summary?.total_points || 0,
           success: unifiedData.summary?.passed_count || 0,
-          error: unifiedData.summary?.failed_count || 0,
-          warning: 0,
+          warning: unifiedData.summary?.failed_count || 0,
+          error: 0,
           notApplicable: unifiedData.summary?.not_applicable_count || 0,
           score: unifiedData.summary?.total_score || 0
         },