Merge #6577 #6579 #6581

6577: fix typos in syntax.md r=matklad a=jakobhellermann

6579: cargo update and add new license r=matklad a=kjeremy

6581: Use vscode-languageclient 7.0.0-next.14 r=matklad a=kjeremy

Co-authored-by: Jakob Hellermann <jakob.hellermann@protonmail.com>
Co-authored-by: kjeremy <kjeremy@gmail.com>

Merge #6553

6553: Auto imports in completion r=matklad a=SomeoneToIgnore


Closes https://github.com/rust-analyzer/rust-analyzer/issues/1062 but does not handle the completion order, since it's a separate task for https://github.com/rust-analyzer/rust-analyzer/issues/4922 , https://github.com/rust-analyzer/rust-analyzer/issues/4922 and maybe something else.

2 quirks in the current implementation:

* traits are not auto imported during method completion

If I understand the current situation right, we cannot search for traits by a **part** of a method name, we need a full name with correct case to get a trait for it.

* VSCode (?) autocompletion is not as rigid as in Intellij Rust as you can notice on the animation.

Intellij is able to refresh the completions on every new symbol added, yet VS Code does not query the completions on every symbol for me.
With a few debug prints placed in RA, I've observed the following behaviour: after the first set of completion suggestions is received, next symbol input does not trigger a server request, if the completions contain this symbol.
When more symbols added, the existing completion suggestions are filtered out until none are left and only then, on the next symbol it queries for completions.
It seems like the only alternative to get an updated set of results is to manually retrigger it with Esc and Ctrl + Space.

Despite the eerie latter bullet, the completion seems to work pretty fine and fast nontheless, but if you have any ideas on how to make it more smooth, I'll gladly try it out.

Co-authored-by: Kirill Bulatov <mail4score@gmail.com>

Merge #6582

6582: Fill the diagnostic code field in publish_diagnostics r=kjeremy a=Veykril

Fixes #6580
Before:
After:

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Fill the diagnostic code field in publish_diagnostics

Use vscode-languageclient 7.0.0-next.14

Merge #6578

6578: Simpify project_model r=matklad a=matklad

bors r+
🤖

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

cargo update and add new license

Simplify

Make code more readable

Minor

Remove dead code

Simplify

fix typos in syntax.md

Merge #6575

6575: Use standard code style for test fixtures r=matklad a=matklad

bors r+
🤖

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

Use standard code style for test fixtures

Merge #6574

6574: **Unwrap Block** supports stand-alone blocks r=matklad a=matklad

bors r+
🤖

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

**Unwrap Block** supports stand-alone blocks

Simplify

Assist target should point at *existing* code

Merge #6573

6573: Add **Ignore Test** assist r=matklad a=matklad

bors r+
🤖

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

Add **Ignore Test** assist

Actually enable eager completion

Merge #6571

6571: Cleanup project model r=matklad a=matklad

bors r+
🤖

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

Remove needless alloc

Compress code

simplify

Merge #6566

6566: Latest LSP 3.16 protocol r=matklad a=kjeremy

Pulls in https://github.com/gluon-lang/lsp-types/pull/186

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

Avoid turning completion objects into builders

Latest LSP 3.16 protocol

Pulls in https://github.com/gluon-lang/lsp-types/pull/186

Remove query aliases

Properly fill the completion settings

Fix the other test

Qualify autoimport completion suggestions

Better filter mod paths

Move autoimport completion into the unqialified_path module

Omit modules during autocompletion

Tweak the search limits a bit

Reuse existing element rendering

Use imports_locator

Rename the module

Allow to configure the merge behavior

Add braces to functions and macros

Tidy up the tests

Use rewriter api to add both changes

Add imports in auto completion

Merge #6565

6565: Prepare to recompute completions on every keystroke r=matklad a=matklad

bors r+
🤖

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

Prepare to recompute completions on every keystroke

If we set `is_incomplete: true`, VS Code will re-query completions
after every keypress.

Merge #6563

6563: Don't complete keywords in struct initializers r=matklad a=Veykril

Fixes #6562

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Don't complete keywords in struct literals

Merge #6558

6558: format string highlighting: handle hex + debug type specifier r=matklad a=ruabmbua

Should fix https://github.com/rust-analyzer/rust-analyzer/issues/6427

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

Merge #6552

6552: Properly handle shorthands in destructure patterns when renaming r=SomeoneToIgnore a=Veykril

Fixes #6548 and #6551.

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

format string highlighting: handle hex + debug type specifier

Cleanup edit_text_range_for_record_field_expr_or_pat

Use shorthand field syntax in destructures

Properly handle shorthands in destructure patterns when renaming

Use shorthand record syntax when renaming struct initializer field

Merge #6544

