Skip to content

Consolidate logging and expose it on debug builds#262

Merged
davertay-j merged 4 commits into
mainfrom
dt/logging_consolidation
Jun 8, 2026
Merged

Consolidate logging and expose it on debug builds#262
davertay-j merged 4 commits into
mainfrom
dt/logging_consolidation

Conversation

@davertay-j

@davertay-j davertay-j commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Renames the sub-system for all the various non-js logs as "Topaz" and adds them to the debug build log persistence. So we will have the existing JsMessage logs for all the Js related stuff, and Topaz logs for all the app native stuff. I am leaving WebView logs alone (and not persisted) as they are usually not very useful and crazy noisy.


Note

Low Risk
Logging identifiers and debug plist settings only; no runtime behavior beyond aligning EventBus debug logging with existing app config.

Overview
Unifies native OSLog output under the Topaz subsystem (with per-module categories such as AppModel, BluetoothEngine, EventBus) instead of separate subsystem names per package.

Adds OSLogPreferences for Topaz in Info.plist so debug builds persist Topaz logs at Debug level with private data enabled, matching the existing JsMessage configuration. WebView logging is intentionally unchanged.

Wires EventBus(enableDebugLogging: appConfig.enableDebugLogging) when constructing the Bluetooth stack so event-bus debug logs follow the same app debug flag as the engine and virtual keyboard.

Reviewed by Cursor Bugbot for commit b8a33c1. Configure here.

@davertay-j davertay-j merged commit 277671b into main Jun 8, 2026
3 checks passed
@davertay-j davertay-j deleted the dt/logging_consolidation branch June 8, 2026 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants