fix: bedrock positional args in eval cli and adapt to tool parser changes in strands-sglang

Author: Lawhy

pyproject.toml

@@ -21,7 +21,7 @@ classifiers = [
     "Topic :: Scientific/Engineering :: Artificial Intelligence",
 ]
 dependencies = [
-    "strands-sglang>=0.1.2",
+    "strands-sglang>=0.1.3",
     "strands-agents-tools",
     "boto3>=1.26.0",
     "transformers>=4.0.0,<5.0.0",

src/strands_env/cli/utils.py

@@ -81,10 +81,10 @@ def build_model_factory(config: ModelConfig, max_concurrency: int) -> ModelFacto
         if not config.model_id:
             raise click.ClickException("--model-id is required for Bedrock backend")
         if config.role_arn:
-            session = get_assumed_role_session(config.role_arn, config.region)
+            boto_session = get_assumed_role_session(config.role_arn, config.region)
         else:
-            session = get_boto3_session(config.region, config.profile_name)
-        return bedrock_model_factory(session, config.model_id, sampling)
+            boto_session = get_boto3_session(config.region, config.profile_name)
+        return bedrock_model_factory(model_id=config.model_id, boto_session=boto_session, sampling_params=sampling)
 
     else:
         raise click.ClickException(f"Unknown backend: {config.backend}")

src/strands_env/core/models.py

@@ -49,7 +49,7 @@
 from strands.models.bedrock import BedrockModel
 from strands.models.openai import OpenAIModel
 from strands_sglang import SGLangClient, SGLangModel
-from strands_sglang.tool_parser import HermesToolCallParser, ToolCallParser
+from strands_sglang.tool_parsers import HermesToolParser, ToolParser
 from transformers import PreTrainedTokenizerBase
 
 #: Factory that produces a fresh `Model` per step (for concurrent step isolation).
@@ -67,7 +67,7 @@ def sglang_model_factory(
     model_id: str,
     tokenizer: PreTrainedTokenizerBase,
     client: SGLangClient,
-    tool_call_parser: ToolCallParser = HermesToolCallParser(),
+    tool_parser: ToolParser = HermesToolParser(),
     sampling_params: dict[str, Any] = DEFAULT_SAMPLING_PARAMS,
     enable_thinking: bool | None = None,
 ) -> ModelFactory:
@@ -83,7 +83,7 @@ def sglang_model_factory(
     return lambda: SGLangModel(
         tokenizer=tokenizer,
         client=client,
-        tool_call_parser=tool_call_parser,
+        tool_parser=tool_parser,
         params=sampling_params,
         model_id=model_id,
         return_logprobs=True,