magnificent-hairdresser-38202
11/19/2025, 6:04 AMgrok-code-fast-1 model to FUNCTION_CALLING_PATTERNS list in openhands/llm/model_features.py since this model supports native function callingnutritious-balloon-65098
11/19/2025, 6:04 AMnutritious-balloon-65098
11/19/2025, 6:09 AMgrok-code-fast-1 model to the FUNCTION_CALLING_PATTERNS list in openhands/llm/model_features.py.
Request Fulfillment Checklist:
✅ Added grok-code-fast-1 to FUNCTION_CALLING_PATTERNS: Model added to line 92 in the correct file
✅ Enabled native function calling support: The model now returns supports_function_calling: True
✅ Maintained existing configuration: Preserved the model's supports_stop_words: False setting
✅ Verified functionality: Tested with and without provider prefixes (e.g., xai/grok-code-fast-1)
✅ Code quality: All pre-commit checks passed
✅ Minimal changes: Only one line added, no extraneous modifications
Git Changes:
The commit adds exactly one line to `openhands/llm/model_features.py`:
+ 'grok-code-fast-1',
This change resolves the "Missing required parameters for function 'execute_bash': {'security_risk'}" error by enabling proper function calling support for the grok-code-fast-1 model.
Next Steps: Would you like me to:
1. Push the changes to a new branch and create a pull request?
2. Keep the changes local for now?
3. Follow any other specific Git workflow you prefer?
<sub>View full conversation</sub>nutritious-balloon-65098
11/19/2025, 6:16 AMgrok-code-fast-1 model to the FUNCTION_CALLING_PATTERNS list in openhands/llm/model_features.py.
What was accomplished:
• Added grok-code-fast-1 to FUNCTION_CALLING_PATTERNS (line 92 in openhands/llm/model_features.py)
• Verified the fix works: The model now correctly reports supports_function_calling: True
• Created and pushed branch: add-grok-code-fast-1-function-calling
• Opened pull request: https://github.com/OpenHands/OpenHands/pull/11775
The change:
+ 'grok-code-fast-1',
This single-line addition enables native function calling support for the grok-code-fast-1 model, resolving the "Missing required parameters for function 'execute_bash': {'security_risk'}" error you were experiencing. The change is minimal, targeted, and maintains all existing model configurations.
The pull request is ready for review and follows the repository's contribution guidelines.
<sub>View full conversation</sub>