Repository navigation
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
rigor789
force-pushed
the
feat/ios-structured-logs
branch
from
October 30, 2026 17:59
2fbe7d3 to
eb97eb3
Compare
rigor789
force-pushed
the
feat/ios-structured-logs
branch
from
October 30, 2026 19:29
eb97eb3 to
32d6c68
Compare
rigor789
force-pushed
the
feat/ios-structured-logs
branch
from
October 30, 2026 19:50
32d6c68 to
bfef406
Compare
Devices stream com.apple.os_trace_relay and simulators use log stream, both selected by executable name, so logs survive closing and reopening the app. --log picks the scope: JS output by default, the app's native logs without debug entries with debug, everything with trace or all. Without a new enough ios-device-lib, devices fall back to the syslog relay. Debug builds ask the runtime for chunk markers through logChunkMarkers in the app's package.json, and the CLI stitches split messages back together per process and thread.
rigor789
force-pushed
the
feat/ios-structured-logs
branch
from
October 30, 2026 22:15
bfef406 to
8783785
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Draft until
ios-device-libis released. CI can't compile this against 0.9.6, because the types for the new stream are missing. After NativeScript/ios-device-lib#88 ships, bump the dependency here and mark this ready.What is the current behavior?
App logs on devices come from
syslog_relay, filtered with fragile regexes. Kernel noise such aswlan0:lines gets through, non-ASCII text is escaped, and there is no way to choose how much native output to show. Long messages are truncated byos_logwith<…>.What is the new behavior?
os_trace_relayand simulators uselog stream --style ndjson, both selected by executable name, so logs survive closing and reopening the app with no pid tracking.--logsets the scope:debug: the whole app process without debug entries, like Xcodetrace/all: everythinglogChunkMarkersto the app'spackage.json, and the CLI stitches split messages back together per process and thread. Webpack doesn't touch that file and the key only re-signs, so build caching isn't affected.lib/common/mobile/ios/ios-log-filter.tsis deleted.print()and stderr capture follow in #6181.Testing
Demo app, 11 messages (up to 100 KB, emoji, 800 lines,
console.error, objects) compared exactly against the expected text, on both launches of each run. The full four-PR stack with the runtime PRs:--log debugand--log trace, before stacking--log traceis large: about 11,000–13,000 extra lines per run of the demo app.Part of a stack
Base: #6178. Needs NativeScript/ios-device-lib#88 released, and NativeScript/ios#530 for stitching.