fix: missed one, thanks unit testing

Author: dotkas

actions/update_actions/scanner.py

@@ -10,16 +10,29 @@
 def find_uses(obj) -> list[str]:
     """Recursively find all 'uses' values in a YAML structure."""
     found = []
-    if isinstance(obj, dict):
-        if isinstance(obj.get("steps"), list):
-            for step in obj["steps"]:
-                if isinstance(step, dict) and isinstance(step.get("uses"), str):
-                    found.append(step["uses"])
-        for value in obj.values():
-            found.extend(find_uses(value))
-    elif isinstance(obj, list):
+
+    # Guard: Handle lists
+    if isinstance(obj, list):
         for item in obj:
             found.extend(find_uses(item))
+        return found
+
+    # Guard: Only process dicts from here
+    if not isinstance(obj, dict):
+        return found
+
+    # Process steps if present
+    if isinstance(obj.get("steps"), list):
+        for step in obj["steps"]:
+            if not isinstance(step, dict):
+                continue
+            if isinstance(step.get("uses"), str):
+                found.append(step["uses"])
+
+    # Recurse into all dict values
+    for value in obj.values():
+        found.extend(find_uses(value))
+
     return found
 
 

actions/update_actions/tests/test_scanner.py

@@ -136,8 +136,8 @@ def test_apply_updates_preserves_non_uses_variables(self):
         updated = scanner.apply_updates(text, upgrades)
 
         # Verify that the uses entries were updated
-        self.assertIn("actions/create-github-app-token@v2.2.1", updated)
-        self.assertIn("actions/setup-python@v6.2.0", updated)
+        self.assertIn("actions/create-github-app-token@v2", updated)
+        self.assertIn("actions/setup-python@v6", updated)
 
         # Verify that non-uses variables are preserved exactly as-is
         self.assertIn('PYTHON_VERSION: "3.14"', updated)