fix: raise ValueError when 'model' is passed instead of 'model_name' in BedrockEmbedding

BedrockEmbedding accepts model_name to specify the Bedrock model ID, but
passing model= was silently ignored, causing the default model to be used
unexpectedly. This adds an explicit check that raises a clear ValueError
guiding users to use model_name instead.

Author: Br1an67

llama-index-integrations/embeddings/llama-index-embeddings-bedrock/llama_index/embeddings/bedrock/base.py

@@ -147,6 +147,12 @@ def __init__(
         output_parser: Optional[BaseOutputParser] = None,
         **kwargs: Any,
     ):
+        if "model" in kwargs:
+            raise ValueError(
+                "The 'model' parameter is not supported. "
+                "Please use 'model_name' instead to specify the Bedrock model ID."
+            )
+
         additional_kwargs = additional_kwargs or {}
 
         session_kwargs = {

llama-index-integrations/embeddings/llama-index-embeddings-bedrock/tests/test_bedrock_embedding.py

@@ -9,6 +9,13 @@ def test_class():
     assert BaseEmbedding.__name__ in names_of_base_classes
 
 
+def test_model_param_raises_error():
+    """Test that passing 'model' instead of 'model_name' raises ValueError."""
+    bedrock_client = boto3.client("bedrock-runtime", region_name="us-east-1")
+    with pytest.raises(ValueError, match="Please use 'model_name' instead"):
+        BedrockEmbedding(model="cohere.embed-multilingual-v3", client=bedrock_client)
+
+
 def test_get_provider_two_part_format():
     """Test _get_provider with 2-part model names (provider.model)."""
     bedrock_client = boto3.client("bedrock-runtime", region_name="us-east-1")