fix: address CodeRabbit review feedback

1. Add stacklevel=2 to all warnings.warn() calls for better tracebacks
   - Helps developers see where the warning originates
2. Fix SUPPORTED_READERS key in fallback test from 'itk' to 'itkreader'
   - Ensures test actually exercises the warn-and-skip fallback path
   - Was using wrong key, preventing the patch from working

These fixes improve warning clarity and test effectiveness per CodeRabbit review.

Signed-off-by: Cipher <cipher@openclaw.ai>

Author: Cipher

monai/transforms/io/array.py

@@ -216,7 +216,7 @@ def __init__(
                         f"If you want to use an alternative reader, do not specify the `reader` argument."
                     ) from e
                 except TypeError:  # the reader doesn't have the corresponding args/kwargs
-                    warnings.warn(f"{_r} is not supported with the given parameters {args} {kwargs}.")
+                    warnings.warn(f"{_r} is not supported with the given parameters {args} {kwargs}.", stacklevel=2)
                     self.register(the_reader())
             elif inspect.isclass(_r):
                 try:
@@ -229,7 +229,7 @@ def __init__(
                         f"If you want to use an alternative reader, do not specify the `reader` argument."
                     ) from e
                 except TypeError:  # the reader doesn't have the corresponding args/kwargs
-                    warnings.warn(f"{_r.__name__} is not supported with the given parameters {args} {kwargs}.")
+                    warnings.warn(f"{_r.__name__} is not supported with the given parameters {args} {kwargs}.", stacklevel=2)
                     self.register(_r())
             else:
                 self.register(_r)  # reader instance, ignoring the constructor args/kwargs
@@ -244,7 +244,7 @@ def register(self, reader: ImageReader):
 
         """
         if not isinstance(reader, ImageReader):
-            warnings.warn(f"Preferably the reader should inherit ImageReader, but got {type(reader)}.")
+            warnings.warn(f"Preferably the reader should inherit ImageReader, but got {type(reader)}.", stacklevel=2)
         self.readers.append(reader)
 
     def __call__(self, filename: Sequence[PathLike] | PathLike, reader: ImageReader | None = None):

tests/transforms/test_load_image.py

@@ -536,13 +536,13 @@ def test_unspecified_reader_falls_back_silently(self):
 
         # Patch a few readers to fail (e.g., ITKReader)
         try:
-            original_itk = SUPPORTED_READERS.get("itk")
+            original_itk = SUPPORTED_READERS.get("itkreader")
 
             def failing_reader(*args, **kwargs):
                 raise OptionalImportError("itk not installed")
 
             # Temporarily replace ITKReader with a failing version
-            SUPPORTED_READERS["itk"] = failing_reader
+            SUPPORTED_READERS["itkreader"] = failing_reader
 
             # Capture log output to verify warn-and-skip was invoked
             with self.assertLogs("LoadImage", level="DEBUG") as cm:
@@ -555,7 +555,7 @@ def failing_reader(*args, **kwargs):
         finally:
             # Restore original reader
             if original_itk is not None:
-                SUPPORTED_READERS["itk"] = original_itk
+                SUPPORTED_READERS["itkreader"] = original_itk
 
     def test_explicit_reader_available_succeeds(self):
         """When the user explicitly names a reader whose package IS installed, no exception is raised."""