Update Amazon Bedrock integration, add chat model #2746
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
dosubot
bot
added
the
auto:enhancement
![dosubot[bot]](https://avatars.githubusercontent.com/in/324583?s=60&v=4){kind=small}
| @@ -3,13 +3,22 @@ import { Bedrock } from "langchain/llms/bedrock"; | |||
| // If no credentials are provided, the default credentials from | |||
There was a problem hiding this comment.
This comment is flagging the change in the PR that requires the BEDROCK_AWS_ACCESS_KEY_ID and BEDROCK_AWS_SECRET_ACCESS_KEY environment variables to be set, and it should be reviewed by maintainers.
| @@ -685,6 +686,8 @@ | |||
| "@qdrant/js-client-rest": "^1.2.0", | |||
There was a problem hiding this comment.
Great work on the PR! This comment is just to flag the addition of new dependencies (@smithy/protocol-http and @smithy/signature-v4) for maintainers to review, as it changes the project's peer/dev/hard dependencies.
| @@ -0,0 +1,299 @@ | |||
| import { SignatureV4 } from "@smithy/signature-v4"; | |||
There was a problem hiding this comment.
This PR adds a new external HTTP request using fetch. Please review this change to ensure it aligns with the project's requirements and security guidelines.
| @@ -0,0 +1,299 @@ | |||
| import { SignatureV4 } from "@smithy/signature-v4"; | |||
There was a problem hiding this comment.
This PR adds a new import and usage of getEnvironmentVariable to access an environment variable. Please review this change to ensure it is implemented correctly and securely.
| @@ -0,0 +1,55 @@ | |||
| /* eslint-disable no-process-env */ | |||
There was a problem hiding this comment.
This PR adds code that explicitly accesses environment variables via process.env. Please review this change to ensure that the environment variables are properly handled and secured.
| @@ -1,386 +1,48 @@ | |||
| /* eslint-disable no-process-env */ | |||
There was a problem hiding this comment.
This PR adds, accesses, reads, or requires environment variables via process.env or getEnvironmentVariable. Please review this change to ensure proper handling and security of environment variables.
| @@ -0,0 +1,430 @@ | |||
| import { test, expect } from "@jest/globals"; | |||
There was a problem hiding this comment.
This PR adds net-new, non-api client external HTTP requests using fetch. Please review this change to ensure it aligns with the project's requirements and best practices.
Supports streaming for Claude-v2, fixes #2744, fixes #2731, fixes #2645