Skip to content

Improve error handling and display#7

Merged
vdemeester merged 1 commit intomainfrom
feature/error-handling
Feb 4, 2026
Merged

Improve error handling and display#7
vdemeester merged 1 commit intomainfrom
feature/error-handling

Conversation

@vdemeester
Copy link
Copy Markdown
Owner

Summary

  • Silence Cobra's default error output and use styled logger instead
  • Strip Dagger traceparent spans from error messages for cleaner output
  • Show error details inline below failed steps and tasks

Before/After

Before:

X Step: init [50.1s] ERROR
Error: execution failed: ... [traceparent:abc123-def456]

After:

X Step: init [50.1s] ERROR
  → failed to resolve image "alpine:latest"
[ERROR] execution failed: failed to resolve image "alpine:latest"

Changes

  • cmd/chisel/run.go: Add SilenceErrors: true, use styled fail() helper
  • pkg/ui/errors.go: New CleanError/CleanErrorMessage helpers to strip Dagger traces
  • pkg/ui/logger.go: Show error message below failed steps/tasks

Test plan

  • Unit tests for CleanErrorMessage (8 tests)
  • Unit tests for error display in logger (3 tests)
  • All existing tests pass

🤖 Generated with Claude Code

- Silence Cobra's default error output, use styled logger instead
- Strip Dagger traceparent spans from error messages for cleaner output
- Show error details inline below failed steps and tasks
- Add CleanError/CleanErrorMessage helpers to pkg/ui

Before:
  X Step: init [50.1s] ERROR
  Error: execution failed: ... [traceparent:abc123-def456]

After:
  X Step: init [50.1s] ERROR
    → failed to resolve image "alpine:latest"
  [ERROR] execution failed: failed to resolve image "alpine:latest"

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@vdemeester vdemeester merged commit 1dfcfd0 into main Feb 4, 2026
3 checks passed
@vdemeester vdemeester deleted the feature/error-handling branch February 4, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant