Auto merge of #12712 - harpsword:fix-rename-crate-root, r=Veykril

fix: ignore renames for crate root

close #12684 . I just ignore renames for crate root in `rename_mod` func.

Auto merge of #12775 - flodiebold:syntax-fixup-if, r=flodiebold

fix: Improve syntax fixup a bit, handle incomplete `if`

- allow appending tokens after a token, not just a node
- allow inserting delimiters (and remove them again)
- fix up `if {}` and `if` without anything following

Improve syntax fixup a bit, handle incomplete `if`

- allow appending tokens after a token, not just a node
- allow inserting delimiters (and remove them again)
- fix up `if {}` and `if` without anything following

Auto merge of #12776 - matklad:nosu, r=matklad

try fixing spurious CI failures on windows

It seems like it may be related to self-update? Anyways, I think we
should be disabling self-update on CI

try fixing spurious CI failures on windows

It seems like it may be related to self-update? Anyways, I think we
should be disabling self-update on CI

Auto merge of #12773 - Veykril:self-compl, r=Veykril

fix: Improve self param completion applicability

Fixes https://github.com/rust-lang/rust-analyzer/issues/9522

fix: Improve self param completion applicability

Auto merge of #12772 - Veykril:nameres, r=Veykril

internal: Remove allocation in DefCollector::reseed_with_unresolved_attribute

Auto merge of #12766 - Veykril:completion-vis, r=Veykril

fix: Don't show qualified path completions for private items

Fixes https://github.com/rust-lang/rust-analyzer/issues/12703

internal: Remove allocation in DefCollector::reseed_with_unresolved_attribute

Auto merge of #12764 - matklad:rustupaction, r=lnicola

Remove deprecate action

https://old.reddit.com/r/rust/comments/vyx4oj/actionsrs_organization_became_unmaintained/

Looking at this holistically, I don't fully understand *why* we need an
action here? Seems like we can just use rustup? nowadays github runners
come with rustup pre-installed.

Remove deprecate action

https://old.reddit.com/r/rust/comments/vyx4oj/actionsrs_organization_became_unmaintained/

Looking at this holistically, I don't fully understand *why* we need an
action here? Seems like we can just use rustup? nowadays github runners
come with rustup pre-installed.

Update .github/workflows/release.yaml

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

Remove deprecate action

https://old.reddit.com/r/rust/comments/vyx4oj/actionsrs_organization_became_unmaintained/

Looking at this holistically, I don't fully understand *why* we need an
action here? Seems like we can just use rustup? nowadays github runners
come with rustup pre-installed.

fix: Don't show qualified path completions for private items

Auto merge of #12765 - Veykril:import-insert-fix, r=Veykril

fix: Fix imports being inserted before doc comments in inline modules

Fixes https://github.com/rust-lang/rust-analyzer/issues/12758

fix: Fix imports being inserted before doc comments in inline modules

Auto merge of #12556 - DorianListens:dscheidt/generic-extract, r=Veykril

fix: Support generics in extract_function assist

This change attempts to resolve issue #7636: Extract into Function does not
create a generic function with constraints when extracting generic code.

In `FunctionBody::analyze_container`, we now traverse the `ancestors` in search
of `AnyHasGenericParams`, and attach any `GenericParamList`s and `WhereClause`s
we find to the `ContainerInfo`.

Later, in `format_function`, we collect all the `GenericParam`s and
`WherePred`s from the container, and filter them to keep only types matching
`TypeParam`s used within the newly extracted function body or param list. We
can then include the new `GenericParamList` and `WhereClause` in the new
function definition.

This change only impacts `TypeParam`s. `LifetimeParam`s and `ConstParam`s are
out of scope for this change.

I've never contributed to this project before, but I did try to follow the style guide. I believe that this change represents an improvement over the status quo, but I think it's also fair to argue that it doesn't fully "fix" the linked issue. I'm totally open to merging this as is, or going further to try to make a more complete solution. Also: if there are other unit or integration tests I should add, please let me know where to look!

Auto merge of #12691 - Veykril:proc-macro-diag, r=Veykril

fix: Fix unresolved proc macro diagnostics pointing to macro expansions

Fixes https://github.com/rust-lang/rust-analyzer/issues/12657

Auto merge of #12735 - feniljain:fix-completions, r=feniljain

fix(completion): `super::` completion at crate root and module depth aware

- should close #12439
- Suggest super according to the depth in tree

fix(completion): `super::` completion at crate root and module depth
aware

Make search for applicable generics more precise

fix: Support generics in extract_function assist

This change attempts to resolve issue #7636: Extract into Function does not
create a generic function with constraints when extracting generic code.

In `FunctionBody::analyze_container`, we now traverse the `ancestors` in search
of `AnyHasGenericParams`, and attach any `GenericParamList`s and `WhereClause`s
we find to the `ContainerInfo`.

Later, in `format_function`, we collect all the `GenericParam`s and
`WherePred`s from the container, and filter them to keep only types matching
`TypeParam`s used within the newly extracted function body or param list. We
can then include the new `GenericParamList` and `WhereClause` in the new
function definition.

