Merge #5252

5252: Fix symbol search in salsa r=matklad a=matklad

Previous solution for binning paths into disjoint directories was
simple and fast -- just a single binary search.

Unfortunatelly, it wasn't coorrect: if the ditr are

  /d
  /d/a
  /d/c

then partitioning the file /d/b/lib.rs won't pick /d as a correct
directory.

The correct solution here is a trie, but it requires exposing path
components.

So, we use a poor man's substitution -- a *vector* of sorted paths,
such that each bucket is prefix-free

closes #5246

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

Fix symbol search in salsa

Previous solution for binning paths into disjoint directories was
simple and fast -- just a single binary search.

Unfortunatelly, it wasn't coorrect: if the ditr are

  /d
  /d/a
  /d/c

then partitioning the file /d/b/lib.rs won't pick /d as a correct
directory.

The correct solution here is a trie, but it requires exposing path
components.

So, we use a poor man's substitution -- a *vector* of sorted paths,
such that each bucket is prefix-free

closes #5246

Merge #5250

5250: :arrow_up: backtrace r=matklad a=matklad

bors r+
🤖

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

:arrow_up: backtrace

Merge pull request #5249 from matklad/ci

check cargo.lock

check cargo.lock

Upstream salsa

Merge #5248

5248: Upstream salsa r=matklad a=matklad

bors r+
🤖

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

Upstream salsa

Merge #5244

5244: Add a command to compute memory usage statistics r=matklad a=jonas-schievink

This allows inspecting memory usage on a live rust-analyzer instance after it has been used interactively.

This will only work with `--features jemalloc`, so maybe it should print something more useful when that's not available? Right now it will just print 0 Bytes for every query.

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

Merge #5247

5247: Modernize unqualified reference completion tests r=matklad a=matklad

bors r+
🤖

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

Modernize unqualified reference completion tests

Merge #5245

5245: Refactor AssistBuilder to manage a SourceChange r=matklad a=theduke

`AssistBuilder` now managaes a full `SourceChange` instead of a
`Vec<SourceFileEdit>`.

This prepares AssistBuilder to handle creation of new files.

Co-authored-by: Christoph Herzog <chris@theduke.at>

Refactor AssistBuilder to manage a SourceChange

