Merge #8679

8679: fix: don't show error message for a valid notification r=matklad a=matklad

bors r+
🤖

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

fix: don't show error message for a valid notification

Closes #6782, #6772

Merge #8677

8677: Return ServerNotInitialized according to the spec r=kjeremy a=kjeremy

According to the spec we should return ServerNotInitialized if the server is waiting for an initialize request and something else comes in.
Upgrading to lsp-server 0.5.1 will do this and retry until the initialize request comes in.

Fixes #8581

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

Return ServerNotInitialized according to the spec

According to the spec we should return ServerNotInitialized if the server is waiting for an initialize request and something else comes in.
Upgrading to lsp-server 0.5.1 will do this and retry until the initialize request comes in.

Fixes #8581

Merge #8617

8617: Add option to opt out of smaller font size for inlay hints. r=SomeoneToIgnore a=jmederosalvarado

As requested on issue #6883 this PR provides an option for users to opt out of the smaller font size for inlay hints. Part of #6883.

Co-authored-by: Jorge Mederos Alvarado <jmederosalvarado@gmail.com>

Fix how and when old inlay hint decorations are disposed

Add config options

Add option to opt out from smaller inlay hints font size

add option to package.json

Merge #8671

8671: minor: simplify r=matklad a=matklad

bors r+
🤖

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

minor: simplify

Merge #8670

8670: fix: make error message in the status bar more useful r=matklad a=matklad

bors r+
🤖

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

fix: make error message in the status bar more useful

At the moment,the popup is just a bazillion of Cargo's "Compiling this\nCompiling that",
which is not that useful.

--quiet still displays error, which is what we needc

Merge #8668

8668: Use more cross-platform utc `date` argument r=matklad a=SomeoneToIgnore

Part of https://github.com/rust-analyzer/rust-analyzer/issues/8571

```
$ docker run -it --rm ubuntu:20.04 bash
root@7393d1e7bbad:/# date -u +%Y-%m-%d
2021-04-26
```

```
$ date -u +%Y-%m-%d
2021-04-26

$ uname -a
Darwin alaptop.local 20.3.0 Darwin Kernel Version 20.3.0: Thu Jan 21 00:07:06 PST 2021; root:xnu-7195.81.3~1/RELEASE_X86_64 x86_64
```

Some of the places where I've change this do not really require it (since macos bin would have failed with `--iso` param also), but I've changed them for consistency.

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

Use -u instead of --utc date flag

Merge #8666

8666: fix: correct version string on windows and mac r=matklad a=matklad

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

fix: add date to the version string on Mac

It doesn't support `--iso`

fix: dont' misundentify nightly as stable in --version on Mac&Win

We used to set `--nightly` in CI, and only for linux. Let's detect this
in xtask instead.

Merge #8661

8661: internal: make normal release after a poin release less annoying r=matklad a=matklad

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

Update xtask/src/release.rs

Co-authored-by: Laurențiu Nicola <lnicola@users.noreply.github.com>

Update xtask/src/release.rs

Co-authored-by: Laurențiu Nicola <lnicola@users.noreply.github.com>

internal: make normal release after a poin release less annoying

Merge #8660

8660: Added `mutable` semantic token modifier for `&mut self` methods r=Veykril a=1c3t3a

This closes #8644.

Co-authored-by: Bastian Kersting <bastian@cmbt.de>

Fixed formatting

Added mutable semantic token modifier for &mut self methods

Merge #8658

8658: Check more carefully for cases where a rename can't be done r=Veykril a=dzhu

Attempting to rename an element of a tuple field would previously replace the type with the new name, which doesn't make sense; now it fails instead.

The check is done in both `prepare_rename` and `rename` so that the case is caught before the user is prompted for a new name. Some other existing failure cases are also now additionally checked in `prepare_rename`.

Closes: #8592
(I threw in some doc edits for a relevant type; of course, I can remove those if the policy here is to be strict about scope of changes within a PR.)

Co-authored-by: Danny Zhu <dzhu@dzhu.us>

Check more carefully for cases where a rename can't be done

Attempting to rename an element of a tuple field would previously
replace the type with the new name, which doesn't make sense; now it
fails instead.

The check is done in both `prepare_rename` and `rename` so that the case
is caught before the user is prompted for a new name. Some other
existing failure cases are also now additionally checked in
`prepare_rename`.

Merge #8656

8656: Update deps r=kjeremy a=kjeremy

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

Update deps

Merge #8647

8647: internal: Split out merge_imports module from helpers::insert_use r=Veykril a=Veykril

bors r+

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

Split out merge_imports module from helpers::insert_use

Merge #8643

8643: fix: correct version string to contain hash, build date and channel r=matklad a=matklad

bors r+
🤖

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

fix: correct version string to contain hash, build date and channel

minor

Merge #8641

8641: minor r=matklad a=matklad

bors r+
🤖

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

minor

Merge #8639

8639: fix: 'configuration.property' error in VS Code r=matklad a=matklad

bors r+
🤖

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

fix: 'configuration.property' error in VS Code

closes #7789

Merge #8591 #8638

8591: Remove SyntaxRewriter usage in insert_use in favor of mutable syntax trees r=matklad a=Veykril

Unfortunately changing `insert_use` to not use `SyntaxRewriter` creates a lot of changes since so much relies on that. But on the other hand this should be the biggest usage of `SyntaxRewriter` I believe.

8638: Remove SyntaxRewriter::from_fn r=Veykril a=Veykril

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

Remove SyntaxRewriter::from_fn

Merge #8637

8637: Tag `yield` and `await` as ControlFlow in semantic highlighting r=Veykril a=Veykril

and shuffle some code around
Closes #8634
bors r+

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

Simplify highlight token match guards

Sort HlMod variants and ALL const

Tag `yield` and `await` as ControlFlow in semantic highlighting

Merge #8317

8317: Convert tuple struct to named struct assist r=Veykril a=unexge

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

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

Add missing test case for "Convert to named struct" assist

Stop iterating reference after made an edit in "Convert to named struct" assist

Merge #8628

8628: minor: Add one more profiling span to add_dep r=lnicola a=lnicola

CC #8623

bors r+

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

Add one more profiling span to add_dep

Merge #8627

8627: Add profiling spans under cargo_to_crate_graph r=lnicola a=lnicola

bors r+

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

Add profiling spans under cargo_to_crate_graph

Merge #8605

8605: internal: Automatically categorize the changelog entries r=matklad a=lnicola

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

Merge #8620

8620: Remove unnecessary braces for extracted block expression r=Veykril a=brandondong

This change addresses the first bullet point of https://github.com/rust-analyzer/rust-analyzer/issues/7839.

Specifically, when extracting block expressions, remove the unneeded extra braces inside the generated function.

Co-authored-by: Brandon <brandondong604@hotmail.com>

Move reference editing logic into own function to make error handling more ease in "Convert to named struct" assist

Merge #8621

8621: feat: make sure nightly regressions don't break users r=lnicola a=matklad

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

feat: make sure nightly regressions don't break users

Remove unnecessary braces for extracted block expression

Merge #8595

8595: Diagnostic paths support specifying `remapPrefix` r=rickvanprim a=rickvanprim

Currently VSCode Problem Matchers will resolve a path like `//foo_crate/src/main.rs` if `${workspaceFolder}/foo_crate/src/main.rs` exists.  Presumably their behavior is functionally a string concatenation that would produce `${workspaceFolder///foo_crate/src/main.rs` and repeated path separators get ignored.

This PR attempts to mimic this behavior by stripping any `Component::RootDir` from `file_name` before joining it to `workspace_root`, and then checking if the file exists.  If it does, this path is used, and if not, the behavior falls through to the existing Rust path join behavior.

Co-authored-by: James Leitch <rickvanprim@gmail.com>

Don't filter equal nodes in reorder assists

Code review feedback.

Diagnostic Remap Path Prefixes added.

Diagnostic paths attempt to use VSCode's path join behavior before defaulting to Rust's path join behavior.

Merge #8570

8570: Flycheck tries to parse both Cargo and Rustc messages. r=rickvanprim a=rickvanprim

This change allows non-Cargo build systems to be used for Flycheck provided they call `rustc` with `--error-format=json` and emit those JSON messages to `stdout`.

Co-authored-by: James Leitch <rickvanprim@gmail.com>

Merge #8616

8616: minor: Bump deps r=lnicola a=lnicola

bors r+

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

Bump other deps

Bump chalk

Bump rustc_lexer

Merge #8615

8615: skip: Remove `ItemTree::all_inner_items` r=jonas-schievink a=jonas-schievink

It's unused.

bors r+

Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>

Remove `ItemTree::all_inner_items`

Merge #8613

8613: skip: Add failing local items test r=jonas-schievink a=jonas-schievink

This is currently broken due to the way ItemTree stores inner items. This test serves as a reminder until this gets fixed.

bors r+

Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>

Add failing local items test

Merge #8611

8611: Add support for fill match arms of boolean values r=flodiebold a=komonad

- Add support for boolean inside tuple

closes #8593

Co-authored-by: Comonad <comonad@foxmail.com>

Use multiple loops instead of `Iterator::chain` in `FindUsages`

Add multi file test for "Convert to named struct" assist

Remove `unwrap`s in "Convert to named struct" assist

Add support for fill match arms of boolean values

- Add support for boolean inside tuple

Merge #8609

8609: Parse outer atttributes for RecordPatField r=Veykril a=Veykril

Fixes #8608
bors r+

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

Parse outer atttributes for RecordPatField

Add larger example for "Convert to named struct" assist

Fix incorrectly replacing method calls in "Convert to named struct" assist

Merge #8606

8606: fix: no more  Registering progress handler for token rustAnalyzer/Ind… r=jonas-schievink a=matklad

…exing failed.

closes #8509

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

fix: no more  Registering progress handler for token rustAnalyzer/Indexing failed.

Add cov_marks to insert_use tests

Automatically categorize the changelog entries

Rewrite extract_struct_from_enum_variant assist

Merge #8600

8600:  fix: no longer get stuck on windows r=matklad a=matklad

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

fix: no longer get stuck on windows

reading both stdout & stderr is a common gotcha, you need to drain them
concurrently to avoid deadlocks. Not sure why I didn't do the right
thing from the start. Seems like I assumed the stderr is short? That's
not the case when cargo spams `compiling xyz` messages

Merge #8602

8602: Fix panic in `replace_derive_with_manual_impl` r=jonas-schievink a=jonas-schievink

bors r+

Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>

Follow testing style guide

Fix panic in `replace_derive_with_manual_impl`

Merge #8601

8601: Trigger "Inline variable" assist when on a use of the variable r=jonas-schievink a=jonas-schievink

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/8530

bors r+

Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>

"Inline variable" when on a use of the variable

Merge #8598

8598: minor: fix ugly line wrapping in the rendered manual r=matklad a=matklad

bors r+
🤖

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

minor: fix ugly line wrapping in the rendered manual

Remove SyntaxRewriter usage in insert_use in favor of ted

Replace SyntaxRewriter usage with ted in reorder_impl assist

Code review feedback.

Merge #8588

8588: internal: Add guidelines for release notes PR descriptions r=matklad a=lnicola

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

internal: Add guidelines for release notes PR descriptions

Merge #8586

8586: Replace SyntaxRewriter usage with ted in eager::eager_macro_recur r=Veykril a=Veykril

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