Require explicit model_name for SWE-Bench converters

[simonrosenberg]

Summary

• require --model-name for swebench and swebenchmultimodal converters; no default, raise if missing
• clarify docstrings with expected model identifier examples and why it matters for attribution
• adjust tests to supply a model name and assert missing-name error

Testing

• uv run pytest tests/test_swebenchmultimodal.py -q

[simonrosenberg]

@juanmichelini I think model_name: str = "OpenHands" is a bit misleading since the real values are llm names so I figured we could make it mandatory

[openhands-ai]

Looks like there are a few issues preventing this PR from being merged!

• GitHub Actions are failing:
  • Pre-commit checks

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #376 at branch `feature/require-model-name`

Feel free to include any additional details that might help me get this PR into a better state.

_{^{You can manage your notification settings}}