Redundant Error Wrapping in errors.go

[jmesout]

Redundant Error Wrapping in errors.go

Severity: Medium
File: errors.go
Location: Lines 297-319

Description

The custom wrapError type and wrap method create nested errors, but Go's standard errors package already provides Unwrap and Is functionality. This custom implementation is redundant and conflicts with standard error handling practices.

Suggested Fix

Remove custom error wrapping and use standard errors package functionality:

// Replace custom wrapping with:
return fmt.Errorf("%w: %v", constError(err.msg), inner)

Labels: bug, refactor