Restructure src and test modules, rename util and type files

Author: aliu39

…ations/featureFlags/launchdarkly/test.ts …/featureFlags/launchdarkly/basic/test.tsdev-packages/browser-integration-tests/suites/integrations/featureFlags/launchdarkly/test.ts renamed to dev-packages/browser-integration-tests/suites/integrations/featureFlags/launchdarkly/basic/test.ts dev-packages/browser-integration-tests/suites/integrations/featureFlags/launchdarkly/test.ts renamed to dev-packages/browser-integration-tests/suites/integrations/featureFlags/launchdarkly/basic/test.ts

@@ -1,11 +1,11 @@
 import { expect } from '@playwright/test';
 
-import { sentryTest } from '../../../../utils/fixtures';
+import { sentryTest } from '../../../../../utils/fixtures';
 
-import { envelopeRequestParser, waitForErrorRequest } from '../../../../utils/helpers';
+import { envelopeRequestParser, waitForErrorRequest } from '../../../../../utils/helpers';
 import { FLAG_BUFFER_SIZE } from '@sentry/browser';
 
-sentryTest('e2e test', async ({ getLocalTestPath, page }) => {
+sentryTest('Basic test with eviction, update, and no async tasks', async ({ getLocalTestPath, page }) => {
   await page.route('https://dsn.ingest.sentry.io/**/*', route => {
     return route.fulfill({
       status: 200,

packages/browser/src/index.ts

@@ -75,5 +75,5 @@ export type { Span } from '@sentry/types';
 export { makeBrowserOfflineTransport } from './transports/offline';
 export { browserProfilingIntegration } from './profiling/integration';
 export { spotlightBrowserIntegration } from './integrations/spotlight';
-export { launchDarklyIntegration, buildLaunchDarklyFlagUsedHandler } from './integrations/launchdarkly';
-export { FLAG_BUFFER_SIZE } from './utils/flags';
+export { launchDarklyIntegration, buildLaunchDarklyFlagUsedHandler } from './integrations/featureFlags/launchdarkly';
+export { FLAG_BUFFER_SIZE } from './utils/featureFlags';

packages/browser/src/integrations/featureFlags/launchdarkly/index.ts

@@ -0,0 +1 @@
+export {launchDarklyIntegration, buildLaunchDarklyFlagUsedHandler} from './integration';

…browser/src/integrations/launchdarkly.ts …featureFlags/launchdarkly/integration.tspackages/browser/src/integrations/launchdarkly.ts renamed to packages/browser/src/integrations/featureFlags/launchdarkly/integration.ts packages/browser/src/integrations/launchdarkly.ts renamed to packages/browser/src/integrations/featureFlags/launchdarkly/integration.ts

@@ -1,7 +1,7 @@
 import type { Client, Event, EventHint, IntegrationFn } from '@sentry/types';
 import type { LDContext, LDEvaluationDetail, LDInspectionFlagUsedHandler } from 'launchdarkly-js-client-sdk';
 
-import { insertToFlagBuffer } from '../utils/flags'
+import { insertToFlagBuffer } from '../../../utils/featureFlags'
 import { defineIntegration, getCurrentScope } from '@sentry/core';
 
 /**
@@ -60,7 +60,6 @@ export function buildLaunchDarklyFlagUsedHandler(): LDInspectionFlagUsedHandler
         }
         const flagBuffer = scopeContexts.flags.values;
         insertToFlagBuffer(flagBuffer, flagKey, flagDetail.value);
-        // TODO: the flag buffer isn't thread-safe, yet this handler and the event processor could access it at the same time.
       }
       return;
     },

packages/browser/src/integrations/featureFlags/launchdarkly/types.ts

@@ -0,0 +1 @@
+//TODO: inline LD types so we can drop it from devDeps

packages/browser/src/utils/flags.ts …ckages/browser/src/utils/featureFlags.tspackages/browser/src/utils/flags.ts renamed to packages/browser/src/utils/featureFlags.ts packages/browser/src/utils/flags.ts renamed to packages/browser/src/utils/featureFlags.ts

@@ -1,5 +1,5 @@
 
-import { insertToFlagBuffer } from '../../src/utils/flags';
+import { insertToFlagBuffer } from '../../src/utils/featureFlags';
 import type { FeatureFlag } from '@sentry/types';
 
 describe('flags', () => {

packages/browser/test/utils/flags.test.ts …/browser/test/utils/featureFlags.test.tspackages/browser/test/utils/flags.test.ts renamed to packages/browser/test/utils/featureFlags.test.ts packages/browser/test/utils/flags.test.ts renamed to packages/browser/test/utils/featureFlags.test.ts

@@ -1,6 +1,6 @@
 import type { Primitive } from './misc';
 import type { SpanOrigin } from './span';
-import type { FeatureFlag } from './flags'
+import type { FeatureFlag } from './featureFlags'
 
 export type Context = Record<string, unknown>;
 

packages/types/src/context.ts

@@ -56,7 +56,7 @@ export type { Event, EventHint, EventType, ErrorEvent, TransactionEvent } from '
 export type { EventProcessor } from './eventprocessor';
 export type { Exception } from './exception';
 export type { Extra, Extras } from './extra';
-export type { FeatureFlag } from './flags';
+export type { FeatureFlag } from './featureFlags';
 // eslint-disable-next-line deprecation/deprecation
 export type { Hub } from './hub';
 export type { Integration, IntegrationClass, IntegrationFn } from './integration';