Shutdown Python REPL correctly (#18437)

* Python REPL: Shutdown CHIP stack properly

* Make sure Storage is deallocated before ChipStack is Shutdown

Otherwise, we later fail to access the builtins.chipStack:
AttributeError: module 'builtins' has no attribute 'chipStack'

* Add comment why we need to explictily set _persistentStorage to None

Author: agners

src/controller/python/chip/ChipReplStartup.py

@@ -12,6 +12,7 @@
 import argparse
 import builtins
 import chip.FabricAdmin
+import atexit
 
 _fabricAdmins = None
 
@@ -98,6 +99,12 @@ def ReplInit(debug):
         logging.getLogger().setLevel(logging.WARN)
 
 
+def StackShutdown():
+    chip.FabricAdmin.FabricAdmin.ShutdownAll()
+    ChipDeviceCtrl.ChipDeviceController.ShutdownAll()
+    builtins.chipStack.Shutdown()
+
+
 def matterhelp(classOrObj=None):
     if (classOrObj is None):
         inspect(builtins.devCtrl, methods=True, help=True, private=False)
@@ -135,5 +142,7 @@ def mattersetdebug(enableDebugMode: bool = True):
 
 builtins.devCtrl = devCtrl
 
+atexit.register(StackShutdown)
+
 console.print(
     '\n\n[blue]Default CHIP Device Controller has been initialized to manage [bold red]fabricAdmins[0][blue], and is available as [bold red]devCtrl')

src/controller/python/chip/ChipStack.py

@@ -318,6 +318,9 @@ def setLogFunct(self, logFunct):
             self._ChipStackLib.pychip_Stack_SetLogFunct(logFunct)
 
     def Shutdown(self):
+        # Make sure PersistentStorage is destructed before chipStack
+        # to avoid accessing builtins.chipStack after destruction.
+        self._persistentStorage = None
         self.Call(lambda: self._ChipStackLib.pychip_Stack_Shutdown())
         self.networkLock = None
         self.completeEvent = None