feat: add Snowflake provider #2488
Merged
michaelneale merged 19 commits intoblock:mainfrom Jun 2, 2025
Merged
Conversation
sfc-gh-twhite
commented
May 8, 2025
sfc-gh-twhite
commented
May 8, 2025
sfc-gh-twhite
force-pushed
the
snowflake-provider
branch
from
e949378 to
11c670c
Compare
sfc-gh-twhite
force-pushed
the
snowflake-provider
branch
from
11c670c to
022d1cd
Compare
michaelneale
approved these changes
Jun 2, 2025
Collaborator
michaelneale
left a comment
michaelneale
left a comment
There was a problem hiding this comment.
LGTM - once build passes happy to get this in. I have updated the branch
cbruyndoncx
pushed a commit
to cbruyndoncx/goose
that referenced
this pull request
Jul 20, 2025
Co-authored-by: Michael Neale <michael.neale@gmail.com>
1 task
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.
Overview
I am looking to add Snowflake as a provider for Goose. This pull request is intended toget feedback early in the process, and I am open to closing and reopening this PR when
it's closer to completion. (if preferred)
I borrowed a lot of the code from the Anthropic provider, which made the
implementation easier, but it isn't 1:1. I've been using claude-3-5-sonnet for CoT.
Current Status
Most of the functionality works for the first message, but there is an issue inpreparing the response to the message with the tool calls. I'm manually adding the
"tool_use" field in the post function, and I will adjust this to ensure the response is more
accurate and dynamic.
This is all working now, I ended up manually parsing out and mapping the tool calls. It might
be possible to use eventsource-client but I was unable get it working properly.
What's Next
Adjust the preparation of the response to include "tool_use" based on the dynamicallyinteraction with the tool.
Perform thorough testing to ensure all edge cases are handled. (I need to get thetests working)
Validate the integration with Snowflake and ensure seamless operation within theGoose framework.