opencode/picoclaw
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #1479 from securityguy/fix/claude-cli-error-surfacing

Browse source 
fix(claude_cli): surface stdout in error when CLI exits non-zero
This commit is contained in:
Mauro 2026-03-17 22:16:42 +01:00 • committed by GitHub
commit 74f2a1513e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
pkg/providers/claude_cli_provider.go
View file

@ -50,10 +50,18 @@ func (p *ClaudeCliProvider) Chat(
cmd.Stderr = &stderr cmd.Stderr = &stderr
if err := cmd.Run(); err != nil { if err := cmd.Run(); err != nil {
if stderrStr := stderr.String(); stderrStr != "" { stderrStr := strings.TrimSpace(stderr.String())
stdoutStr := strings.TrimSpace(stdout.String())
switch {
case stderrStr != "" && stdoutStr != "":
return nil, fmt.Errorf("claude cli error: %w\nstderr: %s\nstdout: %s", err, stderrStr, stdoutStr)
case stderrStr != "":
return nil, fmt.Errorf("claude cli error: %s", stderrStr) return nil, fmt.Errorf("claude cli error: %s", stderrStr)
case stdoutStr != "":
return nil, fmt.Errorf("claude cli error: %w\noutput: %s", err, stdoutStr)
default:
return nil, fmt.Errorf("claude cli error: %w", err)
} }
return nil, fmt.Errorf("claude cli error: %w", err)
} }
return p.parseClaudeCliResponse(stdout.String()) return p.parseClaudeCliResponse(stdout.String())