@glasstrace/sdk 0.14.1 → 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>;
 /**
@@ -488,7 +494,25 @@ declare function registerGlasstrace(options?: GlasstraceOptions): void;
  */
 declare function getDiscoveryHandler(): ((request: Request) => Promise<Response | null>) | null;
 
-type NextConfig = Record<string, unknown>;
+/**
+ * Structural view of Next.js's `NextConfig`. The SDK does not import Next's
+ * type directly because Next is not a peer dependency — this wrapper must
+ * type-check regardless of which Next.js version the consumer has installed.
+ *
+ * The constraint is `object` rather than `Record<string, unknown>` because
+ * Next's actual `NextConfig` is an interface *without* a string index
+ * signature. Requiring `[key: string]: unknown` would fail the assignability
+ * check that caused DISC-1256, reported by Next 16 consumers as:
+ *   > Argument of type 'NextConfig' is not assignable to parameter of type
+ *   > 'NextConfig'. Index signature for type 'string' is missing in type
+ *   > 'NextConfig'.
+ *
+ * `object` accepts every non-primitive value, which is what the wrapper
+ * actually handles at runtime (it shallow-copies the input and reads a few
+ * known properties defensively). Combined with the generic signature on
+ * `withGlasstraceConfig`, callers preserve their exact config subtype.
+ */
+type NextConfig = object;
 /**
  * Wraps the developer's Next.js config to enable source map generation
  * and upload .map files to the ingestion API at build time.
@@ -496,10 +520,26 @@ type NextConfig = Record<string, unknown>;
  * The build NEVER fails because of Glasstrace — all errors are caught
  * and logged as warnings.
  *
+ * ## Turbopack
+ *
+ * Next.js 16 made Turbopack the default bundler for `next build`, and Next
+ * rejects configs that set `webpack` without also setting `turbopack`. This
+ * wrapper therefore seeds an empty `turbopack: {}` when the user has not set
+ * one themselves, preserving existing behaviour for explicit Turbopack configs.
+ *
+ * **Source-map upload is currently webpack-only.** Under Turbopack the build
+ * succeeds, but the afterEmit hook that collects and uploads `.map` files does
+ * not fire. Run `next build --webpack` to get source-map uploads, or wait for
+ * a follow-up release that ports the plugin to Turbopack.
+ *
  * @param nextConfig - The developer's existing Next.js configuration object.
  * @returns A new config object with source map generation and upload enabled.
+ *   The return type mirrors the input type so that caller-side config
+ *   properties are preserved. The `object` constraint (rather than
+ *   `Record<string, unknown>`) is what makes the wrapper accept Next's
+ *   real `NextConfig` interface (DISC-1256).
  */
-declare function withGlasstraceConfig(nextConfig: NextConfig): NextConfig;
+declare function withGlasstraceConfig<T extends NextConfig>(nextConfig: T): T;
 
 /**
  * Returns true when the SDK is in ACTIVE or ACTIVE_DEGRADED state.

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>;
 /**
@@ -488,7 +494,25 @@ declare function registerGlasstrace(options?: GlasstraceOptions): void;
  */
 declare function getDiscoveryHandler(): ((request: Request) => Promise<Response | null>) | null;
 
-type NextConfig = Record<string, unknown>;
+/**
+ * Structural view of Next.js's `NextConfig`. The SDK does not import Next's
+ * type directly because Next is not a peer dependency — this wrapper must
+ * type-check regardless of which Next.js version the consumer has installed.
+ *
+ * The constraint is `object` rather than `Record<string, unknown>` because
+ * Next's actual `NextConfig` is an interface *without* a string index
+ * signature. Requiring `[key: string]: unknown` would fail the assignability
+ * check that caused DISC-1256, reported by Next 16 consumers as:
+ *   > Argument of type 'NextConfig' is not assignable to parameter of type
+ *   > 'NextConfig'. Index signature for type 'string' is missing in type
+ *   > 'NextConfig'.
+ *
+ * `object` accepts every non-primitive value, which is what the wrapper
+ * actually handles at runtime (it shallow-copies the input and reads a few
+ * known properties defensively). Combined with the generic signature on
+ * `withGlasstraceConfig`, callers preserve their exact config subtype.
+ */
+type NextConfig = object;
 /**
  * Wraps the developer's Next.js config to enable source map generation
  * and upload .map files to the ingestion API at build time.
@@ -496,10 +520,26 @@ type NextConfig = Record<string, unknown>;
  * The build NEVER fails because of Glasstrace — all errors are caught
  * and logged as warnings.
  *
+ * ## Turbopack
+ *
+ * Next.js 16 made Turbopack the default bundler for `next build`, and Next
+ * rejects configs that set `webpack` without also setting `turbopack`. This
+ * wrapper therefore seeds an empty `turbopack: {}` when the user has not set
+ * one themselves, preserving existing behaviour for explicit Turbopack configs.
+ *
+ * **Source-map upload is currently webpack-only.** Under Turbopack the build
+ * succeeds, but the afterEmit hook that collects and uploads `.map` files does
+ * not fire. Run `next build --webpack` to get source-map uploads, or wait for
+ * a follow-up release that ports the plugin to Turbopack.
+ *
  * @param nextConfig - The developer's existing Next.js configuration object.
  * @returns A new config object with source map generation and upload enabled.
+ *   The return type mirrors the input type so that caller-side config
+ *   properties are preserved. The `object` constraint (rather than
+ *   `Record<string, unknown>`) is what makes the wrapper accept Next's
+ *   real `NextConfig` interface (DISC-1256).
  */
-declare function withGlasstraceConfig(nextConfig: NextConfig): NextConfig;
+declare function withGlasstraceConfig<T extends NextConfig>(nextConfig: T): T;
 
 /**
  * Returns true when the SDK is in ACTIVE or ACTIVE_DEGRADED state.

package/dist/index.js

@@ -13,7 +13,7 @@ import {
   uploadSourceMaps,
   uploadSourceMapsAuto,
   uploadSourceMapsPresigned
-} from "./chunk-ERGEG4ZQ.js";
+} from "./chunk-2LDBR3F3.js";
 import {
   buildImportGraph,
   discoverTestFiles,
@@ -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.1"
+      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.1");
-  const initResult = await performInit(config, anonKeyForInit, "0.14.1", 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.1", generation, (newApiKey, accountId) => {
+    startHeartbeat(config, anonKeyForInit, "0.15.1", generation, (newApiKey, accountId) => {
       setAuthState(AuthState.CLAIMING);
       emitLifecycleEvent("auth:claim_started", { accountId });
       setResolvedApiKey(newApiKey);
@@ -4898,16 +4935,34 @@ function isDiscoveryEnabled(config) {
 
 // src/config-wrapper.ts
 init_esm_shims();
+function isTurbopackBuild() {
+  if (typeof process === "undefined") return false;
+  const argv = Array.isArray(process.argv) ? process.argv : [];
+  if (argv.includes("--webpack")) return false;
+  if (argv.includes("--turbopack")) return true;
+  if (process.env?.TURBOPACK === "1") return true;
+  return false;
+}
 function withGlasstraceConfig(nextConfig) {
   if (typeof process === "undefined" || typeof process.versions?.node !== "string") {
     return nextConfig != null ? { ...nextConfig } : {};
   }
   const config = nextConfig != null ? { ...nextConfig } : {};
-  const existingExperimental = config.experimental ?? {};
-  config.experimental = { ...existingExperimental, serverSourceMaps: true };
-  const distDir = typeof config.distDir === "string" ? config.distDir : ".next";
-  const existingWebpack = config.webpack;
-  config.webpack = (webpackConfig, context2) => {
+  const bag = config;
+  const existingExperimental = bag.experimental ?? {};
+  bag.experimental = {
+    ...existingExperimental,
+    serverSourceMaps: true
+  };
+  if (bag.turbopack == null) {
+    bag.turbopack = {};
+  }
+  if (isTurbopackBuild()) {
+    warnTurbopackLimitationOnce();
+  }
+  const distDir = typeof bag.distDir === "string" ? bag.distDir : ".next";
+  const existingWebpack = bag.webpack;
+  bag.webpack = (webpackConfig, context2) => {
     let result = webpackConfig;
     if (typeof existingWebpack === "function") {
       result = existingWebpack(webpackConfig, context2);
@@ -4934,6 +4989,14 @@ function withGlasstraceConfig(nextConfig) {
   };
   return config;
 }
+var _turbopackWarningEmitted = false;
+function warnTurbopackLimitationOnce() {
+  if (_turbopackWarningEmitted) return;
+  _turbopackWarningEmitted = true;
+  console.warn(
+    "[glasstrace] Turbopack detected. Source-map upload currently runs only under webpack \u2014 run `next build --webpack` to upload source maps, or wait for the Turbopack port in a future SDK release."
+  );
+}
 async function handleSourceMapUpload(distDir) {
   try {
     const apiKey = process.env.GLASSTRACE_API_KEY;
@@ -4944,7 +5007,7 @@ async function handleSourceMapUpload(distDir) {
       );
       return;
     }
-    const { discoverSourceMapFiles: discoverSourceMapFiles2, computeBuildHash: computeBuildHash2, uploadSourceMaps: uploadSourceMaps2 } = await import("./source-map-uploader-W6VPGY26.js");
+    const { discoverSourceMapFiles: discoverSourceMapFiles2, computeBuildHash: computeBuildHash2, uploadSourceMaps: uploadSourceMaps2 } = await import("./source-map-uploader-3GWUQDTS.js");
     const files = await discoverSourceMapFiles2(distDir);
     if (files.length === 0) {
       console.info("[glasstrace] No source map files found. Skipping upload.");