From 8f6e31e2932dde2cd60854f342885a1d33390299 Mon Sep 17 00:00:00 2001
From: Ayush Raj Chourasia <iter.student.alpha@gmail.com>
Date: Wed, 13 May 2026 18:46:39 +0000
Subject: [PATCH 1/3] Add gameplay review workflows

---
 .../GameplayReviewWorkspace.test.tsx          | 182 +++++
 .../intelligence/GameplayReviewWorkspace.tsx  | 573 ++++++++++++++++
 app/src/pages/Intelligence.tsx                |   6 +-
 app/src/services/gameplayReviewService.ts     | 224 +++++++
 src/core/all.rs                               |   5 +
 src/openhuman/about_app/catalog.rs            |  50 ++
 src/openhuman/gameplay_review/mod.rs          |  14 +
 src/openhuman/gameplay_review/ops.rs          | 630 ++++++++++++++++++
 src/openhuman/gameplay_review/schemas.rs      | 178 +++++
 src/openhuman/gameplay_review/store.rs        | 175 +++++
 src/openhuman/gameplay_review/types.rs        | 173 +++++
 src/openhuman/mod.rs                          |   1 +
 12 files changed, 2210 insertions(+), 1 deletion(-)
 create mode 100644 app/src/components/intelligence/GameplayReviewWorkspace.test.tsx
 create mode 100644 app/src/components/intelligence/GameplayReviewWorkspace.tsx
 create mode 100644 app/src/services/gameplayReviewService.ts
 create mode 100644 src/openhuman/gameplay_review/mod.rs
 create mode 100644 src/openhuman/gameplay_review/ops.rs
 create mode 100644 src/openhuman/gameplay_review/schemas.rs
 create mode 100644 src/openhuman/gameplay_review/store.rs
 create mode 100644 src/openhuman/gameplay_review/types.rs

diff --git a/app/src/components/intelligence/GameplayReviewWorkspace.test.tsx b/app/src/components/intelligence/GameplayReviewWorkspace.test.tsx
new file mode 100644
index 0000000000..8419d66683
--- /dev/null
+++ b/app/src/components/intelligence/GameplayReviewWorkspace.test.tsx
@@ -0,0 +1,182 @@
+import { fireEvent, render, screen, waitFor } from '@testing-library/react';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
+import {
+  analyzeGameplaySession,
+  askGameplaySession,
+  draftGameplayClipMetadata,
+  listGameplaySessions,
+  prepareGameplayFrames,
+  registerGameplaySession,
+  saveGameplayPreset,
+} from '../../services/gameplayReviewService';
+import { GameplayReviewWorkspace } from './GameplayReviewWorkspace';
+
+vi.mock('../../services/gameplayReviewService', () => ({
+  analyzeGameplaySession: vi.fn(),
+  askGameplaySession: vi.fn(),
+  draftGameplayClipMetadata: vi.fn(),
+  flattenClipCandidates: vi.fn((session: any) => session?.analysis?.clip_candidates ?? []),
+  flattenDrafts: vi.fn((session: any) => session?.analysis?.draft_metadata ?? []),
+  flattenHighlights: vi.fn((session: any) => session?.analysis?.highlights ?? []),
+  formatSpoilerMode: vi.fn((mode: string) => (mode === 'full' ? 'Full spoilers' : 'Light spoilers')),
+  listGameplaySessions: vi.fn(),
+  normalizeGameplayError: vi.fn((error: unknown) => (error instanceof Error ? error.message : String(error))),
+  prepareGameplayFrames: vi.fn(),
+  registerGameplaySession: vi.fn(),
+  saveGameplayPreset: vi.fn(),
+}));
+
+const mockedListGameplaySessions = vi.mocked(listGameplaySessions);
+const mockedPrepareGameplayFrames = vi.mocked(prepareGameplayFrames);
+const mockedRegisterGameplaySession = vi.mocked(registerGameplaySession);
+const mockedAnalyzeGameplaySession = vi.mocked(analyzeGameplaySession);
+const mockedAskGameplaySession = vi.mocked(askGameplaySession);
+const mockedDraftGameplayClipMetadata = vi.mocked(draftGameplayClipMetadata);
+const mockedSaveGameplayPreset = vi.mocked(saveGameplayPreset);
+
+describe('GameplayReviewWorkspace', () => {
+  beforeEach(() => {
+    mockedListGameplaySessions.mockResolvedValue([]);
+    mockedPrepareGameplayFrames.mockReset();
+    mockedRegisterGameplaySession.mockReset();
+    mockedAnalyzeGameplaySession.mockReset();
+    mockedAskGameplaySession.mockReset();
+    mockedDraftGameplayClipMetadata.mockReset();
+    mockedSaveGameplayPreset.mockReset();
+  });
+
+  it('imports selected frames, analyzes the session, and asks follow-up questions', async () => {
+    mockedPrepareGameplayFrames.mockResolvedValue([
+      {
+        source_name: 'frame-1.png',
+        file_name: 'frame-1.png',
+        image_ref: 'data:image/png;base64,AAA',
+        captured_at_ms: 123,
+      },
+    ]);
+    mockedRegisterGameplaySession.mockResolvedValue({
+      session_id: 'gameplay-apex-1',
+      game_id: 'Apex Legends',
+      session_title: 'Ranked climb',
+      source_label: '/recordings/apex',
+      spoiler_mode: 'light',
+      preset_id: 'Apex preset',
+      imported_at_ms: 1000,
+      analyzed_at_ms: null,
+      frames: [
+        { file_name: 'frame-1.png', image_ref: 'data:image/png;base64,AAA', captured_at_ms: 123 },
+      ],
+      analysis: null,
+    });
+    mockedAnalyzeGameplaySession.mockResolvedValue({
+      session_id: 'gameplay-apex-1',
+      game_id: 'Apex Legends',
+      session_title: 'Ranked climb',
+      source_label: '/recordings/apex',
+      spoiler_mode: 'light',
+      preset_id: 'Apex preset',
+      imported_at_ms: 1000,
+      analyzed_at_ms: 2000,
+      frames: [
+        { file_name: 'frame-1.png', image_ref: 'data:image/png;base64,AAA', captured_at_ms: 123 },
+      ],
+      analysis: {
+        recap: 'Gameplay recap',
+        highlights: [
+          {
+            id: 'h1',
+            frame_index: 0,
+            captured_at_ms: 123,
+            title: 'Highlight: clutch fight',
+            rationale: 'Clean finish',
+            confidence: 0.93,
+            kind: 'highlight',
+          },
+        ],
+        clip_candidates: [
+          {
+            id: 'c1',
+            frame_index: 0,
+            start_label: 'frame-1.png',
+            end_label: 'frame-1.png',
+            rationale: 'Clean finish',
+            confidence: 0.93,
+          },
+        ],
+        draft_metadata: [
+          {
+            platform: 'twitch',
+            title: 'Apex Legends — Highlight: clutch fight (twitch)',
+            description: 'Session: Ranked climb',
+            tags: ['apex-legends'],
+          },
+        ],
+        follow_up_questions: ['What was the turning point?'],
+        spoiler_note: null,
+      },
+    });
+    mockedAskGameplaySession.mockResolvedValue({
+      answer: 'Best clip candidate for Ranked climb: Highlight: clutch fight — Clean finish',
+      matched_highlights: [],
+      suggested_follow_up: ['What was the turning point?'],
+    });
+    mockedDraftGameplayClipMetadata.mockResolvedValue([
+      {
+        platform: 'twitch',
+        title: 'Apex Legends — Highlight: clutch fight (twitch)',
+        description: 'Session: Ranked climb',
+        tags: ['apex-legends'],
+      },
+    ]);
+    mockedSaveGameplayPreset.mockResolvedValue({});
+
+    const { container } = render(<GameplayReviewWorkspace />);
+
+    await waitFor(() => expect(screen.getByText('No saved sessions yet.')).toBeInTheDocument());
+
+    fireEvent.change(screen.getByPlaceholderText('Apex Legends'), {
+      target: { value: 'Apex Legends' },
+    });
+    fireEvent.change(screen.getByPlaceholderText('Ranked climb on Friday night'), {
+      target: { value: 'Ranked climb' },
+    });
+    fireEvent.change(screen.getByPlaceholderText('/recordings/apex/night-01'), {
+      target: { value: '/recordings/apex' },
+    });
+    fireEvent.change(screen.getByPlaceholderText('Apex coaching preset'), {
+      target: { value: 'Apex preset' },
+    });
+    fireEvent.change(screen.getByPlaceholderText('aim, positioning, fight selection'), {
+      target: { value: 'aim, positioning' },
+    });
+    fireEvent.change(screen.getByPlaceholderText('What should the reviewer pay attention to?'), {
+      target: { value: 'Play clean and keep it spoiler-safe.' },
+    });
+    fireEvent.change(screen.getByPlaceholderText('twitch,kick,youtube'), {
+      target: { value: 'twitch,kick' },
+    });
+
+    const fileInput = container.querySelector('input[type="file"]') as HTMLInputElement | null;
+    expect(fileInput).not.toBeNull();
+    const file = new File([new Uint8Array([1, 2, 3])], 'frame-1.png', { type: 'image/png' });
+    Object.defineProperty(file, 'webkitRelativePath', { value: 'session/frame-1.png' });
+    fireEvent.change(fileInput as HTMLInputElement, {
+      target: { files: [file] },
+    });
+
+    fireEvent.click(screen.getByRole('button', { name: /import and analyze/i }));
+
+    await waitFor(() => expect(mockedPrepareGameplayFrames).toHaveBeenCalled());
+    await waitFor(() => expect(screen.getByText(/Gameplay recap/)).toBeInTheDocument());
+    expect(
+      screen.getByText('Highlight: clutch fight', {
+        selector: 'div.font-medium.text-stone-900',
+      })
+    ).toBeInTheDocument();
+
+    fireEvent.click(screen.getByRole('button', { name: /ask session/i }));
+    await waitFor(() => expect(mockedAskGameplaySession).toHaveBeenCalled());
+    expect(screen.getByText(/Best clip candidate/)).toBeInTheDocument();
+  });
+});
diff --git a/app/src/components/intelligence/GameplayReviewWorkspace.tsx b/app/src/components/intelligence/GameplayReviewWorkspace.tsx
new file mode 100644
index 0000000000..8e9a462818
--- /dev/null
+++ b/app/src/components/intelligence/GameplayReviewWorkspace.tsx
@@ -0,0 +1,573 @@
+import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
+
+import type { ToastNotification } from '../../types/intelligence';
+import {
+  analyzeGameplaySession,
+  askGameplaySession,
+  draftGameplayClipMetadata,
+  flattenClipCandidates,
+  flattenDrafts,
+  flattenHighlights,
+  formatSpoilerMode,
+  listGameplaySessions,
+  normalizeGameplayError,
+  prepareGameplayFrames,
+  registerGameplaySession,
+  saveGameplayPreset,
+  type GameplayFrameInput,
+  type GameplayReviewSession,
+  type SpoilerMode,
+} from '../../services/gameplayReviewService';
+
+interface GameplayReviewWorkspaceProps {
+  onToast?: (toast: Omit<ToastNotification, 'id'>) => void;
+}
+
+interface ImportState {
+  gameId: string;
+  sessionTitle: string;
+  sourceLabel: string;
+  spoilerMode: SpoilerMode;
+  presetName: string;
+  coachingFocus: string;
+  notes: string;
+  audioFeedback: boolean;
+  platforms: string;
+}
+
+const INITIAL_IMPORT_STATE: ImportState = {
+  gameId: '',
+  sessionTitle: '',
+  sourceLabel: '',
+  spoilerMode: 'light',
+  presetName: '',
+  coachingFocus: '',
+  notes: '',
+  audioFeedback: false,
+  platforms: 'twitch,kick,youtube',
+};
+
+const DIRECTORY_INPUT_PROPS: Record<string, string> = {
+  webkitdirectory: '',
+};
+
+function formatTimestamp(ms?: number | null): string {
+  if (!ms) return 'n/a';
+  return new Date(ms).toLocaleString();
+}
+
+function splitPlatforms(value: string): string[] {
+  return value
+    .split(',')
+    .map(item => item.trim())
+    .filter(Boolean);
+}
+
+export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProps) {
+  const [form, setForm] = useState(INITIAL_IMPORT_STATE);
+  const [selectedFiles, setSelectedFiles] = useState<File[]>([]);
+  const [recentSessions, setRecentSessions] = useState<GameplayReviewSession[]>([]);
+  const [activeSession, setActiveSession] = useState<GameplayReviewSession | null>(null);
+  const [question, setQuestion] = useState('What were my best moments?');
+  const [questionAnswer, setQuestionAnswer] = useState<string>('');
+  const [questionBusy, setQuestionBusy] = useState(false);
+  const [importBusy, setImportBusy] = useState(false);
+  const [analysisBusy, setAnalysisBusy] = useState(false);
+  const [draftBusy, setDraftBusy] = useState(false);
+  const [error, setError] = useState<string | null>(null);
+  const fileInputRef = useRef<HTMLInputElement | null>(null);
+
+  const refreshSessions = useCallback(async () => {
+    try {
+      const sessions = await listGameplaySessions();
+      setRecentSessions(sessions);
+      if (!activeSession && sessions.length > 0) {
+        setActiveSession(sessions[0]);
+      }
+    } catch (err) {
+      setError(normalizeGameplayError(err));
+    }
+  }, [activeSession]);
+
+  useEffect(() => {
+    void refreshSessions();
+  }, [refreshSessions]);
+
+  const activeHighlights = useMemo(() => flattenHighlights(activeSession), [activeSession]);
+  const activeDrafts = useMemo(() => flattenDrafts(activeSession), [activeSession]);
+  const activeClips = useMemo(() => flattenClipCandidates(activeSession), [activeSession]);
+
+  const handleSelectFiles = useCallback(() => {
+    fileInputRef.current?.click();
+  }, []);
+
+  const handleFilesChanged = useCallback((event: React.ChangeEvent<HTMLInputElement>) => {
+    const files = Array.from(event.target.files ?? []);
+    setSelectedFiles(files);
+  }, []);
+
+  const handleImportSession = useCallback(async () => {
+    setError(null);
+    if (!form.gameId.trim()) {
+      setError('Game name is required.');
+      return;
+    }
+    if (!form.sessionTitle.trim()) {
+      setError('Session title is required.');
+      return;
+    }
+    if (selectedFiles.length === 0) {
+      setError('Choose a folder or a set of keyframe images first.');
+      return;
+    }
+
+    setImportBusy(true);
+    try {
+      const frames = await prepareGameplayFrames(selectedFiles);
+      if (frames.length === 0) {
+        throw new Error('No image frames were found in that folder.');
+      }
+
+      if (form.presetName.trim()) {
+        await saveGameplayPreset({
+          game_id: form.gameId.trim(),
+          display_name: form.presetName.trim(),
+          coaching_focus: form.coachingFocus
+            .split(',')
+            .map(item => item.trim())
+            .filter(Boolean),
+          audio_feedback: form.audioFeedback,
+          spoiler_mode: form.spoilerMode,
+          notes: form.notes.trim() || null,
+        });
+      }
+
+      const session = await registerGameplaySession({
+        game_id: form.gameId.trim(),
+        session_title: form.sessionTitle.trim(),
+        source_label: form.sourceLabel.trim() || null,
+        spoiler_mode: form.spoilerMode,
+        preset_id: form.presetName.trim() || null,
+        frames: frames.map(frame => ({
+          file_name: frame.file_name,
+          image_ref: frame.image_ref,
+          captured_at_ms: frame.captured_at_ms,
+        } satisfies GameplayFrameInput)),
+      });
+
+      setAnalysisBusy(true);
+      const analyzed = await analyzeGameplaySession({
+        session_id: session.session_id,
+        max_highlights: 5,
+        platforms: splitPlatforms(form.platforms),
+      });
+      setActiveSession(analyzed);
+      setQuestionAnswer('');
+      onToast?.({
+        type: 'success',
+        title: 'Gameplay session analyzed',
+        message: `Reviewed ${analyzed.frames.length} frame(s) for ${analyzed.game_id}.`,
+      });
+      void refreshSessions();
+    } catch (err) {
+      const message = normalizeGameplayError(err);
+      setError(message);
+      onToast?.({
+        type: 'error',
+        title: 'Gameplay review failed',
+        message,
+      });
+    } finally {
+      setAnalysisBusy(false);
+      setImportBusy(false);
+    }
+  }, [form, onToast, refreshSessions, selectedFiles]);
+
+  const handleQuestion = useCallback(async () => {
+    if (!activeSession) {
+      setError('Select or analyze a session first.');
+      return;
+    }
+    setQuestionBusy(true);
+    try {
+      const response = await askGameplaySession({ session_id: activeSession.session_id, question });
+      setQuestionAnswer(response.answer);
+    } catch (err) {
+      setError(normalizeGameplayError(err));
+    } finally {
+      setQuestionBusy(false);
+    }
+  }, [activeSession, question]);
+
+  const handleDraftClips = useCallback(async () => {
+    if (!activeSession) return;
+    setDraftBusy(true);
+    try {
+      const drafts = await draftGameplayClipMetadata({
+        session_id: activeSession.session_id,
+        platform: splitPlatforms(form.platforms)[0] || 'twitch',
+        highlight_id: activeHighlights[0]?.id ?? null,
+      });
+      setActiveSession(prev =>
+        prev
+          ? {
+              ...prev,
+              analysis: prev.analysis
+                ? { ...prev.analysis, draft_metadata: drafts }
+                : prev.analysis,
+            }
+          : prev
+      );
+    } catch (err) {
+      setError(normalizeGameplayError(err));
+    } finally {
+      setDraftBusy(false);
+    }
+  }, [activeHighlights, activeSession, form.platforms]);
+
+  return (
+    <div className="space-y-4" data-testid="gameplay-review-workspace">
+      <div className="rounded-2xl border border-sage-200 bg-gradient-to-br from-white via-sage-50 to-amber-50 p-5 shadow-soft">
+        <div className="flex flex-wrap items-start justify-between gap-4">
+          <div className="space-y-2">
+            <p className="text-xs font-semibold uppercase tracking-[0.24em] text-sage-700">
+              Gameplay review
+            </p>
+            <h2 className="text-2xl font-bold text-stone-900">Import a session, find the clips, draft the post</h2>
+            <p className="max-w-2xl text-sm text-stone-600">
+              Load a folder of keyframes from a long session, generate a concise recap, ask follow-up questions,
+              and turn the strongest moments into platform-ready clip metadata.
+            </p>
+          </div>
+          <div className="rounded-xl border border-sage-200 bg-white/80 px-4 py-3 text-sm text-stone-700 shadow-sm">
+            <div className="font-semibold text-stone-900">Selected files</div>
+            <div>{selectedFiles.length} file(s)</div>
+            <div>{formatSpoilerMode(form.spoilerMode)}</div>
+          </div>
+        </div>
+
+        <div className="mt-4 grid gap-3 md:grid-cols-2 xl:grid-cols-3">
+          <label className="space-y-1 text-sm">
+            <span className="font-medium text-stone-700">Game</span>
+            <input
+              value={form.gameId}
+              onChange={event => setForm(prev => ({ ...prev, gameId: event.target.value }))}
+              className="w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500"
+              placeholder="Apex Legends"
+            />
+          </label>
+          <label className="space-y-1 text-sm">
+            <span className="font-medium text-stone-700">Session title</span>
+            <input
+              value={form.sessionTitle}
+              onChange={event => setForm(prev => ({ ...prev, sessionTitle: event.target.value }))}
+              className="w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500"
+              placeholder="Ranked climb on Friday night"
+            />
+          </label>
+          <label className="space-y-1 text-sm">
+            <span className="font-medium text-stone-700">Source label</span>
+            <input
+              value={form.sourceLabel}
+              onChange={event => setForm(prev => ({ ...prev, sourceLabel: event.target.value }))}
+              className="w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500"
+              placeholder="/recordings/apex/night-01"
+            />
+          </label>
+          <label className="space-y-1 text-sm">
+            <span className="font-medium text-stone-700">Spoiler mode</span>
+            <select
+              value={form.spoilerMode}
+              onChange={event => setForm(prev => ({ ...prev, spoilerMode: event.target.value as SpoilerMode }))}
+              className="w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500"
+            >
+              <option value="off">Off</option>
+              <option value="light">Light</option>
+              <option value="full">Full</option>
+            </select>
+          </label>
+          <label className="space-y-1 text-sm">
+            <span className="font-medium text-stone-700">Preset name</span>
+            <input
+              value={form.presetName}
+              onChange={event => setForm(prev => ({ ...prev, presetName: event.target.value }))}
+              className="w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500"
+              placeholder="Apex coaching preset"
+            />
+          </label>
+          <label className="space-y-1 text-sm">
+            <span className="font-medium text-stone-700">Focus areas</span>
+            <input
+              value={form.coachingFocus}
+              onChange={event => setForm(prev => ({ ...prev, coachingFocus: event.target.value }))}
+              className="w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500"
+              placeholder="aim, positioning, fight selection"
+            />
+          </label>
+          <label className="space-y-1 text-sm md:col-span-2 xl:col-span-3">
+            <span className="font-medium text-stone-700">Preset notes</span>
+            <textarea
+              value={form.notes}
+              onChange={event => setForm(prev => ({ ...prev, notes: event.target.value }))}
+              className="min-h-24 w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500"
+              placeholder="What should the reviewer pay attention to?"
+            />
+          </label>
+          <label className="space-y-1 text-sm md:col-span-2">
+            <span className="font-medium text-stone-700">Clip platforms</span>
+            <input
+              value={form.platforms}
+              onChange={event => setForm(prev => ({ ...prev, platforms: event.target.value }))}
+              className="w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500"
+              placeholder="twitch,kick,youtube"
+            />
+          </label>
+          <label className="flex items-center gap-2 self-end text-sm text-stone-700 md:col-span-1">
+            <input
+              type="checkbox"
+              checked={form.audioFeedback}
+              onChange={event => setForm(prev => ({ ...prev, audioFeedback: event.target.checked }))}
+            />
+            Enable audio summary notes
+          </label>
+        </div>
+
+        <div className="mt-4 flex flex-wrap items-center gap-3">
+          <input
+            ref={fileInputRef}
+            type="file"
+            multiple
+            accept="image/*"
+            {...DIRECTORY_INPUT_PROPS}
+            className="hidden"
+            onChange={handleFilesChanged}
+          />
+          <button
+            type="button"
+            onClick={handleSelectFiles}
+            className="rounded-xl border border-sage-200 bg-white px-4 py-2 text-sm font-semibold text-sage-800 shadow-sm transition hover:bg-sage-50"
+          >
+            Choose folder / frames
+          </button>
+          <button
+            type="button"
+            onClick={handleImportSession}
+            disabled={importBusy || analysisBusy}
+            className="rounded-xl bg-primary-600 px-4 py-2 text-sm font-semibold text-white shadow-sm transition hover:bg-primary-700 disabled:cursor-not-allowed disabled:opacity-60"
+          >
+            {importBusy ? 'Importing…' : 'Import and analyze'}
+          </button>
+          <button
+            type="button"
+            onClick={handleDraftClips}
+            disabled={draftBusy || !activeSession}
+            className="rounded-xl border border-amber-200 bg-white px-4 py-2 text-sm font-semibold text-amber-800 shadow-sm transition hover:bg-amber-50 disabled:cursor-not-allowed disabled:opacity-60"
+          >
+            {draftBusy ? 'Drafting…' : 'Refresh clip metadata'}
+          </button>
+          <div className="text-xs text-stone-500">
+            The core will analyze image frames. For raw video files, point this at extracted keyframes.
+          </div>
+        </div>
+      </div>
+
+      {error && (
+        <div className="rounded-xl border border-coral-200 bg-coral-50 px-4 py-3 text-sm text-coral-800">
+          {error}
+        </div>
+      )}
+
+      <div className="grid gap-4 lg:grid-cols-[1.2fr_0.8fr]">
+        <section className="space-y-4 rounded-2xl border border-stone-200 bg-white p-5 shadow-soft">
+          <div className="flex items-center justify-between gap-2">
+            <div>
+              <h3 className="text-lg font-semibold text-stone-900">Session viewer</h3>
+              <p className="text-sm text-stone-500">Recap, highlights, and clip candidates for the current session.</p>
+            </div>
+            <div className="rounded-full border border-stone-200 bg-stone-50 px-3 py-1 text-xs text-stone-600">
+              {activeSession ? activeSession.game_id : 'No session selected'}
+            </div>
+          </div>
+
+          {activeSession ? (
+            <div className="space-y-4">
+              <div className="rounded-xl border border-sage-200 bg-sage-50 p-4">
+                <div className="flex flex-wrap items-center justify-between gap-2">
+                  <div>
+                    <div className="text-sm font-semibold text-sage-900">{activeSession.session_title}</div>
+                    <div className="text-xs text-sage-700">
+                      Imported {formatTimestamp(activeSession.imported_at_ms)} · {activeSession.frames.length} frame(s)
+                    </div>
+                  </div>
+                  <div className="text-xs font-medium text-sage-700">
+                    {formatSpoilerMode(activeSession.spoiler_mode)}
+                  </div>
+                </div>
+                {activeSession.analysis?.spoiler_note && (
+                  <p className="mt-3 text-sm text-stone-700">{activeSession.analysis.spoiler_note}</p>
+                )}
+                {activeSession.analysis && (
+                  <p className="mt-3 whitespace-pre-line text-sm text-stone-800">{activeSession.analysis.recap}</p>
+                )}
+              </div>
+
+              <div className="grid gap-3 md:grid-cols-2">
+                <div className="rounded-xl border border-stone-200 p-4">
+                  <div className="text-sm font-semibold text-stone-900">Highlights</div>
+                  <div className="mt-3 space-y-3">
+                    {activeHighlights.length > 0 ? (
+                      activeHighlights.map(highlight => (
+                        <article key={highlight.id} className="rounded-lg border border-stone-200 bg-stone-50 p-3">
+                          <div className="flex items-center justify-between gap-2">
+                            <div className="font-medium text-stone-900">{highlight.title}</div>
+                            <span className="rounded-full bg-white px-2 py-0.5 text-[11px] text-stone-500">
+                              {(highlight.confidence * 100).toFixed(0)}%
+                            </span>
+                          </div>
+                          <p className="mt-1 text-xs uppercase tracking-wide text-stone-500">
+                            {highlight.kind} · frame {highlight.frame_index + 1} · {formatTimestamp(highlight.captured_at_ms)}
+                          </p>
+                          <p className="mt-2 text-sm text-stone-700">{highlight.rationale}</p>
+                        </article>
+                      ))
+                    ) : (
+                      <p className="text-sm text-stone-500">No highlights detected yet.</p>
+                    )}
+                  </div>
+                </div>
+
+                <div className="rounded-xl border border-stone-200 p-4">
+                  <div className="text-sm font-semibold text-stone-900">Clip candidates</div>
+                  <div className="mt-3 space-y-3">
+                    {activeClips.length > 0 ? (
+                      activeClips.map(clip => (
+                        <article key={clip.id} className="rounded-lg border border-amber-100 bg-amber-50 p-3">
+                          <div className="font-medium text-amber-950">{clip.start_label}</div>
+                          <div className="text-xs text-amber-800">through {clip.end_label}</div>
+                          <p className="mt-2 text-sm text-stone-700">{clip.rationale}</p>
+                        </article>
+                      ))
+                    ) : (
+                      <p className="text-sm text-stone-500">No clip candidates yet.</p>
+                    )}
+                  </div>
+                </div>
+              </div>
+
+              <div className="rounded-xl border border-stone-200 p-4">
+                <div className="text-sm font-semibold text-stone-900">Platform drafts</div>
+                <div className="mt-3 grid gap-3 md:grid-cols-3">
+                  {activeDrafts.length > 0 ? (
+                    activeDrafts.map(draft => (
+                      <article key={`${draft.platform}-${draft.title}`} className="rounded-lg border border-stone-200 bg-white p-3">
+                        <div className="text-xs font-semibold uppercase tracking-wide text-stone-500">
+                          {draft.platform}
+                        </div>
+                        <div className="mt-1 font-medium text-stone-900">{draft.title}</div>
+                        <p className="mt-2 text-sm text-stone-700 whitespace-pre-line">{draft.description}</p>
+                        <div className="mt-3 flex flex-wrap gap-2">
+                          {draft.tags.map(tag => (
+                            <span key={tag} className="rounded-full bg-stone-100 px-2 py-0.5 text-[11px] text-stone-600">
+                              #{tag}
+                            </span>
+                          ))}
+                        </div>
+                      </article>
+                    ))
+                  ) : (
+                    <p className="text-sm text-stone-500">Draft metadata will appear after analysis.</p>
+                  )}
+                </div>
+              </div>
+
+              <div className="rounded-xl border border-sage-200 bg-sage-50 p-4">
+                <div className="text-sm font-semibold text-sage-900">Ask this session</div>
+                <div className="mt-3 flex flex-col gap-3 lg:flex-row">
+                  <input
+                    value={question}
+                    onChange={event => setQuestion(event.target.value)}
+                    className="flex-1 rounded-xl border border-sage-200 bg-white px-3 py-2 outline-none ring-0 focus:border-primary-500"
+                    placeholder="What were my best fights?"
+                  />
+                  <button
+                    type="button"
+                    onClick={handleQuestion}
+                    disabled={questionBusy}
+                    className="rounded-xl bg-sage-700 px-4 py-2 text-sm font-semibold text-white transition hover:bg-sage-800 disabled:cursor-not-allowed disabled:opacity-60"
+                  >
+                    {questionBusy ? 'Thinking…' : 'Ask session'}
+                  </button>
+                </div>
+                {questionAnswer && (
+                  <p className="mt-3 whitespace-pre-line text-sm text-stone-800">{questionAnswer}</p>
+                )}
+                {activeSession.analysis?.follow_up_questions && activeSession.analysis.follow_up_questions.length > 0 && (
+                  <div className="mt-3 flex flex-wrap gap-2 text-xs text-stone-600">
+                    {activeSession.analysis.follow_up_questions.map(item => (
+                      <button
+                        key={item}
+                        type="button"
+                        onClick={() => setQuestion(item)}
+                        className="rounded-full border border-sage-200 bg-white px-3 py-1 transition hover:bg-sage-50"
+                      >
+                        {item}
+                      </button>
+                    ))}
+                  </div>
+                )}
+              </div>
+            </div>
+          ) : (
+            <div className="rounded-xl border border-dashed border-stone-300 bg-stone-50 p-8 text-center text-sm text-stone-500">
+              Import a folder of keyframes to create the first gameplay session.
+            </div>
+          )}
+        </section>
+
+        <aside className="space-y-4 rounded-2xl border border-stone-200 bg-white p-5 shadow-soft">
+          <div>
+            <h3 className="text-lg font-semibold text-stone-900">Session history</h3>
+            <p className="text-sm text-stone-500">Previously imported or analyzed sessions for this workspace.</p>
+          </div>
+          <div className="space-y-3">
+            {recentSessions.length > 0 ? (
+              recentSessions.map(session => (
+                <button
+                  key={session.session_id}
+                  type="button"
+                  onClick={() => setActiveSession(session)}
+                  className={`w-full rounded-xl border p-3 text-left transition ${
+                    activeSession?.session_id === session.session_id
+                      ? 'border-primary-300 bg-primary-50'
+                      : 'border-stone-200 bg-white hover:bg-stone-50'
+                  }`}
+                >
+                  <div className="flex items-start justify-between gap-3">
+                    <div>
+                      <div className="font-medium text-stone-900">{session.session_title}</div>
+                      <div className="text-xs text-stone-500">{session.game_id}</div>
+                    </div>
+                    <div className="text-right text-[11px] text-stone-500">
+                      <div>{formatSpoilerMode(session.spoiler_mode)}</div>
+                      <div>{session.frames.length} frame(s)</div>
+                    </div>
+                  </div>
+                  <div className="mt-2 text-xs text-stone-500">Imported {formatTimestamp(session.imported_at_ms)}</div>
+                </button>
+              ))
+            ) : (
+              <p className="text-sm text-stone-500">No saved sessions yet.</p>
+            )}
+          </div>
+          <button
+            type="button"
+            onClick={() => void refreshSessions()}
+            className="w-full rounded-xl border border-stone-200 bg-white px-4 py-2 text-sm font-semibold text-stone-700 transition hover:bg-stone-50"
+          >
+            Refresh history
+          </button>
+        </aside>
+      </div>
+    </div>
+  );
+}
diff --git a/app/src/pages/Intelligence.tsx b/app/src/pages/Intelligence.tsx
index 4d15ba3b71..0df2129e74 100644
--- a/app/src/pages/Intelligence.tsx
+++ b/app/src/pages/Intelligence.tsx
@@ -3,6 +3,7 @@ import { useCallback, useEffect, useState } from 'react';
 import { ConfirmationModal } from '../components/intelligence/ConfirmationModal';
 import IntelligenceCallsTab from '../components/intelligence/IntelligenceCallsTab';
 import IntelligenceDreamsTab from '../components/intelligence/IntelligenceDreamsTab';
+import { GameplayReviewWorkspace } from '../components/intelligence/GameplayReviewWorkspace';
 import IntelligenceSubconsciousTab from '../components/intelligence/IntelligenceSubconsciousTab';
 import { MemoryWorkspace } from '../components/intelligence/MemoryWorkspace';
 import { ToastContainer } from '../components/intelligence/Toast';
@@ -20,7 +21,7 @@ import type {
   ToastNotification,
 } from '../types/intelligence';
 
-type IntelligenceTab = 'memory' | 'subconscious' | 'calls' | 'dreams';
+type IntelligenceTab = 'memory' | 'gameplay' | 'subconscious' | 'calls' | 'dreams';
 
 export default function Intelligence() {
   const { aiStatus } = useIntelligenceStats();
@@ -129,6 +130,7 @@ export default function Intelligence() {
 
   const tabs: { id: IntelligenceTab; label: string; comingSoon?: boolean }[] = [
     { id: 'memory', label: 'Memory' },
+    { id: 'gameplay', label: 'Gameplay' },
     { id: 'subconscious', label: 'Subconscious' },
     { id: 'calls', label: 'Calls' },
     { id: 'dreams', label: 'Dreams', comingSoon: true },
@@ -218,6 +220,8 @@ export default function Intelligence() {
             {/* Tab content */}
             {activeTab === 'memory' && <MemoryWorkspace onToast={addToast} />}
 
+            {activeTab === 'gameplay' && <GameplayReviewWorkspace onToast={addToast} />}
+
             {activeTab === 'subconscious' && (
               <IntelligenceSubconsciousTab
                 addSubconsciousTask={addSubconsciousTask}
diff --git a/app/src/services/gameplayReviewService.ts b/app/src/services/gameplayReviewService.ts
new file mode 100644
index 0000000000..5ac8b1e37f
--- /dev/null
+++ b/app/src/services/gameplayReviewService.ts
@@ -0,0 +1,224 @@
+import debug from 'debug';
+
+import { callCoreRpc } from './coreRpcClient';
+
+const log = debug('gameplay-review');
+const errLog = debug('gameplay-review:error');
+
+export type SpoilerMode = 'off' | 'light' | 'full';
+
+export interface GameplayFrameInput {
+  file_name: string;
+  image_ref: string;
+  captured_at_ms?: number | null;
+}
+
+export interface GameplayReviewSessionInput {
+  game_id: string;
+  session_title: string;
+  source_label?: string | null;
+  spoiler_mode?: SpoilerMode | null;
+  preset_id?: string | null;
+  frames: GameplayFrameInput[];
+}
+
+export interface GameplayReviewAnalysisInput {
+  session_id: string;
+  max_highlights?: number;
+  platforms?: string[];
+}
+
+export interface GameplayPresetInput {
+  game_id: string;
+  display_name: string;
+  coaching_focus: string[];
+  audio_feedback: boolean;
+  spoiler_mode: SpoilerMode;
+  notes?: string | null;
+}
+
+export interface GameplayReviewQuestionInput {
+  session_id: string;
+  question: string;
+}
+
+export interface GameplayReviewClipInput {
+  session_id: string;
+  platform?: string | null;
+  highlight_id?: string | null;
+}
+
+export interface GameplayHighlight {
+  id: string;
+  frame_index: number;
+  captured_at_ms?: number | null;
+  title: string;
+  rationale: string;
+  confidence: number;
+  kind: 'highlight' | 'mistake' | 'coaching';
+}
+
+export interface GameplayClipCandidate {
+  id: string;
+  frame_index: number;
+  start_label: string;
+  end_label: string;
+  rationale: string;
+  confidence: number;
+}
+
+export interface GameplayPlatformDraft {
+  platform: string;
+  title: string;
+  description: string;
+  tags: string[];
+}
+
+export interface GameplayReviewAnalysis {
+  recap: string;
+  highlights: GameplayHighlight[];
+  clip_candidates: GameplayClipCandidate[];
+  draft_metadata: GameplayPlatformDraft[];
+  follow_up_questions: string[];
+  spoiler_note?: string | null;
+}
+
+export interface GameplayReviewSession {
+  session_id: string;
+  game_id: string;
+  session_title: string;
+  source_label?: string | null;
+  spoiler_mode: SpoilerMode;
+  preset_id?: string | null;
+  imported_at_ms: number;
+  analyzed_at_ms?: number | null;
+  frames: GameplayFrameInput[];
+  analysis?: GameplayReviewAnalysis | null;
+}
+
+export interface GameplayReviewQuestionResult {
+  answer: string;
+  matched_highlights: GameplayHighlight[];
+  suggested_follow_up: string[];
+}
+
+export interface PreparedGameplayFrame extends GameplayFrameInput {
+  source_name: string;
+}
+
+export async function prepareGameplayFrames(files: File[], maxFrames = 12): Promise<PreparedGameplayFrame[]> {
+  const imageFiles = files
+    .filter(file => file.type.startsWith('image/'))
+    .sort((left, right) => {
+      const leftName = (left.webkitRelativePath || left.name).toLowerCase();
+      const rightName = (right.webkitRelativePath || right.name).toLowerCase();
+      return leftName.localeCompare(rightName);
+    })
+    .slice(0, Math.max(1, maxFrames));
+
+  log('prepareGameplayFrames: selected %d image(s) from %d file(s)', imageFiles.length, files.length);
+  const prepared: PreparedGameplayFrame[] = [];
+  for (const file of imageFiles) {
+    const buffer = await file.arrayBuffer();
+    const bytes = new Uint8Array(buffer);
+    let base64 = '';
+    for (let index = 0; index < bytes.length; index += 0x8000) {
+      base64 += String.fromCharCode(...bytes.subarray(index, index + 0x8000));
+    }
+    prepared.push({
+      source_name: file.webkitRelativePath || file.name,
+      file_name: file.webkitRelativePath || file.name,
+      image_ref: `data:${file.type || 'image/png'};base64,${btoa(base64)}`,
+      captured_at_ms: file.lastModified || null,
+    });
+  }
+  return prepared;
+}
+
+export async function registerGameplaySession(payload: GameplayReviewSessionInput): Promise<GameplayReviewSession> {
+  const result = await callCoreRpc<GameplayReviewSession>({
+    method: 'openhuman.gameplay_review_register_session',
+    params: payload,
+  });
+  return result;
+}
+
+export async function analyzeGameplaySession(
+  payload: GameplayReviewAnalysisInput
+): Promise<GameplayReviewSession> {
+  const result = await callCoreRpc<GameplayReviewSession>({
+    method: 'openhuman.gameplay_review_analyze_session',
+    params: payload,
+  });
+  return result;
+}
+
+export async function listGameplaySessions(gameId?: string): Promise<GameplayReviewSession[]> {
+  const result = await callCoreRpc<GameplayReviewSession[]>({
+    method: 'openhuman.gameplay_review_list_sessions',
+    params: gameId ? { game_id: gameId } : {},
+  });
+  return result;
+}
+
+export async function saveGameplayPreset(payload: GameplayPresetInput): Promise<unknown> {
+  return callCoreRpc({
+    method: 'openhuman.gameplay_review_set_preset',
+    params: payload,
+  });
+}
+
+export async function listGameplayPresets(): Promise<GameplayPresetInput[]> {
+  return callCoreRpc<GameplayPresetInput[]>({ method: 'openhuman.gameplay_review_list_presets' });
+}
+
+export async function askGameplaySession(
+  payload: GameplayReviewQuestionInput
+): Promise<GameplayReviewQuestionResult> {
+  return callCoreRpc<GameplayReviewQuestionResult>({
+    method: 'openhuman.gameplay_review_ask_session',
+    params: payload,
+  });
+}
+
+export async function draftGameplayClipMetadata(
+  payload: GameplayReviewClipInput
+): Promise<GameplayPlatformDraft[]> {
+  return callCoreRpc<GameplayPlatformDraft[]>({
+    method: 'openhuman.gameplay_review_draft_clip_metadata',
+    params: payload,
+  });
+}
+
+export function flattenHighlights(session: GameplayReviewSession | null): GameplayHighlight[] {
+  return session?.analysis?.highlights ?? [];
+}
+
+export function flattenDrafts(session: GameplayReviewSession | null): GameplayPlatformDraft[] {
+  return session?.analysis?.draft_metadata ?? [];
+}
+
+export function flattenClipCandidates(session: GameplayReviewSession | null): GameplayClipCandidate[] {
+  return session?.analysis?.clip_candidates ?? [];
+}
+
+export function formatSpoilerMode(mode: SpoilerMode): string {
+  switch (mode) {
+    case 'off':
+      return 'Spoiler-safe';
+    case 'full':
+      return 'Full spoilers';
+    default:
+      return 'Light spoilers';
+  }
+}
+
+export function normalizeGameplayError(error: unknown): string {
+  if (error instanceof Error) return error.message;
+  if (typeof error === 'string') return error;
+  return 'Gameplay review failed';
+}
+
+export function logGameplayError(message: string, error: unknown): void {
+  errLog(message, error);
+}
diff --git a/src/core/all.rs b/src/core/all.rs
index d9d48251ff..b411b329f8 100644
--- a/src/core/all.rs
+++ b/src/core/all.rs
@@ -153,6 +153,10 @@ fn build_registered_controllers() -> Vec<RegisteredController> {
     controllers.extend(
         crate::openhuman::screen_intelligence::all_screen_intelligence_registered_controllers(),
     );
+    // Desktop gameplay review workflow
+    controllers.extend(
+        crate::openhuman::gameplay_review::all_gameplay_review_registered_controllers(),
+    );
     // Bridge to external skill runtimes
     controllers.extend(crate::openhuman::socket::all_socket_registered_controllers());
     // Discovered SKILL.md skills and their bundled resources
@@ -255,6 +259,7 @@ fn build_declared_controller_schemas() -> Vec<ControllerSchema> {
     schemas
         .extend(crate::openhuman::channels::providers::web::all_web_channel_controller_schemas());
     schemas.extend(crate::openhuman::channels::controllers::all_channels_controller_schemas());
+    schemas.extend(crate::openhuman::gameplay_review::all_gameplay_review_controller_schemas());
     schemas.extend(crate::openhuman::config::all_config_controller_schemas());
     schemas.extend(crate::openhuman::credentials::all_credentials_controller_schemas());
     schemas.extend(crate::openhuman::service::all_service_controller_schemas());
diff --git a/src/openhuman/about_app/catalog.rs b/src/openhuman/about_app/catalog.rs
index 09f8bfb658..2c2639646b 100644
--- a/src/openhuman/about_app/catalog.rs
+++ b/src/openhuman/about_app/catalog.rs
@@ -652,6 +652,56 @@ const CAPABILITIES: &[Capability] = &[
         status: CapabilityStatus::Beta,
         privacy: LOCAL_RAW,
     },
+    Capability {
+        id: "screen_intelligence.gameplay_review_import",
+        name: "Import Gameplay Review Sessions",
+        domain: "screen_intelligence",
+        category: CapabilityCategory::ScreenIntelligence,
+        description: "Import local gameplay keyframes or a recordings folder into a review session.",
+        how_to: "Intelligence > Gameplay > Import session",
+        status: CapabilityStatus::Beta,
+        privacy: LOCAL_RAW,
+    },
+    Capability {
+        id: "screen_intelligence.gameplay_review_analyze",
+        name: "Analyze Gameplay Sessions",
+        domain: "screen_intelligence",
+        category: CapabilityCategory::ScreenIntelligence,
+        description: "Generate a recap, highlight list, and review notes from imported gameplay frames.",
+        how_to: "Intelligence > Gameplay > Analyze session",
+        status: CapabilityStatus::Beta,
+        privacy: LOCAL_RAW,
+    },
+    Capability {
+        id: "screen_intelligence.gameplay_review_clip_metadata",
+        name: "Draft Clip Metadata",
+        domain: "screen_intelligence",
+        category: CapabilityCategory::ScreenIntelligence,
+        description: "Generate platform-ready titles, descriptions, and tags for gameplay clips.",
+        how_to: "Intelligence > Gameplay > Draft clips",
+        status: CapabilityStatus::Beta,
+        privacy: LOCAL_RAW,
+    },
+    Capability {
+        id: "screen_intelligence.gameplay_review_presets",
+        name: "Save Game Coaching Presets",
+        domain: "screen_intelligence",
+        category: CapabilityCategory::ScreenIntelligence,
+        description: "Store game-specific coaching focus, spoiler mode, and notes for repeat reviews.",
+        how_to: "Intelligence > Gameplay > Presets",
+        status: CapabilityStatus::Beta,
+        privacy: LOCAL_RAW,
+    },
+    Capability {
+        id: "screen_intelligence.gameplay_review_spoilers",
+        name: "Set Spoiler Controls",
+        domain: "screen_intelligence",
+        category: CapabilityCategory::ScreenIntelligence,
+        description: "Keep gameplay coaching spoiler-safe or story-aware per session.",
+        how_to: "Intelligence > Gameplay > Spoiler mode",
+        status: CapabilityStatus::Beta,
+        privacy: LOCAL_RAW,
+    },
     Capability {
         id: "screen_intelligence.configure_capture_fps",
         name: "Configure Capture FPS",
diff --git a/src/openhuman/gameplay_review/mod.rs b/src/openhuman/gameplay_review/mod.rs
new file mode 100644
index 0000000000..cfd0dbb02d
--- /dev/null
+++ b/src/openhuman/gameplay_review/mod.rs
@@ -0,0 +1,14 @@
+//! Gameplay session review, highlight detection, and clip drafting.
+
+pub mod ops;
+mod schemas;
+mod store;
+pub mod types;
+
+pub use ops as rpc;
+pub use ops::*;
+pub use schemas::{
+    all_controller_schemas as all_gameplay_review_controller_schemas,
+    all_registered_controllers as all_gameplay_review_registered_controllers,
+};
+pub use types::*;
diff --git a/src/openhuman/gameplay_review/ops.rs b/src/openhuman/gameplay_review/ops.rs
new file mode 100644
index 0000000000..16f01a7bb6
--- /dev/null
+++ b/src/openhuman/gameplay_review/ops.rs
@@ -0,0 +1,630 @@
+use std::cmp::Ordering;
+use std::collections::BTreeSet;
+use std::path::Path;
+
+use chrono::Utc;
+use uuid::Uuid;
+
+use crate::openhuman::config::Config;
+use crate::openhuman::screen_intelligence::{global_engine, CaptureFrame};
+use crate::rpc::RpcOutcome;
+
+use super::store;
+use super::types::{
+    GameplayClipCandidate, GameplayFrameInput, GameplayHighlight, GameplayPlatformDraft,
+    GameplayPresetInput, GameplayReviewAnalysis, GameplayReviewAnalysisInput,
+    GameplayReviewClipInput, GameplayReviewPreset, GameplayReviewQuestionInput,
+    GameplayReviewQuestionResult, GameplayReviewSession, GameplayReviewSessionInput, HighlightKind,
+    SpoilerMode,
+};
+
+const DEFAULT_PLATFORMS: &[&str] = &["twitch", "kick", "youtube"];
+
+pub async fn register_session(
+    payload: GameplayReviewSessionInput,
+) -> Result<RpcOutcome<GameplayReviewSession>, String> {
+    let session = build_session(payload)?;
+    let workspace_dir = workspace_dir().await?;
+    store::save_session(&workspace_dir, &session)?;
+    Ok(RpcOutcome::single_log(
+        session,
+        "gameplay review session registered",
+    ))
+}
+
+pub async fn analyze_session(
+    payload: GameplayReviewAnalysisInput,
+) -> Result<RpcOutcome<GameplayReviewSession>, String> {
+    let workspace_dir = workspace_dir().await?;
+    let mut session = store::load_session(&workspace_dir, &payload.session_id)?
+        .ok_or_else(|| format!("gameplay session not found: {}", payload.session_id))?;
+    let preset = match session.preset_id.as_deref() {
+        Some(game_id) => store::load_preset(&workspace_dir, game_id)?,
+        None => None,
+    };
+    let analysis = analyze_session_frames(&session, preset.as_ref(), payload.max_highlights, &payload.platforms).await;
+    session.analyzed_at_ms = Some(Utc::now().timestamp_millis());
+    session.analysis = Some(analysis);
+    store::save_session(&workspace_dir, &session)?;
+    Ok(RpcOutcome::single_log(
+        session,
+        "gameplay review session analyzed",
+    ))
+}
+
+pub async fn get_session(session_id: String) -> Result<RpcOutcome<GameplayReviewSession>, String> {
+    let workspace_dir = workspace_dir().await?;
+    let session = store::load_session(&workspace_dir, &session_id)?
+        .ok_or_else(|| format!("gameplay session not found: {session_id}"))?;
+    Ok(RpcOutcome::single_log(
+        session,
+        "gameplay review session fetched",
+    ))
+}
+
+pub async fn list_sessions(
+    game_id: Option<String>,
+) -> Result<RpcOutcome<Vec<GameplayReviewSession>>, String> {
+    let workspace_dir = workspace_dir().await?;
+    let mut sessions = store::list_sessions(&workspace_dir)?;
+    if let Some(filter) = game_id.as_deref() {
+        sessions.retain(|session| session.game_id == filter);
+    }
+    Ok(RpcOutcome::single_log(
+        sessions,
+        "gameplay review sessions listed",
+    ))
+}
+
+pub async fn set_preset(
+    payload: GameplayPresetInput,
+) -> Result<RpcOutcome<GameplayReviewPreset>, String> {
+    let workspace_dir = workspace_dir().await?;
+    let preset = store::preset_from_input(payload);
+    store::save_preset(&workspace_dir, &preset)?;
+    Ok(RpcOutcome::single_log(
+        preset,
+        "gameplay review preset saved",
+    ))
+}
+
+pub async fn list_presets() -> Result<RpcOutcome<Vec<GameplayReviewPreset>>, String> {
+    let workspace_dir = workspace_dir().await?;
+    let presets = store::list_presets(&workspace_dir)?;
+    Ok(RpcOutcome::single_log(
+        presets,
+        "gameplay review presets listed",
+    ))
+}
+
+pub async fn ask_session(
+    payload: GameplayReviewQuestionInput,
+) -> Result<RpcOutcome<GameplayReviewQuestionResult>, String> {
+    let workspace_dir = workspace_dir().await?;
+    let session = store::load_session(&workspace_dir, &payload.session_id)?
+        .ok_or_else(|| format!("gameplay session not found: {}", payload.session_id))?;
+    let analysis = session
+        .analysis
+        .clone()
+        .unwrap_or_else(|| GameplayReviewAnalysis {
+            recap: build_recap(&session, &[], None),
+            highlights: Vec::new(),
+            clip_candidates: Vec::new(),
+            draft_metadata: Vec::new(),
+            follow_up_questions: default_follow_up_questions(&session),
+            spoiler_note: None,
+        });
+    let answer = answer_question(&session, analysis.clone(), &payload.question);
+    let matched_highlights = matched_highlights(&analysis.highlights, &payload.question);
+    let suggested_follow_up = if analysis.follow_up_questions.is_empty() {
+        default_follow_up_questions(&session)
+    } else {
+        analysis.follow_up_questions.clone()
+    };
+    Ok(RpcOutcome::single_log(
+        GameplayReviewQuestionResult {
+            answer,
+            matched_highlights,
+            suggested_follow_up,
+        },
+        "gameplay review question answered",
+    ))
+}
+
+pub async fn draft_clip_metadata(
+    payload: GameplayReviewClipInput,
+) -> Result<RpcOutcome<Vec<GameplayPlatformDraft>>, String> {
+    let workspace_dir = workspace_dir().await?;
+    let session = store::load_session(&workspace_dir, &payload.session_id)?
+        .ok_or_else(|| format!("gameplay session not found: {}", payload.session_id))?;
+    let analysis = session
+        .analysis
+        .as_ref()
+        .ok_or_else(|| "session has not been analyzed yet".to_string())?;
+    let drafts = draft_metadata(&session, analysis, payload.platform.as_deref(), payload.highlight_id.as_deref());
+    Ok(RpcOutcome::single_log(
+        drafts,
+        "gameplay review clip metadata drafted",
+    ))
+}
+
+async fn workspace_dir() -> Result<std::path::PathBuf, String> {
+    let config = Config::load_or_init()
+        .await
+        .map_err(|err| format!("gameplay review config load failed: {err}"))?;
+    Ok(store::workspace_dir_from_config(&config))
+}
+
+fn build_session(payload: GameplayReviewSessionInput) -> Result<GameplayReviewSession, String> {
+    if payload.game_id.trim().is_empty() {
+        return Err("game_id is required".to_string());
+    }
+    if payload.session_title.trim().is_empty() {
+        return Err("session_title is required".to_string());
+    }
+    if payload.frames.is_empty() {
+        return Err("at least one frame is required".to_string());
+    }
+
+    let now = Utc::now().timestamp_millis();
+    let session_id = format!(
+        "gameplay-{}-{}",
+        store::slugify(&payload.game_id),
+        Uuid::new_v4().simple()
+    );
+    Ok(GameplayReviewSession {
+        session_id,
+        game_id: payload.game_id.trim().to_string(),
+        session_title: payload.session_title.trim().to_string(),
+        source_label: payload.source_label.map(|value| value.trim().to_string()).filter(|value| !value.is_empty()),
+        spoiler_mode: payload.spoiler_mode.unwrap_or_default(),
+        preset_id: payload.preset_id.map(|value| value.trim().to_string()).filter(|value| !value.is_empty()),
+        imported_at_ms: now,
+        analyzed_at_ms: None,
+        frames: payload.frames,
+        analysis: None,
+    })
+}
+
+async fn analyze_session_frames(
+    session: &GameplayReviewSession,
+    preset: Option<&GameplayReviewPreset>,
+    max_highlights: Option<usize>,
+    platform_overrides: &[String],
+) -> GameplayReviewAnalysis {
+    let limit = max_highlights.unwrap_or(5).clamp(1, 8);
+    let mut highlights = Vec::new();
+    let mut clip_candidates = Vec::new();
+    let platforms: Vec<String> = if platform_overrides.is_empty() {
+        DEFAULT_PLATFORMS
+            .iter()
+            .map(|platform| (*platform).to_string())
+            .collect()
+    } else {
+        platform_overrides.to_vec()
+    };
+
+    for (index, frame) in session.frames.iter().enumerate() {
+        let capture = CaptureFrame {
+            captured_at_ms: frame
+                .captured_at_ms
+                .unwrap_or(session.imported_at_ms.saturating_add((index as i64) * 1000)),
+            reason: "gameplay_review_import".to_string(),
+            app_name: Some(session.game_id.clone()),
+            window_title: Some(session.session_title.clone()),
+            image_ref: Some(frame.image_ref.clone()),
+        };
+
+        let summary = match global_engine().analyze_and_persist_frame(capture).await {
+            Ok(summary) => summary,
+            Err(err) => {
+                log::warn!(
+                    "[gameplay_review] frame analysis fell back to deterministic summary: {}",
+                    err
+                );
+                fallback_summary(session, frame, index)
+            }
+        };
+
+        let kind = classify_kind(&summary.key_text, &summary.actionable_notes, session.spoiler_mode);
+        let title = highlight_title(&summary.key_text, &frame.file_name, kind);
+        let rationale = if summary.actionable_notes.trim().is_empty() {
+            summary.key_text.clone()
+        } else {
+            summary.actionable_notes.clone()
+        };
+        let confidence = summary.confidence.clamp(0.0, 1.0);
+        let highlight = GameplayHighlight {
+            id: format!("{}-{}", session.session_id, index),
+            frame_index: index,
+            captured_at_ms: Some(summary.captured_at_ms),
+            title,
+            rationale,
+            confidence,
+            kind,
+        };
+        highlights.push(highlight);
+    }
+
+    highlights.sort_by(|left, right| {
+        right
+            .confidence
+            .partial_cmp(&left.confidence)
+            .unwrap_or(Ordering::Equal)
+            .then(left.frame_index.cmp(&right.frame_index))
+    });
+    highlights.truncate(limit);
+
+    for highlight in &highlights {
+        clip_candidates.push(clip_candidate_from_highlight(session, highlight));
+    }
+
+    let recap = build_recap(session, &highlights, preset);
+    let draft_metadata = draft_metadata_from_highlights(session, &highlights, preset, &platforms);
+
+    GameplayReviewAnalysis {
+        recap,
+        highlights,
+        clip_candidates,
+        draft_metadata,
+        follow_up_questions: default_follow_up_questions(session),
+        spoiler_note: spoiler_note(session.spoiler_mode),
+    }
+}
+
+fn fallback_summary(
+    session: &GameplayReviewSession,
+    frame: &GameplayFrameInput,
+    index: usize,
+) -> crate::openhuman::screen_intelligence::VisionSummary {
+    let label = frame
+        .file_name
+        .rsplit_once('/')
+        .map(|(_, tail)| tail)
+        .unwrap_or(&frame.file_name)
+        .trim();
+    let captured_at_ms = frame
+        .captured_at_ms
+        .unwrap_or(session.imported_at_ms.saturating_add((index as i64) * 1000));
+    crate::openhuman::screen_intelligence::VisionSummary {
+        id: format!("gameplay-{}-{}", captured_at_ms, Uuid::new_v4()),
+        captured_at_ms,
+        app_name: Some(session.game_id.clone()),
+        window_title: Some(session.session_title.clone()),
+        ui_state: format!("Gameplay frame {index}: {label}"),
+        key_text: format!("Gameplay moment from {}", label),
+        actionable_notes: if session.spoiler_mode == SpoilerMode::Off {
+            "Keep the coaching spoiler-safe.".to_string()
+        } else {
+            "Review the clip for highlights and mistakes.".to_string()
+        },
+        confidence: 0.65,
+    }
+}
+
+fn classify_kind(key_text: &str, notes: &str, spoiler_mode: SpoilerMode) -> HighlightKind {
+    let text = format!("{} {}", key_text, notes).to_ascii_lowercase();
+    if text.contains("mistake") || text.contains("miss") || text.contains("throw") {
+        return HighlightKind::Mistake;
+    }
+    if text.contains("clutch") || text.contains("highlight") || text.contains("fight") || text.contains("kill") {
+        return HighlightKind::Highlight;
+    }
+    if matches!(spoiler_mode, SpoilerMode::Off) && text.contains("story") {
+        return HighlightKind::Coaching;
+    }
+    HighlightKind::Coaching
+}
+
+fn highlight_title(key_text: &str, file_name: &str, kind: HighlightKind) -> String {
+    let source = if key_text.trim().is_empty() { file_name } else { key_text };
+    let base = source
+        .split(['.', '!', '?', '\n'])
+        .next()
+        .unwrap_or(source)
+        .trim();
+    let prefix = match kind {
+        HighlightKind::Highlight => "Highlight",
+        HighlightKind::Mistake => "Mistake",
+        HighlightKind::Coaching => "Review",
+    };
+    format!("{prefix}: {}", truncate(base, 72))
+}
+
+fn clip_candidate_from_highlight(
+    session: &GameplayReviewSession,
+    highlight: &GameplayHighlight,
+) -> GameplayClipCandidate {
+    let start_index = highlight.frame_index.saturating_sub(1);
+    let end_index = (highlight.frame_index + 1).min(session.frames.len().saturating_sub(1));
+    GameplayClipCandidate {
+        id: format!("clip-{}", highlight.id),
+        frame_index: highlight.frame_index,
+        start_label: session
+            .frames
+            .get(start_index)
+            .map(|frame| frame.file_name.clone())
+            .unwrap_or_else(|| format!("frame-{start_index}")),
+        end_label: session
+            .frames
+            .get(end_index)
+            .map(|frame| frame.file_name.clone())
+            .unwrap_or_else(|| format!("frame-{end_index}")),
+        rationale: highlight.rationale.clone(),
+        confidence: highlight.confidence,
+    }
+}
+
+fn draft_metadata(
+    session: &GameplayReviewSession,
+    analysis: &GameplayReviewAnalysis,
+    platform: Option<&str>,
+    highlight_id: Option<&str>,
+) -> Vec<GameplayPlatformDraft> {
+    let filtered = highlight_id
+        .and_then(|needle| analysis.highlights.iter().find(|highlight| highlight.id == needle))
+        .cloned()
+        .or_else(|| analysis.highlights.first().cloned());
+    let highlights = filtered.into_iter().collect::<Vec<_>>();
+    let platforms = match platform {
+        Some(value) if !value.trim().is_empty() => vec![value.trim().to_string()],
+        _ => DEFAULT_PLATFORMS
+            .iter()
+            .map(|platform| (*platform).to_string())
+            .collect(),
+    };
+    draft_metadata_from_highlights(session, &highlights, None, &platforms)
+}
+
+fn draft_metadata_from_highlights(
+    session: &GameplayReviewSession,
+    highlights: &[GameplayHighlight],
+    preset: Option<&GameplayReviewPreset>,
+    platforms: &[String],
+) -> Vec<GameplayPlatformDraft> {
+    let best = highlights.first();
+    let best_title = best
+        .map(|highlight| highlight.title.clone())
+        .unwrap_or_else(|| format!("{} recap", session.session_title));
+    let focus = preset
+        .and_then(|preset| preset.coaching_focus.first())
+        .cloned()
+        .unwrap_or_else(|| "clean execution".to_string());
+
+    platforms
+        .iter()
+        .map(|platform| GameplayPlatformDraft {
+            platform: platform.clone(),
+            title: format!("{} — {} ({})", session.game_id, truncate(&best_title, 64), platform),
+            description: format!(
+                "Session: {}\nFocus: {}\nTop moment: {}\nSpoiler mode: {}",
+                session.session_title,
+                focus,
+                best.map(|highlight| highlight.rationale.clone()).unwrap_or_else(|| "No highlight selected yet.".to_string()),
+                session.spoiler_mode.as_str(),
+            ),
+            tags: build_tags(session, best, preset, platform),
+        })
+        .collect()
+}
+
+fn build_tags(
+    session: &GameplayReviewSession,
+    highlight: Option<&GameplayHighlight>,
+    preset: Option<&GameplayReviewPreset>,
+    platform: &str,
+) -> Vec<String> {
+    let mut tags = BTreeSet::new();
+    tags.insert(store::slugify(&session.game_id));
+    tags.insert(store::slugify(platform));
+    if let Some(highlight) = highlight {
+        tags.insert(store::slugify(&highlight.title));
+    }
+    if let Some(preset) = preset {
+        for focus in &preset.coaching_focus {
+            tags.insert(store::slugify(focus));
+        }
+    }
+    tags.into_iter().take(6).collect()
+}
+
+fn build_recap(
+    session: &GameplayReviewSession,
+    highlights: &[GameplayHighlight],
+    preset: Option<&GameplayReviewPreset>,
+) -> String {
+    let mut recap = format!(
+        "Gameplay recap for {} ({})\nSpoiler mode: {}\nFrames reviewed: {}",
+        session.session_title,
+        session.game_id,
+        session.spoiler_mode.as_str(),
+        session.frames.len()
+    );
+    if let Some(preset) = preset {
+        recap.push_str(&format!("\nPreset: {}", preset.display_name));
+        if !preset.coaching_focus.is_empty() {
+            recap.push_str(&format!("\nCoaching focus: {}", preset.coaching_focus.join(", ")));
+        }
+    }
+    if highlights.is_empty() {
+        recap.push_str("\nNo strong highlights were detected yet.");
+    } else {
+        recap.push_str("\nTop moments:");
+        for highlight in highlights.iter().take(5) {
+            recap.push_str(&format!(
+                "\n- [{}] {} — {}",
+                highlight.kind.as_str(),
+                highlight.title,
+                highlight.rationale
+            ));
+        }
+    }
+    recap
+}
+
+fn default_follow_up_questions(session: &GameplayReviewSession) -> Vec<String> {
+    vec![
+        format!("What were my best moments in {}?", session.session_title),
+        format!("Where did I make mistakes in {}?", session.session_title),
+        format!("What should I post from this {} session?", session.game_id),
+    ]
+}
+
+fn spoiler_note(mode: SpoilerMode) -> Option<String> {
+    match mode {
+        SpoilerMode::Off => Some("Spoiler-safe mode is enabled; avoid story reveals.".to_string()),
+        SpoilerMode::Light => Some("Light spoiler filtering is enabled; keep story beats vague.".to_string()),
+        SpoilerMode::Full => None,
+    }
+}
+
+fn matched_highlights(highlights: &[GameplayHighlight], question: &str) -> Vec<GameplayHighlight> {
+    let lowered = question.to_ascii_lowercase();
+    let wants_mistakes = lowered.contains("mistake") || lowered.contains("throw") || lowered.contains("bad");
+    let wants_highlights = lowered.contains("best") || lowered.contains("highlight") || lowered.contains("clip") || lowered.contains("post");
+    highlights
+        .iter()
+        .filter(|highlight| {
+            (wants_mistakes && matches!(highlight.kind, HighlightKind::Mistake | HighlightKind::Coaching))
+                || (wants_highlights && matches!(highlight.kind, HighlightKind::Highlight))
+                || (!wants_mistakes && !wants_highlights)
+        })
+        .take(3)
+        .cloned()
+        .collect()
+}
+
+fn answer_question(
+    session: &GameplayReviewSession,
+    analysis: GameplayReviewAnalysis,
+    question: &str,
+) -> String {
+    let lowered = question.to_ascii_lowercase();
+    if lowered.contains("best") || lowered.contains("highlight") || lowered.contains("clip") {
+        if let Some(highlight) = analysis.highlights.first() {
+            return format!(
+                "Best clip candidate for {}: {} — {}",
+                session.session_title, highlight.title, highlight.rationale
+            );
+        }
+    }
+    if lowered.contains("mistake") || lowered.contains("throw") || lowered.contains("miss") {
+        if let Some(highlight) = analysis
+            .highlights
+            .iter()
+            .find(|highlight| matches!(highlight.kind, HighlightKind::Mistake | HighlightKind::Coaching))
+        {
+            return format!(
+                "Review point for {}: {} — {}",
+                session.session_title, highlight.title, highlight.rationale
+            );
+        }
+    }
+    if lowered.contains("post") || lowered.contains("title") || lowered.contains("description") || lowered.contains("tags") {
+        if let Some(draft) = analysis.draft_metadata.first() {
+            return format!(
+                "{} draft for {}: {}. Tags: {}",
+                draft.platform,
+                session.session_title,
+                draft.title,
+                draft.tags.join(", ")
+            );
+        }
+    }
+    format!("{}\n\n{}", analysis.recap, question)
+}
+
+fn truncate(value: &str, max_chars: usize) -> String {
+    let chars: Vec<char> = value.chars().collect();
+    if chars.len() <= max_chars {
+        value.to_string()
+    } else {
+        chars[..max_chars].iter().collect()
+    }
+}
+
+impl HighlightKind {
+    pub const fn as_str(self) -> &'static str {
+        match self {
+            Self::Highlight => "highlight",
+            Self::Mistake => "mistake",
+            Self::Coaching => "coaching",
+        }
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+    use crate::openhuman::config::TEST_ENV_LOCK as ENV_LOCK;
+    use tempfile::tempdir;
+
+    fn mock_frame(name: &str, image_ref: &str, captured_at_ms: i64) -> GameplayFrameInput {
+        GameplayFrameInput {
+            file_name: name.to_string(),
+            image_ref: image_ref.to_string(),
+            captured_at_ms: Some(captured_at_ms),
+        }
+    }
+
+    #[tokio::test]
+    async fn register_and_list_sessions_round_trip() {
+        let _guard = ENV_LOCK.lock().unwrap();
+        let tmp = tempdir().unwrap();
+        unsafe { std::env::set_var("OPENHUMAN_WORKSPACE", tmp.path()); }
+
+        let registered = register_session(GameplayReviewSessionInput {
+            game_id: "Apex Legends".to_string(),
+            session_title: "Ranked climb".to_string(),
+            source_label: Some("/recordings/apex".to_string()),
+            spoiler_mode: Some(SpoilerMode::Light),
+            preset_id: None,
+            frames: vec![mock_frame("frame1.png", "data:image/png;base64,AAA", 1000)],
+        })
+        .await
+        .expect("register")
+        .value;
+
+        assert_eq!(registered.game_id, "Apex Legends");
+        let listed = list_sessions(None).await.expect("list").value;
+        assert_eq!(listed.len(), 1);
+        assert_eq!(listed[0].session_id, registered.session_id);
+
+        unsafe { std::env::remove_var("OPENHUMAN_WORKSPACE"); }
+    }
+
+    #[test]
+    fn question_matching_prefers_highlights_for_best_prompt() {
+        let session = GameplayReviewSession {
+            session_id: "session-1".to_string(),
+            game_id: "Game".to_string(),
+            session_title: "Session".to_string(),
+            source_label: None,
+            spoiler_mode: SpoilerMode::Light,
+            preset_id: None,
+            imported_at_ms: 1000,
+            analyzed_at_ms: None,
+            frames: vec![],
+            analysis: None,
+        };
+        let analysis = GameplayReviewAnalysis {
+            recap: "recap".to_string(),
+            highlights: vec![GameplayHighlight {
+                id: "h1".to_string(),
+                frame_index: 0,
+                captured_at_ms: Some(1),
+                title: "Clutch finish".to_string(),
+                rationale: "A clean finish".to_string(),
+                confidence: 0.95,
+                kind: HighlightKind::Highlight,
+            }],
+            clip_candidates: Vec::new(),
+            draft_metadata: Vec::new(),
+            follow_up_questions: vec!["What was the turning point?".to_string()],
+            spoiler_note: None,
+        };
+        let answer = answer_question(&session, analysis.clone(), "What were my best plays?");
+        assert!(answer.contains("Clutch finish"));
+        assert_eq!(matched_highlights(&analysis.highlights, "best").len(), 1);
+    }
+}
diff --git a/src/openhuman/gameplay_review/schemas.rs b/src/openhuman/gameplay_review/schemas.rs
new file mode 100644
index 0000000000..a831d2873a
--- /dev/null
+++ b/src/openhuman/gameplay_review/schemas.rs
@@ -0,0 +1,178 @@
+use serde::de::DeserializeOwned;
+use serde::Deserialize;
+use serde_json::{Map, Value};
+
+use crate::core::all::{ControllerFuture, RegisteredController};
+use crate::core::{ControllerSchema, FieldSchema, TypeSchema};
+use crate::rpc::RpcOutcome;
+
+use super::types::{GameplayPresetInput, GameplayReviewAnalysisInput, GameplayReviewClipInput, GameplayReviewQuestionInput, GameplayReviewSessionInput};
+
+#[derive(Deserialize)]
+struct SessionIdParams {
+    session_id: String,
+}
+
+pub fn all_controller_schemas() -> Vec<ControllerSchema> {
+    vec![
+        schemas("register_session"),
+        schemas("analyze_session"),
+        schemas("get_session"),
+        schemas("list_sessions"),
+        schemas("set_preset"),
+        schemas("list_presets"),
+        schemas("ask_session"),
+        schemas("draft_clip_metadata"),
+    ]
+}
+
+pub fn all_registered_controllers() -> Vec<RegisteredController> {
+    vec![
+        RegisteredController { schema: schemas("register_session"), handler: handle_register_session },
+        RegisteredController { schema: schemas("analyze_session"), handler: handle_analyze_session },
+        RegisteredController { schema: schemas("get_session"), handler: handle_get_session },
+        RegisteredController { schema: schemas("list_sessions"), handler: handle_list_sessions },
+        RegisteredController { schema: schemas("set_preset"), handler: handle_set_preset },
+        RegisteredController { schema: schemas("list_presets"), handler: handle_list_presets },
+        RegisteredController { schema: schemas("ask_session"), handler: handle_ask_session },
+        RegisteredController { schema: schemas("draft_clip_metadata"), handler: handle_draft_clip_metadata },
+    ]
+}
+
+pub fn schemas(function: &str) -> ControllerSchema {
+    match function {
+        "register_session" => ControllerSchema {
+            namespace: "gameplay_review",
+            function: "register_session",
+            description: "Register a gameplay session from imported keyframes.",
+            inputs: vec![FieldSchema { name: "session", ty: TypeSchema::Ref("GameplayReviewSessionInput"), comment: "Imported gameplay session metadata and frames.", required: true }],
+            outputs: vec![json_output("session", "Stored gameplay review session.")],
+        },
+        "analyze_session" => ControllerSchema {
+            namespace: "gameplay_review",
+            function: "analyze_session",
+            description: "Analyze a gameplay session, generate highlights, and draft clip metadata.",
+            inputs: vec![FieldSchema { name: "analysis", ty: TypeSchema::Ref("GameplayReviewAnalysisInput"), comment: "Analysis request with optional highlight cap and platform targets.", required: true }],
+            outputs: vec![json_output("session", "Analyzed gameplay review session.")],
+        },
+        "get_session" => ControllerSchema {
+            namespace: "gameplay_review",
+            function: "get_session",
+            description: "Fetch one gameplay review session by id.",
+            inputs: vec![FieldSchema { name: "session_id", ty: TypeSchema::String, comment: "Session identifier.", required: true }],
+            outputs: vec![json_output("session", "Gameplay review session.")],
+        },
+        "list_sessions" => ControllerSchema {
+            namespace: "gameplay_review",
+            function: "list_sessions",
+            description: "List gameplay review sessions stored in the workspace.",
+            inputs: vec![FieldSchema { name: "game_id", ty: TypeSchema::Option(Box::new(TypeSchema::String)), comment: "Optional game filter.", required: false }],
+            outputs: vec![json_output("sessions", "Gameplay review sessions.")],
+        },
+        "set_preset" => ControllerSchema {
+            namespace: "gameplay_review",
+            function: "set_preset",
+            description: "Save a game-specific coaching preset.",
+            inputs: vec![FieldSchema { name: "preset", ty: TypeSchema::Ref("GameplayPresetInput"), comment: "Game preset definition.", required: true }],
+            outputs: vec![json_output("preset", "Saved gameplay review preset.")],
+        },
+        "list_presets" => ControllerSchema {
+            namespace: "gameplay_review",
+            function: "list_presets",
+            description: "List stored gameplay coaching presets.",
+            inputs: vec![],
+            outputs: vec![json_output("presets", "Gameplay review presets.")],
+        },
+        "ask_session" => ControllerSchema {
+            namespace: "gameplay_review",
+            function: "ask_session",
+            description: "Ask a question against a stored gameplay session.",
+            inputs: vec![FieldSchema { name: "question", ty: TypeSchema::Ref("GameplayReviewQuestionInput"), comment: "Question and session id.", required: true }],
+            outputs: vec![json_output("answer", "Question answer with matched highlights.")],
+        },
+        "draft_clip_metadata" => ControllerSchema {
+            namespace: "gameplay_review",
+            function: "draft_clip_metadata",
+            description: "Draft clip titles, descriptions, and tags for one gameplay highlight.",
+            inputs: vec![FieldSchema { name: "clip", ty: TypeSchema::Ref("GameplayReviewClipInput"), comment: "Clip draft request.", required: true }],
+            outputs: vec![json_output("drafts", "Draft metadata for clip publishing.")],
+        },
+        _ => ControllerSchema {
+            namespace: "gameplay_review",
+            function: "unknown",
+            description: "Unknown gameplay_review controller function.",
+            inputs: vec![],
+            outputs: vec![json_output("error", "Lookup error details.")],
+        },
+    }
+}
+
+fn json_output(name: &'static str, comment: &'static str) -> FieldSchema {
+    FieldSchema { name, ty: TypeSchema::Json, comment, required: true }
+}
+
+fn deserialize_params<T: DeserializeOwned>(params: Map<String, Value>) -> Result<T, String> {
+    serde_json::from_value(Value::Object(params)).map_err(|err| err.to_string())
+}
+
+fn handle_register_session(params: Map<String, Value>) -> ControllerFuture {
+    Box::pin(async move {
+        let payload = deserialize_params::<GameplayReviewSessionInput>(params)?;
+        to_json(crate::openhuman::gameplay_review::rpc::register_session(payload).await?)
+    })
+}
+
+fn handle_analyze_session(params: Map<String, Value>) -> ControllerFuture {
+    Box::pin(async move {
+        let payload = deserialize_params::<GameplayReviewAnalysisInput>(params)?;
+        to_json(crate::openhuman::gameplay_review::rpc::analyze_session(payload).await?)
+    })
+}
+
+fn handle_get_session(params: Map<String, Value>) -> ControllerFuture {
+    Box::pin(async move {
+        let payload = deserialize_params::<SessionIdParams>(params)?;
+        to_json(crate::openhuman::gameplay_review::rpc::get_session(payload.session_id).await?)
+    })
+}
+
+fn handle_list_sessions(params: Map<String, Value>) -> ControllerFuture {
+    Box::pin(async move {
+        #[derive(Deserialize)]
+        struct Params {
+            #[serde(default)]
+            game_id: Option<String>,
+        }
+        let payload = deserialize_params::<Params>(params)?;
+        to_json(crate::openhuman::gameplay_review::rpc::list_sessions(payload.game_id).await?)
+    })
+}
+
+fn handle_set_preset(params: Map<String, Value>) -> ControllerFuture {
+    Box::pin(async move {
+        let payload = deserialize_params::<GameplayPresetInput>(params)?;
+        to_json(crate::openhuman::gameplay_review::rpc::set_preset(payload).await?)
+    })
+}
+
+fn handle_list_presets(_params: Map<String, Value>) -> ControllerFuture {
+    Box::pin(async move { to_json(crate::openhuman::gameplay_review::rpc::list_presets().await?) })
+}
+
+fn handle_ask_session(params: Map<String, Value>) -> ControllerFuture {
+    Box::pin(async move {
+        let payload = deserialize_params::<GameplayReviewQuestionInput>(params)?;
+        to_json(crate::openhuman::gameplay_review::rpc::ask_session(payload).await?)
+    })
+}
+
+fn handle_draft_clip_metadata(params: Map<String, Value>) -> ControllerFuture {
+    Box::pin(async move {
+        let payload = deserialize_params::<GameplayReviewClipInput>(params)?;
+        to_json(crate::openhuman::gameplay_review::rpc::draft_clip_metadata(payload).await?)
+    })
+}
+
+fn to_json<T: serde::Serialize>(value: T) -> ControllerFuture {
+    Box::pin(async move { serde_json::to_value(value).map_err(|err| err.to_string()) })
+}
diff --git a/src/openhuman/gameplay_review/store.rs b/src/openhuman/gameplay_review/store.rs
new file mode 100644
index 0000000000..4d0430dd84
--- /dev/null
+++ b/src/openhuman/gameplay_review/store.rs
@@ -0,0 +1,175 @@
+use std::fs;
+use std::path::{Path, PathBuf};
+
+use crate::openhuman::config::Config;
+
+use super::types::{GameplayPresetInput, GameplayReviewPreset, GameplayReviewSession};
+
+const REVIEW_DIR: &str = "gameplay_review";
+const SESSIONS_DIR: &str = "sessions";
+const PRESETS_DIR: &str = "presets";
+
+pub fn review_root(workspace_dir: &Path) -> PathBuf {
+    workspace_dir.join(REVIEW_DIR)
+}
+
+pub fn sessions_dir(workspace_dir: &Path) -> PathBuf {
+    review_root(workspace_dir).join(SESSIONS_DIR)
+}
+
+pub fn presets_dir(workspace_dir: &Path) -> PathBuf {
+    review_root(workspace_dir).join(PRESETS_DIR)
+}
+
+pub fn slugify(value: &str) -> String {
+    let mut out = String::with_capacity(value.len());
+    let mut last_was_dash = false;
+    for ch in value.trim().to_ascii_lowercase().chars() {
+        let next = if ch.is_ascii_alphanumeric() { ch } else { '-' };
+        if next == '-' {
+            if last_was_dash {
+                continue;
+            }
+            last_was_dash = true;
+        } else {
+            last_was_dash = false;
+        }
+        out.push(next);
+    }
+    let trimmed = out.trim_matches('-');
+    if trimmed.is_empty() {
+        "untitled".to_string()
+    } else {
+        trimmed.to_string()
+    }
+}
+
+pub fn ensure_dirs(workspace_dir: &Path) -> Result<(), String> {
+    fs::create_dir_all(sessions_dir(workspace_dir))
+        .map_err(|err| format!("failed to create gameplay review sessions dir: {err}"))?;
+    fs::create_dir_all(presets_dir(workspace_dir))
+        .map_err(|err| format!("failed to create gameplay review presets dir: {err}"))?;
+    Ok(())
+}
+
+pub fn session_path(workspace_dir: &Path, session_id: &str) -> PathBuf {
+    sessions_dir(workspace_dir).join(format!("{session_id}.json"))
+}
+
+pub fn preset_path(workspace_dir: &Path, game_id: &str) -> PathBuf {
+    presets_dir(workspace_dir).join(format!("{}.json", slugify(game_id)))
+}
+
+pub fn save_session(workspace_dir: &Path, session: &GameplayReviewSession) -> Result<(), String> {
+    ensure_dirs(workspace_dir)?;
+    let path = session_path(workspace_dir, &session.session_id);
+    let tmp_path = path.with_extension("json.tmp");
+    let payload = serde_json::to_string_pretty(session)
+        .map_err(|err| format!("failed to serialize session: {err}"))?;
+    fs::write(&tmp_path, payload).map_err(|err| format!("failed to write session tmp: {err}"))?;
+    fs::rename(&tmp_path, &path).map_err(|err| format!("failed to move session into place: {err}"))?;
+    Ok(())
+}
+
+pub fn load_session(workspace_dir: &Path, session_id: &str) -> Result<Option<GameplayReviewSession>, String> {
+    let path = session_path(workspace_dir, session_id);
+    if !path.exists() {
+        return Ok(None);
+    }
+    let raw = fs::read_to_string(&path).map_err(|err| format!("failed to read session: {err}"))?;
+    let session = serde_json::from_str(&raw)
+        .map_err(|err| format!("failed to parse session {}: {err}", path.display()))?;
+    Ok(Some(session))
+}
+
+pub fn list_sessions(workspace_dir: &Path) -> Result<Vec<GameplayReviewSession>, String> {
+    let dir = sessions_dir(workspace_dir);
+    if !dir.exists() {
+        return Ok(Vec::new());
+    }
+
+    let mut sessions = Vec::new();
+    for entry in fs::read_dir(&dir).map_err(|err| format!("failed to list sessions: {err}"))? {
+        let entry = entry.map_err(|err| format!("failed to read session entry: {err}"))?;
+        let path = entry.path();
+        if path.extension().and_then(|value| value.to_str()) != Some("json") {
+            continue;
+        }
+        let raw = fs::read_to_string(&path)
+            .map_err(|err| format!("failed to read session {}: {err}", path.display()))?;
+        if let Ok(session) = serde_json::from_str::<GameplayReviewSession>(&raw) {
+            sessions.push(session);
+        }
+    }
+
+    sessions.sort_by(|left, right| {
+        right
+            .imported_at_ms
+            .cmp(&left.imported_at_ms)
+            .then(right.analyzed_at_ms.cmp(&left.analyzed_at_ms))
+    });
+    Ok(sessions)
+}
+
+pub fn save_preset(workspace_dir: &Path, preset: &GameplayReviewPreset) -> Result<(), String> {
+    ensure_dirs(workspace_dir)?;
+    let path = preset_path(workspace_dir, &preset.game_id);
+    let payload = serde_json::to_string_pretty(preset)
+        .map_err(|err| format!("failed to serialize preset: {err}"))?;
+    fs::write(&path, payload).map_err(|err| format!("failed to write preset: {err}"))?;
+    Ok(())
+}
+
+pub fn load_preset(
+    workspace_dir: &Path,
+    game_id: &str,
+) -> Result<Option<GameplayReviewPreset>, String> {
+    let path = preset_path(workspace_dir, game_id);
+    if !path.exists() {
+        return Ok(None);
+    }
+    let raw = fs::read_to_string(&path).map_err(|err| format!("failed to read preset: {err}"))?;
+    let preset = serde_json::from_str(&raw)
+        .map_err(|err| format!("failed to parse preset {}: {err}", path.display()))?;
+    Ok(Some(preset))
+}
+
+pub fn list_presets(workspace_dir: &Path) -> Result<Vec<GameplayReviewPreset>, String> {
+    let dir = presets_dir(workspace_dir);
+    if !dir.exists() {
+        return Ok(Vec::new());
+    }
+
+    let mut presets = Vec::new();
+    for entry in fs::read_dir(&dir).map_err(|err| format!("failed to list presets: {err}"))? {
+        let entry = entry.map_err(|err| format!("failed to read preset entry: {err}"))?;
+        let path = entry.path();
+        if path.extension().and_then(|value| value.to_str()) != Some("json") {
+            continue;
+        }
+        let raw = fs::read_to_string(&path)
+            .map_err(|err| format!("failed to read preset {}: {err}", path.display()))?;
+        if let Ok(preset) = serde_json::from_str::<GameplayReviewPreset>(&raw) {
+            presets.push(preset);
+        }
+    }
+
+    presets.sort_by(|left, right| left.display_name.cmp(&right.display_name));
+    Ok(presets)
+}
+
+pub fn workspace_dir_from_config(config: &Config) -> PathBuf {
+    config.workspace_dir.clone()
+}
+
+pub fn preset_from_input(input: GameplayPresetInput) -> GameplayReviewPreset {
+    GameplayReviewPreset {
+        game_id: input.game_id,
+        display_name: input.display_name,
+        coaching_focus: input.coaching_focus,
+        audio_feedback: input.audio_feedback,
+        spoiler_mode: input.spoiler_mode,
+        notes: input.notes,
+        updated_at_ms: chrono::Utc::now().timestamp_millis(),
+    }
+}
diff --git a/src/openhuman/gameplay_review/types.rs b/src/openhuman/gameplay_review/types.rs
new file mode 100644
index 0000000000..6aaea9397b
--- /dev/null
+++ b/src/openhuman/gameplay_review/types.rs
@@ -0,0 +1,173 @@
+use serde::{Deserialize, Serialize};
+
+#[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize)]
+#[serde(rename_all = "snake_case")]
+pub enum SpoilerMode {
+    Off,
+    Light,
+    Full,
+}
+
+impl Default for SpoilerMode {
+    fn default() -> Self {
+        Self::Light
+    }
+}
+
+impl SpoilerMode {
+    pub const fn as_str(self) -> &'static str {
+        match self {
+            Self::Off => "off",
+            Self::Light => "light",
+            Self::Full => "full",
+        }
+    }
+}
+
+#[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize)]
+#[serde(rename_all = "snake_case")]
+pub enum HighlightKind {
+    Highlight,
+    Mistake,
+    Coaching,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayFrameInput {
+    pub file_name: String,
+    pub image_ref: String,
+    #[serde(default)]
+    pub captured_at_ms: Option<i64>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayPresetInput {
+    pub game_id: String,
+    pub display_name: String,
+    #[serde(default)]
+    pub coaching_focus: Vec<String>,
+    #[serde(default)]
+    pub audio_feedback: bool,
+    #[serde(default)]
+    pub spoiler_mode: SpoilerMode,
+    #[serde(default)]
+    pub notes: Option<String>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayReviewSessionInput {
+    pub game_id: String,
+    pub session_title: String,
+    #[serde(default)]
+    pub source_label: Option<String>,
+    #[serde(default)]
+    pub spoiler_mode: Option<SpoilerMode>,
+    #[serde(default)]
+    pub preset_id: Option<String>,
+    pub frames: Vec<GameplayFrameInput>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayReviewQuestionInput {
+    pub session_id: String,
+    pub question: String,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayReviewPreset {
+    pub game_id: String,
+    pub display_name: String,
+    #[serde(default)]
+    pub coaching_focus: Vec<String>,
+    #[serde(default)]
+    pub audio_feedback: bool,
+    #[serde(default)]
+    pub spoiler_mode: SpoilerMode,
+    #[serde(default)]
+    pub notes: Option<String>,
+    pub updated_at_ms: i64,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayHighlight {
+    pub id: String,
+    pub frame_index: usize,
+    #[serde(default)]
+    pub captured_at_ms: Option<i64>,
+    pub title: String,
+    pub rationale: String,
+    pub confidence: f32,
+    pub kind: HighlightKind,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayClipCandidate {
+    pub id: String,
+    pub frame_index: usize,
+    pub start_label: String,
+    pub end_label: String,
+    pub rationale: String,
+    pub confidence: f32,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayPlatformDraft {
+    pub platform: String,
+    pub title: String,
+    pub description: String,
+    pub tags: Vec<String>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayReviewAnalysis {
+    pub recap: String,
+    pub highlights: Vec<GameplayHighlight>,
+    pub clip_candidates: Vec<GameplayClipCandidate>,
+    pub draft_metadata: Vec<GameplayPlatformDraft>,
+    pub follow_up_questions: Vec<String>,
+    #[serde(default)]
+    pub spoiler_note: Option<String>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayReviewSession {
+    pub session_id: String,
+    pub game_id: String,
+    pub session_title: String,
+    #[serde(default)]
+    pub source_label: Option<String>,
+    pub spoiler_mode: SpoilerMode,
+    #[serde(default)]
+    pub preset_id: Option<String>,
+    pub imported_at_ms: i64,
+    #[serde(default)]
+    pub analyzed_at_ms: Option<i64>,
+    pub frames: Vec<GameplayFrameInput>,
+    #[serde(default)]
+    pub analysis: Option<GameplayReviewAnalysis>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayReviewAnalysisInput {
+    pub session_id: String,
+    #[serde(default)]
+    pub max_highlights: Option<usize>,
+    #[serde(default)]
+    pub platforms: Vec<String>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayReviewClipInput {
+    pub session_id: String,
+    #[serde(default)]
+    pub platform: Option<String>,
+    #[serde(default)]
+    pub highlight_id: Option<String>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct GameplayReviewQuestionResult {
+    pub answer: String,
+    pub matched_highlights: Vec<GameplayHighlight>,
+    pub suggested_follow_up: Vec<String>,
+}
diff --git a/src/openhuman/mod.rs b/src/openhuman/mod.rs
index a70c865ff6..6716e59c06 100644
--- a/src/openhuman/mod.rs
+++ b/src/openhuman/mod.rs
@@ -75,3 +75,4 @@ pub mod webview_apis;
 pub mod webview_notifications;
 pub mod whatsapp_data;
 pub mod workspace;
+pub mod gameplay_review;

From eaf3addadebbd98a1425fdf94e6dd1573a1f121f Mon Sep 17 00:00:00 2001
From: Ayush Raj Chourasia <iter.student.alpha@gmail.com>
Date: Wed, 13 May 2026 18:59:31 +0000
Subject: [PATCH 2/3] feat(gameplay): format review workspace

---
 .../GameplayReviewWorkspace.test.tsx          |  16 +-
 .../intelligence/GameplayReviewWorkspace.tsx  | 155 ++++++++++--------
 app/src/pages/Intelligence.tsx                |   2 +-
 app/src/services/gameplayReviewService.ts     |  24 ++-
 4 files changed, 115 insertions(+), 82 deletions(-)

diff --git a/app/src/components/intelligence/GameplayReviewWorkspace.test.tsx b/app/src/components/intelligence/GameplayReviewWorkspace.test.tsx
index 8419d66683..90d0985a6a 100644
--- a/app/src/components/intelligence/GameplayReviewWorkspace.test.tsx
+++ b/app/src/components/intelligence/GameplayReviewWorkspace.test.tsx
@@ -19,9 +19,13 @@ vi.mock('../../services/gameplayReviewService', () => ({
   flattenClipCandidates: vi.fn((session: any) => session?.analysis?.clip_candidates ?? []),
   flattenDrafts: vi.fn((session: any) => session?.analysis?.draft_metadata ?? []),
   flattenHighlights: vi.fn((session: any) => session?.analysis?.highlights ?? []),
-  formatSpoilerMode: vi.fn((mode: string) => (mode === 'full' ? 'Full spoilers' : 'Light spoilers')),
+  formatSpoilerMode: vi.fn((mode: string) =>
+    mode === 'full' ? 'Full spoilers' : 'Light spoilers'
+  ),
   listGameplaySessions: vi.fn(),
-  normalizeGameplayError: vi.fn((error: unknown) => (error instanceof Error ? error.message : String(error))),
+  normalizeGameplayError: vi.fn((error: unknown) =>
+    error instanceof Error ? error.message : String(error)
+  ),
   prepareGameplayFrames: vi.fn(),
   registerGameplaySession: vi.fn(),
   saveGameplayPreset: vi.fn(),
@@ -161,18 +165,14 @@ describe('GameplayReviewWorkspace', () => {
     expect(fileInput).not.toBeNull();
     const file = new File([new Uint8Array([1, 2, 3])], 'frame-1.png', { type: 'image/png' });
     Object.defineProperty(file, 'webkitRelativePath', { value: 'session/frame-1.png' });
-    fireEvent.change(fileInput as HTMLInputElement, {
-      target: { files: [file] },
-    });
+    fireEvent.change(fileInput as HTMLInputElement, { target: { files: [file] } });
 
     fireEvent.click(screen.getByRole('button', { name: /import and analyze/i }));
 
     await waitFor(() => expect(mockedPrepareGameplayFrames).toHaveBeenCalled());
     await waitFor(() => expect(screen.getByText(/Gameplay recap/)).toBeInTheDocument());
     expect(
-      screen.getByText('Highlight: clutch fight', {
-        selector: 'div.font-medium.text-stone-900',
-      })
+      screen.getByText('Highlight: clutch fight', { selector: 'div.font-medium.text-stone-900' })
     ).toBeInTheDocument();
 
     fireEvent.click(screen.getByRole('button', { name: /ask session/i }));
diff --git a/app/src/components/intelligence/GameplayReviewWorkspace.tsx b/app/src/components/intelligence/GameplayReviewWorkspace.tsx
index 8e9a462818..3b554e3f2a 100644
--- a/app/src/components/intelligence/GameplayReviewWorkspace.tsx
+++ b/app/src/components/intelligence/GameplayReviewWorkspace.tsx
@@ -1,6 +1,5 @@
 import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
 
-import type { ToastNotification } from '../../types/intelligence';
 import {
   analyzeGameplaySession,
   askGameplaySession,
@@ -9,15 +8,16 @@ import {
   flattenDrafts,
   flattenHighlights,
   formatSpoilerMode,
+  type GameplayFrameInput,
+  type GameplayReviewSession,
   listGameplaySessions,
   normalizeGameplayError,
   prepareGameplayFrames,
   registerGameplaySession,
   saveGameplayPreset,
-  type GameplayFrameInput,
-  type GameplayReviewSession,
   type SpoilerMode,
 } from '../../services/gameplayReviewService';
+import type { ToastNotification } from '../../types/intelligence';
 
 interface GameplayReviewWorkspaceProps {
   onToast?: (toast: Omit<ToastNotification, 'id'>) => void;
@@ -47,9 +47,7 @@ const INITIAL_IMPORT_STATE: ImportState = {
   platforms: 'twitch,kick,youtube',
 };
 
-const DIRECTORY_INPUT_PROPS: Record<string, string> = {
-  webkitdirectory: '',
-};
+const DIRECTORY_INPUT_PROPS: Record<string, string> = { webkitdirectory: '' };
 
 function formatTimestamp(ms?: number | null): string {
   if (!ms) return 'n/a';
@@ -148,11 +146,14 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
         source_label: form.sourceLabel.trim() || null,
         spoiler_mode: form.spoilerMode,
         preset_id: form.presetName.trim() || null,
-        frames: frames.map(frame => ({
-          file_name: frame.file_name,
-          image_ref: frame.image_ref,
-          captured_at_ms: frame.captured_at_ms,
-        } satisfies GameplayFrameInput)),
+        frames: frames.map(
+          frame =>
+            ({
+              file_name: frame.file_name,
+              image_ref: frame.image_ref,
+              captured_at_ms: frame.captured_at_ms,
+            }) satisfies GameplayFrameInput
+        ),
       });
 
       setAnalysisBusy(true);
@@ -172,11 +173,7 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
     } catch (err) {
       const message = normalizeGameplayError(err);
       setError(message);
-      onToast?.({
-        type: 'error',
-        title: 'Gameplay review failed',
-        message,
-      });
+      onToast?.({ type: 'error', title: 'Gameplay review failed', message });
     } finally {
       setAnalysisBusy(false);
       setImportBusy(false);
@@ -233,10 +230,12 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
             <p className="text-xs font-semibold uppercase tracking-[0.24em] text-sage-700">
               Gameplay review
             </p>
-            <h2 className="text-2xl font-bold text-stone-900">Import a session, find the clips, draft the post</h2>
+            <h2 className="text-2xl font-bold text-stone-900">
+              Import a session, find the clips, draft the post
+            </h2>
             <p className="max-w-2xl text-sm text-stone-600">
-              Load a folder of keyframes from a long session, generate a concise recap, ask follow-up questions,
-              and turn the strongest moments into platform-ready clip metadata.
+              Load a folder of keyframes from a long session, generate a concise recap, ask
+              follow-up questions, and turn the strongest moments into platform-ready clip metadata.
             </p>
           </div>
           <div className="rounded-xl border border-sage-200 bg-white/80 px-4 py-3 text-sm text-stone-700 shadow-sm">
@@ -278,9 +277,10 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
             <span className="font-medium text-stone-700">Spoiler mode</span>
             <select
               value={form.spoilerMode}
-              onChange={event => setForm(prev => ({ ...prev, spoilerMode: event.target.value as SpoilerMode }))}
-              className="w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500"
-            >
+              onChange={event =>
+                setForm(prev => ({ ...prev, spoilerMode: event.target.value as SpoilerMode }))
+              }
+              className="w-full rounded-xl border border-stone-200 bg-white px-3 py-2 outline-none ring-0 transition focus:border-primary-500">
               <option value="off">Off</option>
               <option value="light">Light</option>
               <option value="full">Full</option>
@@ -326,7 +326,9 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
             <input
               type="checkbox"
               checked={form.audioFeedback}
-              onChange={event => setForm(prev => ({ ...prev, audioFeedback: event.target.checked }))}
+              onChange={event =>
+                setForm(prev => ({ ...prev, audioFeedback: event.target.checked }))
+              }
             />
             Enable audio summary notes
           </label>
@@ -345,28 +347,26 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
           <button
             type="button"
             onClick={handleSelectFiles}
-            className="rounded-xl border border-sage-200 bg-white px-4 py-2 text-sm font-semibold text-sage-800 shadow-sm transition hover:bg-sage-50"
-          >
+            className="rounded-xl border border-sage-200 bg-white px-4 py-2 text-sm font-semibold text-sage-800 shadow-sm transition hover:bg-sage-50">
             Choose folder / frames
           </button>
           <button
             type="button"
             onClick={handleImportSession}
             disabled={importBusy || analysisBusy}
-            className="rounded-xl bg-primary-600 px-4 py-2 text-sm font-semibold text-white shadow-sm transition hover:bg-primary-700 disabled:cursor-not-allowed disabled:opacity-60"
-          >
+            className="rounded-xl bg-primary-600 px-4 py-2 text-sm font-semibold text-white shadow-sm transition hover:bg-primary-700 disabled:cursor-not-allowed disabled:opacity-60">
             {importBusy ? 'Importing…' : 'Import and analyze'}
           </button>
           <button
             type="button"
             onClick={handleDraftClips}
             disabled={draftBusy || !activeSession}
-            className="rounded-xl border border-amber-200 bg-white px-4 py-2 text-sm font-semibold text-amber-800 shadow-sm transition hover:bg-amber-50 disabled:cursor-not-allowed disabled:opacity-60"
-          >
+            className="rounded-xl border border-amber-200 bg-white px-4 py-2 text-sm font-semibold text-amber-800 shadow-sm transition hover:bg-amber-50 disabled:cursor-not-allowed disabled:opacity-60">
             {draftBusy ? 'Drafting…' : 'Refresh clip metadata'}
           </button>
           <div className="text-xs text-stone-500">
-            The core will analyze image frames. For raw video files, point this at extracted keyframes.
+            The core will analyze image frames. For raw video files, point this at extracted
+            keyframes.
           </div>
         </div>
       </div>
@@ -382,7 +382,9 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
           <div className="flex items-center justify-between gap-2">
             <div>
               <h3 className="text-lg font-semibold text-stone-900">Session viewer</h3>
-              <p className="text-sm text-stone-500">Recap, highlights, and clip candidates for the current session.</p>
+              <p className="text-sm text-stone-500">
+                Recap, highlights, and clip candidates for the current session.
+              </p>
             </div>
             <div className="rounded-full border border-stone-200 bg-stone-50 px-3 py-1 text-xs text-stone-600">
               {activeSession ? activeSession.game_id : 'No session selected'}
@@ -394,9 +396,12 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
               <div className="rounded-xl border border-sage-200 bg-sage-50 p-4">
                 <div className="flex flex-wrap items-center justify-between gap-2">
                   <div>
-                    <div className="text-sm font-semibold text-sage-900">{activeSession.session_title}</div>
+                    <div className="text-sm font-semibold text-sage-900">
+                      {activeSession.session_title}
+                    </div>
                     <div className="text-xs text-sage-700">
-                      Imported {formatTimestamp(activeSession.imported_at_ms)} · {activeSession.frames.length} frame(s)
+                      Imported {formatTimestamp(activeSession.imported_at_ms)} ·{' '}
+                      {activeSession.frames.length} frame(s)
                     </div>
                   </div>
                   <div className="text-xs font-medium text-sage-700">
@@ -404,10 +409,14 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
                   </div>
                 </div>
                 {activeSession.analysis?.spoiler_note && (
-                  <p className="mt-3 text-sm text-stone-700">{activeSession.analysis.spoiler_note}</p>
+                  <p className="mt-3 text-sm text-stone-700">
+                    {activeSession.analysis.spoiler_note}
+                  </p>
                 )}
                 {activeSession.analysis && (
-                  <p className="mt-3 whitespace-pre-line text-sm text-stone-800">{activeSession.analysis.recap}</p>
+                  <p className="mt-3 whitespace-pre-line text-sm text-stone-800">
+                    {activeSession.analysis.recap}
+                  </p>
                 )}
               </div>
 
@@ -417,7 +426,9 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
                   <div className="mt-3 space-y-3">
                     {activeHighlights.length > 0 ? (
                       activeHighlights.map(highlight => (
-                        <article key={highlight.id} className="rounded-lg border border-stone-200 bg-stone-50 p-3">
+                        <article
+                          key={highlight.id}
+                          className="rounded-lg border border-stone-200 bg-stone-50 p-3">
                           <div className="flex items-center justify-between gap-2">
                             <div className="font-medium text-stone-900">{highlight.title}</div>
                             <span className="rounded-full bg-white px-2 py-0.5 text-[11px] text-stone-500">
@@ -425,7 +436,8 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
                             </span>
                           </div>
                           <p className="mt-1 text-xs uppercase tracking-wide text-stone-500">
-                            {highlight.kind} · frame {highlight.frame_index + 1} · {formatTimestamp(highlight.captured_at_ms)}
+                            {highlight.kind} · frame {highlight.frame_index + 1} ·{' '}
+                            {formatTimestamp(highlight.captured_at_ms)}
                           </p>
                           <p className="mt-2 text-sm text-stone-700">{highlight.rationale}</p>
                         </article>
@@ -441,7 +453,9 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
                   <div className="mt-3 space-y-3">
                     {activeClips.length > 0 ? (
                       activeClips.map(clip => (
-                        <article key={clip.id} className="rounded-lg border border-amber-100 bg-amber-50 p-3">
+                        <article
+                          key={clip.id}
+                          className="rounded-lg border border-amber-100 bg-amber-50 p-3">
                           <div className="font-medium text-amber-950">{clip.start_label}</div>
                           <div className="text-xs text-amber-800">through {clip.end_label}</div>
                           <p className="mt-2 text-sm text-stone-700">{clip.rationale}</p>
@@ -459,15 +473,21 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
                 <div className="mt-3 grid gap-3 md:grid-cols-3">
                   {activeDrafts.length > 0 ? (
                     activeDrafts.map(draft => (
-                      <article key={`${draft.platform}-${draft.title}`} className="rounded-lg border border-stone-200 bg-white p-3">
+                      <article
+                        key={`${draft.platform}-${draft.title}`}
+                        className="rounded-lg border border-stone-200 bg-white p-3">
                         <div className="text-xs font-semibold uppercase tracking-wide text-stone-500">
                           {draft.platform}
                         </div>
                         <div className="mt-1 font-medium text-stone-900">{draft.title}</div>
-                        <p className="mt-2 text-sm text-stone-700 whitespace-pre-line">{draft.description}</p>
+                        <p className="mt-2 text-sm text-stone-700 whitespace-pre-line">
+                          {draft.description}
+                        </p>
                         <div className="mt-3 flex flex-wrap gap-2">
                           {draft.tags.map(tag => (
-                            <span key={tag} className="rounded-full bg-stone-100 px-2 py-0.5 text-[11px] text-stone-600">
+                            <span
+                              key={tag}
+                              className="rounded-full bg-stone-100 px-2 py-0.5 text-[11px] text-stone-600">
                               #{tag}
                             </span>
                           ))}
@@ -475,7 +495,9 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
                       </article>
                     ))
                   ) : (
-                    <p className="text-sm text-stone-500">Draft metadata will appear after analysis.</p>
+                    <p className="text-sm text-stone-500">
+                      Draft metadata will appear after analysis.
+                    </p>
                   )}
                 </div>
               </div>
@@ -493,28 +515,29 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
                     type="button"
                     onClick={handleQuestion}
                     disabled={questionBusy}
-                    className="rounded-xl bg-sage-700 px-4 py-2 text-sm font-semibold text-white transition hover:bg-sage-800 disabled:cursor-not-allowed disabled:opacity-60"
-                  >
+                    className="rounded-xl bg-sage-700 px-4 py-2 text-sm font-semibold text-white transition hover:bg-sage-800 disabled:cursor-not-allowed disabled:opacity-60">
                     {questionBusy ? 'Thinking…' : 'Ask session'}
                   </button>
                 </div>
                 {questionAnswer && (
-                  <p className="mt-3 whitespace-pre-line text-sm text-stone-800">{questionAnswer}</p>
-                )}
-                {activeSession.analysis?.follow_up_questions && activeSession.analysis.follow_up_questions.length > 0 && (
-                  <div className="mt-3 flex flex-wrap gap-2 text-xs text-stone-600">
-                    {activeSession.analysis.follow_up_questions.map(item => (
-                      <button
-                        key={item}
-                        type="button"
-                        onClick={() => setQuestion(item)}
-                        className="rounded-full border border-sage-200 bg-white px-3 py-1 transition hover:bg-sage-50"
-                      >
-                        {item}
-                      </button>
-                    ))}
-                  </div>
+                  <p className="mt-3 whitespace-pre-line text-sm text-stone-800">
+                    {questionAnswer}
+                  </p>
                 )}
+                {activeSession.analysis?.follow_up_questions &&
+                  activeSession.analysis.follow_up_questions.length > 0 && (
+                    <div className="mt-3 flex flex-wrap gap-2 text-xs text-stone-600">
+                      {activeSession.analysis.follow_up_questions.map(item => (
+                        <button
+                          key={item}
+                          type="button"
+                          onClick={() => setQuestion(item)}
+                          className="rounded-full border border-sage-200 bg-white px-3 py-1 transition hover:bg-sage-50">
+                          {item}
+                        </button>
+                      ))}
+                    </div>
+                  )}
               </div>
             </div>
           ) : (
@@ -527,7 +550,9 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
         <aside className="space-y-4 rounded-2xl border border-stone-200 bg-white p-5 shadow-soft">
           <div>
             <h3 className="text-lg font-semibold text-stone-900">Session history</h3>
-            <p className="text-sm text-stone-500">Previously imported or analyzed sessions for this workspace.</p>
+            <p className="text-sm text-stone-500">
+              Previously imported or analyzed sessions for this workspace.
+            </p>
           </div>
           <div className="space-y-3">
             {recentSessions.length > 0 ? (
@@ -540,8 +565,7 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
                     activeSession?.session_id === session.session_id
                       ? 'border-primary-300 bg-primary-50'
                       : 'border-stone-200 bg-white hover:bg-stone-50'
-                  }`}
-                >
+                  }`}>
                   <div className="flex items-start justify-between gap-3">
                     <div>
                       <div className="font-medium text-stone-900">{session.session_title}</div>
@@ -552,7 +576,9 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
                       <div>{session.frames.length} frame(s)</div>
                     </div>
                   </div>
-                  <div className="mt-2 text-xs text-stone-500">Imported {formatTimestamp(session.imported_at_ms)}</div>
+                  <div className="mt-2 text-xs text-stone-500">
+                    Imported {formatTimestamp(session.imported_at_ms)}
+                  </div>
                 </button>
               ))
             ) : (
@@ -562,8 +588,7 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
           <button
             type="button"
             onClick={() => void refreshSessions()}
-            className="w-full rounded-xl border border-stone-200 bg-white px-4 py-2 text-sm font-semibold text-stone-700 transition hover:bg-stone-50"
-          >
+            className="w-full rounded-xl border border-stone-200 bg-white px-4 py-2 text-sm font-semibold text-stone-700 transition hover:bg-stone-50">
             Refresh history
           </button>
         </aside>
diff --git a/app/src/pages/Intelligence.tsx b/app/src/pages/Intelligence.tsx
index 0df2129e74..c53d760678 100644
--- a/app/src/pages/Intelligence.tsx
+++ b/app/src/pages/Intelligence.tsx
@@ -1,9 +1,9 @@
 import { useCallback, useEffect, useState } from 'react';
 
 import { ConfirmationModal } from '../components/intelligence/ConfirmationModal';
+import { GameplayReviewWorkspace } from '../components/intelligence/GameplayReviewWorkspace';
 import IntelligenceCallsTab from '../components/intelligence/IntelligenceCallsTab';
 import IntelligenceDreamsTab from '../components/intelligence/IntelligenceDreamsTab';
-import { GameplayReviewWorkspace } from '../components/intelligence/GameplayReviewWorkspace';
 import IntelligenceSubconsciousTab from '../components/intelligence/IntelligenceSubconsciousTab';
 import { MemoryWorkspace } from '../components/intelligence/MemoryWorkspace';
 import { ToastContainer } from '../components/intelligence/Toast';
diff --git a/app/src/services/gameplayReviewService.ts b/app/src/services/gameplayReviewService.ts
index 5ac8b1e37f..c99b2e15cb 100644
--- a/app/src/services/gameplayReviewService.ts
+++ b/app/src/services/gameplayReviewService.ts
@@ -106,7 +106,10 @@ export interface PreparedGameplayFrame extends GameplayFrameInput {
   source_name: string;
 }
 
-export async function prepareGameplayFrames(files: File[], maxFrames = 12): Promise<PreparedGameplayFrame[]> {
+export async function prepareGameplayFrames(
+  files: File[],
+  maxFrames = 12
+): Promise<PreparedGameplayFrame[]> {
   const imageFiles = files
     .filter(file => file.type.startsWith('image/'))
     .sort((left, right) => {
@@ -116,7 +119,11 @@ export async function prepareGameplayFrames(files: File[], maxFrames = 12): Prom
     })
     .slice(0, Math.max(1, maxFrames));
 
-  log('prepareGameplayFrames: selected %d image(s) from %d file(s)', imageFiles.length, files.length);
+  log(
+    'prepareGameplayFrames: selected %d image(s) from %d file(s)',
+    imageFiles.length,
+    files.length
+  );
   const prepared: PreparedGameplayFrame[] = [];
   for (const file of imageFiles) {
     const buffer = await file.arrayBuffer();
@@ -135,7 +142,9 @@ export async function prepareGameplayFrames(files: File[], maxFrames = 12): Prom
   return prepared;
 }
 
-export async function registerGameplaySession(payload: GameplayReviewSessionInput): Promise<GameplayReviewSession> {
+export async function registerGameplaySession(
+  payload: GameplayReviewSessionInput
+): Promise<GameplayReviewSession> {
   const result = await callCoreRpc<GameplayReviewSession>({
     method: 'openhuman.gameplay_review_register_session',
     params: payload,
@@ -162,10 +171,7 @@ export async function listGameplaySessions(gameId?: string): Promise<GameplayRev
 }
 
 export async function saveGameplayPreset(payload: GameplayPresetInput): Promise<unknown> {
-  return callCoreRpc({
-    method: 'openhuman.gameplay_review_set_preset',
-    params: payload,
-  });
+  return callCoreRpc({ method: 'openhuman.gameplay_review_set_preset', params: payload });
 }
 
 export async function listGameplayPresets(): Promise<GameplayPresetInput[]> {
@@ -198,7 +204,9 @@ export function flattenDrafts(session: GameplayReviewSession | null): GameplayPl
   return session?.analysis?.draft_metadata ?? [];
 }
 
-export function flattenClipCandidates(session: GameplayReviewSession | null): GameplayClipCandidate[] {
+export function flattenClipCandidates(
+  session: GameplayReviewSession | null
+): GameplayClipCandidate[] {
   return session?.analysis?.clip_candidates ?? [];
 }
 

From 7194eb0a0df9dc8106e721140638af3f71c7a25c Mon Sep 17 00:00:00 2001
From: Ayush Raj Chourasia <iter.student.alpha@gmail.com>
Date: Wed, 13 May 2026 19:22:27 +0000
Subject: [PATCH 3/3] fix(gameplay): address review feedback

---
 .../intelligence/GameplayReviewWorkspace.tsx  |   6 +-
 src/openhuman/gameplay_review/ops.rs          | 213 +++++++++++++++++-
 src/openhuman/gameplay_review/schemas.rs      | 105 ++++++++-
 src/openhuman/gameplay_review/store.rs        | 116 +++++++++-
 src/openhuman/mod.rs                          |   2 +-
 5 files changed, 411 insertions(+), 31 deletions(-)

diff --git a/app/src/components/intelligence/GameplayReviewWorkspace.tsx b/app/src/components/intelligence/GameplayReviewWorkspace.tsx
index 3b554e3f2a..f8c35f7c21 100644
--- a/app/src/components/intelligence/GameplayReviewWorkspace.tsx
+++ b/app/src/components/intelligence/GameplayReviewWorkspace.tsx
@@ -79,13 +79,11 @@ export function GameplayReviewWorkspace({ onToast }: GameplayReviewWorkspaceProp
     try {
       const sessions = await listGameplaySessions();
       setRecentSessions(sessions);
-      if (!activeSession && sessions.length > 0) {
-        setActiveSession(sessions[0]);
-      }
+      setActiveSession(currentSession => currentSession ?? sessions[0] ?? null);
     } catch (err) {
       setError(normalizeGameplayError(err));
     }
-  }, [activeSession]);
+  }, []);
 
   useEffect(() => {
     void refreshSessions();
diff --git a/src/openhuman/gameplay_review/ops.rs b/src/openhuman/gameplay_review/ops.rs
index 16f01a7bb6..53151ae5d8 100644
--- a/src/openhuman/gameplay_review/ops.rs
+++ b/src/openhuman/gameplay_review/ops.rs
@@ -3,6 +3,7 @@ use std::collections::BTreeSet;
 use std::path::Path;
 
 use chrono::Utc;
+use log::{debug, trace, warn};
 use uuid::Uuid;
 
 use crate::openhuman::config::Config;
@@ -23,9 +24,28 @@ const DEFAULT_PLATFORMS: &[&str] = &["twitch", "kick", "youtube"];
 pub async fn register_session(
     payload: GameplayReviewSessionInput,
 ) -> Result<RpcOutcome<GameplayReviewSession>, String> {
+    debug!(
+        "[gameplay_review][rpc] register_session start game_id={} frames={} spoiler_mode={}",
+        payload.game_id,
+        payload.frames.len(),
+        payload
+            .spoiler_mode
+            .unwrap_or_default()
+            .as_str()
+    );
     let session = build_session(payload)?;
     let workspace_dir = workspace_dir().await?;
+    debug!(
+        "[gameplay_review][rpc] register_session workspace_dir={} session_id={}",
+        workspace_dir.display(),
+        session.session_id
+    );
     store::save_session(&workspace_dir, &session)?;
+    debug!(
+        "[gameplay_review][rpc] register_session complete session_id={} game_id={}",
+        session.session_id,
+        session.game_id
+    );
     Ok(RpcOutcome::single_log(
         session,
         "gameplay review session registered",
@@ -35,17 +55,67 @@ pub async fn register_session(
 pub async fn analyze_session(
     payload: GameplayReviewAnalysisInput,
 ) -> Result<RpcOutcome<GameplayReviewSession>, String> {
+    debug!(
+        "[gameplay_review][rpc] analyze_session start session_id={} max_highlights={:?} platform_overrides={}",
+        payload.session_id,
+        payload.max_highlights,
+        payload.platforms.len()
+    );
     let workspace_dir = workspace_dir().await?;
     let mut session = store::load_session(&workspace_dir, &payload.session_id)?
         .ok_or_else(|| format!("gameplay session not found: {}", payload.session_id))?;
+    debug!(
+        "[gameplay_review][rpc] analyze_session loaded session_id={} game_id={} frames={} preset_id={:?}",
+        session.session_id,
+        session.game_id,
+        session.frames.len(),
+        session.preset_id
+    );
     let preset = match session.preset_id.as_deref() {
-        Some(game_id) => store::load_preset(&workspace_dir, game_id)?,
-        None => None,
+        Some(game_id) => {
+            debug!(
+                "[gameplay_review][rpc] analyze_session loading preset game_id={}",
+                game_id
+            );
+            store::load_preset(&workspace_dir, game_id)?
+        }
+        None => {
+            trace!(
+                "[gameplay_review][rpc] analyze_session no preset configured session_id={}",
+                session.session_id
+            );
+            None
+        }
     };
-    let analysis = analyze_session_frames(&session, preset.as_ref(), payload.max_highlights, &payload.platforms).await;
+    let analysis = analyze_session_frames(
+        &session,
+        preset.as_ref(),
+        payload.max_highlights,
+        &payload.platforms,
+    )
+    .await;
     session.analyzed_at_ms = Some(Utc::now().timestamp_millis());
     session.analysis = Some(analysis);
     store::save_session(&workspace_dir, &session)?;
+    debug!(
+        "[gameplay_review][rpc] analyze_session complete session_id={} highlights={} clips={} drafts={}",
+        session.session_id,
+        session
+            .analysis
+            .as_ref()
+            .map(|analysis| analysis.highlights.len())
+            .unwrap_or(0),
+        session
+            .analysis
+            .as_ref()
+            .map(|analysis| analysis.clip_candidates.len())
+            .unwrap_or(0),
+        session
+            .analysis
+            .as_ref()
+            .map(|analysis| analysis.draft_metadata.len())
+            .unwrap_or(0)
+    );
     Ok(RpcOutcome::single_log(
         session,
         "gameplay review session analyzed",
@@ -53,9 +123,16 @@ pub async fn analyze_session(
 }
 
 pub async fn get_session(session_id: String) -> Result<RpcOutcome<GameplayReviewSession>, String> {
+    debug!("[gameplay_review][rpc] get_session start session_id={}", session_id);
     let workspace_dir = workspace_dir().await?;
     let session = store::load_session(&workspace_dir, &session_id)?
         .ok_or_else(|| format!("gameplay session not found: {session_id}"))?;
+    debug!(
+        "[gameplay_review][rpc] get_session complete session_id={} game_id={} analyzed={}",
+        session.session_id,
+        session.game_id,
+        session.analysis.is_some()
+    );
     Ok(RpcOutcome::single_log(
         session,
         "gameplay review session fetched",
@@ -65,11 +142,26 @@ pub async fn get_session(session_id: String) -> Result<RpcOutcome<GameplayReview
 pub async fn list_sessions(
     game_id: Option<String>,
 ) -> Result<RpcOutcome<Vec<GameplayReviewSession>>, String> {
+    debug!(
+        "[gameplay_review][rpc] list_sessions start game_id_filter={:?}",
+        game_id
+    );
     let workspace_dir = workspace_dir().await?;
     let mut sessions = store::list_sessions(&workspace_dir)?;
     if let Some(filter) = game_id.as_deref() {
+        let before = sessions.len();
         sessions.retain(|session| session.game_id == filter);
+        debug!(
+            "[gameplay_review][rpc] list_sessions filtered game_id={} before={} after={}",
+            filter,
+            before,
+            sessions.len()
+        );
     }
+    debug!(
+        "[gameplay_review][rpc] list_sessions complete count={}",
+        sessions.len()
+    );
     Ok(RpcOutcome::single_log(
         sessions,
         "gameplay review sessions listed",
@@ -79,9 +171,21 @@ pub async fn list_sessions(
 pub async fn set_preset(
     payload: GameplayPresetInput,
 ) -> Result<RpcOutcome<GameplayReviewPreset>, String> {
+    debug!(
+        "[gameplay_review][rpc] set_preset start game_id={} display_name={} focus_items={} spoiler_mode={}",
+        payload.game_id,
+        payload.display_name,
+        payload.coaching_focus.len(),
+        payload.spoiler_mode.as_str()
+    );
     let workspace_dir = workspace_dir().await?;
     let preset = store::preset_from_input(payload);
     store::save_preset(&workspace_dir, &preset)?;
+    debug!(
+        "[gameplay_review][rpc] set_preset complete game_id={} path_hint={}",
+        preset.game_id,
+        store::preset_path(&workspace_dir, &preset.game_id).display()
+    );
     Ok(RpcOutcome::single_log(
         preset,
         "gameplay review preset saved",
@@ -89,8 +193,13 @@ pub async fn set_preset(
 }
 
 pub async fn list_presets() -> Result<RpcOutcome<Vec<GameplayReviewPreset>>, String> {
+    debug!("[gameplay_review][rpc] list_presets start");
     let workspace_dir = workspace_dir().await?;
     let presets = store::list_presets(&workspace_dir)?;
+    debug!(
+        "[gameplay_review][rpc] list_presets complete count={}",
+        presets.len()
+    );
     Ok(RpcOutcome::single_log(
         presets,
         "gameplay review presets listed",
@@ -100,6 +209,11 @@ pub async fn list_presets() -> Result<RpcOutcome<Vec<GameplayReviewPreset>>, Str
 pub async fn ask_session(
     payload: GameplayReviewQuestionInput,
 ) -> Result<RpcOutcome<GameplayReviewQuestionResult>, String> {
+    debug!(
+        "[gameplay_review][rpc] ask_session start session_id={} question_len={}",
+        payload.session_id,
+        payload.question.len()
+    );
     let workspace_dir = workspace_dir().await?;
     let session = store::load_session(&workspace_dir, &payload.session_id)?
         .ok_or_else(|| format!("gameplay session not found: {}", payload.session_id))?;
@@ -121,6 +235,12 @@ pub async fn ask_session(
     } else {
         analysis.follow_up_questions.clone()
     };
+    debug!(
+        "[gameplay_review][rpc] ask_session complete session_id={} matched_highlights={} suggested_follow_up={}",
+        session.session_id,
+        matched_highlights.len(),
+        suggested_follow_up.len()
+    );
     Ok(RpcOutcome::single_log(
         GameplayReviewQuestionResult {
             answer,
@@ -134,6 +254,12 @@ pub async fn ask_session(
 pub async fn draft_clip_metadata(
     payload: GameplayReviewClipInput,
 ) -> Result<RpcOutcome<Vec<GameplayPlatformDraft>>, String> {
+    debug!(
+        "[gameplay_review][rpc] draft_clip_metadata start session_id={} platform={:?} highlight_id={:?}",
+        payload.session_id,
+        payload.platform,
+        payload.highlight_id
+    );
     let workspace_dir = workspace_dir().await?;
     let session = store::load_session(&workspace_dir, &payload.session_id)?
         .ok_or_else(|| format!("gameplay session not found: {}", payload.session_id))?;
@@ -141,7 +267,17 @@ pub async fn draft_clip_metadata(
         .analysis
         .as_ref()
         .ok_or_else(|| "session has not been analyzed yet".to_string())?;
-    let drafts = draft_metadata(&session, analysis, payload.platform.as_deref(), payload.highlight_id.as_deref());
+    let drafts = draft_metadata(
+        &session,
+        analysis,
+        payload.platform.as_deref(),
+        payload.highlight_id.as_deref(),
+    );
+    debug!(
+        "[gameplay_review][rpc] draft_clip_metadata complete session_id={} drafts={}",
+        session.session_id,
+        drafts.len()
+    );
     Ok(RpcOutcome::single_log(
         drafts,
         "gameplay review clip metadata drafted",
@@ -149,20 +285,35 @@ pub async fn draft_clip_metadata(
 }
 
 async fn workspace_dir() -> Result<std::path::PathBuf, String> {
+    debug!("[gameplay_review][rpc] workspace_dir load start");
     let config = Config::load_or_init()
         .await
         .map_err(|err| format!("gameplay review config load failed: {err}"))?;
-    Ok(store::workspace_dir_from_config(&config))
+    let workspace_dir = store::workspace_dir_from_config(&config);
+    trace!(
+        "[gameplay_review][rpc] workspace_dir resolved path={}",
+        workspace_dir.display()
+    );
+    Ok(workspace_dir)
 }
 
 fn build_session(payload: GameplayReviewSessionInput) -> Result<GameplayReviewSession, String> {
+    debug!(
+        "[gameplay_review][rpc] build_session start game_id={} session_title={} frames={}",
+        payload.game_id,
+        payload.session_title,
+        payload.frames.len()
+    );
     if payload.game_id.trim().is_empty() {
+        warn!("[gameplay_review][rpc] build_session rejected empty game_id");
         return Err("game_id is required".to_string());
     }
     if payload.session_title.trim().is_empty() {
+        warn!("[gameplay_review][rpc] build_session rejected empty session_title");
         return Err("session_title is required".to_string());
     }
     if payload.frames.is_empty() {
+        warn!("[gameplay_review][rpc] build_session rejected empty frames");
         return Err("at least one frame is required".to_string());
     }
 
@@ -193,8 +344,6 @@ async fn analyze_session_frames(
     platform_overrides: &[String],
 ) -> GameplayReviewAnalysis {
     let limit = max_highlights.unwrap_or(5).clamp(1, 8);
-    let mut highlights = Vec::new();
-    let mut clip_candidates = Vec::new();
     let platforms: Vec<String> = if platform_overrides.is_empty() {
         DEFAULT_PLATFORMS
             .iter()
@@ -203,8 +352,39 @@ async fn analyze_session_frames(
     } else {
         platform_overrides.to_vec()
     };
+    debug!(
+        "[gameplay_review][analysis] start session_id={} game_id={} frames={} limit={} platforms={}",
+        session.session_id,
+        session.game_id,
+        session.frames.len(),
+        limit,
+        platforms.len()
+    );
+    if let Some(preset) = preset {
+        trace!(
+            "[gameplay_review][analysis] using preset game_id={} coaching_focus={} audio_feedback={} spoiler_mode={}",
+            preset.game_id,
+            preset.coaching_focus.len(),
+            preset.audio_feedback,
+            preset.spoiler_mode.as_str()
+        );
+    } else {
+        trace!(
+            "[gameplay_review][analysis] no preset for session_id={}",
+            session.session_id
+        );
+    }
+
+    let mut highlights = Vec::new();
+    let mut clip_candidates = Vec::new();
 
     for (index, frame) in session.frames.iter().enumerate() {
+        trace!(
+            "[gameplay_review][analysis] analyze frame session_id={} frame_index={} file_name={}",
+            session.session_id,
+            index,
+            frame.file_name
+        );
         let capture = CaptureFrame {
             captured_at_ms: frame
                 .captured_at_ms
@@ -218,8 +398,10 @@ async fn analyze_session_frames(
         let summary = match global_engine().analyze_and_persist_frame(capture).await {
             Ok(summary) => summary,
             Err(err) => {
-                log::warn!(
-                    "[gameplay_review] frame analysis fell back to deterministic summary: {}",
+                warn!(
+                    "[gameplay_review][analysis] frame analysis fallback session_id={} frame_index={} error={}",
+                    session.session_id,
+                    index,
                     err
                 );
                 fallback_summary(session, frame, index)
@@ -261,6 +443,13 @@ async fn analyze_session_frames(
 
     let recap = build_recap(session, &highlights, preset);
     let draft_metadata = draft_metadata_from_highlights(session, &highlights, preset, &platforms);
+    debug!(
+        "[gameplay_review][analysis] complete session_id={} highlights={} clips={} drafts={}",
+        session.session_id,
+        highlights.len(),
+        clip_candidates.len(),
+        draft_metadata.len()
+    );
 
     GameplayReviewAnalysis {
         recap,
@@ -277,6 +466,12 @@ fn fallback_summary(
     frame: &GameplayFrameInput,
     index: usize,
 ) -> crate::openhuman::screen_intelligence::VisionSummary {
+    trace!(
+        "[gameplay_review][analysis] fallback_summary session_id={} frame_index={} file_name={}",
+        session.session_id,
+        index,
+        frame.file_name
+    );
     let label = frame
         .file_name
         .rsplit_once('/')
diff --git a/src/openhuman/gameplay_review/schemas.rs b/src/openhuman/gameplay_review/schemas.rs
index a831d2873a..154a7a6c73 100644
--- a/src/openhuman/gameplay_review/schemas.rs
+++ b/src/openhuman/gameplay_review/schemas.rs
@@ -2,6 +2,8 @@ use serde::de::DeserializeOwned;
 use serde::Deserialize;
 use serde_json::{Map, Value};
 
+use log::debug;
+
 use crate::core::all::{ControllerFuture, RegisteredController};
 use crate::core::{ControllerSchema, FieldSchema, TypeSchema};
 use crate::rpc::RpcOutcome;
@@ -118,21 +120,54 @@ fn deserialize_params<T: DeserializeOwned>(params: Map<String, Value>) -> Result
 fn handle_register_session(params: Map<String, Value>) -> ControllerFuture {
     Box::pin(async move {
         let payload = deserialize_params::<GameplayReviewSessionInput>(params)?;
-        to_json(crate::openhuman::gameplay_review::rpc::register_session(payload).await?)
+        debug!(
+            "[gameplay_review][controller] register_session session_game_id={} frames={}",
+            payload.game_id,
+            payload.frames.len()
+        );
+        let result = crate::openhuman::gameplay_review::rpc::register_session(payload).await?;
+        debug!(
+            "[gameplay_review][controller] register_session complete session_id={} game_id={}",
+            result.value.session_id,
+            result.value.game_id
+        );
+        to_json(result)
     })
 }
 
 fn handle_analyze_session(params: Map<String, Value>) -> ControllerFuture {
     Box::pin(async move {
         let payload = deserialize_params::<GameplayReviewAnalysisInput>(params)?;
-        to_json(crate::openhuman::gameplay_review::rpc::analyze_session(payload).await?)
+        debug!(
+            "[gameplay_review][controller] analyze_session session_id={} max_highlights={:?} platforms={}",
+            payload.session_id,
+            payload.max_highlights,
+            payload.platforms.len()
+        );
+        let result = crate::openhuman::gameplay_review::rpc::analyze_session(payload).await?;
+        debug!(
+            "[gameplay_review][controller] analyze_session complete session_id={} analyzed={}",
+            result.value.session_id,
+            result.value.analyzed_at_ms.is_some()
+        );
+        to_json(result)
     })
 }
 
 fn handle_get_session(params: Map<String, Value>) -> ControllerFuture {
     Box::pin(async move {
         let payload = deserialize_params::<SessionIdParams>(params)?;
-        to_json(crate::openhuman::gameplay_review::rpc::get_session(payload.session_id).await?)
+        debug!(
+            "[gameplay_review][controller] get_session session_id={}",
+            payload.session_id
+        );
+        let result = crate::openhuman::gameplay_review::rpc::get_session(payload.session_id).await?;
+        debug!(
+            "[gameplay_review][controller] get_session complete session_id={} game_id={}",
+            result.value.session_id,
+            result.value.game_id
+        );
+        to_json(result)
     })
 }
 
@@ -144,32 +179,86 @@ fn handle_list_sessions(params: Map<String, Value>) -> ControllerFuture {
             game_id: Option<String>,
         }
         let payload = deserialize_params::<Params>(params)?;
-        to_json(crate::openhuman::gameplay_review::rpc::list_sessions(payload.game_id).await?)
+        debug!(
+            "[gameplay_review][controller] list_sessions game_id_filter={:?}",
+            payload.game_id
+        );
+        let result = crate::openhuman::gameplay_review::rpc::list_sessions(payload.game_id).await?;
+        debug!(
+            "[gameplay_review][controller] list_sessions complete count={}",
+            result.value.len()
+        );
+        to_json(result)
     })
 }
 
 fn handle_set_preset(params: Map<String, Value>) -> ControllerFuture {
     Box::pin(async move {
         let payload = deserialize_params::<GameplayPresetInput>(params)?;
-        to_json(crate::openhuman::gameplay_review::rpc::set_preset(payload).await?)
+        debug!(
+            "[gameplay_review][controller] set_preset game_id={} display_name={} focus_items={}",
+            payload.game_id,
+            payload.display_name,
+            payload.coaching_focus.len()
+        );
+        let result = crate::openhuman::gameplay_review::rpc::set_preset(payload).await?;
+        debug!(
+            "[gameplay_review][controller] set_preset complete game_id={}",
+            result.value.game_id
+        );
+        to_json(result)
     })
 }
 
 fn handle_list_presets(_params: Map<String, Value>) -> ControllerFuture {
-    Box::pin(async move { to_json(crate::openhuman::gameplay_review::rpc::list_presets().await?) })
+    Box::pin(async move {
+        debug!("[gameplay_review][controller] list_presets start");
+        let result = crate::openhuman::gameplay_review::rpc::list_presets().await?;
+        debug!(
+            "[gameplay_review][controller] list_presets complete count={}",
+            result.value.len()
+        );
+        to_json(result)
+    })
 }
 
 fn handle_ask_session(params: Map<String, Value>) -> ControllerFuture {
     Box::pin(async move {
         let payload = deserialize_params::<GameplayReviewQuestionInput>(params)?;
-        to_json(crate::openhuman::gameplay_review::rpc::ask_session(payload).await?)
+        let session_id = payload.session_id.clone();
+        debug!(
+            "[gameplay_review][controller] ask_session session_id={} question_len={}",
+            session_id,
+            payload.question.len()
+        );
+        let result = crate::openhuman::gameplay_review::rpc::ask_session(payload).await?;
+        debug!(
+            "[gameplay_review][controller] ask_session complete session_id={} matched_highlights={} suggested_follow_up={}",
+            session_id,
+            result.value.matched_highlights.len(),
+            result.value.suggested_follow_up.len()
+        );
+        to_json(result)
     })
 }
 
 fn handle_draft_clip_metadata(params: Map<String, Value>) -> ControllerFuture {
     Box::pin(async move {
         let payload = deserialize_params::<GameplayReviewClipInput>(params)?;
-        to_json(crate::openhuman::gameplay_review::rpc::draft_clip_metadata(payload).await?)
+        let session_id = payload.session_id.clone();
+        debug!(
+            "[gameplay_review][controller] draft_clip_metadata session_id={} platform={:?} highlight_id={:?}",
+            session_id,
+            payload.platform,
+            payload.highlight_id
+        );
+        let result = crate::openhuman::gameplay_review::rpc::draft_clip_metadata(payload).await?;
+        debug!(
+            "[gameplay_review][controller] draft_clip_metadata complete session_id={} drafts={}",
+            session_id,
+            result.value.len()
+        );
+        to_json(result)
     })
 }
 
diff --git a/src/openhuman/gameplay_review/store.rs b/src/openhuman/gameplay_review/store.rs
index 4d0430dd84..65e6e3acb9 100644
--- a/src/openhuman/gameplay_review/store.rs
+++ b/src/openhuman/gameplay_review/store.rs
@@ -1,6 +1,8 @@
 use std::fs;
 use std::path::{Path, PathBuf};
 
+use log::{debug, trace, warn};
+
 use crate::openhuman::config::Config;
 
 use super::types::{GameplayPresetInput, GameplayReviewPreset, GameplayReviewSession};
@@ -45,6 +47,10 @@ pub fn slugify(value: &str) -> String {
 }
 
 pub fn ensure_dirs(workspace_dir: &Path) -> Result<(), String> {
+    debug!(
+        "[gameplay_review][store] ensure_dirs workspace_dir={}",
+        workspace_dir.display()
+    );
     fs::create_dir_all(sessions_dir(workspace_dir))
         .map_err(|err| format!("failed to create gameplay review sessions dir: {err}"))?;
     fs::create_dir_all(presets_dir(workspace_dir))
@@ -52,6 +58,20 @@ pub fn ensure_dirs(workspace_dir: &Path) -> Result<(), String> {
     Ok(())
 }
 
+fn write_json_atomic(path: &Path, entity: &str, payload: &str) -> Result<(), String> {
+    let tmp_path = path.with_extension("json.tmp");
+    trace!(
+        "[gameplay_review][store] write_json_atomic entity={} path={} tmp_path={}",
+        entity,
+        path.display(),
+        tmp_path.display()
+    );
+    fs::write(&tmp_path, payload)
+        .map_err(|err| format!("failed to write {entity} tmp: {err}"))?;
+    fs::rename(&tmp_path, path).map_err(|err| format!("failed to move {entity} into place: {err}"))?;
+    Ok(())
+}
+
 pub fn session_path(workspace_dir: &Path, session_id: &str) -> PathBuf {
     sessions_dir(workspace_dir).join(format!("{session_id}.json"))
 }
@@ -63,28 +83,58 @@ pub fn preset_path(workspace_dir: &Path, game_id: &str) -> PathBuf {
 pub fn save_session(workspace_dir: &Path, session: &GameplayReviewSession) -> Result<(), String> {
     ensure_dirs(workspace_dir)?;
     let path = session_path(workspace_dir, &session.session_id);
-    let tmp_path = path.with_extension("json.tmp");
+    debug!(
+        "[gameplay_review][store] save_session session_id={} game_id={} path={}",
+        session.session_id,
+        session.game_id,
+        path.display()
+    );
     let payload = serde_json::to_string_pretty(session)
         .map_err(|err| format!("failed to serialize session: {err}"))?;
-    fs::write(&tmp_path, payload).map_err(|err| format!("failed to write session tmp: {err}"))?;
-    fs::rename(&tmp_path, &path).map_err(|err| format!("failed to move session into place: {err}"))?;
+    write_json_atomic(&path, "session", &payload)?;
     Ok(())
 }
 
-pub fn load_session(workspace_dir: &Path, session_id: &str) -> Result<Option<GameplayReviewSession>, String> {
+pub fn load_session(
+    workspace_dir: &Path,
+    session_id: &str,
+) -> Result<Option<GameplayReviewSession>, String> {
     let path = session_path(workspace_dir, session_id);
+    trace!(
+        "[gameplay_review][store] load_session session_id={} path={}",
+        session_id,
+        path.display()
+    );
     if !path.exists() {
+        debug!(
+            "[gameplay_review][store] load_session missing session_id={} path={}",
+            session_id,
+            path.display()
+        );
         return Ok(None);
     }
     let raw = fs::read_to_string(&path).map_err(|err| format!("failed to read session: {err}"))?;
     let session = serde_json::from_str(&raw)
         .map_err(|err| format!("failed to parse session {}: {err}", path.display()))?;
+    debug!(
+        "[gameplay_review][store] load_session loaded session_id={} path={}",
+        session_id,
+        path.display()
+    );
     Ok(Some(session))
 }
 
 pub fn list_sessions(workspace_dir: &Path) -> Result<Vec<GameplayReviewSession>, String> {
     let dir = sessions_dir(workspace_dir);
+    trace!(
+        "[gameplay_review][store] list_sessions dir={}",
+        dir.display()
+    );
     if !dir.exists() {
+        debug!(
+            "[gameplay_review][store] list_sessions empty dir={}",
+            dir.display()
+        );
         return Ok(Vec::new());
     }
 
@@ -97,8 +147,13 @@ pub fn list_sessions(workspace_dir: &Path) -> Result<Vec<GameplayReviewSession>,
         }
         let raw = fs::read_to_string(&path)
             .map_err(|err| format!("failed to read session {}: {err}", path.display()))?;
-        if let Ok(session) = serde_json::from_str::<GameplayReviewSession>(&raw) {
-            sessions.push(session);
+        match serde_json::from_str::<GameplayReviewSession>(&raw) {
+            Ok(session) => sessions.push(session),
+            Err(err) => warn!(
+                "[gameplay_review][store] list_sessions skipped invalid session path={} error={}",
+                path.display(),
+                err
+            ),
         }
     }
 
@@ -108,15 +163,25 @@ pub fn list_sessions(workspace_dir: &Path) -> Result<Vec<GameplayReviewSession>,
             .cmp(&left.imported_at_ms)
             .then(right.analyzed_at_ms.cmp(&left.analyzed_at_ms))
     });
+    debug!(
+        "[gameplay_review][store] list_sessions complete dir={} count={}",
+        dir.display(),
+        sessions.len()
+    );
     Ok(sessions)
 }
 
 pub fn save_preset(workspace_dir: &Path, preset: &GameplayReviewPreset) -> Result<(), String> {
     ensure_dirs(workspace_dir)?;
     let path = preset_path(workspace_dir, &preset.game_id);
+    debug!(
+        "[gameplay_review][store] save_preset game_id={} path={}",
+        preset.game_id,
+        path.display()
+    );
     let payload = serde_json::to_string_pretty(preset)
         .map_err(|err| format!("failed to serialize preset: {err}"))?;
-    fs::write(&path, payload).map_err(|err| format!("failed to write preset: {err}"))?;
+    write_json_atomic(&path, "preset", &payload)?;
     Ok(())
 }
 
@@ -125,18 +190,41 @@ pub fn load_preset(
     game_id: &str,
 ) -> Result<Option<GameplayReviewPreset>, String> {
     let path = preset_path(workspace_dir, game_id);
+    trace!(
+        "[gameplay_review][store] load_preset game_id={} path={}",
+        game_id,
+        path.display()
+    );
     if !path.exists() {
+        debug!(
+            "[gameplay_review][store] load_preset missing game_id={} path={}",
+            game_id,
+            path.display()
+        );
         return Ok(None);
     }
     let raw = fs::read_to_string(&path).map_err(|err| format!("failed to read preset: {err}"))?;
     let preset = serde_json::from_str(&raw)
         .map_err(|err| format!("failed to parse preset {}: {err}", path.display()))?;
+    debug!(
+        "[gameplay_review][store] load_preset loaded game_id={} path={}",
+        game_id,
+        path.display()
+    );
     Ok(Some(preset))
 }
 
 pub fn list_presets(workspace_dir: &Path) -> Result<Vec<GameplayReviewPreset>, String> {
     let dir = presets_dir(workspace_dir);
+    trace!(
+        "[gameplay_review][store] list_presets dir={}",
+        dir.display()
+    );
     if !dir.exists() {
+        debug!(
+            "[gameplay_review][store] list_presets empty dir={}",
+            dir.display()
+        );
         return Ok(Vec::new());
     }
 
@@ -149,12 +237,22 @@ pub fn list_presets(workspace_dir: &Path) -> Result<Vec<GameplayReviewPreset>, S
         }
         let raw = fs::read_to_string(&path)
             .map_err(|err| format!("failed to read preset {}: {err}", path.display()))?;
-        if let Ok(preset) = serde_json::from_str::<GameplayReviewPreset>(&raw) {
-            presets.push(preset);
+        match serde_json::from_str::<GameplayReviewPreset>(&raw) {
+            Ok(preset) => presets.push(preset),
+            Err(err) => warn!(
+                "[gameplay_review][store] list_presets skipped invalid preset path={} error={}",
+                path.display(),
+                err
+            ),
         }
     }
 
     presets.sort_by(|left, right| left.display_name.cmp(&right.display_name));
+    debug!(
+        "[gameplay_review][store] list_presets complete dir={} count={}",
+        dir.display(),
+        presets.len()
+    );
     Ok(presets)
 }
 
diff --git a/src/openhuman/mod.rs b/src/openhuman/mod.rs
index 6716e59c06..a9a4b66bdf 100644
--- a/src/openhuman/mod.rs
+++ b/src/openhuman/mod.rs
@@ -21,6 +21,7 @@ pub mod app_state;
 pub mod approval;
 pub mod autocomplete;
 pub mod billing;
+pub mod gameplay_review;
 pub mod channels;
 pub mod composio;
 pub mod config;
@@ -75,4 +76,3 @@ pub mod webview_apis;
 pub mod webview_notifications;
 pub mod whatsapp_data;
 pub mod workspace;
-pub mod gameplay_review;