refact(map-metrics): collect map metrics on event instead of job

[sonikro]

No description provided.

[Copilot]

Pull request overview

Refactors map metrics collection to be driven by SRCDS “Loading map …” UDP log events instead of a scheduled background job, and updates the domain/schema accordingly (dropping server_id from stored metrics).

Changes:

• Add a new loadingMap SRCDS command parser + handler that logs and persists map load metrics on receipt.
• Remove the scheduled server metrics collection routine and the CollectServerMetrics core use case (and its tests/exports).
• Update ServerStatusMetric + SQLite repository + DB migration to remove server_id from server_status_metrics.

Reviewed changes

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

Show a summary per file

File	Description
packages/providers/src/repository/SQliteServerStatusMetricsRepository.ts	Stops writing/returning server_id when persisting metrics.
packages/entrypoints/src/udp/srcdsCommands/index.ts	Registers the new loadingMap parser in the SRCDS command list.
packages/entrypoints/src/udp/srcdsCommands/UDPCommandServices.ts	Extends UDP command DI services with serverStatusMetricsRepository.
packages/entrypoints/src/udp/srcdsCommands/LoadingMap.ts	Implements parsing + handler to log and persist map load metrics.
packages/entrypoints/src/udp/srcdsCommands/LoadingMap.test.ts	Adds tests for parsing and handler behavior.
packages/entrypoints/src/jobs/index.ts	Removes export of the deleted metrics-collection routine.
packages/entrypoints/src/jobs/ServerMetricsCollectionRoutine.ts	Deletes the scheduled job that used to collect metrics periodically.
packages/entrypoints/src/discordBot.ts	Removes routine scheduling and wires serverStatusMetricsRepository into the UDP listener DI.
packages/core/src/usecase/CollectServerMetrics.ts	Deletes the use case previously used by the scheduled job.
packages/core/src/usecase/CollectServerMetrics.test.ts	Deletes tests for the removed use case.
packages/core/src/domain/ServerStatusMetric.ts	Removes serverId from the domain model.
packages/core/index.ts	Stops exporting the removed use case.
migrations/20260207175944_drop_server_id_from_server_status_metrics.ts	Adds migration to drop server_id column from server_status_metrics.

[Copilot]

These handler tests share the same mocked logger.emit across test cases, but the mock is never cleared. That makes assertions like toHaveBeenCalledWith(...) potentially pass due to calls from earlier tests. Clear/reset the logger.emit mock (or vi.clearAllMocks()) within makeSut() or at the start of each test to keep tests independent.