Review fixes for the solution-targeting change:
- Correct the stale premise in comments: current csharp-ls discovers
solutions on its own (recursive .sln/.slnx glob, most-projects
heuristic since 0.19.0, .slnx since 0.18.0). The pin's value is
determinism, a concrete target for the pre-spawn restore, and an
operator override — not a capability the server lacks.
- GORTEX_LSP_CSHARP_SOLUTION gains an explicit off switch (falsy value
or 'none') that disables injection AND auto-detect, leaving the
server's own discovery untouched.
- The resolved pin and its provenance (env vs auto-detect) are logged
at spawn, and env entries that don't resolve for a root log a warning
with the reason instead of being dropped silently.
- The caller-pinned guard also recognizes the =-joined spellings
(--solution=X / -s=X) config args can carry.
- GORTEX_LSP_RESOLVER_CSHARP accepts any set, non-falsy value ('on',
'yes'), mirroring the sibling GORTEX_LSP_RESOLVER vocabulary.
- docs/lsp.md documents all three new env vars, corrects the restore
row to the targeted form, and replaces both edit-the-router-chain
passages with GORTEX_LSP_IDLE_TTL.
pbednarcik
f931fe25ad