fix(errors): use errors.As for wrapped error handling (#462)

IsSilentExit used type assertion which fails on wrapped errors.
Changed to errors.As to properly unwrap and detect SilentExitError.

Added test to verify wrapped error detection works.
This commit is contained in:
Daniel Sauer
2026-01-16 20:05:59 +01:00
committed by GitHub
parent 139f3aeba3
commit 8332a719ab
2 changed files with 11 additions and 7 deletions

View File

@@ -1,6 +1,9 @@
package cmd
import "fmt"
import (
"errors"
"fmt"
)
// SilentExitError signals that the command should exit with a specific code
// without printing an error message. This is used for scripting purposes
@@ -19,12 +22,14 @@ func NewSilentExit(code int) *SilentExitError {
}
// IsSilentExit checks if an error is a SilentExitError and returns its code.
// Uses errors.As to properly handle wrapped errors.
// Returns 0 and false if err is nil or not a SilentExitError.
func IsSilentExit(err error) (int, bool) {
if err == nil {
return 0, false
}
if se, ok := err.(*SilentExitError); ok {
var se *SilentExitError
if errors.As(err, &se) {
return se.Code, true
}
return 0, false

View File

@@ -54,9 +54,9 @@ func TestNewSilentExit(t *testing.T) {
func TestIsSilentExit(t *testing.T) {
tests := []struct {
name string
err error
wantCode int
name string
err error
wantCode int
wantIsSilent bool
}{
{"nil error", nil, 0, false},
@@ -64,7 +64,7 @@ func TestIsSilentExit(t *testing.T) {
{"silent exit code 1", NewSilentExit(1), 1, true},
{"silent exit code 2", NewSilentExit(2), 2, true},
{"other error", errors.New("some error"), 0, false},
// Note: wrapped errors require errors.As fix - see PR #462
{"wrapped silent exit", fmt.Errorf("wrapped: %w", NewSilentExit(5)), 5, true},
}
for _, tt := range tests {
@@ -81,7 +81,6 @@ func TestIsSilentExit(t *testing.T) {
}
func TestSilentExitError_Is(t *testing.T) {
// Test that SilentExitError works with errors.Is
err := NewSilentExit(1)
var target *SilentExitError
if !errors.As(err, &target) {