fix: sendLogToJavaScript

[RohitKushvaha01]

Fixed crash from: #2106

Related to #2105

[greptile-apps]

Greptile Summary

This PR fixes a crash in sendLogToJavaScript by adding a typeof window.log === 'function' guard in the evaluated JavaScript (so an undefined window.log falls back to console.log), correcting the lambda block syntax, and wrapping the evaluateJavascript call in a try-catch inside the Runnable.

• The JavaScript-level guard is the substantive fix: it prevents a JS TypeError when window.log has not yet been defined at evaluation time.
• The lambda is now properly block-form (() -> { ... }), resolving the previous compile error; the try-catch is correctly placed inside the lambda so it guards the actual execution.
• The Log.e call on the catch path passes e.getMessage() which can be null; prefer Log.e(TAG, msg, e) to capture the full stack trace.

Confidence Score: 4/5

The fix is now compilable and the immediate crash is addressed, but the WebView cast path has a silent-failure mode that could drop log messages without indication.

The core JavaScript-level fix (the typeof guard) addresses the root crash. The lambda syntax and try-catch placement are now correct. The one remaining concern is that if the android.webkit.WebView cast fails at runtime, log messages are silently swallowed — the catch block only logs internally and gives no indication to callers that JS evaluation was skipped entirely.

src/plugins/system/android/com/foxdebug/system/System.java — specifically the WebView cast and error handling in sendLogToJavaScript

Important Files Changed

Filename	Overview
src/plugins/system/android/com/foxdebug/system/System.java	sendLogToJavaScript now guards against undefined window.log via a typeof check in the JS string, fixes the lambda block syntax, and wraps the WebView evaluation in a try-catch inside the runnable. Minor: Log.e call uses getMessage() which can be null.

Sequence Diagram

sequenceDiagram
    participant BG as Background Thread
    participant UI as UI Thread
    participant WV as WebView (JS)

    BG->>UI: cordova.getActivity().runOnUiThread(lambda)
    UI->>UI: "try { cast to android.webkit.WebView }"
    alt cast succeeds
        UI->>WV: evaluateJavascript(js, null)
        WV->>WV: "if typeof window.log === function window.log else console.log"
    else cast fails (ClassCastException)
        UI->>UI: catch(Exception e) Log.e(TAG, message)
    end

Loading

_{Reviews (4): Last reviewed commit: "fix: lamda issue" | Re-trigger Greptile}

[RohitKushvaha01]

@greptileai

[RohitKushvaha01]

@greptileai

[RohitKushvaha01]

@greptileai