Repository navigation
Conversation
dune 3.25.0 (released 2026-11-12) gives an executable its public name in `dune describe workspace`, so the license check found none of the shipped executables; it now accepts either name. ocaml-lsp-server 1.27.0, which the lsp pin holds the dev setup to, doesn't build against stdune 3.25, so the dev setup keeps stdune below it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Kbf3gGPsF3fHZn9rBxA7Hm Signed-off-by: Christoph Knittel <ck@cca.io>
ChangelogNo changelog entry. Semantic Diff2 commits · Full page Work with dune 3.25d2996dc · cknitt + Claude · Nov 12, 2026, 20:44 UTC dune 3.25.0 (released 2026-11-12) gives an executable its public name in ✨ Move to lsp 1.28 instead of bounding stdunebb03256 · cknitt + Claude · Nov 12, 2026, 20:48 UTC · new lsp was pinned to 1.27.0 because 1.28 added snippet text edits, which the analysis CLI's edit printing didn't accept; it prints them now. lsp and jsonrpc move to 1.28.1, so the dev setup gets ocaml-lsp-server 1.28.1 instead of 1.27.0, and the stdune bound for it goes. |
lsp was pinned to 1.27.0 because 1.28 added snippet text edits, which the analysis CLI's edit printing didn't accept; it prints them now. lsp and jsonrpc move to 1.28.1, so the dev setup gets ocaml-lsp-server 1.28.1 instead of 1.27.0, and the stdune bound for it goes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Kbf3gGPsF3fHZn9rBxA7Hm Signed-off-by: Christoph Knittel <ck@cca.io>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #9295 +/- ##
==========================================
- Coverage 81.99% 81.99% -0.01%
==========================================
Files 455 455
Lines 65653 65654 +1
==========================================
Hits 53833 53833
- Misses 11820 11821 +1
🚀 New features to boost your workflow:
|
rescript
@rescript/belt
@rescript/darwin-arm64
@rescript/linux-arm64
@rescript/linux-x64
@rescript/runtime
@rescript/win32-x64
commit: |
dune 3.25.0 was released today (2026-11-12). The next fresh resolve of the OPAM environments, which any change of the opam files triggers, picks it up and breaks two CI jobs (seen on #9294, whose test commit changed an opam file):
dune describe workspacereports an executable's public name instead of its private one when it has one (dune #15582).scripts/generateThirdPartyLicenses.jsmatched the private names, found none of the shipped executables and failed. It now accepts either name for each executable. The new--split-public-namesflag would be cleaner, but dune 3.24 rejects it.ocaml-lsp-server, held at 1.27.0 because Fix the master build by requiring lsp below 1.28 #9234 pinnedlspto 1.27.0, and 1.27.0 doesn't build againststdune3.25 (Unix_env.getnow takes anEnv.Var.t).lspwas pinned because 1.28 added snippet text edits (SnippetTextEdit) to a text document edit, which the closed type annotation in the analysis CLI's edit printing (analysis/src/cli.ml) didn't accept. The CLI now prints them like the other two kinds, andlspandjsonrpcmove to 1.28.1; the otherLsp.Typeschanges in 1.28 (new optional fields ofCodeActionandCommand) don't affect analysis, which builds those withcreate. The dev setup then getsocaml-lsp-server1.28.1. The third-party license notices list the new versions; the license text is unchanged.Since this PR changes the opam files, its own CI resolves the environments from scratch with dune 3.25 and lsp 1.28.1.
The benchmarks job fails on this PR, and only on it: it builds the base branch's compiler in this PR's environment, and master's
analysis/src/cli.mldoesn't compile against lsp 1.28.1 (the incompatibility fixed here). The job isn't required, and once this is merged, the base builds again.🤖 Generated with Claude Code
https://claude.ai/code/session_01Kbf3gGPsF3fHZn9rBxA7Hm