fix: avoid loader-lock work in DllMain detach

[H-Chris233]

Problem

DllMain's DLL_PROCESS_DETACH path called SteamUI::CoreUnhook() and SteamClient::CoreUnhook(), which may perform LoadLibrary/FreeLibrary under the loader lock. This is unsafe during process termination and can cause deadlocks or undefined behavior.

Additionally, nothing prevented an external FreeLibrary from unloading the DLL while hooks and worker threads still reference its code.

Changes

1. Pin the module on DLL_PROCESS_ATTACH via GetModuleHandleExA(GET_MODULE_HANDLE_EX_FLAG_PIN) so the DLL cannot be unmapped while still in use.

2. Guard DETACH work: during process termination (pvReserved != nullptr), only call FileWatcher::Stop() to avoid std::thread's joinable-guard std::terminate. Skip CoreUnhook entirely since it may acquire the loader lock.

Testing

• Verified DLL attaches/detaches cleanly in Steam processes.
• FileWatcher thread joins properly during exit (no more std::terminate crashes).

[OpenSteam001]

@codex review

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. Keep them coming!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".