Fix some small bugs and some lint...

"completer" is set to None, not False, when we don't have a completere.
This happens in Python 3.13 right now

In Python 3.13 we need to convert a FrameLocalsProxy to a dictionary in
order to call exec.

Some type lint was also done.

Author: rocky

trepan/interface.py

@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-#   Copyright (C) 2010, 2013, 2015, 2018, 2023 Rocky Bernstein
+#   Copyright (C) 2010, 2013, 2015, 2018, 2023, 2025 Rocky Bernstein
 #   <rocky@gnu.org>
 #
 #   This program is free software: you can redistribute it and/or modify
@@ -49,7 +49,7 @@ def confirm(self, prompt, default):
         it's okay. `prompt' is printed; user response is returned."""
         raise NotImplementedError(NotImplementedMessage)
 
-    def errmsg(self, str, prefix="** "):
+    def errmsg(self, msg, prefix="** "):
         """Common routine for reporting debugger error messages."""
         raise NotImplementedError(NotImplementedMessage)
 
@@ -74,10 +74,10 @@ def msg_nocr(self, msg):
         self.output.write(msg)
         return
 
-    def read_command(self, prompt):
+    def read_command(self, prompt: str):
         raise NotImplementedError(NotImplementedMessage)
 
-    def readline(self, prompt, add_to_history=True):
+    def readline(self, prompt: str, add_to_history=True):
         raise NotImplementedError(NotImplementedMessage)
 
     pass

trepan/interfaces/user.py

@@ -51,8 +51,11 @@
         write_history_file,
     )
 except ImportError:
-    def write_history_file(histfile: str):
+    def write_history_file(_: str):
         return
+    def parse_and_bind(_: str):
+        raise RuntimeError(f"Called {__name__}() when it doesn't exist")
+    read_history_file = set_completer = parse_and_bind
     have_complete = False
 else:
     have_complete = False
@@ -73,8 +76,8 @@ def __init__(self, inp=None, out=None, opts={}):
         self.histfile = None
 
         if self.input.use_history():
-            self.complete = self.user_opts["complete"] and have_complete
-            if self.complete:
+            self.complete = self.user_opts["complete"] if have_complete else None
+            if self.complete is not None:
                 parse_and_bind("tab: complete")
                 set_completer(self.complete)
                 pass
@@ -96,7 +99,8 @@ def __init__(self, inp=None, out=None, opts={}):
         return
 
     def user_write_history_file(self):
-        write_history_file(self.histfile)
+        if self.histfile is not None:
+            write_history_file(self.histfile)
 
     def close(self):
         """Closes both input and output"""
@@ -166,7 +170,7 @@ def read_command(self, prompt=""):
         # Do something with history?
         return line
 
-    def readline(self, prompt=""):
+    def readline(self, prompt="", add_to_history=True):
         if not self.readline == "prompt_toolkit":
             self.output.flush()
             pass

trepan/processor/command/python.py

@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-#  Copyright (C) 2009-2010, 2013, 2015, 2017, 2020, 2023-2024
+#  Copyright (C) 2009-2010, 2013, 2015, 2017, 2020, 2023-2025
 #  Rocky Bernstein
 #
 #  This program is free software: you can redistribute it and/or modify
@@ -119,11 +119,11 @@ def run(self, args):
         finally:
             sys.excepthook = old_sys_excepthook
 
+        interface = proc.intf[-1]
         # restore completion and our history if we can do so.
-        if hasattr(proc.intf[-1], "complete"):
+        if hasattr(interface, "complete") and interface.complete is not None:
             try:
                 from readline import parse_and_bind, set_completer
-
                 parse_and_bind("tab: complete")
                 set_completer(proc.intf[-1].complete)
             except ImportError:
@@ -182,8 +182,11 @@ def runcode(obj, code_obj):
     caller should be prepared to deal with it.
 
     """
+    # In 3.13 FrameProxy was introduced. Not sure
+    my_locals = dict(obj.locals) if not isinstance(obj.locals, dict) else obj.locals
+    my_globals = dict(obj.globals) if not isinstance(obj.globals, dict) else obj.globals
     try:
-        exec(code_obj, obj.locals, obj.globals)
+        exec(code_obj, my_locals, my_globals)
     except SystemExit:
         raise
     except Exception: