Applied Cursor's suggestions to address straightforward TODOs.

[tameware]

No description provided.

[Copilot]

Pull request overview

This PR removes a few straightforward TODOs by switching core-count detection to standard C++ (std::thread::hardware_concurrency()), simplifying a redundant constructor initializer in SolverContext, and cleaning up an obsolete comment in quick-tricks logic.

Changes:

• Use std::thread::hardware_concurrency() in System::get_cores() with a platform-specific fallback when it returns 0.
• Remove redundant thr_(nullptr) from SolverContext constructor initializer list (the member is set immediately in the constructor body).
• Remove stale TODO commentary in QuickTricksPartnerHandNT now that the logic is settled.

Reviewed changes

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

File	Description
library/src/system/system.cpp	Adds <thread> and prefers std::thread::hardware_concurrency() for core detection, falling back to OS APIs.
library/src/solver_context/solver_context.cpp	Cleans up redundant constructor initialization and clarifies ThreadMemoryUsed() accounting comment.
library/src/quick_tricks.cpp	Removes obsolete TODO comments next to existing quick-trick adjustment logic.