@lessonkit/core 1.0.1 → 1.0.2

package/dist/index.cjs

@@ -279,6 +279,11 @@ function invokeTrackingSink(sink, event) {
 function createTrackingClient(opts) {
   const sink = opts?.sink;
   const batchSink = opts?.batchSink;
+  if (batchSink != null && opts?.batch?.enabled === false) {
+    throw new Error(
+      "[lessonkit] tracking.batchSink cannot be used with batch.enabled: false; omit batch.enabled or set it to true"
+    );
+  }
   const batchEnabled = opts?.batch?.enabled ?? Boolean(batchSink);
   const flushIntervalMs = opts?.batch?.flushIntervalMs ?? 5e3;
   const maxBatchSize = opts?.batch?.maxBatchSize ?? 25;
@@ -350,7 +355,7 @@ function createTrackingClient(opts) {
       if (disposed || disposing) return;
       if (buffer.length >= maxBufferSize) {
         buffer.shift();
-        if (!warnedBufferCap && typeof process !== "undefined" && process.env?.NODE_ENV === "development") {
+        if (!warnedBufferCap && isDevEnvironment()) {
           warnedBufferCap = true;
           console.warn(
             `[lessonkit] telemetry batch buffer capped at ${maxBufferSize} events; oldest events are dropped while the sink is unavailable.`
@@ -539,7 +544,8 @@ function createMemoryBackedSessionStorage(session) {
   const warnPersistFailure = () => {
     if (warnedPersistFailure) return;
     warnedPersistFailure = true;
-    if (typeof process !== "undefined" && process.env?.NODE_ENV === "development") {
+    const g = globalThis;
+    if (typeof g.process !== "undefined" && g.process.env?.NODE_ENV === "development") {
       console.warn(
         "[lessonkit] sessionStorage is unavailable or failed; using in-memory session dedupe for this tab (may reset on full reload)."
       );
@@ -580,23 +586,37 @@ function createMemoryBackedSessionStorage(session) {
 function resetStoragePortForTests(storage) {
   storage.resetForTests?.();
 }
+function createInMemorySessionStoragePort() {
+  const memory = /* @__PURE__ */ new Map();
+  return {
+    getItem: (key) => memory.get(key) ?? null,
+    setItem: (key, value) => {
+      memory.set(key, value);
+    },
+    removeItem: (key) => {
+      memory.delete(key);
+    },
+    resetForTests: () => {
+      memory.clear();
+    }
+  };
+}
+function resolveBrowserSessionStorage() {
+  try {
+    if (typeof sessionStorage === "undefined" || sessionStorage == null) {
+      return null;
+    }
+    return sessionStorage;
+  } catch {
+    return null;
+  }
+}
 function createSessionStoragePort() {
-  if (typeof sessionStorage === "undefined") {
-    const memory = /* @__PURE__ */ new Map();
-    return {
-      getItem: (key) => memory.get(key) ?? null,
-      setItem: (key, value) => {
-        memory.set(key, value);
-      },
-      removeItem: (key) => {
-        memory.delete(key);
-      },
-      resetForTests: () => {
-        memory.clear();
-      }
-    };
+  const session = resolveBrowserSessionStorage();
+  if (!session) {
+    return createInMemorySessionStoragePort();
   }
-  return createMemoryBackedSessionStorage(sessionStorage);
+  return createMemoryBackedSessionStorage(session);
 }
 function createGlobalTimer() {
   return {

package/dist/index.js

@@ -206,6 +206,11 @@ function invokeTrackingSink(sink, event) {
 function createTrackingClient(opts) {
   const sink = opts?.sink;
   const batchSink = opts?.batchSink;
+  if (batchSink != null && opts?.batch?.enabled === false) {
+    throw new Error(
+      "[lessonkit] tracking.batchSink cannot be used with batch.enabled: false; omit batch.enabled or set it to true"
+    );
+  }
   const batchEnabled = opts?.batch?.enabled ?? Boolean(batchSink);
   const flushIntervalMs = opts?.batch?.flushIntervalMs ?? 5e3;
   const maxBatchSize = opts?.batch?.maxBatchSize ?? 25;
@@ -277,7 +282,7 @@ function createTrackingClient(opts) {
       if (disposed || disposing) return;
       if (buffer.length >= maxBufferSize) {
         buffer.shift();
-        if (!warnedBufferCap && typeof process !== "undefined" && process.env?.NODE_ENV === "development") {
+        if (!warnedBufferCap && isDevEnvironment()) {
           warnedBufferCap = true;
           console.warn(
             `[lessonkit] telemetry batch buffer capped at ${maxBufferSize} events; oldest events are dropped while the sink is unavailable.`
@@ -466,7 +471,8 @@ function createMemoryBackedSessionStorage(session) {
   const warnPersistFailure = () => {
     if (warnedPersistFailure) return;
     warnedPersistFailure = true;
-    if (typeof process !== "undefined" && process.env?.NODE_ENV === "development") {
+    const g = globalThis;
+    if (typeof g.process !== "undefined" && g.process.env?.NODE_ENV === "development") {
       console.warn(
         "[lessonkit] sessionStorage is unavailable or failed; using in-memory session dedupe for this tab (may reset on full reload)."
       );
@@ -507,23 +513,37 @@ function createMemoryBackedSessionStorage(session) {
 function resetStoragePortForTests(storage) {
   storage.resetForTests?.();
 }
+function createInMemorySessionStoragePort() {
+  const memory = /* @__PURE__ */ new Map();
+  return {
+    getItem: (key) => memory.get(key) ?? null,
+    setItem: (key, value) => {
+      memory.set(key, value);
+    },
+    removeItem: (key) => {
+      memory.delete(key);
+    },
+    resetForTests: () => {
+      memory.clear();
+    }
+  };
+}
+function resolveBrowserSessionStorage() {
+  try {
+    if (typeof sessionStorage === "undefined" || sessionStorage == null) {
+      return null;
+    }
+    return sessionStorage;
+  } catch {
+    return null;
+  }
+}
 function createSessionStoragePort() {
-  if (typeof sessionStorage === "undefined") {
-    const memory = /* @__PURE__ */ new Map();
-    return {
-      getItem: (key) => memory.get(key) ?? null,
-      setItem: (key, value) => {
-        memory.set(key, value);
-      },
-      removeItem: (key) => {
-        memory.delete(key);
-      },
-      resetForTests: () => {
-        memory.clear();
-      }
-    };
+  const session = resolveBrowserSessionStorage();
+  if (!session) {
+    return createInMemorySessionStoragePort();
   }
-  return createMemoryBackedSessionStorage(sessionStorage);
+  return createMemoryBackedSessionStorage(session);
 }
 function createGlobalTimer() {
   return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lessonkit/core",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "private": false,
   "description": "Shared types and telemetry primitives for LessonKit.",
   "license": "Apache-2.0",
@@ -48,8 +48,9 @@
     "lint": "echo \"(no lint configured yet)\""
   },
   "devDependencies": {
+    "@types/node": "^22.13.10",
     "tsup": "^8.5.0",
     "typescript": "^5.8.3",
-    "vitest": "^3.2.4"
+    "vitest": "^4.1.8"
   }
 }