If there are source root dirs nested inside each other, a module might have
more than one possible name, depending on where it's intended to be imported
from.  In this case, a rename should not be suggested unless the module name
doesn't match any possible correct name.  When suggesting a name, the shortest
name should be suggested, since that's more likely to be the intended one.

Fixes #1903

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>

* [Call hierarchy] Update change log for 1.0.1.0

* Add hackage release instructions

* Add missing new line

* Add initial list of merged prs

* Update list of prs

* Add a initial release description

* Add rename plugin

* Add rename demo and last pr

* Restore significant whitespaces

* Correct grammar

* Add some newlines

* Bump up versions

* Complete Changelog

* Ignoring time resolution test

* Remove reference to rename plugin

Co-authored-by: Lei Zhu <julytreee@gmail.com>

* Extend TopLevelRHS pattern to track the where clause

* Don't case split if there's a where clause

* Add tests

Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>

* update ghc-api-compat

* Add ghc-api-compat to 9.0.1

* update sha256

acquired by providing empty sha256 and causing error

* Update sha for ghc-api-compat-9.0.1

Co-authored-by: Jade <software@lfcode.ca>
Co-authored-by: Peter Becich <peterbecich@gmail.com>

* completions for project identifiers

* simplify import completions

Since the projectExportsMap already indexes home modules, there is no need to handle them separately

* Use maxBound of uinteger not Int.

* Add a comment explaining the magic number 2147483647.

* Replace usages of maxBound with maxBoundUinteger in test and bench.

* Update lsp issue link

Co-authored-by: Junyoung "Clare" Jang <jjc9310@gmail.com>
Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Add a newUnivar machinery, and strengthen unification tests

* Get rid of explicit univar constructions

* Cleanup AutoTupleSpec

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>

to rerun other workflows via pr comments
see https://github.com/marketplace/actions/rerun-actions

* Do a cabal update

* Set false explicitly

* Correct outputs

* Show entire diff

* Remove duplicate cabal update

It's quite unclear in the spec, but in
microsoft/language-server-protocol#970
it's suggested that the intention is that the kinds given in `only`
should be used as *prefix* filters of the generated code action kinds.

