Skip to content

fix(health): exclude Rust inline test code from perf findings - #1855

Open
NullSense wants to merge 1 commit into
repowise-dev:mainfrom
NullSense:fix/rust-inline-test-perf-false-positives
Open

fix(health): exclude Rust inline test code from perf findings#1855
NullSense wants to merge 1 commit into
repowise-dev:mainfrom
NullSense:fix/rust-inline-test-perf-false-positives

fix(health): exclude Rust inline test code from perf findings

71e56dc
Select commit
Loading
Failed to load commit list.
Repowise Bot / Repowise / code health succeeded Aug 28, 2026 in 0s

Health gate passed

Repository health 7.3 -> 7.3. No configured gate rule tripped.

Details

Symbols changed (blast radius)

  • packages/core/src/repowise/core/analysis/health/complexity/walker.py::_rust_test_line_ranges: added, 0 caller(s) outside this PR
  • packages/core/src/repowise/core/analysis/health/perf/gated.py::_in_rust_test_range: added, 0 caller(s) outside this PR
  • packages/core/src/repowise/core/analysis/health/complexity/walker.py::walk_file: body, 57 caller(s) outside this PR: packages/core/src/repowise/core/analysis/health/engine.py::HealthAnalyzer::_walk, packages/core/src/repowise/core/analysis/health/refactoring/llm/enrich.py::_validate_extract_class, packages/core/src/repowise/core/analysis/health/refactoring/llm/enrich.py::_validate_extract_method, packages/core/src/repowise/core/analysis/health/refactoring/llm/enrich.py::_validate_split_file, tests/unit/health/test_complexity_walker.py::_walk_classes, tests/unit/health/test_complexity_walker.py::test_javascript_module_level_arrow_callback_is_function_entry, tests/unit/health/test_complexity_walker.py::test_javascript_test_suite_wrapper_callback_is_not_function_entry, tests/unit/health/test_complexity_walker.py::test_pascal_else_if_chain_is_flat_not_nested, tests/unit/health/test_complexity_walker.py::test_pascal_no_class_metrics, tests/unit/health/test_complexity_walker.py::test_python_comment_in_parameter_list_not_counted, tests/unit/health/test_complexity_walker.py::test_tcc_cohesive_vs_split, tests/unit/health/test_complexity_walker.py::test_unsupported_language_has_no_classes, tests/unit/health/test_error_handling.py::_hits, tests/unit/health/test_ff_track_aefg_perf.py::_hits, tests/unit/health/test_ff_track_aefg_perf.py::_kinds, tests/unit/health/test_ff_track_d_exceptions.py::_kinds, tests/unit/health/test_ff_track_iljk_structural.py::test_god_class_reason_quotes_the_brain_methods_own_ccn, tests/unit/health/test_ff_track_iljk_structural.py::test_large_method_fires_on_match_dispatch, tests/unit/health/test_ff_track_iljk_structural.py::test_large_method_still_fires_on_real_branching, tests/unit/health/test_file_nloc.py::test_health_metric_nloc_uses_file_nloc, tests/unit/health/test_file_nloc.py::test_js_file_nloc_excludes_comment_only, tests/unit/health/test_file_nloc.py::test_js_route_file_nloc_excludes_one_comment_line, tests/unit/health/test_file_nloc.py::test_js_route_function_sum_less_than_file_nloc, tests/unit/health/test_file_nloc.py::test_multiline_string_blank_line_does_not_count, tests/unit/health/test_file_nloc.py::test_python_file_nloc_excludes_comment_only, tests/unit/health/test_file_nloc.py::test_trailing_comment_line_still_counts, tests/unit/health/test_file_nloc.py::test_unsupported_language_fallback_nloc, tests/unit/health/test_perf_crossfn.py::_build, tests/unit/health/test_perf_crossfn.py::_facts, tests/unit/health/test_perf_dialects.py::_hits, tests/unit/health/test_perf_dialects.py::_walk, tests/unit/health/test_perf_dialects.py::test_cpp_function_name_resolves_through_the_declarator_chain, tests/unit/health/test_perf_dialects.py::test_cpp_same_collection_nested_range_for_fact, tests/unit/health/test_perf_dialects.py::test_kotlin_same_collection_nested_combinators_fact, tests/unit/health/test_perf_dialects.py::test_python_filesystem_sink_reaches_the_cross_function_bridge, tests/unit/health/test_perf_dialects.py::test_python_point_reads_do_not_become_hot_path_candidates, tests/unit/health/test_perf_dialects.py::test_ruby_same_collection_nested_block_loops_fact, tests/unit/health/test_perf_dialects.py::test_scala_same_collection_nested_loop_fact, tests/unit/health/test_perf_io_in_loop.py::_walk, tests/unit/health/test_perf_io_in_loop.py::test_perf_pass_cases
  • packages/core/src/repowise/core/analysis/health/complexity/models.py::FileComplexity: body, 11 caller(s) outside this PR: packages/core/src/repowise/core/analysis/health/engine.py::HealthAnalyzer::_walk, packages/core/src/repowise/core/analysis/health/engine.py::HealthAnalyzer::analyze, packages/core/src/repowise/core/analysis/health/engine.py::HealthAnalyzer::analyze_async, packages/core/src/repowise/core/analysis/health/sql_complexity.py::walk_sql_file, tests/unit/health/test_dataflow_facts.py::test_one_parse_when_promotion_and_extract_method_hit_same_file, tests/unit/health/test_perf_crossfn.py::test_call_site_identity_prevents_same_name_first_hop_collision, tests/unit/health/test_perf_crossfn.py::test_dispatch_edge_connects_a_resolved_call_to_its_sink, tests/unit/health/test_perf_crossfn.py::test_unreliable_call_cannot_create_a_cross_function_finding, tests/unit/health/test_perf_crossfn.py::test_unresolved_loop_call_does_not_reuse_same_name_outside_loop, tests/unit/health/test_perf_promotion_engine.py::_walked_entry, tests/unit/health/test_perf_promotion_engine.py::test_unsupported_language_degrades_to_silence
  • packages/core/src/repowise/core/analysis/health/perf/gated.py::collect_centrality_gated: body, 3 caller(s) outside this PR: packages/core/src/repowise/core/analysis/health/engine.py::HealthAnalyzer::_apply_crossfn_perf, tests/unit/health/test_perf_phase7b.py::test_centrality_gate_fires_in_a_churny_file, tests/unit/health/test_perf_phase7b.py::test_centrality_gate_silent_without_any_hot_signal