This change only impacts `TypeParam`s. `LifetimeParam`s and `ConstParam`s are
out of scope for this change.

Auto merge of #12755 - Veykril:server-version, r=lnicola

fix: Set server binary version when distributing

fix: Set server binary version when distributing

Auto merge of #12754 - Veykril:vscode-status, r=Veykril

fix: Fix VSCode status bar tooltip not showing the error messages

fix: Fix VSCode status bar tooltip not showing the error messages

Auto merge of #12696 - hi-rustin:rustin-patch-fix, r=Veykril

Add str_ref_to_string fix

close https://github.com/rust-lang/rust-analyzer/issues/11383
When type mismatch is `&str` -> `String` try to fix it.

Auto merge of #12742 - TonalidadeHidrica:fix-obsolete-config-keys, r=Veykril

Fix obsolete config keys

The config keys were drastically reorganized by #12010, but the docs don't reflect the updates, causing inconsistency and confusion.  I checked for such obsolete configuration keys and updated to the new one.  For reproducibility, I attach a small shell script that I used to examine the old keys.  Now the script only detects `cargoExtraArgs` and `overrideCargo`, which originates from other type definition in the code but not from the configuration.

<details><summary>script</summary>

```bash
echo "allowMergingIntoGlobImports
exprFillDefault
importEnforceGranularity
importGranularity
importMergeBehavior
importMergeBehaviour
importGroup
importPrefix
warmup
loadOutDirsFromCheck
runBuildScripts
runBuildScriptsCommand
useRustcWrapperForBuildScripts
enableExperimental
procAttrMacros
breakPoints
exitPoints
yieldPoints
linksInHover
linksInHover
gotoTypeDef
chainingHints
closureReturnTypeHints
hideNamedConstructorHints
parameterHints
reborrowHints
typeHints
lruCapacity
cargoExtraArgs
overrideCargo
rustcSource
enableRangeFormatting
assist\.allowMergingIntoGlobImports
assist\.exprFillDefault
assist\.importEnforceGranularity
assist\.importGranularity
assist\.importMergeBehavior
assist\.importMergeBehaviour
assist\.importGroup
assist\.importPrefix
primeCaches\.enable
cache\.warmup
cargo\.loadOutDirsFromCheck
cargo\.runBuildScripts
cargo\.runBuildScriptsCommand
cargo\.useRustcWrapperForBuildScripts
completion\.snippets
diagnostics\.enableExperimental
experimental\.procAttrMacros
highlighting\.strings
highlightRelated\.breakPoints
highlightRelated\.exitPoints
highlightRelated\.yieldPoints
highlightRelated\.references
hover\.documentation
hover\.linksInHover
hoverActions\.linksInHover
hoverActions\.debug
hoverActions\.enable
hoverActions\.gotoTypeDef
hoverActions\.implementations
hoverActions\.references
hoverActions\.run
inlayHints\.chainingHints
inlayHints\.closureReturnTypeHints
inlayHints\.hideNamedConstructorHints
inlayHints\.parameterHints
inlayHints\.reborrowHints
inlayHints\.typeHints
lruCapacity
runnables\.cargoExtraArgs
runnables\.overrideCargo
rustcSource
rustfmt\.enableRangeFormatting
allFeatures
addCallArgumentSnippets
addCallParenthesis
callInfo\.full
cargo\.allFeatures
checkOnSave\.allFeatures
completion\.addCallArgumentSnippets
completion\.addCallParenthesis" | while read -r pattern
do
  rg '\b'$pattern'\b([^.]|$)' . -g "!crates/rust-analyzer/src/config/patch_old_style.rs" -g "!editors/code/src/config.ts" -g "!a.sh" --no-heading --color=always --line-number
done

exit

excluded
# debug
# enable
# run
# implementations
# references
# documentation
# references
# snippets
# strings
# full
```

</details>

Auto merge of #12747 - jonas-schievink:proc-macro-abi-1.63, r=jonas-schievink

fix: Update 1.63 proc macro ABI to match rustc

