@leanbase-giangnd/js 0.2.2 → 0.2.3

package/package.json

@@ -1,47 +1,48 @@
 {
-  "name": "@leanbase-giangnd/js",
-  "version": "0.2.2",
-  "description": "Leanbase browser SDK - event tracking, autocapture, and session replay",
-  "repository": {
-    "type": "git",
-    "directory": "packages/leanbase"
-  },
-  "author": "leanbase",
-  "license": "Copyrighted by Leanflag Limited",
-  "main": "dist/index.cjs",
-  "module": "dist/index.mjs",
-  "types": "dist/index.d.ts",
-  "unpkg": "dist/leanbase.iife.js",
-  "jsdelivr": "dist/leanbase.iife.js",
-  "files": [
-    "dist",
-    "src",
-    "README.md"
-  ],
-  "publishConfig": {
-    "access": "public"
-  },
-  "dependencies": {
-    "@rrweb/record": "2.0.0-alpha.17",
-    "fflate": "^0.4.8",
-    "@posthog/core": "1.3.1"
-  },
-  "devDependencies": {
-    "jest": "^29.7.0",
-    "jest-environment-jsdom": "^29.7.0",
-    "rollup": "^4.44.1",
-    "rimraf": "^6.0.1",
-    "@posthog-tooling/tsconfig-base": "1.0.0",
-    "@posthog-tooling/rollup-utils": "1.0.0"
-  },
-  "scripts": {
-    "clean": "rimraf dist coverage",
-    "test:unit": "jest -c jest.config.js",
-    "lint": "eslint src test",
-    "lint:fix": "eslint src test --fix",
-    "prebuild": "node -p \"'export const version = \\'' + require('./package.json').version + '\\''\" > src/version.ts",
-    "build": "rollup -c",
-    "dev": "rollup -c -w",
-    "package": "mkdir -p ../../target && pnpm pack --pack-destination ../../target"
-  }
-}
+    "name": "@leanbase-giangnd/js",
+    "version": "0.2.3",
+    "description": "Leanbase browser SDK - event tracking, autocapture, and session replay",
+    "repository": {
+        "type": "git",
+        "directory": "packages/leanbase"
+    },
+    "author": "leanbase",
+    "license": "Copyrighted by Leanflag Limited",
+    "main": "dist/index.cjs",
+    "module": "dist/index.mjs",
+    "types": "dist/index.d.ts",
+    "unpkg": "dist/leanbase.iife.js",
+    "jsdelivr": "dist/leanbase.iife.js",
+    "scripts": {
+        "clean": "rimraf dist coverage",
+        "test:unit": "jest -c jest.config.js",
+        "lint": "eslint src test",
+        "lint:fix": "eslint src test --fix",
+        "prebuild": "node -p \"'export const version = \\'' + require('./package.json').version + '\\''\" > src/version.ts",
+        "build": "rollup -c",
+        "dev": "rollup -c -w",
+        "prepublishOnly": "pnpm lint && pnpm test:unit && pnpm build",
+        "package": "mkdir -p ../../target && pnpm pack --pack-destination ../../target"
+    },
+    "files": [
+        "dist",
+        "src",
+        "README.md"
+    ],
+    "publishConfig": {
+        "access": "public"
+    },
+    "dependencies": {
+        "@posthog/core": "workspace:*",
+        "@rrweb/record": "2.0.0-alpha.17",
+        "fflate": "^0.4.8"
+    },
+    "devDependencies": {
+        "@posthog-tooling/tsconfig-base": "workspace:*",
+        "@posthog-tooling/rollup-utils": "workspace:*",
+        "jest": "catalog:",
+        "jest-environment-jsdom": "catalog:",
+        "rollup": "catalog:",
+        "rimraf": "^6.0.1"
+    }
+}

package/src/extensions/replay/external/lazy-loaded-session-recorder.ts

@@ -759,6 +759,12 @@ export class LazyLoadedSessionRecording {
         this._makeSamplingDecision(this.sessionId)
         await this._startRecorder()
 
+        // If rrweb failed to load/start, do not proceed further.
+        // This prevents installing listeners that assume rrweb is active.
+        if (!this.isStarted) {
+            return
+        }
+
         // calling addEventListener multiple times is safe and will not add duplicates
         addEventListener(window, 'beforeunload', this._onBeforeUnload)
         addEventListener(window, 'offline', this._onOffline)
@@ -1418,13 +1424,19 @@ export class LazyLoadedSessionRecording {
             })
 
         const activePlugins = this._gatherRRWebPlugins()
