Skip to content

Add Rust runtime lock settings#1996

Open
SlowGreek wants to merge 1 commit into
github:mainfrom
SlowGreek:slowgreek-add-model-lock-setting
Open

Add Rust runtime lock settings#1996
SlowGreek wants to merge 1 commit into
github:mainfrom
SlowGreek:slowgreek-add-model-lock-setting

Conversation

@SlowGreek

Copy link
Copy Markdown

Summary

  • add a typed Rust RuntimeSettings API with optional model and context-tier locks
  • forward runtimeSettings.modelLock and runtimeSettings.contextTierLock on session create and resume
  • preserve wire omission when runtime settings are absent

Testing

  • cargo +nightly-2026-04-14 fmt --check
  • cargo clippy --all-features --all-targets -- -D warnings
  • cargo test runtime_settings --all-features
  • all non-E2E Rust unit and integration test targets

Expose typed model and context-tier locks on create and resume session configs and serialize them through the nested runtimeSettings wire payload.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: a93a18e3-4a6d-4667-97fe-9252838f19cf
Copilot AI review requested due to automatic review settings July 15, 2026 23:09
@SlowGreek SlowGreek requested a review from a team as a code owner July 15, 2026 23:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds typed Rust runtime lock settings and forwards them through session create/resume payloads.

Changes:

  • Adds RuntimeSettings with model and context-tier locks.
  • Integrates settings into create/resume configs and wire serialization.
  • Tests serialization and omission when absent.
Show a summary per file
File Description
rust/src/types.rs Defines the public API, builders, forwarding, and tests.
rust/src/wire.rs Adds runtime settings to create/resume wire payloads.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Medium

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.

3 participants