[ruff] Stop parsing diagnostics from other sources for code action requests

[joouha]

Summary

This PR stops ruff server attempting to parse diagnostic data for diagnostics from other language servers.

The issue

When using ruff server with typos-lsp in helix, it is impossible to apply code actions suggested by typos-lsp.

The cause

During a textDocument/codeAction request, the editor will send Diagnostics to ruff server as part of the codeActionContext - these may have been generated by other LSP servers.

According to the spec, the Diagnostic.data field can contain any data. However, ruff server attempts to parse all contextual diagnostics, and assumes the data field adheres to the data structure used by ruff server.

Since other language servers are free to use their own data structure, parsing the diagnostics fails causing an error, which ultimately causes the code action request to fail.

The fix

On code action requests, ruff server should only parse contextual diagnostics generated by ruff server.

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

[MichaReiser]

Nice. Interesting that the client sends all known Diagnostic but that sort of makes sense.