Settings: add automatic steering for active chats - #713
Draft
woosal1337 wants to merge 1 commit into
Draft
woosal1337 wants to merge 1 commit into
woosal1337 wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Steercommand. The host applies the message at its next supported step boundary without interrupting running tools.This change adds no dependency, wire field, migration, or version bump. Existing settings files keep the current queue behavior.
Test plan
cargo test --locked -p zeron-ui --lib -- --test-threads=1: 1,404 passed. New tests cover settings persistence, pointer and keyboard input, durable steering dispatch, and routing across every registered harness before and after loading.cargo test --locked -p zeron-engine --no-fail-fast -- --test-threads=1: 613 passed, 1 failed, 18 ignored. The failure isonline_runtime_shutdown_stops_edge_workers_and_retires_the_graph, which also fails on upstream main atb42fc2b. It counts one request after shutdown. The engine unit suite passes 381 tests, including the new descriptor regression. That regression first failed for Devin, Grok, and OpenCode.cargo test --locked -p zeron-harness --features native-fixture --no-fail-fast -- --test-threads=1: 510 passed, 12 ignored, with an empty, isolatedGEMINI_HOME. Native and ACP fixtures cover the supported protocol paths.git diff --check. Clippy reports no warning on a changed line. Live steering passes with Claude Code's Haiku and Sonnet aliases, and Grok's default model. Those checks cover tool survival, retained context, and ordered delivery of steering and queued messages.The offline routing matrix covers all nine real harnesses and the mock. Live checks cover the three cases above. They do not establish live support for every provider or advertised model.
On macOS, the engine run uses the existing
TMPDIRresolved to its canonical path. This avoids an existing generated-image path assertion that also reproduces on upstream main. No change to either integration test is part of this PR.Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.