Fix Fable 5.1 Anthropic sampling compatibility - #2
Merged
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
1 participant
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.
Planner requests to Fable 5.1 failed with HTTP 400 because the pinned Copilot runtime adds a deprecated temperature parameter. Register an SDK request handler that removes temperature, top_p, and top_k from direct Anthropic Messages requests, preserving credentials, message/tool payloads, streaming, and cancellation. Other endpoints pass through unchanged.
The pinned-runtime mock now rejects deprecated sampling fields, reproducing the reported error before the fix and passing afterward for Planner streaming, allowed/denied tools, authentication failures, and worker results. Added request-boundary tests covering payload preservation and unrelated endpoints.
Validation:
bun run test:anthropic(4 passed), focused tests (14 passed),bun run types, andbun run ci.Live preflight: the deployed pinned runtime, with this request handler, completed an isolated arithmetic inference and terminal tool call using the configured Anthropic key. The usage event identified
claude-fable-5-1. No user documents were accessed or changed.