getgloss 0.8.0 → 0.8.1

package/dist/cli/index.js

@@ -4,6 +4,17 @@
 import { Command } from "commander";
 import openBrowser from "open";
 
+// src/shared/cleanup.ts
+import { readdir, readFile, rm } from "fs/promises";
+
+// src/shared/errors.ts
+function formatError(error) {
+  return error instanceof Error ? error.message : String(error);
+}
+function isFileNotFound(error) {
+  return error instanceof Error && "code" in error && error.code === "ENOENT";
+}
+
 // src/shared/paths.ts
 import { mkdir } from "fs/promises";
 import { homedir } from "os";
@@ -12,7 +23,7 @@ import path from "path";
 // package.json
 var package_default = {
   name: "getgloss",
-  version: "0.8.0",
+  version: "0.8.1",
   description: "Local browser-based diff review for coding-agent loops.",
   type: "module",
   packageManager: "pnpm@10.33.2",
@@ -150,14 +161,435 @@ function globalReviewDiffFile(reviewId) {
 function globalReviewFeedbackFile(reviewId) {
   return path.join(globalReviewDir(reviewId), "feedback.json");
 }
-function globalReviewMarkdownFile(reviewId) {
-  return path.join(globalReviewDir(reviewId), "feedback.md");
+function globalReviewMarkdownFile(reviewId) {
+  return path.join(globalReviewDir(reviewId), "feedback.md");
+}
+function globalReviewResolvedFile(reviewId) {
+  return path.join(globalReviewDir(reviewId), "resolved.json");
+}
+async function ensureDir(dir) {
+  await mkdir(dir, { recursive: true });
+}
+
+// src/shared/types.ts
+var SIDES = ["L", "R"];
+var REVIEW_STATUSES = ["pending", "submitted", "cancelled", "resolved"];
+var DIFF_LINE_TYPES = ["context", "add", "delete"];
+var DIFF_SCOPE_MODES = ["working", "branch", "explicit"];
+var DIFF_FALLBACK_REASONS = ["working-tree-clean", "missing-branch-base"];
+var REVIEW_SCOPE_MODES = ["all", "single", "range"];
+var RESOLUTION_STATUSES = ["partial", "resolved"];
+var REVIEW_UPDATE_REASONS = [
+  "review-resolved",
+  "comment-resolved",
+  "comment-reopened",
+  "turn-created"
+];
+
+// src/shared/validation.ts
+function parseJson(raw, guard, label) {
+  const parsed = JSON.parse(raw);
+  return parseJsonValue(parsed, guard, label);
+}
+function parseJsonValue(value, guard, label) {
+  if (!guard(value)) {
+    throw new Error(`Invalid ${label}`);
+  }
+  return value;
+}
+function isServerInfo(value) {
+  return isRecord(value) && isNumber(value.pid) && isNumber(value.port) && isString(value.version) && isString(value.startedAt) && isString(value.stateDir);
+}
+function isHealthResponse(value) {
+  return isRecord(value) && isBoolean(value.ok) && isString(value.version) && isNumber(value.activeReviews);
+}
+function isClearReviewsResult(value) {
+  return isRecord(value) && isString(value.reviewsDir) && isString(value.cutoff) && isNumber(value.olderThanDays) && isBoolean(value.dryRun) && isArrayOf(value.candidates, isClearReviewEntry) && isArrayOf(value.deleted, isClearReviewEntry) && isArrayOf(value.skipped, isClearReviewSkipped) && isRecord(value.counts) && isNumber(value.counts.candidates) && isNumber(value.counts.deleted) && isNumber(value.counts.skipped);
+}
+function isCreateReviewResponse(value) {
+  return isRecord(value) && hasReviewRegistrationFields(value) && isOptional(value.turn, isReviewTurnSummary);
+}
+function isCreateReviewTurnResponse(value) {
+  return isRecord(value) && hasReviewRegistrationFields(value) && isReviewTurnSummary(value.turn) && isBoolean(value.reused);
+}
+function isListReviewsResponse(value) {
+  return isRecord(value) && isArrayOf(value.reviews, isReviewMeta);
+}
+function isOpenResult(value) {
+  return isRecord(value) && isString(value.reviewId) && isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isString(value.url) && isNumber(value.files) && isOptionalNumber(value.comments) && isOptionalString(value.feedbackPath) && isOptionalString(value.markdownPath) && isOptionalString(value.artifactDir);
+}
+function isResolveResult(value) {
+  return isRecord(value) && value.ok === true && isString(value.reviewId) && isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isReviewStatus(value.status) && isResolutionStatus(value.resolutionStatus) && isResolutionCounts(value.comments) && isString(value.path) && isResolutionBundle(value.resolution);
+}
+function isReviewRecord(value) {
+  return isRecord(value) && isReviewMeta(value.meta) && isArrayOf(value.turns, isReviewTurn) && isDiffPayload(value.diff) && isOptional(value.feedback, isFeedbackBundle) && isOptional(value.resolution, isResolutionBundle);
+}
+function isStoredReviewMeta(value) {
+  return isRecord(value) && isString(value.id) && isString(value.cwd) && isBaseRef(value.base) && isNullableString(value.branch) && isReviewStatus(value.status) && isString(value.createdAt) && isOptionalString(value.submittedAt) && isOptionalString(value.resolvedAt) && isOptionalString(value.artifactDir) && isOptionalString(value.activeTurnId) && isOptional(
+    value.turns,
+    (turns) => isArrayOf(turns, isReviewTurnSummary)
+  ) && isOptionalString(value.feedbackPath) && isOptionalString(value.markdownPath);
+}
+function isReviewMeta(value) {
+  return isStoredReviewMeta(value) && isString(value.artifactDir);
+}
+function hasReviewRegistrationFields(value) {
+  return isReviewMeta(value.meta) && isString(value.url);
+}
+function isDiffPayload(value) {
+  return isRecord(value) && isBaseRef(value.base) && isNullableString(value.branch) && isString(value.cwd) && isDiffScope(value.scope) && isDiffStats(value.stats) && isString(value.rawDiff) && isArrayOf(value.files, isDiffFile) && isOptional(
+    value.commitDiffs,
+    (commitDiffs) => isArrayOf(commitDiffs, isCommitDiff)
+  ) && isString(value.capturedAt);
+}
+function isFeedbackBundle(value) {
+  return isRecord(value) && value.version === 1 && isString(value.reviewId) && isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isString(value.timestamp) && isBaseRef(value.base) && isNullableString(value.branch) && isOptional(value.reviewScope, isReviewScope) && isArrayOf(value.comments, isComment);
+}
+function isResolutionBundle(value) {
+  return isRecord(value) && isString(value.reviewId) && isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isResolutionStatus(value.status) && isNullableString(value.summary) && isNullableString(value.resolvedAt) && isArrayOf(value.comments, isResolvedComment);
+}
+function isReviewEvent(value) {
+  if (!isRecord(value) || !isString(value.reviewId) || !isString(value.type)) {
+    return false;
+  }
+  switch (value.type) {
+    case "review.opened":
+    case "review.cancelled":
+      return true;
+    case "review.turn.created":
+      return isString(value.turnId) && isNumber(value.turnIndex) && isBoolean(value.reused);
+    case "review.submitted":
+      return isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isRecord(value.counts) && isNumber(value.counts.files) && isNumber(value.counts.comments);
+    case "review.updated":
+      return isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isReviewUpdateReason(value.reason) && isReviewStatus(value.status) && isResolutionStatus(value.resolutionStatus) && isResolutionCounts(value.counts);
+    default:
+      return false;
+  }
+}
+function isReviewTurnMeta(value) {
+  return isRecord(value) && isString(value.id) && isNumber(value.index) && isReviewStatus(value.status) && isString(value.createdAt) && isOptionalString(value.submittedAt) && isOptionalString(value.resolvedAt) && isString(value.artifactDir) && isString(value.diffPath) && isOptionalString(value.feedbackPath) && isOptionalString(value.markdownPath) && isOptionalString(value.resolvedPath);
+}
+function isReviewTurnSummary(value) {
+  if (!isRecord(value) || !isReviewTurnMeta(value)) {
+    return false;
+  }
+  return isString(value.capturedAt) && isDiffStats(value.stats) && isResolutionCounts(value.comments);
+}
+function isReviewTurn(value) {
+  if (!isRecord(value) || !isReviewTurnMeta(value)) {
+    return false;
+  }
+  return isDiffPayload(value.diff) && isOptional(value.feedback, isFeedbackBundle) && isOptional(value.resolution, isResolutionBundle);
+}
+function isClearReviewEntry(value) {
+  return isRecord(value) && isString(value.reviewId) && isReviewStatus(value.status) && isString(value.artifactDir) && isString(value.lastActivityAt);
+}
+function isClearReviewSkipped(value) {
+  return isRecord(value) && isString(value.reviewId) && isString(value.artifactDir) && isString(value.reason);
+}
+function isDiffScope(value) {
+  return isRecord(value) && isOneOf(value.mode, DIFF_SCOPE_MODES) && isNullableString(value.requestedBase) && isBaseRef(value.base) && isDiffRef(value.comparison) && (value.fallbackReason === null || isOneOf(value.fallbackReason, DIFF_FALLBACK_REASONS));
+}
+function isDiffRef(value) {
+  return isRecord(value) && isString(value.ref) && isNullableString(value.sha);
+}
+function isBaseRef(value) {
+  return isRecord(value) && isString(value.ref) && isString(value.sha);
+}
+function isDiffStats(value) {
+  return isRecord(value) && isNumber(value.files) && isNumber(value.additions) && isNumber(value.deletions);
+}
+function isDiffCommit(value) {
+  return isRecord(value) && isString(value.sha) && isString(value.shortSha) && isString(value.subject) && isString(value.authorName) && isString(value.authorEmail) && isString(value.authoredAt) && isString(value.committedAt);
+}
+function isCommitDiff(value) {
+  return isRecord(value) && isDiffCommit(value.commit) && isDiffStats(value.stats) && isString(value.rawDiff) && isArrayOf(value.files, isDiffFile);
+}
+function isReviewScope(value) {
+  if (!isRecord(value) || !isOneOf(value.mode, REVIEW_SCOPE_MODES)) {
+    return false;
+  }
+  switch (value.mode) {
+    case "all":
+      return true;
+    case "single":
+      return isString(value.sha);
+    case "range":
+      return isString(value.fromSha) && isString(value.toSha);
+  }
+}
+function isDiffFile(value) {
+  return isRecord(value) && isString(value.path) && isNullableString(value.oldPath) && isNumber(value.additions) && isNumber(value.deletions) && isBoolean(value.isBinary) && isBoolean(value.isDeleted) && isBoolean(value.isNew) && isBoolean(value.isRenamed) && isNullableString(value.language) && isArrayOf(value.hunks, isDiffHunk);
+}
+function isDiffHunk(value) {
+  return isRecord(value) && isNumber(value.oldStart) && isNumber(value.oldLines) && isNumber(value.newStart) && isNumber(value.newLines) && isString(value.header) && isArrayOf(value.lines, isDiffLine);
+}
+function isDiffLine(value) {
+  return isRecord(value) && isOneOf(value.type, DIFF_LINE_TYPES) && isNullableNumber(value.oldLine) && isNullableNumber(value.newLine) && isString(value.content);
+}
+function isComment(value) {
+  return isRecord(value) && isString(value.id) && isString(value.filePath) && isNumber(value.startLine) && isNumber(value.endLine) && isOneOf(value.side, SIDES) && isString(value.body) && isString(value.originalSnippet) && isString(value.createdAt);
+}
+function isResolvedComment(value) {
+  return isRecord(value) && isString(value.commentId) && value.status === "resolved" && isOptionalString(value.summary) && isString(value.resolvedAt);
+}
+function isResolutionCounts(value) {
+  return isRecord(value) && isNumber(value.total) && isNumber(value.resolved) && isNumber(value.open);
+}
+function isReviewStatus(value) {
+  return isOneOf(value, REVIEW_STATUSES);
+}
+function isResolutionStatus(value) {
+  return isOneOf(value, RESOLUTION_STATUSES);
+}
+function isReviewUpdateReason(value) {
+  return isOneOf(value, REVIEW_UPDATE_REASONS);
+}
+function isRecord(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function isArrayOf(value, guard) {
+  return Array.isArray(value) && value.every(guard);
+}
+function isOptional(value, guard) {
+  return value === void 0 || guard(value);
+}
+function isString(value) {
+  return typeof value === "string";
+}
+function isOptionalString(value) {
+  return value === void 0 || isString(value);
+}
+function isNullableString(value) {
+  return value === null || isString(value);
+}
+function isNumber(value) {
+  return typeof value === "number" && Number.isFinite(value);
+}
+function isOptionalNumber(value) {
+  return value === void 0 || isNumber(value);
+}
+function isNullableNumber(value) {
+  return value === null || isNumber(value);
+}
+function isBoolean(value) {
+  return typeof value === "boolean";
+}
+function isOneOf(value, options) {
+  return typeof value === "string" && options.includes(value);
+}
+
+// src/shared/cleanup.ts
+var DEFAULT_REVIEW_RETENTION_DAYS = 30;
+var clearableStatuses = /* @__PURE__ */ new Set(["submitted", "resolved", "cancelled"]);
+var millisecondsPerDay = 24 * 60 * 60 * 1e3;
+async function clearReviewArtifacts(options = {}) {
+  const olderThanDays = normalizeRetentionDays(options.olderThanDays);
+  const dryRun = options.dryRun === true;
+  const now = options.now ?? /* @__PURE__ */ new Date();
+  const cutoff = new Date(now.getTime() - olderThanDays * millisecondsPerDay);
+  const reviewsDir = globalReviewsDir();
+  const candidates = [];
+  const deleted = [];
+  const skipped = [];
+  let entries;
+  try {
+    entries = await readdir(reviewsDir, { withFileTypes: true });
+  } catch (error) {
+    if (isFileNotFound(error)) {
+      return cleanupResult({
+        reviewsDir,
+        cutoff,
+        olderThanDays,
+        dryRun,
+        candidates,
+        deleted,
+        skipped
+      });
+    }
+    throw new Error(`Could not read reviews directory at ${reviewsDir}: ${formatError(error)}`, {
+      cause: error
+    });
+  }
+  for (const entry of entries) {
+    if (!entry.isDirectory()) {
+      continue;
+    }
+    const reviewId = entry.name;
+    const artifactDir = globalReviewDir(reviewId);
+    const candidate = await cleanupCandidate(reviewId, artifactDir, cutoff, skipped);
+    if (!candidate) {
+      continue;
+    }
+    candidates.push(candidate);
+    if (!dryRun) {
+      await rm(artifactDir, { recursive: true, force: true });
+      deleted.push(candidate);
+    }
+  }
+  return cleanupResult({ reviewsDir, cutoff, olderThanDays, dryRun, candidates, deleted, skipped });
+}
+function normalizeRetentionDays(value) {
+  const days = value ?? DEFAULT_REVIEW_RETENTION_DAYS;
+  if (!Number.isInteger(days) || days < 0) {
+    throw new Error("olderThanDays must be a non-negative integer");
+  }
+  return days;
+}
+async function cleanupCandidate(reviewId, artifactDir, cutoff, skipped) {
+  let raw;
+  try {
+    raw = await readFile(globalReviewMetaFile(reviewId), "utf8");
+  } catch (error) {
+    if (isFileNotFound(error)) {
+      skipped.push({ reviewId, artifactDir, reason: "missing metadata" });
+      return null;
+    }
+    skipped.push({ reviewId, artifactDir, reason: `unreadable metadata: ${formatError(error)}` });
+    return null;
+  }
+  let meta;
+  try {
+    meta = parseJson(raw, isStoredReviewMeta, "review metadata");
+  } catch (error) {
+    skipped.push({ reviewId, artifactDir, reason: `invalid metadata: ${formatError(error)}` });
+    return null;
+  }
+  if (meta.id !== reviewId) {
+    skipped.push({ reviewId, artifactDir, reason: `metadata id mismatch: ${meta.id}` });
+    return null;
+  }
+  if (!clearableStatuses.has(meta.status)) {
+    return null;
+  }
+  const turnState = await persistedTurnCleanupState(reviewId, artifactDir, skipped);
+  if (turnState === "preserve") {
+    return null;
+  }
+  const lastActivityAt = latestTimestamp([
+    ...metadataTimestamps(meta),
+    ...turnState === "none" ? [] : turnState.timestamps
+  ]);
+  if (!lastActivityAt) {
+    skipped.push({ reviewId, artifactDir, reason: "missing valid activity timestamp" });
+    return null;
+  }
+  if (Date.parse(lastActivityAt) >= cutoff.getTime()) {
+    return null;
+  }
+  return {
+    reviewId,
+    status: meta.status,
+    artifactDir,
+    lastActivityAt
+  };
+}
+async function persistedTurnCleanupState(reviewId, artifactDir, skipped) {
+  let entries;
+  try {
+    entries = await readdir(globalReviewTurnsDir(reviewId), { withFileTypes: true });
+  } catch (error) {
+    if (isFileNotFound(error)) {
+      return "none";
+    }
+    skipped.push({
+      reviewId,
+      artifactDir,
+      reason: `unreadable turns directory: ${formatError(error)}`
+    });
+    return "preserve";
+  }
+  const turnDirs = entries.filter((entry) => entry.isDirectory());
+  if (turnDirs.length === 0) {
+    return "none";
+  }
+  const timestamps = [];
+  for (const entry of turnDirs) {
+    const turn = await readPersistedTurnMeta(reviewId, entry.name, artifactDir, skipped);
+    if (!turn) {
+      return "preserve";
+    }
+    if (turn.status === "pending" || !clearableStatuses.has(turn.status)) {
+      return "preserve";
+    }
+    timestamps.push(turn.createdAt, turn.submittedAt, turn.resolvedAt);
+  }
+  return { timestamps };
+}
+async function readPersistedTurnMeta(reviewId, turnDirName, artifactDir, skipped) {
+  let raw;
+  try {
+    raw = await readFile(globalReviewTurnMetaFile(reviewId, turnDirName), "utf8");
+  } catch (error) {
+    skipped.push({
+      reviewId,
+      artifactDir,
+      reason: `${isFileNotFound(error) ? "missing" : "unreadable"} turn metadata for ${turnDirName}${isFileNotFound(error) ? "" : `: ${formatError(error)}`}`
+    });
+    return null;
+  }
+  try {
+    const turn = parseJson(raw, isReviewTurnMeta, "review turn metadata");
+    if (turn.id !== turnDirName) {
+      skipped.push({
+        reviewId,
+        artifactDir,
+        reason: `turn metadata id mismatch for ${turnDirName}: ${turn.id}`
+      });
+      return null;
+    }
+    return turn;
+  } catch (error) {
+    skipped.push({
+      reviewId,
+      artifactDir,
+      reason: `invalid turn metadata for ${turnDirName}: ${formatError(error)}`
+    });
+    return null;
+  }
 }
-function globalReviewResolvedFile(reviewId) {
-  return path.join(globalReviewDir(reviewId), "resolved.json");
+function metadataTimestamps(meta) {
+  return [
+    meta.createdAt,
+    meta.submittedAt,
+    meta.resolvedAt,
+    ...(meta.turns ?? []).flatMap((turn) => [
+      turn.createdAt,
+      turn.capturedAt,
+      turn.submittedAt,
+      turn.resolvedAt
+    ])
+  ];
 }
-async function ensureDir(dir) {
-  await mkdir(dir, { recursive: true });
+function latestTimestamp(timestamps) {
+  const latest = Math.max(
+    ...timestamps.map((timestamp) => timestamp ? Date.parse(timestamp) : Number.NaN).filter((timestamp) => Number.isFinite(timestamp))
+  );
+  return Number.isFinite(latest) ? new Date(latest).toISOString() : null;
+}
+function cleanupResult({
+  reviewsDir,
+  cutoff,
+  olderThanDays,
+  dryRun,
+  candidates,
+  deleted,
+  skipped
+}) {
+  return {
+    reviewsDir,
+    cutoff: cutoff.toISOString(),
+    olderThanDays,
+    dryRun,
+    candidates,
+    deleted,
+    skipped,
+    counts: {
+      candidates: candidates.length,
+      deleted: deleted.length,
+      skipped: skipped.length
+    }
+  };
 }
 
 // src/cli/git.ts
@@ -552,26 +984,18 @@ async function assertGitAvailable() {
 // src/cli/lifecycle.ts
 import { execFile, spawn } from "child_process";
 import { closeSync, existsSync, openSync } from "fs";
-import { rm as rm2 } from "fs/promises";
+import { rm as rm3 } from "fs/promises";
 import { userInfo } from "os";
 import { fileURLToPath } from "url";
 import { promisify } from "util";
 import getPort from "get-port";
 
 // src/shared/server-info.ts
-import { readFile } from "fs/promises";
-
-// src/shared/errors.ts
-function formatError(error) {
-  return error instanceof Error ? error.message : String(error);
-}
-function isFileNotFound(error) {
-  return error instanceof Error && "code" in error && error.code === "ENOENT";
-}
+import { readFile as readFile2 } from "fs/promises";
 
 // src/shared/json.ts
 import { randomUUID } from "crypto";
-import { rename, rm, writeFile } from "fs/promises";
+import { rename, rm as rm2, writeFile } from "fs/promises";
 import path3 from "path";
 function serializeJson(value) {
   return `${JSON.stringify(value, null, 2)}
@@ -589,215 +1013,16 @@ async function writeTextFile(filePath, value) {
     await writeFile(tempPath, value);
     await rename(tempPath, filePath);
   } catch (error) {
-    await rm(tempPath, { force: true }).catch(() => void 0);
+    await rm2(tempPath, { force: true }).catch(() => void 0);
     throw error;
   }
 }
 
-// src/shared/types.ts
-var SIDES = ["L", "R"];
-var REVIEW_STATUSES = ["pending", "submitted", "cancelled", "resolved"];
-var DIFF_LINE_TYPES = ["context", "add", "delete"];
-var DIFF_SCOPE_MODES = ["working", "branch", "explicit"];
-var DIFF_FALLBACK_REASONS = ["working-tree-clean", "missing-branch-base"];
-var REVIEW_SCOPE_MODES = ["all", "single", "range"];
-var RESOLUTION_STATUSES = ["partial", "resolved"];
-var REVIEW_UPDATE_REASONS = [
-  "review-resolved",
-  "comment-resolved",
-  "comment-reopened",
-  "turn-created"
-];
-
-// src/shared/validation.ts
-function parseJson(raw, guard, label) {
-  const parsed = JSON.parse(raw);
-  return parseJsonValue(parsed, guard, label);
-}
-function parseJsonValue(value, guard, label) {
-  if (!guard(value)) {
-    throw new Error(`Invalid ${label}`);
-  }
-  return value;
-}
-function isServerInfo(value) {
-  return isRecord(value) && isNumber(value.pid) && isNumber(value.port) && isString(value.version) && isString(value.startedAt) && isString(value.stateDir);
-}
-function isHealthResponse(value) {
-  return isRecord(value) && isBoolean(value.ok) && isString(value.version) && isNumber(value.activeReviews);
-}
-function isCreateReviewResponse(value) {
-  return isRecord(value) && hasReviewRegistrationFields(value) && isOptional(value.turn, isReviewTurnSummary);
-}
-function isCreateReviewTurnResponse(value) {
-  return isRecord(value) && hasReviewRegistrationFields(value) && isReviewTurnSummary(value.turn) && isBoolean(value.reused);
-}
-function isListReviewsResponse(value) {
-  return isRecord(value) && isArrayOf(value.reviews, isReviewMeta);
-}
-function isOpenResult(value) {
-  return isRecord(value) && isString(value.reviewId) && isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isString(value.url) && isNumber(value.files) && isOptionalNumber(value.comments) && isOptionalString(value.feedbackPath) && isOptionalString(value.markdownPath) && isOptionalString(value.artifactDir);
-}
-function isResolveResult(value) {
-  return isRecord(value) && value.ok === true && isString(value.reviewId) && isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isReviewStatus(value.status) && isResolutionStatus(value.resolutionStatus) && isResolutionCounts(value.comments) && isString(value.path) && isResolutionBundle(value.resolution);
-}
-function isReviewRecord(value) {
-  return isRecord(value) && isReviewMeta(value.meta) && isArrayOf(value.turns, isReviewTurn) && isDiffPayload(value.diff) && isOptional(value.feedback, isFeedbackBundle) && isOptional(value.resolution, isResolutionBundle);
-}
-function isStoredReviewMeta(value) {
-  return isRecord(value) && isString(value.id) && isString(value.cwd) && isBaseRef(value.base) && isNullableString(value.branch) && isReviewStatus(value.status) && isString(value.createdAt) && isOptionalString(value.submittedAt) && isOptionalString(value.resolvedAt) && isOptionalString(value.artifactDir) && isOptionalString(value.activeTurnId) && isOptional(
-    value.turns,
-    (turns) => isArrayOf(turns, isReviewTurnSummary)
-  ) && isOptionalString(value.feedbackPath) && isOptionalString(value.markdownPath);
-}
-function isReviewMeta(value) {
-  return isStoredReviewMeta(value) && isString(value.artifactDir);
-}
-function hasReviewRegistrationFields(value) {
-  return isReviewMeta(value.meta) && isString(value.url);
-}
-function isDiffPayload(value) {
-  return isRecord(value) && isBaseRef(value.base) && isNullableString(value.branch) && isString(value.cwd) && isDiffScope(value.scope) && isDiffStats(value.stats) && isString(value.rawDiff) && isArrayOf(value.files, isDiffFile) && isOptional(
-    value.commitDiffs,
-    (commitDiffs) => isArrayOf(commitDiffs, isCommitDiff)
-  ) && isString(value.capturedAt);
-}
-function isFeedbackBundle(value) {
-  return isRecord(value) && value.version === 1 && isString(value.reviewId) && isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isString(value.timestamp) && isBaseRef(value.base) && isNullableString(value.branch) && isOptional(value.reviewScope, isReviewScope) && isArrayOf(value.comments, isComment);
-}
-function isResolutionBundle(value) {
-  return isRecord(value) && isString(value.reviewId) && isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isResolutionStatus(value.status) && isNullableString(value.summary) && isNullableString(value.resolvedAt) && isArrayOf(value.comments, isResolvedComment);
-}
-function isReviewEvent(value) {
-  if (!isRecord(value) || !isString(value.reviewId) || !isString(value.type)) {
-    return false;
-  }
-  switch (value.type) {
-    case "review.opened":
-    case "review.cancelled":
-      return true;
-    case "review.turn.created":
-      return isString(value.turnId) && isNumber(value.turnIndex) && isBoolean(value.reused);
-    case "review.submitted":
-      return isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isRecord(value.counts) && isNumber(value.counts.files) && isNumber(value.counts.comments);
-    case "review.updated":
-      return isOptionalString(value.turnId) && isOptionalNumber(value.turnIndex) && isReviewUpdateReason(value.reason) && isReviewStatus(value.status) && isResolutionStatus(value.resolutionStatus) && isResolutionCounts(value.counts);
-    default:
-      return false;
-  }
-}
-function isReviewTurnMeta(value) {
-  return isRecord(value) && isString(value.id) && isNumber(value.index) && isReviewStatus(value.status) && isString(value.createdAt) && isOptionalString(value.submittedAt) && isOptionalString(value.resolvedAt) && isString(value.artifactDir) && isString(value.diffPath) && isOptionalString(value.feedbackPath) && isOptionalString(value.markdownPath) && isOptionalString(value.resolvedPath);
-}
-function isReviewTurnSummary(value) {
-  if (!isRecord(value) || !isReviewTurnMeta(value)) {
-    return false;
-  }
-  return isString(value.capturedAt) && isDiffStats(value.stats) && isResolutionCounts(value.comments);
-}
-function isReviewTurn(value) {
-  if (!isRecord(value) || !isReviewTurnMeta(value)) {
-    return false;
-  }
-  return isDiffPayload(value.diff) && isOptional(value.feedback, isFeedbackBundle) && isOptional(value.resolution, isResolutionBundle);
-}
-function isDiffScope(value) {
-  return isRecord(value) && isOneOf(value.mode, DIFF_SCOPE_MODES) && isNullableString(value.requestedBase) && isBaseRef(value.base) && isDiffRef(value.comparison) && (value.fallbackReason === null || isOneOf(value.fallbackReason, DIFF_FALLBACK_REASONS));
-}
-function isDiffRef(value) {
-  return isRecord(value) && isString(value.ref) && isNullableString(value.sha);
-}
-function isBaseRef(value) {
-  return isRecord(value) && isString(value.ref) && isString(value.sha);
-}
-function isDiffStats(value) {
-  return isRecord(value) && isNumber(value.files) && isNumber(value.additions) && isNumber(value.deletions);
-}
-function isDiffCommit(value) {
-  return isRecord(value) && isString(value.sha) && isString(value.shortSha) && isString(value.subject) && isString(value.authorName) && isString(value.authorEmail) && isString(value.authoredAt) && isString(value.committedAt);
-}
-function isCommitDiff(value) {
-  return isRecord(value) && isDiffCommit(value.commit) && isDiffStats(value.stats) && isString(value.rawDiff) && isArrayOf(value.files, isDiffFile);
-}
-function isReviewScope(value) {
-  if (!isRecord(value) || !isOneOf(value.mode, REVIEW_SCOPE_MODES)) {
-    return false;
-  }
-  switch (value.mode) {
-    case "all":
-      return true;
-    case "single":
-      return isString(value.sha);
-    case "range":
-      return isString(value.fromSha) && isString(value.toSha);
-  }
-}
-function isDiffFile(value) {
-  return isRecord(value) && isString(value.path) && isNullableString(value.oldPath) && isNumber(value.additions) && isNumber(value.deletions) && isBoolean(value.isBinary) && isBoolean(value.isDeleted) && isBoolean(value.isNew) && isBoolean(value.isRenamed) && isNullableString(value.language) && isArrayOf(value.hunks, isDiffHunk);
-}
-function isDiffHunk(value) {
-  return isRecord(value) && isNumber(value.oldStart) && isNumber(value.oldLines) && isNumber(value.newStart) && isNumber(value.newLines) && isString(value.header) && isArrayOf(value.lines, isDiffLine);
-}
-function isDiffLine(value) {
-  return isRecord(value) && isOneOf(value.type, DIFF_LINE_TYPES) && isNullableNumber(value.oldLine) && isNullableNumber(value.newLine) && isString(value.content);
-}
-function isComment(value) {
-  return isRecord(value) && isString(value.id) && isString(value.filePath) && isNumber(value.startLine) && isNumber(value.endLine) && isOneOf(value.side, SIDES) && isString(value.body) && isString(value.originalSnippet) && isString(value.createdAt);
-}
-function isResolvedComment(value) {
-  return isRecord(value) && isString(value.commentId) && value.status === "resolved" && isOptionalString(value.summary) && isString(value.resolvedAt);
-}
-function isResolutionCounts(value) {
-  return isRecord(value) && isNumber(value.total) && isNumber(value.resolved) && isNumber(value.open);
-}
-function isReviewStatus(value) {
-  return isOneOf(value, REVIEW_STATUSES);
-}
-function isResolutionStatus(value) {
-  return isOneOf(value, RESOLUTION_STATUSES);
-}
-function isReviewUpdateReason(value) {
-  return isOneOf(value, REVIEW_UPDATE_REASONS);
-}
-function isRecord(value) {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-function isArrayOf(value, guard) {
-  return Array.isArray(value) && value.every(guard);
-}
-function isOptional(value, guard) {
-  return value === void 0 || guard(value);
-}
-function isString(value) {
-  return typeof value === "string";
-}
-function isOptionalString(value) {
-  return value === void 0 || isString(value);
-}
-function isNullableString(value) {
-  return value === null || isString(value);
-}
-function isNumber(value) {
-  return typeof value === "number" && Number.isFinite(value);
-}
-function isOptionalNumber(value) {
-  return value === void 0 || isNumber(value);
-}
-function isNullableNumber(value) {
-  return value === null || isNumber(value);
-}
-function isBoolean(value) {
-  return typeof value === "boolean";
-}
-function isOneOf(value, options) {
-  return typeof value === "string" && options.includes(value);
-}
-
 // src/shared/server-info.ts
 async function readServerInfo() {
   let raw;
   try {
-    raw = await readFile(globalServerFile(), "utf8");
+    raw = await readFile2(globalServerFile(), "utf8");
   } catch (error) {
     if (isFileNotFound(error)) {
       return null;
@@ -845,6 +1070,14 @@ var ServerClient = class {
   async listReviews() {
     return this.get("/api/reviews", isListReviewsResponse, "review list response");
   }
+  async clearReviews(request) {
+    return this.post(
+      "/api/maintenance/clear-reviews",
+      request,
+      isClearReviewsResult,
+      "clear reviews response"
+    );
+  }
   async getFeedback(reviewId) {
     return this.get(`/api/reviews/${reviewId}/feedback`, isFeedbackBundle, "feedback response");
   }
@@ -1065,7 +1298,7 @@ async function stopServer(options = {}) {
         stoppedPids.push(pid);
       }
     }
-    await rm2(globalServerFile(), { force: true });
+    await rm3(globalServerFile(), { force: true });
     return { stopped: stoppedPids.length > 0, info: info2, stoppedPids };
   }
   const info = await readServerInfo();
@@ -1135,7 +1368,7 @@ async function waitForPidExit(pid, timeoutMs) {
 async function removeServerInfoForPid(pid) {
   const current = await readServerInfo().catch(() => null);
   if (!current || current.pid === pid) {
-    await rm2(globalServerFile(), { force: true });
+    await rm3(globalServerFile(), { force: true });
   }
 }
 async function isGlossDaemonPid(pid) {
@@ -1175,7 +1408,7 @@ function isGlossDaemonCommand(command) {
 
 // src/server/store.ts
 import { createHash } from "crypto";
-import { readdir, readFile as readFile2 } from "fs/promises";
+import { readdir as readdir2, readFile as readFile3 } from "fs/promises";
 import path4 from "path";
 import { ulid } from "ulid";
 
@@ -1395,6 +1628,15 @@ var ReviewStore = class {
     await this.loadAllReviews();
     return Array.from(this.reviews.values()).map((record) => record.meta).toSorted((a, b) => a.createdAt.localeCompare(b.createdAt));
   }
+  async clearReviewArtifacts(options = {}) {
+    const result = await clearReviewArtifacts(options);
+    if (!result.dryRun) {
+      for (const review of result.deleted) {
+        this.reviews.delete(review.reviewId);
+      }
+    }
+    return result;
+  }
   async get(id) {
     return this.reviews.get(id) ?? await this.loadKnownReview(id);
   }
@@ -1623,7 +1865,7 @@ var ReviewStore = class {
   async loadAllReviews() {
     let entries;
     try {
-      entries = await readdir(globalReviewsDir(), { withFileTypes: true });
+      entries = await readdir2(globalReviewsDir(), { withFileTypes: true });
     } catch (error) {
       if (isFileNotFound(error)) {
         return;
@@ -1647,7 +1889,7 @@ var ReviewStore = class {
     const metaPath = globalReviewMetaFile(id);
     let metaRaw;
     try {
-      metaRaw = await readFile2(metaPath, "utf8");
+      metaRaw = await readFile3(metaPath, "utf8");
     } catch (error) {
       if (isFileNotFound(error)) {
         return this.loadReviewFromTurnsOnly(id);
@@ -1701,7 +1943,7 @@ var ReviewStore = class {
   async loadPersistedTurns(id) {
     let entries;
     try {
-      entries = await readdir(globalReviewTurnsDir(id), { withFileTypes: true });
+      entries = await readdir2(globalReviewTurnsDir(id), { withFileTypes: true });
     } catch (error) {
       if (isFileNotFound(error)) {
         return [];
@@ -1729,8 +1971,8 @@ var ReviewStore = class {
     let diffRaw;
     try {
       [metaRaw, diffRaw] = await Promise.all([
-        readFile2(metaPath, "utf8"),
-        readFile2(diffPath, "utf8")
+        readFile3(metaPath, "utf8"),
+        readFile3(diffPath, "utf8")
       ]);
     } catch (error) {
       if (isFileNotFound(error)) {
@@ -1760,7 +2002,7 @@ var ReviewStore = class {
     const diffPath = globalReviewDiffFile(id);
     let diffRaw;
     try {
-      diffRaw = await readFile2(diffPath, "utf8");
+      diffRaw = await readFile3(diffPath, "utf8");
     } catch (error) {
       if (isFileNotFound(error)) {
         return null;
@@ -1975,7 +2217,7 @@ function requiredPath(value, label) {
 async function readOptionalJsonFile(filePath, guard, label) {
   let raw;
   try {
-    raw = await readFile2(filePath, "utf8");
+    raw = await readFile3(filePath, "utf8");
   } catch (error) {
     if (isFileNotFound(error)) {
       return void 0;
@@ -2140,6 +2382,19 @@ program.command("stop").description("Stop the managed background server").option
     options.all && result.stoppedPids ? `Stopped ${result.stoppedPids.length} Gloss daemon(s)` : result.stopped ? "Gloss server stopped" : "Gloss server was not running"
   );
 });
+program.command("clear").description("Delete old completed review artifacts").option(
+  "--older-than <days>",
+  "delete completed reviews older than this many days",
+  parseOlderThanDays,
+  DEFAULT_REVIEW_RETENTION_DAYS
+).option("--dry-run", "print cleanup candidates without deleting them").action(async (options) => {
+  const globals = program.opts();
+  const result = await clearReviews({
+    olderThanDays: options.olderThan,
+    dryRun: options.dryRun === true
+  });
+  globals.json ? printJson(result) : printPlain(formatClearResult(result));
+});
 program.command("resolve").argument("<reviewId>", "review id").description("Mark a submitted review or one feedback comment as resolved").option("--comment <commentId>", "resolve one submitted feedback comment").option("--summary <text>", "brief summary of the fixes applied").option("--turn <idOrIndex>", "resolve a specific turn for whole-review resolution").action(
   async (reviewId, options) => {
     const globals = program.opts();
@@ -2245,6 +2500,26 @@ async function baseForExistingReview(client, reviewId) {
   const record = await client.getReview(reviewId);
   return record.diff.scope.mode === "explicit" ? record.diff.scope.requestedBase ?? record.diff.base.ref : null;
 }
+async function clearReviews(options) {
+  const info = await readServerInfo();
+  if (info && await isServerResponsive(info)) {
+    return new ServerClient(serverUrl(info)).clearReviews(options);
+  }
+  return clearReviewArtifacts(options);
+}
+function parseOlderThanDays(value) {
+  const days = Number(value);
+  if (!Number.isInteger(days) || days < 0) {
+    throw new Error("--older-than must be a non-negative integer");
+  }
+  return days;
+}
+function formatClearResult(result) {
+  const action = result.dryRun ? "Would delete" : "Deleted";
+  const count = result.dryRun ? result.counts.candidates : result.counts.deleted;
+  const skipped = result.counts.skipped > 0 ? `; skipped ${result.counts.skipped} invalid review artifact(s)` : "";
+  return `${action} ${count} review artifact(s) older than ${result.olderThanDays} day(s) from ${result.reviewsDir}${skipped}`;
+}
 function isWatchTimeout(error) {
   return error instanceof Error && /^watch timed out after/.test(error.message);
 }