@diveinto/obs 1.0.2 → 1.0.3

package/dist/config.d.ts

@@ -71,11 +71,17 @@ type Resolved = Required<Pick<ObsConfig, "service" | "envPrefix" | "defaultVarDi
 };
 export declare function configure(cfg: ObsConfig): void;
 /**
- * The live config. Falling back rather than throwing is deliberate: a log
- * line emitted before `configure()` (an import-time warning, say) should
- * still come out, named honestly, instead of taking the process down.
+ * The live configuration, or a fallback.
+ *
+ * Falling back rather than throwing is deliberate: a log line emitted before
+ * `configure()` (an import-time warning, say) should still come out, named
+ * honestly, instead of taking the process down. The fallback is NOT cached,
+ * so a later `configure()` still wins, and `isConfigured()` lets the file
+ * sinks refuse to invent a path from it.
  */
 export declare function obsConfig(): Resolved;
+/** Whether `configure()` has actually run. See `setupLogging()`. */
+export declare function isConfigured(): boolean;
 /** Test seam: forget the configuration so a test can set a different one. */
 export declare function resetConfigForTests(): void;
 /** Read one of this service's own prefixed variables. */

package/dist/config.js

@@ -14,7 +14,26 @@
  * service name is a no-op and a call with a DIFFERENT one is a mistake worth
  * hearing about.
  */
-let current = null;
+/**
+ * The configuration lives on `globalThis`, NOT in a module-level variable.
+ *
+ * This is not defensiveness, it is a scar. Next can run `instrumentation.ts`
+ * in a DIFFERENT module context from the route handlers, so a module-level
+ * `let` set by `configure()` reads as unset in half the process. When that
+ * happened, the lazy sink initialiser re-ran `setupLogging()` in the second
+ * context, got the fallback configuration, tried to open
+ * `/var/lib/unconfigured/logs/...`, and self-disabled both file sinks. Every
+ * request log went to stdout only and the `.jsonl` and `.log` files silently
+ * stopped receiving anything, which is exactly the surface operators grep.
+ *
+ * `globalThis` is shared across those contexts, which is the same reason
+ * @vercel/otel puts its provider there and why `otelEnabled()` reads the
+ * environment rather than a flag.
+ */
+const SLOT = Symbol.for("diveinto.obs.config");
+function slot() {
+    return globalThis;
+}
 function resolve(cfg) {
     const service = cfg.service.trim();
     if (!service)
@@ -35,24 +54,33 @@ function resolve(cfg) {
 }
 export function configure(cfg) {
     const next = resolve(cfg);
-    if (current && current.service !== next.service) {
+    const held = slot()[SLOT];
+    if (held && held.service !== next.service) {
         // Two services in one process is not a thing this library supports, and
         // the symptom (logs filed under the wrong service) is miserable to chase.
-        throw new Error(`[obs] already configured as '${current.service}'; cannot reconfigure as '${next.service}'`);
+        throw new Error(`[obs] already configured as '${held.service}'; cannot reconfigure as '${next.service}'`);
     }
-    current = next;
+    slot()[SLOT] = next;
 }
 /**
- * The live config. Falling back rather than throwing is deliberate: a log
- * line emitted before `configure()` (an import-time warning, say) should
- * still come out, named honestly, instead of taking the process down.
+ * The live configuration, or a fallback.
+ *
+ * Falling back rather than throwing is deliberate: a log line emitted before
+ * `configure()` (an import-time warning, say) should still come out, named
+ * honestly, instead of taking the process down. The fallback is NOT cached,
+ * so a later `configure()` still wins, and `isConfigured()` lets the file
+ * sinks refuse to invent a path from it.
  */
 export function obsConfig() {
-    return current ?? (current = resolve({ service: "unconfigured", envPrefix: "OBS" }));
+    return slot()[SLOT] ?? resolve({ service: "unconfigured", envPrefix: "OBS" });
+}
+/** Whether `configure()` has actually run. See `setupLogging()`. */
+export function isConfigured() {
+    return slot()[SLOT] !== undefined;
 }
 /** Test seam: forget the configuration so a test can set a different one. */
 export function resetConfigForTests() {
-    current = null;
+    delete slot()[SLOT];
 }
 /** Read one of this service's own prefixed variables. */
 export function envVar(name, env = process.env) {

package/dist/setup.js

@@ -23,7 +23,7 @@
  */
 import { join } from "node:path";
 import { logEvent } from "./api.js";
-import { obsConfig } from "./config.js";
+import { isConfigured, obsConfig } from "./config.js";
 import { makeOtelLogSink, otelEnabled } from "./otel.js";
 import { renderJson, renderPretty } from "./render.js";
 import { configureSinks, makeFileSink, makeStdoutSink, setSinkInitializer } from "./sinks.js";
@@ -116,8 +116,18 @@ export function setupLogging() {
     // The two rotating files are self-hosted only. They are format-locked
     // (<stem>.log = pretty, <stem>.jsonl = json) regardless of LOG_FORMAT, so
     // each file is always what its name promises.
+    //
+    // Belt and braces after the globalThis fix: if this ever runs before
+    // configure() again, write to stdout and say so, rather than deriving a
+    // path from the fallback name. The old behaviour opened
+    // /var/lib/unconfigured/logs/, failed, self-disabled both files, and left
+    // an operator with a silently empty .jsonl.
     const serverless = isServerless();
-    if (!serverless) {
+    if (!serverless && !isConfigured()) {
+        process.stderr.write("[obs] setupLogging() ran before configure(); writing to stdout only. " +
+            "Call configure() or createObs() first, from instrumentation.ts.\n");
+    }
+    if (!serverless && isConfigured()) {
         const targets = resolveFileTargets();
         if (targets.text) {
             sinks.push(makeFileSink(targets.text.path, "pretty", renderPretty, targets.text.rotation, targets.text.retention));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diveinto/obs",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Structured logging, tracing and metrics for Next.js and Node services: one record shape, rotating files, W3C trace context, optional OpenTelemetry export",
   "license": "MIT",
   "private": false,