Make reanalyze a test dependency

reanalyze 2.27.0 is on opam now, so it becomes a pinned test dependency
in rescript.opam like ocamlformat and ounit2 (not on Windows, like
js_of_ocaml: the check runs on Linux). opam install . --deps-only
--with-test installs it for developers, and CI builds it into the cached
switch, so the dead-code step just runs make dead-code instead of
building reanalyze from its tag.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Kbf3gLDo8mBJKpNE25WAwT
Signed-off-by: Christoph Knittel <ck@cca.io>

Author: cknitt

.github/workflows/ci.yml

@@ -276,18 +276,11 @@ jobs:
         run: opam exec -- make playground
 
       # Runs after the playground build, whose browser-profile tree it
-      # analyzes together with the native one. reanalyze is built from its
-      # release tag in the existing switch: an opam install would recompile
-      # the restored static switch, whose musl system package opam misses.
+      # analyzes together with the native one. reanalyze is a test
+      # dependency, installed with the switch.
       - name: Check for dead code
         if: matrix.build_playground
-        shell: bash
-        run: |
-          git -c advice.detachedHead=false clone --quiet --depth 1 --branch v2.27.0 https://github.com/rescript-lang/reanalyze.git "$RUNNER_TEMP/reanalyze"
-          # opam exec finds the repository's local switch from the current
-          # directory, so it changes directory itself.
-          opam exec -- sh -c 'cd "$RUNNER_TEMP/reanalyze" && dune build -p reanalyze @install'
-          REANALYZE="$RUNNER_TEMP/reanalyze/_build/install/default/bin/reanalyze.exe" opam exec -- make dead-code
+        run: opam exec -- make dead-code
 
       - name: Test playground compiler
         if: matrix.build_playground

dune-project

@@ -52,7 +52,13 @@
    (and
     (<> :os "win32")
     :with-test
-    (= 6.5.0)))))
+    (= 6.5.0)))
+  ; make dead-code
+  (reanalyze
+   (and
+    (<> :os "win32")
+    :with-test
+    (= 2.27.0)))))
 
 (package
  (name analysis)

rescript.opam

@@ -24,6 +24,7 @@ depends: [
   "ocaml-lsp-server" {with-dev-setup & >= "1.23.0"}
   "js_of_ocaml" {os != "win32" & with-test & = "6.5.0"}
   "wasm_of_ocaml-compiler" {os != "win32" & with-test & = "6.5.0"}
+  "reanalyze" {os != "win32" & with-test & = "2.27.0"}
 ]
 build: [
   ["dune" "subst"] {dev}

scripts/dead_code.sh

@@ -4,8 +4,8 @@
 # anything.
 #
 # Usage: REANALYZE=/path/to/reanalyze.exe ./scripts/dead_code.sh
-# (defaults to reanalyze.exe on PATH; needs reanalyze 2.27.0 or later, which
-# analyzes several build trees together: opam install reanalyze)
+# (defaults to reanalyze.exe on PATH. reanalyze is a test dependency in
+# rescript.opam, installed by opam install . --deps-only --with-test.)
 
 set -euo pipefail
 cd "$(dirname "$0")/.."