Skip to content

Accept set_route as model switch alias#296

Open
1jehuang wants to merge 1 commit into
masterfrom
fix-set-route-model-alias
Open

Accept set_route as model switch alias#296
1jehuang wants to merge 1 commit into
masterfrom
fix-set-route-model-alias

Conversation

@1jehuang
Copy link
Copy Markdown
Owner

@1jehuang 1jehuang commented Jun 1, 2026

Summary

  • accept legacy set_route requests as an alias for set_model
  • keep serializing model switch requests as set_model
  • add a regression test for set_route deserialization

Fixes #295

Test plan

  • cargo test -p jcode-protocol test_set_route_deserializes_as_set_model_compat_alias
  • rustfmt --check crates/jcode-protocol/src/wire.rs crates/jcode-protocol/src/protocol_tests/misc_events.rs
  • selfdev build / scripts/dev_cargo.sh build --profile selfdev -p jcode --bin jcode

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.

Accept set_route compatibility alias for model switching

1 participant