Skip to content

Fix error when requesting completion to Copilot Chat without tools #30007

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 6, 2025

Conversation

as-cii
Copy link
Member

@as-cii as-cii commented May 6, 2025

The API will return a Bad Request (with no error message) when tools were used previously in the conversation but no tools are provided as part of a new request.

Inserting a dummy tool seems to circumvent this error.

Release Notes:

  • Fixed an error that could sometimes occur when editing using Copilot Chat.

The API will return a Bad Request (with no error message) when tools were used previously in the conversation but no tools are provided as part of a new request.

Inserting a dummy tool seems to circumvent this error.

Co-authored-by: Bennet Bo Fenner <[email protected]>
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label May 6, 2025
@as-cii as-cii enabled auto-merge (squash) May 6, 2025 13:46
@as-cii as-cii merged commit 0f50e6b into main May 6, 2025
20 of 21 checks passed
@as-cii as-cii deleted the fix-copilot-error branch May 6, 2025 14:20
JosephTLyons pushed a commit that referenced this pull request May 6, 2025
…30007)

The API will return a Bad Request (with no error message) when tools
were used previously in the conversation but no tools are provided as
part of a new request.

Inserting a dummy tool seems to circumvent this error.

Release Notes:

- Fixed an error that could sometimes occur when editing using Copilot
Chat.

Co-authored-by: Bennet Bo Fenner <[email protected]>
maxbrunsfeld added a commit that referenced this pull request May 7, 2025
Copilot chat still returns a 400 if the dummy tool uses the `{}` schema.

This is a follow-up to #30007.

Release Notes:

- Fixed a bug where agent edits would fail when using GitHub Copilot
Chat.

Co-authored-by: Agus Zubiaga <[email protected]>
JosephTLyons pushed a commit that referenced this pull request May 7, 2025
Copilot chat still returns a 400 if the dummy tool uses the `{}` schema.

This is a follow-up to #30007.

Release Notes:

- Fixed a bug where agent edits would fail when using GitHub Copilot
Chat.

Co-authored-by: Agus Zubiaga <[email protected]>
JosephTLyons pushed a commit that referenced this pull request May 7, 2025
Copilot chat still returns a 400 if the dummy tool uses the `{}` schema.

This is a follow-up to #30007.

Release Notes:

- Fixed a bug where agent edits would fail when using GitHub Copilot
Chat.

Co-authored-by: Agus Zubiaga <[email protected]>
lj3954 pushed a commit to lj3954/zed that referenced this pull request May 9, 2025
Copilot chat still returns a 400 if the dummy tool uses the `{}` schema.

This is a follow-up to zed-industries#30007.

Release Notes:

- Fixed a bug where agent edits would fail when using GitHub Copilot
Chat.

Co-authored-by: Agus Zubiaga <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant