Remove simplistic interpolation for manifest-path

{manifest-path} interpolation

Use correct invocation strategy config for checkOnSave

Implement invocation strategy config for checkOnSave

Note that due to how cargo works, none of the modes currently work for r-a

Implement invocation strategy config for build scripts

Auto merge of #13423 - Veykril:vscode-vars, r=Veykril

Substitute some VSCode variables in the VSCode client

cc https://github.com/rust-lang/rust-analyzer/issues/13405

Auto merge of #13399 - DropDemBits:assists-format-args-capture-pt2, r=Veykril

Migrate assists to format args captures, part 2

Continuation of #13379

Migrates:

- `generate_constant`
- `generate_default_from_enum_variant`
- `generate_default_from_new`
- `generate_delegate_methods`
- `generate_deref`
- `generate_documentation_template`
- `generate_enum_is_method`
- `generate_enum_projection_method`
- `generate_from_impl_for_enum`
- `generate_function`
- `generate_getter`
- `generate_impl`
- `generate_new`
- `generate_setter`

Substitute some VSCode variables in the VSCode client

Auto merge of #13397 - zyctree:zyctree-patch-2, r=lnicola

fix link in syntax.md

update link in syntax.md

update link in syntax.md

Auto merge of #13354 - Veykril:try-stuff, r=Veykril

feat: Diagnose some incorrect usages of the question mark operator

Trying to figure out how the type stuff in r-a works some more, I think I am doing this correct here but I am not quite sure :)

Diagnose incorrect usages of the question mark operator

Auto merge of #13408 - lowr:patch/bump-chalk-0.86, r=Veykril

Bump chalk

There's a bug in current chalk that prevents us from properly supporting GATs, which is supposed to be fixed in v0.86. Note the following:
- v0.86 is only going to be released next Sunday so I'll keep this PR as draft until then.
- This doesn't compile without https://github.com/rust-lang/chalk/pull/779, which I hope will be included in v0.86. I confirmed this compiles with it locally.

