`GoType::ValueOrError` should store error type

[blaine-arcjet]

Currently, GoType::ValueOrError is only constructed if the Err type is String. This means that it always needs cleanup. If we instead stored the error type, we'd be able to define Result<u32, u32> and avoid the need to cleanup.