fix: only log a message once

New handlers were registered every time the get_logger function was called, which caused the same log line to be printed multiple times in the log.

Initialise the logger once and just let get_logger return a reference instead.

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "fish_ai"
-version = "0.13.1"
+version = "0.13.2"
 authors = [{ name = "Bastian Fredriksson", email = "realiserad@gmail.com" }]
 description = "Provides core functionality for fish-ai, an AI plugin for the fish shell."
 readme = "README.md"

src/fish_ai/engine.py

@@ -25,34 +25,35 @@
 from os import access, R_OK
 from re import match
 
+logger = logging.getLogger()
 
-def get_args():
-    return list.copy(sys.argv[1:])
+if path.exists('/dev/log'):
+    # Syslog on Linux
+    handler = SysLogHandler(address='/dev/log')
+    logger.addHandler(handler)
+elif path.exists('/var/run/syslog'):
+    # Syslog on macOS
+    handler = SysLogHandler(address='/var/run/syslog')
+    logger.addHandler(handler)
+
+if get_config('log'):
+    handler = RotatingFileHandler(path.expanduser(get_config('log')),
+                                  backupCount=0,
+                                  maxBytes=1024*1024)
+    logger.addHandler(handler)
+
+if get_config('debug') == 'True':
+    logger.setLevel(logging.DEBUG)
 
 
 def get_logger():
-    logger = logging.getLogger()
-
-    if path.exists('/dev/log'):
-        # Syslog on Linux
-        handler = SysLogHandler(address='/dev/log')
-        logger.addHandler(handler)
-    elif path.exists('/var/run/syslog'):
-        # Syslog on macOS
-        handler = SysLogHandler(address='/var/run/syslog')
-        logger.addHandler(handler)
-
-    if get_config('log'):
-        handler = RotatingFileHandler(path.expanduser(get_config('log')),
-                                      backupCount=0,
-                                      maxBytes=1024*1024)
-        logger.addHandler(handler)
-
-    if get_config('debug') == 'True':
-        logger.setLevel(logging.DEBUG)
     return logger
 
 
+def get_args():
+    return list.copy(sys.argv[1:])
+
+
 def get_os():
     if platform.system() == 'Linux':
         if isfile('/etc/os-release'):