@remnic/core 9.66.1 → 9.66.2

package/src/activity/timeline/analysis.ts

@@ -0,0 +1,362 @@
+/**
+ * Optional AI analysis over deterministic timeline cards (issue #2050 first slice).
+ *
+ * Injected provider only. parseConfig, registries, and surfaces wait.
+ * Failures leave cards unchanged. Disabled makes zero provider calls.
+ */
+import { extractJsonCandidates } from "../../json-extract.js";
+import { log } from "../../logger.js";
+import { DEFAULT_TIMELINE_CATEGORIES } from "./categories.js";
+import type { TimelineCard, TimelineCategory, TimelineCorrection, TimelineEvidenceRange, TimelineObservation } from "./types.js";
+
+export const TIMELINE_ANALYSIS_PROMPT_VERSION = 1;
+export const TIMELINE_ANALYSIS_BATCH_SIZE = 40;
+export const TIMELINE_ANALYSIS_BATCH_OVERLAP = 2;
+export const TIMELINE_ANALYSIS_DEFAULT_TIMEOUT_MS = 15_000;
+
+export class TimelineAnalysisConfigError extends Error {
+  override readonly name = "TimelineAnalysisConfigError";
+  readonly code = "invalid_config" as const;
+  constructor(message: string) {
+    super(message);
+  }
+}
+
+export type TimelineAnalysisStatus = "ok" | "disabled" | "provider_failed" | "invalid_output";
+
+export interface TimelineAnalysisCompleteInput {
+  prompt: string;
+  provider: string;
+  model: string;
+  signal: AbortSignal;
+}
+
+export type TimelineAnalysisComplete = (input: TimelineAnalysisCompleteInput) => Promise<string>;
+
+export interface TimelineAnalysisInput {
+  enabled: boolean;
+  cards: readonly TimelineCard[];
+  observations: readonly TimelineObservation[];
+  provider?: string;
+  model?: string;
+  complete?: TimelineAnalysisComplete;
+  timeoutMs?: number;
+  date?: string;
+  timezone?: string;
+  categories?: readonly TimelineCategory[];
+  preferences?: readonly string[];
+  priorEdits?: readonly TimelineCorrection[];
+  signal?: AbortSignal;
+}
+
+export interface TimelineAnalysisResult {
+  status: TimelineAnalysisStatus;
+  cards: TimelineCard[];
+  provider?: string;
+  model?: string;
+  promptVersion?: number;
+}
+
+interface AnalysisOp {
+  cardId: string;
+  title?: string;
+  summary?: string;
+  categoryId?: string;
+  confidence?: number;
+  uncertainty?: string;
+  evidenceRange: TimelineEvidenceRange;
+}
+
+const INSTRUCTIONS = [
+  "Use only supplied evidence.",
+  "Preserve chronology.",
+  "Do not invent people, places, or tasks.",
+  "Avoid productivity or emotional claims.",
+  "Merge adjacent activity only when the evidence justifies it.",
+  "Return strict JSON: {\"ops\":[...]} with evidenceRange, title, summary, categoryId, confidence, uncertainty.",
+  "Cite evidence ranges for every op. Empty ops is a valid no-op.",
+].join(" ");
+
+function fail(
+  status: Exclude<TimelineAnalysisStatus, "ok" | "disabled">,
+  cards: readonly TimelineCard[],
+  provider: string,
+  model: string,
+): TimelineAnalysisResult {
+  log.info(`timeline analysis status=${status} provider=${provider} model=${model} promptVersion=${TIMELINE_ANALYSIS_PROMPT_VERSION}`);
+  return {
+    status,
+    cards: cards as TimelineCard[],
+    provider,
+    model,
+    promptVersion: TIMELINE_ANALYSIS_PROMPT_VERSION,
+  };
+}
+
+function requiredToken(value: string | undefined, name: string): string {
+  if (typeof value !== "string" || value.trim().length === 0) {
+    throw new TimelineAnalysisConfigError(`${name} is required`);
+  }
+  return value.trim();
+}
+
+function evidenceKeyOf(observation: TimelineObservation): string {
+  const parsed = Date.parse(observation.capturedAtUtc);
+  const instant = Number.isFinite(parsed) ? new Date(parsed).toISOString() : observation.capturedAtUtc;
+  return `${observation.machine}|${instant}|${observation.contentHash}`;
+}
+
+function sortObservations(observations: readonly TimelineObservation[]): TimelineObservation[] {
+  return [...observations].sort((a, b) => {
+    const time = Date.parse(a.capturedAtUtc) - Date.parse(b.capturedAtUtc);
+    if (time !== 0) return time;
+    if (a.contentHash !== b.contentHash) return a.contentHash < b.contentHash ? -1 : 1;
+    return a.id - b.id;
+  });
+}
+
+function batchObservations(observations: readonly TimelineObservation[]): TimelineObservation[][] {
+  if (observations.length === 0) return [[]];
+  const size = TIMELINE_ANALYSIS_BATCH_SIZE;
+  const overlap = TIMELINE_ANALYSIS_BATCH_OVERLAP;
+  const step = size - overlap;
+  const batches: TimelineObservation[][] = [];
+  for (let start = 0; start < observations.length; start += step) {
+    batches.push(observations.slice(start, start + size));
+    if (start + size >= observations.length) break;
+  }
+  return batches;
+}
+
+function safeObservation(observation: TimelineObservation) {
+  return {
+    id: observation.id,
+    machine: observation.machine,
+    capturedAtUtc: observation.capturedAtUtc,
+    app: observation.app,
+    windowTitle: observation.windowTitle,
+    ...(observation.browserUrl ? { browserUrl: observation.browserUrl } : {}),
+    contentHash: observation.contentHash,
+  };
+}
+
+function safeCard(card: TimelineCard) {
+  return {
+    id: card.id,
+    kind: card.kind,
+    title: card.title,
+    summary: card.summary,
+    categoryId: card.categoryId,
+    confidence: card.confidence,
+    startUtc: card.startUtc,
+    endUtc: card.endUtc,
+    machine: card.machine,
+    evidenceIds: card.evidenceIds,
+    evidenceRange: card.evidenceRange,
+    ...(card.manualEdit ? { manualEdit: card.manualEdit } : {}),
+  };
+}
+
+export function buildTimelineAnalysisPrompt(input: {
+  date?: string;
+  timezone?: string;
+  cards: readonly TimelineCard[];
+  observations: readonly TimelineObservation[];
+  categories: readonly TimelineCategory[];
+  preferences?: readonly string[];
+  priorEdits?: readonly TimelineCorrection[];
+}): string {
+  const payload = {
+    date: input.date ?? null,
+    timezone: input.timezone ?? null,
+    categories: input.categories.map((category) => ({ id: category.id, name: category.name })),
+    preferences: input.preferences ?? [],
+    priorEdits: input.priorEdits ?? [],
+    cards: input.cards.map(safeCard),
+    observations: input.observations.map(safeObservation),
+  };
+  return `${INSTRUCTIONS}\n${JSON.stringify(payload)}`;
+}
+
+async function callComplete(
+  complete: TimelineAnalysisComplete,
+  prompt: string,
+  provider: string,
+  model: string,
+  timeoutMs: number,
+  parent?: AbortSignal,
+): Promise<string> {
+  const { promise: timeoutPromise, reject } = Promise.withResolvers<string>();
+  const failOnce = (message: string, name: string) => {
+    reject(Object.assign(new Error(message), { name }));
+  };
+  const timer = setTimeout(() => failOnce("timeline analysis timed out", "TimeoutError"), timeoutMs);
+  if (parent?.aborted) {
+    clearTimeout(timer);
+    throw Object.assign(new Error("timeline analysis aborted"), { name: "AbortError" });
+  }
+  const onAbort = () => failOnce("timeline analysis aborted", "AbortError");
+  parent?.addEventListener("abort", onAbort, { once: true });
+  const signal = parent ? AbortSignal.any([parent, AbortSignal.timeout(timeoutMs)]) : AbortSignal.timeout(timeoutMs);
+  try {
+    return await Promise.race([complete({ prompt, provider, model, signal }), timeoutPromise]);
+  } finally {
+    clearTimeout(timer);
+    parent?.removeEventListener("abort", onAbort);
+  }
+}
+
+function parseOps(raw: string): AnalysisOp[] | null {
+  if (typeof raw !== "string" || raw.trim().length === 0) return null;
+  for (const candidate of extractJsonCandidates(raw)) {
+    try {
+      const parsed: unknown = JSON.parse(candidate);
+      if (!parsed || typeof parsed !== "object" || Array.isArray(parsed) || !("ops" in parsed)) continue;
+      const ops = parsed.ops;
+      if (!Array.isArray(ops)) continue;
+      const out: AnalysisOp[] = [];
+      let valid = true;
+      for (const item of ops) {
+        const op = asOp(item);
+        if (!op) {
+          valid = false;
+          break;
+        }
+        out.push(op);
+      }
+      if (valid) return out;
+    } catch {
+      continue;
+    }
+  }
+  return null;
+}
+
+function asOp(value: unknown): AnalysisOp | null {
+  if (!value || typeof value !== "object" || Array.isArray(value)) return null;
+  if (!("cardId" in value) || typeof value.cardId !== "string" || value.cardId.length === 0) return null;
+  if (!("evidenceRange" in value)) return null;
+  const range = value.evidenceRange;
+  if (!range || typeof range !== "object" || Array.isArray(range)) return null;
+  if (!("firstKey" in range) || typeof range.firstKey !== "string" || range.firstKey.length === 0) return null;
+  if (!("lastKey" in range) || typeof range.lastKey !== "string" || range.lastKey.length === 0) return null;
+  if ("title" in value && value.title !== undefined && typeof value.title !== "string") return null;
+  if ("summary" in value && value.summary !== undefined && typeof value.summary !== "string") return null;
+  if ("categoryId" in value && value.categoryId !== undefined && typeof value.categoryId !== "string") return null;
+  if ("uncertainty" in value && value.uncertainty !== undefined && typeof value.uncertainty !== "string") return null;
+  if (
+    "confidence" in value &&
+    value.confidence !== undefined &&
+    (typeof value.confidence !== "number" || value.confidence < 0 || value.confidence > 1)
+  ) {
+    return null;
+  }
+  return {
+    cardId: value.cardId,
+    title: "title" in value && typeof value.title === "string" ? value.title : undefined,
+    summary: "summary" in value && typeof value.summary === "string" ? value.summary : undefined,
+    categoryId: "categoryId" in value && typeof value.categoryId === "string" ? value.categoryId : undefined,
+    confidence: "confidence" in value && typeof value.confidence === "number" ? value.confidence : undefined,
+    uncertainty: "uncertainty" in value && typeof value.uncertainty === "string" ? value.uncertainty : undefined,
+    evidenceRange: { firstKey: range.firstKey, lastKey: range.lastKey },
+  };
+}
+
+function knownEvidenceKeys(cards: readonly TimelineCard[], observations: readonly TimelineObservation[]): Set<string> {
+  const keys = new Set<string>();
+  for (const observation of observations) keys.add(evidenceKeyOf(observation));
+  for (const card of cards) {
+    if (card.evidenceRange) {
+      keys.add(card.evidenceRange.firstKey);
+      keys.add(card.evidenceRange.lastKey);
+    }
+  }
+  return keys;
+}
+
+function applyOps(
+  cards: readonly TimelineCard[],
+  ops: readonly AnalysisOp[],
+  categories: readonly TimelineCategory[],
+  evidenceKeys: ReadonlySet<string>,
+): TimelineCard[] | null {
+  if (ops.length === 0) return cards as TimelineCard[];
+  const byId = new Map(cards.map((card) => [card.id, card]));
+  const allowedCategories = new Set(categories.map((category) => category.id));
+  const next = new Map<string, TimelineCard>();
+  for (const op of ops) {
+    const card = byId.get(op.cardId);
+    if (!card) return null;
+    if (!evidenceKeys.has(op.evidenceRange.firstKey) || !evidenceKeys.has(op.evidenceRange.lastKey)) return null;
+    if (op.categoryId !== undefined && !allowedCategories.has(op.categoryId)) return null;
+    if (card.manualEdit) continue;
+    next.set(card.id, {
+      ...card,
+      ...(op.title !== undefined ? { title: op.title } : {}),
+      ...(op.summary !== undefined ? { summary: op.summary } : {}),
+      ...(op.categoryId !== undefined ? { categoryId: op.categoryId } : {}),
+      ...(op.confidence !== undefined ? { confidence: op.confidence } : {}),
+    });
+  }
+  if (next.size === 0) return cards as TimelineCard[];
+  return cards.map((card) => next.get(card.id) ?? card);
+}
+
+/** Analyze one day of cards. Failures return the input cards unchanged. */
+export async function analyzeTimelineCards(input: TimelineAnalysisInput): Promise<TimelineAnalysisResult> {
+  const cards = input.cards as TimelineCard[];
+  if (!input.enabled) {
+    return { status: "disabled", cards };
+  }
+
+  const provider = requiredToken(input.provider, "provider");
+  const model = requiredToken(input.model, "model");
+  const complete = input.complete;
+  if (typeof complete !== "function") {
+    throw new TimelineAnalysisConfigError("complete is required");
+  }
+  const timeoutMs = input.timeoutMs ?? TIMELINE_ANALYSIS_DEFAULT_TIMEOUT_MS;
+  if (!Number.isInteger(timeoutMs) || timeoutMs <= 0) {
+    throw new TimelineAnalysisConfigError("timeoutMs must be a positive integer");
+  }
+  if (input.cards.length === 0 && input.observations.length === 0) {
+    return fail("invalid_output", cards, provider, model);
+  }
+
+  const categories = input.categories ?? DEFAULT_TIMELINE_CATEGORIES;
+  const ordered = sortObservations(input.observations);
+  const batches = batchObservations(ordered);
+  const merged = new Map<string, AnalysisOp>();
+
+  try {
+    for (const batch of batches) {
+      const prompt = buildTimelineAnalysisPrompt({
+        date: input.date,
+        timezone: input.timezone,
+        cards: input.cards,
+        observations: batch,
+        categories,
+        preferences: input.preferences,
+        priorEdits: input.priorEdits,
+      });
+      const raw = await callComplete(complete, prompt, provider, model, timeoutMs, input.signal);
+      const ops = parseOps(raw);
+      if (!ops) return fail("invalid_output", cards, provider, model);
+      for (const op of ops) merged.set(op.cardId, op);
+    }
+  } catch {
+    return fail("provider_failed", cards, provider, model);
+  }
+
+  const applied = applyOps(cards, [...merged.values()], categories, knownEvidenceKeys(cards, ordered));
+  if (!applied) return fail("invalid_output", cards, provider, model);
+
+  log.info(`timeline analysis status=ok provider=${provider} model=${model} promptVersion=${TIMELINE_ANALYSIS_PROMPT_VERSION}`);
+  return {
+    status: "ok",
+    cards: applied,
+    provider,
+    model,
+    promptVersion: TIMELINE_ANALYSIS_PROMPT_VERSION,
+  };
+}

