Avoid looking at local variables for deprecation warnings

Author: abrookins

redisvl/utils/utils.py

@@ -1,3 +1,4 @@
+import inspect
 import json
 from enum import Enum
 from functools import wraps
@@ -76,7 +77,7 @@ def deprecated_argument(argument: str, replacement: Optional[str] = None) -> Cal
     def wrapper(func):
         @wraps(func)
         def inner(*args, **kwargs):
-            argument_names = func.__code__.co_varnames
+            argument_names = list(inspect.signature(func).parameters.keys())
 
             if argument in argument_names:
                 warn(message, DeprecationWarning, stacklevel=2)

tests/unit/test_utils.py

@@ -1,3 +1,4 @@
+import warnings
 import numpy as np
 import pytest
 
@@ -237,3 +238,15 @@ async def test_func(dtype=None, vectorizer=None):
         with pytest.warns(DeprecationWarning):
             result = await test_func(dtype="float32")
         assert result == 1
+        
+    async def test_ignores_local_variable(self):
+        @deprecated_argument("dtype")
+        async def test_func(vectorizer=None):
+            # The presence of this variable should not trigger a warning
+            dtype = "float32"
+            return 1
+
+        # This will raise an error if any warning is emitted
+        with warnings.catch_warnings():
+            warnings.simplefilter("error")
+            await test_func()