rust-analyzer didn't return warning/hint diagnostics on textDocument/diagnostic

[fannheyward]

1. rust-analyzer publishDiagnostics after open rust file, containing the warning/hint level diagnostics
2. switch windows in editor, then the client requests textDocument/diagnostic for the same file, rust-analyzer returns empty diagnostics

Server trace logs, same logs with VSCode+rust-analzyer too, more logs fannheyward/coc-rust-analyzer#1276 (comment)

[Trace - 14:56:20] Received notification 'textDocument/publishDiagnostics'.
Params: {
    "uri": "file:///Users/fannheyward/src/hello-rust/src/main.rs",
    "diagnostics": [
        {
            "range": {...},
            "severity": 2,
            "code": "unused_imports",
            "source": "rustc",
            "message": "unused import: `std::env`\n`#[warn(unused_imports)]` on by default",
            "relatedInformation": [...],
            "tags": [1],
            "data": {...}
        },
        {
            "range": {...}
            },
            "severity": 4,
            "code": "unused_imports",
            "source": "rustc",
            "message": "remove the whole `use` item",
            "relatedInformation": [...]
        }
    ],
    "version": 1
}

...

[Trace - 14:57:31] Sending request 'textDocument/diagnostic - (18)'.
Params: {
    "textDocument": {
        "uri": "file:///Users/fannheyward/src/hello-rust/src/main.rs"
    }
}


[Trace - 14:57:31] Received response 'textDocument/diagnostic - (18)' in 2ms.
Result: {
    "kind": "full",
    "items": []
}

rust-analyzer version: rust-analyzer 0.0.0 (27e824f 2024-12-15)

rustc version: rustc 1.83.0 (90b35a623 2024-11-26)

editor or extension: nvim + coc.nvim

code snippet to reproduce:

use std::env;

fn main() {
    println!("Hello, world!");
}

[Veykril]

We use publishDiagnostics for the rustc/cargo errors, the pull model for our native ones. This works just fine in VSCode and the spec doesn't state anything regarding it. Though according to microsoft/language-server-protocol#1743 (comment) this setup is discouraged.

cc @rchl @jwortmann tagging you because of your comments in #18404 (I forgot about this sorry)

[fannheyward]

This problem is not that RA is us ing push and pull mode at same time, but: RA can publish diagnostics at pushing mode, but no diagnostics at pulling mode.

[fannheyward]

This works just fine in VSCode

From the tracking logs in VSCode, publishDiagnostics has two diagnostics, but empty result in pull request.

[Veykril]

We use the push mode for cargo diagnostics where as pull only pull rust-analyzer native ones. So as currently implemented in r-a there is no overlap between the two (pull diagnostics will never list cargo check ones, push diagnostics will never list rust-analyzer native ones). So clients (except for VSCode for some reason?) will naturally replace push results for a document with the pull results for the given document (as we don't even populate the result-id #18717 fixes that part so there is no distinguishing to be done either way).

I personally do think using both pull and push is valid, but the spec leaves that (+ how to handle such a case) completely unspecified.

[fannheyward]

except for VSCode for some reason?

See my previous comment, VSCode also pulls empty result, just that it didn't replace/update the PROBLEMS list.

[Veykril]

It is correct that the pull there is empty, because the pull only pulls rust-analyzer native diagnostics (the ones we compute), so they do not include the diagnostic computed by us via cargo check (triggered when you save a file). It is intentional by our implementation that pull and push diagnostics do not yield the same set of diagnostics for a document.

[fannheyward]

I see, thank you for the clarification. Maybe I can make an improvement in the client side for the both modes.