@peakflo/agent-harness 0.2.9-rc.3 → 0.2.9-rc.5

package/dist/vendor-backends.d.ts

@@ -47,31 +47,23 @@ export declare const CLAUDE_SDK_PACKAGE = "@anthropic-ai/claude-agent-sdk";
  * runner. Fixed sentences, like every other reason in the report.
  */
 export declare const NOT_RUNNABLE_REASONS: {
-    readonly claudeSdkMissing: "The Claude Agent SDK is not installed. Run `npm install -g @anthropic-ai/claude-agent-sdk` on this machine.";
+    readonly claudeSdkMissing: "The Claude Agent SDK that ships with this runner could not be loaded. Reinstall the runner.";
     readonly sandbox: "A cloud sandbox runs Pi only.";
 };
 /**
- * Directories to look for the SDK in, besides this package's own
- * `node_modules`.
+ * Where the SDK is, or null.
  *
- * WHY IT IS NOT SIMPLY A DEPENDENCY. In 20x the SDK ships inside the app. Here
- * that would add its platform package — about 190 MB, a copy of the Claude
- * binary — to EVERY install of the runner, the cloud sandbox included, for a
- * harness most installs never use. The adapter already points the SDK at the
- * `claude` the user installed, so only the SDK's own JavaScript is needed, and
- * a machine that wants Claude Code installs it once, globally.
- */
-export declare function claudeSdkSearchRoots(env?: NodeJS.ProcessEnv, execPath?: string): string[];
-/**
- * Where the SDK can be loaded from, or null. Synchronous and cheap (a few
- * file reads), so it can be asked on every capability report.
+ * THE SDK IS A DEPENDENCY OF THIS PACKAGE, as it is of the 20x app: installing
+ * the runner installs it, and nothing else has to be done on the machine. So
+ * it is looked for in one place, this package's own dependency tree.
+ *
+ * Null is still possible and still handled, because "declared" is not
+ * "present": an install that was interrupted, pruned, or made with
+ * dependencies left out has no SDK. Claude Code is then reported as not
+ * runnable, with a reason, instead of being routed a turn it must fail.
  *
- * `ownTree` is this package's own resolution, replaced in tests — this
- * repository has the SDK as a dev dependency, so "not installed" could not be
- * tested otherwise.
+ * `ownTree` is replaced in tests, which could not otherwise see "missing".
  */
-export declare function resolveClaudeSdk(roots?: readonly string[], ownTree?: () => string): string | null;
-/** Import the SDK from wherever `resolveClaudeSdk` finds it. Throws when absent. */
-export declare function importClaudeSdk(): Promise<unknown>;
+export declare function resolveClaudeSdk(ownTree?: () => string): string | null;
 /** Whether this runner can build an adapter for a vendor backend right now. */
 export declare function vendorBackendBlocker(backend: string, runtime: string): string | null;

package/dist/vendor-backends.js

@@ -25,14 +25,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.NOT_RUNNABLE_REASONS = exports.CLAUDE_SDK_PACKAGE = exports.VENDOR_BACKENDS = void 0;
 exports.isVendorBackend = isVendorBackend;
 exports.vendorModelId = vendorModelId;
-exports.claudeSdkSearchRoots = claudeSdkSearchRoots;
 exports.resolveClaudeSdk = resolveClaudeSdk;
-exports.importClaudeSdk = importClaudeSdk;
 exports.vendorBackendBlocker = vendorBackendBlocker;
-const node_fs_1 = require("node:fs");
 const node_module_1 = require("node:module");
