@glasstrace/sdk 0.14.2 → 0.15.1

package/dist/index.d.cts

@@ -278,9 +278,15 @@ declare function getOrCreateAnonKey(projectRoot?: string): Promise<AnonApiKey>;
  */
 declare function loadCachedConfig(projectRoot?: string): SdkInitResponse | null;
 /**
- * Persists the init response to `.glasstrace/config`.
- * Silently skipped when `node:fs` is unavailable (non-Node environments).
- * On I/O failure, logs a warning and continues.
+ * Persists the init response to `.glasstrace/config` using atomic
+ * write-temp + rename semantics. Silently skipped when `node:fs` is
+ * unavailable (non-Node environments). On I/O failure, logs a warning.
+ *
+ * Atomicity: the payload is written to `.glasstrace/config.tmp` and then
+ * renamed into place. `rename` is atomic on POSIX filesystems, so readers
+ * either see the previous valid config or the new valid config — never a
+ * truncated or partially-written file (DISC-1247 Scenario 5). If the
+ * rename fails, the temp file is cleaned up on a best-effort basis.
  */
 declare function saveCachedConfig(response: SdkInitResponse, projectRoot?: string): Promise<void>;
 /**

package/dist/index.d.ts

@@ -278,9 +278,15 @@ declare function getOrCreateAnonKey(projectRoot?: string): Promise<AnonApiKey>;
  */
 declare function loadCachedConfig(projectRoot?: string): SdkInitResponse | null;
 /**
- * Persists the init response to `.glasstrace/config`.
- * Silently skipped when `node:fs` is unavailable (non-Node environments).
- * On I/O failure, logs a warning and continues.
+ * Persists the init response to `.glasstrace/config` using atomic
+ * write-temp + rename semantics. Silently skipped when `node:fs` is
+ * unavailable (non-Node environments). On I/O failure, logs a warning.
+ *
+ * Atomicity: the payload is written to `.glasstrace/config.tmp` and then
+ * renamed into place. `rename` is atomic on POSIX filesystems, so readers
+ * either see the previous valid config or the new valid config — never a
+ * truncated or partially-written file (DISC-1247 Scenario 5). If the
+ * rename fails, the temp file is cleaned up on a best-effort basis.
  */
 declare function saveCachedConfig(response: SdkInitResponse, projectRoot?: string): Promise<void>;
 /**

package/dist/index.js

@@ -304,6 +304,7 @@ async function saveCachedConfig(response, projectRoot) {
   const root = projectRoot ?? process.cwd();
   const dirPath = modules.path.join(root, GLASSTRACE_DIR);
   const configPath = modules.path.join(dirPath, CONFIG_FILE);
+  const tmpPath = `${configPath}.tmp`;
   try {
     await modules.fs.mkdir(dirPath, { recursive: true, mode: 448 });
     await modules.fs.chmod(dirPath, 448);
@@ -311,7 +312,20 @@ async function saveCachedConfig(response, projectRoot) {
       response,
       cachedAt: Date.now()
     };
-    await modules.fs.writeFile(configPath, JSON.stringify(cached), { encoding: "utf-8", mode: 384 });
+    await modules.fs.writeFile(tmpPath, JSON.stringify(cached), {
+      encoding: "utf-8",
+      mode: 384
+    });
+    try {
+      await modules.fs.chmod(tmpPath, 384);
+      await modules.fs.rename(tmpPath, configPath);
+    } catch (renameErr) {
+      try {
+        await modules.fs.unlink(tmpPath);
+      } catch {
+      }
+      throw renameErr;
+    }
     await modules.fs.chmod(configPath, 384);
   } catch (err) {
     console.warn(
@@ -4430,7 +4444,6 @@ async function configureOtel(config, sessionManager) {
       await provider.shutdown();
     }
   });
-  registerSignalHandlers();
   registerBeforeExitTrigger();
   const prismaModule = await tryImport("@prisma/instrumentation");
   if (prismaModule) {
@@ -4485,12 +4498,15 @@ var HEARTBEAT_INTERVAL_MS = 5 * 60 * 1e3;
 var BACKOFF_BASE_MS = HEARTBEAT_INTERVAL_MS;
 var BACKOFF_MAX_MS = 30 * 60 * 1e3;
 var BACKOFF_JITTER = 0.2;
+var HEARTBEAT_SHUTDOWN_PRIORITY = 10;
+var SHUTDOWN_MARKER_RELPATH = ".glasstrace/shutdown-requested";
 var heartbeatTimer = null;
 var heartbeatGeneration = 0;
 var backoffAttempts = 0;
 var backoffUntil = 0;
 var tickInProgress = false;
-var _shutdownHandler = null;
+var shutdownHookRegistered = false;
+var shutdownFired = false;
 function startHeartbeat(config, anonKey, sdkVersion, generation, onClaimTransition) {
   if (heartbeatTimer !== null) return;
   heartbeatGeneration = generation;
@@ -4498,7 +4514,7 @@ function startHeartbeat(config, anonKey, sdkVersion, generation, onClaimTransiti
     void heartbeatTick(config, anonKey, sdkVersion, generation, onClaimTransition);
   }, HEARTBEAT_INTERVAL_MS);
   heartbeatTimer.unref();
-  registerShutdownHandlers(config, anonKey, sdkVersion);
+  registerHeartbeatShutdownHook(config, anonKey, sdkVersion);
   if (config.verbose) {
     sdkLog("info", "[glasstrace] Heartbeat started (5-minute interval).");
   }
@@ -4508,7 +4524,26 @@ function stopHeartbeat() {
     clearInterval(heartbeatTimer);
     heartbeatTimer = null;
   }
-  removeShutdownHandlers();
+}
+function checkShutdownMarker(projectRoot) {
+  let fsSync = null;
+  let pathSync = null;
+  try {
+    fsSync = __require("fs");
+    pathSync = __require("path");
+  } catch {
+    return { triggered: false };
+  }
+  const root = projectRoot ?? (typeof process !== "undefined" ? process.cwd() : ".");
+  const markerPath = pathSync.join(root, SHUTDOWN_MARKER_RELPATH);
+  if (!fsSync.existsSync(markerPath)) return { triggered: false };
+  try {
+    fsSync.unlinkSync(markerPath);
+  } catch {
+  }
+  const shutdown = executeShutdown().catch(() => {
+  });
+  return { triggered: true, shutdown };
 }
 async function heartbeatTick(config, anonKey, sdkVersion, generation, onClaimTransition) {
   if (tickInProgress) return;
@@ -4518,6 +4553,14 @@ async function heartbeatTick(config, anonKey, sdkVersion, generation, onClaimTra
       stopHeartbeat();
       return;
     }
+    const markerResult = checkShutdownMarker();
+    if (markerResult.triggered) {
+      stopHeartbeat();
+      if (markerResult.shutdown) {
+        await markerResult.shutdown;
+      }
+      return;
+    }
     if (Date.now() < backoffUntil) {
       if (config.verbose) {
         sdkLog("info", "[glasstrace] Heartbeat skipped (rate-limit backoff).");
@@ -4552,35 +4595,26 @@ async function heartbeatTick(config, anonKey, sdkVersion, generation, onClaimTra
     tickInProgress = false;
   }
 }
-function registerShutdownHandlers(config, anonKey, sdkVersion) {
-  if (typeof process === "undefined" || typeof process.once !== "function") {
-    return;
-  }
-  let shutdownFired = false;
-  const handler = (signal) => {
-    if (shutdownFired) return;
-    shutdownFired = true;
-    if (heartbeatTimer !== null) {
-      clearInterval(heartbeatTimer);
-      heartbeatTimer = null;
+function registerHeartbeatShutdownHook(config, anonKey, sdkVersion) {
+  if (shutdownHookRegistered) return;
+  shutdownHookRegistered = true;
+  registerShutdownHook({
+    name: "heartbeat-final-report",
+    priority: HEARTBEAT_SHUTDOWN_PRIORITY,
+    fn: async () => {
+      if (shutdownFired) return;
+      shutdownFired = true;
+      if (heartbeatTimer !== null) {
+        clearInterval(heartbeatTimer);
+        heartbeatTimer = null;
+      }
+      try {
+        const healthReport = collectHealthReport(sdkVersion);
+        await performInit(config, anonKey, sdkVersion, healthReport);
+      } catch {
+      }
     }
-    const healthReport = collectHealthReport(sdkVersion);
-    void performInit(config, anonKey, sdkVersion, healthReport).catch(() => {
-    }).finally(() => {
-      removeShutdownHandlers();
-      process.kill(process.pid, signal);
-    });
-  };
-  _shutdownHandler = handler;
-  process.once("SIGTERM", _shutdownHandler);
-  process.once("SIGINT", _shutdownHandler);
-}
-function removeShutdownHandlers() {
-  if (_shutdownHandler && typeof process !== "undefined") {
-    process.removeListener("SIGTERM", _shutdownHandler);
-    process.removeListener("SIGINT", _shutdownHandler);
-    _shutdownHandler = null;
-  }
+  });
 }
 
 // src/runtime-state.ts
@@ -4681,7 +4715,7 @@ function registerGlasstrace(options) {
     setCoreState(CoreState.REGISTERING);
     startRuntimeStateWriter({
       projectRoot: process.cwd(),
-      sdkVersion: "0.14.2"
+      sdkVersion: "0.15.1"
     });
     const config = resolveConfig(options);
     if (config.verbose) {
@@ -4697,6 +4731,11 @@ function registerGlasstrace(options) {
     if (config.verbose) {
       console.info("[glasstrace] Not production-disabled.");
     }
+    const existingProbe = trace.getTracerProvider().getTracer("glasstrace-probe");
+    const anotherProviderRegistered = existingProbe.constructor.name !== "ProxyTracer";
+    if (!anotherProviderRegistered) {
+      registerSignalHandlers();
+    }
     const anonymous = isAnonymousMode(config);
     let effectiveKey = config.apiKey;
     initAuthState(anonymous ? AuthState.ANONYMOUS : AuthState.AUTHENTICATED);
@@ -4727,8 +4766,6 @@ function registerGlasstrace(options) {
     }
     setCoreState(CoreState.KEY_PENDING);
     const currentGeneration = registrationGeneration;
-    const existingProbe = trace.getTracerProvider().getTracer("glasstrace-probe");
-    const anotherProviderRegistered = existingProbe.constructor.name !== "ProxyTracer";
     if (anotherProviderRegistered) {
       if (config.verbose) {
         console.info("[glasstrace] Another OTel provider detected \u2014 using existing context manager.");
@@ -4843,8 +4880,8 @@ async function backgroundInit(config, anonKeyForInit, generation) {
   if (config.verbose) {
     console.info("[glasstrace] Background init firing.");
   }
-  const healthReport = collectHealthReport("0.14.2");
-  const initResult = await performInit(config, anonKeyForInit, "0.14.2", healthReport);
+  const healthReport = collectHealthReport("0.15.1");
+  const initResult = await performInit(config, anonKeyForInit, "0.15.1", healthReport);
   if (generation !== registrationGeneration) return;
   const currentState = getCoreState();
   if (currentState === CoreState.SHUTTING_DOWN || currentState === CoreState.SHUTDOWN) {
@@ -4867,7 +4904,7 @@ async function backgroundInit(config, anonKeyForInit, generation) {
   }
   maybeInstallConsoleCapture();
   if (didLastInitSucceed()) {
-    startHeartbeat(config, anonKeyForInit, "0.14.2", generation, (newApiKey, accountId) => {
+    startHeartbeat(config, anonKeyForInit, "0.15.1", generation, (newApiKey, accountId) => {
       setAuthState(AuthState.CLAIMING);
       emitLifecycleEvent("auth:claim_started", { accountId });
       setResolvedApiKey(newApiKey);