feat(acp): stabilize builds and e2e

Added pnpm-managed dependencies (including @types/react), taught the ACP e2e script to build core/adapters before launching the server, fixed worker path resolution, updated Next.js/tsconfig settings, allowed  in acp.config schema, and reintroduced proper tsconfig build settings with rootDir/paths overrides plus devDependencies for adapters so TypeScript can resolve core. After these fixes, npm run acp:e2e now succeeds.

Author: OhadAssulin

packages/acp-server/acp.config.schema.json

@@ -3,6 +3,7 @@
   "type": "object",
   "required": ["enabledAgents", "defaults"],
   "properties": {
+    "$schema": { "type": "string" },
     "enabledAgents": {
       "type": "array",
       "items": {

packages/acp-server/next-env.d.ts

@@ -1,5 +1,6 @@
 /// <reference types="next" />
 /// <reference types="next/image-types/global" />
+/// <reference path="./.next/types/routes.d.ts" />
 
 // NOTE: This file should not be edited
-// see https://nextjs.org/docs/basic-features/typescript for more information.
+// see https://nextjs.org/docs/app/api-reference/config/typescript for more information.

packages/acp-server/next.config.js

@@ -1,9 +1,7 @@
 /** @type {import('next').NextConfig} */
 const nextConfig = {
   output: 'standalone',
-  experimental: {
-    serverComponentsExternalPackages: ['@i-am-bee/acp-sdk'],
-  },
+  serverExternalPackages: ['@i-am-bee/acp-sdk'],
 };
 
 export default nextConfig;

packages/acp-server/package.json

@@ -13,13 +13,17 @@
     "next": "^15.0.0",
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
-    "@i-am-bee/acp-sdk": "^0.3.0",
+    "@i-am-bee/acp-sdk": "^0.0.6",
     "@headless-coder-sdk/core": "workspace:*",
     "@headless-coder-sdk/codex-adapter": "workspace:*",
     "@headless-coder-sdk/claude-adapter": "workspace:*",
     "@headless-coder-sdk/gemini-adapter": "workspace:*",
     "ajv": "^8.17.1"
   },
+  "devDependencies": {
+    "@types/react": "^18.3.3",
+    "@types/react-dom": "^18.3.0"
+  },
   "files": [
     "app",
     "client",

packages/acp-server/scripts/run-e2e.mjs

@@ -8,6 +8,12 @@ const SERVER_PORT = process.env.ACP_PORT ?? '8000';
 const SERVER_URL = process.env.ACP_BASE_URL ?? `http://localhost:${SERVER_PORT}`;
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const PACKAGE_ROOT = path.resolve(__dirname, '..');
+const WORKSPACE_DEPS = [
+  '@headless-coder-sdk/core',
+  '@headless-coder-sdk/codex-adapter',
+  '@headless-coder-sdk/claude-adapter',
+  '@headless-coder-sdk/gemini-adapter',
+];
 
 function log(...args) {
   console.log('[acp-e2e]', ...args);
@@ -22,6 +28,26 @@ function startServer() {
   });
 }
 
+async function runBuilds() {
+  for (const workspace of WORKSPACE_DEPS) {
+    log(`building ${workspace}...`);
+    await new Promise((resolve, reject) => {
+      const child = spawn(
+        'npm',
+        ['run', 'build', '--workspace', workspace],
+        {
+          stdio: 'inherit',
+        },
+      );
+      child.on('exit', code => {
+        if (code === 0) resolve();
+        else reject(new Error(`${workspace} build failed with code ${code}`));
+      });
+      child.on('error', reject);
+    });
+  }
+}
+
 async function waitForServer(timeoutMs = 30000) {
   const start = Date.now();
   while (Date.now() - start < timeoutMs) {
@@ -56,6 +82,7 @@ async function runClient() {
 }
 
 async function main() {
+  await runBuilds();
   const server = startServer();
   const cleanup = () => {
     if (!server.killed) {

packages/acp-server/tsconfig.json

@@ -1,15 +1,37 @@
 {
-  "extends": "next/tsconfig.json",
+  "extends": "../../tsconfig.base.json",
   "compilerOptions": {
+    "jsx": "preserve",
+    "module": "esnext",
+    "moduleResolution": "node",
+    "allowJs": true,
+    "esModuleInterop": true,
+    "resolveJsonModule": true,
+    "isolatedModules": true,
+    "noEmit": true,
+    "lib": [
+      "dom",
+      "dom.iterable",
+      "esnext"
+    ],
     "baseUrl": ".",
     "paths": {
-      "@/*": ["./src/*"]
-    }
+      "@/*": [
+        "./src/*"
+      ]
+    },
+    "incremental": true,
+    "plugins": [
+      {
+        "name": "next"
+      }
+    ]
   },
   "include": [
-    "next-env.d.ts",
     "**/*.ts",
-    "**/*.tsx"
+    "**/*.tsx",
+    "next-env.d.ts",
+    ".next/types/**/*.ts"
   ],
   "exclude": [
     "node_modules"

packages/claude-adapter/package.json

@@ -15,6 +15,7 @@
     "@headless-coder-sdk/core": "^0.12.0"
   },
   "devDependencies": {
-    "typescript": "^5.4.0"
+    "typescript": "^5.4.0",
+    "@headless-coder-sdk/core": "workspace:*"
   }
 }

packages/claude-adapter/tsconfig.build.json

@@ -2,9 +2,12 @@
   "extends": "../../tsconfig.base.json",
   "compilerOptions": {
     "outDir": "dist",
+    "rootDir": "src",
     "declaration": true,
     "emitDeclarationOnly": false,
-    "sourceMap": false
+    "sourceMap": false,
+    "baseUrl": "src",
+    "paths": {}
   },
   "include": ["src/**/*"],
   "exclude": ["node_modules", "dist", "tests"]

packages/codex-adapter/package.json

@@ -15,6 +15,7 @@
     "@openai/codex-sdk": "*"
   },
   "devDependencies": {
-    "typescript": "^5.4.0"
+    "typescript": "^5.4.0",
+    "@headless-coder-sdk/core": "workspace:*"
   }
 }

packages/codex-adapter/src/index.ts

@@ -4,6 +4,7 @@
  */
 
 import { fork, ChildProcess } from 'node:child_process';
+import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { now } from '@headless-coder-sdk/core';
 import type {
@@ -19,7 +20,7 @@ import type {
   Provider,
 } from '@headless-coder-sdk/core';
 
-const WORKER_PATH = fileURLToPath(new URL('./worker.js', import.meta.url));
+const WORKER_PATH = path.join(path.dirname(fileURLToPath(import.meta.url)), 'worker.js');
 const SOFT_KILL_DELAY_MS = 250;
 const HARD_KILL_DELAY_MS = 1500;
 const DONE = Symbol('stream-done');