feat: rebuild review flow as split workspace with server-side scoring

- two-pane workspace: code viewer left, review panel right
- line-anchored findings with inline cards and jump-to-line
- simplified form: merge decision, conclusion, findings, tests
- auto-save drafts to localStorage
- move scoring to POST /api/review/[slug]/submit; reveal and
  scoringHints no longer shipped to the client before submit

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: study8677

src/app/api/review/[slug]/submit/route.ts

@@ -0,0 +1,86 @@
+import { NextResponse } from "next/server";
+
+import { getReviewChallenge } from "@/lib/challenges/review";
+import { scoreReviewDraft, type ReviewDraft, type ReviewLineFinding } from "@/lib/challenges/review-submission";
+
+type ReviewSubmitRouteProps = {
+  params: Promise<{
+    slug: string;
+  }>;
+};
+
+function asString(value: unknown) {
+  return typeof value === "string" ? value : "";
+}
+
+function asLineFinding(value: unknown): ReviewLineFinding | null {
+  if (!value || typeof value !== "object") {
+    return null;
+  }
+
+  const finding = value as Record<string, unknown>;
+  const lineNumber = typeof finding.lineNumber === "number" ? finding.lineNumber : Number(finding.lineNumber);
+
+  if (!Number.isFinite(lineNumber) || lineNumber < 1) {
+    return null;
+  }
+
+  return {
+    id: asString(finding.id),
+    fileName: asString(finding.fileName),
+    lineNumber,
+    severity: asString(finding.severity),
+    problem: asString(finding.problem),
+    fix: asString(finding.fix)
+  };
+}
+
+function normalizeDraft(value: unknown): ReviewDraft {
+  const draft = value && typeof value === "object" ? value as Record<string, unknown> : {};
+  const mergeDecision = asString(draft.mergeDecision);
+  const findings = Array.isArray(draft.findings)
+    ? draft.findings.map(asLineFinding).filter((finding): finding is ReviewLineFinding => Boolean(finding))
+    : [];
+  const legacyFinding = [asString(draft.problem), asString(draft.impact), asString(draft.fix)].filter(Boolean).join("\n");
+
+  return {
+    mergeDecision: mergeDecision === "yes" || mergeDecision === "no" || mergeDecision === "unknown" ? mergeDecision : "",
+    conclusion: asString(draft.conclusion),
+    findings: findings.length > 0 || !legacyFinding
+      ? findings
+      : [{
+          id: "legacy-finding",
+          fileName: "",
+          lineNumber: 1,
+          severity: asString(draft.severity) || "high",
+          problem: legacyFinding,
+          fix: asString(draft.fix)
+        }],
+    tests: asString(draft.tests)
+  };
+}
+
+export async function POST(request: Request, { params }: ReviewSubmitRouteProps) {
+  const { slug } = await params;
+  const challenge = getReviewChallenge(slug);
+
+  if (!challenge) {
+    return NextResponse.json({ error: "Review challenge not found." }, { status: 404 });
+  }
+
+  let body: unknown;
+
+  try {
+    body = await request.json();
+  } catch {
+    return NextResponse.json({ error: "Invalid JSON body." }, { status: 400 });
+  }
+
+  const draft = normalizeDraft(body && typeof body === "object" ? (body as Record<string, unknown>).draft : null);
+  const feedback = scoreReviewDraft(draft, challenge.metadata.scoringHints, challenge.reveal);
+
+  return NextResponse.json({
+    feedback,
+    reveal: challenge.reveal
+  });
+}

src/app/challenges/review/[slug]/page.tsx

@@ -2,10 +2,8 @@ import type { Metadata } from "next";
 import Link from "next/link";
 import { notFound } from "next/navigation";
 
-import { ReviewFileBrowser } from "@/components/challenges/ReviewFileBrowser";
-import { ReviewSubmissionForm } from "@/components/challenges/ReviewSubmissionForm";
+import { ReviewWorkspace } from "@/components/challenges/ReviewWorkspace";
 import { AppShell } from "@/components/layout/AppShell";
-import { Badge } from "@/components/ui/Badge";
 import {
   getReviewBackground,
   getReviewChallenge,
@@ -45,8 +43,7 @@ export default async function ReviewChallengePage({ params }: ReviewChallengePag
     notFound();
   }
 
-  const { metadata, files, reveal } = challenge;
-  const displayId = metadata.order.toString().padStart(3, "0");
+  const { metadata, files } = challenge;
   const pr = getReviewPrBrief(metadata);
   const background = getReviewBackground(metadata);
 
@@ -60,12 +57,11 @@ export default async function ReviewChallengePage({ params }: ReviewChallengePag
           </Link>
           <nav className="nav" aria-label="Challenge navigation">
             <Link href="/">题库</Link>
-            <a className="active" href="#review">
-              Review {displayId}
+            <a className="active" href="#files">
+              Review {metadata.order.toString().padStart(3, "0")}
             </a>
             <a href="#files">题目文件</a>
             <a href="#submit">提交 Review</a>
-            <a href="#rubric">评分</a>
           </nav>
           <div className="actions">
             <Link className="button button-outline" href="/">
@@ -75,110 +71,20 @@ export default async function ReviewChallengePage({ params }: ReviewChallengePag
         </div>
       </header>
 
-      <main className="page challenge-page">
-        <div className="notice">
-          <span>
-            <strong>真实来源改编</strong> {metadata.source.project} 案例，改编为 AI PR 审核题。上游来源在提交 review 后展示。
-          </span>
-          <span className="mono">Review {displayId}</span>
-        </div>
-
-        <section className="challenge-header card">
-          <div className="challenge-kicker">
-            <span className="id">{displayId}</span>
-            <Badge tone="review">Review</Badge>
-            <Badge tone={metadata.difficulty}>{metadata.difficulty}</Badge>
-            <span className="pill">{metadata.language}</span>
-          </div>
-          <h1>{metadata.title.zh}</h1>
-          <p>{metadata.summary.zh}</p>
-        </section>
-
-        <div className="challenge-layout">
-          <article className="challenge-main">
-            <section className="challenge-section card" id="review">
-              <div className="card-head">
-                <h2>待审核的 PR</h2>
-                <span className="mono">
-                  {pr.author} · CI passed
-                </span>
-              </div>
-              <div className="section-body">
-                <p>
-                  <strong>{pr.title}</strong>
-                </p>
-                {pr.body.map((paragraph) => (
-                  <p key={paragraph}>{paragraph}</p>
-                ))}
-              </div>
-            </section>
-
-            <ReviewFileBrowser files={files} defaultFileName={metadata.reviewTarget.file} />
-
-            {background.length > 0 ? (
-              <section className="challenge-section card">
-                <div className="card-head">
-                  <h2>背景</h2>
-                  <span className="mono">{metadata.source.project}</span>
-                </div>
-                <div className="section-body">
-                  {background.map((item) => (
-                    <p key={item}>{item}</p>
-                  ))}
-                </div>
-              </section>
-            ) : null}
-
-            <ReviewSubmissionForm
-              challengeId={metadata.id}
-              scoringHints={metadata.scoringHints}
-              reveal={reveal}
-            />
-          </article>
-
-          <aside className="challenge-side">
-            <section className="card">
-              <div className="card-head">
-                <h2>快速判断</h2>
-              </div>
-              <div className="section-body compact">
-                <p>语言 / 领域：{metadata.language}。</p>
-                <p>类型：Review Mode，不要求你改代码。</p>
-                <p>你是 reviewer：这个 PR 的结论由你负责，可以合并、不能合并、需要更多信息都可能是正确答案。</p>
-                <p>可以使用任何 AI 工具辅助，但评分看的是你给出的证据和判断。</p>
-              </div>
-            </section>
-
-            <section className="card" id="rubric">
-              <div className="card-head">
-                <h2>评分重点</h2>
-              </div>
-              <div className="rubric-list">
-                <div className="rubric-item">
-                  <span>合并判断</span>
-                  <strong>30%</strong>
-                </div>
-                <div className="rubric-item">
-                  <span>核心风险</span>
-                  <strong>30%</strong>
-                </div>
-                <div className="rubric-item">
-                  <span>边界语义</span>
-                  <strong>15%</strong>
-                </div>
-                <div className="rubric-item">
-                  <span>测试质量</span>
-                  <strong>10%</strong>
-                </div>
-                <div className="rubric-item">
-                  <span>修复建议</span>
-                  <strong>15%</strong>
-                </div>
-              </div>
-            </section>
-          </aside>
-        </div>
-      </main>
+      <ReviewWorkspace
+        background={background}
+        challengeId={metadata.id}
+        challengeSlug={metadata.slug}
+        defaultFileName={metadata.reviewTarget.file}
+        difficulty={metadata.difficulty}
+        files={files}
+        language={metadata.language}
+        order={metadata.order}
+        pr={pr}
+        sourceProject={metadata.source.project}
+        summary={metadata.summary}
+        title={metadata.title}
+      />
     </AppShell>
   );
 }