Handle exceptions from property getters during member enumeration

inspect.getmembers in Python 3.13 only suppresses AttributeError when
calling attribute getters. Any other exception (RuntimeError, ValueError,
etc.) propagates uncaught, causing fire.Fire to crash on --help and bare
invocation when a component has a property whose getter raises.

Add GetSafeMembers to inspectutils, which wraps getattr in a broad
exception handler and falls back to None for any attribute that raises.
Replace the two unguarded inspect.getmembers call sites in core.py
(_IsHelpShortcut) and completion.py (VisibleMembers) with GetSafeMembers.

Fixes #672

Author: BhargavKumarNath

fire/completion.py

@@ -359,7 +359,7 @@ def VisibleMembers(component, class_attrs=None, verbose=False):
   if isinstance(component, dict):
     members = component.items()
   else:
-    members = inspect.getmembers(component)
+    members = inspectutils.GetSafeMembers(component)
 
   # If class_attrs has not been provided, compute it.
   if class_attrs is None:

fire/core.py

@@ -225,7 +225,7 @@ def _IsHelpShortcut(component_trace, remaining_args):
         _, remaining_kwargs, _ = _ParseKeywordArgs(remaining_args, fn_spec)
         show_help = target in remaining_kwargs
       else:
-        members = dict(inspect.getmembers(component))
+        members = dict(inspectutils.GetSafeMembers(component))
         show_help = target not in members
 
   if show_help:

fire/inspectutils.py

@@ -347,3 +347,33 @@ def IsCoroutineFunction(fn):
     return inspect.iscoroutinefunction(fn)
   except:  # pylint: disable=bare-except
     return False
+
+def GetSafeMembers(component, predicate=None):
+  """Returns members of a component, skipping attributes that raise on access.
+
+  Like inspect.getmembers, but catches all exceptions raised by property
+  getters or other dynamic attributes during member enumeration. Members
+  that raise are included with a value of None, preserving the member name
+  in the result so that callers can detect its presence without crashing.
+
+  This behaviour differs from inspect.getmembers in Python 3.13+, which
+  only suppresses AttributeError and lets all other exceptions propagate.
+
+  Args:
+    component: The object whose members to retrieve.
+    predicate: An optional predicate to filter members by value.
+  Returns:
+    A list of (name, value) pairs sorted by name. Members whose getters
+    raised are included as (name, None).
+  """
+  results = []
+  for key in dir(component):
+    try:
+      value = getattr(component, key)
+    except Exception:  # pylint: disable=broad-except
+      value = None
+    if predicate and not predicate(value):
+      continue
+    results.append((key, value))
+  results.sort(key=lambda pair: pair[0])
+  return results

fire/inspectutils_test.py

@@ -125,6 +125,43 @@ def testInfoNoDocstring(self):
     info = inspectutils.Info(tc.NoDefaults)
     self.assertEqual(info['docstring'], None, 'Docstring should be None')
 
+  def testGetSafeMembersRaisingProperty(self):
+    class ComponentWithRaisingProperty:
+      @property
+      def status(self):
+        raise RuntimeError('backend unavailable')
+
+    component = ComponentWithRaisingProperty()
+    members = dict(inspectutils.GetSafeMembers(component))
+    self.assertIn('status', members)
+    self.assertIsNone(members['status'])
+
+  def testGetSafeMembersWorkingProperty(self):
+    class ComponentWithWorkingProperty:
+      @property
+      def status(self):
+        return 'all good'
+
+    component = ComponentWithWorkingProperty()
+    members = dict(inspectutils.GetSafeMembers(component))
+    self.assertIn('status', members)
+    self.assertEqual(members['status'], 'all good')
+
+  def testGetSafeMembersMixedProperties(self):
+    class ComponentWithMixedProperties:
+      @property
+      def good(self):
+        return 'ok'
+      @property
+      def bad(self):
+        raise ValueError('unavailable')
+
+    component = ComponentWithMixedProperties()
+    members = dict(inspectutils.GetSafeMembers(component))
+    self.assertIn('good', members)
+    self.assertEqual(members['good'], 'ok')
+    self.assertIn('bad', members)
+    self.assertIsNone(members['bad'])
 
 if __name__ == '__main__':
   testutils.main()