Skip to content

Conversation

@kzu
Copy link

@kzu kzu commented Nov 29, 2025

Fixes #15

The Message proto was missing tool_call_id, which is required per the function calling docs to associate tool responses with their prior invocations.

Changes

  • Added string tool_call_id = 7 to Message in chat.proto

Usage

Message {
  role: ROLE_TOOL
  content: [{ text: "<function result>" }]
  tool_call_id: "<id from prior ToolCall>"
}
Original prompt

Implement a fix for #15


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

* Initial plan

* Add tool_call_id field to Message proto for tool invocation responses

Co-authored-by: kzu <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: kzu <[email protected]>
@kzu kzu requested a review from a team as a code owner November 29, 2025 19:59
kzu added a commit to devlooped/GrokClient that referenced this pull request Nov 30, 2025
kzu added a commit to devlooped/GrokClient that referenced this pull request Nov 30, 2025
kzu added a commit to devlooped/GrokClient that referenced this pull request Nov 30, 2025
kzu added a commit to devlooped/GrokClient that referenced this pull request Nov 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tool_call_id is missing in chat messages, though appears to be required as per docs

1 participant