-
Notifications
You must be signed in to change notification settings - Fork 422
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
[SDK] feat: Support multiple messages in Nebula API and update input props #5972
[SDK] feat: Support multiple messages in Nebula API and update input props #5972
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
42b5eb3
to
4afbd2a
Compare
🦋 Changeset detectedLatest commit: 6f89de8 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
size-limit report 📦
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #5972 +/- ##
=======================================
Coverage 54.79% 54.79%
=======================================
Files 1150 1150
Lines 61233 61245 +12
Branches 5150 5154 +4
=======================================
+ Hits 33551 33559 +8
- Misses 26955 26958 +3
- Partials 727 728 +1
*This pull request uses carry forward flags. Click here to find out more.
|
Makes sense but having a chat api where you just pass a string is nice though What's the difference between user and assistant? |
Any chance we can support a message | messages input param for the chat interface? |
4afbd2a
to
aa9dfa5
Compare
aa9dfa5
to
ba1ed43
Compare
ba1ed43
to
6f89de8
Compare
Fixes TOOL-3132
PR-Codex overview
This PR focuses on enhancing the
Nebula
API by supporting multiple messages in the input structure and updating certain property names for clarity.Detailed summary
prompt
tomessage
inNebula.chat
andNebula.execute
.context
tocontextFilter
in both functions.