Repository navigation
fix: flow-chain + mermaid rendering improvements - #45
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
9cd5da6 to
2e38803
Compare
Merging this PR will degrade performance by 13.2%
|
| Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|
| ❌ | open+ingest |
252.9 ms | 296.5 ms | -14.68% |
| ❌ | warm_no_hit |
545.6 µs | 617.8 µs | -11.69% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing fix/flow-chain-mermaid-v3 (4180f84) with main (2bedb4e)
- Add CALL_ENTER/CALL_EXIT/STMT_END/SWITCH_START/SWITCH_CLOSE markers in extractor
- let_declaration handler: tag innermost call with into_var/is_new_var (NEW)
- assignment_expression handler: tag innermost call with into_var (is_new_var=false), guard with calls_before check
- Switch discriminant extraction fixed for Rust match_expression
- Mermaid: filter CALL_ENTER/CALL_EXIT/STMT_END from node rendering, remap edges, dedup cross-pass
- Mermaid: call label appends ' → {var}' when into_var present
- Mermaid: structural_edges on filtered chain preserves control flow (IF/LOOP/SWITCH)
- Update 21 golden chain tests to new marker format
- Add tests: rust_let_declaration, rust_multiple_let_declarations, rust_let_declaration_method_chain, rust_assignment_expression_no_call_rhs, rust_assignment_expression_with_call_rhs
All 44 chain tests + 21 mermaid/api tests pass.
…lizers; fix clippy unused_mut
2e38803 to
f2923b6
Compare
Codecov Report❌ Patch coverage is ❌ Your patch status has failed because the patch coverage (94.96%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #45 +/- ##
==========================================
+ Coverage 77.93% 78.11% +0.17%
==========================================
Files 96 96
Lines 23067 23299 +232
==========================================
+ Hits 17978 18200 +222
- Misses 5089 5099 +10 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
dd13bff to
40e3b3f
Compare
Summary
Complete overhaul of call-chain extraction and Mermaid flow-chart rendering to show clear structure:
Extractor improvements
Mermaid rendering
Tests
Verified via MCP