@mycadet/assistant 0.12.9 → 0.12.11

package/src/persistence/slow-sync-log.ts

@@ -44,12 +44,6 @@ export const SLOW_SYNC_THRESHOLD_MS = ((): number => {
   return Number.isFinite(raw) && raw > 0 ? raw : 1000;
 })();
 
-/**
- * `check_name` for slow-sync telemetry events. Stable so downstream
- * grouping stays consistent; keep it in sync with any admin query.
- */
-export const SLOW_SYNC_CHECK_NAME = "slow_sync_operation";
-
 /** One recorded section entry: a static label plus monotonic start/end stamps. */
 export interface SectionMark {
   label: string;
@@ -157,23 +151,6 @@ export function reportSlowSync(
     { label, elapsedMs, ...detail },
     "Slow synchronous DB operation blocked the event loop",
   );
-  // Record telemetry via a lazy dynamic import. A static import would pull the
-  // telemetry → consent-cache → config/loader chain into the module graph of
-  // every raw-query / conversation-load caller, where a test's `mock.module`
-  // of the loader leaks across the shared test process and breaks unrelated
-  // files at eval time. Loading it only on the rare slow path keeps that chain
-  // off the hot path's static graph; best-effort, so a failure is swallowed.
-  void import("../telemetry/watchdog-events-store.js")
-    .then(({ recordWatchdogEvent }) => {
-      recordWatchdogEvent({
-        checkName: SLOW_SYNC_CHECK_NAME,
-        value: elapsedMs,
-        detail: { label, ...detail },
-      });
-    })
-    .catch(() => {
-      // Telemetry is best-effort — never let it escape the timed section.
-    });
 }
 
 /**

package/src/persistence/steps.ts

@@ -497,6 +497,7 @@ import { migrateNormalizeOpencodeHostConnections } from "./migrations/383-normal
 import { migrateAddDocumentRevision } from "./migrations/384-add-document-revision.js";
 import { migrateCreateDocumentRevisions } from "./migrations/385-create-document-revisions.js";
 import { migrateConversationsLastReopenedAt } from "./migrations/386-conversations-last-reopened-at.js";
+import { migrateDropTelemetryTables } from "./migrations/387-drop-telemetry-tables.js";
 import type { MigrationStep } from "./migrations/run-migrations.js";
 
 export const migrationSteps: MigrationStep[] = [
@@ -1667,4 +1668,5 @@ export const migrationSteps: MigrationStep[] = [
   migrateAddDocumentRevision,
   migrateCreateDocumentRevisions,
   migrateConversationsLastReopenedAt,
+  migrateDropTelemetryTables,
 ];

package/src/{telemetry → persistence}/tool-audit.ts

@@ -1,36 +1,20 @@
 /**
- * Direct terminals for tool-execution auditing and telemetry.
+ * Direct terminals for tool-execution auditing.
  *
  * The executor and its permission/approval collaborators call these functions
  * inline as each tool call reaches a terminal outcome (executed / error /
- * denied / prompted). Each function owns exactly one side effect:
+ * denied). Each function owns exactly one side effect:
  *
  *   - `recordToolExecuted` / `recordToolError` / `recordToolDenied` write the
  *     always-on `tool_invocations` audit row (redacting secrets from the
  *     stored input/result).
- *   - `recordToolPermissionPrompted` / `recordToolPermissionDecided` write the
- *     consent-gated `permission_prompt` and `permission_decided`
- *     lifecycle-telemetry rows. Both are called from the one site that owns an
- *     interactive prompt (`tools/permission-checker.ts`), so every prompt has
- *     exactly one decision and the two series reconcile.
  *   - `logToolFailure` emits the operator-facing failure log.
  *
- * The telemetry-only columns (payload sizes + model attribution) are the
- * single write-time privacy gate for `tool_executed` telemetry: on a confirmed
- * `share_analytics` opt-out they persist as NULL, which the projection's
- * `arg_bytes IS NOT NULL` filter excludes permanently. An unknown consent
- * state (cold cache) populates the columns — NULLing them would destroy the
- * data irreversibly, and consent is enforced again at flush time and platform
- * ingest, so opted-out rows never leave the device. The audit fields
- * themselves (tool name, decision, redacted input/result previews, duration)
- * are unaffected — `tool_invocations` is a local always-on audit log.
+ * The payload-size + model-attribution columns are part of the local audit
+ * row — `tool_invocations` is a local always-on audit log and nothing in it
+ * ever leaves the device.
  */
 
