Skip to content

Commit 81342d1

Browse files
fix(SDK-118): resolve IterableConfig eqeqeq lint warnings
Replace loose != undefined checks with strict !== and remove eslint-disable suppressions on handler presence flags. Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 41b86a6 commit 81342d1

1 file changed

Lines changed: 7 additions & 24 deletions

File tree

src/core/classes/IterableConfig.ts

Lines changed: 7 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -449,47 +449,30 @@ export class IterableConfig {
449449
inAppDisplayInterval: this.inAppDisplayInterval,
450450
/**
451451
* A boolean indicating if a URL handler is present.
452-
*
453-
* TODO: Figure out if this is purposeful
454452
*/
455-
// eslint-disable-next-line eqeqeq
456-
urlHandlerPresent: this.urlHandler != undefined,
453+
urlHandlerPresent: this.urlHandler !== undefined,
457454
/**
458455
* A boolean indicating if a custom action handler is present.
459-
*
460-
* TODO: Figure out if this is purposeful
461456
*/
462-
// eslint-disable-next-line eqeqeq
463-
customActionHandlerPresent: this.customActionHandler != undefined,
457+
customActionHandlerPresent: this.customActionHandler !== undefined,
464458
/**
465459
* A boolean indicating if an in-app handler is present.
466-
*
467-
* TODO: Figure out if this is purposeful
468460
*/
469-
// eslint-disable-next-line eqeqeq
470-
inAppHandlerPresent: this.inAppHandler != undefined,
461+
inAppHandlerPresent: this.inAppHandler !== undefined,
471462
/**
472463
* A boolean indicating if an authentication handler is present.
473-
*
474-
* TODO: Figure out if this is purposeful
475464
*/
476-
// eslint-disable-next-line eqeqeq
477-
authHandlerPresent: this.authHandler != undefined,
465+
authHandlerPresent: this.authHandler !== undefined,
478466
/**
479467
* A boolean indicating if an embedded message update callback is present.
480-
*
481-
* TODO: Figure out if this is purposeful
482468
*/
483-
// eslint-disable-next-line eqeqeq
484-
onEmbeddedMessageUpdatePresent: this.onEmbeddedMessageUpdate != undefined,
469+
onEmbeddedMessageUpdatePresent:
470+
this.onEmbeddedMessageUpdate !== undefined,
485471
/**
486472
* A boolean indicating if an embedded messaging disabled callback is present.
487-
*
488-
* TODO: Figure out if this is purposeful
489473
*/
490-
// eslint-disable-next-line eqeqeq
491474
onEmbeddedMessagingDisabledPresent:
492-
this.onEmbeddedMessagingDisabled != undefined,
475+
this.onEmbeddedMessagingDisabled !== undefined,
493476
/** The log level for the SDK. */
494477
logLevel: this.logLevel,
495478
expiringAuthTokenRefreshPeriod: this.expiringAuthTokenRefreshPeriod,

0 commit comments

Comments
 (0)