fix: ensure bedrock client handles stop sequences properly
Some checks failed
CodeQL Advanced / Analyze (actions) (push) Has been cancelled
CodeQL Advanced / Analyze (python) (push) Has been cancelled
Notify Downstream / notify-downstream (push) Has been cancelled
Mark stale issues and pull requests / stale (push) Has been cancelled

Co-authored-by: Greyson LaLonde <greyson.r.lalonde@gmail.com>
This commit is contained in:
Vidit Ostwal
2026-01-26 06:58:09 +05:30
committed by GitHub
parent db7aeb5a00
commit 3d771f03fa

View File

@@ -247,7 +247,7 @@ class BedrockCompletion(BaseLLM):
self.top_p = top_p
self.top_k = top_k
self.stream = stream
self.stop_sequences = stop_sequences or []
self.stop_sequences = stop_sequences
# Store advanced features (optional)
self.guardrail_config = guardrail_config
@@ -267,7 +267,7 @@ class BedrockCompletion(BaseLLM):
@property
def stop(self) -> list[str]:
"""Get stop sequences sent to the API."""
return list(self.stop_sequences)
return [] if self.stop_sequences is None else list(self.stop_sequences)
@stop.setter
def stop(self, value: Sequence[str] | str | None) -> None: