@remnic/core 9.22.0 → 9.22.1

package/src/activity/runner.test.ts

@@ -453,3 +453,40 @@ test("an older-day failure does not starve newer dates in a multi-day window", a
     await rm(memoryDir, { recursive: true, force: true });
   }
 });
+
+test("runActivitySyncOnce reports every local date that gained snapshots (issue #1900 backfill)", async () => {
+  const memoryDir = await mkdtemp(path.join(os.tmpdir(), "remnic-activity-runner-"));
+  const store = ActivityStore.open(memoryDir);
+  try {
+    let calls = 0;
+    const client: ActivitySourceClient = {
+      machineLabel: "workstation-a",
+      async verify() {
+        return { ok: true };
+      },
+      async fetchSnapshots() {
+        calls += 1;
+        // Dates are processed oldest-first: day 1 = 2026-07-21, day 2 = 2026-07-22.
+        const capturedAtUtc = calls === 1 ? "2026-07-21T14:00:00.000Z" : "2026-07-22T14:00:00.000Z";
+        return { snapshots: [snapshot({ capturedAtUtc, contentHash: `hash-${calls}` })], nextCursor: null };
+      },
+    };
+
+    const summary = await runActivitySyncOnce({
+      config: enabledConfig({ syncDays: 2 }),
+      memoryDir,
+      store,
+      now: NOW,
+      createSourceClient: () => client,
+    });
+
+    assert.deepEqual(
+      summary.touchedDates,
+      ["2026-07-21", "2026-07-22"],
+      "both days that inserted snapshots are reported so meetings rebuild the older day too",
+    );
+  } finally {
+    store.close();
+    await rm(memoryDir, { recursive: true, force: true });
+  }
+});

package/src/activity/runner.ts

@@ -60,6 +60,12 @@ export interface ActivitySyncRunSummary {
   reindexErrorCount: number;
   totalInserted: number;
   results: ActivitySourceRunItem[];
+  /**
+   * Local dates (YYYY-MM-DD, in the sync timezone) that gained new snapshots
+   * this run — a backfill/rolling window can touch several. Meetings rebuild
+   * uses this so a change to an OLDER day rebuilds that day, not just today.
+   */
+  touchedDates: string[];
 }
 
 export interface ActivitySyncRunOptions {
@@ -148,6 +154,7 @@ export async function runActivitySyncOnce(options: ActivitySyncRunOptions): Prom
       reindexErrorCount: 0,
       totalInserted: 0,
       results: [],
+      touchedDates: [],
     };
   }
 
@@ -163,6 +170,9 @@ export async function runActivitySyncOnce(options: ActivitySyncRunOptions): Prom
   const ownStore = options.store === undefined;
   const store = options.store ?? ActivityStore.open(options.memoryDir);
   const results: ActivitySourceRunItem[] = [];
+  // Local dates that gained new snapshots this run (any source). Drives the
+  // meetings rebuild fan-out so a backfilled OLDER day is rebuilt too.
+  const touchedDates = new Set<string>();
 
   try {
     for (const sourceConfig of options.config.sources) {
@@ -191,6 +201,7 @@ export async function runActivitySyncOnce(options: ActivitySyncRunOptions): Prom
             });
             item.fetched += result.fetched;
             item.inserted += result.inserted;
+            if (result.inserted > 0) touchedDates.add(date);
             item.duplicates += result.duplicates;
             if (result.digestWritten) item.digestsWritten += 1;
             item.cursor = result.cursor;
@@ -233,5 +244,6 @@ export async function runActivitySyncOnce(options: ActivitySyncRunOptions): Prom
     reindexErrorCount: results.filter((item) => item.reindexError !== undefined).length,
     totalInserted: results.reduce((sum, item) => sum + item.inserted, 0),
     results,
+    touchedDates: [...touchedDates].sort(),
   };
 }

package/src/activity/scheduler.test.ts

@@ -50,6 +50,7 @@ const SUMMARY: ActivitySyncRunSummary = {
   reindexErrorCount: 0,
   totalInserted: 0,
   results: [],
+  touchedDates: [],
 };
 
 function enabledConfig(): ActivityConfig {

package/src/activity/sync-registration.ts

@@ -50,6 +50,13 @@ export interface ActivitySyncRegistrarDeps {
   readonly getQmd: () => ActivityIndexRefresher;
   /** Debounced/singleflighted QMD retry used when a tick's reindex fails. */
   readonly requestReindexRetry: () => void;
+  /**
+   * Tail step (issue #1900): observe a completed sync tick so a dependent
+   * subsystem (meeting building) can rebuild the affected day(s). Fired only
+   * after a durable tick and never after teardown. Optional; omitted by hosts
+   * that do not build meetings.
+   */
+  readonly onSynced?: (summary: ActivitySyncRunSummary) => void;
   /** Scheduler factory (injectable for tests); defaults to the real scheduler. */
   readonly createScheduler?: (options: ActivitySyncSchedulerOptions) => ActivitySyncSchedulerLike;
 }
@@ -111,6 +118,12 @@ export class ActivitySyncRegistrar {
             );
             this.deps.requestReindexRetry();
           }
+          // Tail step: a durable tick that ingested snapshots may have changed a
+          // meeting's screen context; notify the dependent builder. Never after
+          // teardown.
+          if (summary.enabled && summary.totalInserted > 0 && !this.disposed) {
+            this.deps.onSynced?.(summary);
+          }
         },
       });
       this.scheduler = scheduler;

package/src/cli/meetings-commands.ts

@@ -0,0 +1,70 @@
+/**
+ * Meetings command group (issue #1900) — extracted from cli.ts so the surface
+ * file stays under its structural ceiling. Mirrors the
+ * registerResearchStatusCommands / registerCreationLedgerCommands seam: a single
+ * registrar wires the `meetings` command (list / show / build) on the parent
+ * `engram` command.
+ *
+ * Every subcommand forwards to the shared runner in meetings/cli.ts so this host
+ * CLI and the standalone `remnic` CLI never fork behavior or formatting.
+ * Behaviour is identical to the inline registration it replaces — the
+ * cli-command-surface contract tests guard against silent drift.
+ */
+
+import type { Orchestrator } from "../orchestrator.js";
+import type { CliCommand } from "../cli.js";
+import { runMeetingsCliCommand } from "../meetings/cli.js";
+
+export function registerMeetingsCommands(cmd: CliCommand, orchestrator: Orchestrator): void {
+  const meetingsCmd = cmd
+    .command("meetings")
+    .description(
+      "Retrospective meetings: list records, show one, build (detect + fuse + store) a day",
+    );
+  const forwardMeetings = async (argv: string[]): Promise<void> => {
+    const service = await orchestrator.getMeetingsService();
+    const code = await runMeetingsCliCommand(
+      { store: service.store, builder: service.builder, config: service.config },
+      argv,
+      { stdout: process.stdout, stderr: process.stderr },
+    );
+    if (code !== 0) process.exitCode = code;
+  };
+
+  meetingsCmd
+    .command("list")
+    .description("List meeting records (all days, or one day)")
+    .option("--date <date>", "Day to list (YYYY-MM-DD)")
+    .option("--json", "JSON output")
+    .action(async (...args: unknown[]) => {
+      const options = (args[0] ?? {}) as Record<string, unknown>;
+      await forwardMeetings([
+        "list",
+        ...(typeof options.date === "string" && options.date.length > 0
+          ? ["--date", options.date]
+          : []),
+        ...(options.json === true ? ["--json"] : []),
+      ]);
+    });
+
+  meetingsCmd
+    .command("show <meeting-id>")
+    .description("Print a stored meeting record")
+    .action(async (...args: unknown[]) => {
+      await forwardMeetings(["show", String(args[0] ?? "")]);
+    });
+
+  meetingsCmd
+    .command("build")
+    .description("Detect + fuse + store the day's meetings")
+    .option("--date <date>", "Day to build (YYYY-MM-DD)")
+    .option("--json", "JSON output")
+    .action(async (...args: unknown[]) => {
+      const options = (args[0] ?? {}) as Record<string, unknown>;
+      await forwardMeetings([
+        "build",
+        ...(typeof options.date === "string" ? ["--date", options.date] : []),
+        ...(options.json === true ? ["--json"] : []),
+      ]);
+    });
+}

package/src/cli.ts

@@ -13,6 +13,7 @@ import { ThreadingManager } from "./threading.js";
 import { bumpMemoryCorpusVersionForDir } from "./memory-corpus-version.js";
 import { utcDayRange } from "./transcript.js";
 import { runWearablesCliCommand } from "./wearables/cli.js";
+import { registerMeetingsCommands } from "./cli/meetings-commands.js";
 import { registerResearchStatusCommands } from "./cli/research-status-commands.js";
 import { registerCreationLedgerCommands } from "./cli/creation-ledger-commands.js";
 import type {
@@ -5197,11 +5198,11 @@ export function registerCli(
             "Wearable transcript sources (Limitless / Bee / Omi): sync, transcripts, search, speakers, corrections",
           );
         const forwardWearables = async (argv: string[]): Promise<void> => {
-          const code = await runWearablesCliCommand(
-            orchestrator.getWearablesService(),
-            argv,
-            { stdout: process.stdout, stderr: process.stderr },
-          );
+          const io = { stdout: process.stdout, stderr: process.stderr };
+          const code = await runWearablesCliCommand(orchestrator.getWearablesService(), argv, io);
+          if (argv[0] === "sync" && code === 0 && orchestrator.config.meetings.enabled) { // drain debounced build
+            await (await orchestrator.getMeetingsService()).flushBuilds();
+          }
           if (code !== 0) process.exitCode = code;
         };
         const stringOpt = (options: Record<string, unknown>, key: string, flag: string): string[] =>
@@ -6230,7 +6231,7 @@ export function registerCli(
 
       registerResearchStatusCommands(cmd, orchestrator);
 
-      registerCreationLedgerCommands(cmd, orchestrator);
+      registerCreationLedgerCommands(cmd, orchestrator); registerMeetingsCommands(cmd, orchestrator);
 
       cmd
         .command("trust-zone-promote")

package/src/extraction-error-classification.ts

@@ -0,0 +1,52 @@
+import type { ExtractionFailureClass } from "./types.js";
+import { isTransientHttpError } from "./connectors/live/transient-errors.js";
+
+/**
+ * Read a numeric HTTP status off a thrown error without re-implementing the
+ * shared transient classifier. Uses `in`/typeof narrowing (no inline casts).
+ */
+function httpStatusFromError(err: unknown): number | undefined {
+  if (!err || typeof err !== "object") return undefined;
+  if ("status" in err && typeof err.status === "number" && Number.isFinite(err.status)) return err.status;
+  if ("statusCode" in err && typeof err.statusCode === "number" && Number.isFinite(err.statusCode)) {
+    return err.statusCode;
+  }
+  return undefined;
+}
+
+/**
+ * Map a thrown extraction error to a coarse failure class for the retry/breaker
+ * layer. 401/403 → `auth_config` (misconfigured provider, open the breaker);
+ * transient (429/5xx/network, per the shared `isTransientHttpError`) → back off;
+ * anything else defaults to `provider_retryable` (fail-open toward retry, but
+ * capped by the per-fingerprint attempt budget). Never throws.
+ */
+export function classifyExtractionThrownError(err: unknown): ExtractionFailureClass {
+  try {
+    const status = httpStatusFromError(err);
+    if (status === 401 || status === 403) return "auth_config";
+    if (isTransientHttpError(err)) return "provider_retryable";
+    return "provider_retryable";
+  } catch {
+    return "provider_retryable";
+  }
+}
+
+/**
+ * Map the gateway fallback's discriminated parse-failure reason to a coarse
+ * extraction failure class. "no models configured" is an auth/config problem;
+ * an HTTP/transport failure backs off; an unparseable-but-present response is a
+ * genuine empty-parse.
+ */
+export function classifyFallbackParseFailure(
+  reason: "no_models" | "empty" | "http_error",
+): ExtractionFailureClass {
+  switch (reason) {
+    case "no_models":
+      return "auth_config";
+    case "http_error":
+      return "provider_retryable";
+    case "empty":
+      return "parse_empty";
+  }
+}

package/src/extraction.ts

@@ -40,7 +40,8 @@ import { normalizeProcedureSteps } from "./procedural/procedure-types.js";
 import { normalizeReasoningTrace } from "./reasoning-trace-types.js";
 import { looksLikeMechanicalTelemetryTranscript } from "./telemetry-transcript.js";
 import { buildFactProvenance, type ProvenanceTurnInput } from "./provenance.js";
-import { isTransientHttpError } from "./connectors/live/transient-errors.js";
+import { classifyExtractionThrownError, classifyFallbackParseFailure } from "./extraction-error-classification.js";
+export { classifyExtractionThrownError, classifyFallbackParseFailure } from "./extraction-error-classification.js";
 import { resolvePipelineProcessingCapabilities } from "./capabilities.js";
 import { resolveMemoryLifecycleCapabilities,
   resolveLocalLlmCapabilities,resolveRecallAuxiliaryCapabilities } from "./capabilities.js";
@@ -50,55 +51,6 @@ type ExtractedFactResult = ExtractionResult["facts"][number];
 type ExtractedEntityResult = ExtractionResult["entities"][number];
 type ExtractedRelationshipResult = NonNullable<ExtractionResult["relationships"]>[number];
 
-/**
- * Read a numeric HTTP status off a thrown error without re-implementing the
- * shared transient classifier. Uses `in`/typeof narrowing (no inline casts).
- */
-function httpStatusFromError(err: unknown): number | undefined {
-  if (!err || typeof err !== "object") return undefined;
-  if ("status" in err && typeof err.status === "number" && Number.isFinite(err.status)) return err.status;
-  if ("statusCode" in err && typeof err.statusCode === "number" && Number.isFinite(err.statusCode)) {
-    return err.statusCode;
-  }
-  return undefined;
-}
-
-/**
- * Map a thrown extraction error to a coarse failure class for the retry/breaker
- * layer. 401/403 → `auth_config` (misconfigured provider, open the breaker);
- * transient (429/5xx/network, per the shared `isTransientHttpError`) → back off;
- * anything else defaults to `provider_retryable` (fail-open toward retry, but
- * capped by the per-fingerprint attempt budget). Never throws.
- */
-export function classifyExtractionThrownError(err: unknown): ExtractionFailureClass {
-  try {
-    const status = httpStatusFromError(err);
-    if (status === 401 || status === 403) return "auth_config";
-    if (isTransientHttpError(err)) return "provider_retryable";
-    return "provider_retryable";
-  } catch {
-    return "provider_retryable";
-  }
-}
-
-/**
- * Map the gateway fallback's discriminated parse-failure reason to a coarse
- * extraction failure class. "no models configured" is an auth/config problem;
- * an HTTP/transport failure backs off; an unparseable-but-present response is a
- * genuine empty-parse.
- */
-export function classifyFallbackParseFailure(
-  reason: "no_models" | "empty" | "http_error",
-): ExtractionFailureClass {
-  switch (reason) {
-    case "no_models":
-      return "auth_config";
-    case "http_error":
-      return "provider_retryable";
-    case "empty":
-      return "parse_empty";
-  }
-}
 const PROACTIVE_MIN_CONFIDENCE = 0.8;
 const CONSOLIDATION_RESPONSE_SCHEMA = `{
   "items": [

package/src/mcp-read-only-tools.ts

@@ -21,6 +21,8 @@ export const MCP_READ_ONLY_TOOL_SUFFIXES: Readonly<Record<string, true>> = {
   transcript_day: true,
   transcript_search: true,
   transcript_memories: true,
+  meetings_list: true,
+  meetings_get: true,
   action_confidence: true,
   capsule_list: true,
   procedural_stats: true,