OmniSharp语言服务器在保存新C文件时崩溃问题分析

问题现象

在使用VSCode进行C#开发时，当用户创建一个新的未保存文件并将其语言模式设置为C#后，在首次保存文件时，OmniSharp语言服务器偶尔会出现崩溃现象。崩溃日志显示服务器抛出了一个InvalidOperationException异常，提示"Unable to determine language for 'untitled:Untitled-1' with LSP language id ''"。

技术背景

OmniSharp是.NET生态中广泛使用的语言服务器协议(LSP)实现，为C#开发者提供智能代码补全、错误检查等功能。在VSCode中，当用户创建新文件时，编辑器会先创建一个"untitled"类型的临时文档，直到用户显式保存文件到磁盘。

问题根源

通过分析日志和代码，我们发现问题的根本原因在于：

1. 当用户创建新文件并设置语言模式时，VSCode会向语言服务器发送文档打开通知(textDocument/didOpen)
2. 服务器会记录这个文档的状态，包括其URI和语言信息
3. 当用户保存文件时，VSCode会先发送文档关闭通知(textDocument/didClose)
4. 服务器在处理关闭通知时会清除该文档的状态信息
5. 但此时可能仍有对该文档的请求在处理队列中
6. 这些后续请求无法找到已清除的文档状态，导致无法确定语言类型而抛出异常

解决方案

针对这个问题，合理的修复方案应包括：

1. 修改语言检测逻辑，当无法确定文档语言时不应抛出异常，而是：

   • 使用默认语言(C#)继续处理
   • 记录警告日志以便开发者了解情况
   • 保持系统稳定性而不是崩溃

2. 不能简单地保留已关闭文档的状态，因为这会导致内存泄漏。正确的做法是优雅地处理状态缺失的情况。

技术实现建议

在Roslyn语言服务器的实现中，应该：

1. 在LanguageInfoProvider类中增加防御性编程
2. 为GetLanguageInformation方法添加合理的默认值处理
3. 确保所有依赖语言信息的操作都能处理未知语言的情况
4. 完善日志记录，帮助开发者诊断类似问题

总结

这类问题在语言服务器实现中较为常见，特别是在处理文档生命周期事件时。良好的实践应包括：

1. 对临时文档和状态变化保持鲁棒性
2. 采用防御性编程处理可能的竞态条件
3. 在状态清理和请求处理之间保持一致性
4. 提供有意义的错误日志而非直接崩溃

通过这种改进，可以显著提升开发者在创建新C#文件时的体验，避免不必要的语言服务器重启。