refactor: replace process::exit with ExitCode

[ymelois]

---

Note

Replaces all std::process::exit(1) calls with idiomatic Rust ExitCode by extracting application logic into a run() function returning Result<(), AppError>, and introducing a new src/error.rs module with a thiserror-derived AppError enum covering config, GitHub, Mistral, and I/O errors.

^{Written by Mendral for commit 5896783.}

[pratrol]

Pratrol Triage Brief: 🟢 High Confidence

Pratrol assessed this pull request across the primary risk vectors below.

Risk Vector	Score	Tier
Contributor Context	69/100	🟡 Medium
Logic Risk (Mistral)	86/100	🟢 High
Overall Confidence	79/100	🟢 High

Summary: Refactored the main function to use ExitCode instead of process::exit for better error handling.

Key Signal: The refactoring improves error handling and code structure.

Recommendation: Review the changes for any potential edge cases in error handling.

[cloudflare-workers-and-pages]

Deploying pratrol with    Cloudflare Pages

Latest commit:	5896783
Status:	✅  Deploy successful!
Preview URL:	https://9fc73e64.pratrol.pages.dev
Branch Preview URL:	https://refactor-use-exit-code-resul.pratrol.pages.dev

View logs

[mendral-app]

🤖 Code Review

Assessment ✅

Clean, idiomatic refactor. No correctness, security, or data loss issues.

• AppError with #[error(transparent)] correctly preserves underlying error messages through the display chain, so operators still see meaningful error text in the single "Fatal error." log line.
• ? propagation in run() is correct — destructors run before exit, unlike process::exit.
• The new src/error.rs module is minimal and appropriate.

Note

Tag @mendral-app with feedback or questions. View session