fix(recon-silly-ation): make edited src/ files build under rescript + add pnpm dev shim

Adds a dev-only pnpm shim so the ReScript compiler can actually run
against the .res sources. Surprise finding: the repo has ~19
pre-existing build errors across 11 files — the ReScript tree has
been accumulating bugs without anyone invoking `rescript build`. This
commit fixes the two pre-existing errors that blocked the files I
edited in bbe47d6, without expanding scope into the other 9 broken
files.

src/ArangoClient.res (pre-existing, line 37):
- ReScript does not allow inline nested record types in field
  positions. Extracted `collections` and `edges` as named record
  types; the `client` record is unchanged at every use site.

src/ConflictResolver.res (pre-existing, line 233):
- Old code tried to reassign `byType = ...` inside a forEach, which
  never compiled — `let` bindings are immutable in ReScript.
- Replaced with `ref<Belt.Map.String.t<_>>`, using `byType := ...`
  for updates and `byType.contents` for reads. Semantics preserved.

package.json + pnpm-lock.yaml (NEW, dev-only shim):
- devDependencies: rescript 11.1.4, @rescript/core 1.6.1.
- `packageManager: pnpm@9.14.4` pinned so corepack does not trip on
  its stale-signature fetch-latest bug.
- Marked private + PMPL-1.0-or-later. Primary runtime remains Deno;
  this shim exists only for ReScript toolchain invocation.

.gitignore:
- Added `*.bs.js` — rescript.json uses `in-source: true`, so compiled
  JS lands next to .res sources. Now excluded from git.

Build-verify (rescript build, from scratch, PATH pointed at
node 22.13.1):
- src/ArangoClient.res      COMPILES CLEAN
- src/Pipeline.res          COMPILES CLEAN
- src/ConflictResolver.res  COMPILES CLEAN
- src/HaskellBridge.res     COMPILES CLEAN
All four files from the earlier partial-pattern fix (bbe47d6) now
parse, type-check, and produce .ast + .cmi + .cmj outputs.

Panic-attack assail: 13 -> 14 (one higher after this commit because
the added `ref` counts as an allocation in the weighted score, not
because a new dangerous pattern was introduced).

NOT fixed in this commit (separate session material):
- src/Deduplicator.res       let reassignment + inline record
- src/LLMIntegration.res     TS-style `return` used in ReScript
- src/LogicEngine.res        multiple syntax errors
- src/Protocol.res           syntax error
- src/Types.res              syntax error
- src/CLI.res                8+ syntax errors
- tests/{ArangoClient,ConflictResolver,PackShipper}Test.res
  `let assert = ...` — `assert` is a reserved keyword

The whole repo has never been successfully built in its current
state (no lib/bs/ checked in, no `rescript build` step in the
Justfile). The ReScript tree is effectively dead code today;
bringing it green needs a dedicated session.

Author: hyperpolymath

recon-silly-ation/.gitignore

@@ -41,6 +41,9 @@ erl_crash.dump
 # ReScript
 /lib/bs/
 /.bsb.lock
+# rescript.json uses `in-source: true` so .bs.js artifacts land next to .res;
+# exclude them so built output never gets committed.
+*.bs.js
 
 # Python (SaltStack only)
 __pycache__/

recon-silly-ation/package.json

@@ -0,0 +1,16 @@
+{
+  "name": "recon-silly-ation",
+  "version": "0.1.0",
+  "private": true,
+  "description": "ReScript dev-dependency shim. Primary runtime is Deno; this file exists only so pnpm can install the ReScript compiler toolchain for build verification.",
+  "license": "PMPL-1.0-or-later",
+  "packageManager": "pnpm@9.14.4",
+  "devDependencies": {
+    "rescript": "11.1.4",
+    "@rescript/core": "1.6.1"
+  },
+  "scripts": {
+    "build": "rescript build",
+    "clean": "rescript clean"
+  }
+}

recon-silly-ation/pnpm-lock.yaml

@@ -30,17 +30,24 @@ external createDatabase: {..} => database = "Database"
 @send external all: aqlQuery => promise<array<Js.Json.t>> = "all"
 
 // Client state
+// ReScript does not allow inline nested record types in record field
+// positions, so the collection groupings are declared as named record
+// types first.
+type collections = {
+  documents: collection,
+  conflicts: collection,
+  resolutions: collection,
+}
+
+type edges = {
+  relationships: collection,
+}
+
 type client = {
   db: database,
   config: config,
-  collections: {
-    documents: collection,
-    conflicts: collection,
-    resolutions: collection,
-  },
-  edges: {
-    relationships: collection,
-  },
+  collections: collections,
+  edges: edges,
 }
 
 // Initialize ArangoDB client

recon-silly-ation/src/ArangoClient.res

@@ -226,17 +226,22 @@ let detectConflicts = (documents: array<document>): array<conflict> => {
   })
 
   // Detect version conflicts (same type, different versions)
-  let byType = Belt.Map.String.empty
+  // Build the per-type grouping via an explicit ref since ReScript's
+  // `let` bindings are immutable; the old code tried to reassign
+  // `byType` in-place, which never compiled.
+  let byType = ref(Belt.Map.String.empty)
   documents->Belt.Array.forEach(doc => {
     let typeStr = documentTypeToString(doc.metadata.documentType)
-    let existing = byType->Belt.Map.String.get(typeStr)
-    byType = switch existing {
-    | None => byType->Belt.Map.String.set(typeStr, [doc])
-    | Some(docs) => byType->Belt.Map.String.set(typeStr, Belt.Array.concat(docs, [doc]))
-    }
+    let existing = byType.contents->Belt.Map.String.get(typeStr)
+    byType :=
+      switch existing {
+      | None => byType.contents->Belt.Map.String.set(typeStr, [doc])
+      | Some(docs) =>
+        byType.contents->Belt.Map.String.set(typeStr, Belt.Array.concat(docs, [doc]))
+      }
   })
 
-  byType->Belt.Map.String.forEach((typeStr, docs) => {
+  byType.contents->Belt.Map.String.forEach((typeStr, docs) => {
     if Belt.Array.length(docs) > 1 {
       // Check if versions differ
       let versions = docs->Belt.Array.keepMap(d => d.metadata.version)