Skip to content

feat: add eyre + color-eyre for rich error reporting - #40

Merged
getappz merged 1 commit into
masterfrom
worktree-mise-eyre
Jul 7, 2026
Merged

feat: add eyre + color-eyre for rich error reporting#40
getappz merged 1 commit into
masterfrom
worktree-mise-eyre

Conversation

@getappz

@getappz getappz commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Closes #37 (partial)

Summary

Add eyre + color-eyre dependencies and install color_eyre error handler at program start for color-coded errors with backtraces and source context — matches mise's error reporting pattern.

Test plan

  • \cargo check\ passes
  • \cargo build\ compiles

Notes for reviewers

  • Risk areas: color_eyre::install() panics on failure — same as mise's pattern
  • Backwards compatibility: additive, no breaking API changes

Summary by CodeRabbit

  • New Features
    • Improved error reporting at startup, making failures easier to understand and diagnose.
    • Added support for richer, more readable error messages throughout the app.

@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Review Change Stack

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 462bb4c6-6a73-4fdf-82f5-ec46d912116b

📥 Commits

Reviewing files that changed from the base of the PR and between e90c6fa and 7b40f59.

📒 Files selected for processing (2)
  • Cargo.toml
  • src/main.rs
 __________________________________________________________
< Sometimes, I pretend to be a compiler to feel important. >
 ----------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
📝 Walkthrough

Walkthrough

Adds eyre and color-eyre crates as dependencies in Cargo.toml and installs the color_eyre error handler at the start of main(), before CLI argument parsing.

Changes

Error Reporting Setup

Layer / File(s) Summary
Dependency and handler installation
Cargo.toml, src/main.rs
Adds eyre and color-eyre dependencies and installs color_eyre::install() in main() prior to Cli::parse(), panicking on install failure.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Related issues: #37 (partial — adds eyre/color-eyre for rich error reporting; other checklist items such as build.rs, edition 2024, thiserror migration, and tooling config remain unaddressed)

Suggested labels: dependencies, enhancement

Suggested reviewers: none identified

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The PR implements the #37 objective to add eyre/color-eyre rich error reporting, and the remaining checklist items are out of scope.
Out of Scope Changes check ✅ Passed The diff only adds eyre/color-eyre support and startup installation, with no unrelated changes visible.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title is concise and accurately summarizes the main change: adding eyre/color-eyre for richer error reporting.
Description check ✅ Passed The description includes summary, test plan, and reviewer notes, matching the template; only the checklist formatting is slightly different.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch worktree-mise-eyre

Comment @coderabbitai help to get the list of available commands.

Installs color_eyre at program start for color-coded errors
with backtraces and source context - matches mise pattern
@getappz
getappz force-pushed the worktree-mise-eyre branch from e90c6fa to 7b40f59 Compare July 7, 2026 10:08
@getappz
getappz merged commit 33acb0f into master Jul 7, 2026
8 of 9 checks passed
@getappz
getappz deleted the worktree-mise-eyre branch July 7, 2026 10:09
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

adopt mise project practices: edition, errors, tooling

1 participant