Improve Thread handling

[PavelLinearB]

✨ PR Description

Purpose: Implements a basic client-server chat system with multi-threading support and message broadcast functionality.

Main changes:

• Created client.py with socket connection handling and message receiving/sending capabilities
• Implemented server.py with multi-threaded client management and message broadcasting
• Added safety features including message size limits and graceful connection handling

This description was generated by LinearB AI and Added by gitStream

[gitstream-cm]

✨ PR Review Suggestion

Bug - src/client/client.py (Line 25)

Details:

Problem: Empty data from socket.recv() is not handled properly, leading to potential infinite loop
Fix: Add check for empty chunk indicating connection closed
Why: When connection is closed, recv() returns empty bytes - needs to break the loop

                chunk = socket.recv(4096)
+                if not chunk:  # Connection closed by remote
+                    break
                data += chunk

Bug - src/server/server.py (Line 35)

Details:

Problem: Similar to client, empty data from socket.recv() is not handled properly
Fix: Add check for empty chunk to detect connection closure
Why: Prevents infinite loop when client disconnects cleanly

                    chunk = self.socket.recv(4096)
+                    if not chunk:  # Connection closed by client
+                        self.signal = False
+                        break
                    data += chunk

This review was generated by LinearB AI and Added by gitStream