dcdpr / jp / 25080330382

66%

push

github

chore(tools): Treat JSON `null` as absent in `opt` argument parsing (#588)

Some LLMs emit explicit `null` values for optional parameters that were
not provided by the caller. Previously, `opt` would attempt to
deserialize `null` via `req`, which could produce a type error instead
of cleanly returning `None`.

The fix matches on the raw `Value` before delegating to `req`, returning
`Ok(None)` for both a missing key and an explicit `Value::Null`. A
dedicated test covers all three cases: explicit null, a present value,
and a fully absent key.

Signed-off-by: Jean Mertz <git@jeanmertz.com>

3 of 3 new or added lines in 1 file covered. (100.0%)

23431 of 36477 relevant lines covered (64.23%)

215.7 hits per line