amp-acp 0.6.0 → 0.6.2

package/dist/index.js

@@ -5104,6 +5104,7 @@ import { spawn } from "node:child_process";
 import fs from "node:fs";
 import { mkdir, readFile, rm, writeFile } from "node:fs/promises";
 import { createRequire } from "node:module";
+import os from "node:os";
 import path from "node:path";
 import { createInterface } from "node:readline";
 var __defProp2 = Object.defineProperty;
@@ -9119,18 +9120,42 @@ var PermissionSchema = exports_external2.object({
     });
   }
 });
-var MCPServerSchema = exports_external2.object({
+var MCPOAuthConfigSchema = exports_external2.object({
+  clientId: exports_external2.string().describe("OAuth client ID"),
+  clientSecret: exports_external2.string().describe("OAuth client secret").optional(),
+  authUrl: exports_external2.string().describe("OAuth authorization URL"),
+  tokenUrl: exports_external2.string().describe("OAuth token URL"),
+  scopes: exports_external2.array(exports_external2.string()).describe("OAuth scopes").optional(),
+  redirectUrl: exports_external2.string().describe("OAuth redirect URL").optional()
+}).describe("OAuth configuration for HTTP MCP servers");
+var MCPStdioServerSchema = exports_external2.object({
   command: exports_external2.string().describe("Command to run the MCP server"),
   args: exports_external2.array(exports_external2.string()).describe("Arguments for the MCP server command").optional(),
   env: exports_external2.record(exports_external2.string(), exports_external2.string()).describe("Environment variables for the server").optional(),
   disabled: exports_external2.boolean().describe("Whether the server is disabled").optional()
 });
+var MCPHttpServerSchema = exports_external2.object({
+  url: exports_external2.string().describe("URL of the HTTP MCP server"),
+  headers: exports_external2.record(exports_external2.string(), exports_external2.string()).describe("HTTP headers to send with requests").optional(),
+  transport: exports_external2.string().describe('Transport type (e.g., "sse")').optional(),
+  oauth: MCPOAuthConfigSchema.describe("OAuth configuration for authentication").optional(),
+  disabled: exports_external2.boolean().describe("Whether the server is disabled").optional()
+});
+var MCPServerSchema = exports_external2.union([MCPStdioServerSchema, MCPHttpServerSchema]);
 var MCPConfigSchema = exports_external2.record(exports_external2.string(), MCPServerSchema).describe("MCP server configurations keyed by server name");
+var VisibilitySchema = exports_external2.enum(["private", "public", "workspace", "group"]);
+var ThreadsNewOptionsSchema = exports_external2.object({
+  visibility: VisibilitySchema.optional().describe("Thread visibility")
+});
+var ThreadsMarkdownOptionsSchema = exports_external2.object({
+  threadId: exports_external2.string().describe("The thread ID to get markdown for")
+});
 var AmpOptionsSchema = exports_external2.object({
   cwd: exports_external2.string().describe("Current working directory").optional(),
-  mode: exports_external2.enum(["smart", "rush", "large"]).describe("Agent mode - controls the model, system prompt, and tool selection").default("smart").optional(),
+  mode: exports_external2.enum(["smart", "rush", "large", "deep"]).describe("Agent mode - controls the model, system prompt, and tool selection").default("smart").optional(),
   dangerouslyAllowAll: exports_external2.boolean().describe("Allow all tool usage without asking for permission").optional(),
-  visibility: exports_external2.enum(["private", "public", "workspace", "group"]).describe("Visibility level for new threads").default("workspace").optional(),
+  archive: exports_external2.boolean().describe("Archive the thread after execution completes").optional(),
+  visibility: VisibilitySchema.describe("Visibility level for new threads").default("workspace").optional(),
   settingsFile: exports_external2.string().describe("Settings file path").optional(),
   logLevel: exports_external2.enum(["debug", "info", "warn", "error", "audit"]).describe("Log level").optional(),
   logFile: exports_external2.string().describe("Log file path").optional(),
@@ -9138,7 +9163,11 @@ var AmpOptionsSchema = exports_external2.object({
   env: exports_external2.record(exports_external2.string(), exports_external2.string()).describe("Additional environment variables").optional(),
   continue: exports_external2.union([exports_external2.boolean(), exports_external2.string()]).describe("Continue the most recent thread (true) or a specific thread by ID (string)").optional(),
   toolbox: exports_external2.string().describe("Folder path with toolbox scripts").optional(),
-  permissions: exports_external2.array(PermissionSchema).describe("Permission rules for tool usage").optional()
+  skills: exports_external2.string().describe("Folder path with custom skills").optional(),
+  permissions: exports_external2.array(PermissionSchema).describe("Permission rules for tool usage").optional(),
+  labels: exports_external2.array(exports_external2.string().regex(/^[a-zA-Z0-9][a-zA-Z0-9-]*$/, "Labels must start with alphanumeric and contain only alphanumeric characters and hyphens").min(1, "Label must be at least 1 character").max(32, "Label must be at most 32 characters")).max(20, "Maximum 20 labels allowed").describe("Labels to add to the thread (repeatable CLI --label)").optional(),
+  enabledTools: exports_external2.array(exports_external2.string()).describe('Array of tool name patterns to enable. Supports glob patterns (e.g., "mcp__metabase__*"). If not provided, all tools are enabled. If provided, only matching tools are enabled.').optional(),
+  systemPrompt: exports_external2.string().describe("Custom system prompt text to append to the base system prompt for integration-specific customizations").optional()
 }).strict();
 var isWindows = process.platform === "win32";
 async function* execute(options) {
@@ -9172,22 +9201,9 @@ async function* execute(options) {
     }
     yield* processOutputStream(ampProcess.stdout);
     const { exitCode, stderr, signal: processSignal } = await waitForProcess(ampProcess, signal);
-    if (exitCode === null) {
-      if (signal?.aborted) {
-        throw new Error("Amp CLI process was aborted");
-      } else if (processSignal) {
-        throw new Error(`Amp CLI process was killed by signal ${processSignal}`);
-      } else {
-        throw new Error("Amp CLI process terminated unexpectedly");
-      }
-    } else if (exitCode !== 0) {
-      const errorDetails = stderr ? `: ${stderr}` : "";
-      throw new Error(`Amp CLI process exited with code ${exitCode}${errorDetails}`);
-    }
+    throwIfProcessFailed(exitCode, stderr, processSignal, signal);
   } catch (error) {
-    if (!ampProcess.killed) {
-      ampProcess.kill(isWindows ? "SIGKILL" : "SIGTERM");
-    }
+    killProcess(ampProcess);
     throw error;
   } finally {
     await cleanupTempFile();
@@ -9198,7 +9214,14 @@ function generateSessionId() {
   return `sdk-${Date.now()}-${randomHex}`;
 }
 async function buildSettingsFile(options, sessionId) {
-  if (!options.permissions && !options.mode) {
+  const settingsFields = [
+    options.permissions,
+    options.mode,
+    options.skills,
+    options.enabledTools,
+    options.systemPrompt
+  ];
+  if (!settingsFields.some(Boolean)) {
     return {
       settingsFilePath: null,
       cleanupTempFile: async () => {}
@@ -9218,11 +9241,22 @@ async function buildSettingsFile(options, sessionId) {
       }
     }
   }
-  mergedSettings["amp.permissions"] = options.permissions;
-  if (options.mode === "large") {
-    mergedSettings["amp.internal.visibleModes"] = (mergedSettings["amp.internal.visibleModes"] ?? []).concat("large");
+  if (options.permissions) {
+    mergedSettings["amp.permissions"] = options.permissions;
+  }
+  if (options.enabledTools) {
+    mergedSettings["amp.tools.enable"] = options.enabledTools;
+  }
+  if (options.systemPrompt) {
+    mergedSettings["amp.systemPrompt"] = options.systemPrompt;
+  }
+  if (options.skills) {
+    mergedSettings["amp.skills.path"] = options.skills;
   }
   await mkdir(tempDir, { recursive: true });
+  if (process.env.AMP_DEBUG) {
+    console.log("[SDK] Writing settings file:", tempSettingsPath, mergedSettings);
+  }
   await writeFile(tempSettingsPath, JSON.stringify(mergedSettings, null, 2), "utf-8");
   return {
     settingsFilePath: tempSettingsPath,
@@ -9243,7 +9277,7 @@ function buildEnvironmentVariables(options) {
   if (options.env) {
     Object.assign(env, options.env);
   }
-  env.AMP_SDK_VERSION = "0.1.0-20251214200908-g3251f72";
+  env.AMP_SDK_VERSION = "0.1.0-20260220044036-g88868e6";
   return env;
 }
 function spawnAmpCli(args, options) {
@@ -9259,6 +9293,21 @@ function spawnAmpCli(args, options) {
   return childProcess;
 }
 function findAmpCommand() {
+  const localAmpCommand = resolveLocalAmpPackageCommand();
+  if (localAmpCommand) {
+    return localAmpCommand;
+  }
+  const homeAmpCommand = resolveLocalAmpCommand();
+  if (homeAmpCommand) {
+    return homeAmpCommand;
+  }
+  const pathAmpCommand = resolveAmpFromPath();
+  if (pathAmpCommand) {
+    return pathAmpCommand;
+  }
+  throw new Error("Could not find local @sourcegraph/amp package or amp binary. Make sure it is installed.");
+}
+function resolveLocalAmpPackageCommand() {
   try {
     const require2 = createRequire(import.meta.url);
     const pkgJsonPath = require2.resolve("@sourcegraph/amp/package.json");
@@ -9276,8 +9325,41 @@ function findAmpCommand() {
     if (error instanceof Error && error.message.includes("Local @sourcegraph/amp")) {
       throw error;
     }
-    throw new Error("Could not find local @sourcegraph/amp package. Make sure it is installed.");
+    return null;
+  }
+}
+function resolveLocalAmpCommand() {
+  const candidates = isWindows ? ["amp.exe", "amp.cmd", "amp"] : ["amp"];
+  const ampBinDir = path.join(os.homedir(), ".amp", "bin");
+  for (const candidate of candidates) {
+    const candidatePath = path.join(ampBinDir, candidate);
+    if (fs.existsSync(candidatePath)) {
+      return {
+        command: candidatePath,
+        args: []
+      };
+    }
+  }
+  return null;
+}
+function resolveAmpFromPath() {
+  const pathEntries = process.env.PATH?.split(path.delimiter) ?? [];
+  const candidates = isWindows ? ["amp.exe", "amp.cmd", "amp"] : ["amp"];
+  for (const pathEntry of pathEntries) {
+    if (!pathEntry) {
+      continue;
+    }
+    for (const candidate of candidates) {
+      const candidatePath = path.join(pathEntry, candidate);
+      if (fs.existsSync(candidatePath)) {
+        return {
+          command: candidatePath,
+          args: []
+        };
+      }
+    }
   }
+  return null;
 }
 function buildCliArgs(options) {
   const args = [];
@@ -9290,6 +9372,9 @@ function buildCliArgs(options) {
   if (options.dangerouslyAllowAll) {
     args.push("--dangerously-allow-all");
   }
+  if (options.archive) {
+    args.push("--archive");
+  }
   if (options.visibility) {
     args.push("--visibility", options.visibility);
   }
@@ -9309,8 +9394,32 @@ function buildCliArgs(options) {
   if (options.mode) {
     args.push("--mode", options.mode);
   }
+  if (options.labels) {
+    for (const label of options.labels) {
+      args.push("--label", label);
+    }
+  }
   return args;
 }
+function throwIfProcessFailed(exitCode, stderr, processSignal, signal) {
+  if (exitCode === null) {
+    if (signal?.aborted) {
+      throw new Error("Amp CLI process was aborted");
+    } else if (processSignal) {
+      throw new Error(`Amp CLI process was killed by signal ${processSignal}`);
+    } else {
+      throw new Error("Amp CLI process terminated unexpectedly");
+    }
+  } else if (exitCode !== 0) {
+    const errorDetails = stderr ? `: ${stderr.trim()}` : "";
+    throw new Error(`Amp CLI process exited with code ${exitCode}${errorDetails}`);
+  }
+}
+function killProcess(proc) {
+  if (!proc.killed) {
+    proc.kill(isWindows ? "SIGKILL" : "SIGTERM");
+  }
+}
 async function* processOutputStream(stdout) {
   if (!stdout) {
     throw new Error("No stdout stream available from Amp CLI process");
@@ -9535,7 +9644,7 @@ function safeJson(x) {
 // package.json
 var package_default = {
   name: "amp-acp",
-  version: "0.6.0",
+  version: "0.6.2",
   private: false,
   type: "module",
   main: "dist/index.js",
@@ -9553,17 +9662,17 @@ var package_default = {
   },
   scripts: {
     build: "bun build src/index.ts --target=node --outdir=dist --entry-naming=[dir]/[name].js",
-    "build:binary": "bun build src/index.ts --compile --outfile dist/amp-acp",
+    "build:binary": "bun build src/index.ts --target=node --outdir=dist --entry-naming=[dir]/[name].js && bun build dist/index.js --compile --outfile dist/amp-acp",
     start: "bun dist/index.js",
     lint: "tsc --noEmit",
     test: "bun test src/",
-    "test:binary": "bun build src/index.ts --compile --outfile dist/amp-acp-test && bun test test/binary.test.ts",
+    "test:binary": "bun build src/index.ts --target=node --outdir=dist --entry-naming=[dir]/[name].js && bun build dist/index.js --compile --outfile dist/amp-acp-test && bun test test/binary.test.ts",
     "test:all": "bun run test && bun run test:binary"
   },
   dependencies: {
     "@agentclientprotocol/sdk": "0.4.8",
     "@sourcegraph/amp": "^0.0.1765685598-g5365e6",
-    "@sourcegraph/amp-sdk": "^0.1.0-20251214200908-g3251f72"
+    "@sourcegraph/amp-sdk": "0.1.0-20260220044036-g88868e6"
   },
   devDependencies: {
     "@types/bun": "^1.2.5",
@@ -9683,7 +9792,8 @@ ${chunk.resource.text}
       }
     }
     const options = {
-      cwd: s.cwd
+      cwd: s.cwd,
+      env: { TERM: "dumb" }
     };
     if (s.mode === "bypass") {
       options.dangerouslyAllowAll = true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "amp-acp",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "private": false,
   "type": "module",
   "main": "dist/index.js",
@@ -18,17 +18,17 @@
   },
   "scripts": {
     "build": "bun build src/index.ts --target=node --outdir=dist --entry-naming=[dir]/[name].js",
-    "build:binary": "bun build src/index.ts --compile --outfile dist/amp-acp",
+    "build:binary": "bun build src/index.ts --target=node --outdir=dist --entry-naming=[dir]/[name].js && bun build dist/index.js --compile --outfile dist/amp-acp",
     "start": "bun dist/index.js",
     "lint": "tsc --noEmit",
     "test": "bun test src/",
-    "test:binary": "bun build src/index.ts --compile --outfile dist/amp-acp-test && bun test test/binary.test.ts",
+    "test:binary": "bun build src/index.ts --target=node --outdir=dist --entry-naming=[dir]/[name].js && bun build dist/index.js --compile --outfile dist/amp-acp-test && bun test test/binary.test.ts",
     "test:all": "bun run test && bun run test:binary"
   },
   "dependencies": {
     "@agentclientprotocol/sdk": "0.4.8",
     "@sourcegraph/amp": "^0.0.1765685598-g5365e6",
-    "@sourcegraph/amp-sdk": "^0.1.0-20251214200908-g3251f72"
+    "@sourcegraph/amp-sdk": "0.1.0-20260220044036-g88868e6"
   },
   "devDependencies": {
     "@types/bun": "^1.2.5",