Various Fixes

[TheAngryByrd]

WHAT

🤖 Generated by Copilot at fb73003

This pull request enhances the LSP server, fixes a line count bug, removes an unused field, and improves the performance of inlay hints. It uses the IFSACSourceText interface more consistently and simplifies some error handling and progress reporting logic. It affects the files AdaptiveFSharpLspServer.fs, Commands.fs, FileSystem.fs, and InlayHints.fs.

🤖 Generated by Copilot at fb73003

docForText fixed
Analyzers and LSP improved
Autumn of refactoring

🐛🔥🚀

WHY

• Return proper LspResult based on exception
• Adds filenames to serverprogress for analyzers
• Various sourceText cleanups

HOW

🤖 Generated by Copilot at fb73003

• Fix line count bug in docForText function by using GetLineCount method instead of Lines property (link)
• Remove unused field weakTable from Hash module in RoslynSourceText module (link)
• Improve performance of getFirstPositionAfterParen function by using GetLineString method instead of Lines property and ToString method (link)
• Add local variable fileName to builtInCompilerAnalyzers function and use it to improve progress messages for analyzers (link, link, link, link)
• Add active pattern Cancelled and function returnException to simplify error handling for LSP requests and avoid code duplication (link)
• Replace explicit returns of internal error results with calls to returnException function in various LSP request handlers (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)

[baronfel]

LGTM - hooray for using text manipulation helpers more generally