Fix old reopen bug

Author: youngcoder45

README.md

@@ -7,7 +7,7 @@ A simple and efficient ModMail bot for Discord, built with `discord.py`. This bo
 - **DM to ModMail**: Users can DM the bot to open a ticket.
 - **Moderator Replies**: Moderators can reply directly from the modmail channel.
 - **Slash Commands**: Includes slash commands for managing modmail.
-- **Session Management**: Handles user sessions with timeouts and locking.
+- **Session Management**: Keeps one active thread per user until a moderator closes it.
 - **Logging**: Logs events to a file and console.
 
 ## Setup
@@ -39,7 +39,6 @@ A simple and efficient ModMail bot for Discord, built with `discord.py`. This bo
     GUILD_ID=your_guild_id
     MODMAIL_CHANNEL_ID=your_modmail_channel_id
     LOG_LEVEL=INFO
-    MODMAIL_RESET_SECONDS=600
     MODMAIL_CONFIRM_TIMEOUT_SECONDS=300
     ```
 

cogs/modmail.py

@@ -471,22 +471,6 @@ async def _persist_sessions_to_file(self):
         except Exception:
             logger.exception("modmail: failed to persist sessions to file")
 
-    def _is_session_expired(self, session: Dict[str, Any]) -> bool:
-        reset_seconds = int(getattr(self.config, 'modmail_reset_seconds', 0) or 0)
-        if reset_seconds <= 0:
-            return False
-
-        last_activity = session.get('last_activity')
-        if not last_activity:
-            return False
-
-        try:
-            last_dt = datetime.fromisoformat(str(last_activity))
-        except Exception:
-            return False
-
-        return (datetime.utcnow() - last_dt) > timedelta(seconds=reset_seconds)
-
     def _is_session_closed(self, session: Dict[str, Any]) -> bool:
         state = str(session.get('state') or '').lower()
         return state in {'closed', 'resolved'}
@@ -585,7 +569,7 @@ async def handle_dm_message(self, message: discord.Message):
                 thread: Optional[discord.Thread] = None
                 session_active = False
                 if session and isinstance(session, dict):
-                    if not self._is_session_closed(session) and not self._is_session_expired(session):
+                    if not self._is_session_closed(session):
                         thread = await self._get_thread_from_session(session, main_channel)
                         session_active = thread is not None
 
@@ -597,7 +581,7 @@ async def handle_dm_message(self, message: discord.Message):
                         pending = None
 
                     if pending is None:
-                        # First DM (or after close/expiry): queue this message and ask.
+                        # First DM (or after explicit close): queue this message and ask.
                         await self._queue_pending_message(user_id, message)
                         prompt = await self._send_modmail_confirmation_prompt(message.author)
                         self._pending_confirmations[user_id]['prompt_message_id'] = prompt.id

utils/config.py

@@ -20,10 +20,9 @@ class Config(BaseSettings):
     topgg_webhook_secret: Optional[str] = Field(default=None)
     redis_url: Optional[str] = Field(default=None)
 
-    # Modmail settings
-    modmail_channel_id: Optional[int] = Field(default=None)
-    modmail_reset_seconds: int = Field(default=600)
-    modmail_confirm_timeout_seconds: int = Field(default=300)
+    # Modmail settings
+    modmail_channel_id: Optional[int] = Field(default=None)
+    modmail_confirm_timeout_seconds: int = Field(default=300)
 
     # CodeBuddy settings
     question_channel_id: Optional[int] = Field(default=None)