Skip to content

fix: handle invalid images in deepfry - #107

Open
vycdev2 wants to merge 1 commit into
vycdev:developfrom
vycdev2:fix/handle-invalid-deepfry-images
Open

fix: handle invalid images in deepfry#107
vycdev2 wants to merge 1 commit into
vycdev:developfrom
vycdev2:fix/handle-invalid-deepfry-images

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Prevent the deepfry command from failing on malformed or unsupported image bytes.
  • Catch image inspection/processing errors and return a user-facing retry message instead of letting the command handler fail.
  • Add a regression test for invalid image data.

Verification

  • dotnet test --no-restore --filter FullyQualifiedName~ImageModuleTests --logger "console;verbosity=minimal" — passed (1/1).
  • dotnet build --no-restore — passed (0 errors; existing SDK/package warnings).
  • dotnet test --no-build --logger "console;verbosity=minimal" — 298 passed, 2 failed in existing MiscModuleTests because this runner uses globalization-invariant mode and cannot load tr-TR.
  • git diff --check upstream/develop...HEAD — passed.
  • dotnet format Morpheus.sln --verify-no-changes --no-restore — failed on pre-existing repository-wide formatting violations outside this change.

Risk

  • Low: only changes error handling for the deepfry command; valid image processing remains unchanged.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

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