Review priority (recent bug-fix history)

  • packages/core/src/repowise/core/analysis/health/complexity/walker.py: fixed 3x in the last ~6 months
  • packages/core/src/repowise/core/analysis/health/complexity/models.py: fixed once in the last ~6 months

Touched hotspots (blast radius)

  • packages/core/src/repowise/core/analysis/health/perf/gated.py: 5 commits/90d, 2 dependents, owner Raghav Chamadiya
  • packages/core/src/repowise/core/analysis/health/complexity/walker.py: 17 commits/90d, 9 dependents, owner Raghav Chamadiya
  • packages/core/src/repowise/core/analysis/health/complexity/models.py: 9 commits/90d, 5 dependents, owner Raghav Chamadiya

Hidden coupling (co-change partners not in this PR)

  • packages/core/src/repowise/core/analysis/health/complexity/walker.py co-changes with packages/core/src/repowise/core/analysis/health/engine.py (6x)
  • packages/core/src/repowise/core/analysis/health/complexity/walker.py co-changes with packages/core/src/repowise/core/analysis/health/biomarkers/registry.py (6x)
  • packages/core/src/repowise/core/analysis/health/complexity/walker.py co-changes with packages/core/src/repowise/core/analysis/health/scoring.py (6x)
  • packages/core/src/repowise/core/analysis/health/complexity/walker.py co-changes with packages/core/src/repowise/core/analysis/health/complexity/languages.py (5x)
  • packages/core/src/repowise/core/analysis/health/perf/gated.py co-changes with packages/core/src/repowise/core/analysis/health/engine.py (4x)

Change risk

  • Raw change-risk score: 8.8/10 (banded moderate, riskier than 52% of this repo's commits)
  • Driven by: more lines added than baseline
  • Driven by: more scattered than baseline