-import { getRawShareAnalytics } from "../cloud/consent-cache.js";
-import {
-  type LifecycleEventAttributes,
-  recordLifecycleEvent,
-} from "../persistence/lifecycle-events-store.js";
 import { redactJsonStringLeaves } from "../security/redact-json.js";
 import { redactSensitiveFields } from "../security/redaction.js";
 import { redactSecrets } from "../security/secret-scanner.js";
@@ -154,87 +138,6 @@ export function recordToolDenied(entry: DeniedAuditEntry): void {
   });
 }
 
-/**
- * How an interactive permission prompt ended.
- *
- * `allow` / `deny` are answers a human gave. `abandoned` covers every way a
- * prompt stops without one: it timed out, a new user message superseded it,
- * the turn was cancelled, or the prompter was disposed (client disconnect /
- * conversation teardown). The daemon resolves all of those to a deny for the
- * agent loop, so without this bucket they are indistinguishable from a human
- * saying no.
- */
-export type PermissionPromptOutcome = "allow" | "deny" | "abandoned";
-
-/**
- * Grouping dimensions recorded on both halves of a permission-prompt pair.
- * The tool and its risk are always known at the prompt site, so they tighten
- * to required; the rest are the lifecycle attributes as declared. Metadata
- * only: never a file path, a command string, or any tool input.
- */
-export interface PermissionPromptTelemetry extends LifecycleEventAttributes {
-  toolName: string;
-  riskLevel: string;
-}
-
-/**
- * Server bounds on the lifecycle `event_name` and `tool_name`. A field over
- * its bound fails validation, and a failed event is acked and discarded rather
- * than retried, so both are clamped here. `event_name` has one variable-length
- * part (the tool name, which can be a long `mcp__server__tool` id) and is much
- * the tighter of the two, which is why `tool_name` carries the full-length
- * value consumers join a prompt to its decision on.
- */
-const LIFECYCLE_EVENT_NAME_LIMIT = 64;
-const LIFECYCLE_TOOL_NAME_LIMIT = 255;
-
-function permissionEventName(
-  prefix: string,
-  toolName: string,
-  suffix: string,
-): string {
-  const budget = LIFECYCLE_EVENT_NAME_LIMIT - prefix.length - suffix.length;
-  return `${prefix}${toolName.slice(0, Math.max(budget, 0))}${suffix}`;
-}
-
-function recordPermissionLifecycleEvent(
-  eventName: string,
-  entry: PermissionPromptTelemetry,
-  failureMessage: string,
-): void {
-  try {
-    recordLifecycleEvent(eventName, {
-      ...entry,
-      toolName: entry.toolName.slice(0, LIFECYCLE_TOOL_NAME_LIMIT),
-    });
-  } catch (err) {
-    log.warn({ err, eventName, toolName: entry.toolName }, failureMessage);
-  }
-}
-
-/** Record the consent-gated telemetry row for an interactive permission prompt. */
-export function recordToolPermissionPrompted(
-  entry: PermissionPromptTelemetry,
-): void {
-  recordPermissionLifecycleEvent(
-    permissionEventName("permission_prompt:", entry.toolName, ""),
-    entry,
-    "Failed to record permission prompt telemetry",
-  );
-}
-
-/** Record the consent-gated telemetry row for how that prompt ended. */
-export function recordToolPermissionDecided(
-  entry: PermissionPromptTelemetry,
-  outcome: PermissionPromptOutcome,
-): void {
-  recordPermissionLifecycleEvent(
-    permissionEventName("permission_decided:", entry.toolName, `:${outcome}`),
-    entry,
-    "Failed to record permission decision telemetry",
-  );
-}
-
 /** Operator-facing failure log: warn for expected failures, error otherwise. */
 function logToolFailure(entry: ErrorAuditEntry): void {
   const meta = {
@@ -303,32 +206,14 @@ function redactToolInput(
 type TelemetryColumns = Pick<ToolInvocationRecord, "argBytes" | "resultBytes"> &
   UsageAttributionColumns;
 
-const NULL_TELEMETRY_COLUMNS: TelemetryColumns = {
-  argBytes: null,
-  resultBytes: null,
-  provider: null,
-  model: null,
-  inferenceProfile: null,
-  inferenceProfileSource: null,
-};
-
 /**
- * Telemetry-only columns (payload sizes + model attribution). NULLed only on a
- * confirmed `share_analytics` opt-out — the projection's `arg_bytes IS NOT
- * NULL` filter excludes NULL rows permanently (the same mechanism that
- * excludes legacy pre-migration rows, see tool-executed-events-store.ts), so
- * an unknown consent state (cold cache) must populate the columns rather than
- * destroy them. Consent is enforced again at flush time and platform ingest,
- * so opted-out rows never ship.
+ * Payload sizes + model attribution, part of the local audit row.
  */
 function telemetryColumns(
   attribution: UsageAttributionSnapshot | null,
   rawInput: string,
   resultBytes: number,
 ): TelemetryColumns {
-  if (getRawShareAnalytics() === false) {
-    return NULL_TELEMETRY_COLUMNS;
-  }
   return {
     argBytes: Buffer.byteLength(rawInput, "utf8"),
     resultBytes,

package/src/plugin-api/index.ts

@@ -294,7 +294,7 @@ export {
 // Absolute path to the active workspace directory. A plugin that reads or
 // writes files under the workspace (e.g. its own `plugins/<name>/data/`
 // directory) resolves them against this instead of hardcoding a base path.
-export { getWorkspaceDir } from "../util/platform.js";
+export { getWorkspaceDir } from "../util/layout.js";
 // `String.prototype.slice` that never cuts a UTF-16 surrogate pair in half.
 // Any text a plugin truncates by character budget and hands to a model must
 // go through this: an orphaned half is invalid UTF-16 that strict provider

package/src/plugin-api/plugin-name-env.ts

@@ -12,7 +12,7 @@ import { isAbsolute, relative, resolve, sep } from "node:path";
 import { fileURLToPath } from "node:url";
 
 import { getCurrentPluginName } from "../plugins/plugin-execution-context.js";
-import { getWorkspacePluginsDir } from "../util/platform.js";
+import { getWorkspacePluginsDir } from "../util/layout.js";
 
 export const PLUGIN_NAME_ENV = "CADET_PLUGIN_NAME";
 

package/src/plugins/__tests__/installed-plugin-dirs.test.ts

@@ -15,7 +15,7 @@ import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { afterAll, beforeEach, describe, expect, test } from "bun:test";
 
-import { getWorkspacePluginsDir } from "../../util/platform.js";
+import { getWorkspacePluginsDir } from "../../util/layout.js";
 import { listInstalledPluginDirs } from "../installed-plugin-dirs.js";
 
 const pluginsDir = getWorkspacePluginsDir();

package/src/plugins/collect-source-versions.ts

@@ -15,7 +15,7 @@
 import { existsSync } from "node:fs";
 import { join } from "node:path";
 
-import { getWorkspaceHooksDir } from "../util/platform.js";
+import { getWorkspaceHooksDir } from "../util/layout.js";
 import { listInstalledPluginDirs } from "./installed-plugin-dirs.js";
 import { snapshotPluginSource } from "./source-fingerprint.js";
 import type { PluginSourceVersion } from "./source-versions.js";

package/src/plugins/defaults/channel/package.json

@@ -9,6 +9,6 @@
     "node": ">=20.12.0"
   },
   "peerDependencies": {
-    "@mycadet/plugin-api": "^0.8.0"
+    "@mycadet/plugin-api": "^0.12.7"
   }
 }

package/src/plugins/defaults/cloud-hosted/package.json

@@ -9,6 +9,6 @@
     "node": ">=20.12.0"
   },
   "peerDependencies": {
-    "@mycadet/plugin-api": "^0.10.9"
+    "@mycadet/plugin-api": "^0.12.7"
   }
 }

package/src/plugins/defaults/compaction/package.json

@@ -9,6 +9,6 @@
     "node": ">=20.12.0"
   },
   "peerDependencies": {
-    "@mycadet/plugin-api": "^0.8.0"
+    "@mycadet/plugin-api": "^0.12.7"
   }
 }

