refactor(shared): reuse shared CI env-var list in telemetry notice

Author: jacekradko

packages/shared/src/__tests__/telemetry.notice.spec.ts

@@ -4,21 +4,9 @@
 import { afterEach, beforeEach, describe, expect, test, vi } from 'vitest';
 
 import { __resetTelemetryNoticeForTests, maybeShowTelemetryNotice } from '../telemetry/notice';
+import { automatedEnvironmentVariables } from '../utils/runtimeEnvironment';
 
-const CI_VARS = [
-  'CI',
-  'CONTINUOUS_INTEGRATION',
-  'BUILD_NUMBER',
-  'GITHUB_ACTIONS',
-  'GITLAB_CI',
-  'CIRCLECI',
-  'TRAVIS',
-  'BUILDKITE',
-  'JENKINS_URL',
-  'TF_BUILD',
-  'DRONE',
-  'CODEBUILD_BUILD_ID',
-];
+const CI_VARS = automatedEnvironmentVariables;
 
 function clearCIEnv() {
   for (const name of CI_VARS) {

packages/shared/src/telemetry/notice.ts

@@ -25,7 +25,7 @@
  * the SDK.
  */
 
-import { isTruthy } from '../underscore';
+import { isAutomatedEnvironment } from '../utils/runtimeEnvironment';
 
 const PROCESS_FLAG = Symbol.for('@clerk/shared.telemetryNoticeShown');
 
@@ -35,21 +35,6 @@ const NOTICE_LINES = [
   'To learn more, including how to opt-out from the telemetry program, visit: https://clerk.com/docs/telemetry.',
 ];
 
-const CI_ENV_VARS = [
-  'CI',
-  'CONTINUOUS_INTEGRATION',
-  'BUILD_NUMBER',
-  'GITHUB_ACTIONS',
-  'GITLAB_CI',
-  'CIRCLECI',
-  'TRAVIS',
-  'BUILDKITE',
-  'JENKINS_URL',
-  'TF_BUILD',
-  'DRONE',
-  'CODEBUILD_BUILD_ID',
-];
-
 function isServerRuntime(): boolean {
   // Skip in browsers.
   if (typeof window !== 'undefined') {
@@ -64,13 +49,6 @@ function isServerRuntime(): boolean {
   return true;
 }
 
-function isCI(): boolean {
-  if (typeof process === 'undefined' || !process.env) {
-    return false;
-  }
-  return CI_ENV_VARS.some(name => isTruthy(process.env[name]));
-}
-
 function hasSeen(): boolean {
   return Boolean((globalThis as Record<symbol, unknown>)[PROCESS_FLAG]);
 }
@@ -110,7 +88,7 @@ export function maybeShowTelemetryNotice(options: MaybeShowTelemetryNoticeOption
     if (!isServerRuntime()) {
       return;
     }
-    if (isCI()) {
+    if (isAutomatedEnvironment()) {
       return;
     }
     if (hasSeen()) {