Phase 2 abgeschlossen: nachträgliche Änderungen, Grabstein, Richtlinie

Der Kern von harter Regel 2. Jede Änderung an einem Verlaufseintrag schreibt
den vorherigen Stand nach CaseEntryVersion und erhöht die Versionsnummer.
Autor und Entstehungszeit bleiben unangetastet, geprüft gegen die Datenbank.

Die Karenzzeit wird serverseitig aus AppSetting gelesen, nie aus dem Request.
Eine Änderung darin wird versioniert und protokolliert, aber nicht sichtbar
markiert - ein Tippfehler, zwei Minuten später korrigiert, soll die Akte nicht
mit "bearbeitet"-Marken zupflastern. Alles danach ist markiert und verlangt
eine Begründung, ebenso jede Änderung an einem fremden Eintrag.

Die Stelle, an der eine naive Umsetzung leckt, hat einen eigenen Test: wird
minClearance eines Eintrags angehoben, erben alle bestehenden Versionen den
neuen Wert. Sonst liest jemand mit niedrigerer Stufe den ursprünglichen Text
einfach in der Historie.

Zurückziehen setzt einen Grabstein mit Pflichtbegründung, der Text bleibt in
der Datenbank. Rückgängig machen geht und wird wiederum protokolliert.
Systemeinträge sind von jeder Änderung ausgenommen.

Dazu die Änderungsübersicht als Reiter an der Akte - sichtbar für alle, die
die Akte lesen dürfen, nicht nur für die Leitung - und die Adminseite für die
Änderungsrichtlinie.

Zwei Testprobleme dabei behoben: AppSetting ist eine einzige Zeile für die
ganze Anwendung, deshalb laufen die Web-Tests jetzt nacheinander, und die
Testdatenbank startet bei jedem Lauf im Auslieferungszustand. Vorher hing das
Ergebnis davon ab, was der vorige Lauf hinterlassen hatte.

63 Tests, zweimal hintereinander grün. Bündel bei 110,5 kB von 120.

web/app/(app)/faelle/actions.ts

@@ -5,6 +5,7 @@ import { redirect } from 'next/navigation'
 import { z } from 'zod'
 import { requireUser } from '@/lib/auth/current'
 import { AccessError, addEntry, changeStatus, createCase, reclassify } from '@/lib/cases/service'
+import { editEntry, restoreEntry, retractEntry } from '@/lib/cases/entryService'
 import { de } from '@/lib/i18n/de'
 
 export type ActionResult = { ok: true } | { ok: false; error: string }
@@ -121,3 +122,62 @@ export async function reclassifyAction(input: unknown): Promise<ActionResult> {
   revalidatePath('/faelle')
   return { ok: true }
 }
+
+const editSchema = z.object({
+  entryId: z.string().min(1),
+  caseNumber: z.string().min(1),
+  body: z.string().trim().min(1).max(20_000),
+  minClearance: z.coerce.number().int().min(0).max(5).optional(),
+  changeReason: z.string().trim().optional(),
+})
+
+export async function editEntryAction(input: unknown): Promise<ActionResult> {
+  const user = await requireUser()
+  const parsed = editSchema.safeParse(input)
+  if (!parsed.success) return { ok: false, error: de.case.errors.default! }
+
+  try {
+    await editEntry(user.permissions, parsed.data)
+  } catch (error) {
+    return { ok: false, error: toMessage(error) }
+  }
+
+  revalidatePath(`/faelle/${parsed.data.caseNumber}`)
+  return { ok: true }
+}
+
+const retractSchema = z.object({
+  entryId: z.string().min(1),
+  caseNumber: z.string().min(1),
+  reason: z.string().trim().min(10),
+})
+
+export async function retractEntryAction(input: unknown): Promise<ActionResult> {
+  const user = await requireUser()
+  const parsed = retractSchema.safeParse(input)
+  if (!parsed.success) return { ok: false, error: de.entry.reasonTooShort }
+
+  try {
+    await retractEntry(user.permissions, parsed.data)
+  } catch (error) {
+    return { ok: false, error: toMessage(error) }
+  }
+
+  revalidatePath(`/faelle/${parsed.data.caseNumber}`)
+  return { ok: true }
+}
+
+export async function restoreEntryAction(input: unknown): Promise<ActionResult> {
+  const user = await requireUser()
+  const parsed = retractSchema.safeParse(input)
+  if (!parsed.success) return { ok: false, error: de.entry.reasonTooShort }
+
+  try {
+    await restoreEntry(user.permissions, parsed.data)
+  } catch (error) {
+    return { ok: false, error: toMessage(error) }
+  }
+
+  revalidatePath(`/faelle/${parsed.data.caseNumber}`)
+  return { ok: true }
+}