fix: Resolve IDisposable memory leaks in image source strategies (#36)

* fix: Resolve IDisposable memory leaks in image source strategies

- Fix Mat object disposal in FileImageSourceStrategy when empty frames/images
- Fix Mat object disposal in CameraImageSourceStrategy when empty frames
- Add proper exception handling with disposal in GetImageFrameAsync
- Prevents OpenCV Mat memory leaks identified by CodeQL analysis

Resolves #35

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* fix: Add comprehensive exception handling for Mat disposal

- Wrap frame operations in try-catch blocks to ensure disposal
- Prevent memory leaks when Read() or Empty() methods throw exceptions
- Address GitHub Advanced Security Bot recommendations
- Improve robustness of OpenCV resource management

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Stephen Theodore <wssmash93@gmail.com>
Co-authored-by: Claude <noreply@anthropic.com>

Author: 3 people

.claude/settings.local.json

@@ -21,7 +21,9 @@
       "Bash(gh pr:*)",
       "Bash(dotnet build:*)",
       "Bash(gh label:*)",
-      "Bash(gh issue edit:*)"
+      "Bash(gh issue edit:*)",
+      "Bash(git commit:*)",
+      "Bash(git push:*)"
     ],
     "deny": [],
     "ask": []

src/MetaExtractor.Infrastructure/Services/CameraImageSourceStrategy.cs

@@ -78,14 +78,23 @@ private void InitializeCamera()
     public Task<Mat?> GetNextFrameAsync()
     {
         var frame = new Mat();
-        _capture.Read(frame);
+        try
+        {
+            _capture.Read(frame);
+
+            if (frame.Empty())
+            {
+                frame.Dispose();
+                return Task.FromResult<Mat?>(null);
+            }
 
-        if (frame.Empty())
+            return Task.FromResult<Mat?>(frame);
+        }
+        catch
         {
+            frame.Dispose();
             return Task.FromResult<Mat?>(null);
         }
-
-        return Task.FromResult<Mat?>(frame);
     }
 
     public void Dispose()

src/MetaExtractor.Infrastructure/Services/FileImageSourceStrategy.cs

@@ -57,14 +57,23 @@ public FileImageSourceStrategy(string filePath)
             return Task.FromResult<Mat?>(null);
 
         var frame = new Mat();
-        _videoCapture.Read(frame);
+        try
+        {
+            _videoCapture.Read(frame);
+
+            if (frame.Empty())
+            {
+                frame.Dispose();
+                return Task.FromResult<Mat?>(null);
+            }
 
-        if (frame.Empty())
+            return Task.FromResult<Mat?>(frame);
+        }
+        catch
         {
+            frame.Dispose();
             return Task.FromResult<Mat?>(null);
         }
-
-        return Task.FromResult<Mat?>(frame);
     }
 
     private Task<Mat?> GetImageFrameAsync()
@@ -76,20 +85,23 @@ public FileImageSourceStrategy(string filePath)
         }
 
         _isImageRead = true;
-        
+
+        Mat? mat = null;
         try
         {
-            var mat = new Mat(_filePath, ImreadModes.Color);
+            mat = new Mat(_filePath, ImreadModes.Color);
 
             if (mat.Empty())
             {
+                mat.Dispose();
                 return Task.FromResult<Mat?>(null);
             }
 
             return Task.FromResult<Mat?>(mat);
         }
         catch (Exception)
         {
+            mat?.Dispose();
             return Task.FromResult<Mat?>(null);
         }
     }