This updates us to the ABI used by rustc 1.63.0-beta.5, which will likely be the ABI of the next stable Rust release. It should also work on nightly (for now, but future changes won't be supported until the rustc version is bumped).

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

Update 1.63 proc macro ABI to match rustc

Auto merge of #12745 - jonas-schievink:metavar-exprs, r=jonas-schievink

feat: Implement `ignore`  and `index` metavar expression

Part of https://github.com/rust-lang/rust-analyzer/issues/11952

Fixes https://github.com/rust-lang/rust-analyzer/issues/12675

Implement `ignore`  and `index` metavar expression

Auto merge of #12744 - hi-rustin:rustin-patch-typos, r=lnicola

Fix typos

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Fix typos

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Fix obsolete `completion.snippets` config

Fix obsolete `enableRangeFormatting` config

Fix `rustcSource` -> `rustc_source`

This only fixes the comments in the internal source,
     which is not mandatory at all

Fix obsolete `diagnostics.experimental.enable`

Fix obsolete `loadOutDirsFromCheck` configs

Fix config keys regarding imports in docs

fix: ignore renames for crate root

Auto merge of #12732 - lnicola:bump-npm, r=lnicola

Bump npm deps

Bump transitive npm deps

Bump @vscode/test-electron

Bump @types/node to 16

Bump typescript and tslib

Bump prettier

Bump @typescript-eslint

Bump vsce

Bump eslint

Bump esbuild

Bump d3 and d3-graphviz

Bump vscode-languageclient

Auto merge of #12727 - DorianListens:dscheidt/extract-var-field-name, r=jonas-schievink

fix: Improve suggested names for extracted variables

When extracting a field expression, if RA was unable to resolve the type of the
field, we would previously fall back to using "var_name" as the variable name.

Now, when the `Expr` being extracted matches a `FieldExpr`, we can use the
`NameRef`'s ident token as a fallback option.

fixes #10035

Fix test

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

fix: Improve suggested names for extracted variables

When extracting a field expression, if RA was unable to resolve the type of the
field, we would previously fall back to using "var_name" as the variable name.

Now, when the `Expr` being extracted matches a `FieldExpr`, we can use the
`NameRef`'s ident token as a fallback option.

fixes #10035

Auto merge of #12706 - DorianListens:dscheidt/closure-args, r=DorianListens

fix: Extract Function misses locals used in closures

This change fixes #12705.

In `FunctionBody::analyze`, we need to search any `ClosureExpr`s we encounter
for any `NameRef`s, to ensure they aren't missed.

fix: Extract Function misses locals used in closures

This change fixes #12705.

In `FunctionBody::analyze`, we need to search any `ClosureExpr`s we encounter
for any `NameRef`s, to ensure they aren't missed.

Auto merge of #12719 - davidlattimore:format-args-no-unsafe, r=jonas-schievink

Remove unnecessary unsafe from format_args expansion

Auto merge of #12676 - DorianListens:dscheidt/extract-fun-trait-impl, r=jonas-schievink

fix: Extract function from trait impl

This change fixes #10036, "Extract to function assist implements nonexistent
trait methods".

When we detect that the extraction is coming from within a trait impl, and that
a `self` param will be necessary, we adjust which `SyntaxNode` to `insert_after`,
and create a new empty `impl` block for the newly extracted function.

Auto merge of #12722 - jonas-schievink:update-urls, r=jonas-schievink

internal: Update remaining GitHub URLs

The old links still work, but it seems less confusing to have them point to the right repo

Update remaining GitHub URLs

Auto merge of #12716 - davidlattimore:fix-publish-crate-names, r=lnicola

publish workflow: Fix names for existing crates

Attempting to publish for example ra_ap_text-edit when ra_ap_text_edit
has already been published is rejected by crates.io.

Remove unnecessary unsafe from format_args expansion

publish workflow: Fix names for existing crates

Attempting to publish for example ra_ap_text-edit when ra_ap_text_edit
has already been published is rejected by crates.io.

Auto merge of #12695 - xuhongxu96:fix-12140, r=jonas-schievink

Complete type param/associated type in trait generic arg per arg index

- Fix #12140
- Also fix tidy check does not work for marks in multiline

fix default type param value position

Auto merge of #12704 - jonas-schievink:smol-paths, r=jonas-schievink

internal: Use `SmallVec` to slightly shrink `ModPath` size

Saves like a megabyte on r-a itself.

Use `SmallVec` to slightly shrink `ModPath` size

check arg_idx >= n_params only if arg_idx >= n_required_params

Handle generic args per arg index
Add more test cases for generic args

Auto merge of #12702 - lnicola:vscode-schemes, r=lnicola

internal: use different schemes for the custom views

Related to #12699, but doesn't fix it because we still register the providers multiple times.

Code: use different schemes for the custom views

Show only assoc type args in the correct arg pos

Add str_ref_to_string fix

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Auto merge of #12694 - hi-rustin:rustin-patch-bors, r=jonas-schievink

Remove useless bors.toml

It seems we do not use bors-ng anymore. So maybe this is useless.

Complete associated type only in trait generic arg
Fix tidy check does not work for marks in multiline

Fix project root assert

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Remove useless bors.toml

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

fix: Fix unresolved proc macro diagnostics pointing to macro expansions

Auto merge of #12690 - Veykril:inert-attrs, r=Veykril

internal: Update inert attribute list

internal: Update inert attribute list

Auto merge of #12687 - flodiebold:override-docs, r=flodiebold

Improve documentation for `buildScripts.overrideCommand` / `checkOnSave.overrideCommand`

Add back restartServerOnConfigChange option

Improve documentation for buildScripts.overrideCommand / checkOnSave.overrideCommand

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

Bump deps

Bump the rest of the deps

Bump chalk

Bump object

Bump tracing-subscriber

Bump arbitrary and derive_arbitrary

Bump cargo_metadata

Bump semver

Bump indexmap

Bump serde_json

Bump serde

Bump smallvec

Bump pulldown-cmark-to-cmark

Bump either