Remove Result<_, InteractError> from interact signature

[bikeshedder]

The return type of interact is questionable at best:

• InteractError::Panic is dangerous to ignore
• InteractError::Aborted will never be created when using the SyncWrapper interface

I'm proposing to remove this Result from the call signature

-pub async fn interact<F, R>(&self, f: F) -> Result<R, InteractError>
+pub async fn interact<F, R>(&self, f: F) -> R

What happens in the error cases?

It just panics. 💥

Why is this a good thing?

I think most users of the library will just log an error and ignore the InteractError anyways. The default behavior should be to escalate the error rather than trying to handle it "gracefully".

For those people that really know what they're doing and need to handle panics an additional interact_safe could be added which does behave exactly like before.

I'm interested to hear from users of this library if they ever had a need for handling a panic (or abortion) after an interact call.