Greptile Summary

Adds a two-line early-return guard in handle_call_expr (Rust JS extractor) for callee nodes of kind "this" or "super", mirroring a pre-existing guard in the TypeScript extractor. Without this guard, extract_callback_reference_calls iterated the argument list of this(b) / super(a) and emitted each identifier argument as a dynamic call; the PTS resolver then matched those names globally, generating false cross-file edges.

• The guard is placed after the existing \"import\" check and before extract_call_info, extract_callback_definition, and extract_callback_reference_calls — all three of which are either no-ops or actively harmful for this/super callees.
• Two unit tests verify that neither this(b) nor super(a) emit their argument identifiers as dynamic calls, and that the this call record itself is still produced by the separate collect_this_call_and_bindings path.

Confidence Score: 5/5

Safe to merge — the change is a narrow, well-scoped guard that removes false-positive call edges without affecting any legitimate call recording.

The three functions skipped by the early return are all no-ops or harmful for this/super callees: extract_call_info returns None for non-identifier/member-expression nodes, extract_callback_definition requires a member_expression callee to proceed, and extract_callback_reference_calls was the root cause of the false edges. The this call record continues to be emitted through the separate collect_this_call_and_bindings path, confirmed by the new test. Two focused regression tests cover both cases and the guard cleanly mirrors the existing TypeScript extractor behavior.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[handle_call_expr called] --> B{callee is import?}
    B -- Yes --> C[handle_dynamic_import and return]
    B -- No --> D{callee is this or super?}
    D -- Yes - NEW GUARD --> E[return early - no spurious dynamic calls emitted]
    D -- No --> F[extract_call_info - push call record]
    F --> G[extract_callback_definition - push anon callback def]
    G --> H[extract_callback_reference_calls - push identifier args as dynamic calls]

Reviews (1): Last reviewed commit: "fix(native): skip callback-reference cal..." | Re-trigger Greptile

Codegraph Impact Analysis

3 functions changed → 1 callers affected across 1 files

• handle_call_expr in crates/codegraph-core/src/extractors/javascript.rs:1212 (1 transitive callers)
• this_call_args_do_not_emit_callback_reference_calls in crates/codegraph-core/src/extractors/javascript.rs:4111 (0 transitive callers)
• super_call_args_do_not_emit_callback_reference_calls in crates/codegraph-core/src/extractors/javascript.rs:4132 (0 transitive callers)