fix: guard against None after Mapping extraction, prevent request_params overriding core fields

- Add second None check in _normalize_token_id_sequence after extracting
  from a Mapping, since .get() can return None even when the key exists
- Move request_params spread before explicit keys (model, prompt,
  temperature, max_tokens, logprobs) so they cannot be silently
  overridden by stray entries in request_params

Made-with: Cursor

Author: benjibc

eval_protocol/integrations/fireworks_v1_completions_client.py

@@ -72,6 +72,8 @@ def _normalize_token_id_sequence(values: Any) -> List[int]:
         return []
     if isinstance(values, Mapping):
         values = values.get("input_ids", values.get("ids", []))
+    if values is None:
+        return []
     if hasattr(values, "tolist") and not isinstance(values, list):
         values = values.tolist()
     if isinstance(values, tuple):
@@ -347,12 +349,12 @@ async def create_completion_from_prompt_ids(
         active_tools = tools if tools is not None else (self.default_tools or None)
         normalized_prompt_token_ids = [int(x) for x in list(prompt_token_ids)]
         request_payload = {
+            **self.request_params,
             "model": self.model_id,
             "prompt": normalized_prompt_token_ids,
             "temperature": self.temperature,
             "max_tokens": self.max_tokens,
             "logprobs": True if self.logprobs else None,
-            **self.request_params,
         }
         if not self.logprobs:
             request_payload.pop("logprobs", None)