Skip to content

Fix slowness issue when performing codegraph context - #35

Merged
hungpham10 merged 5 commits into
mainfrom
fix/fix-slowness-issue-when-performing-codegraph_context
Sep 17, 2026
Merged

hungpham10 merged 5 commits into
mainfrom
fix/fix-slowness-issue-when-performing-codegraph_context

Conversation

@hungpham10

@hungpham10 hungpham10 commented Sep 17, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features

    • Added resumable, timeout-aware caller traversal with resume tokens.
    • Added resume and timeout_ms options to caller queries.
    • Added configurable response detail levels and output formats across tools.
    • Added compact and minimized response layouts with improved symbol table formatting.
  • Bug Fixes

    • Preserved document values, annotation arguments, and meaningful zero/empty values in responses.
    • Improved file path handling and response formatting consistency.
  • Documentation

    • Updated server guidance for context queries, formatting options, detail inheritance, and graph document ingestion.

@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 1ce7c6fe-3393-46cb-aa75-d1efd6f37037

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov-commenter

codecov-commenter commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.65187% with 47 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.35%. Comparing base (8ce436c) to head (0779ddc).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
crates/codegraph-mcp/src/tools.rs 89.02% 26 Missing ⚠️
crates/codegraph-mcp/src/usage.rs 28.57% 10 Missing ⚠️
crates/codegraph-graph/src/lib.rs 96.53% 8 Missing ⚠️
crates/codegraph-mcp/src/lib.rs 90.47% 2 Missing ⚠️
crates/codegraph/src/main.rs 0.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (91.65%) 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      #35      +/-   ##
==========================================
+ Coverage   73.45%   78.35%   +4.90%     
==========================================
  Files          88       88              
  Lines       20245    20718     +473     
==========================================
+ Hits        14870    16233    +1363     
+ Misses       5375     4485     -890     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codspeed

codspeed Bot commented Sep 17, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 28 untouched benchmarks
🆕 10 new benchmarks

Performance Changes

Benchmark BASE HEAD Efficiency
🆕 cold_depth1 N/A 545.1 ms N/A
🆕 warm_broad N/A 2.1 ms N/A
🆕 warm_depth1 N/A 1.6 ms N/A
🆕 warm_depth2 N/A 3.6 ms N/A
🆕 warm_no_hit N/A 548.2 µs N/A
🆕 cold_depth1 N/A 87.5 ms N/A
🆕 warm_broad N/A 825.2 µs N/A
🆕 warm_depth1 N/A 688.7 µs N/A
🆕 warm_depth2 N/A 978.2 µs N/A
🆕 warm_no_hit N/A 547.4 µs N/A

Comparing fix/fix-slowness-issue-when-performing-codegraph_context (0779ddc) with main (8ce436c)

Open in CodSpeed

@hungpham10
hungpham10 merged commit e3daee0 into main Sep 17, 2026
11 of 12 checks passed
@hungpham10
hungpham10 deleted the fix/fix-slowness-issue-when-performing-codegraph_context branch September 17, 2026 13:58
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.

2 participants