claude-code-session-manager 0.83.0 → 0.84.0

package/src/main/lib/telemetryClient.cjs

@@ -83,6 +83,9 @@ const S = {
   errSeen: new Map(),
   dailyTimer: null,
   flushChain: Promise.resolve(),
+  lastError: null,
+  lastFlushAt: null,
+  lastFlushReason: null,
 };
 
 function queuePath() {
@@ -493,16 +496,22 @@ async function sendBatch(channel, wireBatch) {
   }
 }
 
+function describeFailureStatus(status) {
+  return status === 0 ? 'network error' : `HTTP ${status}`;
+}
+
 function applyFailureBackoff(status) {
   if (status === 429 || status === 0 || status >= 500) {
     S.consecutiveFailures += 1;
     const ms = Math.min(BACKOFF_BASE_MS * (2 ** (S.consecutiveFailures - 1)), BACKOFF_MAX_MS);
     S.backoffUntil = Date.now() + ms;
+    S.lastError = { status, message: describeFailureStatus(status), at: Date.now() };
     return;
   }
   if (status >= 400 && status < 500) {
     // Malformed-contract circuit breaker (includes 401) — not retried.
     S.disabledForProcess = true;
+    S.lastError = { status, message: describeFailureStatus(status), at: Date.now() };
   }
 }
 
@@ -543,6 +552,7 @@ async function flushImpl(reason) {
         if (res.ok) {
           S.consecutiveFailures = 0;
           S.backoffUntil = 0;
+          S.lastError = null;
           for (const r of batch) {
             removeFromQueue(r.recordId);
             addToSent(r.recordId);
@@ -564,6 +574,8 @@ async function flushImpl(reason) {
       S.settings = await telemetrySettings.save({ ...S.settings, lastDailyFlushAt: new Date().toISOString() });
     }
   } catch { /* fail-inert */ }
+  S.lastFlushAt = Date.now();
+  S.lastFlushReason = safeReason;
   return result;
 }
 
@@ -584,6 +596,9 @@ function status() {
     consecutiveFailures: S.consecutiveFailures,
     backoffUntil: S.backoffUntil,
     profileBuildCount: S.profileBuildCount,
+    lastError: S.lastError,
+    lastFlushAt: S.lastFlushAt,
+    lastFlushReason: S.lastFlushReason,
   };
 }
 
@@ -591,6 +606,21 @@ function recentRecords() {
   return S.recent.slice();
 }
 
+/**
+ * Opt-out side effect: drops every not-yet-sent record from disk + memory so
+ * a re-enable later never resends a payload that was only ever queued while
+ * the user was opted out. Does NOT touch telemetry-sent.json or the backlog
+ * watermarks — those describe history that already left the machine, and
+ * clearing them would make a re-enable resend it. Never throws.
+ */
+async function clearQueue() {
+  S.queue = [];
+  S.queueIds = new Set();
+  try {
+    await config.writeTextAtomic(queuePath(), '', { mode: 0o600 });
+  } catch { /* best-effort */ }
+}
+
 async function isPending(recordId) {
   await ensureInit();
   return S.queueIds.has(recordId);
@@ -615,6 +645,7 @@ module.exports = {
   status,
   recentRecords,
   isPending,
+  clearQueue,
   queuePath,
   sentPath,
   _setFetchImpl,

package/src/main/lib/telemetryConsent.cjs

@@ -0,0 +1,34 @@
+'use strict';
+
+/**
+ * telemetryConsent — the write-path the `telemetry:set-config` IPC handler
+ * delegates to, pulled out of src/main/index.cjs so it can be unit-tested
+ * without an Electron runtime (index.cjs requires('electron') at module
+ * load, which vitest can't satisfy).
+ *
+ * The one side effect beyond a plain settings save: turning telemetry OFF
+ * clears the pending queue file, so a later re-enable never resends a
+ * payload that only ever accumulated while the user was opted out. It does
+ * NOT touch telemetry-sent.json or the backlog watermarks — those describe
+ * history already delivered, and clearing them would make a re-enable
+ * resend it. See session-manager-operations/architecture/telemetry.md.
+ */
+
+function resolveDeps(deps = {}) {
+  return {
+    telemetrySettings: deps.telemetrySettings || require('./telemetrySettings.cjs'),
+    telemetryClient: deps.telemetryClient || require('./telemetryClient.cjs'),
+  };
+}
+
+async function applyConsentUpdate(cfg, depsOverride) {
+  const deps = resolveDeps(depsOverride);
+  const before = await deps.telemetrySettings.load();
+  const saved = await deps.telemetrySettings.save(cfg);
+  if (before.enabled && !saved.enabled) {
+    await deps.telemetryClient.clearQueue();
+  }
+  return saved;
+}
+
+module.exports = { applyConsentUpdate };

package/src/main/lib/telemetryCounters.cjs

@@ -0,0 +1,45 @@
+'use strict';
+
+/**
+ * telemetryCounters.cjs — the ONE place every usage counter event is emitted
+ * from, so 'session.open'/'epic.create'/'scheduler.job.finish'/'app.launch'
+ * each have exactly one call site's worth of shape logic even though they're
+ * fired from four different subsystems (pty.cjs, epicMint.cjs,
+ * scheduleJobTransitions.cjs, telemetryBoot.cjs).
+ *
+ * Structural props only — no free text, no cwd, no path — so a counter event
+ * can never carry a prompt/transcript/project-name leak by accident. Lazily
+ * requires telemetryClient.cjs (mirrors opsErrorLog.cjs's lazy require of
+ * opsOwnership.cjs) so this module stays Electron-free and importable from a
+ * plain-Node test.
+ */
+
+function track(name, props, deps) {
+  try {
+    const telemetryClient = (deps && deps.telemetryClient) || require('./telemetryClient.cjs');
+    telemetryClient.track(name, props);
+  } catch { /* counters must never break the caller */ }
+}
+
+function trackAppLaunch({ installChannel, appVersion } = {}, deps) {
+  track('app.launch', { installChannel: installChannel || null, appVersion: appVersion || null }, deps);
+}
+
+function trackSessionOpen(deps) {
+  track('session.open', {}, deps);
+}
+
+function trackEpicCreate(deps) {
+  track('epic.create', {}, deps);
+}
+
+function trackSchedulerJobFinish({ status } = {}, deps) {
+  track('scheduler.job.finish', { status: status || null }, deps);
+}
+
+module.exports = {
+  trackAppLaunch,
+  trackSessionOpen,
+  trackEpicCreate,
+  trackSchedulerJobFinish,
+};

package/src/main/pty.cjs

@@ -22,6 +22,7 @@ const { checkInsideHome } = require('./lib/insideHome.cjs');
 const { sendIfAlive } = require('./lib/sendToRenderer.cjs');
 const opsErrorLog = require('./lib/opsErrorLog.cjs');
 const { resolveEpicSpawnCwd } = require('./lib/epicSpawnCwd.cjs');
+const telemetryCounters = require('./lib/telemetryCounters.cjs');
 
 // Absolute path to the installed package root (src/main/ -> ../../), shown in
 // the remediation message so the user can cd there and rebuild.
@@ -200,6 +201,7 @@ class PtyManager {
       return { pid: null, cwd, reattached: false, error: String(err?.message || 'spawn-failed') };
     }
     console.log('[pty] spawned pid=', proc.pid, 'for tabId=', tabId);
+    telemetryCounters.trackSessionOpen();
 
     const spawnedAt = Date.now();
     let gotData = false;