Feat: Replace flake8, black and husky with ruff + pre-commit

[MarceloRobert]

Important

This PR will be broken in 2, first we will reformat the files bypassing all checks and then we will push the configs and a .git-blame-ignore-revs so that we can use as many rules as possible, all while keeping the blame ability

Changes

• Replaced Flake8 + Black with Ruff
• Updated CI to use Ruff
• Auto-fixed backend linting and formatting issues
• Replaced Husky with Pre-commit
• Updated documentation

Note

I decided to ignore the /commands folder from linting as there were a lot of functions that went over the complexity limit of 10 (which was previously 11) not only because there would be a lot of refactoring but also because it is pretty easy for the commands to get complex.

How to test

• Update frontend dependencies with pnpm install (this will uninstall Husky)
• Update backend dependencies with poetry install (this will install Ruff and Pre-commit and uninstall Black and Flake8)
• Check backend formatting with poetry run ruff format --check
• Check backend linting with poetry run ruff check
• Verify Git is no longer using a custom hooks path (Husky removed) with git config --get core.hooksPath
  • Expected: no output (or empty value).
• Install repository hooks from pre-commit config: poetry -C backend run pre-commit install --hook-type pre-commit --hook-type pre-push --install-hooks
  • Expected: installation messages for both pre-commit and pre-push.
• Validate pre-commit config: poetry -C backend run pre-commit validate-config
• Run all pre-commit stage checks manually: poetry -C backend run pre-commit run --hook-stage pre-commit --all-files
  • Expected: Ruff checks + frontend lint/typecheck hooks pass.
• Run all pre-push stage checks manually: poetry -C backend run pre-commit run --hook-stage pre-push --all-files
  • Expected: pre-push stage passes (including the frontend pre-push guard hook).
• Smoke-test with real hook execution:
  • Make a small local change and run git commit to confirm pre-commit hooks execute.
  • Run git push --dry-run (or push to a test branch) to confirm pre-push hooks execute.

Reference / Useful Links

• https://docs.astral.sh/ruff/formatter/black/
• https://docs.astral.sh/ruff/configuration/
• https://docs.astral.sh/ruff/settings/
• https://docs.astral.sh/ruff/rules/
• https://www.flake8rules.com/
• Allow one empty newline after class header astral-sh/ruff#21380

Closes #1845

[dede999]

I don't like pretty much the idea of so many unrelated changes, but as they were made by the new linter rules, there is no real point in complaining about them. I only many a small suggestion but it must not lock the merge of the PR it it takes too long

[dede999]

nit: is it possible to extract this script (lines 31-47) to a shell script? Would it take too much time?

[MarceloRobert]

I could but then we would be going back to having separate files for the pre-push of the frontend, I don't think this is needed for now