@koolbase/core 10.2.0 → 10.3.0

package/dist/cjs/analytics.d.ts

@@ -4,13 +4,27 @@ export declare class KoolbaseAnalytics {
     private queue;
     private deviceId;
     private userId?;
+    private getSignedInUser?;
     private environmentId?;
     private userProperties;
     private sessionId;
     private appVersion;
     private flushTimer?;
     private initialized;
-    constructor(config: KoolbaseConfig);
+    /**
+     * getSignedInUser lets events carry the signed-in user without the app
+     * having to say so.
+     *
+     * identify() existed and nothing errored when an app never called it, so
+     * every event landed anonymous and retention, funnels and per-user
+     * analysis were quietly worthless — found in a real project as 53 events,
+     * 8 registered users and not one event carrying a user id. The Flutter SDK
+     * fixed this in 11.2.0; the TypeScript SDKs kept the bug until now.
+     *
+     * identify() still wins, for an app with its own identity system, and
+     * reset() releases that override.
+     */
+    constructor(config: KoolbaseConfig, getSignedInUser?: () => string | null);
     init(appVersion?: string): Promise<void>;
     track(eventName: string, properties?: Record<string, unknown>): void;
     screenView(screenName: string, properties?: Record<string, unknown>): void;

package/dist/cjs/analytics.js

@@ -9,7 +9,20 @@ const DEVICE_ID_KEY = 'koolbase:device_id';
 const FLUSH_INTERVAL_MS = 30000;
 const MAX_BATCH_SIZE = 20;
 class KoolbaseAnalytics {
-    constructor(config) {
+    /**
+     * getSignedInUser lets events carry the signed-in user without the app
+     * having to say so.
+     *
+     * identify() existed and nothing errored when an app never called it, so
+     * every event landed anonymous and retention, funnels and per-user
+     * analysis were quietly worthless — found in a real project as 53 events,
+     * 8 registered users and not one event carrying a user id. The Flutter SDK
+     * fixed this in 11.2.0; the TypeScript SDKs kept the bug until now.
+     *
+     * identify() still wins, for an app with its own identity system, and
+     * reset() releases that override.
+     */
+    constructor(config, getSignedInUser) {
         this.queue = [];
         this.deviceId = '';
         this.userProperties = {};
@@ -17,6 +30,7 @@ class KoolbaseAnalytics {
         this.appVersion = '1.0.0';
         this.initialized = false;
         this.config = config;
+        this.getSignedInUser = getSignedInUser;
     }
     // ─── Init ─────────────────────────────────────────────────────────────────
     async init(appVersion) {
@@ -42,7 +56,8 @@ class KoolbaseAnalytics {
     track(eventName, properties) {
         const event = {
             device_id: this.deviceId,
-            user_id: this.userId,
+            // The explicit override first, then whoever is signed in.
+            user_id: this.userId ?? this.getSignedInUser?.() ?? undefined,
             environment_id: this.environmentId,
             event_name: eventName,
             properties: properties ?? {},

package/dist/esm/analytics.d.ts

@@ -4,13 +4,27 @@ export declare class KoolbaseAnalytics {
     private queue;
     private deviceId;
     private userId?;
+    private getSignedInUser?;
     private environmentId?;
     private userProperties;
     private sessionId;
     private appVersion;
     private flushTimer?;
     private initialized;
-    constructor(config: KoolbaseConfig);
+    /**
+     * getSignedInUser lets events carry the signed-in user without the app
+     * having to say so.
+     *
+     * identify() existed and nothing errored when an app never called it, so
+     * every event landed anonymous and retention, funnels and per-user
+     * analysis were quietly worthless — found in a real project as 53 events,
+     * 8 registered users and not one event carrying a user id. The Flutter SDK
+     * fixed this in 11.2.0; the TypeScript SDKs kept the bug until now.
+     *
+     * identify() still wins, for an app with its own identity system, and
+     * reset() releases that override.
+     */
+    constructor(config: KoolbaseConfig, getSignedInUser?: () => string | null);
     init(appVersion?: string): Promise<void>;
     track(eventName: string, properties?: Record<string, unknown>): void;
     screenView(screenName: string, properties?: Record<string, unknown>): void;

package/dist/esm/analytics.js

@@ -6,7 +6,20 @@ const DEVICE_ID_KEY = 'koolbase:device_id';
 const FLUSH_INTERVAL_MS = 30000;
 const MAX_BATCH_SIZE = 20;
 export class KoolbaseAnalytics {
-    constructor(config) {
+    /**
+     * getSignedInUser lets events carry the signed-in user without the app
+     * having to say so.
+     *
+     * identify() existed and nothing errored when an app never called it, so
+     * every event landed anonymous and retention, funnels and per-user
+     * analysis were quietly worthless — found in a real project as 53 events,
+     * 8 registered users and not one event carrying a user id. The Flutter SDK
+     * fixed this in 11.2.0; the TypeScript SDKs kept the bug until now.
+     *
+     * identify() still wins, for an app with its own identity system, and
+     * reset() releases that override.
+     */
+    constructor(config, getSignedInUser) {
         this.queue = [];
         this.deviceId = '';
         this.userProperties = {};
@@ -14,6 +27,7 @@ export class KoolbaseAnalytics {
         this.appVersion = '1.0.0';
         this.initialized = false;
         this.config = config;
+        this.getSignedInUser = getSignedInUser;
     }
     // ─── Init ─────────────────────────────────────────────────────────────────
     async init(appVersion) {
@@ -39,7 +53,8 @@ export class KoolbaseAnalytics {
     track(eventName, properties) {
         const event = {
             device_id: this.deviceId,
-            user_id: this.userId,
+            // The explicit override first, then whoever is signed in.
+            user_id: this.userId ?? this.getSignedInUser?.() ?? undefined,
             environment_id: this.environmentId,
             event_name: eventName,
             properties: properties ?? {},

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@koolbase/core",
-  "version": "10.2.0",
+  "version": "10.3.0",
   "description": "Koolbase SDK core \u2014 shared behaviour behind @koolbase/react-native and @koolbase/js. Install one of those, not this.",
   "main": "./dist/cjs/index.js",
   "types": "./dist/esm/index.d.ts",