From e1ff938af4d224c8ad78ed5c3f21e0af8ec05e06 Mon Sep 17 00:00:00 2001
From: Chandra Penke <chandrapenke@gmail.com>
Date: Mon, 27 Nov 2023 18:28:23 -0800
Subject: [PATCH 1/2] fix: honor disableIdle flag

Closes #576
---
 docs/generated/changelog.html     |  4 +++-
 packages/auth-client/src/index.ts | 25 +++++++++++++------------
 2 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/docs/generated/changelog.html b/docs/generated/changelog.html
index 4a8fb5916..1e4259db5 100644
--- a/docs/generated/changelog.html
+++ b/docs/generated/changelog.html
@@ -11,7 +11,9 @@ <h1>Agent-JS Changelog</h1>
 
     <section>
       <h2>Version x.x.x</h2>
-      <ul></ul>
+      <ul>
+        <li>fix: honor disableIdle flag</li>
+      </ul>
       <h2>Version 0.20.2</h2>
       <ul>
         <li>chore: lowering prettier version for CI</li>
diff --git a/packages/auth-client/src/index.ts b/packages/auth-client/src/index.ts
index 2b73fe829..12048dfb5 100644
--- a/packages/auth-client/src/index.ts
+++ b/packages/auth-client/src/index.ts
@@ -329,15 +329,19 @@ export class AuthClient {
     // The event handler for processing events from the IdP.
     private _eventHandler?: (event: MessageEvent) => void,
   ) {
-    const logout = this.logout.bind(this);
-    const idleOptions = _createOptions?.idleOptions;
+    this._registerDefaultIdleCallback();
+  }
+
+  private _registerDefaultIdleCallback() {
+    const idleOptions = this._createOptions?.idleOptions;
     /**
      * Default behavior is to clear stored identity and reload the page.
      * By either setting the disableDefaultIdleCallback flag or passing in a custom idle callback, we will ignore this config
      */
     if (!idleOptions?.onIdle && !idleOptions?.disableDefaultIdleCallback) {
       this.idleManager?.registerCallback(() => {
-        logout();
+        console.log('User is idle. Logging out.');
+        this.logout();
         location.reload();
       });
     }
@@ -372,17 +376,14 @@ export class AuthClient {
     this._identity = DelegationIdentity.fromDelegation(key, this._chain);
 
     this._idpWindow?.close();
-    if (!this.idleManager) {
-      const idleOptions = this._createOptions?.idleOptions;
+    const idleOptions = this._createOptions?.idleOptions;
+    // create the idle manager on a successful login if we haven't disabled it
+    // and it doesn't already exist.
+    if (!this.idleManager && !idleOptions?.disableIdle) {
       this.idleManager = IdleManager.create(idleOptions);
-
-      if (!idleOptions?.onIdle && !idleOptions?.disableDefaultIdleCallback) {
-        this.idleManager?.registerCallback(() => {
-          this.logout();
-          location.reload();
-        });
-      }
+      this._registerDefaultIdleCallback();
     }
+
     this._removeEventListener();
     delete this._idpWindow;
 

From 64068381fa9ee0d5437643547cbc6bdbb2b3aabe Mon Sep 17 00:00:00 2001
From: Chandra Penke <chandrapenke@gmail.com>
Date: Thu, 30 Nov 2023 23:12:54 -0800
Subject: [PATCH 2/2] fix: remove logging

Co-authored-by: Kyle Peacock <kylpeacock@gmail.com>
---
 packages/auth-client/src/index.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/packages/auth-client/src/index.ts b/packages/auth-client/src/index.ts
index 12048dfb5..dd9a89152 100644
--- a/packages/auth-client/src/index.ts
+++ b/packages/auth-client/src/index.ts
@@ -340,7 +340,6 @@ export class AuthClient {
      */
     if (!idleOptions?.onIdle && !idleOptions?.disableDefaultIdleCallback) {
       this.idleManager?.registerCallback(() => {
-        console.log('User is idle. Logging out.');
         this.logout();
         location.reload();
       });