Skip to content

Conversation

@kevinmessiaen
Copy link
Member

Description

Group message by role when using Claude bedrock client, this allow some edge case to work such as:

resp = claude_client.complete(messages=[
    ChatMessage(role='system', content='This is a test'),
    ChatMessage(role='user', content='This is the first message'),
    ChatMessage(role='user', content='This is the second message'),
    ChatMessage(role='user', content='Is it working?'),
])

Type of Change

  • 📚 Examples / docs / tutorials / dependencies update
  • 🔧 Bug fix (non-breaking change which fixes an issue)
  • 🥂 Improvement (non-breaking change which improves an existing feature)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🔐 Security fix
@kevinmessiaen kevinmessiaen requested a review from henchaves June 4, 2024 06:05
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 4, 2024

Please retry analysis of this Pull-Request directly on SonarCloud

@kevinmessiaen kevinmessiaen merged commit 7e16c0d into main Jun 4, 2024
@kevinmessiaen kevinmessiaen deleted the feature/handle-consecutive-messages-claude branch June 4, 2024 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants