Skip to content

fix(bridge): split notification cache lock#180

Closed
vcheckk wants to merge 2 commits into
bug-ops:mainfrom
vcheckk:restart-issue-104
Closed

fix(bridge): split notification cache lock#180
vcheckk wants to merge 2 commits into
bug-ops:mainfrom
vcheckk:restart-issue-104

Conversation

@vcheckk

@vcheckk vcheckk commented Jun 13, 2026

Copy link
Copy Markdown

Closes #104.

What changed

  • Splits notification-cache access out of the translator lock path to reduce notification pump starvation.

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
@bug-ops

bug-ops commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Closing this PR as part of a batch closure of all submissions from this contributor.

The entire set of 15 PRs (#180#194) was submitted within ~3 minutes on 2026-06-13, indicating AI-generated code submitted without proper individual review or integration into the project's contribution workflow.

Specific issues identified across the batch:

If you would like to contribute to this project, please open individual issues for discussion before submitting code, and follow the contribution guidelines.

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): split NotificationCache out of Translator lock to eliminate notification_pump starvation

2 participants