Skip to content

test(bridge): document lock-free translator context#182

Draft
vcheckk wants to merge 4 commits into
bug-ops:mainfrom
vcheckk:restart-issue-114
Draft

test(bridge): document lock-free translator context#182
vcheckk wants to merge 4 commits into
bug-ops:mainfrom
vcheckk:restart-issue-114

Conversation

@vcheckk

@vcheckk vcheckk commented Jun 13, 2026

Copy link
Copy Markdown

Closes #114.

Stacking / readiness

What changed

  • Adds regression coverage documenting that HandlerContext stores Arc<Translator> directly rather than an outer request-wide lock.

Validation

  • Branch-local cargo test -p mcpls-core passed.
  • Branch-local cargo clippy -p mcpls-core --all-targets -- -D warnings passed.
  • Included in standalone integration branch restart-standalone-integration.
  • Integration branch cargo test -p mcpls-core passed.
  • Integration branch cargo clippy -p mcpls-core --all-targets -- -D warnings passed.
  • Real rust-analyzer smoke passed on the integration branch: 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 testing Test-related changes mcpls-core mcpls-core crate changes labels Jun 13, 2026
@vcheckk

vcheckk commented Jun 13, 2026

Copy link
Copy Markdown
Author

Readiness note: keeping this PR as draft because the current branch is stacked on #180 and #181 / issues #104 and #108. It now passes local cargo test -p mcpls-core and cargo clippy -p mcpls-core --all-targets -- -D warnings, but it should be reviewed after the dependency chain or rebuilt/rebased once those land.

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 testing Test-related changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

research(bridge): upgrade Arc<Mutex<Translator>> to Arc<RwLock<Translator>> after #104 and #108 land

1 participant