Merge #4614

4614: Fix some clippy perf warnings r=kjeremy a=kjeremy

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

Fix some clippy perf warnings

Fix conflict between nightly branch and tag

Merge pull request #4612 from matklad/fix-tag

Try fixing release tags

Try fixing release tags

closes #4319

Merge #4602 #4603

4602: Add boolean literal semantic token type to package.json r=matklad a=lnicola

Closes #4583.

CC @GrayJack

4603: Add self keyword semantic token type r=matklad a=lnicola

Not sure if this is warranted a new token type or just a modifier.

---

CC #4583, @GrayJack

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

Merge #4611

4611: Document `parentModule` experimental LSP request r=matklad a=matklad

bors r+
🤖

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

Document `parentModule` experimental LSP request

Merge #4610

4610: Cleanup lsp extensions on the client side r=matklad a=matklad

bors r+
🤖

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

Cleanup lsp extensions on the client side

Merge #4607

4607: Less rust-analyzer specific onEnter r=matklad a=matklad

bors r+
🤖

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

Less rust-analyzer specific onEnter

Merge #4605

4605: Reorganize TypeScript r=matklad a=matklad

bors r+
🤖

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

Flatten commands.ts

Flatten simple commands

Add self keyword semantic token type

Remove dead code

Add boolean literals to package.json

Merge #4601

4601: Introduce `toggle inlay hints` vscode command r=matklad a=Veetaha

Users now can assign a shortcut for this command
via the general vscode
keybindings ui or `keybindings.json` file

<details>
<summary>Demo</summary>


</details>

<details>
<summary>Howto assign a shortcut</summary>


</details>

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

Introduce `toggle inlay hints` vscode command

Users now can assign a shortcut for this command
via the general vscode
keybindings ui or `keybinding.json file`

Closes: #4599

Merge #4593

4593: Document some rust-analyzer specific protocol extensions r=matklad a=matklad

bors r+
🤖

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

Fix formatting

Document some rust-analyzer specific protocol extensions

Document matchingBrace LSP request

Merge #4556

4556: More highlighting improvements r=matthewjasper a=matthewjasper

* Separate `true` and `false` from keywords (this matches the Textmate grammar).
* Handle more cases in `highlight_name_by_syntax`.

Co-authored-by: Matthew Jasper <mjjasper1@gmail.com>

Handle more cases in `highlight_name_by_syntax`

Highlight `true` and `false` as literals

Fix bors config?

Merge pull request #4574 from lnicola/bump-deps

Bump some dependencies

Reorgonise extensions docs

Fix tests

Fix Cargo.lock

Merge pull request #4548 from bnjjj/fix_4464

add support of feature flag for runnables

Merge pull request #4495 from vsrs/fixture_meta

Test fixtures parsing improvements

Merge pull request #4474 from georgewfraser/color_attrs

Color attribute functions

Bump more crates

Bump more npm deps

Bump some npm deps

Bump rustc_lexer

Merge pull request #4590 from Julian-Wollersberger/update_unescape

Update to rustc_lexer version 660

add support of feature flag for runnables #4464

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

Some FIXMEs were outdated:
* Done at line 243: "Add validation of `crate` keyword not appearing in the middle of the symbol path"
* Already happened: "Remove validation of unterminated literals (it is already implemented in `tokenize()`)"
* Happens in `unescape()`: "Add validation of character literal containing only a single char"
* Missing: "raw string literals and raw byte string literals"

Update to rustc_lexer version 660.
Change `unescape_*()` to `unescape_literal()`.

add support of feature flag for runnables #4464

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

Merge pull request #4538 from vsrs/vscode_tests

vscode client side tests

vscode tests configuration for more verbose output

OS matrix for TypeScript

xvfb-action fix

Try GabrielBB/xvfb to run vscode tests

Start xvfb manually on ubuntu

try to run TypeScript CI on windows

Add `npm test` to CI

Merge #4555 #4575

4555: VSCode: added patchelf after download for NixOS support r=matklad a=cab404

This adds Nix support, and fixes #4542

4575: Use Chalk's built-in representations for fn items and pointers r=matklad a=flodiebold

The `TypeName::FnDef` was just added; the function pointer variant has existed for a while, I just forgot about it because it's special (because fn pointers can be higher-ranked over lifetimes).

We *could* also make `FnPtr` a separate `Ty` variant instead of a `TypeCtor` variant, which would make the conversion code a bit less special-casey, but it doesn't seem worth doing right now.

Co-authored-by: Vladimir Serov <me@cab404.ru>
Co-authored-by: Cabia Rangris <me@cab404.ru>
Co-authored-by: Florian Diebold <florian.diebold@freiheit.com>

Merge #4578

4578: Remove unnecessary clone that prevented clippy from moving on r=matklad a=kjeremy

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

Merge #4559

4559: Module name on hover shows another newline after it r=matklad a=Arthamys

This changes the display of hover information to add a newline between the module path of the item and the signature of the item, as suggested in #3813

**Before**


**After**


Co-authored-by: Galilée 'Bill' Enguehard <galilee.enguehard@gmail.com>

Fix resolve_proc_macro heavy test

And a few drive-bys

Remove unnecessary clone that prevented clippy from moving on

Use Chalk's Ty::Function for function pointer types

