reducers: propagate tool call action metadata#211
Merged
Conversation
- Ensures provider metadata stays synchronized when tool calls move through later lifecycle actions, which lets clients render updated tool-call UI context beyond the initial start event. - Extends reducer parity coverage so each tool-call-scoped action that carries metadata is exercised across implementations. - Updates the language client reducers and changelogs so generated packages document and preserve the same protocol behavior. (Commit message generated by Copilot)
Yoyokrazy
approved these changes
Jun 9, 2026
sandy081
added a commit
that referenced
this pull request
Jun 10, 2026
The rebased tool-call metadata change reintroduced an explicit `List<Annotation>` type on the annotationsRemoved reducer branch, which resolved to `kotlin.Annotation` instead of the generated type and broke `compileKotlin`. Drop the explicit annotation so the element type is inferred from `state.annotations`, matching the sibling branches. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
joshmouch
added a commit
to joshmouch/agent-host-protocol
that referenced
this pull request
Jun 10, 2026
Adopt the shared round-trip corpus's input/acceptableOutputs format and fix the
wire-fidelity drift it surfaces, plus propagate tool-call action metadata.
- TypesRoundTripFixtures: decode each fixture's `input` into the real generated
type, re-encode, and assert structural equality with the single canonical
acceptableOutputs[0] (key-order-independent, key-presence-sensitive). Add a
standalone ProtocolVersionConstants test in place of the removed 021-023.
- generate-csharp.ts: ChangesetOperationRangeTarget.Range is now the canonical
TextRange (was a flat {Start,End} pair); ActionEnvelope.Origin omits when
absent instead of serializing null. Regenerated.
- Reducers: the six tool-call lifecycle handlers propagate the action's _meta
onto the tool-call state (parity with upstream microsoft#211).
320/320 tests pass on net8.0 and net9.0.
joshmouch
added a commit
to joshmouch/agent-host-protocol
that referenced
this pull request
Jun 10, 2026
Adopt the shared round-trip corpus's input/acceptableOutputs format and fix the
wire-fidelity drift it surfaces, plus propagate tool-call action metadata.
- TypesRoundTripFixtures: decode each fixture's `input` into the real generated
type, re-encode, and assert structural equality with the single canonical
acceptableOutputs[0] (key-order-independent, key-presence-sensitive). Add a
standalone ProtocolVersionConstants test in place of the removed 021-023.
- generate-csharp.ts: ChangesetOperationRangeTarget.Range is now the canonical
TextRange (was a flat {Start,End} pair); ActionEnvelope.Origin omits when
absent instead of serializing null. Regenerated.
- Reducers: the six tool-call lifecycle handlers propagate the action's _meta
onto the tool-call state (parity with upstream microsoft#211).
320/320 tests pass on net8.0 and net9.0.
joshmouch
added a commit
to joshmouch/agent-host-protocol
that referenced
this pull request
Jun 10, 2026
Adopt the shared round-trip corpus's input/acceptableOutputs format and fix the
wire-fidelity drift it surfaces, plus propagate tool-call action metadata.
- TypesRoundTripFixtures: decode each fixture's `input` into the real generated
type, re-encode, and assert structural equality with the single canonical
acceptableOutputs[0] (key-order-independent, key-presence-sensitive). Add a
standalone ProtocolVersionConstants test in place of the removed 021-023.
- generate-csharp.ts: ChangesetOperationRangeTarget.Range is now the canonical
TextRange (was a flat {Start,End} pair); ActionEnvelope.Origin omits when
absent instead of serializing null. Regenerated.
- Reducers: the six tool-call lifecycle handlers propagate the action's _meta
onto the tool-call state (parity with upstream microsoft#211).
320/320 tests pass on net8.0 and net9.0.
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
2 participants
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.
(Commit message generated by Copilot)