Ruff test

[IanDTM]

This pull request primarily improves code quality and maintainability in the backend of the Whack-a-Mole game by adding comprehensive docstrings to WebSocket event handlers, refactoring logic for clarity, and enhancing thread safety. It also introduces minor development environment tweaks.

Backend code improvements:

• Added detailed docstrings to the handle_start_game and handle_whack WebSocket event handlers in app.py, clearly documenting their purpose, arguments, emitted events, side effects, thread safety, and example usage. [1] [2]
• Refactored the whack attempt logic by extracting it into a new helper function _process_whack_attempt, improving code readability and ensuring thread-safe state updates.
• Simplified the handle_whack function by removing redundant checks, using the helper for processing, and unifying result emission and logging.

Development environment/configuration:

• Added an editor ruler at 90 characters in .vscode/settings.json to encourage consistent code formatting.
• Removed the --fix argument from the ruff hook in .pre-commit-config.yaml, likely to prevent automatic code modifications during pre-commit checks.

[IanDTM]

@copilot please look at the code and look for any security issues

[Copilot]

@IanDTM I've opened a new pull request, #5, to work on those changes. Once the pull request is ready, I'll request review from you.

[Copilot]

Pull request overview

This PR enhances the Whack-a-Mole game backend code quality through comprehensive documentation and refactoring. The main focus is on adding detailed docstrings to WebSocket event handlers and extracting game logic into a reusable helper function for improved maintainability.

• Added comprehensive docstrings to handle_start_game and handle_whack WebSocket handlers documenting parameters, emitted events, side effects, and thread safety considerations
• Refactored whack processing logic into a separate _process_whack_attempt helper function to improve code organization and ensure consistent thread-safe state updates
• Adjusted development environment settings (editor ruler, pre-commit hook configuration)

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
whack-a-mole/backend/app.py	Added detailed docstrings to event handlers, extracted whack logic into helper function, improved logging and code clarity
whack-a-mole/.pre-commit-config.yaml	Removed --fix argument from ruff hook to prevent automatic code modifications during pre-commit
.vscode/settings.json	Added editor ruler at 90 characters for consistent code formatting

[Copilot]

@IanDTM I've opened a new pull request, #6, to work on those changes. Once the pull request is ready, I'll request review from you.