fix: guard fmt.Errorf %w against nil err

errors.Wrap(nil, "msg") returned nil, but fmt.Errorf("msg: %w", nil)
returns a non-nil error. Add nil check before wrapping.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Ahmet Alp Balkan
2026-03-08 16:50:28 -07:00
parent 44c37100bf
commit 7eb2221a43
2 changed files with 8 additions and 4 deletions

View File

@@ -42,8 +42,10 @@ func (op SwitchOp) Run(_, stderr io.Writer) error {
if err != nil {
return fmt.Errorf("failed to switch context: %w", err)
}
err = printer.Success(stderr, "Switched to context \"%s\".", printer.SuccessColor.Sprint(newCtx))
return fmt.Errorf("print error: %w", err)
if err = printer.Success(stderr, "Switched to context \"%s\".", printer.SuccessColor.Sprint(newCtx)); err != nil {
return fmt.Errorf("print error: %w", err)
}
return nil
}
// switchContext switches to specified context name.

View File

@@ -53,8 +53,10 @@ func (kf *kubeconfigFile) Reset() error {
if err := kf.Truncate(0); err != nil {
return fmt.Errorf("failed to truncate file: %w", err)
}
_, err := kf.Seek(0, 0)
return fmt.Errorf("failed to seek in file: %w", err)
if _, err := kf.Seek(0, 0); err != nil {
return fmt.Errorf("failed to seek in file: %w", err)
}
return nil
}
func kubeconfigPath() (string, error) {