-const node_path_1 = require("node:path");
-const node_url_1 = require("node:url");
 exports.VENDOR_BACKENDS = ['codex', 'claude-code', 'cursor'];
 function isVendorBackend(name) {
     return exports.VENDOR_BACKENDS.includes(name);
@@ -65,94 +60,30 @@ exports.CLAUDE_SDK_PACKAGE = '@anthropic-ai/claude-agent-sdk';
  * runner. Fixed sentences, like every other reason in the report.
  */
 exports.NOT_RUNNABLE_REASONS = {
-    claudeSdkMissing: 'The Claude Agent SDK is not installed. Run `npm install -g @anthropic-ai/claude-agent-sdk` on this machine.',
+    claudeSdkMissing: 'The Claude Agent SDK that ships with this runner could not be loaded. Reinstall the runner.',
     sandbox: 'A cloud sandbox runs Pi only.'
 };
 /**
- * Directories to look for the SDK in, besides this package's own
- * `node_modules`.
+ * Where the SDK is, or null.
  *
- * WHY IT IS NOT SIMPLY A DEPENDENCY. In 20x the SDK ships inside the app. Here
- * that would add its platform package — about 190 MB, a copy of the Claude
- * binary — to EVERY install of the runner, the cloud sandbox included, for a
- * harness most installs never use. The adapter already points the SDK at the
- * `claude` the user installed, so only the SDK's own JavaScript is needed, and
- * a machine that wants Claude Code installs it once, globally.
- */
-function claudeSdkSearchRoots(env = process.env, execPath = process.execPath) {
-    const roots = [
-        env.PF_CLAUDE_AGENT_SDK_DIR,
-        // `npm install -g` beside the running node: <prefix>/lib/node_modules
-        (0, node_path_1.join)((0, node_path_1.dirname)((0, node_path_1.dirname)(execPath)), 'lib', 'node_modules'),
-        // Windows: <prefix>\node_modules
-        (0, node_path_1.join)((0, node_path_1.dirname)(execPath), 'node_modules'),
-        '/opt/homebrew/lib/node_modules',
-        '/usr/local/lib/node_modules'
-    ];
-    return [...new Set(roots.filter((root) => Boolean(root)))];
-}
-/** The file a package directory loads as, read from its own package.json. */
-function packageEntry(directory) {
-    try {
-        const manifest = JSON.parse((0, node_fs_1.readFileSync)((0, node_path_1.join)(directory, 'package.json'), 'utf8'));
-        const root = manifest.exports && typeof manifest.exports === 'object'
-            ? (manifest.exports['.'] ??
-                manifest.exports)
-            : manifest.exports;
-        const fromExports = typeof root === 'string'
-            ? root
-            : root && typeof root === 'object'
-                ? (root.import ??
-                    root.default)
-                : undefined;
-        const entry = [
-            fromExports,
-            manifest.module,
-            manifest.main,
-            'index.js'
-        ].find((candidate) => typeof candidate === 'string');
-        const file = entry ? (0, node_path_1.join)(directory, entry) : null;
-        return file && (0, node_fs_1.existsSync)(file) ? file : null;
-    }
-    catch {
-        return null;
-    }
-}
-/**
- * Where the SDK can be loaded from, or null. Synchronous and cheap (a few
- * file reads), so it can be asked on every capability report.
+ * THE SDK IS A DEPENDENCY OF THIS PACKAGE, as it is of the 20x app: installing
+ * the runner installs it, and nothing else has to be done on the machine. So
+ * it is looked for in one place, this package's own dependency tree.
+ *
+ * Null is still possible and still handled, because "declared" is not
+ * "present": an install that was interrupted, pruned, or made with
+ * dependencies left out has no SDK. Claude Code is then reported as not
+ * runnable, with a reason, instead of being routed a turn it must fail.
  *
- * `ownTree` is this package's own resolution, replaced in tests — this
- * repository has the SDK as a dev dependency, so "not installed" could not be
- * tested otherwise.
+ * `ownTree` is replaced in tests, which could not otherwise see "missing".
  */
-function resolveClaudeSdk(roots = claudeSdkSearchRoots(), ownTree = () => (0, node_module_1.createRequire)(__filename).resolve(exports.CLAUDE_SDK_PACKAGE)) {
+function resolveClaudeSdk(ownTree = () => (0, node_module_1.createRequire)(__filename).resolve(exports.CLAUDE_SDK_PACKAGE)) {
     try {
         return ownTree();
     }
     catch {
-        // Not in this package's own tree.
-    }
-    for (const root of roots) {
-        const entry = packageEntry((0, node_path_1.join)(root, exports.CLAUDE_SDK_PACKAGE));
-        if (entry)
-            return entry;
-    }
-    return null;
-}
-/** Import the SDK from wherever `resolveClaudeSdk` finds it. Throws when absent. */
-async function importClaudeSdk() {
-    const resolved = resolveClaudeSdk();
-    if (!resolved) {
-        throw new Error(exports.NOT_RUNNABLE_REASONS.claudeSdkMissing);
+        return null;
     }
-    /*
-     * `new Function` keeps this a REAL dynamic import in the compiled CommonJS
-     * output. Written as `import(x)`, TypeScript turns it into `require(x)`,
-     * and the SDK is an ES module that `require` cannot load on every Node.
-     */
-    const dynamicImport = new Function('specifier', 'return import(specifier)');
-    return dynamicImport((0, node_url_1.pathToFileURL)(resolved).href);
 }
 /** Whether this runner can build an adapter for a vendor backend right now. */
 function vendorBackendBlocker(backend, runtime) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@peakflo/agent-harness",
-  "version": "0.2.9-rc.3",
+  "version": "0.2.9-rc.5",
   "description": "Headless Peakflo coding-agent harness. One codebase, three hosting modes: a Daytona cloud sandbox, a customer VM, and the 20x desktop.",
   "license": "MIT",
   "type": "commonjs",
@@ -29,7 +29,6 @@
     "test": "jest"
   },
   "devDependencies": {
-    "@anthropic-ai/claude-agent-sdk": "^0.3.195",
     "@opencode-ai/sdk": "^1.15.13",
     "@types/node": "^22.10.0",
     "tsx": "^4.19.3",
@@ -37,6 +36,7 @@
     "undici": "^7.21.0"
   },
   "dependencies": {
+    "@anthropic-ai/claude-agent-sdk": "0.3.258",
     "@modelcontextprotocol/client": "2.0.0",
     "@peculiar/x509": "^2.1.0",
     "reflect-metadata": "^0.2.2"