-
Notifications
You must be signed in to change notification settings - Fork 74
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
Added getMessage
function under ai subpath
#1709
Conversation
🦋 Changeset detectedLatest commit: 5f7472c 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great start! Left a few comments/questions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe I'm missing some context, but why is this functionality here and not in the frontend libraries? There is no backend being created/provisioned which is the sole purpose of this library.
Removed duplicated files for `changeset`
* @returns The response from the AI model. | ||
* @throws Eror If the response from the AI model does not contain a message. | ||
*/ | ||
export const getMessage = async ({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How are customers expected to consume this function? Do they import it in their lambda function code? Could we just expose some prefilled defineFunction
definitions that they could use directly and not have to touch lambda code at all?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe we expect to have a lot of customizable within the lambda execution.
…o take in toolUseStrategy
4232af5
to
a3bde1c
Compare
bb975a1
to
a3bde1c
Compare
…dded handleToolUseStrategy method to handle toolUseStrategy logic
…ups=20(staff),12(everyone),61(localaccounts),79(_appserverusr),80(admin),81(_appserveradm),98(_lpadmin),701(com.apple.sharepoint.group.1),33(_appstore),100(_lpoperator),204(_developer),250(_analyticsusers),395(com.apple.access_ftp),398(com.apple.access_screensharing),399(com.apple.access_ssh),400(com.apple.access_remote_ae),702(com.apple.sharepoint.group.2) and
Problem
Issue number, if available:
Changes
Corresponding docs PR, if applicable:
Validation
Checklist
run-e2e
label set.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.