That is to say, if the client asks for `only = [ CodeActionRefactor ]`,
we should give them all kinds of refactoring code actions, including
those whose kind is `CodeActionRefactorInline` (because as "hierarchical
strings" they are represented as `"refactor"` and `"refactor.inline"`).

This is quite important for the client: e.g. I hit this because I wanted
to ask for all the import quickfixes so I could present them to the user
to pick one. But they use various subkinds of `"quickfix.import"`, so
currently you cannot ask for them all (asking for `"quickfix.import"`
currentl returns nothing!).

The ipmlemention is a little ugly: this needs some helper funcitons in
`lsp`, which I'll make a PR for separately, but I didn't want to block
this.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Update nix flake

Doing a nixpkgs update gains ghc 8.10.6 support.

lsp-types needed updating to satisfy bounds so I did that.

* Add ghc-lib-parser override required for ormolu

* Put the override in the right spot, oopsie

* Pass --print-build-logs to nix commands

* Fix tracy on darwin

* Remove the macOS hack since it got merged

* It appears this is actually ghc8107 by now

Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>
Co-authored-by: Potato Hatsue <1793913507@qq.com>

* auto complete functions from imports

* address PR comments

* clean up

* remove duplicate HashMap import

* use lookupDefault

* fuzzy match filter

* add field to exportsMap

* generate map from modIFace

* Update ghcide/src/Development/IDE/Types/Exports.hs

Co-authored-by: Pepe Iborra <pepeiborra@me.com>

* module name text alias

* use hashset; enable local modules

* local module imports now working

* derive map from exportMap

* generate maps from list

* clean up

* addressing PR comments

* clean up

* clean up

* useWithStaleFast

Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>
Co-authored-by: alexnaspoleap <88672779+alexnaspoleap@users.noreply.github.com>
Co-authored-by: Alex Naspo <alex.naspo@leapyear.io>
Co-authored-by: Pepe Iborra <pepeiborra@me.com>
Co-authored-by: Pepe Iborra <pepeiborra@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Including the update of some .cabal files to make `cabal check` happy

* Complete contributing guide

With the contents of ./CONTRIBUTING.md (mainly the pre commit hook to format)

* Delete CONTRIBUTING.md

Included in the main documentation

* Move style guidelines

As building and testing are more important imo

* Correct grammar

Co-authored-by: Jan Hrcek <2716069+jhrcek@users.noreply.github.com>

* Note cabal run behaviour

Co-Authored-By: jhrcek

Co-authored-by: Jan Hrcek <2716069+jhrcek@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Plugin in config files

List the config files where a plugin should be added

* Correct grammar

Co-authored-by: Pepe Iborra <pepeiborra@gmail.com>

Co-authored-by: Pepe Iborra <pepeiborra@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Remove ghc-api source snapshot

We can replace the source snapshot with Cabal conditions.
This will unblock Hackage uploads until
https://github.com/haskell/haskell-language-server/pull/2128 lands

* fix 9.0.1 build

* add allow-newer for ghc-api-compat

* fix version bounds for 9.0.1

* fix hls-eval-plugin

* stack fixes

* added new versions of ghc-api-compat

* remove source snapshot from ghc901 cabal project

* add a gitpod descriptor

* preinstall haskell extension and error lens

* before

* fix tasks

* workspace

* cabal dir

* stack root

* Make HLS work in gitpod

* tweaks

* Fix GHC version

* Correct instance for incoming

* Get rid of constant delay

* Add metatactic for let-bindings

* Add test for simple let bindings

* Label hole numbers in the documentation

* Sort imports

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Co-authored-by: Junyoung "Clare" Jang <jjc9310@gmail.com>

* Remove HasSrcSpan import

* Amend extendImportHandler' function

* Amend old import tests

* Adapt tests for windows newline issue

Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>

It fails to build with transient segfaults

Co-authored-by: Junyoung "Clare" Jang <jjc9310@gmail.com>

GitHub now supports citation files https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/creating-a-repository-on-github/about-citation-files



Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Add stack yaml for ghc-8.10.7

* support for ghc-8.10.7

* Use void as suggested by hlint

* Remove duplicate build

* Update extra-deps for ghc-8.10.7

Copied from #2138 stack-8.10.6.yaml

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

* allow newer ghc for ghc-api-compat-8.10.6

Co-authored-by: Junyoung "Clare" Jang <jjc9310@gmail.com>
Co-authored-by: peterbecich <peterbecich@gmail.com>

Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: alexnaspoleap <88672779+alexnaspoleap@users.noreply.github.com>

* update LTS for GHC 8.10.6

https://www.stackage.org/lts-18.7



* update GHC 8.10.6 extra-deps

* Use void as suggested by hlint

Co-authored-by: jneira <atreyu.bbb@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Move pragmas completion to pragmas plugin

* update pragmas plugin tests

* move options pragmas to pragmas plugin

return empty list from logic completions when opts or pragma start

fix empty list logic

Co-authored-by: Alex Naspo <alex.naspo@leapyear.io>
Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>

Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* List all available plugins

* Make hlint happy

Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>

* Update ghc-9.0.1 support

* Use hie-bios master for stack

* Enable rest of available plugins

* Clean cabal allow-newer

* Disable class plugin

* Use void as suggested by hlint

* Use hackage versions

* Disable refineImports plugin

* Comment out refineImports

and use hackage released versions for blaze-textual and hie-bios

* Use extra < 1.7.10

Co-authored-by: Junyoung "Clare" Jang <jjc9310@gmail.com>

* Support call hierarchy on pattern matching

* Make result satisfied with the way VSCode processes data

* Version bump

* Support for ghc-8.10.6

* Always miss this one

* Update tested-with .cabal field

* Pin optparse-applicative for stylish-haskell

* Add needed extra-deps

* WIP abstract LSP, take the pain out of writing LSP stuff

* Finish making commands

* Separate code lenses and actions

* Pull out types

* Finalize the abstract API

* Bug fix in JSON; first connected abstract handler

* Add ContinuationResult for better control over how edits work

* Remove IO from TacticProviders; use LspEnv instead

* installInteractions

* Pull TacticCodeActions into their own file

* Misc cleanup

* Haddock

* Fix bug in codelens

* Port EmptyCase to Interaction

* Rename makeTacticCodeAction -> makeTacticInteraction

* Support for partial timeouts in upcoming refinery v5

* asum instead of choice for assumption

* Don't count it as using a term if you only destruct it

* Let interactions return multiple results --- aka also info messages

* Update refinery lower bounds

* Revert "Update refinery lower bounds"

This reverts commit 53199b3b781884d7bd85ac8fb3efa1d2da915ea6.

* Pull refinery from the future

* Fix tests

* Add -XNumDecimals

* Fix AutoTypeLevel test

* Continue to emit errors

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* replace head with safer uncons

* if empty, stay empty

* add test for removal of operators ending with '.'

* make remove-import-actions tests pass

* add more tests and debug output for modifyBinding

* implement reviewers suggestions

* restore ghcide/bench/example/HLS

Co-authored-by: Pepe Iborra <pepeiborra@gmail.com>