cli/multiselect: skip root.unmount, let renderer.destroy reclaim container (workaround for opentui-react Embind BindingError); bump v0.1.5

Author: knhn1004

Formula/agentlock.rb

@@ -1,10 +1,10 @@
 class Agentlock < Formula
   desc "Locally-hosted, open-source firewall for AI coding agents"
   homepage "https://openagentlock.github.io/OpenAgentLock"
-  url "https://registry.npmjs.org/@openagentlock/cli/-/cli-0.1.4.tgz"
-  sha256 "969b8730f29dae20f96326209f6c5ffcf881458027f733c8c735df257ba26b09"
+  url "https://registry.npmjs.org/@openagentlock/cli/-/cli-0.1.5.tgz"
+  sha256 "REPLACE_AFTER_NPM_PUBLISH"
   license "FSL-1.1-Apache-2.0"
-  version "0.1.4"
+  version "0.1.5"
 
   depends_on "bun"
 

cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openagentlock/cli",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "type": "module",
   "license": "SEE LICENSE IN LICENSE",
   "description": "OpenAgentLock CLI — a firewall for AI coding agents. Detects local agent harnesses (Claude Code, Codex CLI, Cursor, OpenCode, Cline, Gemini CLI, Continue, Copilot), gates risky tool calls via a Go control plane, anchors decisions in a Rust Merkle ledger.",

cli/src/tui/multiselect.tsx

@@ -197,31 +197,23 @@ export async function multiselect<T extends string>(
     let finished = false;
     const finish = (result: T[] | null): void => {
       // Guard against the keypress handler firing finish twice (Enter
-      // can produce both `return` and `enter` on some terminals; q/esc
-      // similarly). React's reconciler does not tolerate a second
-      // unmount commit on a torn-down container — symptom is
-      // BindingError "Expected null or instance of Node" inside
-      // commitBeforeMutationEffects → clearContainer.
+      // can produce both `return` and `enter` on some terminals).
       if (finished) return;
       finished = true;
-      // Resolve first so the caller's `await` settles before teardown.
-      // Defer unmount + destroy to the next microtask so React's
-      // current commit cycle (the keypress that triggered finish)
-      // finishes before the reconciler tries to mutate a container we
-      // are about to clear.
+
+      // opentui-react's reconciler crashes inside clearContainer when
+      // we try to formally unmount the React root — the WASM/Embind
+      // binding does a strict instanceof check that fails across the
+      // unmount→destroy boundary ("Expected null or instance of Node,
+      // got an instance of Node"). The reliable workaround is to skip
+      // root.unmount() entirely and let renderer.destroy() reclaim the
+      // container in one step. Process exit handles the React leak.
+      try {
+        renderer.destroy();
+      } catch {
+        // best-effort
+      }
       resolve(result);
-      queueMicrotask(() => {
-        try {
-          root.unmount();
-        } catch {
-          // best-effort
-        }
-        try {
-          renderer.destroy();
-        } catch {
-          // best-effort
-        }
-      });
     };
 
     root.render(