Merge #10964

10964: minor: Move synstructure hack out of ItemTree lowering r=jonas-schievink a=jonas-schievink

Don't set the name to `None` for the rest of the IDE if it's generated by synstructure.

bors r+

Co-authored-by: Jonas Schievink <jonas.schievink@ferrous-systems.com>

Move synstructure hack out of ItemTree lowering

Merge #10950

10950: internal: completion: split out more PathKinds from ImmediateLocation r=Veykril a=Veykril

bors r+

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

internal: completion: split out more PathKinds from ImmediateLocation

Merge #10960

10960: fix: fix handling of macros in `extern` blocks r=jonas-schievink a=jonas-schievink

- Removes knowledge of what's in an extern block from `ItemTree`.
- Treats extern blocks as `AssocContainerId` and renames the latter to `ItemContainerId`.
- Threads the container through name resolution (which is a bit messy).
- Considers statics to be associated items, since they can be in an extern block. (it might be better to further distinguish potentially-associated-items from potentially-in-an-extern-block-items, but currently I don't expect much of a benefit)
- Adds a test for the incorrect-case diagnostic demonstrating the impact of this change.

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

bors r+

Co-authored-by: Jonas Schievink <jonas.schievink@ferrous-systems.com>

Treat extern blocks as item containers

Merge #10958

10958: internal: add "Shuffle Crate Graph" command r=jonas-schievink a=jonas-schievink

May be useful for debugging issues like https://github.com/rust-analyzer/rust-analyzer/issues/10084

bors r+

Co-authored-by: Jonas Schievink <jonas.schievink@ferrous-systems.com>

Update lsp-extensions.md

internal: add "Shuffle Crate Graph" command

Merge #10957

10957: fix: Fix some TryToNav impls not upmapping ranges out of macros r=Veykril a=Veykril

bors r+

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

Adjust incorrect runnable tests

Simplify

Fix some TryToNav impls not upmapping ranges out of macros

Merge #10953

10953: minor: Bump `tracing` r=lnicola a=lnicola

bors r+

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

Bump tracing

Merge #10949

10949: Bump parser step limit a little r=lnicola a=lnicola

Fixes #10948

This doesn't actually make the limit configurable, but at least uses the same `Limit` struct, so we can fix all of them at once when we get to it.

We also bump the limit from 10 to 15M, which is a small enough increase to not worry about making the experience worse for non-`windows` users. We still crash when we reach the limit.

bors r+

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

Bump parser step limit a little

Merge #10944

10944: internal: Prefer resolution of inert attributes r=Veykril a=Veykril

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/10942
bors r+

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

internal: Prefer resolution of inert attributes

Merge #10943

10943: feat: Enable completions for attributes r=Veykril a=Veykril

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

feat: Enable flyimport completions for attributes

Merge #10940

10940: minor: Fix HttpsProxyAgent creation and avoid deprecated call r=lnicola a=lnicola

bors r+

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

Fix HttpsProxyAgent creation and avoid deprecated call

Merge #10939

10939: minor: Clean up some unused cross-crate dependencies r=lnicola a=lnicola

bors r+

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

Clean up some unused cross-crate dependencies

Merge #10938

10938: minor: Bump d3 r=lnicola a=lnicola

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

Bump d3

Merge #10937

10937: fix: Add highlighting hack back for unresolved attributes r=Veykril a=Veykril

cc https://github.com/rust-analyzer/rust-analyzer/issues/10935
bors r+

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

fix: Add highlighting hack back for unresolved attributes

Merge #10902

10902: Handle multiple cargo check quick fix spans r=Veykril a=brandondong

Resolves https://github.com/rust-analyzer/rust-analyzer/issues/10705.

**Cause:**
- For a cargo check diagnostic with multiple spans, only a single quick fix action would be created at the location of `spans[0]`. Additionally, the hover window details would only show the location of `spans[0]` next to the message.

**Fix:**
- Allow cargo check quick fix actions to be triggerable from multiple selection ranges. Specifically, if the selection intersects with any of the replacement spans, the quick fix action is shown.
- No change in behavior for the hover window details. It's pretty minor and I think showing multiple locations next to the message may be more confusing anyways.

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

Update expected test results

Handle multiple cargo check quick fix spans

Refactor away unnecessary Vec

Merge #10930

10930: fix: Fix self keyword not being tagged as such in highlighting properly r=Veykril a=Veykril

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/10575
bors r+

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

fix: Fix self keyword not being tagged as such in highlighting properly

Merge #10929

10929: internal: Split up macro/attribute semantic tokens a bit more r=Veykril a=Veykril

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

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

Update semantic token docs

Introduce SymbolKind::Derive

Introduce SymbolKind::Attribute

Merge #10928

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

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

Bump pulldown-cmark-to-cmark

Bump miow

Bump countme

Bump chalk

Bump most deps

Merge #10927

10927: minor: bump the rest of the npm deps r=lnicola a=lnicola

bors r+

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

Bump rest of the npm deps

Bump node-fetch

Bump vsce

Bump eslint

Bump engine in lockfile

Merge #10925

10925: minor: Simplify `.vscodeignore` r=lnicola a=lnicola

The quadruple negation we used to have here was quite confusing:

```gitignore
**
!out
out/**
!out/src
```

Let's ignore everything and cherry-pick what we want to include.

~~And also include the extension sourcemap, just in case someone wants to debug the extension.~~

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

Simplify `.vscodeignore`

Merge #10924

10924: minor: Add LICENSE file to make vsce happy r=lnicola a=lnicola

Newer `vsce` versions error out if a `LICENSE` file is missing, even if the license is specified in `package.json`. Concatenate the two licenses we use and add a file with that to make a future `vsce` upgrade easier.

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

Merge #10926

10926: minor: Bump Node to 14 in release workflow r=lnicola a=lnicola

bors r+

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

Bump Node to 14 in release workflow

Add LICENSE file to make vsce happy

Merge #10921

10921: minor: Update list of safe intrinsics r=Veykril a=Veykril

Closes https://github.com/rust-analyzer/rust-analyzer/issues/4798
bors r+

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

minor: Update list of safe intrinsics

Merge #10920

10920: minor: Shorten spans of mismatched_arg_count diag r=Veykril a=Veykril

Fixes #5289
bors r+

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

minor: Shorten spans of mismatched_arg_count diag

Merge #10919

10919: internal: Define missing semantic token hierarchies in package.json r=Veykril a=Veykril

Closes https://github.com/rust-analyzer/rust-analyzer/issues/10077
bors r+

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

Add some more semantic token to textmate grammar mapping

Define missing semantic token hierarchies in package.json

Merge #10916

10916: feat: ide: Convert while let to loop r=Veykril a=rainy-me

close #10901

Co-authored-by: rainy-me <github@yue.coffee>

fix: defer condition construction

Merge #10915

10915: feat: Resolve builtin-attr and tools in ide layer r=Veykril a=Veykril

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

Improve hover message for inert attributes

Merge #10917

10917: minor: Bump some npm deps r=lnicola a=lnicola

bors r+

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

Bump engine

Bump esbuild

Bump typescript

Bump @types/vscode

Bump eslint

Bump vscode-test

Bump tslib

Bump node-fetch and @types/node-fetch

Bump @types/node

Bump d3

feat: handle while let to loop

Rename things: Tool -> ToolModule

Basic hover for builtin-attr and tool modules

Simplify

Remove syntax highlighting hack for builtin attrs

feat: Resolve builtin-attr and tools in ide layer

Merge #10906

10906: [first contributation] fix: `add return type` assit works when there's missing whitespace r=Veykril a=izik1

I feel like the way I handled whitespace here isn't... Right? Maybe it should be folded into `InsertOrReplace::Insert`

Also, sorry about the commit name, I am _not_ good at writing user facing commit messages

Also sorry about the test names, could be clearer I feel

Co-authored-by: Skyler Rain Ross <orangesnowfox@gmail.com>

refactor(assist/add_return_type): avoid threading `needs_whitespace`

Merge #10903

10903: internal: Replace `mocha` with a custom test runner r=Aloso a=lnicola

Closes #10723

r? `@Aloso`

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

fix: add return type works when there's missing whitespace

Merge #10905

10905: minor: Fix proxyStrictSSL check r=lnicola a=lnicola

bors r+

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

Fix proxyStrictSSL check

Merge #10891

10891: fix: emit trait names in moniker identifier r=Veykril a=tjdevries

Hi,

Not sure if this is the best fix for the problem, but I noticed that functions that part of a trait are not namespaced with the trait name, so this could lead to problems if you had a function and a trait-function both defined in the same module.

Thanks!

TJ

Co-authored-by: TJ DeVries <devries.timothyj@gmail.com>

Replace mocha with a custom test runner

fixup: include more information for impls

fixup: properly handle all associated items

Merge #10899

10899: ide: hack to make self not unresolved reference in async trait wrapped impl's r=Veykril a=jhgg

fixes #10708

this is a bit hacky, but it "works".

i'm not sure how to even write a test for this though, but i've confirmed it works via manual testing...

Co-authored-by: Jake Heinz <jh@discordapp.com>

add a fixme:

simplify

ide: hack to make self not unresolved reference in async trait wrapped impl's

Merge #10897

10897: internal: Try to fix Code tests r=lnicola a=lnicola

It looks like `--noEmit` broke our unit tests:


r? `@ChayimFriedman2`

I suppose we might still:

 - compile stuff twice
 - not detect failing tests on CI (since we didn't notice this)

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