package/src/activity/timeline/index.ts

@@ -6,3 +6,8 @@ export * from "./build.js";
 export * from "./corrections.js";
 export * from "./query.js";
 export * from "./journal-recap.js";
+export * from "./journal-recap-persist.js";
+export * from "./weekly.js";
+export * from "./weekly-persist.js";
+export * from "./analysis.js";
+export * from "./analysis-batch.js";

package/src/activity/timeline/journal-recap-persist.test.ts

@@ -0,0 +1,148 @@
+import assert from "node:assert/strict";
+import { mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import path from "node:path";
+import test from "node:test";
+
+import { journalPath } from "../journal.js";
+import { persistDeterministicJournal } from "./journal-recap-persist.js";
+import { renderDeterministicJournal } from "./journal-recap.js";
+import type { TimelineCard } from "./types.js";
+
+const DATE = "2026-08-17";
+const TZ = "UTC";
+
+function card(
+  overrides: Partial<TimelineCard> & Pick<TimelineCard, "id" | "startUtc" | "endUtc">,
+): TimelineCard {
+  return {
+    kind: "activity",
+    title: "Untitled",
+    summary: "none",
+    categoryId: "development",
+    confidence: 1,
+    dayKey: DATE,
+    timezone: TZ,
+    machine: "ws-a",
+    evidenceIds: [],
+    evidenceRange: null,
+    ...overrides,
+  };
+}
+
+function activityCards(): TimelineCard[] {
+  return [
+    card({
+      id: "act-1",
+      title: "Terminal",
+      startUtc: "2026-08-17T14:00:00.000Z",
+      endUtc: "2026-08-17T15:30:00.000Z",
+    }),
+  ];
+}
+
+function otherCards(): TimelineCard[] {
+  return [
+    card({
+      id: "act-2",
+      title: "Browser",
+      startUtc: "2026-08-17T16:00:00.000Z",
+      endUtc: "2026-08-17T17:00:00.000Z",
+    }),
+  ];
+}
+
+function withMemoryDir(fn: (memoryDir: string) => void): void {
+  const memoryDir = mkdtempSync(path.join(tmpdir(), "remnic-journal-recap-persist-"));
+  try {
+    fn(memoryDir);
+  } finally {
+    rmSync(memoryDir, { recursive: true, force: true });
+  }
+}
+
+test("persist is a hard no-op without force", () => {
+  withMemoryDir((memoryDir) => {
+    const first = persistDeterministicJournal({
+      memoryDir,
+      date: DATE,
+      cards: activityCards(),
+      timezone: TZ,
+    });
+    assert.equal(first.wrote, true);
+    assert.equal(first.path, journalPath(memoryDir, DATE));
+    const original = readFileSync(first.path);
+    assert.equal(
+      original.toString("utf8"),
+      renderDeterministicJournal(activityCards(), { date: DATE, timezone: TZ }),
+    );
+
+    const second = persistDeterministicJournal({
+      memoryDir,
+      date: DATE,
+      cards: otherCards(),
+      timezone: TZ,
+    });
+    assert.equal(second.wrote, false);
+    assert.equal(second.path, first.path);
+    assert.deepEqual(readFileSync(second.path), original);
+  });
+});
+
+test("persist with force overwrites the existing file", () => {
+  withMemoryDir((memoryDir) => {
+    persistDeterministicJournal({
+      memoryDir,
+      date: DATE,
+      cards: activityCards(),
+      timezone: TZ,
+    });
+    const original = readFileSync(journalPath(memoryDir, DATE), "utf8");
+    const forced = persistDeterministicJournal({
+      memoryDir,
+      date: DATE,
+      cards: otherCards(),
+      timezone: TZ,
+      force: true,
+    });
+    assert.equal(forced.wrote, true);
+    const next = readFileSync(forced.path, "utf8");
+    assert.notEqual(next, original);
+    assert.equal(next, renderDeterministicJournal(otherCards(), { date: DATE, timezone: TZ }));
+  });
+});
+
+test("empty day still writes a valid journal file", () => {
+  withMemoryDir((memoryDir) => {
+    const result = persistDeterministicJournal({
+      memoryDir,
+      date: DATE,
+      cards: [],
+      timezone: TZ,
+    });
+    assert.equal(result.wrote, true);
+    const relative = path.relative(memoryDir, result.path).replace(/\\/g, "/");
+    assert.equal(relative, "journal/2026-08-17.md");
+    assert.equal(
+      readFileSync(result.path, "utf8"),
+      renderDeterministicJournal([], { date: DATE, timezone: TZ }),
+    );
+  });
+});
+
+test("existing foreign file without force stays byte-identical", () => {
+  withMemoryDir((memoryDir) => {
+    const filePath = journalPath(memoryDir, DATE);
+    mkdirSync(path.dirname(filePath), { recursive: true });
+    writeFileSync(filePath, "user notes\n");
+    const original = readFileSync(filePath);
+    const result = persistDeterministicJournal({
+      memoryDir,
+      date: DATE,
+      cards: activityCards(),
+      timezone: TZ,
+    });
+    assert.equal(result.wrote, false);
+    assert.deepEqual(readFileSync(filePath), original);
+  });
+});

package/src/activity/timeline/journal-recap-persist.ts

@@ -0,0 +1,37 @@
+/**
+ * Persist a deterministic daily journal recap (issue #2051).
+ *
+ * Writes journal/<YYYY-MM-DD>.md through writeJournalFile. Existing files
+ * stay byte-identical unless force is set. Body is renderDeterministicJournal.
+ */
+import { journalPath, writeJournalFile } from "../journal.js";
+import { renderDeterministicJournal } from "./journal-recap.js";
+import type { TimelineCard } from "./types.js";
+
+export interface PersistDeterministicJournalInput {
+  memoryDir: string;
+  date: string;
+  cards: readonly TimelineCard[];
+  timezone: string;
+  force?: boolean;
+}
+
+export interface PersistDeterministicJournalResult {
+  path: string;
+  wrote: boolean;
+}
+
+export function persistDeterministicJournal(
+  input: PersistDeterministicJournalInput,
+): PersistDeterministicJournalResult {
+  const filePath = journalPath(input.memoryDir, input.date);
+  const wrote = writeJournalFile(
+    filePath,
+    renderDeterministicJournal(input.cards, {
+      date: input.date,
+      timezone: input.timezone,
+    }),
+    input.force === true,
+  );
+  return { path: filePath, wrote };
+}

package/src/activity/timeline/weekly-persist.test.ts

@@ -0,0 +1,121 @@
+import assert from "node:assert/strict";
+import { mkdtempSync, readFileSync, readdirSync, rmSync, statSync } from "node:fs";
+import { tmpdir } from "node:os";
+import path from "node:path";
+import test from "node:test";
+
+import { RECALL_FALLBACK_DIRS } from "../../utils/category-dir.js";
+import type { TimelineCategory } from "./types.js";
+import { persistWeeklySnapshot } from "./weekly-persist.js";
+import { buildWeeklyActivitySummary } from "./weekly.js";
+
+const WEEK_START = "2026-07-13T00:00:00.000Z";
+const WEEK_END = "2026-07-20T00:00:00.000Z";
+const OTHER_WEEK_START = "2026-07-20T00:00:00.000Z";
+const OTHER_WEEK_END = "2026-07-27T00:00:00.000Z";
+
+const CATEGORIES: TimelineCategory[] = [
+  { id: "development", name: "Development", color: "#333333", description: "d", order: 1 },
+];
+
+function summary(weekStartUtc = WEEK_START, weekEndUtc = WEEK_END) {
+  return buildWeeklyActivitySummary([], {
+    timezone: "UTC",
+    weekStartUtc,
+    weekEndUtc,
+    categories: CATEGORIES,
+  });
+}
+
+function withMemoryDir(fn: (memoryDir: string) => void): void {
+  const memoryDir = mkdtempSync(path.join(tmpdir(), "remnic-weekly-persist-"));
+  try {
+    fn(memoryDir);
+  } finally {
+    rmSync(memoryDir, { recursive: true, force: true });
+  }
+}
+
+function assertOutsideRecallRoots(filePath: string, memoryDir: string): void {
+  const relative = path.relative(memoryDir, filePath);
+  assert.ok(relative.startsWith(`activity${path.sep}weekly${path.sep}`));
+  assert.ok(
+    !RECALL_FALLBACK_DIRS.some((dir) => relative === dir || relative.startsWith(`${dir}${path.sep}`)),
+  );
+}
+
+test("same inputs skip rewrite by content hash", () => {
+  withMemoryDir((memoryDir) => {
+    const input = {
+      memoryDir,
+      namespace: "team",
+      summary: summary(),
+      sourceRevision: "rev-1",
+      configHash: "cfg-1",
+    };
+    const first = persistWeeklySnapshot(input);
+    assert.equal(first.written, true);
+    assertOutsideRecallRoots(first.path, memoryDir);
+    const firstBytes = readFileSync(first.path, "utf8");
+    const firstStat = statSync(first.path);
+    const snapshot = JSON.parse(firstBytes) as { contentHash: string };
+    assert.equal(typeof snapshot.contentHash, "string");
+    assert.equal(snapshot.contentHash.length, 64);
+
+    const second = persistWeeklySnapshot(input);
+    assert.equal(second.written, false);
+    assert.equal(second.path, first.path);
+    assert.equal(readFileSync(first.path, "utf8"), firstBytes);
+    assert.equal(statSync(first.path).mtimeMs, firstStat.mtimeMs);
+  });
+});
+
+test("changed sourceRevision or configHash writes a new file and leaves other weeks", () => {
+  withMemoryDir((memoryDir) => {
+    const base = {
+      memoryDir,
+      namespace: "team",
+      summary: summary(),
+      sourceRevision: "rev-1",
+      configHash: "cfg-1",
+    };
+    const original = persistWeeklySnapshot(base);
+    const originalBytes = readFileSync(original.path, "utf8");
+
+    const revised = persistWeeklySnapshot({ ...base, sourceRevision: "rev-2" });
+    assert.notEqual(revised.path, original.path);
+    assert.equal(revised.written, true);
+    assert.equal(readFileSync(original.path, "utf8"), originalBytes);
+
+    const reconfigured = persistWeeklySnapshot({ ...base, configHash: "cfg-2" });
+    assert.notEqual(reconfigured.path, original.path);
+    assert.notEqual(reconfigured.path, revised.path);
+    assert.equal(readFileSync(original.path, "utf8"), originalBytes);
+
+    const otherWeek = persistWeeklySnapshot({
+      ...base,
+      summary: summary(OTHER_WEEK_START, OTHER_WEEK_END),
+    });
+    assert.notEqual(otherWeek.path, original.path);
+    assert.equal(readFileSync(original.path, "utf8"), originalBytes);
+    assert.equal(JSON.parse(readFileSync(otherWeek.path, "utf8")).summary.weekStartUtc, OTHER_WEEK_START);
+  });
+});
+
+test("snapshots are namespace scoped", () => {
+  withMemoryDir((memoryDir) => {
+    const shared = {
+      memoryDir,
+      summary: summary(),
+      sourceRevision: "rev-1",
+      configHash: "cfg-1",
+    };
+    const alpha = persistWeeklySnapshot({ ...shared, namespace: "alpha" });
+    const beta = persistWeeklySnapshot({ ...shared, namespace: "beta" });
+    assert.notEqual(alpha.path, beta.path);
+    assert.equal(JSON.parse(readFileSync(alpha.path, "utf8")).namespace, "alpha");
+    assert.equal(JSON.parse(readFileSync(beta.path, "utf8")).namespace, "beta");
+    assert.equal(readdirSync(path.dirname(alpha.path)).length, 1);
+    assert.equal(readdirSync(path.dirname(beta.path)).length, 1);
+  });
+});