agentjido / req_llm / 0b2b562e96007dde6b7b6906a0c99c6ff218032b

49%

Build Type

push

github

Committed by web-flow

Commit Message

fix: Handle JSON Schema in Tool.Inspect protocol (#129)

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

Run Details

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

3 existing lines in 2 files now uncovered.

3370 of 6599 relevant lines covered (51.07%)

444.23 hits per line