Introduce an asynchronous Redis instance to be used in async functions.
Also, this change migrates most of the sync cache usage to the new async
cache.

misc: Add Redis async cache

Convert `MobyGamesHandler` methods to be asynchronous, and use an
`httpx` async client, instead of `requests` sync client.

Creates an example config file for Batocera/RetroBat-based mappings, as popular distributions so should hopefully save people a few hours.

Tested with a setup using RetroBat v6.0.3 folder structures, and only keeping matches for which I'm certain.

Create config.batocera-retrobat.yml

…ests

misc: Migrate MobyGamesHandler to async

Updates after discovering all the extra cool icons already available (but not advertised in supported platforms!) and sorting.

Update config.batocera-retrobat.yml

For filesystem resource handler, `requests` calls have been replaced
with `httpx`, and file I/O has been replaced with `anyio` utils.

The existing approach to save covers and screenshots, by calling
`shutil.copyfileobj` with the raw response is no longer needed. `httpx`
does not provide a file-like object when streaming [1], so there's no
easy drop-in replacement.

However, the applied solution correctly builds the file iteratively, by
consuming the response in chunks.

[1] encode/httpx#2296

…e-handler-to-async

misc: Migrate filesystem resource handler to async

`pytest-vcr` [1] is no longer maintained (last commit was in 2020), and
`VCR.py` recommends [2] `pytest-recording` instead.

Also, `pytest-vcr` does not correctly support `httpx` [3].

This change also removes unused cassettes.

[1] https://github.com/ktosiek/pytest-vcr
[2] https://vcrpy.readthedocs.io/en/latest/usage.html#pytest-integration
[3] ktosiek/pytest-vcr#46

…test-recording

misc: Replace pytest-vcr dependency with pytest-recording

This change mainly refactors the `scan_platforms` function, moving part
of its logic to `_identify_platform`, `_identify_firmware`, and
`_identify_rom`.

The logic is simpler this way, each smaller function returns `ScanStats`
that can be merged by the caller, and it simplifies future performance
improvements.

misc: Refactor scan process by splitting single function

[ROMM-1025] Replace game-and-watch with g-and-w

Public collections

Convert `IGDBBaseHandler` methods to be asynchronous, and use an `httpx`
async client, instead of `requests` sync client.

This change also removes the direct dependency with `requests`, as the
project no longer uses it, preferring `httpx` instead.

misc: Migrate IGDBBaseHandler to async

`ASYNC` rules [1] tries to find issues regarding asynchronous code. This
change enables `ruff` to start reporting these issues, and fixes
existing warnings.

[1] https://docs.astral.sh/ruff/rules/#flake8-async-async

fix: Enable flake8-async rules in ruff and fix warnings

This improvement avoids extra IGDB API requests when a received Rom is
an exact match. It avoids up to 2 requests per Rom, when an exact match
is found.

Only for this scheduled task, the run was failing because there was no
context initialization that created the httpx session.

Other schedules work as expected, because they do have context
initialization in the functions they call to (e.g. `scan_platforms`).

fix: Initialize context on scheduled task

…match

fix: Return early if IGDB finds an exact match

Pytest v8.2 introduced the `PYTEST_VERSION` environment variable [1],
that can be used to check if code is running from within a pytest run.

This way, we can avoid checking the loaded `sys` modules.

[1] https://docs.pytest.org/en/stable/changelog.html#id57

…ct-pytest-runs

misc: Use PYTEST_VERSION variable to detect Pytest runs

Replace use of synchronous `stream_zip` implementation, with
`async_stream_zip`. [1]

[1] https://stream-zip.docs.trade.gov.uk/async-interface/

misc: Use async interface for stream-zip

[ROMM-1029] Store and re-use last saved biois/save/state/core

More to come!

I have better ideas of what to do with this one. Since with previous mobile platforms, I've had an example phone with the logo on the screen, I want to do that for Palm as well. May also come back at some point and do that for J2ME if I have time but I think with that one, the java logo works as a platform icon for now. Here, I'm less certain.

Additional platform icons again!

[ROMM-1018] Fix scanning selected roms

[ROMM-1020] Fix firmware pagination on mount

Hotfix and link from related games

Hotfix for the previously submitted additional icons. Corrects low res/white background issue.

Fix incorrectly converted platform icons

Fix tyo in feontend

[ROMM-1012] Fix multi-file download in gakllery view

[ROMM-1031] Remove persistent overlay prevents back navigation

Co-authored-by: Zurdi <34356590+zurdi15@users.noreply.github.com>

[ROMM-1019] Clickable filter buttons on details view

[ROMM-1022] Add button to show duplicates

Added link to home button

Add workflow that runs typecheck on PRs

Fix FastAPI and nginx configuration, to make the application correctly
redirect URLs. This is specially useful when URLs ended with forward
slash are redirected to their stripped version.

Included changes:
* Stop removing the `/api` prefix in nginx rewrite rules, so FastAPI
  knows what's the original URL path being requested.
* Use `$http_host` in nginx, so FastAPI receives both the original host
  and port, to build the redirect URL (as `$host` does not include the
  port, if present).
* Make all FastAPI included routers know their prefix, to correctly
  route incoming requests.

This fix was found based on a report that redirects from URLs ended with
forward slash were not working [1].

[1] #1051 (comment)

fix: Backend URL redirection logic

Fix CSRF token issues

According to multiple FastAPI discussions [1], FastAPI only includes a
built-in mechanism to redirect requests including a trailing slash, to
its variation without slash, using a `307` status code.

This can be an issue when certain clients do not send the same headers
on the redirected request.

This change adds a custom FastAPI `APIRouter`, that registers both route
path variations (with and without trailing slash), while only marking
the path without slash for being included in the OpenAPI schema.

[1] fastapi/fastapi#7298

misc: Make backend handle URLs with trailing slash

Drop ROMM_AUTH_USERNAME and ROMM_AUTH_PASSWORD env variables