perf(flutter): Move Android JNI work to core worker to avoid work on main isolate (#3713) * perf(flutter): Optimize Android scope sync Send large Android scope payloads as JSON bytes instead of recursively constructing Java maps and lists through JNI. This keeps nested user data structured while reducing per-entry JNI object churn. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * docs(flutter): Document JNI payload guidance Clarify that large or arbitrary Dart collection payloads should cross JNI as JSON bytes, while primitives and small controlled payloads can use direct conversion. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * style(flutter): Wrap JNI JSON reader helper Keep the Android JSON reader helper within ktlint formatting limits after adding the scope sync byte-array bridge. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * style(flutter): Apply ktlint when-branch formatting Wrap all native JSON conversion when branches consistently so ktlint accepts the multiline Kotlin helper added for Android scope sync. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * ref(flutter): Reuse native data normalizer Use the existing native boundary normalizer before encoding Android scope payloads as JSON bytes instead of maintaining a second normalization helper. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * fix(flutter): Preserve nulls in Android JSON bridge Keep null values when converting JSON object and array payloads on Android so the bridge remains lossless before native model deserialization. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * perf(flutter): Reuse Android JSON deserializers Cache JSON deserializers for scope sync payloads and replace the broad Any extension with a private helper function for Kotlin JSON conversion. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * ref(flutter): Use SDK JSON object reader Use the Java SDK JSON reader for context payload parsing instead of a custom recursive org.json conversion helper. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * docs(flutter): Clarify JSON reader parsing Document that the Android JSON reader accepts root-level primitives so future changes do not replace it with object-only parsing. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * fix(flutter): Wrap Android primitive contexts Route Android context values through sentry-java's typed overloads so primitive Dart context values are serialized as valid context objects. Regenerate JNI bindings after removing the unused object overload entrypoint. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * fix(flutter): Release Android JNI refs Release replay callback JNI handles after processing replay privacy options, and avoid creating duplicate JNI strings when loading debug images. This reduces leaked global refs in Android replay and debug image paths. Fixes #3696 Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * test(flutter): Expect wrapped Android contexts Update the native context sync test to match the Android bridge's valid serialized shape for primitive context values. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * fix(flutter): Release replay JNI refs safely Use a single arena for replay callback JNI temporaries and avoid releasing map keys before removing privacy options from the payload. Refs #3696 Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * ref(flutter): Move Android JNI work to core worker Route Android native scope and context work through the core worker so JNI calls can run off the main isolate when the worker is available. Keep current-isolate fallbacks for calls made before startup. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * fix(flutter): Harden Android core worker Handle worker request failures without crashing callers and normalize payloads before sending them across isolates. Track JNI strings as they are allocated so partial failures still release native refs. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * fix(flutter): Await Android scope worker sync Return worker request futures for Android scope updates so awaited scope observer calls complete after native scope synchronization finishes. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * fix(flutter): Serialize Android scope updates Route paired Android scope mutations through the same worker queue so call order is preserved across breadcrumb and context updates. Handle worker startup failures without delaying native SDK initialization. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * fix(flutter): Skip Android worker reads after close Return null for Android core worker read paths after shutdown so closed workers do not fall back to JNI work on the main isolate. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * style(flutter): Format Android core worker Apply formatting to the Android core worker and clarify that its internal queue preserves JNI request order. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * ref(flutter): Expand Android worker update helpers Use explicit worker request helpers for each Android scope mutation so request construction and error logging stay local to each operation. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * ref(flutter): Reuse Android scope normalizer Route Android core worker scope payloads through the shared normalizer to avoid broadening behavior in this refactor. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor * ref(flutter): Use normalize for worker payloads Align Android worker scope payload normalization with the shared helper without changing the surrounding worker request flow. Co-Authored-By: GPT-5.5 Co-authored-by: Cursor --------- Co-authored-by: GPT-5.5 Co-authored-by: Cursor

Uh oh!

Uh oh!

Investigate JNI for possible memory leaks and misuse #3696

Description

Description

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions