feat(projects): Aktivitaetsverlauf je Projekt
Was ist hier eigentlich passiert - eine Zeitleiste auf der Projektseite beantwortet das jetzt: Statuswechsel, Notizen, Aufgaben, neue Commits. Der Verlauf soll erzaehlen, nicht protokollieren. Deshalb bekommen Status und Repo-Verknuepfung einen eigenen Eintrag, waehrend alle uebrigen Felder zu einem zusammengefasst werden - sonst ertraenkt eine Feinjustage der Farbe alles Wichtige. Ein Titel, der sich aendert, kommt gar nicht vor. Beim Abgleich ist Zurueckhaltung noch wichtiger: er laeuft im Zweifel stuendlich. Vermerkt wird nur, wenn der letzte Commit ein anderer ist als beim letzten Mal, und beim ersten Abgleich gar nicht - dass ein Repo Commits hat, ist keine Neuigkeit. Fehlschlaege stehen nur beim Uebergang in den Fehlerzustand drin, sonst schlaegt ein abgelaufenes Token stuendlich neu auf. Der Text wird beim Schreiben formuliert, nicht beim Anzeigen. Ein Eintrag soll auch dann lesbar bleiben, wenn es die Notiz, um die es ging, laengst nicht mehr gibt. Zweihundert Eintraege je Projekt bleiben erhalten, der taegliche Durchgang stutzt den Rest. Der Export nimmt den Verlauf nicht mit - er beschriebe, was in dieser Instanz passiert ist, und verwiese auf Konten, die es anderswo nicht gibt. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -1,4 +1,6 @@
|
||||
import type { ProjectStatus } from "@prisma/client";
|
||||
import { NextResponse } from "next/server";
|
||||
import { describeFields, describeStatus, recordActivities, shorten } from "@/lib/activity";
|
||||
import { requireUserApi } from "@/lib/auth/guard";
|
||||
import { prisma } from "@/lib/db";
|
||||
import { parseRepoUrl } from "@/lib/git";
|
||||
@@ -98,6 +100,8 @@ export async function PATCH(request: Request, { params }: Params) {
|
||||
|
||||
await prisma.project.update({ where: { id }, data: update });
|
||||
|
||||
await noteChanges(id, auth.user.id, existing, update);
|
||||
|
||||
// Wird die Ableitung eingeschaltet, soll der Wert sofort stimmen und nicht
|
||||
// erst beim naechsten Aufgabenwechsel.
|
||||
if (data.progressFromTasks === true) {
|
||||
@@ -112,6 +116,59 @@ export async function PATCH(request: Request, { params }: Params) {
|
||||
return NextResponse.json({ project });
|
||||
}
|
||||
|
||||
/**
|
||||
* Haelt fest, was sich geaendert hat. Status und Repo-Verknuepfung bekommen
|
||||
* einen eigenen Eintrag – sie sind die Vorgaenge, die man spaeter sucht. Alles
|
||||
* uebrige fasst ein Eintrag zusammen, sonst ertraenkt eine Feinjustage der
|
||||
* Farbe den Verlauf.
|
||||
*/
|
||||
async function noteChanges(
|
||||
projectId: string,
|
||||
userId: string,
|
||||
before: { status: ProjectStatus; repoUrl: string | null },
|
||||
update: Record<string, unknown>,
|
||||
) {
|
||||
const rest = Object.keys(update).filter(
|
||||
(key) => key !== "status" && key !== "slug" && !key.startsWith("repo"),
|
||||
);
|
||||
const entries: Parameters<typeof recordActivities>[0] = [];
|
||||
|
||||
if (update.status !== undefined && update.status !== before.status) {
|
||||
const to = update.status as ProjectStatus;
|
||||
entries.push({
|
||||
projectId,
|
||||
userId,
|
||||
kind: "STATUS_CHANGED",
|
||||
summary: describeStatus(before.status, to),
|
||||
meta: { from: before.status, to },
|
||||
});
|
||||
}
|
||||
|
||||
if (update.repoUrl !== undefined && update.repoUrl !== before.repoUrl) {
|
||||
const url = update.repoUrl as string | null;
|
||||
entries.push({
|
||||
projectId,
|
||||
userId,
|
||||
kind: "REPO_LINKED",
|
||||
summary: url ? `Repository verknüpft: ${shorten(url)}` : "Repository-Verknüpfung entfernt",
|
||||
meta: { repoUrl: url },
|
||||
});
|
||||
}
|
||||
|
||||
const felder = describeFields(rest);
|
||||
if (felder) {
|
||||
entries.push({
|
||||
projectId,
|
||||
userId,
|
||||
kind: "PROJECT_UPDATED",
|
||||
summary: felder,
|
||||
meta: { fields: rest },
|
||||
});
|
||||
}
|
||||
|
||||
await recordActivities(entries);
|
||||
}
|
||||
|
||||
export async function DELETE(_request: Request, { params }: Params) {
|
||||
const auth = await requireUserApi();
|
||||
if (!auth.ok) return auth.response;
|
||||
|
||||
Reference in New Issue
Block a user