Skip to content

Make LogBox configurable#6007

Merged
j0ntz merged 1 commit intodevelopfrom
jon/logbox-config
May 8, 2026
Merged

Make LogBox configurable#6007
j0ntz merged 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: adds a new env flag that only affects developer-facing LogBox warning/error popups and slightly tightens linting coverage for src/app.ts.

Overview
Makes React Native LogBox behavior configurable via a new ENV.LOGBOX_DISABLE flag (wired through envConfig.ts), allowing environments to fully suppress LogBox popups when desired.

Updates app startup to either ignoreAllLogs() when disabled or continue ignoring only a small set of known noisy warnings, and records the new option in CHANGELOG.md.

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


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
Copy link
Copy Markdown
Contributor

@peachbits peachbits left a comment

Choose a reason for hiding this comment

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

Please remove or update the comment per bugbot's comment before merging

@j0ntz j0ntz force-pushed the jon/logbox-config branch from 4671def to 5838a1a Compare May 8, 2026 19:49
@j0ntz
Copy link
Copy Markdown
Contributor Author

j0ntz commented May 8, 2026

Updated the stale comment in src/app.ts per bugbot's note (fixup folded into 5838a1a).

In preparation for making it easier for agents to control the sim
@j0ntz j0ntz force-pushed the jon/logbox-config branch from 5838a1a to 08e0bd9 Compare May 8, 2026 20:20
@j0ntz j0ntz merged commit 659966b into develop May 8, 2026
4 of 5 checks passed
@j0ntz j0ntz deleted the jon/logbox-config branch May 8, 2026 20:26
j0ntz pushed a commit that referenced this pull request May 8, 2026
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.

2 participants