Raise UserError when tool system_prompt returns a None

[cmungall]

Previously

@agent.system_prompt
async def func():
    return None

would raise an error at time of model calling

This PR checks defensively checks what is being returned, and will raise a hopefully more informative UserError. Note this all requires a type check override, since returning None is forbidden. However, if we treat user code as input then it's possible to violate at runtime.

Also include a unit test, and for an additional ultra-edgy case of returning a non-string from system_prompt

Fixes #1057

[cmungall]

Either intentionally or accidentally returning a number seems quite unlikely but I decided to cast to str and add a test.

[DouweM]

@cmungall Thanks for working on this and our apologies for the late response... There's another PR that adds explicitly support for None system prompts: #864. That one's been stalled for a bit as well, but I think we'll get it merged soon. So I'll close this for the moment, due to be reopened if we decide not to go the other route.