fix: use %w instead of %s for error wrapping in DetectPacInstallation

chmouel · claude · theakshaypant · commit aeb85ab653e9 · 2026-06-17T17:07:35.000+05:30
Preserves the error chain for errors.Is/errors.As callers.

Co-Authored-By: Claude &lt;noreply@anthropic.com&gt;
diff --git a/pkg/cmd/tknpac/bootstrap/bootstrap.go b/pkg/cmd/tknpac/bootstrap/bootstrap.go
@@ -271,7 +271,7 @@ func DetectPacInstallation(ctx context.Context, wantedNS string, run *params.Run
 	if err == nil {
 		return installed, cm.Namespace, nil
 	}
-	return installed, "", fmt.Errorf("could not detect Pipelines as Code configmap on the cluster, please specify the namespace in which pac is installed: %s", err.Error())
+	return installed, "", fmt.Errorf("could not detect Pipelines as Code configmap on the cluster, please specify the namespace in which pac is installed: %w", err)
 }
 
 func getConfigMap(ctx context.Context, run *params.Run) (*corev1.ConfigMap, error) {

Original file line number	Diff line number	Diff line change
`@@ -271,7 +271,7 @@ func DetectPacInstallation(ctx context.Context, wantedNS string, run *params.Run`
`271`	`271`	`if err == nil {`
`272`	`272`	`return installed, cm.Namespace, nil`
`273`	`273`	`}`
`274`		`- return installed, "", fmt.Errorf("could not detect Pipelines as Code configmap on the cluster, please specify the namespace in which pac is installed: %s", err.Error())`
	`274`	`+ return installed, "", fmt.Errorf("could not detect Pipelines as Code configmap on the cluster, please specify the namespace in which pac is installed: %w", err)`
`275`	`275`	`}`
`276`	`276`
`277`	`277`	`func getConfigMap(ctx context.Context, run params.Run) (corev1.ConfigMap, error) {`