Merge #4733

4733: Cargo update r=kjeremy a=kjeremy

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

Cargo update

Merge #4502

4502: Mark fixes from diagnostics as quick fixes r=kjeremy a=kjeremy

Populates the diagnostic UI with fixes:

Before:

After:

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

Mark fixes from diagnostics as quick fixes

Merge #4730

4730: Document rust-project.json r=matklad a=matklad

bors r+
🤖

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

Document rust-project.json

Merge #4678

4678: Unsquish parameter types in tooltips for macro-generated functions r=aloucks a=aloucks

Note the missing whitespace between `:` and the parameter type.

Before:

After:

Co-authored-by: Aaron Loucks <aloucks@cofront.net>

Use split1 when formatting function signature params

Merge #4660

4660: Enable hover and autocomplete docs on macro generated items r=aloucks a=aloucks

Enable hover and autocomplete docs on macro generated items. This de-sugars doc comments into `doc` attributes in some cases, but not all. Comments and `doc` attributes are then merged together.

This PR is essentially a partial implementation of what's being suggested #3182, but it's not all the way there yet. ~I still need to add unit tests~, but I wanted to first get feedback on whether or not this was an acceptable path forward.

Fixes #4564
Fixes #3984
Fixes #3180
Related #3182


Co-authored-by: Aaron Loucks <aloucks@cofront.net>

Update generated feature docs

Consolidate documentation expansion and merging

Removes the duplicated `expand_doc_attrs` and `merge_doc_comments_and_attrs`
functions from `ra_ide` and exposes the same functionality via
`ra_hir::Documentation::from_ast`.

Add basic hover and completion doc tests for macro generated items

Enable hover and autocomplete docs on macro generated items

Merge #4727

4727: Drop test for old format r=matklad a=matklad

bors r+
🤖

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

Drop test for old format

Merge #4726

4726: Groundwork for specifying the set of projects via config r=matklad a=matklad

bors r+
🤖

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

Groundwork for specifying the set of projects via config

Rename ProjectRoot -> ProjectManifest

Move project discovery

Merge #4724

4724: Rename WorldState -> GlobalState r=matklad a=matklad

bors r+
🤖

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

Rename WorldState -> GlobalState

Merge #4723

4723: Derive local roots from Workspaces r=matklad a=matklad

bors r+
🤖

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

Drop no-project test

Eventually, we should support "just open random rust file" use case,
we don't really do this now, so let's avoid spending time on it until
we fix it properly.

Derive local roots from Workspaces

Put important things on top

Merge pull request #4382 from woody77/json_cfgs

Begin transition to new fields for JsonProject crate cfgs

Merge #4715

4715: Don't exclude dirs that appear in project root parent path r=matklad a=Veetaha

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

Merge #4679

4679: Update a comment for the new source organization r=matklad a=nelhage

Co-authored-by: Nelson Elhage <nelhage@nelhage.com>

Merge #4703

4703: Start documenting review process r=matklad a=matklad

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

Start documenting review process

Merge #4721

4721: Hide squiggly for unused and unnecessary diagnostics r=matklad a=GabbeV

Fixes #4229

When working with JavaScript or TypeScript in VSCode unused valiables are faded but don't have a squiggle. This PR makes rust-analyzer work similarly by setting the severity to hint when applying the unnecessary tag.

VSCode usually shows a squiggle for error, warning and information and shows three dots for hint. When the unnecessary tag is present the squiggles will still show up but the three dots will not.

This is my first contribution to open source. Please tell me if i need to do anything more to get this PR considered.

Co-authored-by: Gabriel Valfridsson <gabriel.valfridsson@gmail.com>

Hide squiggly for unused and unnecessary

Merge #4720

4720: Add highlight support for unsafe fn calls and raw ptr deref r=matklad a=Nashenas88

Addresses ide highlighting for #190

Co-authored-by: Paul Daniel Faria <Nashenas88@users.noreply.github.com>

Add highlight support for unsafe fn calls and raw ptr deref

Merge #4710

4710: New runnables r=matklad a=matklad

bors d=@vsrs

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

Don't exclude dirs that appear in project root parent path

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

Update a comment for the new source organization

Unsquish parameter types in tooltips for macro-generated functions

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>