Skip to content
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

Porting functionality in upstream http commands #3

Open
1412031 opened this issue May 14, 2024 · 1 comment
Open

Porting functionality in upstream http commands #3

1412031 opened this issue May 14, 2024 · 1 comment

Comments

@1412031
Copy link

1412031 commented May 14, 2024

Greetings, I've wrote a couple of wrappers around APIs like OpenAI and Anthropic and I've spent considerable amounts of time hacking around parsing sse when streaming.

At some point I would like to share them in nushell_scripts for others to use and your plugin is perfect for parsing the response.

I've been wondering, is there anything (apart from time and effort) stopping one from incorporating the parser in nushell instead of using it separately like a plugin?

SSE have become quite a thing lastly and having them handled without any external dependencies could be quite useful in many cases.

In any case, thanks for authoring this!

@cablehead
Copy link
Owner

Hi! I'd love to see your wrappers. Those type of APIs are just the thing I'm hoping from sse can support.

My understanding is that the nu community is generally looking to move things from builtins towards plugins (the dataframe builtins have been moved to a polars plugin). I'm not certain what the consensus is for http related things. My personal preference would be to move the current builtin http commands out to a plugin: https://github.com/cablehead/nu_plugin_http If that http plugin pans out, I'll merge from sse with it.

Want to swing by the nushell discord to discuss in the #plugins channel?

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

No branches or pull requests

2 participants