agentjido / req_llm / cd102022105135b91a6ffa04f71b81add5cc4f78-PR-129

51%

main: 52%

Pull #129

github

fix: Handle JSON Schema in Tool.Inspect protocol

The Tool Inspect protocol was crashing when inspecting tools with
JSON Schema (map) parameter schemas because it only handled
NimbleOptions (keyword list) format.

Changes:
- Updated Inspect protocol to check if parameter_schema is list or map
- For lists (NimbleOptions): Show param count (e.g., "2 params")
- For maps (JSON Schema): Show param count from properties field
  with "(JSON Schema)" suffix (e.g., "2 params (JSON Schema)")
- Handle empty schemas for both formats appropriately

Tests:
- Added 4 new test cases covering NimbleOptions and JSON Schema
  formats with both populated and empty parameter schemas
- Fixed trailing whitespace in existing test
- All 29 tests passing

7 of 8 new or added lines in 1 file covered. (87.5%)

3367 of 6599 relevant lines covered (51.02%)

444.74 hits per line