@steve31415/baselib 3.13.0 → 3.14.0

package/dist/log-core.d.ts

@@ -152,5 +152,9 @@ export declare function shapeMeta(meta: Record<string, unknown>): {
     issues: ShapeIssue[];
     leaves: number;
 };
+/** Toggle strict shape checking — for tests that exercise the rewrite/WARN
+ *  behavior itself. Restore with setLogShapeStrict(true) (or leave it: vitest
+ *  isolates module state per file). */
+export declare function setLogShapeStrict(on: boolean): void;
 /** Forget which sites have been flagged (tests). */
 export declare function resetLogShapeWarnings(): void;

package/dist/log-core.js

@@ -377,6 +377,12 @@ function buildEvent(opts, level, message, meta) {
  *  the shape rules in this process, a WARN `kind=log-shape` naming the site. */
 export function makeEvents(opts, level, message, meta) {
     const { event, issues } = buildEvent(opts, level, message, meta);
+    if (issues.length > 0 && strict) {
+        throw new Error(`log-shape (strict): "${message}" — ${issues.map(describeIssue).join('; ')}. ` +
+            'Fix the logging site: keys are columns (OPERATIONS.md). ' +
+            '(Strict mode is on under vitest so shape bugs fail tests instead of ' +
+            'shipping as rewrites; setLogShapeStrict(false) to test the rewrite itself.)');
+    }
     const fresh = issues.filter((i) => {
         const key = `${i.kind}\n${message}`;
         if (flagged.has(key))
@@ -457,6 +463,27 @@ export function shapeMeta(meta) {
 // Sites already flagged in this process, by issue kind and message: the
 // first offending event is worth a WARN, the ten-thousandth is not.
 const flagged = new Set();
+// Strict mode: makeEvents throws on a shape issue instead of rewriting.
+// On by default under vitest so a shape bug fails the suite that exercises
+// the logging site rather than reaching production as a rewritten event
+// with a once-per-process WARN. LOG_SHAPE_STRICT=1/0 overrides the default
+// either way (production stays permissive: a logger must never throw there).
+function strictDefault() {
+    // log-core is shared with the browser build; no bare `process` access.
+    if (typeof process === 'undefined' || !process.env)
+        return false;
+    const explicit = process.env.LOG_SHAPE_STRICT;
+    if (explicit !== undefined && explicit !== '')
+        return explicit !== '0';
+    return process.env.VITEST !== undefined;
+}
+let strict = strictDefault();
+/** Toggle strict shape checking — for tests that exercise the rewrite/WARN
+ *  behavior itself. Restore with setLogShapeStrict(true) (or leave it: vitest
+ *  isolates module state per file). */
+export function setLogShapeStrict(on) {
+    strict = on;
+}
 /** Forget which sites have been flagged (tests). */
 export function resetLogShapeWarnings() {
     flagged.clear();

package/dist/log.d.ts

@@ -1,6 +1,6 @@
 import { Logger, LogShipper, ShipperOptions } from './log-core.js';
-export { LogShipper, serializeError, truncate } from './log-core.js';
-export type { Logger, LogEvent, LogLevel, ShipperOptions, ShippedBatch, ShipFailureReason, } from './log-core.js';
+export { LogShipper, makeEvent, makeEvents, serializeError, setLogShapeStrict, truncate } from './log-core.js';
+export type { EventFactoryOptions, Logger, LogEvent, LogLevel, ShipperOptions, ShippedBatch, ShipFailureReason, } from './log-core.js';
 export interface LoggerOptions {
     /** App name, e.g. 'auth'. Becomes the `app` field queried in Axiom. */
     app: string;

package/dist/log.js

@@ -9,7 +9,10 @@
 // log-core's key-shape guard rewrites offenders and WARNs once per site.
 import { writeSync } from 'node:fs';
 import { LogShipper, makeEvents, truncate, } from './log-core.js';
-export { LogShipper, serializeError, truncate } from './log-core.js';
+// makeEvent/makeEvents are exported for app test fakes: routing a recording
+// logger's calls through makeEvents applies the key-shape guard (strict under
+// vitest) to fake-logged sites too.
+export { LogShipper, makeEvent, makeEvents, serializeError, setLogShapeStrict, truncate } from './log-core.js';
 // Registry of every shipper created here (plus any registered explicitly),
 // used by flushAllLoggers and graceful shutdown. Shipper-level on purpose:
 // with() children share their root's shipper, so registering loggers leaked

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@steve31415/baselib",
-  "version": "3.13.0",
+  "version": "3.14.0",
   "description": "Plasticine new-world shared platform library: logging, auth, service-to-service auth, db, HTTP, sync, app updates",
   "type": "module",
   "license": "MIT",