haskell / haskell/HieDb

configurable limit in searchDef

Open
#34 3 comments 1 reaction 0 assignees View on GitHub
Dominant language
Haskell
Stars
69
Forks
30
PR merge metrics
No merged PRs in 30d

Description

currently, `searchDef` is implemented like
https://github.com/wz1000/HieDb/blob/bb7367f513eb17b4e81ed1339b8a583bfc521326/src/HieDb/Query.hs#L121-L126
the `LIMIT` on line 126 is problematic for HLS's workspace symbol handler, since it frequently omits symbols from a workspace_symbols query in a way that's opaque to the user
https://github.com/haskell/haskell-language-server/blob/3be343acf76f05fb9b8adb73625439a5d40b362c/ghcide/src/Development/IDE/Core/Actions.hs#L123-L127

I think it's fine to either

1. remove the `LIMIT` line
2. add a `Maybe Int` parameter or similar to make this limit configurable

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.