review comments

Author: aliu39

dev-packages/browser-integration-tests/suites/integrations/featureFlags/launchdarkly/basic/test.ts

@@ -22,7 +22,7 @@ sentryTest('Basic test with eviction, update, and no async tasks', async ({ getL
   const url = await getLocalTestUrl({ testDir: __dirname, skipDsnRouteHandler: true });
   await page.goto(url);
 
-  await page.waitForFunction(bufferSize => {
+  await page.evaluate(bufferSize => {
     const ldClient = (window as any).initializeLD();
     for (let i = 1; i <= bufferSize; i++) {
       ldClient.variation(`feat${i}`, false);

dev-packages/browser-integration-tests/suites/integrations/featureFlags/launchdarkly/withScope/test.ts

@@ -25,7 +25,7 @@ sentryTest('Flag evaluations in forked scopes are stored separately.', async ({
   const forkedReqPromise = waitForErrorRequest(page, event => !!event.tags && event.tags.isForked === true);
   const mainReqPromise = waitForErrorRequest(page, event => !!event.tags && event.tags.isForked === false);
 
-  await page.waitForFunction(() => {
+  await page.evaluate(() => {
     const Sentry = (window as any).Sentry;
     const errorButton = document.querySelector('#error') as HTMLButtonElement;
     const ldClient = (window as any).initializeLD();

dev-packages/browser-integration-tests/suites/integrations/featureFlags/openfeature/withScope/test.ts

@@ -25,7 +25,7 @@ sentryTest('Flag evaluations in forked scopes are stored separately.', async ({
   const forkedReqPromise = waitForErrorRequest(page, event => !!event.tags && event.tags.isForked === true);
   const mainReqPromise = waitForErrorRequest(page, event => !!event.tags && event.tags.isForked === false);
 
-  await page.waitForFunction(() => {
+  await page.evaluate(() => {
     const Sentry = (window as any).Sentry;
     const errorButton = document.querySelector('#error') as HTMLButtonElement;
     const client = (window as any).initialize();

packages/browser/src/index.ts

@@ -68,6 +68,5 @@ export type { Span } from '@sentry/core';
 export { makeBrowserOfflineTransport } from './transports/offline';
 export { browserProfilingIntegration } from './profiling/integration';
 export { spotlightBrowserIntegration } from './integrations/spotlight';
-export { copyFlagsFromScopeToEvent, insertFlagToScope } from './utils/featureFlags';
 export { launchDarklyIntegration, buildLaunchDarklyFlagUsedHandler } from './integrations/featureFlags/launchdarkly';
 export { openFeatureIntegration, OpenFeatureIntegrationHook } from './integrations/featureFlags/openfeature';