-        this._stopRrweb = rrwebRecord({
-            emit: (event) => {
-                this.onRRwebEmit(event)
-            },
-            plugins: activePlugins,
-            ...sessionRecordingOptions,
-        })
+        try {
+            this._stopRrweb = rrwebRecord({
+                emit: (event) => {
+                    this.onRRwebEmit(event)
+                },
+                plugins: activePlugins,
+                ...sessionRecordingOptions,
+            })
+        } catch (e) {
+            logger.error('failed to start rrweb recorder', e)
+            this._stopRrweb = undefined
+            return
+        }
 
         // We reset the last activity timestamp, resetting the idle timer
         this._lastActivityTimestamp = Date.now()

package/src/extensions/replay/session-recording.ts

@@ -213,22 +213,29 @@ export class SessionRecording {
         // If extensions provide an init function, use it. Otherwise, fall back to the local LazyLoadedSessionRecording
         if (assignableWindow.__PosthogExtensions__?.initSessionRecording) {
             if (!this._lazyLoadedSessionRecording) {
-                this._lazyLoadedSessionRecording = assignableWindow.__PosthogExtensions__?.initSessionRecording(
-                    this._instance
-                )
-                ;(this._lazyLoadedSessionRecording as any)._forceAllowLocalhostNetworkCapture =
-                    this._forceAllowLocalhostNetworkCapture
+                const maybeRecording = assignableWindow.__PosthogExtensions__?.initSessionRecording(this._instance)
+                if (maybeRecording && typeof (maybeRecording as any).start === 'function') {
+                    this._lazyLoadedSessionRecording = maybeRecording
+                    ;(this._lazyLoadedSessionRecording as any)._forceAllowLocalhostNetworkCapture =
+                        this._forceAllowLocalhostNetworkCapture
+                } else {
+                    log.warn(
+                        'initSessionRecording was present but did not return a recorder instance; falling back to local recorder'
+                    )
+                }
             }
 
-            try {
-                const maybePromise: any = this._lazyLoadedSessionRecording!.start(startReason)
-                if (maybePromise && typeof maybePromise.catch === 'function') {
-                    maybePromise.catch((e: any) => logger.error('error starting session recording', e))
+            if (this._lazyLoadedSessionRecording) {
+                try {
+                    const maybePromise: any = this._lazyLoadedSessionRecording.start(startReason)
+                    if (maybePromise && typeof maybePromise.catch === 'function') {
+                        maybePromise.catch((e: any) => logger.error('error starting session recording', e))
+                    }
+                } catch (e: any) {
+                    logger.error('error starting session recording', e)
                 }
-            } catch (e: any) {
-                logger.error('error starting session recording', e)
+                return
             }
-            return
         }
 
         if (!this._lazyLoadedSessionRecording) {

package/src/leanbase.ts

@@ -93,6 +93,10 @@ export class Leanbase extends PostHogCore {
     consent!: ConsentManager
     sessionRecording?: SessionRecording
     isRemoteConfigLoaded?: boolean
+    private _remoteConfigLoadAttempted: boolean = false
+    private _remoteConfigResolved: boolean = false
+    private _featureFlagsResolved: boolean = false
+    private _maybeStartedSessionRecording: boolean = false
     personProcessingSetOncePropertiesSent = false
     isLoaded: boolean = false
     initialPageviewCaptured: boolean
@@ -130,11 +134,21 @@ export class Leanbase extends PostHogCore {
 
         if (this.sessionManager && this.config.cookieless_mode !== 'always') {
             this.sessionRecording = new SessionRecording(this)
-            this.sessionRecording.startIfEnabledOrStop()
         }
 
+        // Start session recording only once flags + remote config have been resolved.
+        // This matches the PostHog browser SDK where replay activation is driven by remote config and flags.
         if (this.config.preloadFeatureFlags !== false) {
-            this.reloadFeatureFlags()
+            this.reloadFeatureFlags({
+                // eslint-disable-next-line @typescript-eslint/no-unused-vars
+                cb: (_err) => {
+                    this._featureFlagsResolved = true
+                    this._maybeStartSessionRecording()
+                },
+            })
+        } else {
+            // If feature flags preload is explicitly disabled, treat this requirement as satisfied.
+            this._featureFlagsResolved = true
         }
 
         this.config.loaded?.(this)
@@ -146,9 +160,25 @@ export class Leanbase extends PostHogCore {
             }, 1)
         }
 
-        addEventListener(document, 'DOMContentLoaded', () => {
-            this.loadRemoteConfig()
-        })
+        const triggerRemoteConfigLoad = (reason: 'immediate' | 'dom' | 'no-document') => {
+            logger.info(`remote config load triggered via ${reason}`)
+            void this.loadRemoteConfig()
+        }
+
+        if (document) {
+            if (document.readyState === 'loading') {
+                logger.info('remote config load deferred until DOMContentLoaded')
+                const onDomReady = () => {
+                    document?.removeEventListener('DOMContentLoaded', onDomReady)
+                    triggerRemoteConfigLoad('dom')
+                }
+                addEventListener(document, 'DOMContentLoaded', onDomReady, { once: true } as any)
+            } else {
+                triggerRemoteConfigLoad('immediate')
+            }
+        } else {
+            triggerRemoteConfigLoad('no-document')
+        }
         addEventListener(window, 'onpagehide' in self ? 'pagehide' : 'unload', this.capturePageLeave.bind(this), {
             passive: false,
         })
@@ -191,11 +221,20 @@ export class Leanbase extends PostHogCore {
     }
 
     async loadRemoteConfig() {
-        if (!this.isRemoteConfigLoaded) {
+        if (this._remoteConfigLoadAttempted) {
+            return
+        }
+        this._remoteConfigLoadAttempted = true
+
+        try {
             const remoteConfig = await this.reloadRemoteConfigAsync()
             if (remoteConfig) {
                 this.onRemoteConfig(remoteConfig as RemoteConfig)
             }
+        } finally {
+            // Regardless of success/failure, we consider remote config "resolved" so replay isn't blocked forever.
+            this._remoteConfigResolved = true
+            this._maybeStartSessionRecording()
         }
     }
 
@@ -210,6 +249,29 @@ export class Leanbase extends PostHogCore {
         this.isRemoteConfigLoaded = true
         this.replayAutocapture?.onRemoteConfig(config)
         this.sessionRecording?.onRemoteConfig(config)
+
+        // Remote config has been applied; allow replay start if flags are also ready.
+        this._remoteConfigResolved = true
+        this._maybeStartSessionRecording()
+    }
+
+    private _maybeStartSessionRecording(): void {
+        if (this._maybeStartedSessionRecording) {
+            return
+        }
+        if (!this.sessionRecording) {
+            return
+        }
+        if (!this._featureFlagsResolved || !this._remoteConfigResolved) {
+            return
+        }
+
+        this._maybeStartedSessionRecording = true
+        try {
+            this.sessionRecording.startIfEnabledOrStop()
+        } catch (e) {
+            logger.error('Failed to start session recording', e)
+        }
     }
 
     fetch(url: string, options: PostHogFetchOptions): Promise<PostHogFetchResponse> {

package/src/version.ts

@@ -1 +1 @@
-export const version = '0.2.2'
+export const version = '0.2.3'

package/LICENSE

@@ -1,37 +0,0 @@
-Copyright 2025 Leanflag Limited
-This project is a fork of PostHog/posthog-js.
-
-All rights reserved. This software is proprietary to Leanflag Limited and may only be used for Leanflag Limited products and internal purposes. Redistribution, modification, or commercial use outside of Leanflag Limited is strictly prohibited without explicit written permission from Leanflag Limited.
-
-
----
-
-Copyright 2020 Posthog / Hiberly, Inc.
-
-Copyright 2015 Mixpanel, Inc.
-
-
-Some files in this codebase contain code from getsentry/sentry-javascript by Software, Inc. dba Sentry.
-In such cases it is explicitly stated in the file header. This license only applies to the relevant code in such cases. 
-
-MIT License
-
-Copyright (c) 2012 Functional Software, Inc. dba Sentry
-
-Permission is hereby granted, free of charge, to any person obtaining a copy of
-this software and associated documentation files (the "Software"), to deal in
-the Software without restriction, including without limitation the rights to
-use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies
-of the Software, and to permit persons to whom the Software is furnished to do
-so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.