Skip to content

fix: replace panic-based error handling with recoverable error paths#46

Open
Ishita-190 wants to merge 2 commits intomofa-org:mainfrom
Ishita-190:ishi
Open

fix: replace panic-based error handling with recoverable error paths#46
Ishita-190 wants to merge 2 commits intomofa-org:mainfrom
Ishita-190:ishi

Conversation

@Ishita-190
Copy link
Copy Markdown
Contributor

@Ishita-190 Ishita-190 commented Feb 25, 2026

Fixes: #41
Files changed:

Changes made:

  • Startup logic that previously unwrapped optional engine state now handles missing state gracefully.
  • Replaced expect("... mutex poisoned") with recovery logic using into_inner() so the UI can continue operating even if a mutex becomes poisoned after a prior failure.
  • Parsing paths no longer rely on runtime unwrap() calls. Failures now return clearer errors instead of panicking during execution.
  • Test assertions that previously used raw unwrap() now use expect() with descriptive messages to improve debugging when failures occur.

All changes are working as expected.

@Ishita-190
Copy link
Copy Markdown
Contributor Author

@BH3GEI please take a look!

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.

bug: potential runtime crashes from unwrap/expect usage

1 participant