Fix booking payment schemas for Responses API

[Liang-Chun Tsai (ltsai-dev)]

Summary

Azure OpenAI Responses rejects the regex lookaround emitted by Pydantic for Decimal tool inputs, preventing external booking evaluations from starting. This change exposes payment amounts as finite JSON numbers while preserving Decimal conversion and monetary rounding inside each tool.

Changes

• Change charge, refund, and dispute amount request fields to finite float values with Responses-compatible JSON schemas.
• Convert validated inputs through Decimal(str(value)) before two-decimal ROUND_HALF_UP quantization.
• Add production-schema, non-finite input, rounding, and scale regression coverage.

Test plan

• PYTHONPATH=servers\\tb_business_ops_servers_202606 python -m pytest --quiet servers\\tb_business_ops_servers_202606\\tests\\external_booking\\payment_api\\test_process_charge.py servers\\tb_business_ops_servers_202606\\tests\\external_booking\\payment_api\\test_process_charge_dispute.py servers\\tb_business_ops_servers_202606\\tests\\external_booking\\payment_api\\test_process_refund.py
• 71 tests passed.

Related issues

N/A

[Copilot]

Pull request overview

This PR updates the external booking payment tools’ request schemas to avoid Pydantic Decimal JSON-schema regex features (lookarounds) that Azure OpenAI Responses rejects, while keeping internal monetary handling in Decimal with consistent ROUND_HALF_UP quantization.

Changes:

• Switch charge_amount, refund_amount, and dispute_amount tool inputs from Decimal to finite float (allow_inf_nan=False) to emit Responses-compatible JSON schemas.
• Convert validated float inputs via Decimal(str(value)) and quantize to 2 decimal places using ROUND_HALF_UP before persisting.
• Add tests covering schema shape (type: number, no pattern), rejection of non-finite inputs, and rounding behavior regression.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
servers/tb_business_ops_servers_202606/tb_business_ops_servers_202606/toolslib/external_booking/payment_api/tools/process_refund.py	Makes refund amount a finite JSON number while preserving internal Decimal rounding.
servers/tb_business_ops_servers_202606/tb_business_ops_servers_202606/toolslib/external_booking/payment_api/tools/process_charge.py	Makes charge amount a finite JSON number and converts to Decimal for validation/rounding.
servers/tb_business_ops_servers_202606/tb_business_ops_servers_202606/toolslib/external_booking/payment_api/tools/process_charge_dispute.py	Makes dispute amount a finite JSON number and quantizes via Decimal.
servers/tb_business_ops_servers_202606/tests/external_booking/payment_api/test_process_refund.py	Adds schema + non-finite validation coverage and pins float->Decimal rounding behavior.
servers/tb_business_ops_servers_202606/tests/external_booking/payment_api/test_process_charge.py	Adds schema + non-finite validation coverage and pins float->Decimal rounding behavior.
servers/tb_business_ops_servers_202606/tests/external_booking/payment_api/test_process_charge_dispute.py	Adds schema + non-finite validation coverage and pins float->Decimal rounding behavior.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.