refactor: remove QuizAttempt model and related functionality across backend and frontend

backend/alembic/versions/0027_quiz_attempts.py

@@ -1,38 +0,0 @@
-"""quiz_attempts: a person's last finished run of a quiz, shown again when they reopen it
-
-Revision ID: 0027_quiz_attempts
-Revises: 0026_plan_copy
-Create Date: 2026-10-08
-"""
-import sqlalchemy as sa
-from alembic import op
-
-revision = "0027_quiz_attempts"
-down_revision = "0026_plan_copy"
-branch_labels = None
-depends_on = None
-
-
-def upgrade() -> None:
-    # 0001 builds from the live models, so a fresh database may already have the table.
-    if "quiz_attempts" in sa.inspect(op.get_bind()).get_table_names():
-        return
-    op.create_table(
-        "quiz_attempts",
-        sa.Column("id", sa.Uuid(), primary_key=True),
-        sa.Column("created_at", sa.DateTime(timezone=True), nullable=False),
-        sa.Column("updated_at", sa.DateTime(timezone=True), nullable=False),
-        sa.Column("workspace_id", sa.Uuid(), sa.ForeignKey("workspaces.id", ondelete="CASCADE"), nullable=False),
-        sa.Column("artifact_id", sa.Uuid(), sa.ForeignKey("artifacts.id", ondelete="CASCADE"), nullable=False),
-        sa.Column("user_id", sa.Uuid(), sa.ForeignKey("users.id", ondelete="CASCADE"), nullable=False),
-        sa.Column("score", sa.Integer(), nullable=False),
-        sa.Column("total", sa.Integer(), nullable=False),
-        sa.Column("answers", sa.JSON(), nullable=False),
-        sa.UniqueConstraint("artifact_id", "user_id"),
-    )
-    op.create_index("ix_quiz_attempts_workspace_id", "quiz_attempts", ["workspace_id"])
-    op.create_index("ix_quiz_attempts_artifact_id", "quiz_attempts", ["artifact_id"])
-
-
-def downgrade() -> None:
-    op.drop_table("quiz_attempts")

backend/app/models/__init__.py

@@ -18,7 +18,6 @@ from app.models.content import (
     MessageFeedback,
     Notebook,
     NotebookDocument,
-    QuizAttempt,
     ReportTemplateCache,
     SocialAccount,
     Source,
@@ -37,7 +36,6 @@ from app.models.support import SupportConversation, SupportMessage
 from app.models.user import AuthProvider, BillingEvent, PlanRecord, Subscription, User, Workspace, WorkspaceMember
 
 __all__ = [
-    "QuizAttempt",
     "SupportConversation",
     "SupportMessage",
     "Artifact",

backend/app/models/content.py

@@ -173,21 +173,6 @@ class ArtifactFeedback(IdMixin, TimestampMixin, Base):
     snapshot: Mapped[dict] = mapped_column(JSON, default=dict)
 
 
-class QuizAttempt(IdMixin, TimestampMixin, Base):
-    """A person's last finished run of a quiz: what they answered and the score, so opening the quiz again shows the result.
-    One row per quiz and person; finishing it again replaces it."""
-
-    __tablename__ = "quiz_attempts"
-    __table_args__ = (UniqueConstraint("artifact_id", "user_id"),)
-
-    workspace_id: Mapped[uuid.UUID] = _ws_fk()
-    artifact_id: Mapped[uuid.UUID] = mapped_column(Uuid, ForeignKey("artifacts.id", ondelete="CASCADE"), index=True)
-    user_id: Mapped[uuid.UUID] = mapped_column(Uuid, ForeignKey("users.id", ondelete="CASCADE"))
-    score: Mapped[int] = mapped_column(Integer)
-    total: Mapped[int] = mapped_column(Integer)
-    answers: Mapped[list] = mapped_column(JSON, default=list)  # per question: choice (indexes), text, correct
-
-
 class ReportTemplateCache(IdMixin, TimestampMixin, Base):
     """The suggested report templates already written for some sources, found again by a hash of exactly what the AI read (each
     post's stored ideas and topics, and the topic). The same input gives the same templates at once, so the Create report dialog

backend/app/routers/artifacts.py

@@ -11,7 +11,7 @@ from app.auth import Ctx, get_ctx
 from app.config import settings
 from app.infographics.image import ImageUnavailable, render_png
 from app.infographics.themes import theme_id
-from app.models import Artifact, ArtifactFeedback, CalendarItem, Chat, Document, Job, Notebook, QuizAttempt, UserSavedVoice
+from app.models import Artifact, ArtifactFeedback, CalendarItem, Chat, Document, Job, Notebook, UserSavedVoice
 from app.routers.notebooks import ensure_archive_notebook
 from app.services import artifact_feedback
 from app.services.artifacts import latest_jobs, serialize_artifact, start_artifact
@@ -345,37 +345,6 @@ def rate_artifact(artifact_id: uuid.UUID, body: FeedbackIn, ctx: Ctx = Depends(g
     return {"feedback": artifact_feedback.feedback_out(row)}
 
 
-class AttemptAnswer(BaseModel):
-    choice: list[int] = Field(default_factory=list, max_length=20)
-    text: str = Field("", max_length=2000)
-    correct: bool | None = None
-
-
-class AttemptIn(BaseModel):
-    answers: list[AttemptAnswer] = Field(max_length=200)
-    score: int = Field(ge=0)
-
-
-@router.put("/{artifact_id}/attempt")
-def save_quiz_attempt(artifact_id: uuid.UUID, body: AttemptIn, ctx: Ctx = Depends(get_ctx)):
-    """The person's finished run of a quiz, kept so the result shows when they open it again. Finishing again replaces it."""
-    a = get_artifact_or_404(ctx, artifact_id)
-    if a.type != "quiz" or a.status != "ready":
-        raise HTTPException(400, "Only a finished quiz keeps a result")
-    total = len((a.content_json or {}).get("questions") or [])
-    if len(body.answers) != total:
-        raise HTTPException(400, "The answers do not match the quiz")
-    row = ctx.db.scalar(select(QuizAttempt).where(QuizAttempt.artifact_id == a.id, QuizAttempt.user_id == ctx.user.id))
-    if not row:
-        row = QuizAttempt(workspace_id=ctx.workspace.id, artifact_id=a.id, user_id=ctx.user.id)
-        ctx.db.add(row)
-    row.total = total
-    row.score = min(body.score, total)
-    row.answers = [x.model_dump() for x in body.answers]
-    ctx.db.commit()
-    return {"attempt": artifact_feedback.attempt_out(row)}
-
-
 @router.get("/{artifact_id}/image")
 def infographic_image(artifact_id: uuid.UUID, layout: Literal["landscape", "portrait"] = "landscape", ctx: Ctx = Depends(get_ctx)):
     """A finished infographic as a PNG, wide (the default) or tall, for its Download button."""

backend/app/services/artifact_feedback.py

@@ -6,7 +6,7 @@ import uuid
 from sqlalchemy import select
 from sqlalchemy.orm import Session
 
-from app.models import Artifact, ArtifactFeedback, QuizAttempt
+from app.models import Artifact, ArtifactFeedback
 
 # What each kind can be marked down for (the ids the app sends; anything else is dropped).
 REASONS: dict[str, set[str]] = {
@@ -24,23 +24,14 @@ def feedback_out(f: ArtifactFeedback | None) -> dict | None:
     return {"rating": "up" if f.rating > 0 else "down", "reasons": f.reasons or [], "comment": f.comment}
 
 
-def attempt_out(a: QuizAttempt | None) -> dict | None:
-    return {"score": a.score, "total": a.total, "answers": a.answers or []} if a else None
-
-
 def attach(db: Session, user_id: uuid.UUID, items: list[dict]) -> list[dict]:
     """Add `feedback` (this person's rating, or None) to each serialized artifact that can be rated. One query."""
     ids = [uuid.UUID(i["id"]) for i in items if i.get("type") in RATEABLE]
     rows = {f.artifact_id: f for f in db.scalars(select(ArtifactFeedback).where(
         ArtifactFeedback.artifact_id.in_(ids), ArtifactFeedback.user_id == user_id))} if ids else {}
-    quiz_ids = [uuid.UUID(i["id"]) for i in items if i.get("type") == "quiz"]
-    attempts = {a.artifact_id: a for a in db.scalars(select(QuizAttempt).where(
-        QuizAttempt.artifact_id.in_(quiz_ids), QuizAttempt.user_id == user_id))} if quiz_ids else {}
     for i in items:
         if i.get("type") in RATEABLE:
             i["feedback"] = feedback_out(rows.get(uuid.UUID(i["id"])))
-        if i.get("type") == "quiz":
-            i["attempt"] = attempt_out(attempts.get(uuid.UUID(i["id"])))
     return items
 
 

frontend/src/api/endpoints.ts

@@ -1,7 +1,6 @@
 import { api, apiBlob, del, patch, post, postForm, put, qs, uploadFile } from "./client";
 import type {
   AnswerFeedback,
-  QuizAttempt,
   Artifact,
   ArtifactType,
   BillingStatus,
@@ -161,9 +160,6 @@ export const artifactsApi = {
   /** Thumbs up or down (null takes it back) on a finished report, quiz, flashcard set or infographic. */
   rate: (id: string, body: { rating: "up" | "down" | null; reasons?: string[]; comment?: string }) =>
     put<{ feedback: AnswerFeedback | null }>(`/api/artifacts/${id}/feedback`, body),
-  /** A quiz's finished run, kept so the result shows when it is opened again. */
-  saveAttempt: (id: string, body: { answers: QuizAttempt["answers"]; score: number }) =>
-    put<{ attempt: QuizAttempt }>(`/api/artifacts/${id}/attempt`, body),
   retry: (id: string) => post<Artifact>(`/api/artifacts/${id}/retry`),
   /** A slide deck as a PDF (exactly as shown) or an editable PowerPoint. */
   slidesFile: (id: string, ext: "pdf" | "pptx") => apiBlob(`/api/artifacts/${id}/slides.${ext}`),

frontend/src/api/types.ts

@@ -190,12 +190,8 @@ export type Artifact = {
   provider?: string | null;
   /** This person's thumbs up or down on a finished report, quiz, flashcard set or infographic. */
   feedback?: AnswerFeedback | null;
-  /** A quiz: this person's last finished run (what they answered, the score), shown again when it is reopened. */
-  attempt?: QuizAttempt | null;
 };
 
-export type QuizAttempt = { score: number; total: number; answers: { choice: number[]; text: string; correct: boolean | null }[] };
-
 export type TopicStatus = "rising" | "steady" | "dormant";
 
 export type TopicNode = {

frontend/src/components/ArtifactFeedback.tsx

@@ -62,8 +62,8 @@ export function ArtifactFeedback({ artifact, label, compact = false }: { artifac
   const initial = artifact.feedback ?? null;
   const fb = useSyncExternalStore(subscribe, () => (ratings.has(artifact.id) ? ratings.get(artifact.id)! : initial), () => initial);
   const [open, setOpen] = useState(false);
-  const [reasons, setReasons] = useState<string[]>(initial?.reasons ?? []);
-  const [comment, setComment] = useState(initial?.comment ?? "");
+  const [reasons, setReasons] = useState<string[]>(fb?.reasons ?? []);
+  const [comment, setComment] = useState(fb?.comment ?? "");
   const [error, setError] = useState<string | null>(null);
   const [thanks, setThanks] = useState(false);
   const options = REASONS[artifact.type] ?? [];
@@ -116,6 +116,18 @@ export function ArtifactFeedback({ artifact, label, compact = false }: { artifac
         {thanks && <span className="mono muted small">Thanks, saved.</span>}
         {error && <span className="error-text small">Could not save that. Try again.</span>}
       </div>
+      {!open && fb && (fb.comment || fb.reasons.length > 0) && (
+        <div className="afb-saved">
+          <span className="mono muted small">Your review</span>
+          {fb.reasons.length > 0 && (
+            <div className="fb-chips">
+              {fb.reasons.map((id) => <span key={id} className="fb-chip is-on">{options.find((o) => o.id === id)?.label ?? id}</span>)}
+            </div>
+          )}
+          {fb.comment && <p className="afb-saved-text">{fb.comment}</p>}
+          <button type="button" className="link-btn small" onClick={() => { setReasons(fb.reasons); setComment(fb.comment ?? ""); setOpen(true); }}>Edit</button>
+        </div>
+      )}
       {open && fb && (
         <div className="fb-why afb-why" ref={whyRef}>
           <p className="mono muted small">{fb.rating === "up" ? "What worked well? (optional)" : "What was wrong? (optional)"}</p>

frontend/src/components/Quiz.tsx

@@ -1,6 +1,6 @@
 import { useMemo, useState, type ReactNode } from "react";
-import { artifactsApi, type GenerateBody } from "../api/endpoints";
-import type { Artifact, ChatSummary, QuizAttempt, QuizQuestion, QuizQuestionType } from "../api/types";
+import type { GenerateBody } from "../api/endpoints";
+import type { Artifact, ChatSummary, QuizQuestion, QuizQuestionType } from "../api/types";
 import { ArtifactFeedback } from "./ArtifactFeedback";
 import { CheckIcon, ChevronIcon } from "./icons/Icons";
 import { Pills, SourceFocusFields, sourceSummary, type SourceSelection } from "./SourceFocusFields";
@@ -188,21 +188,11 @@ export function ResultScene({ score, total, tone }: { score: number; total: numb
 }
 
 /** The quiz itself, without a frame: in a modal (QuizPlayer) or inside a report. `onExit` adds a Close button. */
-export function QuizRunner({ questions, onExit, feedback, saved, onFinish }: {
-  questions: QuizQuestion[];
-  onExit?: () => void;
-  feedback?: ReactNode;
-  /** A finished run kept earlier: opening the quiz again starts on its result. */
-  saved?: QuizAttempt | null;
-  /** Called with the answers when the last question is done, so the result can be kept. */
-  onFinish?: (answers: QuizAttempt["answers"], score: number) => void;
-}) {
-  const restored = saved && saved.answers.length === questions.length ? saved : null;
+export function QuizRunner({ questions, onExit, feedback }: { questions: QuizQuestion[]; onExit?: () => void; feedback?: ReactNode }) {
   const [i, setI] = useState(0);
-  const [answers, setAnswers] = useState<Answer[]>(() =>
-    restored ? restored.answers.map((a) => ({ choice: a.choice, text: a.text, checked: true, correct: a.correct })) : questions.map(blank));
-  const [done, setDone] = useState(!!restored);
-  const [finished, setFinished] = useState(!!restored); // the results have been reached: a question opened from them leads back to them
+  const [answers, setAnswers] = useState<Answer[]>(() => questions.map(blank));
+  const [done, setDone] = useState(false);
+  const [finished, setFinished] = useState(false); // the results have been reached: a question opened from them leads back to them
   const [detail, setDetail] = useState(false); // the per-question results, under the score
 
   const q = questions[i];
@@ -279,7 +269,6 @@ export function QuizRunner({ questions, onExit, feedback, saved, onFinish }: {
     if (!last) return setI(i + 1);
     setFinished(true);
     setDone(true);
-    onFinish?.(answers.map((x) => ({ choice: x.choice, text: x.text, correct: x.correct })), score);
   };
   const pick = (k: number) => {
     if (a.checked) return;
@@ -372,20 +361,11 @@ export function QuizRunner({ questions, onExit, feedback, saved, onFinish }: {
   );
 }
 
-// A finished run is shown on every open of the quiz, also before the list is fetched again, so the latest copy is kept here.
-const attempts = new Map<string, QuizAttempt>();
-
 export function QuizPlayer({ artifact, onClose }: { artifact: Artifact; onClose: () => void }) {
   const questions = useMemo(() => (artifact.content.questions ?? []) as QuizQuestion[], [artifact]);
-  const saved = attempts.get(artifact.id) ?? artifact.attempt ?? null;
-  const keep = (answers: QuizAttempt["answers"], score: number) => {
-    attempts.set(artifact.id, { score, total: questions.length, answers });
-    artifactsApi.saveAttempt(artifact.id, { answers, score }).catch(() => undefined); // the result is also kept here for this visit
-  };
   return (
     <StudyOverlay title={artifact.title} onClose={onClose}>
-      <QuizRunner questions={questions} onExit={onClose} saved={saved} onFinish={keep}
-                  feedback={<ArtifactFeedback artifact={artifact} label="How was this quiz?" />} />
+      <QuizRunner questions={questions} onExit={onClose} feedback={<ArtifactFeedback artifact={artifact} label="How was this quiz?" />} />
     </StudyOverlay>
   );
 }
@@ -395,12 +375,10 @@ export function QuizBody({ artifact }: { artifact: Artifact }) {
   const [playing, setPlaying] = useState(false);
   const c = artifact.content;
   const from = sourceSummary(c.source);
-  const last = attempts.get(artifact.id) ?? artifact.attempt ?? null; // read again whenever the player closes (the state change re-renders)
   return (
     <div className="artifact-body">
       <p className="muted">{c.question_count ?? 0} questions · {c.difficulty ?? "medium"} · from {from}</p>
-      {last && <p className="muted small">Last score {last.score} / {last.total}</p>}
-      <button type="button" className="btn btn-small" onClick={() => setPlaying(true)}>{last ? "View result" : "Start quiz"}</button>
+      <button type="button" className="btn btn-small" onClick={() => setPlaying(true)}>Start quiz</button>
       {playing && <QuizPlayer artifact={artifact} onClose={() => setPlaying(false)} />}
     </div>
   );

frontend/src/styles/pages.css

@@ -9928,3 +9928,28 @@ body.nbv-resizing iframe {
 .modal.modal-wide.so-modal {
   height: min(760px, calc(100vh - 32px));
 }
+
+/* The review a person already gave, shown again under the thumbs when the quiz, cards, report or infographic is reopened */
+.afb-saved {
+  display: grid;
+  gap: 6px;
+  justify-items: start;
+  margin-top: 8px;
+  padding: 10px 12px;
+  border: 1px solid var(--line);
+  border-radius: var(--radius-sm);
+  background: var(--surface);
+}
+
+.afb-saved .fb-chip {
+  cursor: default;
+}
+
+.afb-saved-text {
+  margin: 0;
+  color: var(--text-2);
+  font-size: 0.88rem;
+  line-height: 1.45;
+  overflow-wrap: anywhere;
+  white-space: pre-wrap;
+}