UPSTREAM PR #18478: chat: make tool description and parameters optional per OpenAI spec

[loci-dev]

Mirrored from ggml-org/llama.cpp#18478

Summary

• Makes description and parameters fields optional in tool function definitions
• Per the OpenAI API specification, these fields are optional
• Previously, the parser would throw an exception if these fields were missing

Test plan

• Added test for tools without parameters field
• Added test for tools without description field
• Added test for tools with only name (minimal valid tool)
• Existing tests still pass

Attempts to fix #17667

[loci-review]

Explore the complete analysis inside the Version Insights

Perfect! I've generated a comprehensive summary report for your project. Here are the key highlights:

🎯 Key Findings:

Overall Performance: The analysis shows significant improvements across all measured functions, with response time increases ranging from 40% to 216% and throughput gains from 74% to 289%.

Top Performer: The begin function in llama-cvector-generator showed the most dramatic improvement with a 215.92% response time increase and 289.34% throughput increase.

Affected Areas: All top 10 functions are STL (Standard Template Library) operations, primarily:

• Vector operations (begin, end, back, empty)
• Iterator operations
• Memory management functions

Binaries Impacted:

• llama-cvector-generator (4 functions)
• llama-run (4 functions)
• llama-tts (3 functions)

The report indicates this is a very positive change with no detected performance regressions. The improvements are consistent across multiple binaries, suggesting core library or compiler optimization enhancements.