Skip to content

fix(go): resolve receiver method calls (#830 with main merged) - #868

Merged
tirth8205 merged 7 commits into
mainfrom
pr-830-main-merge
Aug 19, 2026
Merged

fix(go): resolve receiver method calls (#830 with main merged)#868
tirth8205 merged 7 commits into
mainfrom
pr-830-main-merge

Conversation

@tirth8205

Copy link
Copy Markdown
Owner

Carries #830 (author merlincat11) with current main merged in and the test-anchor conflict against #833 resolved by keeping both test blocks. Validated locally: tests/test_multilang.py 464 passed, full suite 2475 passed, ruff and mypy clean.

Opened because the source branch needs a conflict resolution that cannot be pushed to the fork from here. Merging this lands the original commits unchanged, so #830 will show as merged.

Fixes #829.

@github-actions

Copy link
Copy Markdown

code-review-graph review

Overall risk: 0.55 (MEDIUM) — 46 changed function(s)/class(es), 0 affected flow(s), 34 test gap(s)

Risk-scored changes

Risk Level Symbol Location Tested
0.55 medium code_review_graph/parser.py::CodeParser._get_member_call_receiver_method code_review_graph/parser.py:10895 no
0.40 medium code_review_graph/parser.py::CodeParser._extract_from_tree code_review_graph/parser.py:6248 no
0.40 medium code_review_graph/parser.py::CodeParser._go_receiver_binding_index code_review_graph/parser.py:14870 yes
0.40 medium code_review_graph/parser.py::CodeParser.scope_key code_review_graph/parser.py:14878 no
0.40 medium code_review_graph/parser.py::CodeParser._go_receiver_is_shadowed code_review_graph/parser.py:14979 yes
0.35 low code_review_graph/parser.py::CodeParser._go_function_parameters_bind_name code_review_graph/parser.py:14858 no
0.30 low code_review_graph/parser.py::CodeParser code_review_graph/parser.py:2405 yes
0.30 low code_review_graph/parser.py::CodeParser._go_field_binds_name code_review_graph/parser.py:14841 no
0.30 low tests/fixtures/sample_go.go::InMemoryRepo.SaveAndReturn tests/fixtures/sample_go.go:41 no
0.30 low tests/fixtures/sample_go.go::ShadowA tests/fixtures/sample_go.go:45 no

Test gaps

  • code_review_graph/parser.py::CodeParser._resolve_call_targets (code_review_graph/parser.py:4652)
  • code_review_graph/parser.py::CodeParser._extract_from_tree (code_review_graph/parser.py:6248)
  • code_review_graph/parser.py::CodeParser._extract_functions (code_review_graph/parser.py:10310)
  • code_review_graph/parser.py::CodeParser._extract_calls (code_review_graph/parser.py:10640)
  • code_review_graph/parser.py::CodeParser._get_member_call_receiver_method (code_review_graph/parser.py:10895)
  • ...and 29 more without direct tests

Token savings: this graph-backed report used ~202,502 fewer tokens (~96%) than reading every changed file in full (estimated, chars/4 approximation).


Powered by code-review-graph — local-first analysis; no code leaves the CI runner.

@tirth8205
tirth8205 merged commit 0178d3b into main Aug 19, 2026
17 checks passed
@tirth8205
tirth8205 deleted the pr-830-main-merge branch August 19, 2026 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: (go) preserve selector receivers in CALLS edges

2 participants