Files
mcpedia/packages/core/src/revision.service.ts
T
asepharyana b92f6f91fa feat(mcpedia): Phase 4 — operability + correctness hardening
Reinterpreted from the plan's YAGNI 'Scale-out' (OpenSearch/object-storage
/multi-tenant deferred at KB scale). Phase 4 = make the Phase 3 async +
revision system correct, secure, observable, deployable.

- T1 (correctness bug): restoreRevision now rebuilds semantic chunks via new
  @mcpedia/core reindexChunks(slug) so semantic/hybrid search stay consistent
  after a restore (previously document_chunks held the NEW body while
  documents.body held the restored OLD body -> stale search).
- T2 (security): /hooks/* git-sync webhooks now require x-webhook-secret header
  matching WEBHOOK_SECRET (401 otherwise); API fails fast at startup if unset.
  Added WEBHOOK_SECRET to @mcpedia/config + .env.example; set real secret in .env.
- T3 (UX): web doc page shows a History panel (revision no/reason/date/length)
  with per-revision Restore; app/api/revisions/restore/route.ts calls
  restoreRevision + revalidatePath (server-component only, no client JS).
- T4: listRevisions gains offset paging; summary never includes body.
- T5 (ops): deploy/mcpedia-api.service + deploy/mcpedia-worker.service systemd
  units (Restart=on-failure, EnvironmentFile=.env). Not auto-enabled on host.

Verified against live imrnes Redis + Postgres: turbo typecheck+build green;
restore-rebuilds-chunks (marker present -> gone after restore); webhook 401/200;
web restore route redirects to doc + reverts body; revisions API returns summary
(no body); systemd-analyze verify passes.
2026-08-19 21:54:54 +07:00

125 lines
3.4 KiB
TypeScript

import { db } from "@mcpedia/db";
import { documents, documentRevisions, documentChunks } from "@mcpedia/db/schema";
import { reindexChunks } from "./index.service";
import { eq, desc, and, sql } from "drizzle-orm";
import { toMeta } from "./row-map";
import type { DocumentMeta } from "@mcpedia/types";
export interface RevisionSummary {
id: string;
slug: string;
revisionNo: number;
title: string;
reason: string;
createdAt: string;
bodyLength: number;
}
/** List revisions for a slug, newest first. `offset` enables paging. */
export async function listRevisions(
slug: string,
limit = 20,
offset = 0,
): Promise<RevisionSummary[]> {
const [doc] = await db
.select({ id: documents.id })
.from(documents)
.where(eq(documents.slug, slug));
if (!doc) return [];
const rows = await db
.select({
id: documentRevisions.id,
slug: documentRevisions.slug,
revisionNo: documentRevisions.revisionNo,
title: documentRevisions.title,
reason: documentRevisions.reason,
createdAt: documentRevisions.createdAt,
bodyLength: sql<number>`length(${documentRevisions.body})`,
})
.from(documentRevisions)
.where(eq(documentRevisions.documentId, doc.id))
.orderBy(desc(documentRevisions.revisionNo))
.limit(limit)
.offset(offset);
return rows.map((r) => ({
id: r.id,
slug: r.slug,
revisionNo: r.revisionNo,
title: r.title,
reason: r.reason,
createdAt: r.createdAt.toISOString(),
bodyLength: r.bodyLength,
}));
}
/** Fetch a single revision's full body. */
export async function getRevision(
id: string,
): Promise<{ id: string; revisionNo: number; body: string; meta: unknown } | null> {
const [row] = await db
.select({
id: documentRevisions.id,
revisionNo: documentRevisions.revisionNo,
body: documentRevisions.body,
meta: documentRevisions.meta,
})
.from(documentRevisions)
.where(eq(documentRevisions.id, id));
if (!row) return null;
return {
id: row.id,
revisionNo: row.revisionNo,
body: row.body,
meta: row.meta,
};
}
/** Restore a revision: write its body+metadata back into the live `documents` row. */
export async function restoreRevision(
id: string,
): Promise<{ slug: string; documentId: string } | null> {
const [rev] = await db
.select({
id: documentRevisions.id,
documentId: documentRevisions.documentId,
slug: documentRevisions.slug,
title: documentRevisions.title,
body: documentRevisions.body,
meta: documentRevisions.meta,
})
.from(documentRevisions)
.where(eq(documentRevisions.id, id));
if (!rev) return null;
const m = rev.meta as {
type?: string;
section?: string;
status?: string;
author?: string;
tags?: string[];
};
await db
.update(documents)
.set({
title: rev.title,
type: (m.type as any) ?? "documentation",
section: (m.section as any) ?? "docs",
status: (m.status as any) ?? "published",
author: m.author ?? "",
tags: m.tags ?? [],
body: rev.body,
updatedAt: new Date(),
})
.where(eq(documents.id, rev.documentId));
// Rebuild semantic chunks + embeddings from the restored body so semantic
// and hybrid search stay consistent (otherwise document_chunks would hold
// the NEW body's chunks while documents.body holds the OLD/restore body).
await reindexChunks(rev.slug);
return { slug: rev.slug, documentId: rev.documentId };
}