Draw the rest of the owl

collector changes WIP

lower traits

Hook up query, add impls, lower moar

Start item tree

Merge #5034

5034: Cleanup project.json deserialization r=matklad a=matklad

bors r+
🤖

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

Cleanup project.json deserialization

Rename json_project -> project_json

Merge #5015 #5027 #5028

5015: Account for updated module ids when determining whether a resolution is changed r=matklad a=Nashenas88

Fixes #4943

5027: Make Debug less verbose for VfsPath and use Display in analysis-stats r=matklad a=lnicola

5028: Remove namedExports config r=matklad a=lnicola

Fixes a warning:

```
(!) Plugin commonjs: The namedExports option from "@rollup/plugin-commonjs" is deprecated. Named exports are now handled automatically.
```

Co-authored-by: Paul Daniel Faria <Nashenas88@users.noreply.github.com>
Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Merge #5029

5029: Be more explicit about absolute paths at various places r=matklad a=matklad

bors r+
🤖

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

Fix formatting issue with glob shadow test

Be more explicit about absolute paths at various places

Use a macro in push_res to simplify code, simplify glob_shadowed_def assertion

Remove namedExports config

Make Debug less verbose for VfsPath and use Display in analysis-stats

Merge #5023

5023: Fix string literal inference in match r=flodiebold a=lnicola

Fixes #5020

r? @flodiebold

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

Merge #5024 #5026

5024: Simplify r=matklad a=matklad

bors r+
🤖

5026: Disable file watching when running slow tests r=matklad a=matklad

This should rid us of the intermittent test failure

https://github.com/rust-analyzer/rust-analyzer/pull/5017#issuecomment-648717983

bors r+
🤖

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

Simplify

Disable file watching when running slow tests

This should rid us of the intermittent test failure

https://github.com/rust-analyzer/rust-analyzer/pull/5017#issuecomment-648717983

Fix string literal inference in match

Simplify

Merge #5022

5022: Simplify r=matklad a=matklad

bors r+
🤖

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

Simplify

Simplify

Simplify

Merge #5021

5021: Reduce code dupication around fixtures r=matklad a=matklad

bors r+
🤖

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

Remove duplication

More consistent usage of fixtures

Use fixtures more

Merge #5018

5018: Simplify r=matklad a=matklad

bors r+
🤖

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

Simplify

Merge #4940

4940: Add support for marking doctest items as distinct from normal code r=ltentrup a=Nashenas88

This adds `HighlightTag::Generic | HighlightModifier::Injected` as the default highlight for all elements within a doctest. Please feel free to suggest that a new tag be created or a different one used.


Fixes #4929
Fixes #4939

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

Account for updated module ids when determining whether a resolution is "changed"

Improve readability be replacing hard-to-read if-else branches with a match

Merge #5014

5014: Use only one code-path for parsing fixtures r=matklad a=matklad

This removes leading newlines everywhere, shifting all ranges in tests
by one

bors r+
🤖

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

Use only one code-path for parsing fixtures

This removes leading newlines everywhere, shifting all ranges in tests
by one

Centralize fixture parsing for assists

Merge #5008

5008: Remove useless test code r=matklad a=matklad

bors r+
🤖

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

Remove useless test code

Merge #5013

5013: More principled indentation trimming in fixtures r=matklad a=matklad

bors r+
🤖

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

More principled indentation trimming in fixtures

Minor, rename

Merge #5012

5012: Slightly better name r=matklad a=matklad

bors r+
🤖

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

Slightly better name

Merge #5011

5011: Simplify fixtures r=matklad a=matklad

bors r+
🤖

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

Complicate

Fixing test fallout unfortunately requires more work, we need to do
it, but let's merge something at least!

Docs for Fixture

Cut problematic dependency

Nicer API

Simplify

Remove unrelated change

Simplify

Simplify

Drop rarely used fixture functionality

Move fixtures to a separate file

Update comment for pop_and_inject

Co-authored-by: Leander Tentrup <ltentrup@users.noreply.github.com>

Update injection mechanism and stop injecting through highlight element, switch to more general new highlight tag, generic

Add punctuation highlighting for highlighting punctuation in doctests, fix highlighting in doctests

Add default color and opacity for documentation and injected, respectively, in the html generator

Add support for marking doctest items as distinct from normal code, add default tag to all doctest elements

Merge #4976

4976: New VFS r=matklad a=matklad

Still a draft, but mostly working already.

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

New VFS

Merge #5010

5010: Remove relative-path dependency r=matklad a=matklad

bors r+
🤖

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

Remove relative-path dependency

Merge #5004

5004: Fix panic in split/merge import assists r=matklad a=lnicola

Fixes #4368 #4905

Not sure if this is the best solution here. Maybe the `make` functions should be fallible? We generally seem to be playing whack-a-mole with panics in assists, although most of them are `unwrap`s in the assist code.

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