`AssistBuilder`` now managaes a full `SourceChange` instead of a
Vec<SourceFileEdit>.

This prepares AssistBuilder to handle creation of new files.

Add a command to compute memory usage statistics

cleanup

Merge #5235 #5236 #5241

5235: Don't ping people in PRs r=matklad a=lnicola

5236: Disable ES module interop r=matklad a=lnicola

5241: Clippy perf warnings r=matklad a=kjeremy

Removes redundant clones

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>
Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>
Co-authored-by: kjeremy <kjeremy@gmail.com>

Update docs/dev/README.md

Merge #5243

5243: Consider EPERM error as other vscode processes using rust-analyzer r=matklad a=Veetaha

According to: https://github.com/rust-analyzer/rust-analyzer/issues/5009#issuecomment-654561497

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

Consider EPERM error as other vscode processes using rust-analyzer

Merge #5242

5242: Switch to fully dynamically dispatched salsa r=matklad a=matklad

This improves compile times quite a bit

bors r+
🤖

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

Switch to fully dynamically dispatched salsa

This improves compile times quite a bit

Clippy perf warnings

Removes redundant clones

Merge #5240

5240: Update crates r=kjeremy a=kjeremy

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

Update crates

Merge #5239

5239: sponsors link r=matklad a=matklad

bors r+
🤖

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

sponsors link

Disable ES module interop

Don't ping people in PRs

Merge #5234

5234: Fix: allow for binaries from $PATH to pass validity check r=matklad a=Veetaha

Tackles https://github.com/rust-analyzer/rust-analyzer/pull/5229#issuecomment-654151387
cc @matklad @lnicola
Apparently `fs.existsSync()` works only with real paths and not with `$PATH` env var

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

Fix: allow for binaries from $PATH to pass validity check

Merge #5233

5233: Update salsa r=jonas-schievink a=jonas-schievink

bors r+

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

Update salsa

Merge #5229

5229: Improve client logging (use output channel and more log levels) r=matklad a=Veetaha

The improvements:
* Separate output channel allows viewing the logs belonging to only our extension (without the intervention of other vscode extensions)
* All the objects in the output channel are always expanded so users only need to `Ctrl + A and Ctrl + C` to copy the entire output to send us and nothing more (e.g. currently users need to expand the object which is not obvious for them and we may lose the logs this way, see two comments: https://github.com/rust-analyzer/rust-analyzer/issues/5009#issuecomment-651361137
* More log levels allows us to be more granular in disabling only optional verbose debug-level output and leave the logs for us as developers to understand the context of user issues.
* For `log.error(...)` invocations we reveal `Rust Analyzer Client` channel automatically so that users don't have to do any additional actions to get the logs output window visible

Demo:

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

Merge #5231

5231: Simplify file watcher r=matklad a=matklad

bors r+
🤖

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

Simplify file watcher

Revert "Dispose logger on extension deactivation"

This reverts commit 13872543e074adc153b440660beda441fd562f53.
That commit was wrong because we use-after-free the logger

Dispose logger on extension deactivation

Improve client logging (use output channel and more log levels)

Merge #5228

5228: Fix typo r=matklad a=matklad

bors r+
🤖

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

Fix typo

Merge #5227

5227: AbsPath r=matklad a=matklad

bors r+
🤖

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

AbsPath

Merge #5226

5226: Modernise item completion tests r=matklad a=matklad

bors r+
🤖

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

Modernise item completion tests

Merge #4864

4864: Add optional target to crates in json project r=Nashenas88 a=Nashenas88

Lookup default cfgs per target when generating cfg list.

Should fully address #4840

CC @woody77

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

Fallback to target in cfg if not specified in project json config

Add optional target to crates in json project, lookup default cfgs per target when generating cfg list

Merge #5225

5225: Alight details in comkplation list r=matklad a=matklad

bors r+
🤖

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

Alight details in comkplation list

Modernize qual path completion tests

Modernize snippet completion tests

Merge #5223

5223: update deps r=kjeremy a=kjeremy

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

Merge #5222

5222: Add Item change to the set of benches r=matklad a=matklad

bors r+
🤖

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

Add Item change to the set of benches

Fix bench to work with relative paths

update deps

Merge #5221

5221: Modernize postfix completion tests r=matklad a=matklad

bors r+
🤖

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

Modernize postfix completion tests

Modernize pattern completion tests

Merge #5220

5220: Fix lookup in tests r=matklad a=matklad

bors r+
🤖

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

Fix lookup in tests

Macro tests

Modernize tests

Merge #5219

5219: Cleanup dot completiont tests r=matklad a=matklad

bors r+
🤖

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

Cleanup dot completiont tests

Comments

Better tests for completion scoring

Modernize completion presentation tests

Unify naming

Cleanup tests

Merge #5197

5197: SSR internal refactorings r=davidlattimore a=davidlattimore

- Extract error code out to a separate module
- Improve error reporting when a test fails
- Refactor matching code
- Update tests so that all paths in search patterns can be resolved

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

SSR: Update tests so that all paths in patterns can be resolved

SSR: Refactor matching code.

Mutable state is now stored in the enum Phase.
MatchState, since it now has no mutable state is renamed Matcher.
MatchInputs is merged into Matcher

SSR: Improve error reporting when a test fails

SSR: Extract error code out to a separate module

This is to make reusing it outside of parsing easier in a subsequent
change.

Merge #5215

5215: Cleanup more completion tests r=matklad a=matklad

bors r+

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

Cleanup more completion tests

Merge #5214

5214: Cleanup presentation tests r=matklad a=matklad

bors r+
🤖

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

Cleanup presentation tests

Merge #5213

5213: Add AssistKind::Generate r=matklad a=matklad

bors r+
🤖

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

Add AssistKind::Generate

Unify naming of generating assists

Merge #5212

5212: Fix module renaming r=matklad a=matklad

bors r+
🤖

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

Fix module renaming

Merge #5203

5203: Fix typo in description of vscode setting r=matklad a=Nashenas88

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

Compress rename tests

Use expect for rename tests

Adhere to code style in rename tests

Merge #5211

5211: Fix inference of indexing argument (partly) r=flodiebold a=flodiebold

We need to add the `T: Index<Arg>` obligation to be resolved later as well, otherwise we can't make inferences about `Arg` later based on the `Index` impls.

This still doesn't fix indexing with integer variables though; there's a further problem with Chalk floundering because of the variable, I think.

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

Fix inference of indexing argument (partly)

We need to add the `T: Index<Arg>` obligation to be resolved later as well,
otherwise we can't make inferences about `Arg` later based on the `Index` impls.

This still doesn't fix indexing with integer variables though; there's a further
problem with Chalk floundering because of the variable, I think.

Merge #5209

5209: Fixes to memory usage stats r=matklad a=jonas-schievink

This brings the unaccounted memory down from 287mb to 250mb, and displays memory used by VFS and "other" allocations.

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

Merge #5210

5210: Fix workspace reloading r=matklad a=matklad

bors r+
🤖

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

Fix workspace reloading

Track VFS and remaining/unaccounted memory

Fix memory usage accounting for interned queries

Merge #5207

5207: :arrow_up: salsa r=matklad a=matklad

bors r+
🤖

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

:arrow_up: salsa

Merge #5205

5205: Refactor attribut completion tests r=matklad a=matklad

bors r+
🤖

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

Refactor attribut completion tests