Fix review comments

Merge remote-tracking branch 'upstream/master' into compute-lazy-assits

# Conflicts:
# crates/rust-analyzer/src/to_proto.rs

Preliminary implementation of lazy CodeAssits

Merge #4710

4710: New runnables r=matklad a=matklad

bors d=@vsrs

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Merge #4711

4711: Disable rust-analyzer.{cargo,checkOnSave}.allFeatures by default r=matklad a=lnicola

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Disable rust-analyzer.{cargo,checkOnSave}.allFeatures by default

Fix tests

Spec better runnables

New runnables API

Merge #4709

4709: Minor r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Minor

Merge #4708

4708: Move run commands to commands.ts r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Move run commands to commands.ts

Merge #4658

4658: Fix problem with format string tokenization r=matklad a=ruabmbua

Fixed by just not handling closing curlybrace escaping.

Closes https://github.com/rust-analyzer/rust-analyzer/issues/4637

Co-authored-by: Roland Ruckerbauer <roland.rucky@gmail.com>

Merge #4580

4580: Fix invoking cargo without consulting CARGO env var or standard installation paths r=matklad a=Veetaha

Followup for #4329

The pr essentially fixes [this bug](https://youtu.be/EzQ7YIIo1rY?t=2189)

cc @lefticus

Co-authored-by: veetaha <veetaha2@gmail.com>

Merge #4701

4701: Simplify r=kjeremy a=Veetaha

Co-authored-by: veetaha <veetaha2@gmail.com>

Simplify

Merge #4688

4688: Document initilizationOptions used by rust-analyzer r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Subscribe to protocol changes

Document initilizationOptions used by rust-analyzer

Merge #4686

4686: Drop unused document r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Drop unused document

Merge #4682

4682: Rename assist r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Rename assist

Cleanup imports

Merge #4675

4675: Small refactor r=kiljacken a=Veetaha

Co-authored-by: veetaha <veetaha2@gmail.com>

Small refactor

Merge #4674

4674: Recursively search submodules to find modules in which a definition is visible. r=matklad a=umanwizard

Co-authored-by: Brennan Vincent <brennan@materialize.io>

Merge #4669

4669: Always derive from `process.env` when spawning a child process/shell execution r=matklad a=Dfinity-Alin

This is useful when an extension (e.g. [Nix Environment Selector](https://marketplace.visualstudio.com/items?itemName=arrterian.nix-env-selector)) or [launch
configuration](https://stackoverflow.com/questions/57641460/set-env-var-for-node-js-when-launching-through-vs-code) sets one or more environment variables.

When `env` is not explicitly specified in the options passed to
`child_process.spawn()` or `vscode.ShellExecution()`, then `process.env` gets
applied automatically. But when an explicit `env` is set, it should inherit from
`process.env` rather than replace it completely.

Co-authored-by: Alin Sinpalean <alin.sinpalean@dfinity.org>

add test

recursively search submodules

Merge #4673

4673: Update deps r=kjeremy a=kjeremy

Co-authored-by: Jeremy Kolb <kjeremy@gmail.com>

Update deps

Merge #4672

4672: Link docs and code r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Link to the specific source line

Merge #4671

4671: docs r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

docs

Merge #4670

4670: Move assists documentation into the manual r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Move assists documentation into the manual

Merge #4667

4667: Infer labelled breaks correctly r=flodiebold a=robojumper

Fixes #4663.

Co-authored-by: robojumper <robojumper@gmail.com>

Always derive from `process.env` when spawning a child process/shell execution

This is useful when an extension (e.g. Nix Environment Selector) or launch
configuration sets one or more environment variables.

When `env` is not explicitly specified in the options passed to
`child_process.spawn()` or `vscode.ShellExecution()`, then `process.env` gets
applied automatically. But when an explicit `env` is set, it should inherit from
`process.env` rather than replace it completely.

Merge #4668

4668: Rename user/readme.adoc -> user/manual.adoc r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Rename user/readme.adoc -> user/manual.adoc

Merge #4664

4664: Generate feature documentation from code r=matklad a=matklad

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Force / slashes on windows

Tweak whitespace

fmt

correctly infer labelled breaks

Move the rest of the features to generated docs

labelled break test

Doc more features

Don't require module docs for Features and Assists

Specify actions

Ignore HTML

Simplify

Change Runnable.bin -> Runnable.kind

As per matklad, we now pass the responsibility for finding the binary to the frontend.
Also, added caching for finding the binary path to reduce
the amount of filesystem interactions.

Fix tests, apply code review proposals

Fix invoking cargo without consulting CARGO or standard installation paths

Generate features docs from source

Generalize

Merge #4662

4662: minor r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

minor

Test case for format string highlighting of closing curlybrace

Fix problem with format string tokenization

Fixed by just not handling closing curlybrace escaping.

Merge #4654

4654: Add semantic highlight to ? operator r=matklad a=ruabmbua

Made it an operator with controlFlow modifier.

To highlight in vscode as red:

```json
"editor.semanticTokenColorCustomizations": {
        "enabled": true,
        "rules": {
            "operator.controlFlow": "#ff0000",
        }
}
```


https://github.com/rust-analyzer/rust-analyzer/issues/4597

Co-authored-by: Roland Ruckerbauer <roland.rucky@gmail.com>

Merge #4562

4562: Assist: replace anonymous lifetime with a named one r=matklad a=jbalint

(fixes #4523)

Co-authored-by: Jess Balint <jbalint@gmail.com>

Merge #4653

4653: Fix match ergonomics in closure parameters r=matklad a=flodiebold

Fixes #4476.

Co-authored-by: Florian Diebold <florian.diebold@freiheit.com>

Merge #4655

4655: Fix typo in docs/dev/lsp-extensions.md: automagiacally -> automagically r=kjeremy a=theHamsta

This may be a typo...

Co-authored-by: Stephan Seitz <stephan.seitz@fau.de>

Fix typo in docs/dev/lsp-extensions.md: automagiacally -> automagically

Merge #4651

4651: Use first match branch in case of type mismatch, not last r=kjeremy a=flodiebold

The comment says this was intentional, but I do agree with #4304 that it makes
more sense the other way around (for if/else as well).

Fixes #4304.

Co-authored-by: Florian Diebold <florian.diebold@freiheit.com>

Merge #4652

4652: Upgrade Chalk r=kjeremy a=flodiebold

Fixes #4072.

Co-authored-by: Florian Diebold <florian.diebold@freiheit.com>

Add semantic highlight to QUESTION token

Made it an operator with controlFlow modifier.

Fix match ergonomics in closure parameters

Fixes #4476.

Merge #4648

4648: Support raw_ref_op's raw reference operator r=matklad a=robojumper

Fixes #4642.

This syntax (and its semantics) are implemented in rustc behind the `raw_ref_op` feature.

It is not entirely clear whether this is the syntax that will become stable, but [it seems like](https://github.com/rust-lang/rust/pull/72279) rust-analyzer must still support this unstable syntax to support future stable rust.

Also fixes a random inference failure involving a direct coercion from `&[T, _]` to `*const [T]`.

Co-authored-by: robojumper <robojumper@gmail.com>

Upgrade Chalk

Fixes #4072.

Use first match branch in case of type mismatch, not last

The comment says this was intentional, but I do agree with #4304 that it makes
more sense the other way around (for if/else as well).

Fixes #4304.

add support for naming anon lifetimes in function return type

Support raw_ref_op's raw reference operator

Merge #4640

4640: Add `inRustProject` when-clause for commands in vscode r=vsrs a=vsrs

At the moment all rust-analyzer commands always visible in the command palette, even if there is no rust project opened.

This PR adds special [when-clause](https://code.visualstudio.com/docs/getstarted/keybindings#_when-clause-contexts) context. This context also might be used in key bindings.

Co-authored-by: vsrs <vit@conrlab.com>
Co-authored-by: vsrs <62505555+vsrs@users.noreply.github.com>

Add `inRustProject` context documentation.

Merge #4641

4641: Upgrade Chalk r=matklad a=flodiebold

Chalk newly added TypeName::Never and Array; I implemented the conversion for
Never, but not Array since that expects a const argument.

Co-authored-by: Florian Diebold <flodiebold@gmail.com>

Merge #4646

4646: Link forum topic r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Link forum topic

Merge #4639

4639: Fixed missing newline on each field on "Missing structure fields". r=flodiebold a=flipbit03

Co-authored-by: Cadu <cadu.coelho@gmail.com>

Upgrade Chalk

Chalk newly added TypeName::Never and Array; I implemented the conversion for
Never, but not Array since that expects a const argument.

Add `inRustProject` when-clause for commands.

Fixed missing newline on each field on "Missing structure fields".

Merge #4592

4592: fix textedit range returned for completion when left token is a keyword r=bnjjj a=bnjjj

close #4545

Co-authored-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>

fix textedit range returned for completion when left token is a keyword

Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>

Merge #4596

4596: Strip leading underscores of argument names in function/method r=matklad a=kuy

Closes #4510

### Goal

When I select a function/method from completions, I get a snippet that doesn't contain leading underscores of argument names.

### Solution

- Option 1: All signatures don't contain underscores
- Option 2: Keep same signature, but inserted snippet doesn't contain underscores

I choose Option 2 because I think that leading underscores is a part of "signature". Users should get correct signatures. On the other hand, trimming underscores is an assist by IDE.

### Other impls.

rls: Complete argument names with underscores (same as actual ra)
IntelliJ Rust: Doesn't complete argument names
VSCode (TypeScript): Doesn't complete argument names

### Working example


Co-authored-by: Yuki Kodama <endflow.net@gmail.com>

Merge #4625 #4629

4625: Partially fix displaying inlay hints in Github PR diff views r=matklad a=Veetaha

See the comment in https://github.com/rust-analyzer/rust-analyzer/issues/4608#issuecomment-63424257

It partially fixes the left side of diff view (the one where old code is displayed), but the diff editor with new code changes still has `file` scheme and will proceed displaying inlay hints...

4629: Fix the `should_panic` snippet r=matklad a=eminence

Closes #4628

Co-authored-by: veetaha <veetaha2@gmail.com>
Co-authored-by: Andrew Chin <achin@eminence32.net>

Merge #4633

4633: typo r=matklad a=Veetaha

Co-authored-by: Veetaha <veetaha2@gmail.com>

typo

Merge #4632

4632: Document inlay hints and runnables r=matklad a=matklad

We want to change those, but let's document what we have in meantime

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Document inlay hints and runnables

We want to change those, but let's document what we have in meantime

Merge #4622

4622: Pass trivially copy types as copy r=matklad a=kjeremy

Co-authored-by: kjeremy <kjeremy@gmail.com>

Fix the `should_panic` snippet

Closes #4628

Partially fix displaying inlay hints in Github PR diff views

Merge #4623

4623: Update deps r=kjeremy a=kjeremy

Co-authored-by: kjeremy <kjeremy@gmail.com>

Update lexer