DeleteContext reads filter["id"] unchecked (L423), then asserts id.(string) (L437, L445). A filter without a string id, like {"deleted_at": nil} or {"id": 42}, panics.
Fix: use comma-ok for the lookup and the assertion, and return an error. UpdateContext in the same file already does this.
Found while reviewing #251 / #247.
DeleteContextreadsfilter["id"]unchecked (L423), then assertsid.(string)(L437, L445). A filter without a stringid, like{"deleted_at": nil}or{"id": 42}, panics.Fix: use comma-ok for the lookup and the assertion, and return an error.
UpdateContextin the same file already does this.Found while reviewing #251 / #247.