Phase 1: Anmeldung, Rechte und Rollenverwaltung

Auth.js übernimmt nur den OAuth-Ablauf mit state und PKCE, Scope identify. Das
Zugangstoken des Nutzers wird danach verworfen: die Rollen holt der Bot mit
seinem eigenen Token. Die Sitzung steht in der Datenbank, im Cookie liegt nur
userId und sessionId. Ohne die Zeile in Session ist das Cookie wertlos, und nur
so lässt sich eine Sitzung zurückziehen.

Wer sich anmelden darf, steckt in lib/auth/signInPolicy.ts statt im Callback:
das ist die Regel, an der alles hängt, und sie gehört einzeln prüfbar. Kein
Guild-Mitglied, kein Zugang. Keine Rolle mit grantsMembership, kein Zugang.
Notfall-Admins umgehen die Rolle, aber nicht die Guild-Mitgliedschaft.

RoleMapping trägt jetzt grantsMembership, category und isPresent. Der Worker
spiegelt jede Rolle der Guild hierher, damit im Adminbereich nichts von Hand
eingetragen werden muss. Gelöschte Rollen werden nicht entfernt, sondern auf
isPresent = false gesetzt: beim Umzug auf den echten Server bleiben sie als
Vorlage stehen.

Der Worker gleicht über Gateway-Ereignisse und zusätzlich alle fünf Minuten
voll ab. Verliert jemand seine AfiS-Rolle, beendet er dessen Sitzungen sofort
und setzt isActive auf false — der Datensatz bleibt, damit Autorenangaben in
alten Akten gültig bleiben. Das passiert im Worker und nicht in der Web-App,
weil es dort erst beim nächsten Request des Betroffenen griffe.

Zwei Widersprüche der Dokumente aufgelöst, festgehalten in
docs/13-entscheidungen.md: der Ausschluss schlägt auch den Admin (das
Codebeispiel in 03-auth-und-rechte.md übergeht ihn, die Reihenfolge eine Zeile
darüber nicht), und versiegelte Akten fallen im Where-Fragment heraus statt in
einer Prüfung danach.

45 Tests, davon die Negativfälle aus docs/03 gegen eine echte Datenbank: Stufe 2
sieht die Stufe-4-Akte nicht und findet sie auch nicht in der Trefferzahl, ein
Ausgeschlossener mit Stufe 5 kommt nicht heran, ein Rollenentzug beendet die
Sitzungen, und ein Ausfall der Discord-API sieht nicht aus wie eine verweigerte
Mitgliedschaft.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Harlekin
2026-09-01 19:11:53 +02:00
co-authored by Claude Opus 5
parent dbca57cedf
commit ea389c12fa
38 changed files with 2369 additions and 110 deletions
+38 -4
View File
@@ -2,6 +2,7 @@ import { describe, expect, it } from 'vitest'
import { readFileSync, readdirSync, statSync } from 'node:fs'
import { join } from 'node:path'
import { de } from '@/lib/i18n/de'
import { OPT_IN_PERMISSIONS, PERMISSIONS } from '@/lib/auth/permissions'
describe('Sprachkonvention', () => {
it('haelt die Stufenbezeichnungen aus den Texten heraus', () => {
@@ -19,9 +20,21 @@ describe('Sprachkonvention', () => {
it('haelt deutsche Oberflaechentexte aus dem JSX heraus', () => {
// Umlaute und Eszett sind der billigste verlaessliche Marker fuer einen
// deutschen Text. Alles Sichtbare gehoert nach lib/i18n/de.ts, siehe CLAUDE.md.
//
// Geprueft werden Komponenten und Seiten (.tsx) sowie Server Actions:
// deren Fehlermeldungen landen in der Oberflaeche. Kommentare zaehlen
// nicht, die duerfen und sollen deutsch sein.
const offenders: string[] = []
const roots = ['app', 'components']
function stripComments(source: string): string {
// Blockkommentare durch gleich viele Zeilenumbrueche ersetzen, damit die
// Zeilennummern stimmen bleiben.
return source
.replace(/\/\*[\s\S]*?\*\//g, (match) => match.replace(/[^\n]/g, ' '))
.replace(/\/\/.*$/gm, '')
}
function walk(dir: string) {
for (const name of readdirSync(dir)) {
const path = join(dir, name)
@@ -29,11 +42,10 @@ describe('Sprachkonvention', () => {
walk(path)
continue
}
if (!name.endsWith('.tsx')) continue
const source = readFileSync(path, 'utf8')
if (!name.endsWith('.tsx') && name !== 'actions.ts') continue
const source = stripComments(readFileSync(path, 'utf8'))
for (const [index, line] of source.split('\n').entries()) {
const withoutComments = line.replace(/\/\/.*$/, '').replace(/\/\*.*?\*\//g, '')
if (/[äöüÄÖÜß]/.test(withoutComments)) {
if (/[äöüÄÖÜß]/.test(line)) {
offenders.push(`${path}:${index + 1}`)
}
}
@@ -44,3 +56,25 @@ describe('Sprachkonvention', () => {
expect(offenders).toEqual([])
})
})
describe('Rechte-Enum', () => {
it('kennt die vier Rechte, die Phase 1 anlegen muss', () => {
for (const permission of [
'entry.edit_own',
'entry.edit_any',
'entry.retract',
'case.edit_closed',
]) {
expect(PERMISSIONS).toContain(permission)
}
})
it('haelt drei davon im Auslieferungszustand zurueck', () => {
// Der Leiter schaltet sie bewusst frei, siehe docs/03-auth-und-rechte.md.
expect([...OPT_IN_PERMISSIONS].sort()).toEqual([
'entry.edit_any',
'entry.edit_own',
'entry.retract',
])
})
})