package/src/plugins/defaults/documents/package.json

@@ -9,6 +9,6 @@
     "node": ">=20.12.0"
   },
   "peerDependencies": {
-    "@mycadet/plugin-api": "^0.8.0"
+    "@mycadet/plugin-api": "^0.12.7"
   }
 }

package/src/plugins/defaults/empty-response/package.json

@@ -9,6 +9,6 @@
     "node": ">=20.12.0"
   },
   "peerDependencies": {
-    "@mycadet/plugin-api": "^0.8.0"
+    "@mycadet/plugin-api": "^0.12.7"
   }
 }

package/src/plugins/defaults/exploration-drift/package.json

@@ -10,6 +10,6 @@
     "node": ">=20.12.0"
   },
   "peerDependencies": {
-    "@mycadet/plugin-api": "^0.8.0"
+    "@mycadet/plugin-api": "^0.12.7"
   }
 }

package/src/plugins/defaults/image-fallback/package.json

@@ -9,6 +9,6 @@
     "node": ">=20.12.0"
   },
   "peerDependencies": {
-    "@mycadet/plugin-api": "^0.8.0"
+    "@mycadet/plugin-api": "^0.12.7"
   }
 }

package/src/plugins/defaults/image-recovery/package.json

@@ -9,6 +9,6 @@
     "node": ">=20.12.0"
   },
   "peerDependencies": {
-    "@mycadet/plugin-api": "^0.8.0"
+    "@mycadet/plugin-api": "^0.12.7"
   }
 }