Two breaking changes from v0.84:
- `TypeFolder` has been split into `TypeFolder` and `FallibleTypeFolder` (https://github.com/rust-lang/chalk/pull/772)
- `ProjectionTy::self_type_parameter()` has been removed (https://github.com/rust-lang/chalk/pull/778)

Bump chalk to 0.86

Two breaking changes:
- `TypeFolder` has been split into `TypeFolder` and `FallibleTypeFolder`
- `ProjectionTy::self_type_parameter()` has been removed

Auto merge of #13421 - rust-lang:Veykril-patch-1, r=Veykril

Update guide.md to reflect support for proc-macros

Update guide.md to reflect support for proc-macros

Auto merge of #13402 - HKalbasi:patch-1, r=Veykril

Cast runnableEnv items to string

fix #13390

An alternative approach could be raising an error if there is non string values.

Auto merge of #13420 - volsa:master, r=Veykril

fix: Ignore auto-import assist on parameter names

Fixes #13105; before & after

https://user-images.githubusercontent.com/29666622/195999489-0474c93a-b2bf-41c4-b7da-a4242a8082d8.mov

https://user-images.githubusercontent.com/29666622/195999571-605ee09c-bc6f-4ee5-bfe4-73e37254c647.mov

fix #13105: Ignore auto-import assist on parameter names

Auto merge of #13419 - lnicola:bump-setup-node, r=lnicola

Bump `actions/setup-node`

Fixes more Node 12 deprecation warnings.

Bump actions/setup-node

Auto merge of #13418 - lnicola:bump-deps, r=lnicola

Bump deps

Add new license

Bump the rest of the deps

Bump notify

Bump proc-macro2

Bump pulldown-cmark-to-cmark

Bump mimalloc

Bump rowan

Bump url

Bump libc

Bump dashmap

Bump home

Bump tracing

Bump smallvec

Bump anyhow, arbitrary, itertools, semver, serde

Auto merge of #13415 - lnicola:ovsx-timeout, r=lnicola

internal: Set a timeout when publishing to OVSX

CC https://github.com/rust-lang/rust-analyzer/issues/11080#issuecomment-1279212120

Set a timeout when publishing to OVSX

Auto merge of #13414 - lnicola:bump-cache, r=lnicola

minor: Bump `Swatinem/rust-cache` to move off Node 12

Bump Swatinem/rust-cache to move off Node 12

Auto merge of #13413 - rust-lang:revert-13367-fix-problem-matchers, r=jonas-schievink

Revert "fix: in VSCode, correctly resolve relative paths to errors"

Reverts rust-lang/rust-analyzer#13367

I didn't manage to figure out what exactly is the issue, so reverting this to fix https://github.com/rust-lang/rust-analyzer/issues/13404 before monday.

Revert "fix: in VSCode, correctly resolve relative paths to errors"

Auto merge of #13401 - lnicola:opt-tests, r=lnicola

Run `analysis-stats` on CI, with `opt-level = 1`

We might want to run `analysis-stats` on PRs, and this makes it less unbearable.

Run analysis-stats on CI

fix formatting

Cast runnableEnv items to string

Avoid format! in favor of to_string

Set opt-level = 1 on dev profile

Migrate assists to format args captures, part 2

Auto merge of #13358 - btwotwo:feature/env-vars-autocompletion, r=Veykril

feat: Autocomplete Cargo-defined env vars in `env!` and `option_env!` (#12448)

Closes #12448

Important to know:

- Variables are taken from https://doc.rust-lang.org/cargo/reference/environment-variables.html and hardcoded as a const array.
- For the sake of simplicity I didn't include the autocompletion of `CARGO_BIN_EXE_<name>` and `OUT_DIR` since it would require information about build.rs and binary name. If somebody knows an easy way of obtaining them I can add those vars as well :)

Formatting

Fix formatting for cargo vars list

Rename `get_outer_macro` to `macro_call_for_string_token`

Remove extra parameter, access Db through semantics

Auto merge of #13382 - lowr:fix/reorder-dyn-bounds-on-render, r=lowr

fix: reorder dyn bounds on render

Fixes #13368

#13192 changed the order of dyn bounds, violating the [contract](https://github.com/rust-lang/rust-analyzer/blob/3a69435af7a1e6273744085cb251adb2b9c30a03/crates/hir-ty/src/display.rs#L896-L901) with `write_bounds_like_dyn_trait()` on render. The projection bounds are expected to come right after the trait bound they are accompanied with.

Although the reordering procedure can be made a bit more efficient, I opted for relying only on the [invariants](https://github.com/rust-lang/rust-analyzer/blob/3a69435af7a1e6273744085cb251adb2b9c30a03/crates/hir-ty/src/lower.rs#L995-L998) currently documented in `lower_dyn_trait()`. It's not the hottest path and dyn bounds tend to be short so I believe it shouldn't hurt performance noticeably.

Update docs/dev/syntax.md

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

fix: reorder dyn bounds on render

fix link in syntax.md

Auto merge of #13357 - Veykril:minicore, r=Veykril

internal: Allow minicore flags specification to be order independent

Auto merge of #13329 - Veykril:rustc-proc-macro, r=Veykril

Use $crate instead of std for panic builtin_fn_macro

This should be closer to the expected output and gets rid of a few type mismatches in rustc/library

Auto merge of #13392 - Veykril:spec-pref, r=Veykril

Prefer similar tokens when expanding macros speculatively

Should improve completions in proc-macros in some cases

Prefer similar tokens when expanding macros speculatively

Auto merge of #13391 - dvdsk:update-manual, r=lnicola

Update manual now stable can be installed with rustup

this a new PR for #13374 as `bors squash` seemed to have broken `bors`
_______
`rustup` can now install `rust-analyzer` for the stable tool-chain. This commit removes the note that `rustup` can only install for the nightly branch and adjusts the command.

I also added a note on how to find the path to the `rust-analyzer` binary when installed using `rustup`, and suggestions on how to work around it not being placed in `~/.cargo/bin`.

I thought it would be ideal to point everyone to use `rustup run stable rust-analyzer` to start `rust-analyzer`. That would make it trivial to switch to nightly however I could not get this to work in `nvim` therefore I left it as a suggestion at the end.

Update manual now stable can be installed with rustup

`rustup` can now install `rust-analyzer` for the stable tool-chain. This commit removes the note that `rustup` can only install for the nightly branch and adjusts the command.

I also added a note on how to find the path to the `rust-analyzer` binary when installed using `rustup`, and suggestions on how to work around it not being placed in `~/.cargo/bin`.

I thought it would be ideal to point everyone to use `rustup run stable rust-analyzer` to start `rust-analyzer`. That would make it trivial to switch to nightly however I could not get this to work in `nvim` therefore I left it as a suggestion at the end.

Auto merge of #13386 - Veykril:completions-ref, r=Veykril

Refactor completions expansion

Depends on https://github.com/rust-lang/rust-analyzer/pull/13384
Diff is unfortunately massive as I changed the functions in the analysis module from associated ones to standalone (unfortunately without an extra commit)

Auto merge of #13380 - Veykril:cfg-pat-params, r=Veykril

Honor cfg attributes on params when lowering their patterns

Closes https://github.com/rust-lang/rust-analyzer/issues/13375

Auto merge of #13385 - Veykril:progress, r=Veykril

Don't report build-scripts and proc-macros as metadata progress

Seems somewhat confusing to me, given `metadata` is already the step we do for workspace loading

Refactor completions expansion

Don't report build-scripts and proc-macros are metadata progress

Auto merge of #13384 - Veykril:macro-fallbacks, r=Veykril

Expand unmatched mbe fragments to reasonable default token trees

Currently we expand unmatched fragments by not replacing them at all, leaving us with `$ident`. This trips up the parser or subsequent macro calls. Instead it makes more sense to replace these with some reasonable default depending on the fragment kind which should make more recursive macro calls work better for completions.

Expand unmatched mbe fragments to reasonable default token trees

Currently we expand unmatched fragments by not replacing them at all,
leaving us with `$ident`. This trips up the parser or subsequent macro
calls. Instead it makes more sense to replace these with some reasonable
default depending on the fragment kind which should make more recursive
macro calls work better for completions.

Auto merge of #13333 - feniljain:fix_assists, r=Veykril

fix(generate_method): correct method indentation inside generated impl and change gen loc

should fix #10619

Use $crate instead of std for panic builtin_fn_macro

This should be closer to the expected output and gets rid of a few
type mismatches in rustc/library

Auto merge of #13303 - jplatte:convert-named-struct-to-tuple-struct, r=Veykril

Add convert_named_struct_to_tuple_struct assist

Closes #11643, since the assist for converting in the other direction is already there (I based most of the implementation and all of the tests on it).

Auto merge of #13367 - matklad:fix-problem-matchers, r=Veykril

fix: in VSCode, correctly resolve relative paths to errors

VS Code problem matcher are restricted to be static "regexes". You can't create a problem matcher dynamically, and you can't use custom code in lieu of problem matcher.

This creates a problem for rust/cargo compiler errors. They use paths relative to the root of the Cargo workspace, but VS Code doesn't necessary know where that root is.

Luckily, there's a way out: our current problem matcher is defined like this:

    "fileLocation": [ "autoDetect", "${workspaceRoot}" ],

That means that relative pahts would be resoleved relative to workspace root. VS Code allows to specify a command inside `${}`. So we can plug custom logic there to fetch Cargo's workspace root!

And that's exactly what this PR is doing!

Honor cfg attributes on params when lowering their patterns

Auto merge of #13377 - DropDemBits:doc-link-underline-range, r=Veykril

fix: Underline only the intra-doc link itself instead of the whole doc comment

Auto merge of #13376 - DropDemBits:strip-generic-bounds-and-defaults, r=Veykril

internal: Add `GenericParamList::to_generic_args` and `{TypeParam,ConstParam}::remove_default` APIs

Also fixes `generate_impl` not removing the default const param value, though it seems that no one has encountered or reported that issue yet 😅

This initially started out as refactoring `utils::generate_impl_text_inner` to understand it better (which was the reason for adding `{TypeParam,ConstParam}::remove_default`), but ended up also finding another place that needed `GenericParamList::to_generic_args`, hence its addition in here.

Underline only the intra-doc link instead of the whole doc comment

Have `to_generic_args` return `ast::GenericArgList`

Add `{TypeParam, ConstParam}::remove_default`

Also includes a drive-by refactor of `utils::generate_impl_text_inner`,
since that's what drove this change

Add `GenericParamList::to_generic_args`

fix: in VSCode, correctly resolve relative paths to errors

VS Code problem matcher are restricted to be static "regexes". You can't
create a problem matcher dynamically, and you can't use custom code in
lieu of problem matcher.

This creates a problem for rust/cargo compiler errors. They use paths
relative to the root of the Cargo workspace, but VS Code doesn't
necessary know where that root is.

Luckily, there's a way out: our current problem matcher is defined like
this:

    "fileLocation": [ "autoDetect", "${workspaceRoot}" ],

That means that relative pahts would be resoleved relative to workspace
root. VS Code allows to specify a command inside `${}`. So we can plug
custom logic there to fetch Cargo's workspace root!

And that's exactly what this PR is doing!

Bump once_cell

Auto merge of #13366 - matklad:xflags, r=lnicola

internal: :arrow_up: xflags

The main change here should be that flags are not inhereted, so

   $ rust-analyzer analysis-stats . -v -v

would do what it should do

We also no longer Don\'t

internal: :arrow_up: xflags

The main change here should be that flags are not inhereted, so

   $ rust-analyzer analysis-stats . -v -v

would do what it should do

We also no longer Don\'t

Auto merge of #13362 - WaffleLapkin:go_to_def_fix_doc_include_str, r=Veykril

fix: Make go-to-def work for `#[doc = include_str!("path")]`

See the added test, go-to-def on `#[doc = include_str!("path$0")]` should navigate to `path`.

Fix go-to-def for `#[doc = include_str!("path")]`

Pass context to env vars completion

Restrict auto-completion for only built-in macros

Make helper method less specific

Formatting

Remove unnecessary dereference

Use helper method in is_format_string

Replace if with option, add detail for each env variable completion

Add tests for env var completion

Add const list of cargo-defined env variables with descriptions

Add helper method to get a macro name from passed string

Use expanded version of text for env var completion