fix: Do not reexport integration setup functions

getsentry · Oct 24, 2018 · 7c8aad3 · 7c8aad3
1 parent 839326f
commit 7c8aad3
Show file tree

Hide file tree

Showing 3 changed files with 99 additions and 99 deletions.
diff --git a/packages/core/src/baseclient.ts b/packages/core/src/baseclient.ts
@@ -16,7 +16,7 @@ import { consoleSandbox, uuid4 } from '@sentry/utils/misc';
 import { truncate } from '@sentry/utils/string';
 import { BackendClass } from './basebackend';
 import { Dsn } from './dsn';
-import { IntegrationIndex, setupIntegrations } from './integrations';
+import { IntegrationIndex, setupIntegrations } from './integration';
 import { Backend, Client, Options } from './interfaces';
 
 /**

diff --git a/packages/core/src/integration.ts b/packages/core/src/integration.ts
@@ -0,0 +1,98 @@
+import { Integration } from '@sentry/types';
+import { logger } from '@sentry/utils/logger';
+import { Options } from './interfaces';
+
+export const installedIntegrations: string[] = [];
+
+/** Map of integrations assigned to a client */
+export interface IntegrationIndex {
+  [key: string]: Integration;
+}
+
+/** Gets integration to install */
+export function getIntegrationsToSetup(options: Options): Integration[] {
+  const defaultIntegrations = (options.defaultIntegrations && [...options.defaultIntegrations]) || [];
+  const userIntegrations = options.integrations;
+  let integrations: Integration[] = [];
+  if (Array.isArray(userIntegrations)) {
+    const userIntegrationsNames = userIntegrations.map(i => i.name);
+    const pickedIntegrationsNames = [];
+
+    // Leave only unique default integrations, that were not overridden with provided user integrations
+    for (const defaultIntegration of defaultIntegrations) {
+      if (
+        userIntegrationsNames.indexOf(getIntegrationName(defaultIntegration)) === -1 &&
+        pickedIntegrationsNames.indexOf(getIntegrationName(defaultIntegration)) === -1
+      ) {
+        integrations.push(defaultIntegration);
+        pickedIntegrationsNames.push(getIntegrationName(defaultIntegration));
+      }
+    }
+
+    // Don't add same user integration twice
+    for (const userIntegration of userIntegrations) {
+      if (pickedIntegrationsNames.indexOf(getIntegrationName(userIntegration)) === -1) {
+        integrations.push(userIntegration);
+        pickedIntegrationsNames.push(getIntegrationName(userIntegration));
+      }
+    }
+  } else if (typeof userIntegrations === 'function') {
+    integrations = userIntegrations(defaultIntegrations);
+    integrations = Array.isArray(integrations) ? integrations : [integrations];
+  } else {
+    return [...defaultIntegrations];
+  }
+
+  return integrations;
+}
+
+/** Setup given integration */
+export function setupIntegration(integration: Integration, options: Options): void {
+  if (installedIntegrations.indexOf(getIntegrationName(integration)) !== -1) {
+    return;
+  }
+
+  try {
+    integration.setupOnce();
+  } catch (_Oo) {
+    /** @deprecated */
+    // TODO: Remove in v5
+    logger.warn(`Integration ${getIntegrationName(integration)}: The install method is deprecated. Use "setupOnce".`);
+
+    // tslint:disable:deprecation
+    if (integration.install) {
+      integration.install(options);
+    }
+    // tslint:enable:deprecation
+  }
+
+  installedIntegrations.push(getIntegrationName(integration));
+  logger.log(`Integration installed: ${getIntegrationName(integration)}`);
+}
+
+/**
+ * Given a list of integration instances this installs them all. When `withDefaults` is set to `true` then all default
+ * integrations are added unless they were already provided before.
+ * @param integrations array of integration instances
+ * @param withDefault should enable default integrations
+ */
+export function setupIntegrations<O extends Options>(options: O): IntegrationIndex {
+  const integrations: IntegrationIndex = {};
+  getIntegrationsToSetup(options).forEach(integration => {
+    integrations[getIntegrationName(integration)] = integration;
+    setupIntegration(integration, options);
+  });
+  return integrations;
+}
+
+/**
+ * Returns the integration static id.
+ * @param integration Integration to retrieve id
+ */
+function getIntegrationName(integration: Integration): string {
+  /**
+   * @depracted
+   */
+  // tslint:disable-next-line:no-unsafe-any
+  return (integration as any).constructor.id || integration.name;
+}
diff --git a/packages/core/src/integrations/index.ts b/packages/core/src/integrations/index.ts
@@ -1,105 +1,7 @@
-// tslint:disable:deprecation
-import { Integration } from '@sentry/types';
-import { logger } from '@sentry/utils/logger';
-import { Options } from '../interfaces';
-
 export { Dedupe } from './dedupe';
 export { FunctionToString } from './functiontostring';
 export { SDKInformation } from './sdkinformation';
 export { InboundFilters } from './inboundfilters';
 
 export { Debug } from './pluggable/debug';
 export { RewriteFrames } from './pluggable/rewriteframes';
