print a nicer error on timeout/cancel

[Stebalien]

Don't say "context canceled". Just say "canceled".

fixes ipfs/kubo#3785

[Stebalien]

We don't need a cmdkit.Error, we just need the error message.

[Stebalien]

This now prints:

Error: canceled

When a user hits ^C and:

Error: timed out

When the command times out.

[djdv]

I like this change either way, but wonder if prepending "request" is necessary.

Error: request canceled
Error: request timed-out

vs

Error: canceled
Error: timed-out

Edit: I feel like the verbose line information etc. was the problem in the original issue rather than an additional word "context".
Or perhaps, cancellations shouldn't be considered an error to the canceler.
Need @whyrusleeping's insight 👁

[Stebalien]

Edit: I feel like the verbose line information etc. was the problem in the original issue rather than an additional word "context".

That was probably the motivation for the original issue but I've also seen users confused about the term "context". That's a go-ism and, really, an implementation detail.

At the end of the day, we just want to communicate:

1. That the command failed.
2. Why it failed.

I'd also like to avoid "request" because that really doesn't have anything to do with what the user is trying to do (they shouldn't even need to consider the fact that ipfs ... is sending a "request" to the daemon).

[Stebalien]

We can bikeshed this in a followup PR.