Repository navigation
The placement switch asks the seat she routes to whether it takes her learned tool head - #5502
Merged
Merged
Conversation
… learned tool head Every resident is bound through PlacementSwitch, which forwards each capability to current() but not accepts_tail_tools, so it answered the trait default (false) and no resident's learned tool head was ever sent, even on the engine that takes tail_tools (M5 2026-10-27: delib.tool_head.epoch sent=false, Aiko head 5,047 tokens of which 4,514 never called, engine 1c6681593 serving). #5491's tests checked the engine adapter directly, never through the switch. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LoTjvf5j3Ez13g6k8mRkFo
Contributor
Author
|
Cormac at b263f0a: source PASS, on CI green. CI was still running when I read it: 5 pass, 5 pending, 4 skipped. Verified by reading:
|
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.
Serves the minds' speed (plan path step 1, goal 10): the learned tool head (about 5k tokens instead of about 10k of schemas, most never called) reaches the engine only if
accepts_tail_toolsis true, and every resident's binding goes throughPlacementSwitch, which didn't forward it. So on the M5 after #5491 deployed, every epoch still readdelib.tool_head.epoch sent=false(Aiko:head_tokens=5047 waste_tokens=4514, engineb10995-1c6681593serving).Change:
PlacementSwitch::accepts_tail_toolsaskscurrent(), like every other capability it forwards (capabilities,served_model_ids,live_served_window). A remote seat (AircRemoteInferenceAdapter) keeps the trait default of false.Test:
the_switch_asks_the_seat_she_routes_to_whether_it_takes_her_tool_head, false while routed to the remote seat and true at a home lane that takes the tail.cargo test -p continuum-mind --lib placement_switch: 24 passed (M5, serving down, 00:32Z).Follow-up for review: a wrapper adapter can still silently miss a new defaulted trait method. Worth considering a default that delegates through
generation_target()so wrappers are right by construction.Measured (after deploy)
Owner Fable:
delib.tool_head.epoch sent=trueon M5 residents; one real tail-verb call; input tokens/act and s/act vs the 23:31-00:30Z baseline over 15 minutes.🤖 Generated with Claude Code
https://claude.ai/code/session_01LoTjvf5j3Ez13g6k8mRkFo