Skip to content

Conversation

ntindle
Copy link
Member

@ntindle ntindle commented Oct 3, 2025

We struggle to identify where issues are coming from feature flags and which are from normal use. This adds that split on the frontend.

Changes ๐Ÿ—๏ธ

Include sentry in the LD initialization

Checklist ๐Ÿ“‹

For code changes:

  • I have clearly listed my changes in the PR description
  • I have made a test plan
  • I have tested my changes according to the test plan:
    • Test that launch darkly flags get attached to the frontend (browser only)

@ntindle ntindle requested a review from a team as a code owner October 3, 2025 18:47
@ntindle ntindle requested review from Swiftyos and Pwuts and removed request for a team October 3, 2025 18:47
@github-project-automation github-project-automation bot moved this to ๐Ÿ†• Needs initial review in AutoGPT development kanban Oct 3, 2025
@github-actions github-actions bot added the platform/frontend AutoGPT Platform - Front end label Oct 3, 2025
Copy link

netlify bot commented Oct 3, 2025

โœ… Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
๐Ÿ”จ Latest commit fc05fe1
๐Ÿ” Latest deploy log https://app.netlify.com/projects/auto-gpt-docs-dev/deploys/68e01a52a25dc60008e762de

Copy link

qodo-merge-pro bot commented Oct 3, 2025

PR Reviewer Guide ๐Ÿ”

Here are some key observations to aid the review process:

โฑ๏ธย Estimated effort to review: 2 ๐Ÿ”ต๐Ÿ”ตโšชโšชโšช
๐Ÿงชย No relevant tests
๐Ÿ”’ย No security concerns identified
โšกย Recommended focus areas for review

Configuration Validity

Verify that the Sentry LaunchDarkly integration is compatible with the current Sentry SDK version and is correctly initialized in the browser bundle; ensure no SSR contexts attempt to use it.

integrations: [
  Sentry.captureConsoleIntegration(),
  Sentry.extraErrorDataIntegration(),
  Sentry.browserProfilingIntegration(),
  Sentry.httpClientIntegration(),
  Sentry.launchDarklyIntegration(),
  Sentry.replayIntegration({
    unmask: [".sentry-unmask, [data-sentry-unmask]"],
Runtime Availability

Confirm that Sentry is always available on the client when building the LaunchDarkly inspectors; guard for environments where Sentry may be disabled to avoid runtime errors.

options={{
  bootstrap: "localStorage",
  inspectors: [Sentry.buildLaunchDarklyFlagUsedHandler()],
}}
Behavior Change

Adding the LaunchDarkly inspector may increase event volume to Sentry; validate performance and sampling settings so flag-used breadcrumbs donโ€™t flood reports.

options={{
  bootstrap: "localStorage",
  inspectors: [Sentry.buildLaunchDarklyFlagUsedHandler()],
}}

@github-actions github-actions bot added the size/s label Oct 3, 2025
Copy link

netlify bot commented Oct 3, 2025

โœ… Deploy Preview for auto-gpt-docs canceled.

Name Link
๐Ÿ”จ Latest commit fc05fe1
๐Ÿ” Latest deploy log https://app.netlify.com/projects/auto-gpt-docs/deploys/68e01a523eef2e0007cc071a

Copy link

coderabbitai bot commented Oct 3, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

โœจ Finishing touches
๐Ÿงช Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ntindle/secrt-1680-add-launch-darkly-flag-status-to-sentry-reports-frontend

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

deepsource-io bot commented Oct 3, 2025

Here's the code health analysis summary for commits 7f8cf36..fc05fe1. View details on DeepSourceย โ†—.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScriptโœ…ย SuccessView Checkย โ†—
DeepSource Python LogoPythonโœ…ย SuccessView Checkย โ†—

๐Ÿ’ก If youโ€™re a repository administrator, you can configure the quality gates from the settings.

@ntindle ntindle requested review from 0ubbe and majdyz October 3, 2025 18:49
@github-project-automation github-project-automation bot moved this from ๐Ÿ†• Needs initial review to ๐Ÿ‘๐Ÿผ Mergeable in AutoGPT development kanban Oct 6, 2025
@ntindle ntindle added this pull request to the merge queue Oct 6, 2025
Merged via the queue into dev with commit b86aa8b Oct 6, 2025
34 checks passed
@ntindle ntindle deleted the ntindle/secrt-1680-add-launch-darkly-flag-status-to-sentry-reports-frontend branch October 6, 2025 14:01
@github-project-automation github-project-automation bot moved this from ๐Ÿ‘๐Ÿผ Mergeable to โœ… Done in AutoGPT development kanban Oct 6, 2025
@github-project-automation github-project-automation bot moved this to Done in Frontend Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: โœ… Done
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants