Skip to content

fix(bridge): release translator lock before diagnostics request#191

Open
vcheckk wants to merge 2 commits into
bug-ops:mainfrom
vcheckk:restart-issue-159
Open

fix(bridge): release translator lock before diagnostics request#191
vcheckk wants to merge 2 commits into
bug-ops:mainfrom
vcheckk:restart-issue-159

Conversation

@vcheckk

@vcheckk vcheckk commented Jun 13, 2026

Copy link
Copy Markdown

Closes #159.

What changed

  • Prepares diagnostics requests under translator state, then releases translator state before the LSP request.

Validation

  • Included in standalone integration branch restart-standalone-integration.
  • cargo test -p mcpls-core passed.
  • cargo clippy -p mcpls-core --all-targets -- -D warnings passed.
  • Real rust-analyzer smoke passed: cargo test -p mcpls-core --test integration_tests integration::rust_analyzer_tests::test_hover_on_u64_type -- --ignored --exact --nocapture.

@github-actions github-actions Bot added rust Rust code changes mcpls-core mcpls-core crate changes labels Jun 13, 2026
@vcheckk vcheckk marked this pull request as ready for review June 13, 2026 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mcpls-core mcpls-core crate changes rust Rust code changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(bridge): translator lock held across 30s pull_diagnostics blocks all concurrent MCP tools

1 participant