Merge pull request '[agent/claude:opus] Fix CodeRabbit findings. Verify each against current code, f...' (#3) from agent/fix-coderabbit-findings--verify-each-aga into main

Virgil · Virgil · commit 876119cddeff · 2026-03-17T13:53:58.000Z
diff --git a/git.go b/git.go
@@ -290,9 +290,12 @@ func (e *GitError) Error() string {
 	stderr := strings.TrimSpace(e.Stderr)
 
 	if stderr != "" {
-		return coreerr.E("", fmt.Sprintf("git command %q failed: %s", cmd, stderr), nil).Error()
+		return fmt.Sprintf("git command %q failed: %s", cmd, stderr)
 	}
-	return coreerr.E("", fmt.Sprintf("git command %q failed", cmd), e.Err).Error()
+	if e.Err != nil {
+		return fmt.Sprintf("git command %q failed: %v", cmd, e.Err)
+	}
+	return fmt.Sprintf("git command %q failed", cmd)
 }
 
 // Unwrap returns the underlying error for error chain inspection.

Original file line number	Diff line number	Diff line change
`@@ -290,9 +290,12 @@ func (e *GitError) Error() string {`
`290`	`290`	`stderr := strings.TrimSpace(e.Stderr)`
`291`	`291`
`292`	`292`	`if stderr != "" {`
`293`		`- return coreerr.E("", fmt.Sprintf("git command %q failed: %s", cmd, stderr), nil).Error()`
	`293`	`+ return fmt.Sprintf("git command %q failed: %s", cmd, stderr)`
`294`	`294`	`}`
`295`		`- return coreerr.E("", fmt.Sprintf("git command %q failed", cmd), e.Err).Error()`
	`295`	`+ if e.Err != nil {`
	`296`	`+ return fmt.Sprintf("git command %q failed: %v", cmd, e.Err)`
	`297`	`+ }`
	`298`	`+ return fmt.Sprintf("git command %q failed", cmd)`
`296`	`299`	`}`
`297`	`300`
`298`	`301`	`// Unwrap returns the underlying error for error chain inspection.`