6544: add suggestion ..Default::default() for remaining struct fields in a constructor r=bnjjj a=bnjjj

I'm not sure I should import `assists` crate inside `completions`, maybe we should move out `FamousDefs` from `assists` ? Let me know :)

close #6492

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

add suggestion ..Default::default() for remaining struct fields in a constructor #6492

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

Merge #6545

6545: Simplify project model r=matklad a=matklad

bors r+
🤖

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

Remove dead code

Cleanup workspace loading a tiny bit

add suggestion ..Default::default() for remaining struct fields in a constructor #6492

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

Move tricky workspace logic to a separate module

Merge #6543

6543: cleanup r=matklad a=matklad

bors r+
🤖

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

cleanup

Merge #6519

6519: Add "Open Cargo.toml" action r=matklad a=p3achyjr

## What is it?

This adds an "open cargo.toml" action from the vscode shell, resolves #6462

## Test

Ran ```cargo xtask install --server``` and ```cargo xtask install --client```, then ```Developer: Reload Window```.


When clicked:


Co-authored-by: Anatol Liu <axlui@anatols-mbp.lan>

Merge #6542

6542: Revert "Upgrade version of npms lockfile" r=matklad a=matklad

bors r+
🤖

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

Revert "Upgrade version of npms lockfile"

This reverts commit 3d559afc11b230d52b13c5540706b85a7eeb3490.

VS Code uses nodejs 12 still, so its better to stick to the format it
understands.

add open Cargo.toml action

Merge #6538

6538: Upgrade version of npms lockfile r=matklad a=matklad

bors r+
🤖

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

Upgrade version of npms lockfile

Merge #6524

6524: Add support for loading rustc private crates r=matklad a=xldenis

This PR presents a solution to the problem of making`rustc_private` crates visible to `rust-analyzer`.
Currently developers add dependencies to those crates behind a `cfg(NOT_A_TARGET)` target to prevent `cargo` from building them.
This solution is unsatisfactory since it requires modifying `Cargo.toml` and causes problems for collaboration or CI.

The proposed solution suggested by @matklad is to allow users to give RA a path where the `rustc` sources could be found and then load that like a normal workspace.

This PR implements this solution by adding a `rustcSource` configuration item and adding the cargo metadata to the crate graph if it is provided.

------

I have no idea how this should be tested, or if this code is generally tested at all. I've locally run the extension with these changes and it correctly loads the relevant crates on a `rustc_private` project of mine.

Co-authored-by: Xavier Denis <xldenis@gmail.com>

Address review comments

Merge #6514

6514: Fix extract_struct_from_enum_variant not updating record references r=Veykril a=Veykril

Related to #6510

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Add extra test to extract_struct_from_enum_variant

Merge #6513

6513: Support qualified function calls in remove_unused_param r=Veykril a=Veykril

Also adds a test to check that it removes unused params across files.

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Support qualified function calls in remove_unused_param

Use Module::find_use_path_prefixed in extract_struct_from_enum_variant assist

Fix extract_struct_from_enum_variant not updating record references

Merge #6529

6529: Do not insert imports before inner comments r=Veykril a=SomeoneToIgnore

Co-authored-by: Kirill Bulatov <mail4score@gmail.com>

Remove the fixme

Add a FIXME for non-unified inner attributes

Do not insert imports before inner comments

Merge #6534

6534: Fix attachment of inner doc comments r=matklad a=matklad

bors r+
🤖

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

Fix attachment of inner doc comments

Simplify code

Merge #6531

6531: Update crates r=matklad a=kjeremy

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

Update tests

Update crates

Merge #6530

6530: Do not use yanked once_cell r=kjeremy a=kjeremy

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

Do not use yanked once_cell

Reorder rustc_private loading

Add support for loading rustc private crates

Merge #6526

6526: Replace RacyFlag with OnceCell r=matklad a=matklad

bors r+
🤖

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

Replace RacyFlag with OnceCell

Merge #6523

6523: Be more pedantic when checking codeAction/resolve support r=matklad a=matklad

bors r+
🤖

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

Be more pedantic when checking codeAction/resolve support

Merge #6521

6521: Switch to upstream protocol for resolving code action r=matklad a=matklad

Note that we have to maintain custom implementation on the client
side: I don't see how to marry bulitin resolve support with groups and
snippets.

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

Switch to upstream protocol for resolving code action

Note that we have to maintain custom implementation on the client
side: I don't see how to marry bulitin resolve support with groups and
snippets.

Merge #6512

6512: Don't replace parent node when inserting as first child in algo::diff r=SomeoneToIgnore a=Veykril

This makes the diff a bit more detailed.

See https://github.com/rust-analyzer/rust-analyzer/pull/6287#issuecomment-723889267 for context
cc @SomeoneToIgnore

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>