@in-the-loop-labs/pair-review 2.4.0 → 2.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "Your AI-powered code review partner - Close the feedback loop with AI coding agents",
   "main": "src/server.js",
   "bin": {

package/plugin/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pair-review",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "pair-review app integration — Open PRs and local changes in the pair-review web UI, run server-side AI analysis, and address review feedback. Requires the pair-review MCP server.",
   "author": {
     "name": "in-the-loop-labs",

package/plugin-code-critic/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "code-critic",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "AI-powered code review analysis — Run three-level AI analysis and implement-review-fix loops directly in your coding agent. Works standalone, no server required.",
   "author": {
     "name": "in-the-loop-labs",

package/src/chat/acp-bridge.js

@@ -15,10 +15,19 @@ const { Writable, Readable } = require('stream');
 const logger = require('../utils/logger');
 const { version: pkgVersion } = require('../../package.json');
 
+// Lazy-load the ESM-only ACP SDK via dynamic import (cached after first call)
+let _acpModule = null;
+async function loadAcp() {
+  if (!_acpModule) {
+    _acpModule = await import('@agentclientprotocol/sdk');
+  }
+  return _acpModule;
+}
+
 // Default dependencies (overridable for testing)
 const defaults = {
   spawn,
-  acp: require('@agentclientprotocol/sdk'),
+  acp: null, // loaded lazily via loadAcp(); tests inject via _deps
   Writable,
   Readable,
 };
@@ -156,6 +165,11 @@ class AcpBridge extends EventEmitter {
    * @returns {Promise<void>}
    */
   async _initializeConnection(proc, deps) {
+    // Lazy-load ACP SDK if not injected (e.g. via _deps in tests)
+    if (!deps.acp) {
+      deps.acp = await loadAcp();
+    }
+
     const stream = deps.acp.ndJsonStream(
       deps.Writable.toWeb(proc.stdin),
       deps.Readable.toWeb(proc.stdout)