This repository was archived by the owner on Jun 5, 2025. It is now read-only.
Fixes to the last_user_block method#1076
Closed
jhrozek wants to merge 29 commits intostacklok:mainfrom
Closed
Conversation
These are necessary to fully decouple pipeline steps from message representation.
Also, message/content/text interface to be used throughout the pipeline.
…tacklok#1026) In case the message is just a text and not a MessageContent, we need to set self.content. Otherwise, the method will be called on MessageContent which is already implemented.
* copilot: Normalize into OpenAI types * copilot: Use native OpenAI types when parsing SSE stream * copilot: Streaming normalizer no longer needed
Additionally, more fixes to pipeline steps.
This allows using the openai types without normalizers.
Also, moved openai typed and generators in their own module, and, last but not least, restored the typewriter-effect!
Also, fixed typo in prompt.
Also, fixing default streaming behaviour for Ollama requests.
For good, hopefully...
…ypes (stacklok#1040) * Default the function literal, copilot doesn't set it * return in get_text in ChoiceDelta * tool_calls is a valid stop reason for copilot * Add set_text to openAI tool message
Also, tests and fixes.
Also, moar fixes.
* Fix typo * Rely on redacted_secrets_count to detect if this was the first message
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
I had to temporarily disable the OpenInterpreter case and the Aider case while I test those two clients specifically.