admin review: show + edit submission hours before approving

The review queue had no way to see or change a record's tracked hours, so
an over-counted Hackatime figure couldn't be deflated before approval.

- page.tsx: fetch Optional - Override Hours Spent + Hackatime project name
  into each queue row
- AdminQueue: per-row Hours number input with the stored value shown for
  reference and a "Save hours" action
- /api/admin/review: new "hours" action that rewrites Override Hours Spent
  (rounded to 0.1) without touching Approved / Review Status

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01X3z1tR9YabNCw8146cPzT7

Author: plastuchino

app/admin/page.tsx

@@ -12,6 +12,8 @@ const TELESCREEN_BASE = "https://joe-cool.jollyy.dev/billy/overview";
 // leak into this page's payload even by accident.
 const QUEUE_FIELDS = [
   SUBMISSION_FIELDS.hackatimeId,
+  SUBMISSION_FIELDS.hackatimeProjects,
+  SUBMISSION_FIELDS.overrideHours,
   SUBMISSION_FIELDS.codeUrl,
   SUBMISSION_FIELDS.playableUrl,
   SUBMISSION_FIELDS.lapseLinks,
@@ -51,12 +53,15 @@ export default async function AdminPage({
     const screenshot = record.fields[SUBMISSION_FIELDS.screenshot] as
       | Array<{ url: string }>
       | undefined;
+    const hoursRaw = record.fields[SUBMISSION_FIELDS.overrideHours];
     return {
       id: record.id,
       telescreenLink: `${TELESCREEN_BASE}?u=${encodeURIComponent(hackatimeId)}`,
       codeUrl: String(record.fields[SUBMISSION_FIELDS.codeUrl] ?? ""),
       playableUrl: String(record.fields[SUBMISSION_FIELDS.playableUrl] ?? ""),
       lapseLinks: String(record.fields[SUBMISSION_FIELDS.lapseLinks] ?? ""),
+      hackatimeProjects: String(record.fields[SUBMISSION_FIELDS.hackatimeProjects] ?? ""),
+      hours: typeof hoursRaw === "number" ? hoursRaw : 0,
       screenshotUrl: screenshot?.[0]?.url ?? null,
       approved: Boolean(record.fields[SUBMISSION_FIELDS.approved]),
       reviewStatus: String(record.fields[SUBMISSION_FIELDS.reviewStatus] ?? "Pending"),

app/api/admin/review/route.ts

@@ -10,7 +10,7 @@ import {
 } from "../../../../src/lib/airtable";
 import { getIdentity } from "../../../../src/lib/hackclub";
 
-const ACTIONS = ["approve", "reject", "fraud"] as const;
+const ACTIONS = ["approve", "reject", "fraud", "hours"] as const;
 type Action = (typeof ACTIONS)[number];
 
 export async function POST(request: Request) {
@@ -36,6 +36,23 @@ export async function POST(request: Request) {
   }
 
   const reviewedAt = new Date().toISOString();
+
+  // "hours" is an adjustment action, not a verdict — it only rewrites the
+  // record's hours (letting a reviewer deflate an over-counted Hackatime
+  // figure before approving) and leaves Approved / Review Status untouched.
+  if (action === "hours") {
+    const hours = Number(body.hours);
+    if (!Number.isFinite(hours) || hours < 0) {
+      return NextResponse.json({ error: "invalid_hours" }, { status: 400 });
+    }
+    await updateAirtableRecord(recordId, {
+      [SUBMISSION_FIELDS.overrideHours]: Math.round(hours * 10) / 10,
+      [SUBMISSION_FIELDS.reviewedAt]: reviewedAt,
+      [SUBMISSION_FIELDS.reviewedBy]: identity.primary_email,
+    });
+    return NextResponse.json({ ok: true });
+  }
+
   const reviewFields: Record<string, unknown> = {
     [SUBMISSION_FIELDS.reviewedAt]: reviewedAt,
     [SUBMISSION_FIELDS.reviewedBy]: identity.primary_email,

app/components/admin/AdminQueue.tsx

@@ -9,6 +9,8 @@ export type AdminSubmissionRow = {
   codeUrl: string;
   playableUrl: string;
   lapseLinks: string;
+  hackatimeProjects: string;
+  hours: number;
   screenshotUrl: string | null;
   approved: boolean;
   reviewStatus: string;
@@ -26,9 +28,14 @@ export default function AdminQueue({
   filter: Filter;
 }) {
   const [rejectDraft, setRejectDraft] = useState<Record<string, string>>({});
+  const [hoursDraft, setHoursDraft] = useState<Record<string, string>>({});
   const [busy, setBusy] = useState<string | null>(null);
 
-  async function act(recordId: string, action: "approve" | "reject" | "fraud") {
+  async function act(
+    recordId: string,
+    action: "approve" | "reject" | "fraud" | "hours",
+    extra?: Record<string, unknown>,
+  ) {
     const message = action === "reject" ? rejectDraft[recordId]?.trim() : undefined;
     if (action === "reject" && !message) return;
 
@@ -37,7 +44,7 @@ export default function AdminQueue({
       const res = await fetch("/api/admin/review", {
         method: "POST",
         headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ recordId, action, message }),
+        body: JSON.stringify({ recordId, action, message, ...extra }),
       });
       if (res.ok) {
         window.location.reload();
@@ -59,7 +66,14 @@ export default function AdminQueue({
 
       {rows.length === 0 && <p className="opacity-60">No submissions in this view.</p>}
 
-      {rows.map((row) => (
+      {rows.map((row) => {
+        const hoursValue = hoursDraft[row.id] ?? String(row.hours);
+        const parsedHours = Number(hoursValue);
+        const hoursValid = Number.isFinite(parsedHours) && parsedHours >= 0;
+        const hoursChanged =
+          hoursValid && Math.round(parsedHours * 10) / 10 !== Math.round(row.hours * 10) / 10;
+
+        return (
         <div key={row.id} className="card bg-base-200 p-4 gap-3">
           <div className="flex gap-4 items-start flex-wrap">
             {row.screenshotUrl && (
@@ -77,12 +91,35 @@ export default function AdminQueue({
                 Playable URL
               </a>
               {row.lapseLinks && <p>Lapse: {row.lapseLinks}</p>}
+              {row.hackatimeProjects && <p>Project: {row.hackatimeProjects}</p>}
               <p className="opacity-60">
                 {row.approved ? "Approved" : row.reviewStatus}
               </p>
             </div>
           </div>
 
+          <div className="flex gap-2 flex-wrap items-center">
+            <label className="text-sm opacity-70">Hours</label>
+            <input
+              type="number"
+              step="0.1"
+              min="0"
+              className="input input-bordered input-sm w-24"
+              value={hoursValue}
+              onChange={(e) => setHoursDraft((d) => ({ ...d, [row.id]: e.target.value }))}
+            />
+            <span className="text-xs opacity-60">
+              stored: {Math.round(row.hours * 10) / 10}h
+            </span>
+            <button
+              className="btn btn-sm"
+              disabled={busy === row.id || !hoursValid || !hoursChanged}
+              onClick={() => act(row.id, "hours", { hours: parsedHours })}
+            >
+              Save hours
+            </button>
+          </div>
+
           <div className="flex gap-2 flex-wrap items-center">
             <button
               className="btn btn-success btn-sm"
@@ -120,7 +157,8 @@ export default function AdminQueue({
             </div>
           </details>
         </div>
-      ))}
+        );
+      })}
     </div>
   );
 }