Fix panic in split and merge import assists

Merge #5002

5002: Fix underflow panic when doctests are at top of file r=Nashenas88 a=Nashenas88

While debugging a comment at the top of a test string, I discovered that the offset calculations could underflow and panic. This only seemed to occur in tests, I assume because it's running a debug mode. The wrapping is quickly fixed later on in release mode, which is why this seems to have gone unnoticed. The new checks ensure the value is always positive or zero.

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

Merge #4992 #4993 #4994 #4997 #4998

4992: Never disable error logging on the frontend r=matklad a=Veetaha

4993: Make bootstrap error message more informative and better-fitting r=matklad a=Veetaha

Now this better fits standard vscode extension activation failure message and suggests enabling verbose logs.


4994: Decouple http file stream logic from temp dir logic r=matklad a=Veetaha

Followup for #4989

4997: Update manual.adoc r=matklad a=gwutz

GNOME Builder (Nightly) supports now rust-analyzer

4998: Disrecommend trace.server: "verbose" for regular users r=matklad a=Veetaha

This option has never been useful for me, I wonder if anyone finds regular users can use this for sending logs

Co-authored-by: Veetaha <veetaha2@gmail.com>
Co-authored-by: Günther Wagner <info@gunibert.de>

Merge #4999

4999: SSR: Allow matching of whole macro calls r=matklad a=davidlattimore

Matching within macro calls is to come later and matching of macro calls within macro calls later still.

Co-authored-by: David Lattimore <dml@google.com>

Update docs/user/manual.adoc

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

Fix underflow panic when doctests are at top of file

Merge #5000

5000: Remove RelativePathBuf from fixture r=matklad a=matklad

The paths in fixture are not really relative (the default one is
`/main.rs`), so it doesn't make sense to use `RelativePathBuf` here.

bors r+
🤖

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

Remove RelativePathBuf from fixture

The paths in fixture are not really relative (the default one is
`/main.rs`), so it doesn't make sense to use `RelativePathBuf` here.

SSR: Allow matching of whole macro calls

Matching within macro calls is to come later and matching of macro calls within macro calls later still.

Disrecommend trace.server: "verbose" for regular users

Update manual.adoc

GNOME Builder (Nightly) supports now rust-analyzer

Decouple http file stream logic from temp dir logic

Never disable error logging on the frontend

Make bootstrap error message more informative and better-fitting

Merge #4989

4989: Hotfix skipping the first chunks of the artifacts r=matklad a=Veetaha

Quick hotfix.
fixes: #4986, #4987

The stream starts being consumed once we put a handler for `data` event.
When extracting `stream.pipeline()` under `withTempFile` in #4963 I didn't move it into the scope too, which due to preliminary awaiting for async operations with the file system allowed for the first chunks of the file to be skipped

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

Hotfix skipping the first chunks of the artifacts

Merge #4988

4988: Don't offer to add missing fields of unknown types r=flodiebold a=lnicola

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

Don't offer to add missing fields of unknown types

Merge #4900

4900: Self variant enum res fix r=BGluth a=BGluth

Fixes #4789.

This is my first PR for this project, so it's probably worth giving it an extra close look.

A few things that I wasn't sure about:
- Is `resolve_path` really the best place to perform this check? It seemed like a natural place, but perhaps there's a better place?
- When handling the new variant `PathResolution::VariantDef`, I couldn't see an obvious variant of `TypeNs` to return in `in_type_ns` for Unions and Structs.

Co-authored-by: BGluth <gluthb@gmail.com>

Enum variants with `Self::[variant]` now resolve (#4879)

Merge #4984

4984: Add sponsors section to changelog r=matklad a=matklad

bors r+
🤖

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

Add sponsors section to changelog

Merge #4983

4983: Add paths::RelPath[Buf] r=matklad a=matklad

bors r+
🤖

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

Add paths::RelPath[Buf]

Merge #4899

4899: don't complete top level attrs inside nested attrs and add better labels r=matklad a=bnjjj

close #4890

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

Merge #4921

4921: Allow SSR to match type references, items, paths and patterns r=davidlattimore a=davidlattimore

Part of #3186

Co-authored-by: David Lattimore <dml@google.com>

Allow SSR to match type references, items, paths and patterns

Part of #3186

don't complete top level attrs inside nested attrs and add better labels #4899

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

Merge #4928

4928: Fix renaming mod in use tree r=matklad a=unexge

Fixes #4825

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

Created goto Self enum variant test

don't complete top level attrs inside nested attrs and add better labels #4899

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

Merge #4971

4971: cargo update r=kjeremy a=kjeremy

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

Update test for new serde

cargo update

Merge #4969

4969: Handle bindings after @ in patterns r=flodiebold a=jonas-schievink

This is unstable, behind the `bindings_after_at` feature gate, but the semantics are fairly clear, and this is used at lot in rustc.

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