Repository navigation
Update T3 contract to 5fe7568 - #7
Merged
Merged
Conversation
Moves the T3 pin from 8d199b4 to 5fe7568. A project can now turn a provider off, and delegate_task refuses that provider even though orchestrator_capabilities still lists it. The adapter's model-per-role steps now fall back to another provider in that case. Adds write_pull_request_guides and read_pull_request_guide to the contract. t3_thread_read now rejects unknown params, and the adapter passes only threadId and view. Release 0.2.1.
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.
Moves the T3 pin from
8d199b4to5fe7568(3 commits). Release 0.2.1.Breaks a mapping
8ac6846feat(settings): providers can be turned on or off per project (#17879).delegate_taskandcreate_threadsnow refuse a provider instance the project turned off, withinvalid_request"Provider instance ... is disabled for project ...".orchestrator_capabilitiesstill lists that instance. The adapter's fallback ("use the closest model of the same provider") would retry the refused provider, so Model per role step 2 now says to run the lane on another provider and say so in the reply.Opportunities
write_pull_request_guidesandread_pull_request_guide(#18143) write and read the review guide a maintainer sees in a PR's Guide tab. The adapter's "Opening a PR" bullet could have the root callwrite_pull_request_guidesfor every PR it links, and the Shipping playbook could checkread_pull_request_guide(stale,run) before it asks for a merge.Irrelevant to the skills
1 change.
t3_thread_readis now strict and rejects unknown params (#17823). The adapter passes onlythreadIdandview, both in the schema.Checks
bun run check,bun test tools, andbun run typecheckpass.