Skip to content

Commit a58ccda

Browse files
committed
Use list inclusion on filter_by to bypass operation limits
1 parent dad837a commit a58ccda

File tree

1 file changed

+12
-10
lines changed

1 file changed

+12
-10
lines changed

src/hexdocs/services/hexdocs.gleam

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -195,16 +195,18 @@ fn new_search_body(
195195
}
196196

197197
fn get_filter_by_packages(packages: List(version.Package)) -> String {
198-
use <- bool.guard(when: list.is_empty(packages), return: "")
199-
packages
200-
|> list.filter_map(fn(p) {
201-
case p.status {
202-
version.Found(ver) -> Ok(p.name <> "-" <> ver)
203-
_ -> Error(Nil)
204-
}
205-
})
206-
|> list.map(string.append("package:=", _))
207-
|> string.join("||")
198+
let filtered_packages =
199+
packages
200+
|> list.filter_map(fn(p) {
201+
case p.status {
202+
version.Found(ver) -> Ok("`" <> p.name <> "-" <> ver <> "`")
203+
_ -> Error(Nil)
204+
}
205+
})
206+
case filtered_packages {
207+
[] -> ""
208+
_ -> "package:=" <> "[" <> string.join(filtered_packages, ",") <> "]"
209+
}
208210
}
209211

210212
pub fn snippet(doc: String, search_input: String) -> String {

0 commit comments

Comments
 (0)