Skip to content

Make LogBox configurable#6007

Open
j0ntz wants to merge 1 commit intodevelopfrom
jon/logbox-config
Open

Make LogBox configurable#6007
j0ntz wants to merge 1 commit intodevelopfrom
jon/logbox-config

Conversation

@j0ntz
Copy link
Copy Markdown
Contributor

@j0ntz j0ntz commented Apr 30, 2026

In preparation for making it easier for agents to control the sim

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

Note

Low Risk
Low risk configuration change that only affects developer warning/error overlays; default behavior is unchanged unless LOGBOX_DISABLE is enabled.

Overview
Adds a new LOGBOX_DISABLE boolean to env.json parsing (envConfig.ts) and uses it at app startup (app.ts) to optionally call LogBox.ignoreAllLogs().

When the flag is not set, the app keeps the existing behavior of only suppressing a small set of known noisy LogBox messages. Updates the CHANGELOG to document the new option.

Reviewed by Cursor Bugbot for commit 4671def. Bugbot is set up for automated code reviews on this repo. Configure here.


In preparation for making it easier for agents to control the sim
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 4671def. Configure here.

Comment thread src/app.ts
'Require cycle:',
'Attempted to end a Span which has already ended.'
])
if (ENV.LOGBOX_DISABLE) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale comment no longer matches code behavior

Low Severity

The comment "Uncomment the next line to remove popup warning/error boxes." is now stale — there is no longer a commented-out line to uncomment. The behavior is instead controlled by ENV.LOGBOX_DISABLE. This violates the remove-stale-comments rule: comments that no longer match reality need to be updated or removed.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 4671def. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant