We should embed errors when reporting upwards

[tobsan]

A common code path is something like:

val, err := SomeFunc(somearg)
if err != nil {
    return fmt.Errorf("Couldn't do some thing with %s", somearg)
}

However, in doing that we lose some error context, because we throw away the actual error message from SomeFunc. A better way would be to do the below, which we ideally should do everywhere.

return fmt.Errorf("Couldn't do some thing with %s: %w", somearg, err)

[obbardc]

I wonder if there's a golangci linter for this as well?

[tobsan]

Looks like the wrapcheck linter does this, or something very close to it.

[obbardc]

also created go-debos/fakemachine#278 for this

[obbardc]

The wrapcheck linter is a bit crazy, needs a proper error message for all errors rather than simply returning err. Let's see if AI can automatically fix it for me.