feat(notes): add review preview and note sorting

.mockapi/behavior.md

@@ -133,7 +133,7 @@ Require the visible deck and next parent, reject duplicate visible titles in the
 
 Status: inferred
 
-Require the visible deck and return visible note list items for that deck. Support `sortField=title|updated` and `sortDirection=asc|desc`.
+Require the visible deck and return visible note list items for that deck. Support `sortField=dueAt|progress|title|updatedAt` and `sortDirection=asc|desc`.
 
 ## operation:createNote
 

api/mock-server/src/app.test.ts

@@ -247,6 +247,34 @@ describe('mock api app', () => {
     )
   })
 
+  it('sorts deck notes by due date', async () => {
+    const app = await newMockApiApp()
+
+    const response = await app.fetch(
+      new Request('http://localhost/api/v1/decks/world-history/notes?sortField=dueAt&sortDirection=desc'),
+    )
+
+    expect(response.status).toBe(200)
+    await expect(json<Array<{ id: string }>>(response)).resolves.toEqual([
+      expect.objectContaining({ id: 'postwar-institutions' }),
+      expect.objectContaining({ id: 'industrial-revolution-causes' }),
+    ])
+  })
+
+  it('sorts deck notes by progress', async () => {
+    const app = await newMockApiApp()
+
+    const response = await app.fetch(
+      new Request('http://localhost/api/v1/decks/world-history/notes?sortField=progress&sortDirection=asc'),
+    )
+
+    expect(response.status).toBe(200)
+    await expect(json<Array<{ id: string }>>(response)).resolves.toEqual([
+      expect.objectContaining({ id: 'postwar-institutions' }),
+      expect.objectContaining({ id: 'industrial-revolution-causes' }),
+    ])
+  })
+
   it('exposes admin state reset and inspection endpoints', async () => {
     const app = await newMockApiApp()
 

api/mock-server/src/features/decks/repository.ts

@@ -5,7 +5,7 @@ import { visible } from '../../lib/softDelete.ts'
 import { byStringField } from '../../lib/sort.ts'
 
 type SortDirection = 'asc' | 'desc'
-type DeckSortField = 'dueToday' | 'title' | 'updated'
+type DeckSortField = 'dueToday' | 'title' | 'updatedAt'
 
 const sortDecks = (
   decks: DeckRecord[],
@@ -23,7 +23,7 @@ const sortDecks = (
       return (left.dueToday - right.dueToday) * direction
     }
 
-    if (sortField === 'updated') {
+    if (sortField === 'updatedAt') {
       return left.updatedAt.localeCompare(right.updatedAt) * direction
     }
 

api/mock-server/src/features/decks/service.ts

@@ -184,7 +184,9 @@ export class DeckService {
     return {
       sortDirection: query.sortDirection === 'desc' ? 'desc' : 'asc',
       sortField:
-        query.sortField === 'dueToday' || query.sortField === 'title' || query.sortField === 'updated'
+        query.sortField === 'dueToday' ||
+        query.sortField === 'title' ||
+        query.sortField === 'updatedAt'
           ? query.sortField
           : undefined,
     } as const

api/mock-server/src/features/folders/repository.ts

@@ -5,7 +5,7 @@ import { visible } from '../../lib/softDelete.ts'
 import { byStringField } from '../../lib/sort.ts'
 
 type SortDirection = 'asc' | 'desc'
-type FolderSortField = 'title' | 'updated'
+type FolderSortField = 'title' | 'updatedAt'
 
 const sortFolders = (
   folders: FolderRecord[],
@@ -19,7 +19,7 @@ const sortFolders = (
   const direction = sortDirection === 'desc' ? -1 : 1
 
   return [...folders].sort((left, right) => {
-    if (sortField === 'updated') {
+    if (sortField === 'updatedAt') {
       return left.updatedAt.localeCompare(right.updatedAt) * direction
     }
 

api/mock-server/src/features/folders/service.ts

@@ -181,7 +181,8 @@ export class FolderService {
   private parseSortQuery(query: { sortField?: string; sortDirection?: string } = {}) {
     return {
       sortDirection: query.sortDirection === 'desc' ? 'desc' : 'asc',
-      sortField: query.sortField === 'title' || query.sortField === 'updated' ? query.sortField : undefined,
+      sortField:
+        query.sortField === 'title' || query.sortField === 'updatedAt' ? query.sortField : undefined,
     } as const
   }
 

api/mock-server/src/features/notes/repository.test.ts

@@ -0,0 +1,50 @@
+import { describe, expect, it } from 'vitest'
+
+import { newMemoryMockStateStore } from '../../lib/stateStore.ts'
+import { NotesRepository } from './repository.ts'
+
+describe('NotesRepository', () => {
+  it('sorts deck notes by due date', async () => {
+    const notes = new NotesRepository(await newMemoryMockStateStore())
+
+    expect(
+      notes
+        .listByDeck('world-history', {
+          sortDirection: 'asc',
+          sortField: 'dueAt',
+        })
+        .map((note) => note.id),
+    ).toEqual(['industrial-revolution-causes', 'postwar-institutions'])
+
+    expect(
+      notes
+        .listByDeck('world-history', {
+          sortDirection: 'desc',
+          sortField: 'dueAt',
+        })
+      .map((note) => note.id),
+    ).toEqual(['postwar-institutions', 'industrial-revolution-causes'])
+  })
+
+  it('sorts deck notes by progress', async () => {
+    const notes = new NotesRepository(await newMemoryMockStateStore())
+
+    expect(
+      notes
+        .listByDeck('world-history', {
+          sortDirection: 'asc',
+          sortField: 'progress',
+        })
+        .map((note) => note.id),
+    ).toEqual(['postwar-institutions', 'industrial-revolution-causes'])
+
+    expect(
+      notes
+        .listByDeck('world-history', {
+          sortDirection: 'desc',
+          sortField: 'progress',
+        })
+        .map((note) => note.id),
+    ).toEqual(['industrial-revolution-causes', 'postwar-institutions'])
+  })
+})

api/mock-server/src/features/notes/repository.ts

@@ -5,7 +5,7 @@ import { visible } from '../../lib/softDelete.ts'
 import { byStringField } from '../../lib/sort.ts'
 
 type SortDirection = 'asc' | 'desc'
-type NoteSortField = 'title' | 'updated'
+type NoteSortField = 'dueAt' | 'progress' | 'title' | 'updatedAt'
 
 const sortNotes = (
   notes: NoteDetailRecord[],
@@ -19,7 +19,15 @@ const sortNotes = (
   const direction = sortDirection === 'desc' ? -1 : 1
 
   return [...notes].sort((left, right) => {
-    if (sortField === 'updated') {
+    if (sortField === 'dueAt') {
+      return (new Date(left.dueAt).getTime() - new Date(right.dueAt).getTime()) * direction
+    }
+
+    if (sortField === 'progress') {
+      return (left.progress - right.progress) * direction
+    }
+
+    if (sortField === 'updatedAt') {
       return left.updatedAt.localeCompare(right.updatedAt) * direction
     }
 

api/mock-server/src/features/notes/service.ts

@@ -49,7 +49,13 @@ export class NotesService {
   listNotesByDeck(deckId: string, query?: { sortField?: string; sortDirection?: string }): NoteListItem[] {
     const deck = this.decks.require(deckId)
     const sortQuery = query ?? {}
-    const sortField = sortQuery.sortField === 'title' || sortQuery.sortField === 'updated' ? sortQuery.sortField : undefined
+    const sortField =
+      sortQuery.sortField === 'dueAt' ||
+      sortQuery.sortField === 'progress' ||
+      sortQuery.sortField === 'title' ||
+      sortQuery.sortField === 'updatedAt'
+        ? sortQuery.sortField
+        : undefined
     const sortDirection = sortQuery.sortDirection === 'desc' ? 'desc' : 'asc'
 
     return this.notes.listByDeck(deck.id ?? '', { sortField, sortDirection }).map(toNoteListItem)

api/openapi/shared/components.yaml

@@ -197,21 +197,23 @@ components:
       enum:
         - dueToday
         - title
-        - updated
+        - updatedAt
       example: title
 
     FolderSortField:
       type: string
       enum:
         - title
-        - updated
+        - updatedAt
       example: title
 
     NoteSortField:
       type: string
       enum:
+        - dueAt
+        - progress
         - title
-        - updated
+        - updatedAt
       example: title
 
     SortDirection:

ui/src/core/i18n/resources/ar.ts

@@ -132,7 +132,6 @@ export const ar = {
     },
     labels: {
       deleted: 'حذف {{value}}',
-      due: 'مستحق: {{value}}',
       reviewed: 'تمت المراجعة: {{value}}',
       updated: 'تم التحديث {{value}}',
       updatedUppercase: 'تم التحديث {{value}}',

ui/src/core/i18n/resources/bg.ts

@@ -116,7 +116,6 @@ export const bg = {
     },
     labels: {
       deleted: 'Изтрито {{value}}',
-      due: 'Срок: {{value}}',
       reviewed: 'Преговорено: {{value}}',
       updated: 'Обновено {{value}}',
       updatedUppercase: 'ОБНОВЕНО {{value}}',

ui/src/core/i18n/resources/bs.ts

@@ -120,7 +120,6 @@ export const bs = {
     },
     labels: {
       deleted: 'Izbrisano {{value}}',
-      due: 'Rok: {{value}}',
       reviewed: 'Ponovljeno: {{value}}',
       updated: 'Ažurirano {{value}}',
       updatedUppercase: 'AŽURIRANO {{value}}',

ui/src/core/i18n/resources/ca.ts

@@ -120,7 +120,6 @@ export const ca = {
     },
     labels: {
       deleted: 'Suprimit {{value}}',
-      due: 'Venciment: {{value}}',
       reviewed: 'Repassat: {{value}}',
       updated: 'Actualitzat {{value}}',
       updatedUppercase: 'ACTUALITZAT {{value}}',

ui/src/core/i18n/resources/cs.ts

@@ -124,7 +124,6 @@ export const cs = {
     },
     labels: {
       deleted: 'Smazáno {{value}}',
-      due: 'Termín: {{value}}',
       reviewed: 'Opakováno: {{value}}',
       updated: 'Aktualizováno {{value}}',
       updatedUppercase: 'AKTUALIZOVÁNO {{value}}',

ui/src/core/i18n/resources/da.ts

@@ -116,7 +116,6 @@ export const da = {
     },
     labels: {
       deleted: 'Slettet {{value}}',
-      due: 'Forfalder: {{value}}',
       reviewed: 'Gentaget: {{value}}',
       updated: 'Opdateret {{value}}',
       updatedUppercase: 'OPDATERET {{value}}',

ui/src/core/i18n/resources/de.ts

@@ -116,7 +116,6 @@ export const de = {
     },
     labels: {
       deleted: 'Gelöscht {{value}}',
-      due: 'Fällig: {{value}}',
       reviewed: 'Wiederholt: {{value}}',
       updated: 'Aktualisiert {{value}}',
       updatedUppercase: 'AKTUALISIERT {{value}}',

ui/src/core/i18n/resources/el.ts

@@ -116,7 +116,6 @@ export const el = {
     },
     labels: {
       deleted: 'Διαγράφηκε {{value}}',
-      due: 'Προθεσμία: {{value}}',
       reviewed: 'Επαναλήφθηκε: {{value}}',
       updated: 'Ενημερώθηκε {{value}}',
       updatedUppercase: 'ΕΝΗΜΕΡΩΘΗΚΕ {{value}}',

ui/src/core/i18n/resources/en-US.ts

@@ -116,7 +116,6 @@ export const enUS = {
     },
     labels: {
       deleted: 'Deleted {{value}}',
-      due: 'Due: {{value}}',
       reviewed: 'Reviewed: {{value}}',
       updated: 'Updated {{value}}',
       updatedUppercase: 'UPDATED {{value}}',

ui/src/core/i18n/resources/es.ts

@@ -116,7 +116,6 @@ export const es = {
     },
     labels: {
       deleted: 'Eliminado {{value}}',
-      due: 'Vence: {{value}}',
       reviewed: 'Repasado: {{value}}',
       updated: 'Actualizado {{value}}',
       updatedUppercase: 'ACTUALIZADO {{value}}',

ui/src/core/i18n/resources/et.ts

@@ -116,7 +116,6 @@ export const et = {
     },
     labels: {
       deleted: 'Kustutatud {{value}}',
-      due: 'Tähtaeg: {{value}}',
       reviewed: 'Korratud: {{value}}',
       updated: 'Uuendatud {{value}}',
       updatedUppercase: 'UUENDATUD {{value}}',

ui/src/core/i18n/resources/fa.ts

@@ -116,7 +116,6 @@ export const fa = {
     },
     labels: {
       deleted: 'حذف‌شده {{value}}',
-      due: 'موعد: {{value}}',
       reviewed: 'مرور شد: {{value}}',
       updated: 'به‌روزرسانی‌شده {{value}}',
       updatedUppercase: 'به‌روزرسانی‌شده {{value}}',

ui/src/core/i18n/resources/fi.ts

@@ -116,7 +116,6 @@ export const fi = {
     },
     labels: {
       deleted: 'Poistettu {{value}}',
-      due: 'Erääntyy: {{value}}',
       reviewed: 'Kerrattu: {{value}}',
       updated: 'Päivitetty {{value}}',
       updatedUppercase: 'PÄIVITETTY {{value}}',

ui/src/core/i18n/resources/fr.ts

@@ -116,7 +116,6 @@ export const fr = {
     },
     labels: {
       deleted: 'Supprimé {{value}}',
-      due: 'À réviser : {{value}}',
       reviewed: 'Révisé : {{value}}',
       updated: 'Mis à jour {{value}}',
       updatedUppercase: 'MIS À JOUR {{value}}',

ui/src/core/i18n/resources/he.ts

@@ -120,7 +120,6 @@ export const he = {
     },
     labels: {
       deleted: 'נמחק {{value}}',
-      due: 'מועד: {{value}}',
       reviewed: 'נסקר: {{value}}',
       updated: 'עודכן {{value}}',
       updatedUppercase: 'עודכן {{value}}',

ui/src/core/i18n/resources/hr.ts

@@ -120,7 +120,6 @@ export const hr = {
     },
     labels: {
       deleted: 'Izbrisano {{value}}',
-      due: 'Rok: {{value}}',
       reviewed: 'Ponovljeno: {{value}}',
       updated: 'Ažurirano {{value}}',
       updatedUppercase: 'AŽURIRANO {{value}}',