Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
ce224bc2a4 | ||
|
|
a264eb0dce | ||
|
|
afe57c4428 | ||
|
|
0ef0b3d5ca | ||
|
|
9cdd51fba8 | ||
|
|
5713facc75 | ||
|
|
52f2d0ce8e | ||
|
|
7b60667633 | ||
|
|
a122187feb | ||
|
|
1fe41ac2a5 |
+6
-1
@@ -69,9 +69,14 @@ ADMINER_PORT="127.0.0.1:8080"
|
|||||||
# und Passkeys sind davon nicht betroffen.
|
# und Passkeys sind davon nicht betroffen.
|
||||||
APP_SECRET="bitte-aendern-mindestens-32-zeichen-langer-zufallswert"
|
APP_SECRET="bitte-aendern-mindestens-32-zeichen-langer-zufallswert"
|
||||||
|
|
||||||
# Lebensdauer einer Anmeldung in Tagen.
|
# Absolute Lebensdauer einer Anmeldung in Tagen – ab dem Anmelden gerechnet
|
||||||
|
# und nicht verlaengerbar.
|
||||||
SESSION_TTL_DAYS="30"
|
SESSION_TTL_DAYS="30"
|
||||||
|
|
||||||
|
# Leerlauffrist in Stunden: so lange darf eine Sitzung unbenutzt bleiben.
|
||||||
|
# Wird bei jeder Nutzung neu angestossen. 0 schaltet die Pruefung ab.
|
||||||
|
SESSION_IDLE_HOURS="8"
|
||||||
|
|
||||||
# ── Benachrichtigungen ───────────────────────────────────────
|
# ── Benachrichtigungen ───────────────────────────────────────
|
||||||
# ntfy und Webhook brauchen nichts weiter – die Ziele werden in der
|
# ntfy und Webhook brauchen nichts weiter – die Ziele werden in der
|
||||||
# Anwendung unter "Mein Konto" eingerichtet.
|
# Anwendung unter "Mein Konto" eingerichtet.
|
||||||
|
|||||||
+49
-3
@@ -22,8 +22,19 @@ sonst würden Konten ausgesperrt. Nach der Einrichtung ist `/setup` dauerhaft ge
|
|||||||
|
|
||||||
### 0.2 Anmeldung
|
### 0.2 Anmeldung
|
||||||
|
|
||||||
- **Benutzername und Passwort.** Mindestens 10 Zeichen, offensichtliche Passwörter werden
|
- **Benutzername und Passwort.** Mindestens 10 Zeichen. Abgelehnt werden naheliegende
|
||||||
abgelehnt. Hashing mit scrypt.
|
Passwörter, reine Ziffernfolgen, zu wenig verschiedene Zeichen (`abababab`),
|
||||||
|
durchlaufende Folgen (`abcdefgh`) und alles, was den eigenen Benutzernamen enthält.
|
||||||
|
**Keine Zusammensetzungsregeln** — ASVS 2.1.9 rät ausdrücklich davon ab, weil sie zu
|
||||||
|
`Passwort1!` führen statt zu Entropie.
|
||||||
|
- **Hashing mit scrypt**, `N = 2^17, r = 8, p = 1` nach dem OWASP Password Storage Cheat
|
||||||
|
Sheet (rund 0,5 s und 128 MiB je Prüfung). Die Parameter stehen im Hash, ältere bleiben
|
||||||
|
also lesbar — und werden bei der nächsten Anmeldung still auf den aktuellen Stand
|
||||||
|
gehoben. Das ist der einzige Moment, in dem das Klartextpasswort vorliegt.
|
||||||
|
- **Gleiche Antwort für alles Fehlgeschlagene.** Ob ein Konto existiert, gesperrt,
|
||||||
|
deaktiviert oder reines Passkey-Konto ist, sagt die Anmeldung erst, wenn das Passwort
|
||||||
|
stimmt. Davor gibt es genau einen Text und eine vergleichbare Antwortzeit
|
||||||
|
(`fakeVerifyDelay`), damit sich Benutzernamen nicht durchprobieren lassen.
|
||||||
- **Passkey (WebAuthn).** Fingerabdruck, Gesichtserkennung oder Sicherheitsschlüssel. Die
|
- **Passkey (WebAuthn).** Fingerabdruck, Gesichtserkennung oder Sicherheitsschlüssel. Die
|
||||||
Anmeldung läuft über auffindbare Credentials, ein Benutzername ist also nicht nötig.
|
Anmeldung läuft über auffindbare Credentials, ein Benutzername ist also nicht nötig.
|
||||||
Mehrere Passkeys pro Konto, benennbar und einzeln entfernbar.
|
Mehrere Passkeys pro Konto, benennbar und einzeln entfernbar.
|
||||||
@@ -518,7 +529,32 @@ soll jemand gewollt haben. Danach wandert der Zustand allerdings mit — erledig
|
|||||||
das Issue, ein Zurückziehen öffnet es wieder. Schlägt der Aufruf fehl, zieht die Aufgabe
|
das Issue, ein Zurückziehen öffnet es wieder. Schlägt der Aufruf fehl, zieht die Aufgabe
|
||||||
trotzdem um; ein nicht erreichbares Git-System darf das Abhaken nicht verhindern.
|
trotzdem um; ein nicht erreichbares Git-System darf das Abhaken nicht verhindern.
|
||||||
|
|
||||||
**Git → Nebula** über Webhooks, siehe unten.
|
**Git → Nebula** auf zwei Wegen, die sich ergänzen:
|
||||||
|
|
||||||
|
- **Webhook** — sofort, sobald das Issue geschlossen wird. Setzt voraus, dass die Instanz
|
||||||
|
von außen erreichbar ist und der Haken eingetragen wurde (siehe 4.7).
|
||||||
|
- **Beim Abgleich** — verzögert, aber verlässlich. Der Abgleich fragt den Zustand
|
||||||
|
verknüpfter Issues nach und hakt ab, was in Git geschlossen ist. Damit funktioniert der
|
||||||
|
Rückweg auch ohne Webhook, ohne öffentliche Erreichbarkeit und selbst dann, wenn ein
|
||||||
|
Haken einmal ausfällt.
|
||||||
|
|
||||||
|
Gespart wird, wo es geht: Nummern, die der gerade geholte Snapshot als offen führt,
|
||||||
|
werden nicht einzeln nachgefragt. Umgekehrt gilt das nicht — die Liste der offenen
|
||||||
|
Issues ist gedeckelt, aus dem Fehlen einer Nummer folgt also nichts. Pro Durchgang
|
||||||
|
fragt der Abgleich höchstens 25 Issues nach.
|
||||||
|
|
||||||
|
**Der Abgleich hakt nur ab, er nimmt nie zurück.** Er sieht Zustände, keine Ereignisse:
|
||||||
|
ein offenes Issue kann auch bedeuten, dass das Schließen von Nebula aus fehlgeschlagen
|
||||||
|
ist. Ein symmetrischer Abgleich würde dann eine bewusst erledigte Aufgabe wieder
|
||||||
|
aufmachen. Das Zurücknehmen bleibt dem Webhook vorbehalten, der ein echtes „reopened“
|
||||||
|
sieht.
|
||||||
|
|
||||||
|
Ein gelöschtes Issue ist kein erledigtes — die Aufgabe bleibt stehen.
|
||||||
|
|
||||||
|
In beiden Fällen bewirkt ein geschlossenes Issue dasselbe wie ein Haken in der
|
||||||
|
Oberfläche: bei einer wiederkehrenden Aufgabe entsteht die nächste Fassung, und der
|
||||||
|
Vorgang landet im Projektverlauf (ohne Benutzer — das hat nicht jemand in Nebula getan,
|
||||||
|
sondern das Git-System gemeldet).
|
||||||
|
|
||||||
**Die Zustände passen nicht aufeinander, und das bleibt so.** Nebula kennt
|
**Die Zustände passen nicht aufeinander, und das bleibt so.** Nebula kennt
|
||||||
`Offen · In Arbeit · Blockiert · Erledigt`, ein Issue nur offen oder geschlossen. Nach
|
`Offen · In Arbeit · Blockiert · Erledigt`, ein Issue nur offen oder geschlossen. Nach
|
||||||
@@ -760,6 +796,16 @@ nicht verloren gehen. Indizes auf Status, Verbindung und Änderungsdatum.
|
|||||||
`GIT_BLOCK_PRIVATE` und `GIT_ALLOWED_HOSTS`.
|
`GIT_BLOCK_PRIVATE` und `GIT_ALLOWED_HOSTS`.
|
||||||
- **Rate-Limiting** für Anmeldung (je IP und je Konto), ausgehende Git-Aufrufe und
|
- **Rate-Limiting** für Anmeldung (je IP und je Konto), ausgehende Git-Aufrufe und
|
||||||
WebAuthn-Optionen.
|
WebAuthn-Optionen.
|
||||||
|
- **Sitzungen**: Zufallstoken mit 256 Bit, in der Datenbank nur als SHA-256-Hash — wer die
|
||||||
|
Datenbank liest, kann keine Sitzung übernehmen. Geprüft wird bei jeder Anfrage, Abmelden
|
||||||
|
und Sperren wirken also sofort. Zwei Fristen nach ASVS 3.3.2: **absolut** 30 Tage ab
|
||||||
|
Anmeldung (`SESSION_TTL_DAYS`) und **im Leerlauf** 8 Stunden (`SESSION_IDLE_HOURS`).
|
||||||
|
Ein Passwortwechsel entwertet alle Sitzungen und stellt für das eigene Gerät eine neue
|
||||||
|
aus.
|
||||||
|
- **Cookies**: `HttpOnly`, `SameSite=Strict`, `Secure` sobald die App über HTTPS läuft, und
|
||||||
|
dann zusätzlich mit dem Präfix `__Host-`. Der Präfix lässt den Browser selbst erzwingen,
|
||||||
|
dass kein `Domain` gesetzt ist — eine andere Subdomain kann den Cookie damit nicht
|
||||||
|
überschreiben und keine Sitzung unterschieben.
|
||||||
- Access Tokens verschlüsselt mit **AES-256-GCM**, Schlüssel per **scrypt** aus
|
- Access Tokens verschlüsselt mit **AES-256-GCM**, Schlüssel per **scrypt** aus
|
||||||
`APP_SECRET` abgeleitet, Chiffrat versioniert (`v1.<iv>.<tag>.<data>`).
|
`APP_SECRET` abgeleitet, Chiffrat versioniert (`v1.<iv>.<tag>.<data>`).
|
||||||
- Tokens werden nie an den Browser gesendet — alle API-Antworten nutzen eine Feldauswahl
|
- Tokens werden nie an den Browser gesendet — alle API-Antworten nutzen eine Feldauswahl
|
||||||
|
|||||||
@@ -118,7 +118,8 @@ Alle Variablen sind in [`.env.example`](.env.example) dokumentiert.
|
|||||||
| `APP_URL` | Öffentliche Adresse — bestimmt die WebAuthn-RP-ID | `http://localhost:3000` |
|
| `APP_URL` | Öffentliche Adresse — bestimmt die WebAuthn-RP-ID | `http://localhost:3000` |
|
||||||
| `DATABASE_URL` | PostgreSQL-Verbindung (Prisma) | `postgresql://nebula:nebula@localhost:5432/nebula` |
|
| `DATABASE_URL` | PostgreSQL-Verbindung (Prisma) | `postgresql://nebula:nebula@localhost:5432/nebula` |
|
||||||
| `APP_SECRET` | Schlüssel für die Token-Verschlüsselung (AES-256-GCM) | — **unbedingt setzen** |
|
| `APP_SECRET` | Schlüssel für die Token-Verschlüsselung (AES-256-GCM) | — **unbedingt setzen** |
|
||||||
| `SESSION_TTL_DAYS` | Lebensdauer einer Anmeldung in Tagen | `30` |
|
| `SESSION_TTL_DAYS` | Absolute Lebensdauer einer Anmeldung in Tagen | `30` |
|
||||||
|
| `SESSION_IDLE_HOURS` | Nach so vielen Stunden ohne Nutzung läuft die Sitzung ab (`0` = aus) | `8` |
|
||||||
| `PORT` | Port der Anwendung | `3000` |
|
| `PORT` | Port der Anwendung | `3000` |
|
||||||
| `APP_PORT` | Host-Port-Mapping des App-Containers | `127.0.0.1:3000` |
|
| `APP_PORT` | Host-Port-Mapping des App-Containers | `127.0.0.1:3000` |
|
||||||
| `POSTGRES_USER` / `POSTGRES_PASSWORD` / `POSTGRES_DB` | Zugangsdaten des DB-Containers | `nebula` |
|
| `POSTGRES_USER` / `POSTGRES_PASSWORD` / `POSTGRES_DB` | Zugangsdaten des DB-Containers | `nebula` |
|
||||||
|
|||||||
+234
@@ -1,4 +1,13 @@
|
|||||||
import { expect, test, type APIRequestContext } from "@playwright/test";
|
import { expect, test, type APIRequestContext } from "@playwright/test";
|
||||||
|
import { PrismaClient } from "@prisma/client";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Dieselbe Datenbank wie global-setup.ts. Gebraucht fuer den einen Fall, in
|
||||||
|
* dem ein Zustand herzustellen ist, den die API bewusst nicht anbietet.
|
||||||
|
*/
|
||||||
|
const DATABASE_URL =
|
||||||
|
process.env.E2E_DATABASE_URL ??
|
||||||
|
"postgresql://nebula:nebula@localhost:5432/nebula_e2e_test?schema=public";
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Tests ueber HTTP gegen die laufende Anwendung – also durch Middleware,
|
* Tests ueber HTTP gegen die laufende Anwendung – also durch Middleware,
|
||||||
@@ -279,6 +288,125 @@ test("Mehrere Repositories je Projekt", async ({ request, playwright }) => {
|
|||||||
expect(zumSchluss.repositories[0].repoName).toBe("swr");
|
expect(zumSchluss.repositories[0].repoName).toBe("swr");
|
||||||
});
|
});
|
||||||
|
|
||||||
|
test("Die Anmeldung verraet nicht, welche Konten es gibt", async ({ request, playwright }) => {
|
||||||
|
await login(request, ADMIN);
|
||||||
|
|
||||||
|
// Ein Konto anlegen und stilllegen.
|
||||||
|
const angelegt = await request.post("/api/admin/users", {
|
||||||
|
data: {
|
||||||
|
username: "stillgelegt",
|
||||||
|
password: "ein-ausreichend-langes-passwort",
|
||||||
|
role: "USER",
|
||||||
|
},
|
||||||
|
});
|
||||||
|
expect(angelegt.status()).toBe(201);
|
||||||
|
const id = (await angelegt.json()).user.id;
|
||||||
|
expect((await request.patch(`/api/admin/users/${id}`, { data: { isActive: false } })).ok())
|
||||||
|
.toBeTruthy();
|
||||||
|
|
||||||
|
const anonym = await playwright.request.newContext({
|
||||||
|
baseURL: test.info().project.use.baseURL,
|
||||||
|
});
|
||||||
|
|
||||||
|
// Falsches Passwort auf dem stillgelegten Konto ...
|
||||||
|
const stillgelegt = await anonym.post("/api/auth/login", {
|
||||||
|
data: { username: "stillgelegt", password: "falsches-passwort-hier" },
|
||||||
|
});
|
||||||
|
// ... und auf einem Konto, das es gar nicht gibt.
|
||||||
|
const erfunden = await anonym.post("/api/auth/login", {
|
||||||
|
data: { username: "gibtesnicht", password: "falsches-passwort-hier" },
|
||||||
|
});
|
||||||
|
|
||||||
|
// Beides muss ununterscheidbar sein – sonst laesst sich durchprobieren,
|
||||||
|
// welche Benutzernamen vergeben sind. Frueher kam hier 403 gegen 401.
|
||||||
|
expect(stillgelegt.status()).toBe(erfunden.status());
|
||||||
|
expect(stillgelegt.status()).toBe(401);
|
||||||
|
expect(await stillgelegt.json()).toEqual(await erfunden.json());
|
||||||
|
|
||||||
|
// Mit dem richtigen Passwort darf die Anwendung dann Klartext reden.
|
||||||
|
const mitPasswort = await anonym.post("/api/auth/login", {
|
||||||
|
data: { username: "stillgelegt", password: "ein-ausreichend-langes-passwort" },
|
||||||
|
});
|
||||||
|
expect(mitPasswort.status()).toBe(403);
|
||||||
|
expect((await mitPasswort.json()).error).toMatch(/deaktiviert/);
|
||||||
|
|
||||||
|
await anonym.dispose();
|
||||||
|
await request.delete(`/api/admin/users/${id}`);
|
||||||
|
});
|
||||||
|
|
||||||
|
test("Abgleich hakt Aufgaben ab, deren Issue geschlossen wurde", async ({ request }) => {
|
||||||
|
await login(request, ADMIN);
|
||||||
|
|
||||||
|
// Ein oeffentliches Repository mit sicher geschlossenen Issues. Der
|
||||||
|
// Abgleich laeuft anonym gegen die GitHub-API – kein Token noetig.
|
||||||
|
const projekt = (
|
||||||
|
await (
|
||||||
|
await request.post("/api/projects", {
|
||||||
|
data: { name: "Rueckweg", repoUrl: "https://github.com/vercel/swr" },
|
||||||
|
})
|
||||||
|
).json()
|
||||||
|
).project;
|
||||||
|
const repoId = projekt.repositories[0].id;
|
||||||
|
|
||||||
|
const angelegt = await request.post(`/api/projects/${projekt.id}/tasks`, {
|
||||||
|
data: { title: "Haengt an einem geschlossenen Issue" },
|
||||||
|
});
|
||||||
|
const taskId = (await angelegt.json()).task.id;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Die Verknuepfung direkt in der Datenbank setzen.
|
||||||
|
*
|
||||||
|
* Ueber die API gibt es dafuer keinen Weg, und das ist richtig so: "als
|
||||||
|
* Issue anlegen" wuerde ein neues Issue in einem fremden Repository
|
||||||
|
* erzeugen. Ein Endpunkt nur fuer Tests waere eine Hintertuer im
|
||||||
|
* Produktionscode – dafuer greift der Test lieber selbst zur Datenbank.
|
||||||
|
*
|
||||||
|
* Issue 1 von vercel/swr ist seit Jahren geschlossen.
|
||||||
|
*/
|
||||||
|
const prisma = new PrismaClient({
|
||||||
|
datasources: { db: { url: DATABASE_URL } },
|
||||||
|
});
|
||||||
|
try {
|
||||||
|
await prisma.task.update({
|
||||||
|
where: { id: taskId },
|
||||||
|
data: { sourceRepositoryId: repoId, sourceIssue: 1 },
|
||||||
|
});
|
||||||
|
} finally {
|
||||||
|
await prisma.$disconnect();
|
||||||
|
}
|
||||||
|
|
||||||
|
const vorher = await (await request.get(`/api/projects/${projekt.id}`)).json();
|
||||||
|
const alsAngelegt = vorher.project.tasks.find(
|
||||||
|
(t: { id: string }) => t.id === taskId,
|
||||||
|
);
|
||||||
|
expect(alsAngelegt.sourceIssue).toBe(1);
|
||||||
|
expect(alsAngelegt.status).toBe("TODO");
|
||||||
|
|
||||||
|
// Abgleich anstossen – er fragt den Zustand des Issues nach.
|
||||||
|
const sync = await request.post(`/api/repositories/${repoId}/sync`);
|
||||||
|
const syncBody = await sync.json();
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Dieser Test greift als einziger wirklich nach draussen. Ist GitHub nicht
|
||||||
|
* erreichbar oder das anonyme Rate-Limit ausgeschoepft, sagt er das statt
|
||||||
|
* rot zu werden – ein fremder Dienst darf die Testreihe nicht bestimmen.
|
||||||
|
*/
|
||||||
|
if (!sync.ok()) {
|
||||||
|
test.skip(true, `GitHub nicht erreichbar: ${syncBody.message ?? sync.status()}`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const nachher = await (await request.get(`/api/projects/${projekt.id}`)).json();
|
||||||
|
const abgehakt = nachher.project.tasks.find((t: { id: string }) => t.id === taskId);
|
||||||
|
expect(abgehakt.status).toBe("DONE");
|
||||||
|
expect(abgehakt.doneAt).not.toBeNull();
|
||||||
|
|
||||||
|
// Und ein zweiter Lauf aendert nichts mehr – abgehakt bleibt abgehakt,
|
||||||
|
// ohne dass erneut nachgefragt wird.
|
||||||
|
const nochmal = await request.post(`/api/repositories/${repoId}/sync`);
|
||||||
|
expect((await nochmal.json()).message ?? "").not.toMatch(/abgehakt/);
|
||||||
|
});
|
||||||
|
|
||||||
test("Webhook: Signatur entscheidet, sonst passiert nichts", async ({ request, playwright }) => {
|
test("Webhook: Signatur entscheidet, sonst passiert nichts", async ({ request, playwright }) => {
|
||||||
await login(request, ADMIN);
|
await login(request, ADMIN);
|
||||||
|
|
||||||
@@ -635,6 +763,112 @@ test("Projektvorlagen: aus Projekt sichern und anwenden", async ({ request, play
|
|||||||
expect((await request.delete(`/api/templates/${vorlage.id}`)).ok()).toBeTruthy();
|
expect((await request.delete(`/api/templates/${vorlage.id}`)).ok()).toBeTruthy();
|
||||||
});
|
});
|
||||||
|
|
||||||
|
test("Verlauf blaettert per Cursor, ohne Zeilen zu verlieren", async ({ request, playwright }) => {
|
||||||
|
await login(request, ADMIN);
|
||||||
|
|
||||||
|
// Die bisherigen Tests haben reichlich Verlauf erzeugt. Erst alles auf
|
||||||
|
// einmal holen, dann in kleinen Seiten – beides muss dieselbe Menge sein.
|
||||||
|
const alles = await (await request.get("/api/activity?limit=100")).json();
|
||||||
|
expect(alles.activities.length).toBeGreaterThan(5);
|
||||||
|
|
||||||
|
const ids: string[] = [];
|
||||||
|
let cursor: { vor: string; id: string } | null = null;
|
||||||
|
for (let seite = 0; seite < 60; seite += 1) {
|
||||||
|
const query = cursor
|
||||||
|
? `?limit=3&vor=${encodeURIComponent(cursor.vor)}&id=${encodeURIComponent(cursor.id)}`
|
||||||
|
: "?limit=3";
|
||||||
|
const antwort = await request.get(`/api/activity${query}`);
|
||||||
|
expect(antwort.ok()).toBeTruthy();
|
||||||
|
const seiteBody = await antwort.json();
|
||||||
|
ids.push(...seiteBody.activities.map((eintrag: { id: string }) => eintrag.id));
|
||||||
|
if (!seiteBody.weitere) break;
|
||||||
|
cursor = seiteBody.naechster;
|
||||||
|
expect(cursor).not.toBeNull();
|
||||||
|
}
|
||||||
|
|
||||||
|
// Der eigentliche Punkt: keine Dubletten und keine Luecken, auch wenn
|
||||||
|
// mehrere Eintraege denselben Zeitstempel tragen.
|
||||||
|
expect(new Set(ids).size).toBe(ids.length);
|
||||||
|
expect(ids.length).toBe(alles.activities.length);
|
||||||
|
|
||||||
|
// Absteigend sortiert.
|
||||||
|
const zeiten: number[] = alles.activities.map((eintrag: { createdAt: string }) =>
|
||||||
|
Date.parse(eintrag.createdAt),
|
||||||
|
);
|
||||||
|
for (let i = 1; i < zeiten.length; i += 1) {
|
||||||
|
expect(zeiten[i]).toBeLessThanOrEqual(zeiten[i - 1]);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Unbrauchbare Cursor fangen von vorn an, statt zu scheitern.
|
||||||
|
const kaputt = await request.get("/api/activity?vor=keindatum&id=x");
|
||||||
|
expect(kaputt.ok()).toBeTruthy();
|
||||||
|
expect((await kaputt.json()).activities.length).toBeGreaterThan(0);
|
||||||
|
|
||||||
|
// Und ein fremdes Konto sieht ausschliesslich seinen eigenen Verlauf.
|
||||||
|
const fremd = await playwright.request.newContext({
|
||||||
|
baseURL: test.info().project.use.baseURL,
|
||||||
|
});
|
||||||
|
await login(fremd, ZWEITER);
|
||||||
|
const fremderVerlauf = await (await fremd.get("/api/activity")).json();
|
||||||
|
const eigeneIds = new Set(ids);
|
||||||
|
for (const eintrag of fremderVerlauf.activities) {
|
||||||
|
expect(eigeneIds.has(eintrag.id)).toBe(false);
|
||||||
|
}
|
||||||
|
await fremd.dispose();
|
||||||
|
});
|
||||||
|
|
||||||
|
test("Volltextsuche findet auch Aufgaben", async ({ request, playwright }) => {
|
||||||
|
await login(request, ADMIN);
|
||||||
|
|
||||||
|
const projekt = (
|
||||||
|
await (await request.post("/api/projects", { data: { name: "Suchprojekt" } })).json()
|
||||||
|
).project;
|
||||||
|
|
||||||
|
for (const task of [
|
||||||
|
{
|
||||||
|
title: "Dockerfile aufraeumen",
|
||||||
|
description: "Die Schichten im Container neu ordnen, damit der Bau schneller wird.",
|
||||||
|
},
|
||||||
|
{ title: "Rechnungen exportieren", description: "Monatlicher Export als CSV." },
|
||||||
|
{ title: "Erledigtes mit Container", description: "Auch das soll gefunden werden.", status: "DONE" },
|
||||||
|
{ title: "Ohne Bezug", description: "Voellig anderer Inhalt." },
|
||||||
|
]) {
|
||||||
|
expect((await request.post(`/api/projects/${projekt.id}/tasks`, { data: task })).status()).toBe(
|
||||||
|
201,
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Titel und Beschreibung werden durchsucht.
|
||||||
|
const container = await (await request.get("/api/search?q=Container")).json();
|
||||||
|
const titel = container.tasks.map((treffer: { title: string }) => treffer.title);
|
||||||
|
expect(titel).toContain("Dockerfile aufraeumen");
|
||||||
|
// Erledigtes bleibt drin – wer sucht, sucht oft gerade das Abgehakte.
|
||||||
|
expect(titel).toContain("Erledigtes mit Container");
|
||||||
|
expect(titel).not.toContain("Ohne Bezug");
|
||||||
|
|
||||||
|
// Deutsche Stammformen: "Rechnung" findet "Rechnungen".
|
||||||
|
const rechnung = await (await request.get("/api/search?q=Rechnung")).json();
|
||||||
|
expect(rechnung.tasks.map((t: { title: string }) => t.title)).toContain(
|
||||||
|
"Rechnungen exportieren",
|
||||||
|
);
|
||||||
|
|
||||||
|
// Das Projekt taucht in projectIds auf, damit das Brett es einblendet.
|
||||||
|
expect(container.projectIds).toContain(projekt.id);
|
||||||
|
|
||||||
|
// Zu kurze Eingaben liefern leere Listen statt Unsinn.
|
||||||
|
const kurz = await (await request.get("/api/search?q=a")).json();
|
||||||
|
expect(kurz.tasks).toEqual([]);
|
||||||
|
expect(kurz.hits).toEqual([]);
|
||||||
|
|
||||||
|
// Und ein fremdes Konto findet nichts davon.
|
||||||
|
const fremd = await playwright.request.newContext({
|
||||||
|
baseURL: test.info().project.use.baseURL,
|
||||||
|
});
|
||||||
|
await login(fremd, ZWEITER);
|
||||||
|
expect((await (await fremd.get("/api/search?q=Container")).json()).tasks).toHaveLength(0);
|
||||||
|
await fremd.dispose();
|
||||||
|
});
|
||||||
|
|
||||||
test("Der Bildoptimierer ist zu", async ({ playwright }) => {
|
test("Der Bildoptimierer ist zu", async ({ playwright }) => {
|
||||||
// /_next/image reicht jedes Bild durch sharp und damit durch libvips, fuer
|
// /_next/image reicht jedes Bild durch sharp und damit durch libvips, fuer
|
||||||
// das es regelmaessig Schwachstellen gibt. Nebula benutzt next/image
|
// das es regelmaessig Schwachstellen gibt. Nebula benutzt next/image
|
||||||
|
|||||||
+70
-1
@@ -223,6 +223,75 @@ test("Aufgabenuebersicht sammelt ueber alle Projekte und hakt ab", async ({ page
|
|||||||
).toBeVisible();
|
).toBeVisible();
|
||||||
});
|
});
|
||||||
|
|
||||||
|
test("Wochenrueckblick zeigt, was diese Woche passiert ist", async ({ page }) => {
|
||||||
|
// Erreichbar vom Dashboard aus, ohne vierten Punkt in der Kopfleiste.
|
||||||
|
await page.getByRole("link", { name: /Wochenrückblick/ }).click();
|
||||||
|
await expect(page).toHaveURL(/\/rueckblick$/);
|
||||||
|
await expect(page.getByRole("heading", { name: "Diese Woche", level: 1 })).toBeVisible();
|
||||||
|
|
||||||
|
// Die Vorgaenge der bisherigen Tests liegen in dieser Woche, es muss also
|
||||||
|
// etwas dastehen – und das Projekt gehoert an jede Zeile.
|
||||||
|
await expect(page.getByRole("link", { name: "Oberflaechentest" }).first()).toBeVisible();
|
||||||
|
|
||||||
|
// Blättern: zurück und wieder her.
|
||||||
|
await page.getByRole("link", { name: "Eine Woche zurück" }).click();
|
||||||
|
await expect(page).toHaveURL(/woche=-1/);
|
||||||
|
await expect(page.getByRole("heading", { name: "Vorige Woche", level: 1 })).toBeVisible();
|
||||||
|
|
||||||
|
await page.getByRole("link", { name: "Diese Woche", exact: true }).click();
|
||||||
|
await expect(page).toHaveURL(/\/rueckblick$/);
|
||||||
|
|
||||||
|
// Auch ueber die Schnellsuche erreichbar.
|
||||||
|
await page.goto("/");
|
||||||
|
await page.keyboard.press("Control+k");
|
||||||
|
await page.getByPlaceholder(/^Projekt, /).fill("Rückblick");
|
||||||
|
await page.keyboard.press("Enter");
|
||||||
|
await expect(page).toHaveURL(/\/rueckblick/);
|
||||||
|
});
|
||||||
|
|
||||||
|
test("Globale Zeitleiste zeigt den Verlauf ueber alle Projekte", async ({ page }) => {
|
||||||
|
// Vom Wochenrueckblick aus verlinkt.
|
||||||
|
await page.goto("/rueckblick");
|
||||||
|
await page.getByRole("link", { name: /Ganzer Verlauf/ }).click();
|
||||||
|
await expect(page).toHaveURL(/\/verlauf$/);
|
||||||
|
await expect(page.getByRole("heading", { name: "Verlauf", level: 1 })).toBeVisible();
|
||||||
|
|
||||||
|
// Eintraege mit Projektbezug – ohne den ist eine uebergreifende Liste
|
||||||
|
// nicht zu lesen.
|
||||||
|
await expect(page.getByRole("link", { name: "Oberflaechentest" }).first()).toBeVisible();
|
||||||
|
await expect(page.getByText(/Vorgänge geladen|Vorgang geladen/)).toBeVisible();
|
||||||
|
|
||||||
|
// Auch ueber die Schnellsuche erreichbar.
|
||||||
|
await page.goto("/");
|
||||||
|
await page.keyboard.press("Control+k");
|
||||||
|
await page.getByPlaceholder(/^Projekt, /).fill("Verlauf");
|
||||||
|
await page.keyboard.press("Enter");
|
||||||
|
await expect(page).toHaveURL(/\/verlauf/);
|
||||||
|
});
|
||||||
|
|
||||||
|
test("Aufgabendashboard laesst sich durchsuchen", async ({ page }) => {
|
||||||
|
await page.getByRole("link", { name: "Aufgaben", exact: true }).click();
|
||||||
|
await expect(page.getByRole("heading", { name: "Aufgaben", level: 1 })).toBeVisible();
|
||||||
|
|
||||||
|
const suchfeld = page.getByLabel("Aufgaben durchsuchen");
|
||||||
|
await expect(suchfeld).toBeVisible();
|
||||||
|
|
||||||
|
// Aus den frueheren Tests gibt es "Erste Aufgabe" und "Zweite Aufgabe".
|
||||||
|
await suchfeld.fill("Erste");
|
||||||
|
await expect(page.getByRole("listitem").filter({ hasText: "Erste Aufgabe" })).toBeVisible();
|
||||||
|
await expect(page.getByRole("listitem").filter({ hasText: "Zweite Aufgabe" })).toBeHidden();
|
||||||
|
|
||||||
|
// Ohne Treffer erklaert der Leerzustand, worin gesucht wurde.
|
||||||
|
await suchfeld.fill("gibtesnichtxyz");
|
||||||
|
await expect(page.getByText("Keine Treffer")).toBeVisible();
|
||||||
|
await expect(page.getByText(/Titel, Beschreibung, Labels und Projektname/)).toBeVisible();
|
||||||
|
|
||||||
|
// Zuruecksetzen bringt alles wieder.
|
||||||
|
await page.getByRole("button", { name: "Suche zuruecksetzen" }).click();
|
||||||
|
await expect(suchfeld).toHaveValue("");
|
||||||
|
await expect(page.getByRole("listitem").filter({ hasText: "Erste Aufgabe" })).toBeVisible();
|
||||||
|
});
|
||||||
|
|
||||||
test("Schnellsuche findet Projekt und Notiz", async ({ page }) => {
|
test("Schnellsuche findet Projekt und Notiz", async ({ page }) => {
|
||||||
await page.keyboard.press("Control+k");
|
await page.keyboard.press("Control+k");
|
||||||
await expect(page.getByRole("dialog", { name: "Schnellsuche" })).toBeVisible();
|
await expect(page.getByRole("dialog", { name: "Schnellsuche" })).toBeVisible();
|
||||||
@@ -230,7 +299,7 @@ test("Schnellsuche findet Projekt und Notiz", async ({ page }) => {
|
|||||||
// Ohne Eingabe stehen Projekte und Bereiche bereit.
|
// Ohne Eingabe stehen Projekte und Bereiche bereit.
|
||||||
await expect(page.getByRole("dialog").getByText("Oberflaechentest")).toBeVisible();
|
await expect(page.getByRole("dialog").getByText("Oberflaechentest")).toBeVisible();
|
||||||
|
|
||||||
await page.getByPlaceholder(/Projekt, Notiz oder Bereich/).fill("Zaunkoenig");
|
await page.getByPlaceholder(/^Projekt, /).fill("Zaunkoenig");
|
||||||
await expect(page.getByRole("dialog").getByText(/in Oberflaechentest/)).toBeVisible();
|
await expect(page.getByRole("dialog").getByText(/in Oberflaechentest/)).toBeVisible();
|
||||||
|
|
||||||
// Enter springt zum ersten Treffer.
|
// Enter springt zum ersten Treffer.
|
||||||
|
|||||||
+1
-1
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "nebula",
|
"name": "nebula",
|
||||||
"version": "1.7.0",
|
"version": "1.10.0",
|
||||||
"private": true,
|
"private": true,
|
||||||
"description": "Nebula – Dashboard fuer Vibe-Coding Projekte mit Git-Anbindung (GitHub, GitLab, Gitea)",
|
"description": "Nebula – Dashboard fuer Vibe-Coding Projekte mit Git-Anbindung (GitHub, GitLab, Gitea)",
|
||||||
"scripts": {
|
"scripts": {
|
||||||
|
|||||||
@@ -0,0 +1,20 @@
|
|||||||
|
-- Volltextsuche ueber Aufgaben.
|
||||||
|
--
|
||||||
|
-- Wie Note_fulltext_idx ein Ausdrucksindex: Prisma kann so etwas im Schema
|
||||||
|
-- nicht abbilden und kennt ihn deshalb nicht. Nachgeprueft mit
|
||||||
|
-- "prisma migrate diff" gegen das Schema - es kommt "empty migration"
|
||||||
|
-- heraus, der Index wird also weder angelegt noch geloescht. Trotzdem beim
|
||||||
|
-- Erzeugen kuenftiger Migrationen kurz hinsehen.
|
||||||
|
--
|
||||||
|
-- Faellt er doch einmal weg, funktioniert die Suche weiter, nur langsamer:
|
||||||
|
-- Postgres berechnet to_tsvector dann je Zeile.
|
||||||
|
--
|
||||||
|
-- Der Ausdruck muss Zeichen fuer Zeichen dem in src/app/api/search/route.ts
|
||||||
|
-- entsprechen, sonst greift der Index nicht. Die zweistellige Form von
|
||||||
|
-- to_tsvector mit fester Konfiguration ist IMMUTABLE und damit indizierbar.
|
||||||
|
--
|
||||||
|
-- Anders als bei der Notiz ist auch die Beschreibung optional, deshalb steht
|
||||||
|
-- coalesce um beide Spalten.
|
||||||
|
|
||||||
|
CREATE INDEX "Task_fulltext_idx" ON "Task"
|
||||||
|
USING GIN (to_tsvector('german', coalesce("title", '') || ' ' || coalesce("description", '')));
|
||||||
@@ -47,7 +47,7 @@ export async function POST(request: Request) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
const weak = validatePasswordStrength(parsed.data.newPassword);
|
const weak = validatePasswordStrength(parsed.data.newPassword, user.username);
|
||||||
if (weak) {
|
if (weak) {
|
||||||
return NextResponse.json({ error: weak, fields: { newPassword: weak } }, { status: 422 });
|
return NextResponse.json({ error: weak, fields: { newPassword: weak } }, { status: 422 });
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,58 @@
|
|||||||
|
import { NextResponse } from "next/server";
|
||||||
|
import { requireUserApi } from "@/lib/auth/guard";
|
||||||
|
import { prisma } from "@/lib/db";
|
||||||
|
import {
|
||||||
|
SEITEN_ORDNUNG,
|
||||||
|
cursorAus,
|
||||||
|
seitenBedingung,
|
||||||
|
seitenGroesse,
|
||||||
|
} from "@/lib/timeline";
|
||||||
|
|
||||||
|
export const dynamic = "force-dynamic";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Der Verlauf ueber alle Projekte, neueste Eintraege zuerst.
|
||||||
|
*
|
||||||
|
* Geblaettert wird per Cursor (`vor` = Zeitpunkt, `id` = Eintrag), nicht per
|
||||||
|
* Offset – siehe lib/timeline. Zurueck kommt der Cursor fuer die naechste
|
||||||
|
* Seite gleich mit, damit der Aufrufer ihn nicht selbst zusammensetzen muss.
|
||||||
|
*/
|
||||||
|
export async function GET(request: Request) {
|
||||||
|
const auth = await requireUserApi();
|
||||||
|
if (!auth.ok) return auth.response;
|
||||||
|
|
||||||
|
const params = new URL(request.url).searchParams;
|
||||||
|
const cursor = cursorAus(params.get("vor"), params.get("id"));
|
||||||
|
const take = seitenGroesse(params.get("limit"));
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Einen mehr holen als angefragt: daran laesst sich ablesen, ob es
|
||||||
|
* weitergeht, ohne dafuer zaehlen zu muessen.
|
||||||
|
*/
|
||||||
|
const gefunden = await prisma.activity.findMany({
|
||||||
|
where: seitenBedingung(auth.user.id, cursor),
|
||||||
|
orderBy: SEITEN_ORDNUNG,
|
||||||
|
take: take + 1,
|
||||||
|
select: {
|
||||||
|
id: true,
|
||||||
|
kind: true,
|
||||||
|
summary: true,
|
||||||
|
createdAt: true,
|
||||||
|
project: { select: { id: true, name: true, accent: true } },
|
||||||
|
user: { select: { username: true, displayName: true } },
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
const weitere = gefunden.length > take;
|
||||||
|
const activities = weitere ? gefunden.slice(0, take) : gefunden;
|
||||||
|
const letzter = activities[activities.length - 1];
|
||||||
|
|
||||||
|
return NextResponse.json({
|
||||||
|
activities: activities.map((eintrag) => ({
|
||||||
|
...eintrag,
|
||||||
|
createdAt: eintrag.createdAt.toISOString(),
|
||||||
|
})),
|
||||||
|
weitere,
|
||||||
|
naechster: weitere && letzter ? { vor: letzter.createdAt.toISOString(), id: letzter.id } : null,
|
||||||
|
});
|
||||||
|
}
|
||||||
@@ -69,7 +69,7 @@ export async function PATCH(request: Request, { params }: Params) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
if (password) {
|
if (password) {
|
||||||
const weak = validatePasswordStrength(password);
|
const weak = validatePasswordStrength(password, target.username);
|
||||||
if (weak) {
|
if (weak) {
|
||||||
return NextResponse.json({ error: weak, fields: { password: weak } }, { status: 422 });
|
return NextResponse.json({ error: weak, fields: { password: weak } }, { status: 422 });
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -47,7 +47,7 @@ export async function POST(request: Request) {
|
|||||||
|
|
||||||
const { username, displayName, email, password, role } = parsed.data;
|
const { username, displayName, email, password, role } = parsed.data;
|
||||||
|
|
||||||
const weak = validatePasswordStrength(password);
|
const weak = validatePasswordStrength(password, username);
|
||||||
if (weak) {
|
if (weak) {
|
||||||
return NextResponse.json({ error: weak, fields: { password: weak } }, { status: 422 });
|
return NextResponse.json({ error: weak, fields: { password: weak } }, { status: 422 });
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
import { NextResponse } from "next/server";
|
import { NextResponse } from "next/server";
|
||||||
import { createPendingLogin, setMfaCookie } from "@/lib/auth/mfa";
|
import { createPendingLogin, setMfaCookie } from "@/lib/auth/mfa";
|
||||||
import { fakeVerifyDelay, verifyPassword } from "@/lib/auth/password";
|
import { fakeVerifyDelay, hashPassword, needsRehash, verifyPassword } from "@/lib/auth/password";
|
||||||
import { createSession, setSessionCookie } from "@/lib/auth/session";
|
import { createSession, setSessionCookie } from "@/lib/auth/session";
|
||||||
import { lockRemainingSeconds, noteFailedLogin, noteSuccessfulLogin } from "@/lib/auth/users";
|
import { lockRemainingSeconds, noteFailedLogin, noteSuccessfulLogin } from "@/lib/auth/users";
|
||||||
import { prisma } from "@/lib/db";
|
import { prisma } from "@/lib/db";
|
||||||
@@ -42,6 +42,30 @@ export async function POST(request: Request) {
|
|||||||
return jsonError(GENERIC, 401);
|
return jsonError(GENERIC, 401);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Erst das Passwort pruefen, dann ueber den Kontozustand reden.
|
||||||
|
*
|
||||||
|
* Frueher standen die Abfragen auf gesperrt, deaktiviert und
|
||||||
|
* "kein Passwort hinterlegt" **vor** der Pruefung und beantworteten sich
|
||||||
|
* damit ohne jede Kenntnis des Passworts. Wer einen Benutzernamen riet,
|
||||||
|
* bekam an der Antwort gesagt, ob es ihn gibt – genau die Aufzaehlbarkeit,
|
||||||
|
* die der einheitliche Text GENERIC sonst verhindert.
|
||||||
|
*
|
||||||
|
* Jetzt gilt: falsches Passwort → immer dieselbe Antwort. Die
|
||||||
|
* hilfreichere Auskunft bekommt nur, wer das Passwort kennt.
|
||||||
|
*/
|
||||||
|
const valid = user.passwordHash
|
||||||
|
? await verifyPassword(password, user.passwordHash)
|
||||||
|
: // Kein Passwort hinterlegt (reines Passkey-Konto): trotzdem Zeit
|
||||||
|
// verbrennen, damit sich der Fall nicht an der Antwortdauer zeigt.
|
||||||
|
await fakeVerifyDelay().then(() => false);
|
||||||
|
|
||||||
|
if (!valid) {
|
||||||
|
// Fehlversuche nur zaehlen, wo es ein Passwort zu erraten gibt.
|
||||||
|
if (user.passwordHash) await noteFailedLogin(user.id);
|
||||||
|
return jsonError(GENERIC, 401);
|
||||||
|
}
|
||||||
|
|
||||||
const locked = lockRemainingSeconds(user.lockedUntil);
|
const locked = lockRemainingSeconds(user.lockedUntil);
|
||||||
if (locked > 0) {
|
if (locked > 0) {
|
||||||
return jsonError(
|
return jsonError(
|
||||||
@@ -51,22 +75,20 @@ export async function POST(request: Request) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
if (!user.isActive) {
|
if (!user.isActive) {
|
||||||
await fakeVerifyDelay();
|
|
||||||
return jsonError("Dieses Konto ist deaktiviert.", 403);
|
return jsonError("Dieses Konto ist deaktiviert.", 403);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!user.passwordHash) {
|
/**
|
||||||
await fakeVerifyDelay();
|
* Der Aufwand beim Hashen ist gestiegen. Wer sich mit einem Passwort
|
||||||
return jsonError(
|
* anmeldet, dessen Hash noch nach den alten Vorgaben erzeugt wurde,
|
||||||
"Fuer dieses Konto ist kein Passwort hinterlegt – bitte mit Passkey anmelden.",
|
* bekommt hier still einen neuen – es ist der einzige Moment, in dem das
|
||||||
401,
|
* Klartextpasswort vorliegt. Nebenlaeufig, damit die Anmeldung nicht auf
|
||||||
);
|
* die halbe Sekunde wartet.
|
||||||
}
|
*/
|
||||||
|
if (needsRehash(user.passwordHash)) {
|
||||||
const valid = await verifyPassword(password, user.passwordHash);
|
void hashPassword(password)
|
||||||
if (!valid) {
|
.then((neu) => prisma.user.update({ where: { id: user.id }, data: { passwordHash: neu } }))
|
||||||
await noteFailedLogin(user.id);
|
.catch(() => undefined);
|
||||||
return jsonError(GENERIC, 401);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// Das Passwort stimmt. Ist ein zweiter Faktor eingerichtet, entsteht hier
|
// Das Passwort stimmt. Ist ein zweiter Faktor eingerichtet, entsteht hier
|
||||||
|
|||||||
@@ -39,7 +39,7 @@ export async function POST(request: Request) {
|
|||||||
|
|
||||||
const { username, displayName, email, password } = parsed.data;
|
const { username, displayName, email, password } = parsed.data;
|
||||||
|
|
||||||
const weak = validatePasswordStrength(password);
|
const weak = validatePasswordStrength(password, username);
|
||||||
if (weak) {
|
if (weak) {
|
||||||
return NextResponse.json({ error: weak, fields: { password: weak } }, { status: 422 });
|
return NextResponse.json({ error: weak, fields: { password: weak } }, { status: 422 });
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -34,7 +34,7 @@ export async function POST(request: Request) {
|
|||||||
|
|
||||||
const { username, displayName, email, password, mode } = parsed.data;
|
const { username, displayName, email, password, mode } = parsed.data;
|
||||||
|
|
||||||
const weak = validatePasswordStrength(password);
|
const weak = validatePasswordStrength(password, username);
|
||||||
if (weak) {
|
if (weak) {
|
||||||
return NextResponse.json(
|
return NextResponse.json(
|
||||||
{ error: weak, fields: { password: weak } },
|
{ error: weak, fields: { password: weak } },
|
||||||
|
|||||||
@@ -12,16 +12,30 @@ interface Hit {
|
|||||||
snippet: string;
|
snippet: string;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
interface TaskHit {
|
||||||
|
projectId: string;
|
||||||
|
taskId: string;
|
||||||
|
title: string;
|
||||||
|
status: string;
|
||||||
|
/** Ausschnitt aus der Beschreibung; leer, wenn nur der Titel passte. */
|
||||||
|
snippet: string;
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Volltextsuche ueber Notizen.
|
* Volltextsuche ueber Notizen und Aufgaben.
|
||||||
*
|
*
|
||||||
* Die Suche im Board arbeitet auf den bereits geladenen Projekten und deckt
|
* Die Suche im Board arbeitet auf den bereits geladenen Projekten und deckt
|
||||||
* Name, Kurzbeschreibung und Tags ab. Notizen liegen dort nicht vor – die
|
* Name, Kurzbeschreibung und Tags ab. Notizen und Aufgaben liegen dort nicht
|
||||||
* mitzuladen wuerde die Seite unnoetig aufblaehen. Deshalb dieser Endpunkt:
|
* vor – die mitzuladen wuerde die Seite unnoetig aufblaehen. Deshalb dieser
|
||||||
* Postgres sucht mit seiner deutschen Textsuche direkt in der Datenbank.
|
* Endpunkt: Postgres sucht mit seiner deutschen Textsuche direkt in der
|
||||||
|
* Datenbank.
|
||||||
*
|
*
|
||||||
* ts_headline liefert gleich den passenden Ausschnitt mit; die Hervorhebung
|
* ts_headline liefert gleich den passenden Ausschnitt mit; die Hervorhebung
|
||||||
* kommt als [[…]] zurueck, damit die Oberflaeche kein HTML einbetten muss.
|
* kommt als [[…]] zurueck, damit die Oberflaeche kein HTML einbetten muss.
|
||||||
|
*
|
||||||
|
* Das Feld heisst weiterhin "hits" und meint die Notizen – umbenennen haette
|
||||||
|
* nur Aufrufer gebrochen, ohne etwas zu verbessern. Aufgaben kommen als
|
||||||
|
* eigenes Feld dazu.
|
||||||
*/
|
*/
|
||||||
export async function GET(request: Request) {
|
export async function GET(request: Request) {
|
||||||
const auth = await requireUserApi();
|
const auth = await requireUserApi();
|
||||||
@@ -29,7 +43,7 @@ export async function GET(request: Request) {
|
|||||||
|
|
||||||
const query = (new URL(request.url).searchParams.get("q") ?? "").trim();
|
const query = (new URL(request.url).searchParams.get("q") ?? "").trim();
|
||||||
if (query.length < 2) {
|
if (query.length < 2) {
|
||||||
return NextResponse.json({ query, hits: [], projectIds: [] });
|
return NextResponse.json({ query, hits: [], tasks: [], projectIds: [] });
|
||||||
}
|
}
|
||||||
|
|
||||||
// websearch_to_tsquery versteht Eingaben wie "docker -compose" oder
|
// websearch_to_tsquery versteht Eingaben wie "docker -compose" oder
|
||||||
@@ -57,11 +71,48 @@ export async function GET(request: Request) {
|
|||||||
LIMIT 50
|
LIMIT 50
|
||||||
`;
|
`;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Dieselbe Rechnung fuer Aufgaben.
|
||||||
|
*
|
||||||
|
* Der Ausdruck in to_tsvector muss Zeichen fuer Zeichen dem in der
|
||||||
|
* Migration entsprechen (Task_fulltext_idx), sonst greift der Index nicht
|
||||||
|
* und Postgres rechnet ihn je Zeile neu.
|
||||||
|
*
|
||||||
|
* Erledigtes bleibt drin: wer sucht, sucht oft gerade das, was er letzte
|
||||||
|
* Woche abgehakt hat.
|
||||||
|
*/
|
||||||
|
const tasks = await prisma.$queryRaw<TaskHit[]>`
|
||||||
|
SELECT
|
||||||
|
t."projectId" AS "projectId",
|
||||||
|
t."id" AS "taskId",
|
||||||
|
t."title" AS "title",
|
||||||
|
t."status"::text AS "status",
|
||||||
|
ts_headline(
|
||||||
|
'german',
|
||||||
|
coalesce(t."description", ''),
|
||||||
|
websearch_to_tsquery('german', ${query}),
|
||||||
|
'StartSel=[[, StopSel=]], MaxWords=18, MinWords=6, ShortWord=2, MaxFragments=1'
|
||||||
|
) AS "snippet"
|
||||||
|
FROM "Task" t
|
||||||
|
JOIN "Project" p ON p."id" = t."projectId"
|
||||||
|
WHERE p."ownerId" = ${auth.user.id}
|
||||||
|
AND to_tsvector('german', coalesce(t."title", '') || ' ' || coalesce(t."description", ''))
|
||||||
|
@@ websearch_to_tsquery('german', ${query})
|
||||||
|
ORDER BY ts_rank(
|
||||||
|
to_tsvector('german', coalesce(t."title", '') || ' ' || coalesce(t."description", '')),
|
||||||
|
websearch_to_tsquery('german', ${query})
|
||||||
|
) DESC
|
||||||
|
LIMIT 50
|
||||||
|
`;
|
||||||
|
|
||||||
return NextResponse.json({
|
return NextResponse.json({
|
||||||
query,
|
query,
|
||||||
hits,
|
hits,
|
||||||
|
tasks,
|
||||||
// Die Oberflaeche braucht vor allem die betroffenen Projekte, um sie im
|
// Die Oberflaeche braucht vor allem die betroffenen Projekte, um sie im
|
||||||
// Board einzublenden.
|
// Board einzublenden – jetzt aus beiden Quellen.
|
||||||
projectIds: Array.from(new Set(hits.map((hit) => hit.projectId))),
|
projectIds: Array.from(
|
||||||
|
new Set([...hits.map((hit) => hit.projectId), ...tasks.map((hit) => hit.projectId)]),
|
||||||
|
),
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -3,7 +3,7 @@ import { tryDecryptSecret } from "@/lib/crypto";
|
|||||||
import { prisma } from "@/lib/db";
|
import { prisma } from "@/lib/db";
|
||||||
import { rateLimit } from "@/lib/security/rateLimit";
|
import { rateLimit } from "@/lib/security/rateLimit";
|
||||||
import { syncRepository } from "@/lib/sync";
|
import { syncRepository } from "@/lib/sync";
|
||||||
import { statusForIssueState } from "@/lib/tasks/issues";
|
import { completeTaskFromIssue, statusForIssueState } from "@/lib/tasks/issues";
|
||||||
import { refreshDerivedProgress } from "@/lib/tasks";
|
import { refreshDerivedProgress } from "@/lib/tasks";
|
||||||
import { parseEvent, verifySignature } from "@/lib/webhooks/verify";
|
import { parseEvent, verifySignature } from "@/lib/webhooks/verify";
|
||||||
|
|
||||||
@@ -118,13 +118,19 @@ export async function POST(request: Request, { params }: Params) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
const ziel = statusForIssueState(ereignis.open, task.status);
|
const ziel = statusForIssueState(ereignis.open, task.status);
|
||||||
if (ziel !== task.status) {
|
|
||||||
|
if (ziel === "DONE" && task.status !== "DONE") {
|
||||||
|
// Ueber die gemeinsame Funktion, damit ein geschlossenes Issue
|
||||||
|
// dasselbe bewirkt wie ein Haken in der Oberflaeche: naechste Fassung
|
||||||
|
// einer wiederkehrenden Aufgabe und ein Eintrag im Verlauf.
|
||||||
|
await completeTaskFromIssue(task.id, "webhook");
|
||||||
|
} else if (ziel !== task.status) {
|
||||||
|
// Wieder geoeffnet. Das darf nur der Webhook, weil hier ein echtes
|
||||||
|
// "reopened" vorliegt – der Abgleich sieht nur Zustaende und wuerde
|
||||||
|
// sonst eine bewusst erledigte Aufgabe aufmachen.
|
||||||
await prisma.task.update({
|
await prisma.task.update({
|
||||||
where: { id: task.id },
|
where: { id: task.id },
|
||||||
data: {
|
data: { status: ziel, doneAt: null },
|
||||||
status: ziel,
|
|
||||||
doneAt: ziel === "DONE" ? new Date() : null,
|
|
||||||
},
|
|
||||||
});
|
});
|
||||||
await refreshDerivedProgress(task.projectId);
|
await refreshDerivedProgress(task.projectId);
|
||||||
}
|
}
|
||||||
|
|||||||
+11
-1
@@ -1,4 +1,5 @@
|
|||||||
import { CircleDot, GitBranch, Lightbulb, Rocket, Trophy } from "lucide-react";
|
import Link from "next/link";
|
||||||
|
import { CalendarRange, CircleDot, GitBranch, Lightbulb, Rocket, Trophy } from "lucide-react";
|
||||||
import { ProjectBoard } from "@/components/ProjectBoard";
|
import { ProjectBoard } from "@/components/ProjectBoard";
|
||||||
import { StatCard } from "@/components/StatCard";
|
import { StatCard } from "@/components/StatCard";
|
||||||
import { requireUserPage } from "@/lib/auth/guard";
|
import { requireUserPage } from "@/lib/auth/guard";
|
||||||
@@ -50,6 +51,15 @@ export default async function DashboardPage() {
|
|||||||
Ideen festhalten, Status pflegen, Notizen sammeln – und live sehen, was in den
|
Ideen festhalten, Status pflegen, Notizen sammeln – und live sehen, was in den
|
||||||
verknuepften Repositories bei GitHub, GitLab und Gitea passiert.
|
verknuepften Repositories bei GitHub, GitLab und Gitea passiert.
|
||||||
</p>
|
</p>
|
||||||
|
{/* Kein weiterer Punkt in der Kopfleiste – die ist auf dem Telefon
|
||||||
|
schon eng. Der Rueckblick haengt dort, wo man ohnehin hinsieht. */}
|
||||||
|
<Link
|
||||||
|
href="/rueckblick"
|
||||||
|
className="mt-3 inline-flex items-center gap-1.5 text-sm text-violet-300 underline-offset-2 hover:underline"
|
||||||
|
>
|
||||||
|
<CalendarRange className="h-4 w-4" />
|
||||||
|
Wochenrückblick ansehen
|
||||||
|
</Link>
|
||||||
</header>
|
</header>
|
||||||
|
|
||||||
{/* Zwei Spalten schon auf dem Telefon: einspaltig fuellten die fuenf
|
{/* Zwei Spalten schon auf dem Telefon: einspaltig fuellten die fuenf
|
||||||
|
|||||||
@@ -85,8 +85,16 @@ export default async function ProjectPage({ params }: PageProps) {
|
|||||||
progressFromTasks={detail.progressFromTasks}
|
progressFromTasks={detail.progressFromTasks}
|
||||||
/>
|
/>
|
||||||
|
|
||||||
|
{/*
|
||||||
|
min-w-0 an beiden Spalten ist hier nicht Kosmetik, sondern der Kern:
|
||||||
|
ein Rasterkind hat "min-width: auto" und weigert sich damit, unter
|
||||||
|
die Inhaltsbreite seines breitesten Kindes zu schrumpfen. Die
|
||||||
|
Heatmap ist ueber 700px breit – auf dem Telefon zog sie damit die
|
||||||
|
ganze Spalte und mit ihr die Seite auseinander. Erst mit min-w-0
|
||||||
|
greift das overflow-x-auto darin ueberhaupt.
|
||||||
|
*/}
|
||||||
<div className="grid gap-6 lg:grid-cols-12">
|
<div className="grid gap-6 lg:grid-cols-12">
|
||||||
<div className="space-y-6 lg:col-span-5">
|
<div className="min-w-0 space-y-6 lg:col-span-5">
|
||||||
<section className="glass p-5">
|
<section className="glass p-5">
|
||||||
<h2 className="flex items-center gap-2 text-sm font-semibold uppercase tracking-wider text-slate-300">
|
<h2 className="flex items-center gap-2 text-sm font-semibold uppercase tracking-wider text-slate-300">
|
||||||
<FileText className="h-4 w-4 text-cyan-300" />
|
<FileText className="h-4 w-4 text-cyan-300" />
|
||||||
@@ -115,7 +123,7 @@ export default async function ProjectPage({ params }: PageProps) {
|
|||||||
<NotesPanel projectId={detail.id} notes={detail.notes} />
|
<NotesPanel projectId={detail.id} notes={detail.notes} />
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div className="space-y-6 lg:col-span-7">
|
<div className="min-w-0 space-y-6 lg:col-span-7">
|
||||||
<GitPanel
|
<GitPanel
|
||||||
project={detail}
|
project={detail}
|
||||||
connections={connections as unknown as ConnectionSafe[]}
|
connections={connections as unknown as ConnectionSafe[]}
|
||||||
|
|||||||
@@ -0,0 +1,139 @@
|
|||||||
|
import type { Metadata } from "next";
|
||||||
|
import Link from "next/link";
|
||||||
|
import { ArrowLeft, ArrowRight, CalendarRange, History } from "lucide-react";
|
||||||
|
import { WeeklyReview } from "@/components/WeeklyReview";
|
||||||
|
import { requireUserPage } from "@/lib/auth/guard";
|
||||||
|
import { prisma } from "@/lib/db";
|
||||||
|
import { tagesschluessel, wocheUm, wochenName, wochenTitel, zaehleWoche } from "@/lib/review";
|
||||||
|
import type { ReviewDay, ReviewEntry } from "@/types";
|
||||||
|
|
||||||
|
export const dynamic = "force-dynamic";
|
||||||
|
|
||||||
|
export const metadata: Metadata = { title: "Wochenrückblick" };
|
||||||
|
|
||||||
|
/** Wie weit sich zurueckblaettern laesst – der Verlauf haelt ohnehin nur 200 je Projekt. */
|
||||||
|
const MAX_RUECKBLICK = 25;
|
||||||
|
|
||||||
|
export default async function ReviewPage({
|
||||||
|
searchParams,
|
||||||
|
}: {
|
||||||
|
searchParams: Promise<{ woche?: string }>;
|
||||||
|
}) {
|
||||||
|
const user = await requireUserPage();
|
||||||
|
const params = await searchParams;
|
||||||
|
|
||||||
|
// Nur nach hinten blaettern: die Zukunft ist im Rueckblick leer.
|
||||||
|
const gewuenscht = Number(params.woche ?? 0);
|
||||||
|
const versatz = Number.isFinite(gewuenscht)
|
||||||
|
? Math.min(0, Math.max(-MAX_RUECKBLICK, Math.trunc(gewuenscht)))
|
||||||
|
: 0;
|
||||||
|
|
||||||
|
const woche = wocheUm(new Date(), versatz);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Der Verlauf kennt keinen Besitzer – die Trennung laeuft ueber das
|
||||||
|
* Projekt. Die Bedingung steht deshalb als erste in der where-Klausel.
|
||||||
|
*/
|
||||||
|
const activities = await prisma.activity.findMany({
|
||||||
|
where: {
|
||||||
|
project: { ownerId: user.id },
|
||||||
|
createdAt: { gte: woche.von, lt: woche.bis },
|
||||||
|
},
|
||||||
|
include: {
|
||||||
|
project: { select: { id: true, name: true, accent: true } },
|
||||||
|
user: { select: { username: true, displayName: true } },
|
||||||
|
},
|
||||||
|
// meta kommt mit, weil die Kennzahlen daran ablesen, was erledigt
|
||||||
|
// wurde – verlaesslicher als der Anzeigetext.
|
||||||
|
orderBy: { createdAt: "desc" },
|
||||||
|
// Deckel gegen eine Woche, in der ein Abgleich Amok gelaufen ist.
|
||||||
|
take: 500,
|
||||||
|
});
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Nach Kalendertagen buendeln – auf dem Server, damit Server-Render und
|
||||||
|
* Hydration nicht auseinanderlaufen. "Welcher Tag" haengt an der Zeitzone.
|
||||||
|
*/
|
||||||
|
const nachTag = new Map<string, ReviewEntry[]>();
|
||||||
|
for (const eintrag of activities) {
|
||||||
|
const schluessel = tagesschluessel(eintrag.createdAt);
|
||||||
|
const liste = nachTag.get(schluessel) ?? [];
|
||||||
|
liste.push({
|
||||||
|
id: eintrag.id,
|
||||||
|
kind: eintrag.kind,
|
||||||
|
summary: eintrag.summary,
|
||||||
|
createdAt: eintrag.createdAt.toISOString(),
|
||||||
|
project: eintrag.project,
|
||||||
|
user: eintrag.user,
|
||||||
|
});
|
||||||
|
nachTag.set(schluessel, liste);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Neuester Tag zuerst, und nur Tage, an denen etwas war.
|
||||||
|
const tage: ReviewDay[] = [...woche.tage]
|
||||||
|
.reverse()
|
||||||
|
.map((tag) => ({ tag, eintraege: nachTag.get(tag) ?? [] }))
|
||||||
|
.filter((eintrag) => eintrag.eintraege.length > 0);
|
||||||
|
|
||||||
|
const kennzahlen = zaehleWoche(activities);
|
||||||
|
|
||||||
|
return (
|
||||||
|
<div className="space-y-5">
|
||||||
|
<header className="fade-up pt-4">
|
||||||
|
<p className="text-[11px] font-semibold uppercase tracking-[0.3em] text-violet-300/70">
|
||||||
|
Über alle Projekte
|
||||||
|
</p>
|
||||||
|
<div className="mt-2 flex flex-wrap items-end justify-between gap-3">
|
||||||
|
<div>
|
||||||
|
<h1 className="text-2xl font-semibold tracking-tight text-white sm:text-3xl">
|
||||||
|
{wochenName(woche)}
|
||||||
|
</h1>
|
||||||
|
<p className="mt-1 flex items-center gap-1.5 text-sm text-slate-400">
|
||||||
|
<CalendarRange className="h-3.5 w-3.5" />
|
||||||
|
{wochenTitel(woche)}
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<nav className="flex items-center gap-2" aria-label="Woche wählen">
|
||||||
|
<Link
|
||||||
|
href={`/rueckblick?woche=${versatz - 1}`}
|
||||||
|
className="btn btn-ghost px-2.5 py-1.5 text-xs"
|
||||||
|
aria-label="Eine Woche zurück"
|
||||||
|
// Am Ende des Zeitraums bleibt der Verweis stehen, wird aber
|
||||||
|
// wirkungslos – die Seite klemmt den Wert ohnehin.
|
||||||
|
aria-disabled={versatz <= -MAX_RUECKBLICK}
|
||||||
|
>
|
||||||
|
<ArrowLeft className="h-3.5 w-3.5" />
|
||||||
|
Früher
|
||||||
|
</Link>
|
||||||
|
{versatz < 0 ? (
|
||||||
|
<Link
|
||||||
|
href={`/rueckblick?woche=${versatz + 1}`}
|
||||||
|
className="btn btn-ghost px-2.5 py-1.5 text-xs"
|
||||||
|
aria-label="Eine Woche vor"
|
||||||
|
>
|
||||||
|
Später
|
||||||
|
<ArrowRight className="h-3.5 w-3.5" />
|
||||||
|
</Link>
|
||||||
|
) : null}
|
||||||
|
{versatz !== 0 ? (
|
||||||
|
<Link href="/rueckblick" className="btn btn-ghost px-2.5 py-1.5 text-xs">
|
||||||
|
Diese Woche
|
||||||
|
</Link>
|
||||||
|
) : null}
|
||||||
|
<Link href="/verlauf" className="btn btn-ghost px-2.5 py-1.5 text-xs">
|
||||||
|
<History className="h-3.5 w-3.5" />
|
||||||
|
Ganzer Verlauf
|
||||||
|
</Link>
|
||||||
|
</nav>
|
||||||
|
</div>
|
||||||
|
</header>
|
||||||
|
|
||||||
|
<WeeklyReview
|
||||||
|
tage={tage}
|
||||||
|
kennzahlen={kennzahlen}
|
||||||
|
istDieseWoche={versatz === 0}
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -0,0 +1,77 @@
|
|||||||
|
import type { Metadata } from "next";
|
||||||
|
import Link from "next/link";
|
||||||
|
import { CalendarRange, History } from "lucide-react";
|
||||||
|
import { Timeline } from "@/components/Timeline";
|
||||||
|
import { requireUserPage } from "@/lib/auth/guard";
|
||||||
|
import { prisma } from "@/lib/db";
|
||||||
|
import { SEITE, SEITEN_ORDNUNG, seitenBedingung } from "@/lib/timeline";
|
||||||
|
import type { ReviewEntry } from "@/types";
|
||||||
|
|
||||||
|
export const dynamic = "force-dynamic";
|
||||||
|
|
||||||
|
export const metadata: Metadata = { title: "Verlauf" };
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Der Verlauf ueber alle Projekte, fortlaufend.
|
||||||
|
*
|
||||||
|
* Die erste Seite kommt serverseitig mit, damit sofort etwas dasteht; alles
|
||||||
|
* Weitere holt der Knopf per Cursor nach. Anders als der Wochenrueckblick
|
||||||
|
* hat diese Seite keine Grenze und kein Fazit – sie laeuft zurueck, solange
|
||||||
|
* es Eintraege gibt.
|
||||||
|
*/
|
||||||
|
export default async function TimelinePage() {
|
||||||
|
const user = await requireUserPage();
|
||||||
|
|
||||||
|
const gefunden = await prisma.activity.findMany({
|
||||||
|
where: seitenBedingung(user.id, null),
|
||||||
|
orderBy: SEITEN_ORDNUNG,
|
||||||
|
// Einen mehr, um zu wissen, ob es weitergeht.
|
||||||
|
take: SEITE + 1,
|
||||||
|
select: {
|
||||||
|
id: true,
|
||||||
|
kind: true,
|
||||||
|
summary: true,
|
||||||
|
createdAt: true,
|
||||||
|
project: { select: { id: true, name: true, accent: true } },
|
||||||
|
user: { select: { username: true, displayName: true } },
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
const weitere = gefunden.length > SEITE;
|
||||||
|
const erste: ReviewEntry[] = (weitere ? gefunden.slice(0, SEITE) : gefunden).map((eintrag) => ({
|
||||||
|
id: eintrag.id,
|
||||||
|
kind: eintrag.kind,
|
||||||
|
summary: eintrag.summary,
|
||||||
|
createdAt: eintrag.createdAt.toISOString(),
|
||||||
|
project: eintrag.project,
|
||||||
|
user: eintrag.user,
|
||||||
|
}));
|
||||||
|
|
||||||
|
return (
|
||||||
|
<div className="space-y-5">
|
||||||
|
<header className="fade-up pt-4">
|
||||||
|
<p className="text-[11px] font-semibold uppercase tracking-[0.3em] text-violet-300/70">
|
||||||
|
Über alle Projekte
|
||||||
|
</p>
|
||||||
|
<div className="mt-2 flex flex-wrap items-end justify-between gap-3">
|
||||||
|
<div>
|
||||||
|
<h1 className="flex items-center gap-2 text-2xl font-semibold tracking-tight text-white sm:text-3xl">
|
||||||
|
<History className="h-6 w-6 text-violet-300" />
|
||||||
|
Verlauf
|
||||||
|
</h1>
|
||||||
|
<p className="mt-1 text-sm text-slate-400">
|
||||||
|
Alles, was passiert ist – vom Neuesten rückwärts.
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<Link href="/rueckblick" className="btn btn-ghost px-3 py-1.5 text-xs">
|
||||||
|
<CalendarRange className="h-3.5 w-3.5" />
|
||||||
|
Wochenrückblick
|
||||||
|
</Link>
|
||||||
|
</div>
|
||||||
|
</header>
|
||||||
|
|
||||||
|
<Timeline erste={erste} weitere={weitere} />
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -1,23 +1,9 @@
|
|||||||
"use client";
|
"use client";
|
||||||
|
|
||||||
import { useState } from "react";
|
import { useState } from "react";
|
||||||
import {
|
import { Bot, History, Loader2 } from "lucide-react";
|
||||||
AlertTriangle,
|
import { ACTIVITY_KINDS } from "@/components/activityKinds";
|
||||||
ArrowRight,
|
import type { ActivityEntry } from "@/types";
|
||||||
Bot,
|
|
||||||
Download,
|
|
||||||
GitCommitHorizontal,
|
|
||||||
History,
|
|
||||||
Link2,
|
|
||||||
Loader2,
|
|
||||||
NotebookPen,
|
|
||||||
Pencil,
|
|
||||||
Plus,
|
|
||||||
Signpost,
|
|
||||||
Sparkles,
|
|
||||||
Trash2,
|
|
||||||
} from "lucide-react";
|
|
||||||
import type { ActivityEntry, ActivityKind } from "@/types";
|
|
||||||
import { formatDateTime, formatRelative } from "@/lib/utils";
|
import { formatDateTime, formatRelative } from "@/lib/utils";
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -29,21 +15,6 @@ import { formatDateTime, formatRelative } from "@/lib/utils";
|
|||||||
* obersten fuenf liest.
|
* obersten fuenf liest.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
const KINDS: Record<ActivityKind, { icon: typeof Sparkles; color: string }> = {
|
|
||||||
PROJECT_CREATED: { icon: Sparkles, color: "text-violet-300" },
|
|
||||||
PROJECT_UPDATED: { icon: Pencil, color: "text-slate-400" },
|
|
||||||
STATUS_CHANGED: { icon: Signpost, color: "text-cyan-300" },
|
|
||||||
NOTE_ADDED: { icon: NotebookPen, color: "text-blue-300" },
|
|
||||||
NOTE_UPDATED: { icon: Pencil, color: "text-blue-300" },
|
|
||||||
NOTE_DELETED: { icon: Trash2, color: "text-rose-300" },
|
|
||||||
TASK_ADDED: { icon: Plus, color: "text-emerald-300" },
|
|
||||||
TASK_MOVED: { icon: ArrowRight, color: "text-violet-300" },
|
|
||||||
TASK_DELETED: { icon: Trash2, color: "text-rose-300" },
|
|
||||||
TASKS_IMPORTED: { icon: Download, color: "text-amber-300" },
|
|
||||||
REPO_LINKED: { icon: Link2, color: "text-cyan-300" },
|
|
||||||
REPO_COMMITS: { icon: GitCommitHorizontal, color: "text-emerald-300" },
|
|
||||||
SYNC_FAILED: { icon: AlertTriangle, color: "text-rose-300" },
|
|
||||||
};
|
|
||||||
|
|
||||||
export function ActivityFeed({
|
export function ActivityFeed({
|
||||||
projectId,
|
projectId,
|
||||||
@@ -88,7 +59,7 @@ export function ActivityFeed({
|
|||||||
) : (
|
) : (
|
||||||
<ol className="mt-4 space-y-0">
|
<ol className="mt-4 space-y-0">
|
||||||
{entries.map((entry, index) => {
|
{entries.map((entry, index) => {
|
||||||
const meta = KINDS[entry.kind] ?? KINDS.PROJECT_UPDATED;
|
const meta = ACTIVITY_KINDS[entry.kind] ?? ACTIVITY_KINDS.PROJECT_UPDATED;
|
||||||
const Icon = meta.icon;
|
const Icon = meta.icon;
|
||||||
const letzter = index === entries.length - 1;
|
const letzter = index === entries.length - 1;
|
||||||
|
|
||||||
|
|||||||
@@ -0,0 +1,126 @@
|
|||||||
|
import Link from "next/link";
|
||||||
|
import { ACTIVITY_KINDS } from "@/components/activityKinds";
|
||||||
|
import { accentOf } from "@/lib/status";
|
||||||
|
import { cn } from "@/lib/utils";
|
||||||
|
import type { ReviewDay, ReviewEntry } from "@/types";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Darstellung von Verlaufseintraegen – eine Zeile und die Gruppierung nach
|
||||||
|
* Kalendertagen.
|
||||||
|
*
|
||||||
|
* Bewusst ohne "use client" und ohne Hooks: so laesst sich beides sowohl aus
|
||||||
|
* einer Server-Komponente (Wochenrueckblick) als auch aus einer
|
||||||
|
* Client-Komponente (Zeitleiste mit Nachladen) heraus benutzen. Zwei Kopien
|
||||||
|
* derselben Zeile waeren beim naechsten Feinschliff auseinandergelaufen.
|
||||||
|
*
|
||||||
|
* Alle Formate mit fester Zeitzone – dieselbe Ueberlegung wie in lib/utils:
|
||||||
|
* sonst weichen Server-Render und Hydration voneinander ab.
|
||||||
|
*/
|
||||||
|
|
||||||
|
const WOCHENTAG = new Intl.DateTimeFormat("de-DE", {
|
||||||
|
timeZone: "Europe/Berlin",
|
||||||
|
weekday: "long",
|
||||||
|
});
|
||||||
|
const TAG_MONAT = new Intl.DateTimeFormat("de-DE", {
|
||||||
|
timeZone: "Europe/Berlin",
|
||||||
|
day: "numeric",
|
||||||
|
month: "long",
|
||||||
|
});
|
||||||
|
const TAG_MONAT_JAHR = new Intl.DateTimeFormat("de-DE", {
|
||||||
|
timeZone: "Europe/Berlin",
|
||||||
|
day: "numeric",
|
||||||
|
month: "long",
|
||||||
|
year: "numeric",
|
||||||
|
});
|
||||||
|
const UHRZEIT = new Intl.DateTimeFormat("de-DE", {
|
||||||
|
timeZone: "Europe/Berlin",
|
||||||
|
hour: "2-digit",
|
||||||
|
minute: "2-digit",
|
||||||
|
});
|
||||||
|
|
||||||
|
/** "2026-08-17" → ein Zeitpunkt mittags, damit die Zeitzone nichts verschiebt. */
|
||||||
|
function alsDatum(tag: string): Date {
|
||||||
|
const [j, m, t] = tag.split("-").map(Number);
|
||||||
|
return new Date(Date.UTC(j, m - 1, t, 12));
|
||||||
|
}
|
||||||
|
|
||||||
|
export function ActivityDayGroups({
|
||||||
|
tage,
|
||||||
|
/** Bei Tagen aus anderen Jahren gehoert die Jahreszahl dazu. */
|
||||||
|
mitJahr = false,
|
||||||
|
}: {
|
||||||
|
tage: ReviewDay[];
|
||||||
|
mitJahr?: boolean;
|
||||||
|
}) {
|
||||||
|
const format = mitJahr ? TAG_MONAT_JAHR : TAG_MONAT;
|
||||||
|
|
||||||
|
return (
|
||||||
|
<div className="space-y-6">
|
||||||
|
{tage.map((tag) => {
|
||||||
|
const datum = alsDatum(tag.tag);
|
||||||
|
return (
|
||||||
|
<section key={tag.tag}>
|
||||||
|
<div className="mb-2.5 flex flex-wrap items-baseline gap-x-2.5 gap-y-1">
|
||||||
|
<h2 className="text-sm font-semibold uppercase tracking-wider text-slate-300">
|
||||||
|
{WOCHENTAG.format(datum)}
|
||||||
|
</h2>
|
||||||
|
<span className="text-xs text-slate-500">{format.format(datum)}</span>
|
||||||
|
<span className="rounded-full bg-white/6 px-2 py-0.5 text-[11px] tabular-nums text-slate-400">
|
||||||
|
{tag.eintraege.length}
|
||||||
|
</span>
|
||||||
|
<span className="h-px flex-1 bg-gradient-to-r from-white/10 to-transparent" />
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<ul className="glass divide-y divide-white/6 overflow-hidden">
|
||||||
|
{tag.eintraege.map((eintrag) => (
|
||||||
|
<ActivityRow key={eintrag.id} eintrag={eintrag} />
|
||||||
|
))}
|
||||||
|
</ul>
|
||||||
|
</section>
|
||||||
|
);
|
||||||
|
})}
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
export function ActivityRow({ eintrag }: { eintrag: ReviewEntry }) {
|
||||||
|
const art = ACTIVITY_KINDS[eintrag.kind] ?? ACTIVITY_KINDS.PROJECT_UPDATED;
|
||||||
|
const Icon = art.icon;
|
||||||
|
const accent = accentOf(eintrag.project.accent);
|
||||||
|
|
||||||
|
return (
|
||||||
|
<li className="flex items-start gap-2.5 px-3 py-2.5 sm:gap-3 sm:px-4">
|
||||||
|
<span className={cn("mt-0.5 shrink-0", art.color)} title={art.label}>
|
||||||
|
<Icon className="h-4 w-4" />
|
||||||
|
</span>
|
||||||
|
|
||||||
|
<div className="min-w-0 flex-1">
|
||||||
|
<p className="text-sm leading-snug text-slate-200">{eintrag.summary}</p>
|
||||||
|
<div className="mt-1 flex flex-wrap items-center gap-x-2 gap-y-1 text-[11px] text-slate-500">
|
||||||
|
{/* Das Projekt gehoert in jede Zeile – ohne es ist eine
|
||||||
|
projektuebergreifende Liste nicht zu lesen. */}
|
||||||
|
<Link
|
||||||
|
href={`/projects/${eintrag.project.id}`}
|
||||||
|
className="inline-flex max-w-[12rem] items-center gap-1.5 truncate text-slate-400 underline-offset-2 hover:text-violet-300 hover:underline"
|
||||||
|
>
|
||||||
|
<span
|
||||||
|
className="h-1.5 w-1.5 shrink-0 rounded-full"
|
||||||
|
style={{ backgroundColor: accent.from }}
|
||||||
|
aria-hidden="true"
|
||||||
|
/>
|
||||||
|
{eintrag.project.name}
|
||||||
|
</Link>
|
||||||
|
<span className="tabular-nums">{UHRZEIT.format(new Date(eintrag.createdAt))} Uhr</span>
|
||||||
|
{eintrag.user ? (
|
||||||
|
<span className="hidden xs:inline">
|
||||||
|
{eintrag.user.displayName?.trim() || eintrag.user.username}
|
||||||
|
</span>
|
||||||
|
) : (
|
||||||
|
// Ohne Benutzer: das war der Zeitgeber oder ein Webhook.
|
||||||
|
<span className="hidden text-slate-600 xs:inline">automatisch</span>
|
||||||
|
)}
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</li>
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -5,9 +5,11 @@ import { createPortal } from "react-dom";
|
|||||||
import { useRouter } from "next/navigation";
|
import { useRouter } from "next/navigation";
|
||||||
import {
|
import {
|
||||||
Bell,
|
Bell,
|
||||||
|
CalendarRange,
|
||||||
CheckSquare,
|
CheckSquare,
|
||||||
CornerDownLeft,
|
CornerDownLeft,
|
||||||
GitBranch,
|
GitBranch,
|
||||||
|
History,
|
||||||
LayoutDashboard,
|
LayoutDashboard,
|
||||||
Loader2,
|
Loader2,
|
||||||
NotebookPen,
|
NotebookPen,
|
||||||
@@ -16,8 +18,9 @@ import {
|
|||||||
Settings,
|
Settings,
|
||||||
UserRound,
|
UserRound,
|
||||||
} from "lucide-react";
|
} from "lucide-react";
|
||||||
import type { ProjectStatus } from "@prisma/client";
|
import type { ProjectStatus, TaskStatus } from "@prisma/client";
|
||||||
import { statusOf } from "@/lib/status";
|
import { statusOf } from "@/lib/status";
|
||||||
|
import { TASK_STATUS_META } from "@/lib/tasks";
|
||||||
import { cn } from "@/lib/utils";
|
import { cn } from "@/lib/utils";
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -38,6 +41,14 @@ interface ProjectHit {
|
|||||||
tags: string[];
|
tags: string[];
|
||||||
}
|
}
|
||||||
|
|
||||||
|
interface TaskHit {
|
||||||
|
projectId: string;
|
||||||
|
taskId: string;
|
||||||
|
title: string;
|
||||||
|
status: string;
|
||||||
|
snippet: string;
|
||||||
|
}
|
||||||
|
|
||||||
interface NoteHit {
|
interface NoteHit {
|
||||||
projectId: string;
|
projectId: string;
|
||||||
noteId: string;
|
noteId: string;
|
||||||
@@ -56,6 +67,20 @@ interface Action {
|
|||||||
|
|
||||||
const ACTIONS: Action[] = [
|
const ACTIONS: Action[] = [
|
||||||
{ id: "dashboard", label: "Dashboard", hint: "Alle Projekte", icon: LayoutDashboard, href: "/" },
|
{ id: "dashboard", label: "Dashboard", hint: "Alle Projekte", icon: LayoutDashboard, href: "/" },
|
||||||
|
{
|
||||||
|
id: "rueckblick",
|
||||||
|
label: "Wochenrückblick",
|
||||||
|
hint: "Was diese Woche passiert ist",
|
||||||
|
icon: CalendarRange,
|
||||||
|
href: "/rueckblick",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
id: "verlauf",
|
||||||
|
label: "Verlauf",
|
||||||
|
hint: "Alles, was passiert ist – über alle Projekte",
|
||||||
|
icon: History,
|
||||||
|
href: "/verlauf",
|
||||||
|
},
|
||||||
{
|
{
|
||||||
id: "tasks",
|
id: "tasks",
|
||||||
label: "Aufgaben",
|
label: "Aufgaben",
|
||||||
@@ -91,7 +116,8 @@ const ACTIONS: Action[] = [
|
|||||||
type Entry =
|
type Entry =
|
||||||
| { kind: "action"; key: string; action: Action }
|
| { kind: "action"; key: string; action: Action }
|
||||||
| { kind: "project"; key: string; project: ProjectHit }
|
| { kind: "project"; key: string; project: ProjectHit }
|
||||||
| { kind: "note"; key: string; note: NoteHit; projectName: string };
|
| { kind: "note"; key: string; note: NoteHit; projectName: string }
|
||||||
|
| { kind: "task"; key: string; task: TaskHit; projectName: string };
|
||||||
|
|
||||||
export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
||||||
const router = useRouter();
|
const router = useRouter();
|
||||||
@@ -100,6 +126,7 @@ export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
|||||||
const [query, setQuery] = useState("");
|
const [query, setQuery] = useState("");
|
||||||
const [projects, setProjects] = useState<ProjectHit[] | null>(null);
|
const [projects, setProjects] = useState<ProjectHit[] | null>(null);
|
||||||
const [notes, setNotes] = useState<NoteHit[]>([]);
|
const [notes, setNotes] = useState<NoteHit[]>([]);
|
||||||
|
const [aufgaben, setAufgaben] = useState<TaskHit[]>([]);
|
||||||
const [active, setActive] = useState(0);
|
const [active, setActive] = useState(0);
|
||||||
const [loading, setLoading] = useState(false);
|
const [loading, setLoading] = useState(false);
|
||||||
const listRef = useRef<HTMLDivElement>(null);
|
const listRef = useRef<HTMLDivElement>(null);
|
||||||
@@ -138,6 +165,7 @@ export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
|||||||
const needle = query.trim();
|
const needle = query.trim();
|
||||||
if (needle.length < 2) {
|
if (needle.length < 2) {
|
||||||
setNotes([]);
|
setNotes([]);
|
||||||
|
setAufgaben([]);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
const controller = new AbortController();
|
const controller = new AbortController();
|
||||||
@@ -147,8 +175,9 @@ export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
|||||||
signal: controller.signal,
|
signal: controller.signal,
|
||||||
});
|
});
|
||||||
if (!response.ok) return;
|
if (!response.ok) return;
|
||||||
const body = (await response.json()) as { hits: NoteHit[] };
|
const body = (await response.json()) as { hits: NoteHit[]; tasks: TaskHit[] };
|
||||||
setNotes(body.hits ?? []);
|
setNotes(body.hits ?? []);
|
||||||
|
setAufgaben(body.tasks ?? []);
|
||||||
} catch {
|
} catch {
|
||||||
// Abbruch beim Weitertippen ist der Normalfall.
|
// Abbruch beim Weitertippen ist der Normalfall.
|
||||||
}
|
}
|
||||||
@@ -163,6 +192,7 @@ export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
|||||||
if (!open) {
|
if (!open) {
|
||||||
setQuery("");
|
setQuery("");
|
||||||
setNotes([]);
|
setNotes([]);
|
||||||
|
setAufgaben([]);
|
||||||
setActive(0);
|
setActive(0);
|
||||||
}
|
}
|
||||||
}, [open]);
|
}, [open]);
|
||||||
@@ -202,6 +232,17 @@ export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
|||||||
for (const project of passendeProjekte.slice(0, 8)) {
|
for (const project of passendeProjekte.slice(0, 8)) {
|
||||||
result.push({ kind: "project", key: `p-${project.id}`, project });
|
result.push({ kind: "project", key: `p-${project.id}`, project });
|
||||||
}
|
}
|
||||||
|
// Aufgaben vor Notizen: wer tippt, sucht haeufiger etwas zu Tun als
|
||||||
|
// etwas Notiertes.
|
||||||
|
for (const treffer of aufgaben.slice(0, 5)) {
|
||||||
|
result.push({
|
||||||
|
kind: "task",
|
||||||
|
key: `t-${treffer.taskId}`,
|
||||||
|
task: treffer,
|
||||||
|
projectName:
|
||||||
|
(projects ?? []).find((project) => project.id === treffer.projectId)?.name ?? "Projekt",
|
||||||
|
});
|
||||||
|
}
|
||||||
for (const hit of notes.slice(0, 5)) {
|
for (const hit of notes.slice(0, 5)) {
|
||||||
result.push({
|
result.push({
|
||||||
kind: "note",
|
kind: "note",
|
||||||
@@ -215,7 +256,7 @@ export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
|||||||
result.push({ kind: "action", key: `a-${action.id}`, action });
|
result.push({ kind: "action", key: `a-${action.id}`, action });
|
||||||
}
|
}
|
||||||
return result;
|
return result;
|
||||||
}, [query, projects, notes, isAdmin]);
|
}, [query, projects, notes, aufgaben, isAdmin]);
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
setActive((current) => Math.min(current, Math.max(0, entries.length - 1)));
|
setActive((current) => Math.min(current, Math.max(0, entries.length - 1)));
|
||||||
@@ -226,6 +267,7 @@ export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
|||||||
setOpen(false);
|
setOpen(false);
|
||||||
if (entry.kind === "action") router.push(entry.action.href);
|
if (entry.kind === "action") router.push(entry.action.href);
|
||||||
else if (entry.kind === "project") router.push(`/projects/${entry.project.id}`);
|
else if (entry.kind === "project") router.push(`/projects/${entry.project.id}`);
|
||||||
|
else if (entry.kind === "task") router.push(`/projects/${entry.task.projectId}`);
|
||||||
else router.push(`/projects/${entry.note.projectId}`);
|
else router.push(`/projects/${entry.note.projectId}`);
|
||||||
},
|
},
|
||||||
[router],
|
[router],
|
||||||
@@ -276,7 +318,7 @@ export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
|||||||
value={query}
|
value={query}
|
||||||
onChange={(event) => setQuery(event.target.value)}
|
onChange={(event) => setQuery(event.target.value)}
|
||||||
onKeyDown={onInputKey}
|
onKeyDown={onInputKey}
|
||||||
placeholder="Projekt, Notiz oder Bereich…"
|
placeholder="Projekt, Aufgabe, Notiz oder Bereich…"
|
||||||
className="w-full bg-transparent text-sm text-slate-100 outline-none placeholder:text-slate-600"
|
className="w-full bg-transparent text-sm text-slate-100 outline-none placeholder:text-slate-600"
|
||||||
/>
|
/>
|
||||||
{loading ? <Loader2 className="h-3.5 w-3.5 animate-spin text-slate-600" /> : null}
|
{loading ? <Loader2 className="h-3.5 w-3.5 animate-spin text-slate-600" /> : null}
|
||||||
@@ -330,6 +372,23 @@ export function CommandPalette({ isAdmin }: { isAdmin: boolean }) {
|
|||||||
{statusOf(entry.project.status).short}
|
{statusOf(entry.project.status).short}
|
||||||
</span>
|
</span>
|
||||||
</>
|
</>
|
||||||
|
) : entry.kind === "task" ? (
|
||||||
|
<>
|
||||||
|
<CheckSquare className="mt-0.5 h-4 w-4 shrink-0 text-emerald-300/70" />
|
||||||
|
<span className="min-w-0">
|
||||||
|
<span className="block truncate text-sm text-slate-200">
|
||||||
|
{entry.task.title}
|
||||||
|
<span className="ml-1.5 text-[11px] text-slate-500">
|
||||||
|
in {entry.projectName}
|
||||||
|
</span>
|
||||||
|
</span>
|
||||||
|
<span className="block truncate text-[11px] text-slate-500">
|
||||||
|
{entry.task.snippet.replace(/\[\[|\]\]/g, "").trim() ||
|
||||||
|
TASK_STATUS_META[entry.task.status as TaskStatus]?.label ||
|
||||||
|
"Aufgabe"}
|
||||||
|
</span>
|
||||||
|
</span>
|
||||||
|
</>
|
||||||
) : (
|
) : (
|
||||||
<>
|
<>
|
||||||
<NotebookPen className="mt-0.5 h-4 w-4 shrink-0 text-violet-300/70" />
|
<NotebookPen className="mt-0.5 h-4 w-4 shrink-0 text-violet-300/70" />
|
||||||
|
|||||||
@@ -3,6 +3,7 @@
|
|||||||
import { useEffect, useMemo, useRef, useState } from "react";
|
import { useEffect, useMemo, useRef, useState } from "react";
|
||||||
import Link from "next/link";
|
import Link from "next/link";
|
||||||
import {
|
import {
|
||||||
|
CheckSquare,
|
||||||
Columns3,
|
Columns3,
|
||||||
Database,
|
Database,
|
||||||
Download,
|
Download,
|
||||||
@@ -16,7 +17,7 @@ import {
|
|||||||
Upload,
|
Upload,
|
||||||
X,
|
X,
|
||||||
} from "lucide-react";
|
} from "lucide-react";
|
||||||
import type { ProjectStatus } from "@prisma/client";
|
import type { ProjectStatus, TaskStatus } from "@prisma/client";
|
||||||
import { BulkBar } from "@/components/BulkBar";
|
import { BulkBar } from "@/components/BulkBar";
|
||||||
import { ImportDialog } from "@/components/ImportDialog";
|
import { ImportDialog } from "@/components/ImportDialog";
|
||||||
import { KanbanBoard } from "@/components/KanbanBoard";
|
import { KanbanBoard } from "@/components/KanbanBoard";
|
||||||
@@ -25,9 +26,18 @@ import { ProjectDialog } from "@/components/ProjectDialog";
|
|||||||
import { ProjectList } from "@/components/ProjectList";
|
import { ProjectList } from "@/components/ProjectList";
|
||||||
import { Popover } from "@/components/ui/Popover";
|
import { Popover } from "@/components/ui/Popover";
|
||||||
import { STATUS_META, STATUS_ORDER } from "@/lib/status";
|
import { STATUS_META, STATUS_ORDER } from "@/lib/status";
|
||||||
|
import { TASK_STATUS_META } from "@/lib/tasks";
|
||||||
import { cn } from "@/lib/utils";
|
import { cn } from "@/lib/utils";
|
||||||
import type { ConnectionSafe, ProjectListItem } from "@/types";
|
import type { ConnectionSafe, ProjectListItem } from "@/types";
|
||||||
|
|
||||||
|
interface TaskHit {
|
||||||
|
projectId: string;
|
||||||
|
taskId: string;
|
||||||
|
title: string;
|
||||||
|
status: string;
|
||||||
|
snippet: string;
|
||||||
|
}
|
||||||
|
|
||||||
interface NoteHit {
|
interface NoteHit {
|
||||||
projectId: string;
|
projectId: string;
|
||||||
noteId: string;
|
noteId: string;
|
||||||
@@ -135,6 +145,7 @@ export function ProjectBoard({
|
|||||||
const [dataMenuOpen, setDataMenuOpen] = useState(false);
|
const [dataMenuOpen, setDataMenuOpen] = useState(false);
|
||||||
const dataButtonRef = useRef<HTMLButtonElement>(null);
|
const dataButtonRef = useRef<HTMLButtonElement>(null);
|
||||||
const [noteHits, setNoteHits] = useState<NoteHit[]>([]);
|
const [noteHits, setNoteHits] = useState<NoteHit[]>([]);
|
||||||
|
const [taskHits, setTaskHits] = useState<TaskHit[]>([]);
|
||||||
const [selection, setSelection] = useState<Set<string>>(new Set());
|
const [selection, setSelection] = useState<Set<string>>(new Set());
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -147,6 +158,7 @@ export function ProjectBoard({
|
|||||||
const needle = query.trim();
|
const needle = query.trim();
|
||||||
if (needle.length < 2) {
|
if (needle.length < 2) {
|
||||||
setNoteHits([]);
|
setNoteHits([]);
|
||||||
|
setTaskHits([]);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -157,8 +169,9 @@ export function ProjectBoard({
|
|||||||
signal: controller.signal,
|
signal: controller.signal,
|
||||||
});
|
});
|
||||||
if (!response.ok) return;
|
if (!response.ok) return;
|
||||||
const body = (await response.json()) as { hits: NoteHit[] };
|
const body = (await response.json()) as { hits: NoteHit[]; tasks: TaskHit[] };
|
||||||
setNoteHits(body.hits ?? []);
|
setNoteHits(body.hits ?? []);
|
||||||
|
setTaskHits(body.tasks ?? []);
|
||||||
} catch {
|
} catch {
|
||||||
// Abgebrochene Anfragen sind der Normalfall beim Tippen.
|
// Abgebrochene Anfragen sind der Normalfall beim Tippen.
|
||||||
}
|
}
|
||||||
@@ -171,8 +184,12 @@ export function ProjectBoard({
|
|||||||
}, [query]);
|
}, [query]);
|
||||||
|
|
||||||
const noteHitProjectIds = useMemo(
|
const noteHitProjectIds = useMemo(
|
||||||
() => new Set(noteHits.map((hit) => hit.projectId)),
|
() =>
|
||||||
[noteHits],
|
new Set([
|
||||||
|
...noteHits.map((hit) => hit.projectId),
|
||||||
|
...taskHits.map((hit) => hit.projectId),
|
||||||
|
]),
|
||||||
|
[noteHits, taskHits],
|
||||||
);
|
);
|
||||||
|
|
||||||
const counts = useMemo(() => {
|
const counts = useMemo(() => {
|
||||||
@@ -200,7 +217,7 @@ export function ProjectBoard({
|
|||||||
project.repositories.some((repo) =>
|
project.repositories.some((repo) =>
|
||||||
(repo.snapshot?.fullName ?? "").toLowerCase().includes(needle),
|
(repo.snapshot?.fullName ?? "").toLowerCase().includes(needle),
|
||||||
) ||
|
) ||
|
||||||
// Treffer in Notizen kommen vom Server dazu.
|
// Treffer in Notizen und Aufgaben kommen vom Server dazu.
|
||||||
noteHitProjectIds.has(project.id)
|
noteHitProjectIds.has(project.id)
|
||||||
);
|
);
|
||||||
});
|
});
|
||||||
@@ -434,6 +451,60 @@ export function ProjectBoard({
|
|||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
|
{/* Treffer in Aufgaben */}
|
||||||
|
{taskHits.length > 0 ? (
|
||||||
|
<div className="glass fade-up p-4">
|
||||||
|
<h2 className="mb-2.5 flex items-center gap-2 text-[11px] font-semibold uppercase tracking-wider text-slate-400">
|
||||||
|
<CheckSquare className="h-3.5 w-3.5 text-emerald-300" />
|
||||||
|
Treffer in Aufgaben
|
||||||
|
<span className="rounded-full bg-white/6 px-2 py-0.5 tabular-nums text-slate-400">
|
||||||
|
{taskHits.length}
|
||||||
|
</span>
|
||||||
|
</h2>
|
||||||
|
<ul className="space-y-1">
|
||||||
|
{taskHits.slice(0, 8).map((hit) => {
|
||||||
|
const project = projects.find((entry) => entry.id === hit.projectId);
|
||||||
|
const status = TASK_STATUS_META[hit.status as TaskStatus];
|
||||||
|
return (
|
||||||
|
<li key={hit.taskId}>
|
||||||
|
<Link
|
||||||
|
href={`/projects/${hit.projectId}`}
|
||||||
|
className="block rounded-lg px-2.5 py-2 transition-colors hover:bg-white/5"
|
||||||
|
>
|
||||||
|
<span className="flex flex-wrap items-baseline gap-x-2">
|
||||||
|
<span className="text-sm text-slate-200">{hit.title}</span>
|
||||||
|
<span className="text-[11px] text-slate-500">
|
||||||
|
· {project?.name ?? "Projekt"}
|
||||||
|
</span>
|
||||||
|
{status ? (
|
||||||
|
<span
|
||||||
|
className="inline-flex items-center gap-1 text-[11px]"
|
||||||
|
style={{ color: status.color }}
|
||||||
|
>
|
||||||
|
{status.label}
|
||||||
|
</span>
|
||||||
|
) : null}
|
||||||
|
</span>
|
||||||
|
{/* Nur wenn die Beschreibung etwas beitraegt – passte
|
||||||
|
allein der Titel, ist der Ausschnitt leer. */}
|
||||||
|
{hit.snippet.trim() ? (
|
||||||
|
<span className="mt-0.5 block text-xs leading-snug text-slate-400">
|
||||||
|
{highlight(hit.snippet)}
|
||||||
|
</span>
|
||||||
|
) : null}
|
||||||
|
</Link>
|
||||||
|
</li>
|
||||||
|
);
|
||||||
|
})}
|
||||||
|
</ul>
|
||||||
|
{taskHits.length > 8 ? (
|
||||||
|
<p className="mt-2 px-2.5 text-[11px] text-slate-600">
|
||||||
|
… und {taskHits.length - 8} weitere
|
||||||
|
</p>
|
||||||
|
) : null}
|
||||||
|
</div>
|
||||||
|
) : null}
|
||||||
|
|
||||||
{/* Treffer in Notizen */}
|
{/* Treffer in Notizen */}
|
||||||
{noteHits.length > 0 ? (
|
{noteHits.length > 0 ? (
|
||||||
<div className="glass fade-up p-4">
|
<div className="glass fade-up p-4">
|
||||||
|
|||||||
@@ -0,0 +1,126 @@
|
|||||||
|
"use client";
|
||||||
|
|
||||||
|
import { useMemo, useState } from "react";
|
||||||
|
import { ChevronDown, History, Loader2 } from "lucide-react";
|
||||||
|
import { ActivityDayGroups } from "@/components/ActivityRow";
|
||||||
|
import { tagesschluessel } from "@/lib/review";
|
||||||
|
import type { ReviewDay, ReviewEntry } from "@/types";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Die fortlaufende Zeitleiste.
|
||||||
|
*
|
||||||
|
* Client-Komponente, weil hier nachgeladen wird – das ist der Unterschied
|
||||||
|
* zum Wochenrueckblick, der komplett auf dem Server entsteht.
|
||||||
|
*
|
||||||
|
* Die Buendelung nach Tagen passiert hier im Browser. Das ist unbedenklich,
|
||||||
|
* weil tagesschluessel eine feste Zeitzone benutzt: Server und Browser kommen
|
||||||
|
* damit auf denselben Kalendertag, und die erste Seite hydriert ohne
|
||||||
|
* Abweichung.
|
||||||
|
*/
|
||||||
|
export function Timeline({
|
||||||
|
erste,
|
||||||
|
weitere: gabWeitere,
|
||||||
|
}: {
|
||||||
|
erste: ReviewEntry[];
|
||||||
|
weitere: boolean;
|
||||||
|
}) {
|
||||||
|
const [eintraege, setEintraege] = useState(erste);
|
||||||
|
const [weitere, setWeitere] = useState(gabWeitere);
|
||||||
|
const [laedt, setLaedt] = useState(false);
|
||||||
|
const [fehler, setFehler] = useState<string | null>(null);
|
||||||
|
|
||||||
|
const tage: ReviewDay[] = useMemo(() => {
|
||||||
|
// Map behaelt die Einfuegereihenfolge – die Eintraege kommen schon
|
||||||
|
// absteigend, also stimmt die Reihenfolge der Tage von selbst.
|
||||||
|
const nachTag = new Map<string, ReviewEntry[]>();
|
||||||
|
for (const eintrag of eintraege) {
|
||||||
|
const schluessel = tagesschluessel(new Date(eintrag.createdAt));
|
||||||
|
const liste = nachTag.get(schluessel) ?? [];
|
||||||
|
liste.push(eintrag);
|
||||||
|
nachTag.set(schluessel, liste);
|
||||||
|
}
|
||||||
|
return [...nachTag.entries()].map(([tag, liste]) => ({ tag, eintraege: liste }));
|
||||||
|
}, [eintraege]);
|
||||||
|
|
||||||
|
async function mehr() {
|
||||||
|
const letzter = eintraege[eintraege.length - 1];
|
||||||
|
if (!letzter) return;
|
||||||
|
|
||||||
|
setLaedt(true);
|
||||||
|
setFehler(null);
|
||||||
|
try {
|
||||||
|
const antwort = await fetch(
|
||||||
|
`/api/activity?vor=${encodeURIComponent(letzter.createdAt)}&id=${encodeURIComponent(letzter.id)}`,
|
||||||
|
);
|
||||||
|
if (!antwort.ok) throw new Error("Verlauf konnte nicht geladen werden.");
|
||||||
|
const nutzlast = (await antwort.json()) as {
|
||||||
|
activities: ReviewEntry[];
|
||||||
|
weitere: boolean;
|
||||||
|
};
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Gegen Dubletten absichern, auch wenn der Cursor sie ausschliesst:
|
||||||
|
* ein zweimal gedrueckter Knopf oder ein wiederholter Aufruf soll
|
||||||
|
* keine Zeile verdoppeln, und React braucht eindeutige Schluessel.
|
||||||
|
*/
|
||||||
|
setEintraege((bisher) => {
|
||||||
|
const bekannt = new Set(bisher.map((eintrag) => eintrag.id));
|
||||||
|
return [...bisher, ...nutzlast.activities.filter((eintrag) => !bekannt.has(eintrag.id))];
|
||||||
|
});
|
||||||
|
setWeitere(nutzlast.weitere);
|
||||||
|
} catch (gefangen) {
|
||||||
|
setFehler(gefangen instanceof Error ? gefangen.message : String(gefangen));
|
||||||
|
} finally {
|
||||||
|
setLaedt(false);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (eintraege.length === 0) {
|
||||||
|
return (
|
||||||
|
<div className="glass fade-up flex flex-col items-center justify-center gap-4 px-6 py-16 text-center">
|
||||||
|
<span className="grid h-14 w-14 place-items-center rounded-2xl bg-gradient-to-br from-violet-500/25 to-cyan-400/20 text-violet-200">
|
||||||
|
<History className="h-6 w-6" />
|
||||||
|
</span>
|
||||||
|
<div>
|
||||||
|
<h2 className="text-lg font-semibold text-white">Noch kein Verlauf</h2>
|
||||||
|
<p className="mt-1 max-w-md text-sm text-slate-400">
|
||||||
|
Sobald du ein Projekt anlegst, Aufgaben verschiebst oder Notizen schreibst, steht es
|
||||||
|
hier.
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
return (
|
||||||
|
<div className="space-y-5">
|
||||||
|
{/* Die Jahreszahl gehoert dazu: eine Zeitleiste reicht weit zurueck,
|
||||||
|
und "3. Januar" allein sagt dann nicht genug. */}
|
||||||
|
<ActivityDayGroups tage={tage} mitJahr />
|
||||||
|
|
||||||
|
{fehler ? (
|
||||||
|
<p className="rounded-lg border border-rose-500/30 bg-rose-500/10 px-3 py-2 text-sm text-rose-200">
|
||||||
|
{fehler}
|
||||||
|
</p>
|
||||||
|
) : null}
|
||||||
|
|
||||||
|
<div className="flex flex-col items-center gap-2 pt-1">
|
||||||
|
{weitere ? (
|
||||||
|
<button type="button" className="btn btn-ghost" onClick={mehr} disabled={laedt}>
|
||||||
|
{laedt ? (
|
||||||
|
<Loader2 className="h-4 w-4 animate-spin" />
|
||||||
|
) : (
|
||||||
|
<ChevronDown className="h-4 w-4" />
|
||||||
|
)}
|
||||||
|
Ältere laden
|
||||||
|
</button>
|
||||||
|
) : (
|
||||||
|
<p className="text-xs text-slate-600">Das war alles.</p>
|
||||||
|
)}
|
||||||
|
<p className="text-[11px] text-slate-600 tabular-nums">
|
||||||
|
{eintraege.length} {eintraege.length === 1 ? "Vorgang" : "Vorgänge"} geladen
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -0,0 +1,108 @@
|
|||||||
|
import { CalendarRange, CheckSquare, FolderGit2, NotebookPen, Plus } from "lucide-react";
|
||||||
|
import { ActivityDayGroups } from "@/components/ActivityRow";
|
||||||
|
import type { Kennzahlen } from "@/lib/review";
|
||||||
|
import type { ReviewDay } from "@/types";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Der Wochenrueckblick.
|
||||||
|
*
|
||||||
|
* Bewusst eine Server-Komponente: es gibt nichts zu bedienen, alles steht
|
||||||
|
* beim Rendern fest. Das spart dem Browser den ganzen Verlauf als JavaScript.
|
||||||
|
*
|
||||||
|
* Die Tage kommen fertig gebuendelt und in der richtigen Reihenfolge – die
|
||||||
|
* Zuordnung "welcher Kalendertag" haengt an der Zeitzone und gehoert
|
||||||
|
* deshalb auf den Server.
|
||||||
|
*/
|
||||||
|
|
||||||
|
export function WeeklyReview({
|
||||||
|
tage,
|
||||||
|
kennzahlen,
|
||||||
|
istDieseWoche,
|
||||||
|
}: {
|
||||||
|
tage: ReviewDay[];
|
||||||
|
/** Fertig gezaehlt vom Server – die Anzeige rechnet nichts nach. */
|
||||||
|
kennzahlen: Kennzahlen;
|
||||||
|
istDieseWoche: boolean;
|
||||||
|
}) {
|
||||||
|
const { gesamt, projekte, erledigt, neueAufgaben, notizen } = kennzahlen;
|
||||||
|
|
||||||
|
if (gesamt === 0) {
|
||||||
|
return (
|
||||||
|
<div className="glass fade-up flex flex-col items-center justify-center gap-4 px-6 py-16 text-center">
|
||||||
|
<span className="grid h-14 w-14 place-items-center rounded-2xl bg-gradient-to-br from-violet-500/25 to-cyan-400/20 text-violet-200">
|
||||||
|
<CalendarRange className="h-6 w-6" />
|
||||||
|
</span>
|
||||||
|
<div>
|
||||||
|
<h2 className="text-lg font-semibold text-white">
|
||||||
|
{istDieseWoche ? "Diese Woche war es still" : "In dieser Woche ist nichts passiert"}
|
||||||
|
</h2>
|
||||||
|
<p className="mt-1 max-w-md text-sm text-slate-400">
|
||||||
|
{istDieseWoche
|
||||||
|
? "Sobald du Aufgaben abhakst, Notizen schreibst oder etwas gepusht wird, steht es hier."
|
||||||
|
: "Vielleicht lohnt ein Blick auf eine frühere Woche."}
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
return (
|
||||||
|
<div className="space-y-5">
|
||||||
|
{/* Kennzahlen */}
|
||||||
|
<div className="grid grid-cols-2 gap-3 sm:gap-4 lg:grid-cols-4">
|
||||||
|
<Kennzahl label="Vorgänge" wert={gesamt} icon={CalendarRange} color="#a78bfa" />
|
||||||
|
<Kennzahl label="Projekte berührt" wert={projekte} icon={FolderGit2} color="#38bdf8" />
|
||||||
|
<Kennzahl label="Aufgaben erledigt" wert={erledigt} icon={CheckSquare} color="#34d399" />
|
||||||
|
<Kennzahl label="Neue Aufgaben" wert={neueAufgaben} icon={Plus} color="#fbbf24" />
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{notizen > 0 ? (
|
||||||
|
<p className="flex items-center gap-1.5 px-1 text-xs text-slate-500">
|
||||||
|
<NotebookPen className="h-3.5 w-3.5" />
|
||||||
|
Dazu {notizen} {notizen === 1 ? "neue Notiz" : "neue Notizen"}.
|
||||||
|
</p>
|
||||||
|
) : null}
|
||||||
|
|
||||||
|
{/* Verlauf nach Tagen – dieselbe Darstellung wie in der Zeitleiste. */}
|
||||||
|
<ActivityDayGroups tage={tage} />
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
function Kennzahl({
|
||||||
|
label,
|
||||||
|
wert,
|
||||||
|
icon: Icon,
|
||||||
|
color,
|
||||||
|
}: {
|
||||||
|
label: string;
|
||||||
|
wert: number;
|
||||||
|
icon: typeof CalendarRange;
|
||||||
|
color: string;
|
||||||
|
}) {
|
||||||
|
return (
|
||||||
|
<div className="glass relative overflow-hidden p-3 sm:p-4">
|
||||||
|
<div
|
||||||
|
className="pointer-events-none absolute -right-6 -top-8 h-24 w-24 rounded-full opacity-25 blur-2xl"
|
||||||
|
style={{ backgroundColor: color }}
|
||||||
|
aria-hidden="true"
|
||||||
|
/>
|
||||||
|
<div className="flex items-start justify-between gap-2">
|
||||||
|
<div className="min-w-0">
|
||||||
|
<p className="text-[11px] font-semibold uppercase leading-tight tracking-[0.14em] text-slate-500">
|
||||||
|
{label}
|
||||||
|
</p>
|
||||||
|
<p className="mt-1.5 text-2xl font-semibold tabular-nums text-white sm:text-3xl">
|
||||||
|
{wert}
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
<span
|
||||||
|
className="grid h-8 w-8 shrink-0 place-items-center rounded-xl sm:h-9 sm:w-9"
|
||||||
|
style={{ backgroundColor: `${color}1f`, color }}
|
||||||
|
>
|
||||||
|
<Icon className="h-4 w-4" />
|
||||||
|
</span>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -0,0 +1,43 @@
|
|||||||
|
import {
|
||||||
|
AlertTriangle,
|
||||||
|
ArrowRight,
|
||||||
|
Download,
|
||||||
|
GitCommitHorizontal,
|
||||||
|
Link2,
|
||||||
|
NotebookPen,
|
||||||
|
Pencil,
|
||||||
|
Plus,
|
||||||
|
Signpost,
|
||||||
|
Sparkles,
|
||||||
|
Trash2,
|
||||||
|
} from "lucide-react";
|
||||||
|
import type { ActivityKind } from "@prisma/client";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Symbol und Farbe je Art eines Verlaufseintrags.
|
||||||
|
*
|
||||||
|
* Ausgelagert, weil sie inzwischen an zwei Stellen gebraucht werden: im
|
||||||
|
* Verlauf eines Projekts und im Wochenrueckblick ueber alle Projekte. Zwei
|
||||||
|
* Kopien liefen beim naechsten neuen Eintragstyp auseinander.
|
||||||
|
*
|
||||||
|
* Bewusst ohne "use client" und ohne Prisma-Zugriff – nur Typen und
|
||||||
|
* Symbole, damit beide Seiten es importieren koennen.
|
||||||
|
*/
|
||||||
|
export const ACTIVITY_KINDS: Record<
|
||||||
|
ActivityKind,
|
||||||
|
{ icon: typeof Sparkles; color: string; label: string }
|
||||||
|
> = {
|
||||||
|
PROJECT_CREATED: { icon: Sparkles, color: "text-violet-300", label: "Projekt angelegt" },
|
||||||
|
PROJECT_UPDATED: { icon: Pencil, color: "text-slate-400", label: "Projekt bearbeitet" },
|
||||||
|
STATUS_CHANGED: { icon: Signpost, color: "text-cyan-300", label: "Status geändert" },
|
||||||
|
NOTE_ADDED: { icon: NotebookPen, color: "text-blue-300", label: "Notiz" },
|
||||||
|
NOTE_UPDATED: { icon: Pencil, color: "text-blue-300", label: "Notiz bearbeitet" },
|
||||||
|
NOTE_DELETED: { icon: Trash2, color: "text-rose-300", label: "Notiz gelöscht" },
|
||||||
|
TASK_ADDED: { icon: Plus, color: "text-emerald-300", label: "Aufgabe angelegt" },
|
||||||
|
TASK_MOVED: { icon: ArrowRight, color: "text-violet-300", label: "Aufgabe verschoben" },
|
||||||
|
TASK_DELETED: { icon: Trash2, color: "text-rose-300", label: "Aufgabe gelöscht" },
|
||||||
|
TASKS_IMPORTED: { icon: Download, color: "text-amber-300", label: "Aufgaben übernommen" },
|
||||||
|
REPO_LINKED: { icon: Link2, color: "text-cyan-300", label: "Repository verknüpft" },
|
||||||
|
REPO_COMMITS: { icon: GitCommitHorizontal, color: "text-emerald-300", label: "Commits" },
|
||||||
|
SYNC_FAILED: { icon: AlertTriangle, color: "text-rose-300", label: "Abgleich fehlgeschlagen" },
|
||||||
|
};
|
||||||
@@ -10,7 +10,9 @@ import {
|
|||||||
Loader2,
|
Loader2,
|
||||||
Plus,
|
Plus,
|
||||||
Repeat,
|
Repeat,
|
||||||
|
Search,
|
||||||
Square,
|
Square,
|
||||||
|
X,
|
||||||
} from "lucide-react";
|
} from "lucide-react";
|
||||||
import type { TaskStatus } from "@prisma/client";
|
import type { TaskStatus } from "@prisma/client";
|
||||||
import { QuickCapture } from "@/components/QuickCapture";
|
import { QuickCapture } from "@/components/QuickCapture";
|
||||||
@@ -42,6 +44,7 @@ export function TaskOverview({ tasks }: { tasks: TaskOverviewItem[] }) {
|
|||||||
const router = useRouter();
|
const router = useRouter();
|
||||||
const [statusFilter, setStatusFilter] = useState<TaskStatus[]>([]);
|
const [statusFilter, setStatusFilter] = useState<TaskStatus[]>([]);
|
||||||
const [projektFilter, setProjektFilter] = useState("");
|
const [projektFilter, setProjektFilter] = useState("");
|
||||||
|
const [suche, setSuche] = useState("");
|
||||||
const [zeigeErledigte, setZeigeErledigte] = useState(false);
|
const [zeigeErledigte, setZeigeErledigte] = useState(false);
|
||||||
const [erfassenOffen, setErfassenOffen] = useState(false);
|
const [erfassenOffen, setErfassenOffen] = useState(false);
|
||||||
const [laufend, setLaufend] = useState<Set<string>>(new Set());
|
const [laufend, setLaufend] = useState<Set<string>>(new Set());
|
||||||
@@ -63,16 +66,34 @@ export function TaskOverview({ tasks }: { tasks: TaskOverviewItem[] }) {
|
|||||||
}, [tasks]);
|
}, [tasks]);
|
||||||
|
|
||||||
const sichtbar = useMemo(() => {
|
const sichtbar = useMemo(() => {
|
||||||
|
const nadel = suche.toLowerCase().trim();
|
||||||
|
|
||||||
return tasks.filter((task) => {
|
return tasks.filter((task) => {
|
||||||
// Erledigtes fuellt die Liste sonst mit dem, was gerade nicht ansteht.
|
// Erledigtes fuellt die Liste sonst mit dem, was gerade nicht ansteht.
|
||||||
if (!zeigeErledigte && task.status === "DONE" && !statusFilter.includes("DONE")) {
|
// Wer allerdings sucht, sucht auch in Erledigtem – sonst findet man
|
||||||
|
// die Aufgabe von letzter Woche nicht wieder.
|
||||||
|
if (
|
||||||
|
!zeigeErledigte &&
|
||||||
|
!nadel &&
|
||||||
|
task.status === "DONE" &&
|
||||||
|
!statusFilter.includes("DONE")
|
||||||
|
) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
if (statusFilter.length > 0 && !statusFilter.includes(task.status)) return false;
|
if (statusFilter.length > 0 && !statusFilter.includes(task.status)) return false;
|
||||||
if (projektFilter && task.project.id !== projektFilter) return false;
|
if (projektFilter && task.project.id !== projektFilter) return false;
|
||||||
return true;
|
if (!nadel) return true;
|
||||||
|
|
||||||
|
// Die Aufgaben liegen hier ohnehin alle vor – das filtert ohne
|
||||||
|
// zusaetzliche Abfrage und ohne Verzoegerung beim Tippen.
|
||||||
|
return (
|
||||||
|
task.title.toLowerCase().includes(nadel) ||
|
||||||
|
(task.description ?? "").toLowerCase().includes(nadel) ||
|
||||||
|
task.labels.some((label) => label.toLowerCase().includes(nadel)) ||
|
||||||
|
task.project.name.toLowerCase().includes(nadel)
|
||||||
|
);
|
||||||
});
|
});
|
||||||
}, [tasks, statusFilter, projektFilter, zeigeErledigte]);
|
}, [tasks, statusFilter, projektFilter, zeigeErledigte, suche]);
|
||||||
|
|
||||||
const faecher = useMemo(() => {
|
const faecher = useMemo(() => {
|
||||||
return DUE_BUCKET_ORDER.map((bucket) => ({
|
return DUE_BUCKET_ORDER.map((bucket) => ({
|
||||||
@@ -155,6 +176,27 @@ export function TaskOverview({ tasks }: { tasks: TaskOverviewItem[] }) {
|
|||||||
|
|
||||||
{/* Steuerleiste */}
|
{/* Steuerleiste */}
|
||||||
<div className="glass flex flex-col gap-3 p-3 sm:p-4">
|
<div className="glass flex flex-col gap-3 p-3 sm:p-4">
|
||||||
|
<div className="relative">
|
||||||
|
<Search className="pointer-events-none absolute left-3 top-1/2 h-4 w-4 -translate-y-1/2 text-slate-500" />
|
||||||
|
<input
|
||||||
|
className="field pl-9"
|
||||||
|
placeholder="Aufgaben durchsuchen – Titel, Beschreibung, Labels…"
|
||||||
|
value={suche}
|
||||||
|
onChange={(event) => setSuche(event.target.value)}
|
||||||
|
aria-label="Aufgaben durchsuchen"
|
||||||
|
/>
|
||||||
|
{suche ? (
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
onClick={() => setSuche("")}
|
||||||
|
className="absolute right-2 top-1/2 -translate-y-1/2 rounded p-1 text-slate-500 hover:text-white"
|
||||||
|
aria-label="Suche zuruecksetzen"
|
||||||
|
>
|
||||||
|
<X className="h-3.5 w-3.5" />
|
||||||
|
</button>
|
||||||
|
) : null}
|
||||||
|
</div>
|
||||||
|
|
||||||
<div className="flex flex-wrap items-center gap-2">
|
<div className="flex flex-wrap items-center gap-2">
|
||||||
{TASK_STATUS_ORDER.map((status) => {
|
{TASK_STATUS_ORDER.map((status) => {
|
||||||
const meta = TASK_STATUS_META[status];
|
const meta = TASK_STATUS_META[status];
|
||||||
@@ -220,12 +262,18 @@ export function TaskOverview({ tasks }: { tasks: TaskOverviewItem[] }) {
|
|||||||
</span>
|
</span>
|
||||||
<div>
|
<div>
|
||||||
<h2 className="text-lg font-semibold text-white">
|
<h2 className="text-lg font-semibold text-white">
|
||||||
{tasks.length === 0 ? "Noch keine Aufgaben" : "Nichts offen"}
|
{tasks.length === 0
|
||||||
|
? "Noch keine Aufgaben"
|
||||||
|
: suche.trim()
|
||||||
|
? "Keine Treffer"
|
||||||
|
: "Nichts offen"}
|
||||||
</h2>
|
</h2>
|
||||||
<p className="mt-1 max-w-md text-sm text-slate-400">
|
<p className="mt-1 max-w-md text-sm text-slate-400">
|
||||||
{tasks.length === 0
|
{tasks.length === 0
|
||||||
? "Aufgaben entstehen im Projekt oder hier über „Neue Aufgabe“."
|
? "Aufgaben entstehen im Projekt oder hier über „Neue Aufgabe“."
|
||||||
: "Alles abgehakt, was zu den Filtern passt."}
|
: suche.trim()
|
||||||
|
? `Keine Aufgabe passt zu „${suche.trim()}“ – gesucht wird in Titel, Beschreibung, Labels und Projektname.`
|
||||||
|
: "Alles abgehakt, was zu den Filtern passt."}
|
||||||
</p>
|
</p>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
+2
-2
@@ -46,7 +46,7 @@ export function setMfaCookie(response: NextResponse, token: string) {
|
|||||||
name: MFA_COOKIE,
|
name: MFA_COOKIE,
|
||||||
value: token,
|
value: token,
|
||||||
httpOnly: true,
|
httpOnly: true,
|
||||||
sameSite: "lax",
|
sameSite: "strict",
|
||||||
secure: COOKIES_SECURE,
|
secure: COOKIES_SECURE,
|
||||||
path: "/",
|
path: "/",
|
||||||
maxAge: Math.floor(MFA_TTL_MS / 1000),
|
maxAge: Math.floor(MFA_TTL_MS / 1000),
|
||||||
@@ -58,7 +58,7 @@ export function clearMfaCookie(response: NextResponse) {
|
|||||||
name: MFA_COOKIE,
|
name: MFA_COOKIE,
|
||||||
value: "",
|
value: "",
|
||||||
httpOnly: true,
|
httpOnly: true,
|
||||||
sameSite: "lax",
|
sameSite: "strict",
|
||||||
secure: COOKIES_SECURE,
|
secure: COOKIES_SECURE,
|
||||||
path: "/",
|
path: "/",
|
||||||
maxAge: 0,
|
maxAge: 0,
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
import { describe, expect, it } from "vitest";
|
import { describe, expect, it } from "vitest";
|
||||||
import { PASSWORD_MIN_LENGTH } from "./constants";
|
import { PASSWORD_MIN_LENGTH } from "./constants";
|
||||||
import { hashPassword, validatePasswordStrength, verifyPassword } from "./password";
|
import { hashPassword, needsRehash, validatePasswordStrength, verifyPassword } from "./password";
|
||||||
|
|
||||||
describe("hashPassword und verifyPassword", () => {
|
describe("hashPassword und verifyPassword", () => {
|
||||||
it("erkennt das richtige Passwort wieder", async () => {
|
it("erkennt das richtige Passwort wieder", async () => {
|
||||||
@@ -64,3 +64,82 @@ describe("validatePasswordStrength", () => {
|
|||||||
expect(validatePasswordStrength("a".repeat(300))).toMatch(/hoechstens/);
|
expect(validatePasswordStrength("a".repeat(300))).toMatch(/hoechstens/);
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe("Aufwand nach OWASP", () => {
|
||||||
|
it("hasht mit den heute vorgesehenen Parametern", async () => {
|
||||||
|
// OWASP Password Storage Cheat Sheet: N = 2^17, r = 8, p = 1.
|
||||||
|
const hash = await hashPassword("ein-langes-testpasswort");
|
||||||
|
const [, N, r, p] = hash.split("$");
|
||||||
|
expect(Number(N)).toBe(131072);
|
||||||
|
expect(Number(r)).toBe(8);
|
||||||
|
expect(Number(p)).toBe(1);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("prueft alte Hashes weiterhin, weil die Parameter im Hash stehen", async () => {
|
||||||
|
// Von Hand mit den frueheren Werten erzeugt – muss gueltig bleiben,
|
||||||
|
// sonst waere ein Anheben des Aufwands eine Aussperrung.
|
||||||
|
const alt =
|
||||||
|
"scrypt$32768$8$1$" +
|
||||||
|
Buffer.from("salzsalzsalzsalz").toString("base64url") +
|
||||||
|
"$";
|
||||||
|
// Echten Hash mit alten Parametern bauen:
|
||||||
|
const { scrypt } = await import("node:crypto");
|
||||||
|
const { promisify } = await import("node:util");
|
||||||
|
const s = promisify(scrypt) as (a: string, b: Buffer, c: number, d: object) => Promise<Buffer>;
|
||||||
|
const salt = Buffer.from("salzsalzsalzsalz");
|
||||||
|
const derived = await s("ein-langes-testpasswort", salt, 64, {
|
||||||
|
N: 32768,
|
||||||
|
r: 8,
|
||||||
|
p: 1,
|
||||||
|
maxmem: 96 * 1024 * 1024,
|
||||||
|
});
|
||||||
|
const alterHash = `scrypt$32768$8$1$${salt.toString("base64url")}$${derived.toString("base64url")}`;
|
||||||
|
expect(alt.length).toBeGreaterThan(0);
|
||||||
|
|
||||||
|
await expect(verifyPassword("ein-langes-testpasswort", alterHash)).resolves.toBe(true);
|
||||||
|
// ... und wird als erneuerungsbeduerftig erkannt.
|
||||||
|
expect(needsRehash(alterHash)).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("laesst frische Hashes in Ruhe", async () => {
|
||||||
|
expect(needsRehash(await hashPassword("ein-langes-testpasswort"))).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("erkennt gleichwertige Kombinationen als ausreichend", () => {
|
||||||
|
// N=2^16 mit p=2 ergibt denselben Gesamtaufwand wie N=2^17 mit p=1 und
|
||||||
|
// ist nach OWASP ebenso zulaessig.
|
||||||
|
expect(needsRehash("scrypt$65536$8$2$c2FsdA$aGFzaA")).toBe(false);
|
||||||
|
// N=2^15 mit p=1 dagegen nicht – das war der alte Zustand.
|
||||||
|
expect(needsRehash("scrypt$32768$8$1$c2FsdA$aGFzaA")).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("haelt Unsinn fuer erneuerungsbeduerftig", () => {
|
||||||
|
expect(needsRehash("voelliger-unsinn")).toBe(true);
|
||||||
|
expect(needsRehash(null)).toBe(false);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe("Schwache Muster", () => {
|
||||||
|
it("weist Wiederholungen ab", () => {
|
||||||
|
expect(validatePasswordStrength("aaaaaaaaaaaa")).toBeTruthy();
|
||||||
|
expect(validatePasswordStrength("abababababab")).toBeTruthy();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("weist Zeichenfolgen ab", () => {
|
||||||
|
expect(validatePasswordStrength("abcdefghijkl")).toBeTruthy();
|
||||||
|
expect(validatePasswordStrength("87654321abcd")).toBeTruthy();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("weist Passwoerter mit dem Benutzernamen ab", () => {
|
||||||
|
expect(validatePasswordStrength("chefin-geheimnis", "chefin")).toMatch(/Benutzernamen/);
|
||||||
|
// Ohne Benutzernamen ist dasselbe Passwort in Ordnung.
|
||||||
|
expect(validatePasswordStrength("chefin-geheimnis")).toBeNull();
|
||||||
|
// Sehr kurze Namen zaehlen nicht – sonst scheitert fast alles.
|
||||||
|
expect(validatePasswordStrength("ab-langes-passwort", "ab")).toBeNull();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("laesst ein gutes Passwort mit Wiederholung durch", () => {
|
||||||
|
// Ein "ll" ist kein Grund zur Ablehnung, solange genug Vielfalt da ist.
|
||||||
|
expect(validatePasswordStrength("Rollladen-Fenster-42")).toBeNull();
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|||||||
+122
-10
@@ -17,11 +17,24 @@ const scrypt = promisify(scryptCallback) as (
|
|||||||
options: { N: number; r: number; p: number; maxmem: number },
|
options: { N: number; r: number; p: number; maxmem: number },
|
||||||
) => Promise<Buffer>;
|
) => Promise<Buffer>;
|
||||||
|
|
||||||
const PARAMS = { N: 32768, r: 8, p: 1 };
|
/**
|
||||||
|
* Arbeitsaufwand nach der Empfehlung des OWASP Password Storage Cheat Sheet:
|
||||||
|
* N = 2^17, r = 8, p = 1. Frueher stand hier N = 2^15 mit p = 1 – das liegt
|
||||||
|
* unter *jeder* der dort genannten zulaessigen Kombinationen (zu 2^15
|
||||||
|
* gehoerte p = 3) und war damit rund dreimal billiger zu knacken als
|
||||||
|
* vorgesehen.
|
||||||
|
*
|
||||||
|
* Gemessen kostet das rund 0,5 s je Pruefung und 128 MiB. Der Speicher
|
||||||
|
* klingt nach viel, ist aber gedeckelt: Node rechnet scrypt im
|
||||||
|
* libuv-Threadpool, standardmaessig vier Faeden parallel. Mehr als rund
|
||||||
|
* 512 MiB koennen also nicht gleichzeitig belegt sein, und davor liegen
|
||||||
|
* noch die Ratenbegrenzungen je IP und je Benutzername.
|
||||||
|
*/
|
||||||
|
const PARAMS = { N: 131072, r: 8, p: 1 };
|
||||||
const KEY_LENGTH = 64;
|
const KEY_LENGTH = 64;
|
||||||
// scrypt braucht rund 128 * N * r Bytes; der Standardwert von 32 MB reicht
|
// scrypt braucht rund 128 * N * r Bytes – bei diesen Werten 128 MiB.
|
||||||
// dafuer nicht ganz.
|
// Etwas Luft nach oben, damit die Pruefung nicht am Deckel scheitert.
|
||||||
const MAX_MEM = 96 * 1024 * 1024;
|
const MAX_MEM = 192 * 1024 * 1024;
|
||||||
|
|
||||||
export async function hashPassword(password: string): Promise<string> {
|
export async function hashPassword(password: string): Promise<string> {
|
||||||
const salt = randomBytes(16);
|
const salt = randomBytes(16);
|
||||||
@@ -72,6 +85,30 @@ export async function verifyPassword(
|
|||||||
return timingSafeEqual(derived, expected);
|
return timingSafeEqual(derived, expected);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Ist der gespeicherte Hash mit schwaecheren Parametern erzeugt worden als
|
||||||
|
* heute ueblich?
|
||||||
|
*
|
||||||
|
* Der Aufwand laesst sich nur fuer neue Passwoerter anheben – bestehende
|
||||||
|
* Hashes bleiben sonst fuer immer auf dem alten Stand. OWASP empfiehlt
|
||||||
|
* deshalb, bei der naechsten erfolgreichen Anmeldung still neu zu hashen.
|
||||||
|
* Das geht nur dort, weil das Klartextpasswort ausschliesslich in diesem
|
||||||
|
* Moment vorliegt.
|
||||||
|
*/
|
||||||
|
export function needsRehash(stored: string | null | undefined): boolean {
|
||||||
|
if (!stored) return false;
|
||||||
|
|
||||||
|
const parts = stored.split("$");
|
||||||
|
if (parts.length !== 6 || parts[0] !== "scrypt") return true;
|
||||||
|
|
||||||
|
const [N, r, p] = [Number(parts[1]), Number(parts[2]), Number(parts[3])];
|
||||||
|
if (!Number.isFinite(N) || !Number.isFinite(r) || !Number.isFinite(p)) return true;
|
||||||
|
|
||||||
|
// Verglichen wird der Gesamtaufwand, nicht Feld fuer Feld: N=2^16 mit p=2
|
||||||
|
// ist nach OWASP genauso zulaessig wie N=2^17 mit p=1.
|
||||||
|
return N * r * p < PARAMS.N * PARAMS.r * PARAMS.p;
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Verbrennt vergleichbar viel Zeit wie eine echte Pruefung. Wird aufgerufen,
|
* Verbrennt vergleichbar viel Zeit wie eine echte Pruefung. Wird aufgerufen,
|
||||||
* wenn es den Benutzernamen gar nicht gibt – sonst liesse sich an der
|
* wenn es den Benutzernamen gar nicht gibt – sonst liesse sich an der
|
||||||
@@ -84,27 +121,102 @@ export async function fakeVerifyDelay(): Promise<void> {
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Naheliegende Passwoerter.
|
||||||
|
*
|
||||||
|
* ASVS 2.1.7 verlangt einen Abgleich gegen bekannte geleakte Passwoerter.
|
||||||
|
* Ein Dienst wie Have-I-Been-Pwned schiede dafuer aus – Nebula ruft
|
||||||
|
* grundsaetzlich nichts von aussen, und ein Passwort-Praefix nach draussen
|
||||||
|
* zu geben widerspraeche dem erst recht. Diese Liste ist der Kompromiss:
|
||||||
|
* die haeufigsten deutschen und englischen Kandidaten, dazu weiter unten
|
||||||
|
* Muster statt Einzelwoerter, was mehr abdeckt als jede Liste dieser Groesse.
|
||||||
|
*/
|
||||||
const OBVIOUS = new Set([
|
const OBVIOUS = new Set([
|
||||||
"password", "passwort", "12345678", "123456789", "1234567890",
|
// Klassiker
|
||||||
"qwertzuiop", "qwertyuiop", "administrator", "changeme", "letmein",
|
"password", "password1", "password123", "passwort", "passwort1",
|
||||||
"willkommen", "nebula123", "geheim123",
|
"passwort123", "kennwort", "geheim", "geheim123", "changeme", "letmein",
|
||||||
|
"welcome", "willkommen", "willkommen1", "administrator", "admin1234",
|
||||||
|
"iloveyou", "sunshine", "princess", "dragon", "monkey", "football",
|
||||||
|
"baseball", "superman", "trustno1", "starwars", "whatever", "freedom",
|
||||||
|
// Zahlen und Tastaturlaeufe
|
||||||
|
"12345678", "123456789", "1234567890", "87654321", "11111111",
|
||||||
|
"qwertzuiop", "qwertyuiop", "asdfghjkl", "qwertz123", "qwerty123",
|
||||||
|
"1qay2wsx", "1q2w3e4r", "1q2w3e4r5t", "asdf1234", "qwer1234",
|
||||||
|
// Deutsch
|
||||||
|
"sommer2024", "sommer2025", "sommer2026", "winter2024", "winter2025",
|
||||||
|
"winter2026", "fussball", "deutschland", "hallowelt", "sonnenschein",
|
||||||
|
"schatz123", "liebling", "blumen123", "internet", "computer",
|
||||||
|
// Projektnah
|
||||||
|
"nebula", "nebula123", "nebula2026", "dashboard", "vibecoding",
|
||||||
]);
|
]);
|
||||||
|
|
||||||
export { PASSWORD_MIN_LENGTH };
|
export { PASSWORD_MIN_LENGTH };
|
||||||
|
|
||||||
/** Gibt eine Fehlermeldung zurueck oder null, wenn das Passwort taugt. */
|
/**
|
||||||
export function validatePasswordStrength(password: string): string | null {
|
* Laenge der laengsten auf- oder absteigenden Zeichenfolge ("abcd", "4321").
|
||||||
|
*
|
||||||
|
* Zurueckgegeben wird die Laenge und nicht nur ein Ja/Nein: entscheidend ist,
|
||||||
|
* *wie viel* des Passworts aus der Folge besteht. Ein "de" mitten in einem
|
||||||
|
* Wort ist harmlos, "abcdefghijkl" ist das ganze Passwort.
|
||||||
|
*/
|
||||||
|
function laengsteFolge(wert: string): number {
|
||||||
|
if (wert.length < 2) return wert.length;
|
||||||
|
let beste = 1;
|
||||||
|
let auf = 1;
|
||||||
|
let ab = 1;
|
||||||
|
for (let i = 1; i < wert.length; i += 1) {
|
||||||
|
const schritt = wert.charCodeAt(i) - wert.charCodeAt(i - 1);
|
||||||
|
auf = schritt === 1 ? auf + 1 : 1;
|
||||||
|
ab = schritt === -1 ? ab + 1 : 1;
|
||||||
|
beste = Math.max(beste, auf, ab);
|
||||||
|
}
|
||||||
|
return beste;
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Gibt eine Fehlermeldung zurueck oder null, wenn das Passwort taugt.
|
||||||
|
*
|
||||||
|
* Bewusst **keine** Zusammensetzungsregeln ("mindestens ein Sonderzeichen").
|
||||||
|
* ASVS 2.1.9 raet ausdruecklich davon ab: sie treiben Leute zu "Passwort1!"
|
||||||
|
* und senken die Entropie, statt sie zu heben. Geprueft wird stattdessen
|
||||||
|
* auf Laenge und auf das, was nachweislich schwach ist.
|
||||||
|
*/
|
||||||
|
export function validatePasswordStrength(
|
||||||
|
password: string,
|
||||||
|
/** Benutzername des Kontos, falls bekannt – er darf nicht im Passwort stehen. */
|
||||||
|
username?: string | null,
|
||||||
|
): string | null {
|
||||||
if (password.length < PASSWORD_MIN_LENGTH) {
|
if (password.length < PASSWORD_MIN_LENGTH) {
|
||||||
return `Das Passwort muss mindestens ${PASSWORD_MIN_LENGTH} Zeichen haben.`;
|
return `Das Passwort muss mindestens ${PASSWORD_MIN_LENGTH} Zeichen haben.`;
|
||||||
}
|
}
|
||||||
if (password.length > 200) {
|
if (password.length > 200) {
|
||||||
return "Das Passwort darf hoechstens 200 Zeichen haben.";
|
return "Das Passwort darf hoechstens 200 Zeichen haben.";
|
||||||
}
|
}
|
||||||
if (OBVIOUS.has(password.toLowerCase())) {
|
|
||||||
|
const klein = password.toLowerCase();
|
||||||
|
|
||||||
|
if (OBVIOUS.has(klein)) {
|
||||||
return "Dieses Passwort ist zu naheliegend.";
|
return "Dieses Passwort ist zu naheliegend.";
|
||||||
}
|
}
|
||||||
if (/^\d+$/.test(password)) {
|
if (/^\d+$/.test(password)) {
|
||||||
return "Nur Ziffern reichen nicht – bitte Buchstaben oder Zeichen ergaenzen.";
|
return "Nur Ziffern reichen nicht – bitte Buchstaben oder Zeichen ergaenzen.";
|
||||||
}
|
}
|
||||||
|
// Zu wenig verschiedene Zeichen – deckt "aaaaaaaa" ebenso ab wie
|
||||||
|
// "abababab" oder "12121212", ohne dafuer je ein eigenes Muster zu
|
||||||
|
// brauchen.
|
||||||
|
if (new Set(klein).size <= 4) {
|
||||||
|
return "Das Passwort benutzt zu wenige verschiedene Zeichen.";
|
||||||
|
}
|
||||||
|
// Und wenn der groesste Teil des Passworts eine durchlaufende Folge ist.
|
||||||
|
if (laengsteFolge(klein) >= Math.max(6, Math.ceil(password.length * 0.6))) {
|
||||||
|
return "Zeichenfolgen wie „abcd“ oder „4321“ sind zu leicht zu raten.";
|
||||||
|
}
|
||||||
|
// Ein Passwort, das den Benutzernamen enthaelt, ist der erste Versuch
|
||||||
|
// jedes Angreifers.
|
||||||
|
const name = username?.trim().toLowerCase();
|
||||||
|
if (name && name.length >= 3 && klein.includes(name)) {
|
||||||
|
return "Das Passwort darf den Benutzernamen nicht enthalten.";
|
||||||
|
}
|
||||||
|
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
|||||||
+33
-6
@@ -2,7 +2,12 @@ import { createHash, randomBytes } from "node:crypto";
|
|||||||
import { cookies } from "next/headers";
|
import { cookies } from "next/headers";
|
||||||
import type { NextResponse } from "next/server";
|
import type { NextResponse } from "next/server";
|
||||||
import type { User } from "@prisma/client";
|
import type { User } from "@prisma/client";
|
||||||
import { COOKIES_SECURE, SESSION_COOKIE, SESSION_TTL_MS } from "@/lib/config";
|
import {
|
||||||
|
COOKIES_SECURE,
|
||||||
|
SESSION_COOKIE,
|
||||||
|
SESSION_IDLE_MS,
|
||||||
|
SESSION_TTL_MS,
|
||||||
|
} from "@/lib/config";
|
||||||
import { prisma } from "@/lib/db";
|
import { prisma } from "@/lib/db";
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -68,7 +73,10 @@ export function setSessionCookie(response: NextResponse, token: string, expiresA
|
|||||||
name: SESSION_COOKIE,
|
name: SESSION_COOKIE,
|
||||||
value: token,
|
value: token,
|
||||||
httpOnly: true,
|
httpOnly: true,
|
||||||
sameSite: "lax",
|
// Strict statt Lax: Nebula hat keinen Einstieg von aussen, den eine
|
||||||
|
// fremde Seite anstossen duerfte. Der Preis ist, dass ein Klick von
|
||||||
|
// einer anderen Seite auf die Anmeldung fuehrt – ein Neuladen genuegt.
|
||||||
|
sameSite: "strict",
|
||||||
secure: COOKIES_SECURE,
|
secure: COOKIES_SECURE,
|
||||||
path: "/",
|
path: "/",
|
||||||
expires: expiresAt,
|
expires: expiresAt,
|
||||||
@@ -80,7 +88,10 @@ export function clearSessionCookie(response: NextResponse) {
|
|||||||
name: SESSION_COOKIE,
|
name: SESSION_COOKIE,
|
||||||
value: "",
|
value: "",
|
||||||
httpOnly: true,
|
httpOnly: true,
|
||||||
sameSite: "lax",
|
// Strict statt Lax: Nebula hat keinen Einstieg von aussen, den eine
|
||||||
|
// fremde Seite anstossen duerfte. Der Preis ist, dass ein Klick von
|
||||||
|
// einer anderen Seite auf die Anmeldung fuehrt – ein Neuladen genuegt.
|
||||||
|
sameSite: "strict",
|
||||||
secure: COOKIES_SECURE,
|
secure: COOKIES_SECURE,
|
||||||
path: "/",
|
path: "/",
|
||||||
maxAge: 0,
|
maxAge: 0,
|
||||||
@@ -109,11 +120,27 @@ export async function getSessionUser(): Promise<SessionUser | null> {
|
|||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Leerlaufgrenze. Vorher wurde lastSeenAt zwar gefuehrt, aber nie
|
||||||
|
// ausgewertet – ein abgegriffener Cookie galt damit bis zum absoluten
|
||||||
|
// Ablauf, auch wenn das Konto wochenlang unberuehrt war.
|
||||||
|
const untaetig = Date.now() - session.lastSeenAt.getTime();
|
||||||
|
if (SESSION_IDLE_MS > 0 && untaetig > SESSION_IDLE_MS) {
|
||||||
|
await prisma.session.delete({ where: { id: session.id } }).catch(() => undefined);
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
if (!session.user.isActive) return null;
|
if (!session.user.isActive) return null;
|
||||||
|
|
||||||
// Hoechstens einmal pro Stunde schreiben, damit nicht jede Anfrage die
|
/**
|
||||||
// Datenbank anfasst.
|
* lastSeenAt nachfuehren, aber nicht bei jeder Anfrage schreiben.
|
||||||
if (Date.now() - session.lastSeenAt.getTime() > 60 * 60 * 1000) {
|
*
|
||||||
|
* Das Intervall muss deutlich unter der Leerlaufgrenze liegen, sonst
|
||||||
|
* verkuerzt die Schreibbremse sie: bei stuendlichem Schreiben und einer
|
||||||
|
* Grenze von einer Stunde waere die Sitzung schon nach Sekunden faellig.
|
||||||
|
* Ein Zwanzigstel der Frist, hoechstens fuenf Minuten.
|
||||||
|
*/
|
||||||
|
const intervall = SESSION_IDLE_MS > 0 ? Math.min(SESSION_IDLE_MS / 20, 5 * 60 * 1000) : 60 * 60 * 1000;
|
||||||
|
if (untaetig > intervall) {
|
||||||
void prisma.session
|
void prisma.session
|
||||||
.update({ where: { id: session.id }, data: { lastSeenAt: new Date() } })
|
.update({ where: { id: session.id }, data: { lastSeenAt: new Date() } })
|
||||||
.catch(() => undefined);
|
.catch(() => undefined);
|
||||||
|
|||||||
@@ -41,6 +41,101 @@ export const KIND_META: Record<ChangeKind, { label: string; className: string }>
|
|||||||
};
|
};
|
||||||
|
|
||||||
export const CHANGELOG: Release[] = [
|
export const CHANGELOG: Release[] = [
|
||||||
|
{
|
||||||
|
version: "1.10.0",
|
||||||
|
date: "2026-08-18",
|
||||||
|
headline: "Rückblick, Zeitleiste und eine Suche, die auch Aufgaben findet.",
|
||||||
|
changes: [
|
||||||
|
{
|
||||||
|
kind: "neu",
|
||||||
|
text:
|
||||||
|
"Wochenrückblick über alle Projekte: Kennzahlen der Woche und darunter die " +
|
||||||
|
"Vorgänge nach Tagen gebündelt. Blättern in frühere Wochen inbegriffen. " +
|
||||||
|
"Erreichbar über die Schnellsuche und einen Verweis vom Dashboard.",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
kind: "neu",
|
||||||
|
text:
|
||||||
|
"Globale Zeitleiste: derselbe Verlauf, aber fortlaufend und ohne Wochengrenze, " +
|
||||||
|
"mit Nachladen älterer Einträge. Geblättert wird per Cursor — bei mehreren " +
|
||||||
|
"Einträgen im selben Augenblick verliert oder verdoppelt das keine Zeilen.",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
kind: "neu",
|
||||||
|
text:
|
||||||
|
"Das Aufgabendashboard hat ein Suchfeld über Titel, Beschreibung, Labels und " +
|
||||||
|
"Projektname. Wer sucht, sucht dabei auch in Erledigtem — sonst findet man die " +
|
||||||
|
"Aufgabe von letzter Woche nicht wieder.",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
kind: "besser",
|
||||||
|
text:
|
||||||
|
"Die Volltextsuche kannte bisher nur Notizen und findet jetzt auch Aufgaben — " +
|
||||||
|
"mit denselben deutschen Stammformen: „Rechnung“ findet „Rechnungen“. Treffer " +
|
||||||
|
"erscheinen im Projektbrett und in der Schnellsuche.",
|
||||||
|
},
|
||||||
|
],
|
||||||
|
},
|
||||||
|
{
|
||||||
|
version: "1.9.0",
|
||||||
|
date: "2026-08-17",
|
||||||
|
headline: "Geschlossene Issues haken ihre Aufgabe auch ohne Webhook ab.",
|
||||||
|
changes: [
|
||||||
|
{
|
||||||
|
kind: "neu",
|
||||||
|
text:
|
||||||
|
"Der Abgleich prüft jetzt den Zustand verknüpfter Issues: was in Git geschlossen " +
|
||||||
|
"wurde, wird in Nebula abgehakt. Bisher konnte das nur der Webhook — war keiner " +
|
||||||
|
"eingerichtet, blieb der Rückweg ganz aus.",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
kind: "besser",
|
||||||
|
text:
|
||||||
|
"Ein über Git geschlossenes Issue bewirkt jetzt dasselbe wie ein Haken in der " +
|
||||||
|
"Oberfläche: bei wiederkehrenden Aufgaben entsteht die nächste Fassung, und der " +
|
||||||
|
"Vorgang steht im Projektverlauf.",
|
||||||
|
},
|
||||||
|
],
|
||||||
|
},
|
||||||
|
{
|
||||||
|
version: "1.8.0",
|
||||||
|
date: "2026-08-15",
|
||||||
|
headline: "Durchgang durch Anmeldung und Sitzungen nach OWASP.",
|
||||||
|
changes: [
|
||||||
|
{
|
||||||
|
kind: "besser",
|
||||||
|
text:
|
||||||
|
"Passwörter werden deutlich aufwendiger gehasht (scrypt mit N=2^17 statt 2^15) — " +
|
||||||
|
"das entspricht der Empfehlung des OWASP Password Storage Cheat Sheet. Bestehende " +
|
||||||
|
"Passwörter werden bei der nächsten Anmeldung still auf den neuen Stand gebracht.",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
kind: "behoben",
|
||||||
|
text:
|
||||||
|
"Die Anmeldung verriet, welche Benutzernamen existieren: gesperrte, deaktivierte " +
|
||||||
|
"und reine Passkey-Konten antworteten anders als unbekannte — und das schon vor " +
|
||||||
|
"der Passwortprüfung. Jetzt gibt es die Auskunft erst, wenn das Passwort stimmt.",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
kind: "neu",
|
||||||
|
text:
|
||||||
|
"Sitzungen laufen nach acht Stunden ohne Nutzung ab, zusätzlich zur bestehenden " +
|
||||||
|
"Frist von 30 Tagen. Einstellbar über SESSION_IDLE_HOURS; 0 schaltet es ab.",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
kind: "besser",
|
||||||
|
text:
|
||||||
|
"Das Sitzungscookie trägt bei HTTPS den Präfix „__Host-“ und gilt nur noch " +
|
||||||
|
"SameSite=Strict. Beides muss einmalig neu angemeldet werden.",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
kind: "besser",
|
||||||
|
text:
|
||||||
|
"Die Passwortprüfung erkennt jetzt Wiederholungen, Zeichenfolgen und Passwörter, " +
|
||||||
|
"die den eigenen Benutzernamen enthalten.",
|
||||||
|
},
|
||||||
|
],
|
||||||
|
},
|
||||||
{
|
{
|
||||||
version: "1.7.0",
|
version: "1.7.0",
|
||||||
date: "2026-08-15",
|
date: "2026-08-15",
|
||||||
|
|||||||
+32
-3
@@ -73,18 +73,47 @@ export const WEBAUTHN_AVAILABLE =
|
|||||||
/** Cookies nur ueber TLS ausliefern, wenn die App per https laeuft. */
|
/** Cookies nur ueber TLS ausliefern, wenn die App per https laeuft. */
|
||||||
export const COOKIES_SECURE = APP_URL_PARSED.protocol === "https:";
|
export const COOKIES_SECURE = APP_URL_PARSED.protocol === "https:";
|
||||||
|
|
||||||
/** Lebensdauer einer Sitzung. */
|
/**
|
||||||
|
* Absolute Lebensdauer einer Sitzung – ab Anmeldung, nicht verlaengerbar.
|
||||||
|
* Nach dieser Frist ist Schluss, egal wie aktiv jemand war.
|
||||||
|
*/
|
||||||
export const SESSION_TTL_DAYS = Number(process.env.SESSION_TTL_DAYS ?? 30);
|
export const SESSION_TTL_DAYS = Number(process.env.SESSION_TTL_DAYS ?? 30);
|
||||||
export const SESSION_TTL_MS = SESSION_TTL_DAYS * 24 * 60 * 60 * 1000;
|
export const SESSION_TTL_MS = SESSION_TTL_DAYS * 24 * 60 * 60 * 1000;
|
||||||
|
|
||||||
export const SESSION_COOKIE = "nebula_session";
|
/**
|
||||||
|
* Leerlauffrist: so lange darf eine Sitzung unbenutzt bleiben.
|
||||||
|
*
|
||||||
|
* ASVS 3.3.2 verlangt beides – eine absolute *und* eine Leerlaufgrenze.
|
||||||
|
* Vorher gab es nur die absolute, ein abgegriffener Cookie galt also bis zu
|
||||||
|
* 30 Tage, unabhaengig davon, ob das Konto benutzt wurde.
|
||||||
|
*
|
||||||
|
* Acht Stunden sind der Kompromiss: ein Arbeitstag am Stueck bleibt
|
||||||
|
* angemeldet, ueber Nacht laeuft die Sitzung ab. Wem das zu streng ist,
|
||||||
|
* setzt SESSION_IDLE_HOURS hoch; 0 schaltet die Leerlaufgrenze ab.
|
||||||
|
*/
|
||||||
|
export const SESSION_IDLE_HOURS = Number(process.env.SESSION_IDLE_HOURS ?? 8);
|
||||||
|
export const SESSION_IDLE_MS =
|
||||||
|
SESSION_IDLE_HOURS > 0 ? SESSION_IDLE_HOURS * 60 * 60 * 1000 : 0;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Name des Sitzungscookies.
|
||||||
|
*
|
||||||
|
* Mit dem Praefix "__Host-" erzwingt der Browser selbst, was wir ohnehin
|
||||||
|
* setzen: Secure, Pfad "/" und kein Domain-Attribut. Letzteres ist der
|
||||||
|
* eigentliche Gewinn – eine andere Subdomain kann den Cookie dann nicht
|
||||||
|
* ueberschreiben und damit keine Sitzung unterschieben.
|
||||||
|
*
|
||||||
|
* Nur bei https, denn der Praefix verlangt zwingend Secure; ohne TLS wuerde
|
||||||
|
* der Browser den Cookie schlicht verwerfen.
|
||||||
|
*/
|
||||||
|
export const SESSION_COOKIE = COOKIES_SECURE ? "__Host-nebula_session" : "nebula_session";
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Cookie fuer die halbfertige Anmeldung zwischen Passwort und zweitem Faktor.
|
* Cookie fuer die halbfertige Anmeldung zwischen Passwort und zweitem Faktor.
|
||||||
* Kurzlebig – wer den Code nicht binnen fuenf Minuten eingibt, faengt von
|
* Kurzlebig – wer den Code nicht binnen fuenf Minuten eingibt, faengt von
|
||||||
* vorne an.
|
* vorne an.
|
||||||
*/
|
*/
|
||||||
export const MFA_COOKIE = "nebula_mfa";
|
export const MFA_COOKIE = COOKIES_SECURE ? "__Host-nebula_mfa" : "nebula_mfa";
|
||||||
export const MFA_TTL_MS = 5 * 60 * 1000;
|
export const MFA_TTL_MS = 5 * 60 * 1000;
|
||||||
|
|
||||||
/** Wird beim Start einmal geloggt, damit Fehlkonfiguration frueh auffaellt. */
|
/** Wird beim Start einmal geloggt, damit Fehlkonfiguration frueh auffaellt. */
|
||||||
|
|||||||
@@ -71,6 +71,15 @@ export class GiteaClient implements GitClient {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
async getIssueState(ref: RepoRef, number: number): Promise<"open" | "closed" | null> {
|
||||||
|
const issue = await getJson<{ state?: string } | null>(
|
||||||
|
`${this.repoBase(ref)}/issues/${number}`,
|
||||||
|
{ headers: this.headers(), allow404: true },
|
||||||
|
);
|
||||||
|
if (!issue?.state) return null;
|
||||||
|
return issue.state === "closed" ? "closed" : "open";
|
||||||
|
}
|
||||||
|
|
||||||
async check(): Promise<ConnectionCheck> {
|
async check(): Promise<ConnectionCheck> {
|
||||||
try {
|
try {
|
||||||
const version = await getJsonSafe<{ version?: string }>(
|
const version = await getJsonSafe<{ version?: string }>(
|
||||||
|
|||||||
@@ -70,6 +70,15 @@ export class GitHubClient implements GitClient {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
async getIssueState(ref: RepoRef, number: number): Promise<"open" | "closed" | null> {
|
||||||
|
const issue = await getJson<{ state?: string } | null>(
|
||||||
|
`${this.repoBase(ref)}/issues/${number}`,
|
||||||
|
{ headers: this.headers(), allow404: true },
|
||||||
|
);
|
||||||
|
if (!issue?.state) return null;
|
||||||
|
return issue.state === "closed" ? "closed" : "open";
|
||||||
|
}
|
||||||
|
|
||||||
async check(): Promise<ConnectionCheck> {
|
async check(): Promise<ConnectionCheck> {
|
||||||
try {
|
try {
|
||||||
if (!this.token) {
|
if (!this.token) {
|
||||||
|
|||||||
@@ -70,6 +70,16 @@ export class GitLabClient implements GitClient {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
async getIssueState(ref: RepoRef, number: number): Promise<"open" | "closed" | null> {
|
||||||
|
const issue = await getJson<{ state?: string } | null>(
|
||||||
|
`${this.apiBase}/projects/${this.projectPath(ref)}/issues/${number}`,
|
||||||
|
{ headers: this.headers(), allow404: true },
|
||||||
|
);
|
||||||
|
if (!issue?.state) return null;
|
||||||
|
// GitLab nennt den offenen Zustand "opened".
|
||||||
|
return issue.state === "closed" ? "closed" : "open";
|
||||||
|
}
|
||||||
|
|
||||||
async check(): Promise<ConnectionCheck> {
|
async check(): Promise<ConnectionCheck> {
|
||||||
try {
|
try {
|
||||||
if (!this.token) {
|
if (!this.token) {
|
||||||
|
|||||||
@@ -190,6 +190,17 @@ export interface GitClient {
|
|||||||
* geoeffnet – daher der Schalter statt zweier Methoden.
|
* geoeffnet – daher der Schalter statt zweier Methoden.
|
||||||
*/
|
*/
|
||||||
setIssueState(ref: RepoRef, number: number, open: boolean): Promise<void>;
|
setIssueState(ref: RepoRef, number: number, open: boolean): Promise<void>;
|
||||||
|
/**
|
||||||
|
* Liest den Zustand eines einzelnen Issues.
|
||||||
|
*
|
||||||
|
* Gebraucht fuer den Abgleich: der Snapshot fuehrt nur die *offenen*
|
||||||
|
* Issues und ist ausserdem gedeckelt – aus dem Fehlen einer Nummer laesst
|
||||||
|
* sich deshalb nicht schliessen, dass sie geschlossen wurde. Sie koennte
|
||||||
|
* ebenso geloescht oder einfach jenseits des Deckels sein.
|
||||||
|
*
|
||||||
|
* null heisst: gibt es nicht (mehr).
|
||||||
|
*/
|
||||||
|
getIssueState(ref: RepoRef, number: number): Promise<"open" | "closed" | null>;
|
||||||
/** Repos, auf die der Token Zugriff hat (fuer die Auswahl beim Verknuepfen). */
|
/** Repos, auf die der Token Zugriff hat (fuer die Auswahl beim Verknuepfen). */
|
||||||
listRepos(query?: string): Promise<RepoListItem[]>;
|
listRepos(query?: string): Promise<RepoListItem[]>;
|
||||||
/** Vollstaendiger Stand eines Repositories. */
|
/** Vollstaendiger Stand eines Repositories. */
|
||||||
|
|||||||
@@ -0,0 +1,173 @@
|
|||||||
|
import { describe, expect, it } from "vitest";
|
||||||
|
import {
|
||||||
|
berlinMitternacht,
|
||||||
|
tagesschluessel,
|
||||||
|
wocheUm,
|
||||||
|
wochenName,
|
||||||
|
wochenTitel,
|
||||||
|
zaehleWoche,
|
||||||
|
} from "./review";
|
||||||
|
|
||||||
|
describe("tagesschluessel", () => {
|
||||||
|
it("rechnet in Berliner Kalendertagen, nicht in UTC", () => {
|
||||||
|
// 22:30 UTC ist in Berlin schon der Folgetag (Sommerzeit, UTC+2).
|
||||||
|
expect(tagesschluessel(new Date("2026-08-17T22:30:00Z"))).toBe("2026-08-18");
|
||||||
|
expect(tagesschluessel(new Date("2026-08-17T21:00:00Z"))).toBe("2026-08-17");
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe("berlinMitternacht", () => {
|
||||||
|
it("trifft den Sommerzeit-Beginn des Tages", () => {
|
||||||
|
// Sommerzeit: Berlin = UTC+2, Mitternacht dort ist 22:00 UTC am Vortag.
|
||||||
|
expect(berlinMitternacht("2026-08-17").toISOString()).toBe("2026-08-16T22:00:00.000Z");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("trifft den Winterzeit-Beginn des Tages", () => {
|
||||||
|
// Winterzeit: UTC+1.
|
||||||
|
expect(berlinMitternacht("2026-01-12").toISOString()).toBe("2026-01-11T23:00:00.000Z");
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe("wocheUm", () => {
|
||||||
|
/** Ein Dienstag. */
|
||||||
|
const DIENSTAG = new Date("2026-08-18T09:00:00Z");
|
||||||
|
|
||||||
|
it("beginnt am Montag und endet am folgenden Montag", () => {
|
||||||
|
const woche = wocheUm(DIENSTAG);
|
||||||
|
expect(woche.tage[0]).toBe("2026-08-17");
|
||||||
|
expect(woche.tage[6]).toBe("2026-08-23");
|
||||||
|
expect(woche.tage).toHaveLength(7);
|
||||||
|
expect(woche.von.toISOString()).toBe("2026-08-16T22:00:00.000Z");
|
||||||
|
expect(woche.bis.toISOString()).toBe("2026-08-23T22:00:00.000Z");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("zaehlt den Montag selbst zur laufenden Woche", () => {
|
||||||
|
const montag = new Date("2026-08-17T06:00:00Z");
|
||||||
|
expect(wocheUm(montag).tage[0]).toBe("2026-08-17");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("zaehlt den Sonntagabend noch zur ablaufenden Woche", () => {
|
||||||
|
// 21:00 UTC am Sonntag ist in Berlin 23:00 – noch derselbe Tag.
|
||||||
|
const sonntagAbend = new Date("2026-08-23T21:00:00Z");
|
||||||
|
expect(wocheUm(sonntagAbend).tage[0]).toBe("2026-08-17");
|
||||||
|
// Zwei Stunden spaeter ist Montag und damit die naechste Woche.
|
||||||
|
const montagFrueh = new Date("2026-08-23T22:30:00Z");
|
||||||
|
expect(wocheUm(montagFrueh).tage[0]).toBe("2026-08-24");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("blättert rückwärts", () => {
|
||||||
|
expect(wocheUm(DIENSTAG, -1).tage[0]).toBe("2026-08-10");
|
||||||
|
expect(wocheUm(DIENSTAG, -3).tage[0]).toBe("2026-07-27");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("kommt über den Monats- und Jahreswechsel", () => {
|
||||||
|
const neujahr = new Date("2027-01-01T12:00:00Z"); // ein Freitag
|
||||||
|
const woche = wocheUm(neujahr);
|
||||||
|
expect(woche.tage[0]).toBe("2026-12-28");
|
||||||
|
expect(woche.tage[6]).toBe("2027-01-03");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("bleibt über die Zeitumstellung hinweg sieben Tage lang", () => {
|
||||||
|
// Am 25.10.2026 wird auf Winterzeit zurueckgestellt – die Woche hat
|
||||||
|
// dann 169 Stunden. "von + 7 Tage" waere hier eine Stunde daneben.
|
||||||
|
const inDerWoche = new Date("2026-10-21T12:00:00Z");
|
||||||
|
const woche = wocheUm(inDerWoche);
|
||||||
|
expect(woche.tage[0]).toBe("2026-10-19");
|
||||||
|
expect(woche.tage[6]).toBe("2026-10-25");
|
||||||
|
const stunden = (woche.bis.getTime() - woche.von.getTime()) / 3_600_000;
|
||||||
|
expect(stunden).toBe(169);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("liefert bei der Umstellung im Fruehjahr 167 Stunden", () => {
|
||||||
|
// Vorgestellt wird am letzten Sonntag im Maerz – 2026 ist das der 29.,
|
||||||
|
// er liegt also in der Woche ab Montag, dem 23.
|
||||||
|
const inDerWoche = new Date("2026-03-25T12:00:00Z");
|
||||||
|
const woche = wocheUm(inDerWoche);
|
||||||
|
expect(woche.tage[0]).toBe("2026-03-23");
|
||||||
|
expect(woche.tage[6]).toBe("2026-03-29");
|
||||||
|
const stunden = (woche.bis.getTime() - woche.von.getTime()) / 3_600_000;
|
||||||
|
expect(stunden).toBe(167);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("hat in einer Woche ohne Umstellung genau 168 Stunden", () => {
|
||||||
|
const woche = wocheUm(new Date("2026-03-31T12:00:00Z"));
|
||||||
|
expect(woche.tage[0]).toBe("2026-03-30");
|
||||||
|
expect((woche.bis.getTime() - woche.von.getTime()) / 3_600_000).toBe(168);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe("Beschriftung", () => {
|
||||||
|
const DIENSTAG = new Date("2026-08-18T09:00:00Z");
|
||||||
|
|
||||||
|
it("nennt die laufende und die vorige Woche beim Namen", () => {
|
||||||
|
expect(wochenName(wocheUm(DIENSTAG))).toBe("Diese Woche");
|
||||||
|
expect(wochenName(wocheUm(DIENSTAG, -1))).toBe("Vorige Woche");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("nennt weiter zurueckliegende Wochen mit Zeitraum", () => {
|
||||||
|
expect(wochenName(wocheUm(DIENSTAG, -2))).toMatch(/\d/);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("nennt den Monat nur einmal, wenn die Woche nicht darueber hinausgeht", () => {
|
||||||
|
expect(wochenTitel(wocheUm(DIENSTAG))).toBe("17.–23. August 2026");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("nennt beide Monate, wenn die Woche sie ueberspannt", () => {
|
||||||
|
// 31.08. bis 06.09.2026.
|
||||||
|
const woche = wocheUm(new Date("2026-09-02T12:00:00Z"));
|
||||||
|
expect(wochenTitel(woche)).toBe("31. August–6. September 2026");
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe("zaehleWoche", () => {
|
||||||
|
const eintrag = (
|
||||||
|
kind: Parameters<typeof zaehleWoche>[0][number]["kind"],
|
||||||
|
projectId: string,
|
||||||
|
meta?: unknown,
|
||||||
|
) => ({ kind, projectId, meta });
|
||||||
|
|
||||||
|
it("zaehlt Erledigtes am meta-Feld, nicht am Text", () => {
|
||||||
|
const zahlen = zaehleWoche([
|
||||||
|
eintrag("TASK_MOVED", "p1", { from: "TODO", to: "DONE" }),
|
||||||
|
// Verschoben, aber nicht erledigt – darf nicht mitzaehlen.
|
||||||
|
eintrag("TASK_MOVED", "p1", { from: "TODO", to: "DOING" }),
|
||||||
|
]);
|
||||||
|
expect(zahlen.erledigt).toBe(1);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("zaehlt einen Eintrag ohne meta nicht als erledigt", () => {
|
||||||
|
// Aeltere Eintraege koennen kein "to" haben. Lieber untertreiben als
|
||||||
|
// eine Zahl behaupten, die aus dem Anzeigetext geraten waere.
|
||||||
|
expect(zaehleWoche([eintrag("TASK_MOVED", "p1")]).erledigt).toBe(0);
|
||||||
|
expect(zaehleWoche([eintrag("TASK_MOVED", "p1", "unsinn")]).erledigt).toBe(0);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("zaehlt neue Aufgaben aus beiden Quellen", () => {
|
||||||
|
const zahlen = zaehleWoche([
|
||||||
|
eintrag("TASK_ADDED", "p1"),
|
||||||
|
eintrag("TASKS_IMPORTED", "p2"),
|
||||||
|
]);
|
||||||
|
expect(zahlen.neueAufgaben).toBe(2);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("zaehlt beruehrte Projekte ohne Dubletten", () => {
|
||||||
|
const zahlen = zaehleWoche([
|
||||||
|
eintrag("NOTE_ADDED", "p1"),
|
||||||
|
eintrag("NOTE_ADDED", "p1"),
|
||||||
|
eintrag("REPO_COMMITS", "p2"),
|
||||||
|
]);
|
||||||
|
expect(zahlen.projekte).toBe(2);
|
||||||
|
expect(zahlen.notizen).toBe(2);
|
||||||
|
expect(zahlen.gesamt).toBe(3);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("kommt mit einer leeren Woche zurecht", () => {
|
||||||
|
expect(zaehleWoche([])).toEqual({
|
||||||
|
gesamt: 0,
|
||||||
|
projekte: 0,
|
||||||
|
erledigt: 0,
|
||||||
|
neueAufgaben: 0,
|
||||||
|
notizen: 0,
|
||||||
|
});
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -0,0 +1,203 @@
|
|||||||
|
import type { ActivityKind } from "@prisma/client";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Wochenrueckblick: welche Woche, und was zaehlt darin.
|
||||||
|
*
|
||||||
|
* Die Wochenrechnung steht hier als reine Funktion und nicht in der Seite –
|
||||||
|
* Wochengrenzen sind der Teil, der still falsch wird, und so laesst er sich
|
||||||
|
* pruefen.
|
||||||
|
*/
|
||||||
|
|
||||||
|
/** Wochenbeginn ist Montag, wie in Deutschland ueblich (ISO 8601). */
|
||||||
|
export const WOCHENBEGINN = 1;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Kalendertag in fester Zeitzone.
|
||||||
|
*
|
||||||
|
* Dieselbe Ueberlegung wie bei den Faelligkeiten in lib/tasks: im Container
|
||||||
|
* laeuft die Anwendung nach UTC. Um 23:30 Berliner Zeit waere dort noch der
|
||||||
|
* Vortag, und ein Eintrag von Sonntagabend fiele in die falsche Woche.
|
||||||
|
*/
|
||||||
|
const TAG_FORMAT = new Intl.DateTimeFormat("en-CA", {
|
||||||
|
timeZone: "Europe/Berlin",
|
||||||
|
year: "numeric",
|
||||||
|
month: "2-digit",
|
||||||
|
day: "2-digit",
|
||||||
|
});
|
||||||
|
|
||||||
|
/** "2026-08-17" in Berliner Zeit. */
|
||||||
|
export function tagesschluessel(wert: Date): string {
|
||||||
|
return TAG_FORMAT.format(wert);
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Wochentag in Berliner Zeit, 0 = Sonntag. */
|
||||||
|
function wochentag(wert: Date): number {
|
||||||
|
const [jahr, monat, tag] = tagesschluessel(wert).split("-").map(Number);
|
||||||
|
return new Date(Date.UTC(jahr, monat - 1, tag)).getUTCDay();
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface Woche {
|
||||||
|
/** Beginn des Montags, als Zeitpunkt fuer die Abfrage. */
|
||||||
|
von: Date;
|
||||||
|
/** Beginn des folgenden Montags – ausschliesslich. */
|
||||||
|
bis: Date;
|
||||||
|
/** 0 = laufende Woche, -1 = vorige, … */
|
||||||
|
versatz: number;
|
||||||
|
/** Die sieben Kalendertage der Woche, Montag zuerst. */
|
||||||
|
tage: string[];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Grenzen der Woche, in der `jetzt` liegt – um `versatz` Wochen verschoben.
|
||||||
|
*
|
||||||
|
* Gerechnet wird ueber die Kalendertage in Berliner Zeit und dann in UTC
|
||||||
|
* zurueckgerechnet. Der Umweg ist noetig, weil zwischen zwei Montagen eine
|
||||||
|
* Zeitumstellung liegen kann: "sieben mal 24 Stunden" waere dann um eine
|
||||||
|
* Stunde daneben.
|
||||||
|
*/
|
||||||
|
export function wocheUm(jetzt: Date, versatz = 0): Woche {
|
||||||
|
const [jahr, monat, tag] = tagesschluessel(jetzt).split("-").map(Number);
|
||||||
|
const heuteUtc = Date.UTC(jahr, monat - 1, tag);
|
||||||
|
|
||||||
|
// Rueckwaerts bis zum Montag.
|
||||||
|
const abstandZumMontag = (wochentag(jetzt) - WOCHENBEGINN + 7) % 7;
|
||||||
|
const montag = new Date(heuteUtc - (abstandZumMontag - versatz * 7) * 86_400_000);
|
||||||
|
|
||||||
|
const tage: string[] = [];
|
||||||
|
for (let i = 0; i < 7; i += 1) {
|
||||||
|
tage.push(new Date(montag.getTime() + i * 86_400_000).toISOString().slice(0, 10));
|
||||||
|
}
|
||||||
|
|
||||||
|
const von = berlinMitternacht(tage[0]);
|
||||||
|
return {
|
||||||
|
von,
|
||||||
|
// Ende ist die Mitternacht *nach* dem Sonntag – nicht "von + 7 Tage":
|
||||||
|
// liegt eine Zeitumstellung dazwischen, ist die Woche 167 oder 169
|
||||||
|
// Stunden lang.
|
||||||
|
bis: berlinMitternacht(naechsterTag(tage[6])),
|
||||||
|
versatz,
|
||||||
|
tage,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Ein Kalendertag weiter, rein auf der Zeichenkette gerechnet. */
|
||||||
|
function naechsterTag(tag: string): string {
|
||||||
|
const [j, m, t] = tag.split("-").map(Number);
|
||||||
|
return new Date(Date.UTC(j, m - 1, t) + 86_400_000).toISOString().slice(0, 10);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Um wie viel geht die Berliner Uhr zu diesem Zeitpunkt der UTC voraus?
|
||||||
|
*
|
||||||
|
* Ermittelt statt fest verdrahtet, weil zwischen Sommer- und Winterzeit eine
|
||||||
|
* Stunde Unterschied liegt und ein Rueckblick ueber die Umstellung hinweg
|
||||||
|
* sonst um eine Stunde verrutscht.
|
||||||
|
*/
|
||||||
|
function zonenversatzMs(zeitpunkt: Date): number {
|
||||||
|
// sv-SE liefert "2026-08-17 14:30:00" – nah genug an ISO, um es nach dem
|
||||||
|
// Austausch des Leerzeichens wieder einlesen zu koennen.
|
||||||
|
const berlin = new Intl.DateTimeFormat("sv-SE", {
|
||||||
|
timeZone: "Europe/Berlin",
|
||||||
|
dateStyle: "short",
|
||||||
|
timeStyle: "medium",
|
||||||
|
}).format(zeitpunkt);
|
||||||
|
return Date.parse(`${berlin.replace(" ", "T")}Z`) - zeitpunkt.getTime();
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Der Zeitpunkt, an dem in Berlin dieser Kalendertag beginnt. */
|
||||||
|
export function berlinMitternacht(tag: string): Date {
|
||||||
|
const basis = new Date(`${tag}T00:00:00Z`);
|
||||||
|
// Die Umstellung findet nachts um zwei statt, nie um Mitternacht – der
|
||||||
|
// Versatz an dieser Stelle ist also eindeutig.
|
||||||
|
return new Date(basis.getTime() - zonenversatzMs(basis));
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface Kennzahlen {
|
||||||
|
gesamt: number;
|
||||||
|
projekte: number;
|
||||||
|
erledigt: number;
|
||||||
|
neueAufgaben: number;
|
||||||
|
notizen: number;
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Was ein Verlaufseintrag fuer die Zaehlung hergeben muss. */
|
||||||
|
export interface ZaehlbarerEintrag {
|
||||||
|
kind: ActivityKind;
|
||||||
|
projectId: string;
|
||||||
|
meta?: unknown;
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Kennzahlen einer Woche.
|
||||||
|
*
|
||||||
|
* "Erledigt" kommt aus dem `meta`-Feld (`to: "DONE"`) und nicht aus dem
|
||||||
|
* Anzeigetext. Der Text ist zum Lesen da und aendert sich mit der
|
||||||
|
* Formulierung – eine Zaehlung, die daran haengt, wird beim naechsten
|
||||||
|
* umformulierten Einzeiler still falsch.
|
||||||
|
*
|
||||||
|
* Fuer aeltere Eintraege ohne `to` bleibt ein Rueckfall auf den Text: der
|
||||||
|
* Verlauf haelt bis zu 200 Eintraege je Projekt, darunter also durchaus
|
||||||
|
* welche von vor dieser Aenderung.
|
||||||
|
*/
|
||||||
|
export function zaehleWoche(eintraege: ZaehlbarerEintrag[]): Kennzahlen {
|
||||||
|
let erledigt = 0;
|
||||||
|
let neueAufgaben = 0;
|
||||||
|
let notizen = 0;
|
||||||
|
const projekte = new Set<string>();
|
||||||
|
|
||||||
|
for (const eintrag of eintraege) {
|
||||||
|
projekte.add(eintrag.projectId);
|
||||||
|
|
||||||
|
if (eintrag.kind === "TASK_ADDED" || eintrag.kind === "TASKS_IMPORTED") {
|
||||||
|
neueAufgaben += 1;
|
||||||
|
} else if (eintrag.kind === "NOTE_ADDED") {
|
||||||
|
notizen += 1;
|
||||||
|
} else if (eintrag.kind === "TASK_MOVED" && istErledigt(eintrag)) {
|
||||||
|
erledigt += 1;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return { gesamt: eintraege.length, projekte: projekte.size, erledigt, neueAufgaben, notizen };
|
||||||
|
}
|
||||||
|
|
||||||
|
function istErledigt(eintrag: ZaehlbarerEintrag): boolean {
|
||||||
|
const meta = eintrag.meta;
|
||||||
|
if (meta && typeof meta === "object" && "to" in meta) {
|
||||||
|
return (meta as { to?: unknown }).to === "DONE";
|
||||||
|
}
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
const MONAT_FORMAT = new Intl.DateTimeFormat("de-DE", {
|
||||||
|
timeZone: "Europe/Berlin",
|
||||||
|
month: "long",
|
||||||
|
});
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Beschriftung einer Woche: "17.–23. August 2026", ueber einen
|
||||||
|
* Monatswechsel hinweg "31. August–6. September 2026".
|
||||||
|
*
|
||||||
|
* Der Punkt hinter der Tageszahl kommt von Hand: Intl liefert bei
|
||||||
|
* `day: "numeric"` nur die Ziffer, und mit `day: "2-digit"` staende dort
|
||||||
|
* "07." statt "7.".
|
||||||
|
*/
|
||||||
|
export function wochenTitel(woche: Woche): string {
|
||||||
|
const teile = (tag: string) => {
|
||||||
|
const [j, m, t] = tag.split("-").map(Number);
|
||||||
|
return { tag: t, monat: MONAT_FORMAT.format(new Date(Date.UTC(j, m - 1, t, 12))), jahr: j };
|
||||||
|
};
|
||||||
|
|
||||||
|
const von = teile(woche.tage[0]);
|
||||||
|
const bis = teile(woche.tage[6]);
|
||||||
|
const gleicherMonat = von.monat === bis.monat && von.jahr === bis.jahr;
|
||||||
|
|
||||||
|
const links = gleicherMonat ? `${von.tag}.` : `${von.tag}. ${von.monat}`;
|
||||||
|
return `${links}–${bis.tag}. ${bis.monat} ${bis.jahr}`;
|
||||||
|
}
|
||||||
|
|
||||||
|
/** "Diese Woche", "Vorige Woche", sonst der Zeitraum. */
|
||||||
|
export function wochenName(woche: Woche): string {
|
||||||
|
if (woche.versatz === 0) return "Diese Woche";
|
||||||
|
if (woche.versatz === -1) return "Vorige Woche";
|
||||||
|
return wochenTitel(woche);
|
||||||
|
}
|
||||||
+22
-1
@@ -5,6 +5,7 @@ import { clientForConnection, createClient, parseRepoUrl } from "@/lib/git";
|
|||||||
import type { GitClient } from "@/lib/git/types";
|
import type { GitClient } from "@/lib/git/types";
|
||||||
import { notifyAfterSync } from "@/lib/notify";
|
import { notifyAfterSync } from "@/lib/notify";
|
||||||
import { repoTitle } from "@/lib/repositories";
|
import { repoTitle } from "@/lib/repositories";
|
||||||
|
import { reconcileLinkedIssues } from "@/lib/tasks/issues";
|
||||||
import type { RepoSnapshotData } from "@/lib/git/types";
|
import type { RepoSnapshotData } from "@/lib/git/types";
|
||||||
|
|
||||||
export interface SyncResult {
|
export interface SyncResult {
|
||||||
@@ -62,11 +63,31 @@ export async function syncRepository(repositoryId: string): Promise<SyncResult>
|
|||||||
// README hineingeschrieben werden koennen.
|
// README hineingeschrieben werden koennen.
|
||||||
await maybeRefreshCalendar(repository.id, client, ref);
|
await maybeRefreshCalendar(repository.id, client, ref);
|
||||||
await maybeRefreshReadme(repository.id, client, ref);
|
await maybeRefreshReadme(repository.id, client, ref);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Verknuepfte Aufgaben nachziehen: was in Git geschlossen wurde, wird
|
||||||
|
* hier abgehakt. Der Webhook macht das sofort – ist keiner eingerichtet
|
||||||
|
* oder faellt er aus, greift der Abgleich.
|
||||||
|
*
|
||||||
|
* Die offenen Issues aus dem gerade geholten Snapshot gehen mit hinein:
|
||||||
|
* fuer jede Nummer, die dort steht, ist die Nachfrage gespart.
|
||||||
|
*/
|
||||||
|
const abgehakt = await reconcileLinkedIssues(
|
||||||
|
repository.id,
|
||||||
|
client,
|
||||||
|
ref,
|
||||||
|
data.issues.map((issue) => issue.number),
|
||||||
|
);
|
||||||
|
|
||||||
// Nach dem Speichern, damit die Meldung den neuen Stand sieht.
|
// Nach dem Speichern, damit die Meldung den neuen Stand sieht.
|
||||||
await notifyAfterSync(repository.id);
|
await notifyAfterSync(repository.id);
|
||||||
return {
|
return {
|
||||||
ok: true,
|
ok: true,
|
||||||
message: `Snapshot aktualisiert: ${data.branches.length} Branches, ${data.tags.length} Tags.`,
|
message:
|
||||||
|
`Snapshot aktualisiert: ${data.branches.length} Branches, ${data.tags.length} Tags.` +
|
||||||
|
(abgehakt > 0
|
||||||
|
? ` ${abgehakt} Aufgabe(n) abgehakt, weil das Issue geschlossen wurde.`
|
||||||
|
: ""),
|
||||||
fetchedAt: new Date().toISOString(),
|
fetchedAt: new Date().toISOString(),
|
||||||
};
|
};
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
|
|||||||
@@ -1,6 +1,9 @@
|
|||||||
import type { TaskStatus } from "@prisma/client";
|
import type { TaskStatus } from "@prisma/client";
|
||||||
|
import { recordActivity, shorten } from "@/lib/activity";
|
||||||
import { prisma } from "@/lib/db";
|
import { prisma } from "@/lib/db";
|
||||||
import { clientForConnection } from "@/lib/git";
|
import { clientForConnection } from "@/lib/git";
|
||||||
|
import type { GitClient, RepoRef } from "@/lib/git/types";
|
||||||
|
import { nextPosition, refreshDerivedProgress, spawnRecurrence } from "@/lib/tasks";
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Die Bruecke zwischen Nebula-Aufgabe und Git-Issue.
|
* Die Bruecke zwischen Nebula-Aufgabe und Git-Issue.
|
||||||
@@ -68,3 +71,130 @@ export async function mirrorStatusToIssue(
|
|||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Hakt eine Aufgabe ab, weil ihr Issue geschlossen wurde.
|
||||||
|
*
|
||||||
|
* Bewusst an einer Stelle fuer beide Wege – Webhook und Abgleich. Vorher
|
||||||
|
* setzte der Webhook nur den Status und liess dabei aus, was die Oberflaeche
|
||||||
|
* tut: die naechste Fassung einer wiederkehrenden Aufgabe anlegen und den
|
||||||
|
* Vorgang in den Verlauf schreiben.
|
||||||
|
*
|
||||||
|
* Gibt false zurueck, wenn nichts zu tun war (Aufgabe fehlt oder ist schon
|
||||||
|
* erledigt) – der Aufrufer kann daran zaehlen, was er bewirkt hat.
|
||||||
|
*/
|
||||||
|
export async function completeTaskFromIssue(
|
||||||
|
taskId: string,
|
||||||
|
herkunft: "webhook" | "abgleich",
|
||||||
|
): Promise<boolean> {
|
||||||
|
const task = await prisma.task.findUnique({
|
||||||
|
where: { id: taskId },
|
||||||
|
select: { id: true, title: true, status: true, projectId: true, sourceIssue: true },
|
||||||
|
});
|
||||||
|
if (!task || task.status === "DONE") return false;
|
||||||
|
|
||||||
|
await prisma.task.update({
|
||||||
|
where: { id: task.id },
|
||||||
|
data: {
|
||||||
|
status: "DONE",
|
||||||
|
doneAt: new Date(),
|
||||||
|
position: await nextPosition(task.projectId, "DONE"),
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
// Wie beim Abhaken von Hand: eine wiederkehrende Aufgabe erzeugt ihre
|
||||||
|
// naechste Fassung.
|
||||||
|
const folge = await spawnRecurrence(task.id);
|
||||||
|
await refreshDerivedProgress(task.projectId);
|
||||||
|
|
||||||
|
await recordActivity({
|
||||||
|
projectId: task.projectId,
|
||||||
|
// Kein userId: das hat nicht ein Mensch in Nebula getan, sondern das
|
||||||
|
// Git-System gemeldet.
|
||||||
|
kind: "TASK_MOVED",
|
||||||
|
summary:
|
||||||
|
`Issue #${task.sourceIssue} geschlossen → „${shorten(task.title, 40)}“ erledigt` +
|
||||||
|
(herkunft === "abgleich" ? " (beim Abgleich bemerkt)" : "") +
|
||||||
|
(folge ? ", nächste angelegt" : ""),
|
||||||
|
// "from"/"to" wie beim Verschieben von Hand – der Wochenrueckblick
|
||||||
|
// zaehlt daran ab, was erledigt wurde, statt den Text zu durchsuchen.
|
||||||
|
meta: {
|
||||||
|
taskId: task.id,
|
||||||
|
issue: task.sourceIssue,
|
||||||
|
source: herkunft,
|
||||||
|
from: task.status,
|
||||||
|
to: "DONE",
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Wie viele Issues ein Abgleich hoechstens einzeln nachfragt. */
|
||||||
|
const MAX_NACHFRAGEN = 25;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Gleicht beim Abgleich die verknuepften Issues ab: was in Git geschlossen
|
||||||
|
* ist, wird in Nebula abgehakt.
|
||||||
|
*
|
||||||
|
* Warum zusaetzlich zum Webhook: der Webhook wirkt sofort, setzt aber voraus,
|
||||||
|
* dass die Instanz von aussen erreichbar ist und der Haken eingetragen wurde.
|
||||||
|
* Ist er das nicht – oder faellt er einmal aus –, bliebe der Rueckweg sonst
|
||||||
|
* ganz aus. Hier kommt er verzoegert, aber verlaesslich.
|
||||||
|
*
|
||||||
|
* **Nur abhaken, nie zuruecknehmen.** Der Abgleich sieht Zustaende, keine
|
||||||
|
* Ereignisse. Ein offenes Issue kann auch bedeuten, dass das Schliessen von
|
||||||
|
* Nebula aus fehlgeschlagen ist – dann wuerde ein symmetrischer Abgleich eine
|
||||||
|
* bewusst erledigte Aufgabe wieder aufmachen. Das Zuruecknehmen bleibt dem
|
||||||
|
* Webhook vorbehalten, der ein echtes "reopened" sieht.
|
||||||
|
*
|
||||||
|
* Fehler werden geschluckt: ein nicht erreichbares Issue darf den Abgleich
|
||||||
|
* des Repositories nicht scheitern lassen.
|
||||||
|
*/
|
||||||
|
export async function reconcileLinkedIssues(
|
||||||
|
repositoryId: string,
|
||||||
|
client: GitClient,
|
||||||
|
ref: RepoRef,
|
||||||
|
/** Nummern, die der Snapshot gerade als offen gemeldet hat. */
|
||||||
|
offeneNummern: number[] = [],
|
||||||
|
): Promise<number> {
|
||||||
|
const offen = new Set(offeneNummern);
|
||||||
|
|
||||||
|
const kandidaten = await prisma.task.findMany({
|
||||||
|
where: {
|
||||||
|
sourceRepositoryId: repositoryId,
|
||||||
|
sourceIssue: { not: null },
|
||||||
|
status: { not: "DONE" },
|
||||||
|
},
|
||||||
|
select: { id: true, sourceIssue: true },
|
||||||
|
orderBy: { createdAt: "asc" },
|
||||||
|
});
|
||||||
|
|
||||||
|
let abgehakt = 0;
|
||||||
|
let nachfragen = 0;
|
||||||
|
|
||||||
|
for (const task of kandidaten) {
|
||||||
|
const nummer = task.sourceIssue;
|
||||||
|
if (nummer === null) continue;
|
||||||
|
|
||||||
|
// Steht die Nummer in der Liste der offenen Issues, ist nichts zu tun –
|
||||||
|
// und der Aufruf gespart. Umgekehrt gilt das nicht: die Liste ist
|
||||||
|
// gedeckelt, aus dem Fehlen folgt also nichts.
|
||||||
|
if (offen.has(nummer)) continue;
|
||||||
|
|
||||||
|
if (nachfragen >= MAX_NACHFRAGEN) break;
|
||||||
|
nachfragen += 1;
|
||||||
|
|
||||||
|
try {
|
||||||
|
const zustand = await client.getIssueState(ref, nummer);
|
||||||
|
// null heisst "gibt es nicht mehr" – geloescht ist nicht erledigt,
|
||||||
|
// also bleibt die Aufgabe stehen.
|
||||||
|
if (zustand !== "closed") continue;
|
||||||
|
if (await completeTaskFromIssue(task.id, "abgleich")) abgehakt += 1;
|
||||||
|
} catch {
|
||||||
|
// Nichts erreichbar, nichts geaendert – der naechste Lauf versucht es.
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return abgehakt;
|
||||||
|
}
|
||||||
|
|||||||
@@ -0,0 +1,64 @@
|
|||||||
|
import { describe, expect, it } from "vitest";
|
||||||
|
import { SEITE, SEITE_MAX, cursorAus, seitenBedingung, seitenGroesse } from "./timeline";
|
||||||
|
|
||||||
|
const ID = "cmstxnrzs00011cx4nd5tiqbt";
|
||||||
|
|
||||||
|
describe("cursorAus", () => {
|
||||||
|
it("liest Zeitpunkt und ID", () => {
|
||||||
|
const cursor = cursorAus("2026-08-17T22:30:00.000Z", ID);
|
||||||
|
expect(cursor?.id).toBe(ID);
|
||||||
|
expect(cursor?.createdAt.toISOString()).toBe("2026-08-17T22:30:00.000Z");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("faengt ohne Cursor von vorn an", () => {
|
||||||
|
expect(cursorAus(null, null)).toBeNull();
|
||||||
|
expect(cursorAus("2026-08-17T22:30:00.000Z", null)).toBeNull();
|
||||||
|
expect(cursorAus(null, ID)).toBeNull();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("weist Unbrauchbares ab, statt es in die Abfrage zu geben", () => {
|
||||||
|
expect(cursorAus("kein datum", ID)).toBeNull();
|
||||||
|
expect(cursorAus("2026-08-17T22:30:00.000Z", "kurz")).toBeNull();
|
||||||
|
expect(cursorAus("2026-08-17T22:30:00.000Z", "'; DROP TABLE x --")).toBeNull();
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe("seitenBedingung", () => {
|
||||||
|
it("filtert immer auf den Besitzer", () => {
|
||||||
|
expect(seitenBedingung("u1", null)).toEqual({ project: { ownerId: "u1" } });
|
||||||
|
});
|
||||||
|
|
||||||
|
it("behaelt den Besitzer auch mit Cursor", () => {
|
||||||
|
const wo = seitenBedingung("u1", { createdAt: new Date("2026-08-17T22:30:00Z"), id: ID });
|
||||||
|
expect(wo.project).toEqual({ ownerId: "u1" });
|
||||||
|
});
|
||||||
|
|
||||||
|
it("fasst gleichzeitige Eintraege ueber die ID auseinander", () => {
|
||||||
|
// Der eigentliche Zweck: beim Uebernehmen von Issues entstehen mehrere
|
||||||
|
// Zeilen mit identischem createdAt. Ohne den zweiten Zweig wuerde die
|
||||||
|
// naechste Seite sie ueberspringen oder wiederholen.
|
||||||
|
const zeitpunkt = new Date("2026-08-17T22:30:00Z");
|
||||||
|
const wo = seitenBedingung("u1", { createdAt: zeitpunkt, id: ID });
|
||||||
|
expect(wo.OR).toEqual([
|
||||||
|
{ createdAt: { lt: zeitpunkt } },
|
||||||
|
{ createdAt: zeitpunkt, id: { lt: ID } },
|
||||||
|
]);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe("seitenGroesse", () => {
|
||||||
|
it("nimmt die Vorgabe, wenn nichts angefragt wurde", () => {
|
||||||
|
expect(seitenGroesse(null)).toBe(SEITE);
|
||||||
|
expect(seitenGroesse("unsinn")).toBe(SEITE);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("deckelt nach oben und unten", () => {
|
||||||
|
expect(seitenGroesse("1000")).toBe(SEITE_MAX);
|
||||||
|
expect(seitenGroesse("0")).toBe(1);
|
||||||
|
expect(seitenGroesse("-5")).toBe(1);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("nimmt einen gueltigen Wert an", () => {
|
||||||
|
expect(seitenGroesse("25")).toBe(25);
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -0,0 +1,77 @@
|
|||||||
|
import type { Prisma } from "@prisma/client";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Fortlaufender Verlauf ueber alle Projekte.
|
||||||
|
*
|
||||||
|
* Geblaettert wird per Cursor, nicht per `skip`. Der Verlauf schreibt
|
||||||
|
* mehrere Eintraege im selben Augenblick – beim Uebernehmen von Issues etwa
|
||||||
|
* entstehen zehn Zeilen mit identischem `createdAt`. Ein Offset ueber eine
|
||||||
|
* solche Grenze hinweg ueberspringt Zeilen oder zeigt sie zweimal, und zwar
|
||||||
|
* lautlos. Der Cursor aus (createdAt, id) ist dagegen eindeutig.
|
||||||
|
*/
|
||||||
|
|
||||||
|
/** Wie viele Eintraege eine Seite hat. */
|
||||||
|
export const SEITE = 40;
|
||||||
|
/** Deckel gegen einen Aufrufer, der sich alles auf einmal holen will. */
|
||||||
|
export const SEITE_MAX = 100;
|
||||||
|
|
||||||
|
export interface Cursor {
|
||||||
|
/** Zeitpunkt des letzten gezeigten Eintrags. */
|
||||||
|
createdAt: Date;
|
||||||
|
/** Seine ID – entscheidet bei gleichem Zeitpunkt. */
|
||||||
|
id: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Liest einen Cursor aus Abfrageparametern.
|
||||||
|
*
|
||||||
|
* Gibt null zurueck, wenn nichts oder Unbrauchbares ankommt – dann faengt
|
||||||
|
* die Seite eben von vorn an. Ein Fehler waere hier die schlechtere Antwort:
|
||||||
|
* ein abgeschnittener Verweis soll die Seite nicht sperren.
|
||||||
|
*/
|
||||||
|
export function cursorAus(vor: string | null, id: string | null): Cursor | null {
|
||||||
|
if (!vor || !id) return null;
|
||||||
|
const zeitpunkt = new Date(vor);
|
||||||
|
if (Number.isNaN(zeitpunkt.getTime())) return null;
|
||||||
|
// Die ID ist eine cuid; alles andere weisen wir ab, statt es in die
|
||||||
|
// Abfrage zu geben.
|
||||||
|
if (!/^[a-z0-9]{20,32}$/i.test(id)) return null;
|
||||||
|
return { createdAt: zeitpunkt, id };
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Die where-Bedingung fuer eine Seite.
|
||||||
|
*
|
||||||
|
* Besitzer zuerst – wie ueberall im Projekt, damit fremde Datensaetze gar
|
||||||
|
* nicht erst getroffen werden. Der Verlauf selbst kennt keinen Besitzer, die
|
||||||
|
* Bedingung laeuft deshalb ueber das Projekt.
|
||||||
|
*
|
||||||
|
* Der Cursor bedeutet "aelter als der letzte gezeigte Eintrag": entweder
|
||||||
|
* frueher, oder gleichzeitig und mit kleinerer ID. Genau so sortiert auch
|
||||||
|
* die Abfrage, weshalb beides zusammenpasst.
|
||||||
|
*/
|
||||||
|
export function seitenBedingung(ownerId: string, cursor: Cursor | null): Prisma.ActivityWhereInput {
|
||||||
|
const basis: Prisma.ActivityWhereInput = { project: { ownerId } };
|
||||||
|
if (!cursor) return basis;
|
||||||
|
|
||||||
|
return {
|
||||||
|
...basis,
|
||||||
|
OR: [
|
||||||
|
{ createdAt: { lt: cursor.createdAt } },
|
||||||
|
{ createdAt: cursor.createdAt, id: { lt: cursor.id } },
|
||||||
|
],
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Sortierung, die zur Cursor-Bedingung passt. Beide muessen gleich sein. */
|
||||||
|
export const SEITEN_ORDNUNG: Prisma.ActivityOrderByWithRelationInput[] = [
|
||||||
|
{ createdAt: "desc" },
|
||||||
|
{ id: "desc" },
|
||||||
|
];
|
||||||
|
|
||||||
|
/** Begrenzt eine gewuenschte Seitengroesse auf das Erlaubte. */
|
||||||
|
export function seitenGroesse(roh: string | null): number {
|
||||||
|
const angefragt = roh === null ? Number.NaN : Number(roh);
|
||||||
|
if (!Number.isFinite(angefragt)) return SEITE;
|
||||||
|
return Math.min(Math.max(Math.trunc(angefragt), 1), SEITE_MAX);
|
||||||
|
}
|
||||||
+12
-2
@@ -15,7 +15,17 @@ import { NextResponse, type NextRequest } from "next/server";
|
|||||||
*/
|
*/
|
||||||
|
|
||||||
const SAFE_METHODS = new Set(["GET", "HEAD", "OPTIONS"]);
|
const SAFE_METHODS = new Set(["GET", "HEAD", "OPTIONS"]);
|
||||||
const SESSION_COOKIE = "nebula_session";
|
|
||||||
|
/**
|
||||||
|
* Beide moeglichen Namen des Sitzungscookies.
|
||||||
|
*
|
||||||
|
* Bei https traegt er den Praefix "__Host-", sonst nicht (der Praefix
|
||||||
|
* verlangt zwingend Secure). Welcher gilt, entscheidet lib/config anhand
|
||||||
|
* der APP_URL – hier im Edge-Runtime wird schlicht auf beide geschaut. Es
|
||||||
|
* geht ohnehin nur um die grobe Vorpruefung; verbindlich entscheidet
|
||||||
|
* requireUserPage bzw. requireUserApi.
|
||||||
|
*/
|
||||||
|
const SESSION_COOKIES = ["__Host-nebula_session", "nebula_session"];
|
||||||
|
|
||||||
/** Ohne Anmeldung erreichbar. */
|
/** Ohne Anmeldung erreichbar. */
|
||||||
const PUBLIC_PATHS = ["/login", "/setup", "/register"];
|
const PUBLIC_PATHS = ["/login", "/setup", "/register"];
|
||||||
@@ -110,7 +120,7 @@ export function middleware(request: NextRequest) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// ── 2. Zugangsschranke ─────────────────────────────────────
|
// ── 2. Zugangsschranke ─────────────────────────────────────
|
||||||
const hasSession = Boolean(request.cookies.get(SESSION_COOKIE)?.value);
|
const hasSession = SESSION_COOKIES.some((name) => Boolean(request.cookies.get(name)?.value));
|
||||||
if (!hasSession && !isPublicPath(pathname)) {
|
if (!hasSession && !isPublicPath(pathname)) {
|
||||||
if (pathname.startsWith("/api/")) {
|
if (pathname.startsWith("/api/")) {
|
||||||
return NextResponse.json({ error: "Nicht angemeldet." }, { status: 401 });
|
return NextResponse.json({ error: "Nicht angemeldet." }, { status: 401 });
|
||||||
|
|||||||
@@ -100,6 +100,24 @@ export interface TaskOverviewItem {
|
|||||||
project: { id: string; name: string; accent: string; status: ProjectStatus };
|
project: { id: string; name: string; accent: string; status: ProjectStatus };
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/** Ein Verlaufseintrag im Wochenrueckblick – mit Projekt, ueber Projekte hinweg. */
|
||||||
|
export interface ReviewEntry {
|
||||||
|
id: string;
|
||||||
|
kind: ActivityKind;
|
||||||
|
summary: string;
|
||||||
|
/** ISO – ein Date ueberlebt den Weg zum Browser nicht. */
|
||||||
|
createdAt: string;
|
||||||
|
project: { id: string; name: string; accent: string };
|
||||||
|
user: { username: string; displayName: string | null } | null;
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Ein Kalendertag der Woche samt seiner Eintraege. */
|
||||||
|
export interface ReviewDay {
|
||||||
|
/** "2026-08-17", in Berliner Zeit bestimmt. */
|
||||||
|
tag: string;
|
||||||
|
eintraege: ReviewEntry[];
|
||||||
|
}
|
||||||
|
|
||||||
export interface RepoListEntry {
|
export interface RepoListEntry {
|
||||||
fullPath: string;
|
fullPath: string;
|
||||||
owner: string;
|
owner: string;
|
||||||
|
|||||||
Reference in New Issue
Block a user