-
-export const installedIntegrations: string[] = [];
-
-/** Map of integrations assigned to a client */
-export interface IntegrationIndex {
-  [key: string]: Integration;
-}
-
-/** Gets integration to install */
-export function getIntegrationsToSetup(options: Options): Integration[] {
-  const defaultIntegrations = (options.defaultIntegrations && [...options.defaultIntegrations]) || [];
-  const userIntegrations = options.integrations;
-  let integrations: Integration[] = [];
-  if (Array.isArray(userIntegrations)) {
-    const userIntegrationsNames = userIntegrations.map(i => i.name);
-    const pickedIntegrationsNames = [];
-
-    // Leave only unique default integrations, that were not overridden with provided user integrations
-    for (const defaultIntegration of defaultIntegrations) {
-      if (
-        userIntegrationsNames.indexOf(getIntegrationName(defaultIntegration)) === -1 &&
-        pickedIntegrationsNames.indexOf(getIntegrationName(defaultIntegration)) === -1
-      ) {
-        integrations.push(defaultIntegration);
-        pickedIntegrationsNames.push(getIntegrationName(defaultIntegration));
-      }
-    }
-
-    // Don't add same user integration twice
-    for (const userIntegration of userIntegrations) {
-      if (pickedIntegrationsNames.indexOf(getIntegrationName(userIntegration)) === -1) {
-        integrations.push(userIntegration);
-        pickedIntegrationsNames.push(getIntegrationName(userIntegration));
-      }
-    }
-  } else if (typeof userIntegrations === 'function') {
-    integrations = userIntegrations(defaultIntegrations);
-    integrations = Array.isArray(integrations) ? integrations : [integrations];
-  } else {
-    return [...defaultIntegrations];
-  }
-
-  return integrations;
-}
-
-/** Setup given integration */
-export function setupIntegration(integration: Integration, options: Options): void {
-  if (installedIntegrations.indexOf(getIntegrationName(integration)) !== -1) {
-    return;
-  }
-
-  try {
-    integration.setupOnce();
-  } catch (_Oo) {
-    /** @deprecated */
-    // TODO: Remove in v5
-    logger.warn(`Integration ${getIntegrationName(integration)}: The install method is deprecated. Use "setupOnce".`);
-
-    if (integration.install) {
-      integration.install(options);
-    }
-  }
-
-  installedIntegrations.push(getIntegrationName(integration));
-  logger.log(`Integration installed: ${getIntegrationName(integration)}`);
-}
-
-/**
- * Given a list of integration instances this installs them all. When `withDefaults` is set to `true` then all default
- * integrations are added unless they were already provided before.
- * @param integrations array of integration instances
- * @param withDefault should enable default integrations
- */
-export function setupIntegrations<O extends Options>(options: O): IntegrationIndex {
-  const integrations: IntegrationIndex = {};
-  getIntegrationsToSetup(options).forEach(integration => {
-    integrations[getIntegrationName(integration)] = integration;
-    setupIntegration(integration, options);
-  });
-  return integrations;
-}
-
-/**
- * Returns the integration static id.
- * @param integration Integration to retrieve id
- */
-function getIntegrationName(integration: Integration): string {
-  /**
-   * @depracted
-   */
-  // tslint:disable-next-line:no-unsafe-any
-  return (integration as any).constructor.id || integration.name;
-}