Skip to content

Validate returned tool call arguments#278

Merged
Kamilbenkirane merged 1 commit into
mainfrom
validate-returned-tool-calls
May 6, 2026
Merged

Validate returned tool call arguments#278
Kamilbenkirane merged 1 commit into
mainfrom
validate-returned-tool-calls

Conversation

@Kamilbenkirane

Copy link
Copy Markdown
Member

Summary

  • validate returned user-defined tool calls against their Pydantic parameter models
  • normalize validated arguments with model_dump(mode="json", exclude_unset=True) while preserving provider extra fields
  • enforce validation for non-streaming and final streaming text outputs

Validation

  • pre-commit on commit/push: Ruff, Ruff format, mypy, test mypy, Bandit, coverage tests
  • focused local run: uv run pytest tests/unit_tests/test_tool_call_validation.py tests/unit_tests/test_client.py tests/unit_tests/test_streaming.py
  • focused local run: uv run ruff check src/celeste/tools.py src/celeste/client.py src/celeste/streaming.py tests/unit_tests/test_tool_call_validation.py
  • focused local run: uv run mypy src/celeste/tools.py src/celeste/client.py src/celeste/streaming.py

@Kamilbenkirane Kamilbenkirane merged commit 976d683 into main May 6, 2026
11 checks passed
@claude

claude Bot commented May 6, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant