From a52be47121293caeaf9b8e24deacf5fe09676df6 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Fri, 2 Jan 2026 21:06:47 +0000
Subject: [PATCH] Fix shutdown error: file descriptor cannot be a negative
 integer

Improved error handling in bot shutdown process:
- Check connection state before attempting to quit IRC
- Use thread-safe event loop shutdown
- Filter expected shutdown errors from error logs
- Changed shutdown errors to debug level logging

This prevents the "file descriptor cannot be a negative integer (-1)"
error from appearing as an ERROR when cleanly shutting down the bot.
---
 chatbot.py | 31 ++++++++++++++++++++++---------
 1 file changed, 22 insertions(+), 9 deletions(-)

diff --git a/chatbot.py b/chatbot.py
index 5cd62d5..5df2194 100644
--- a/chatbot.py
+++ b/chatbot.py
@@ -232,9 +232,14 @@ class EugenBot(irc.bot.SingleServerIRCBot):
             self.logger.info("Starting IRC bot...")
             super().start()
         except Exception as e:
-            self.logger.error(f"Bot error: {str(e)}")
-            if self.dashboard:
-                self.dashboard.log_event("error", {"error": f"Bot crashed: {str(e)}"})
+            # Ignore errors related to shutdown (file descriptor issues)
+            error_msg = str(e)
+            if "file descriptor" in error_msg.lower() and not self.is_running:
+                self.logger.debug(f"Shutdown cleanup: {error_msg}")
+            else:
+                self.logger.error(f"Bot error: {error_msg}")
+                if self.dashboard:
+                    self.dashboard.log_event("error", {"error": f"Bot crashed: {error_msg}"})
 
     def stop(self):
         """Stop the bot"""
@@ -243,14 +248,22 @@ class EugenBot(irc.bot.SingleServerIRCBot):
         if self.dashboard:
             self.dashboard.log_event("info", {"message": "Shutting down..."})
 
+        # Safely disconnect from IRC
         try:
-            self.connection.quit("Bot shutting down")
-            self.die()
-        except:
-            pass
+            if hasattr(self, 'connection') and self.connection and self.connection.connected:
+                self.connection.quit("Bot shutting down")
+        except Exception as e:
+            self.logger.debug(f"Error during disconnect: {str(e)}")
 
-        if self.loop:
-            self.loop.stop()
+        # Stop IRC bot
+        try:
+            self.die()
+        except Exception as e:
+            self.logger.debug(f"Error during die(): {str(e)}")
+
+        # Stop event loop
+        if self.loop and self.loop.is_running():
+            self.loop.call_soon_threadsafe(self.loop.stop)
 
 
 def check_env_file():