Function pointers can be 'higher-ranked' over lifetimes, which is why they're
not an application type in Chalk, but since we don't model lifetimes it doesn't
matter for us yet.

Use Chalk's built-in representation of function item types

Merge #4573

4573: Chalk cleanups r=matklad a=flodiebold

 - split off the `Interner` impl and the conversion code into their own modules
 - use `TypeCtorId` as `AdtId` directly (instead of raw `InternId`s), and rename the type alias `StructId` -> `AdtId`
 - provide missing Chalk debug impls

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

Update tests to accept new hover formatting

Fix formatting error when no modpath is present

Provide missing Chalk debug methods

Use TypeCtorId as AdtId directly, and rename the type alias StructId -> AdtId

Split up chalk module a bit

Merge #4570

4570: Use Chalk's built-in impls r=matklad a=flodiebold

This contains two changes:
 - Chalk has begun adding built-in representations of primitive types; use these in our type conversion logic. There's one somewhat 'iffy' part here, namely references; we don't keep track of lifetimes, but Chalk does, so it will expect a lifetime parameter on references. If we didn't provide that, it could cause crashes in Chalk code that expects the lifetime, so I rather hackily add an (always the same) lifetime placeholder during conversion. I expect that we'll fully switch to using Chalk's types everywhere before we add lifetime support, so I think this is the best solution for now.
 - let Chalk know about well-known traits (from lang items), so it can apply its built-in impls.

Before:
```
Total expressions: 181485
Expressions of unknown type: 2940 (1%)
Expressions of partially unknown type: 2884 (1%)
Type mismatches: 901
Inference: 37.821210245s, 0b allocated 0b resident
Total: 53.399467609s, 0b allocated 0b resident
```

After:
```
Total expressions: 181485
Expressions of unknown type: 2923 (1%)
Expressions of partially unknown type: 2879 (1%)
Type mismatches: 734
Inference: 39.157752509s, 0b allocated 0b resident
Total: 54.110767621s, 0b allocated 0b resident
```

(I will start splitting up `chalk.rs` in a separate PR, since it's getting pretty big...)

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

Merge #4572

4572: Link upstream issues r=matklad a=matklad

bors r+
🤖

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

Link upstream issues

Modify hover info to show module path separated by a line

Add some tests for Chalk built-in trait impls

Merge #4571

4571: KISS SourceChange r=matklad a=matklad

The idea behind requiring the label is a noble one, but we are not
really using it consistently anyway, and it should be easy to retrofit
later, should we need it.

bors r+

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

KISS SourceChange

The idea behind requiring the label is a noble one, but we are not
really using it consistently anyway, and it should be easy to retrofit
later, should we need it.

Merge #4569

4569: CodeAction groups r=matklad a=matklad

bors r+
🤖

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

Provide Chalk well-known traits

Use Chalk's built-in representations of primitive types

For references, we make sure Chalk actually gets a lifetime here.

CodeAction groups

Merge #4568

4568: Update Chalk r=matklad a=flodiebold

As always, this just makes compilation work, we don't use the newly available
functionality yet.

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

Update Chalk

As always, this just makes compilation work, we don't use the newly available
functionality yet.

Merge #4516

4516: LSP: Two stage initialization r=kjeremy a=kjeremy

Fills in server information.

Derives CodeAction capabilities from the client. If code action literals
are unsupported we fall back to the "simple support" which just sends back
commands (this is already supported in our config). The difference being
that we did not adjust our server capabilities so that if the client was
checking for `CodeActionProvider: "true"` in the response that would have failed.

Part of #144
Fixes #4130 (the specific case called out in that issue)

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

add support of feature flag for runnables #4464

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

Merge #4560

4560: Use WorkspaceEdit for ssr r=matklad a=matklad

bors r+
🤖

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

Use WorkspaceEdit for ssr

Merge branch 'master' of github.com:rust-analyzer/rust-analyzer into modname_spacing

Add extra newline after module path in hover info

Closes issue #3813

Merge branch 'nix-patchelf' of github.com:cab404/rust-analyzer into nix-patchelf

editors/vscode: forgotten await in os check

editors/vscode: patchelf-ing without intermediate files

Merge #4557

4557: Formalize JoinLines protocol extension r=matklad a=matklad

bors r+
🤖

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

Formalize JoinLines protocol extension

Update editors/code/src/main.ts

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

editor/vscode: lint

editors/vscode: removing original file after patching

editors/vscode: added patchelf after download

Merge #4553

4553: Cleanup r=matklad a=matklad

bors r+
🤖

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

Cleanup

Cleanup TextEdit

Merge #4552

4552: Transition OnEnter to WorkspaceSnippetEdit r=matklad a=matklad

This also changes our handiling of snippet edits on the client side.
`editor.insertSnippet` unfortunately forces indentation, which we
really don't want to have to deal with. So, let's just implement our
manual hacky way of dealing with a simple subset of snippets we
actually use in rust-analyzer

bors r+
🤖

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

Remove dead code for handling cursor positions

Transition OnEnter to WorkspaceSnippetEdit

This also changes our handiling of snippet edits on the client side.
`editor.insertSnippet` unfortunately forces indentation, which we
really don't want to have to deal with. So, let's just implement our
manual hacky way of dealing with a simple subset of snippets we
actually use in rust-analyzer