package/src/plugins/defaults/max-tokens-continue/package.json

@@ -9,6 +9,6 @@
     "node": ">=20.12.0"
   },
   "peerDependencies": {
-    "@mycadet/plugin-api": ">=0.8.0"
+    "@mycadet/plugin-api": "^0.12.7"
   }
 }

package/src/plugins/defaults/memory/__tests__/db-async-query.test.ts

@@ -34,7 +34,7 @@ const {
   parseChangesFromStdout,
 } = await import("../../../../persistence/db-async-query.js");
 const { findSqlite3 } = await import("../../../../util/sqlite3-runtime.js");
-const { getDbPath } = await import("../../../../util/platform.js");
+const { getDbPath } = await import("../../../../util/layout.js");
 
 await initializeDb();
 

package/src/plugins/defaults/memory/__tests__/db-maintenance.test.ts

@@ -26,7 +26,7 @@ const { maybeRunDbMaintenance, maybeRunPassiveWalCheckpoint } =
   await import("../../../../persistence/db-maintenance.js");
 const { getLastInteractiveUserMessageTimestamp } =
   await import("../../../../persistence/conversation-crud.js");
-const { getDbPath } = await import("../../../../util/platform.js");
+const { getDbPath } = await import("../../../../util/layout.js");
 
 await initializeDb();
 

package/src/plugins/defaults/memory/__tests__/memory-retrospective-job.test.ts

@@ -132,21 +132,6 @@ mock.module("../../../../persistence/conversation-tool-surface.js", () => ({
   },
 }));
 
-const watchdogEvents: Array<{
-  checkName: string;
-  value?: number | null;
-  detail?: Record<string, unknown> | null;
-}> = [];
-mock.module("../../../../telemetry/watchdog-events-store.js", () => ({
-  recordWatchdogEvent: (record: {
-    checkName: string;
-    value?: number | null;
-    detail?: Record<string, unknown> | null;
-  }) => {
-    watchdogEvents.push(record);
-  },
-}));
-
 mock.module("../../../../daemon/conversation-registry.js", () => ({
   findConversation: (id: string | undefined) => {
     if (!id) {
@@ -494,7 +479,6 @@ function priorRetroMessage(rememberContents: string[]) {
 
 describe("memoryRetrospectiveJob", () => {
   beforeEach(() => {
-    watchdogEvents.length = 0;
     mockState = null;
     stateUpserts = [];
     lastRunAtBumps = [];
@@ -604,16 +588,6 @@ describe("memoryRetrospectiveJob", () => {
     expect(lastRunAtBumps).toHaveLength(0);
     expect(wakeCalls).toHaveLength(0);
     expect(forkCalls).toHaveLength(0);
-    // Every job run emits exactly one outcome counter for the health chart.
-    expect(
-      watchdogEvents.filter((e) => e.checkName === "memory_retrospective_run"),
-    ).toEqual([
-      {
-        checkName: "memory_retrospective_run",
-        value: 1,
-        detail: { outcome: "no_new_messages" },
-      },
-    ]);
   });
 
   test("assistant-only slice with the user-activity gate on: completed as a no-op, both pointers untouched", async () => {
@@ -642,15 +616,6 @@ describe("memoryRetrospectiveJob", () => {
     expect(lastRunAtBumps).toHaveLength(0);
     expect(wakeCalls).toHaveLength(0);
     expect(forkCalls).toHaveLength(0);
-    expect(
-      watchdogEvents.filter((e) => e.checkName === "memory_retrospective_run"),
-    ).toEqual([
-      {
-        checkName: "memory_retrospective_run",
-        value: 1,
-        detail: { outcome: "no_user_activity" },
-      },
-    ]);
   });
 
   test("a user text row in the slice passes the user-activity gate", async () => {
@@ -828,17 +793,6 @@ describe("memoryRetrospectiveJob", () => {
     const outcome = await memoryRetrospectiveJob(makeJob(), stubConfig);
 
     expect(outcome.kind).toBe("wake_failed");
-    // The outcome counter carries the wake-failure reason so a fleet-wide
-    // spike is attributable (e.g. provider outage vs busy conversations).
-    expect(
-      watchdogEvents.filter((e) => e.checkName === "memory_retrospective_run"),
-    ).toEqual([
-      {
-        checkName: "memory_retrospective_run",
-        value: 1,
-        detail: { outcome: "wake_failed", reason: "timeout" },
-      },
-    ]);
     if (outcome.kind === "wake_failed") {
       expect(outcome.reason).toBe("timeout");
     }
@@ -869,20 +823,6 @@ describe("memoryRetrospectiveJob", () => {
     // Retryable: cooldown bump only, orphan fork cleaned up.
     expect(lastRunAtBumps).toHaveLength(1);
     expect(deletedConversationIds).toEqual(["fork-conv-1"]);
-    expect(
-      watchdogEvents.filter((e) => e.checkName === "memory_retrospective_run"),
-    ).toEqual([
-      {
-        checkName: "memory_retrospective_run",
-        value: 1,
-        detail: {
-          outcome: "no_usable_output",
-          // The failure reason names what was missing, so an operator reading
-          // the job row can tell a lost write from an unfinished review.
-          reason: "run committed neither a memory write nor a concluding reply",
-        },
-      },
-    ]);
   });
 
   test("a run whose reply reads as a conclusion but that the provider cut short: no_usable_output, prior retrospective preserved", async () => {
@@ -1118,17 +1058,6 @@ describe("memoryRetrospectiveJob", () => {
       expect(stateUpserts).toHaveLength(1);
       expect(stateUpserts[0]!.lastProcessedMessageId).toBe("m3");
       expect(stateUpserts[0]!.rememberedLog).toEqual([]);
-      expect(
-        watchdogEvents.filter(
-          (e) => e.checkName === "memory_retrospective_run",
-        ),
-      ).toEqual([
-        {
-          checkName: "memory_retrospective_run",
-          value: 1,
-          detail: { outcome: "invoked", noFindings: true },
-        },
-      ]);
     },
   );
 
@@ -1309,17 +1238,6 @@ describe("memoryRetrospectiveJob", () => {
     expect(stateUpserts).toHaveLength(0);
     expect(lastRunAtBumps).toHaveLength(1);
     expect(deletedConversationIds).toEqual(["fork-conv-1"]);
-    // A thrown run still records an outcome counter (as "error") before the
-    // rethrow — exception-flavored outages must show in the health metric.
-    expect(
-      watchdogEvents.filter((e) => e.checkName === "memory_retrospective_run"),
-    ).toEqual([
-      {
-        checkName: "memory_retrospective_run",
-        value: 1,
-        detail: { outcome: "error", reason: "LLM provider 503" },
-      },
-    ]);
   });
 
   test("missing conversationId payload: no_new_messages, no side effects", async () => {

package/src/plugins/defaults/memory/__tests__/memory-retrospective-skill-card.test.ts

@@ -35,21 +35,6 @@ type ConversationStub = {
 } | null;
 let conversationOverrides: Record<string, ConversationStub> = {};
 
-const watchdogEvents: Array<{
-  checkName: string;
-  value?: number | null;
-  detail?: Record<string, unknown> | null;
-}> = [];
-mock.module("../../../../telemetry/watchdog-events-store.js", () => ({
-  recordWatchdogEvent: (record: {
-    checkName: string;
-    value?: number | null;
-    detail?: Record<string, unknown> | null;
-  }) => {
-    watchdogEvents.push(record);
-  },
-}));
-
 mock.module("../../../../persistence/conversation-crud.js", () => ({
   getMessagesAfter: (_id: string, _afterId: string | null) => [],
   getMessages: (_id: string) => [],
@@ -171,7 +156,6 @@ function makeInsertJob(payload: Record<string, unknown>): MemoryJob {
 
 describe("memory-retrospective skill card", () => {
   beforeEach(() => {
-    watchdogEvents.length = 0;
     addMessageCalls = [];
     addMessageThrowsFor = null;
     addMessageDeduplicatesFor = null;
@@ -334,31 +318,6 @@ describe("memory-retrospective skill card", () => {
     expect(publishedConversationIds).toHaveLength(0);
   });
 
-  test("a delivered card emits the skill_card_delivered counter; a dedup does not", async () => {
-    await insertSkillCardMessage("conv-source", "conv-run-telemetry", [
-      { skillId: "s-1", name: "Skill One", description: "d1" },
-      { skillId: "s-2", name: "Skill Two", description: "d2" },
-    ]);
-    expect(
-      watchdogEvents.filter((e) => e.checkName === "skill_card_delivered"),
-    ).toEqual([
-      {
-        checkName: "skill_card_delivered",
-        value: 2,
-        detail: { skill_count: 2 },
-      },
-    ]);
-
-    // Retried delivery for the same run dedups on clientMessageId — the
-    // counter must not fire again.
-    await insertSkillCardMessage("conv-source", "conv-run-telemetry", [
-      { skillId: "s-1", name: "Skill One", description: "d1" },
-    ]);
-    expect(
-      watchdogEvents.filter((e) => e.checkName === "skill_card_delivered"),
-    ).toHaveLength(1);
-  });
-
   test("insert is best-effort: a persistence failure never throws", async () => {
     addMessageThrowsFor = "src-conv-9";
 

package/src/plugins/defaults/memory/__tests__/memory-tier-boundary-guard.test.ts

@@ -193,7 +193,6 @@ interface TierKeyExemption {
  *   home for raw tier-predicate reads, and it compares v2 against v3.
  * - `config/loader.ts` (v3) — seeds/normalizes the persisted `memory.v3`
  *   shape.
- * - `telemetry/config-setting-snapshot.ts` (both) — reports both tier keys
  *   as-is.
  * - the named migrations: append-only history that rewrites or reads the
  *   historical tier keys it was written for. Listed per file, not per
@@ -240,7 +239,6 @@ const TIER_KEY_READ_ALLOWLIST: readonly TierKeyExemption[] = [
   },
   { path: "src/plugins/defaults/memory/v2/", keys: ["v2"] },
   { path: "src/plugins/defaults/memory/v3/", keys: ["v3"] },
-  { path: "src/telemetry/config-setting-snapshot.ts", keys: ["v2", "v3"] },
   {
     path: "src/workspace/migrations/085-memory-v2-bm25-b-reembed-disabled-v2-pages.ts",
     keys: ["v2"],

package/src/plugins/defaults/memory/memory-retrospective-job.ts

@@ -80,7 +80,6 @@ import {
 import { resolveUserSlug } from "../../../prompts/persona-resolver.js";
 import type { SystemPromptPersonaOverride } from "../../../prompts/system-prompt.js";
 import { wakeAgentForOpportunity } from "../../../runtime/agent-wake.js";
-import { recordWatchdogEvent } from "../../../telemetry/watchdog-events-store.js";
 import { findMostRecentRetrospectiveFor } from "./find-most-recent-retrospective-for.js";
 import { getLogger } from "./logging.js";
 import {
@@ -134,9 +133,6 @@ const FOLLOW_UP_JOB_TYPES: readonly MemoryJobType[] = [] as const;
  */
 export const STALE_SOURCE_PROCESSING_OVERRIDE_MS = 6 * 60 * 60 * 1000;
 
-/** Watchdog check_name for the per-run retrospective outcome counter. */
-const MEMORY_RETROSPECTIVE_RUN_CHECK_NAME = "memory_retrospective_run";
-
 /**
  * The agent-loop exit that means the MODEL ended the run: it answered without
  * asking for another tool. Under every other exit something ended the run for
@@ -191,55 +187,10 @@ export async function memoryRetrospectiveJob(
     return { kind: "disabled" };
   }
 
-  // Central health counter (admin analytics groups on the watchdog
-  // check_name): one event per run with its outcome kind. A run that
-  // throws records outcome "error" before the exception continues to the
-  // jobs worker's retry machinery, so a fleet-wide spike in
-  // `wake_failed`/`error` (e.g. a provider outage on the retrospective's
-  // resolved model) is visible without log access. The emitter itself
-  // never throws — the run's outcome must reach the jobs worker
-  // regardless.
-  const emitRunOutcome = (
-    outcome: string,
-    detail?: { reason?: string; noFindings?: boolean },
-  ): void => {
-    try {
-      recordWatchdogEvent({
-        checkName: MEMORY_RETROSPECTIVE_RUN_CHECK_NAME,
-        value: 1,
-        detail: {
-          outcome,
-          ...(detail?.reason ? { reason: detail.reason.slice(0, 200) } : {}),
-          ...(detail?.noFindings !== undefined
-            ? { noFindings: detail.noFindings }
-            : {}),
-        },
-      });
-    } catch {
-      // recordWatchdogEvent already no-ops on opt-out and a missing
-      // telemetry DB; anything past that is not worth surfacing here.
-    }
-  };
-
-  let outcome: MemoryRetrospectiveOutcome;
-  try {
-    outcome = await runForkBasedRetrospective(sourceConversationId, config, {
-      enforceSweepLookback: true,
-      enforceUserActivityGate: true,
-    });
-  } catch (err) {
-    emitRunOutcome("error", {
-      reason: err instanceof Error ? err.message : String(err),
-    });
-    throw err;
-  }
-  emitRunOutcome(outcome.kind, {
-    ...(outcome.kind === "wake_failed" || outcome.kind === "no_usable_output"
-      ? { reason: outcome.reason }
-      : {}),
-    ...(outcome.kind === "invoked" ? { noFindings: outcome.noFindings } : {}),
+  return await runForkBasedRetrospective(sourceConversationId, config, {
+    enforceSweepLookback: true,
+    enforceUserActivityGate: true,
   });
-  return outcome;
 }
 
 // ---------------------------------------------------------------------------