Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

Pass in copilot extra headers when analyzing output packages #346

Merged
merged 1 commit into from
Dec 15, 2024

Conversation

jhrozek
Copy link
Contributor

@jhrozek jhrozek commented Dec 14, 2024

When querying Copilot, the LLM requires certain headers to be explicitly
set to accept a request. We were already parsing them and storing in
context and even using in the input pipeline, but forgot to use them in
the output pipeline to extract packages from the inbound (LLM-to-client)
snippets.

Fixes: #335

When querying Copilot, the LLM requires certain headers to be explicitly
set to accept a request. We were already parsing them and storing in
context and even using in the input pipeline, but forgot to use them in
the output pipeline to extract packages from the inbound (LLM-to-client)
snippets.

Fixes: #335
@lukehinds lukehinds merged commit c556c6e into stacklok:main Dec 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing Editor-Version header causing 400 Bad Request when querying for packages in output snippets
2 participants