Skip to content

Commit 6518260

Browse files
committed
fix: fall back to lib/bs directory for deps without .sourcedirs.json
Prebuilt dependencies like @rescript/react have .cmt files in lib/bs/ but no .sourcedirs.json. The mtime-based detection introduced in f178c45 returned None for these, causing the analysis to skip the dependency entirely and breaking completions for ReactDOM, ReactEvent, JsxEvent, etc. on macOS CI.
1 parent 59b3b3f commit 6518260

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

analysis/src/BuildSystem.ml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ let getLibBs path =
5050
if mtime (sourcedirs lsp) > mtime (sourcedirs bs) then Some lsp else Some bs
5151
| true, false -> Some bs
5252
| false, true -> Some lsp
53-
| false, false -> None
53+
| false, false -> Files.ifExists bs
5454

5555
let getStdlib base =
5656
match getRuntimeDir base with

0 commit comments

Comments
 (0)