refactor: split app structure

2026-06-15 12:24:49 +08:00
parent e15c931e91
commit b9468c7485
17 changed files with 1263 additions and 1090 deletions
@@ -0,0 +1,119 @@
+from __future__ import annotations
+
+from fastapi import APIRouter, Depends, HTTPException, Query
+
+from ..auth import verify_admin_auth
+from ..config import (
+    ACCOUNTS_PATH,
+    ADMIN_TASKS_PATH,
+    CLASSNOTES_PATH,
+    COURSE_SUMMARIES_ROOT,
+    OPERATION_LOGS_PATH,
+    write_lock,
+)
+from ..data import (
+    append_operation_log,
+    approve_admin_task,
+    list_admin_tasks,
+    list_operation_logs,
+    query_course_summaries,
+    reject_admin_task,
+)
+
+
+router = APIRouter()
+
+
+@router.get("/api/admin/tasks")
+def admin_tasks(
+    status_filter: str = Query("", alias="status"),
+    task_type: str = Query("", alias="type"),
+    _user: str = Depends(verify_admin_auth),
+):
+    try:
+        return list_admin_tasks(ADMIN_TASKS_PATH, status_filter=status_filter, task_type=task_type)
+    except ValueError as exc:
+        raise HTTPException(status_code=500, detail=str(exc)) from exc
+
+
+@router.get("/api/admin/operation-logs")
+def admin_operation_logs(
+    limit: int = Query(100, ge=1, le=500),
+    operation: str = Query(""),
+    status_filter: str = Query("", alias="status"),
+    student: str = Query(""),
+    _user: str = Depends(verify_admin_auth),
+):
+    return list_operation_logs(
+        OPERATION_LOGS_PATH,
+        limit=limit,
+        operation=operation,
+        status_filter=status_filter,
+        student=student,
+    )
+
+
+@router.get("/api/admin/course-summaries")
+def admin_course_summaries(
+    q: str = Query(""),
+    student: str = Query(""),
+    teacher: str = Query(""),
+    subject: str = Query(""),
+    date_from: str = Query(""),
+    date_to: str = Query(""),
+    limit: int = Query(200, ge=1, le=1000),
+    _user: str = Depends(verify_admin_auth),
+):
+    try:
+        return query_course_summaries(
+            COURSE_SUMMARIES_ROOT,
+            q=q,
+            student=student,
+            teacher=teacher,
+            subject=subject,
+            date_from=date_from,
+            date_to=date_to,
+            limit=limit,
+        )
+    except ValueError as exc:
+        raise HTTPException(status_code=400, detail=str(exc)) from exc
+
+
+@router.post("/api/admin/tasks/{task_id}/approve")
+def admin_approve_task(task_id: int, _user: str = Depends(verify_admin_auth)):
+    try:
+        with write_lock:
+            result = approve_admin_task(ADMIN_TASKS_PATH, CLASSNOTES_PATH, ACCOUNTS_PATH, task_id)
+            task = result.get("task", {})
+            append_operation_log(
+                OPERATION_LOGS_PATH,
+                "admin_task_approve",
+                "approved",
+                task_id=task_id,
+                task_type=str(task.get("type") or ""),
+                student=str(task.get("student") or task.get("corrected", {}).get("student") or ""),
+                source_id=str(task.get("source_id") or ""),
+                backup_id=str(result.get("backup_id") or task.get("backup_id") or ""),
+            )
+    except ValueError as exc:
+        raise HTTPException(status_code=409, detail=str(exc)) from exc
+    return {"ok": True, **result}
+
+
+@router.post("/api/admin/tasks/{task_id}/reject")
+def admin_reject_task(task_id: int, _user: str = Depends(verify_admin_auth)):
+    try:
+        with write_lock:
+            task = reject_admin_task(ADMIN_TASKS_PATH, task_id)
+            append_operation_log(
+                OPERATION_LOGS_PATH,
+                "admin_task_reject",
+                "rejected",
+                task_id=task_id,
+                task_type=str(task.get("type") or ""),
+                student=str(task.get("student") or task.get("corrected", {}).get("student") or ""),
+                source_id=str(task.get("source_id") or ""),
+            )
+    except ValueError as exc:
+        raise HTTPException(status_code=400, detail=str(exc)) from exc
+    return {"ok": True, "task": task}