@love-moon/conductor-cli 0.11.1 → 0.12.0

package/CHANGELOG.md

@@ -1,5 +1,69 @@
 # @love-moon/conductor-cli
 
+## 0.12.0
+
+### Minor Changes
+
+- b749180: Add `conductor remote cp` for copying a single file to or from another daemon's
+  host, and fold the existing `remote-exec` command into a `conductor remote`
+  umbrella alongside it.
+
+  - `conductor remote exec ...` replaces `conductor remote-exec ...`. Behaviour is
+    unchanged; only the command path moved.
+  - `conductor remote cp <src> <dst>` follows scp: exactly one side is written as
+    `<daemon>:<path>`, and that decides the direction. Contents are verified end
+    to end with SHA-256 and written via a temporary file plus rename, so an
+    interrupted copy never truncates an existing file.
+  - `conductor remote cp -r` copies a directory, matching scp's destination
+    rules. It packs with `tar`, moves the one tarball through the ordinary
+    transfer and unpacks on the other side, so permissions and symlinks survive
+    and the wire protocol is unchanged. Needs `tar` and `remote_exec` on the
+    target; the limit applies to the compressed size.
+  - New `remote_file` daemon capability, declinable with `remote_file: false` in
+    the config or `CONDUCTOR_REMOTE_FILE=0`, kept separate from `remote_exec` so
+    version skew fails with a clean 409 instead of a timeout.
+  - Transfers are chunked at 32 MiB with `Content-Range` and resume automatically
+    after a network blip, from the server's offset going up and from the local
+    partial file coming down. Chunking is also what keeps the server safe at the
+    raised limit: no single request is large or long-lived.
+  - The per-file limit is now 1 GiB, bounded by a global staged-bytes budget
+    (8 GiB), a per-user budget (2 GiB), a free-disk floor, and reservation by
+    declared size at creation.
+  - Bytes travel over HTTP through the backend rather than the agent WebSocket.
+    The socket only carries a small control message. See RFC 0037.
+
+  Deploying this needs the updated `web/nginx_conf`: the file-transfer routes set
+  `client_max_body_size` to 64m — a per-chunk cap, not a per-file one — and turn
+  off `proxy_request_buffering`.
+
+### Patch Changes
+
+- @love-moon/conductor-sdk@0.12.0
+- @love-moon/ai-sdk@0.12.0
+
+## 0.11.2
+
+### Patch Changes
+
+- 938efe7: Fix "New task from this" when the successor runs on a different daemon.
+
+  Picking another daemon in the dialog created the successor row and then killed
+  it one second later with `new task failed: Could not resolve resume cwd`. The
+  server intentionally drops the source machine's paths for a cross-daemon fork,
+  and the target daemon had nothing left to resolve: no launch-config `cwd`, no
+  worktree, no local project path (the default project has none, and a project
+  bound to another daemon is refused by design), and no source session in its own
+  session store.
+
+  Fork modes now fall back to a fresh `<workspace>/<date>/<run>` directory — the
+  same fallback `create_task` already uses — so the successor starts like a
+  brand-new task on that machine and picks up the conversation through the
+  resume-context URL. In-place restarts still fail loudly, because they must reuse
+  the original working directory.
+
+  - @love-moon/conductor-sdk@0.11.2
+  - @love-moon/ai-sdk@0.11.2
+
 ## 0.11.1
 
 ### Patch Changes

package/bin/conductor-config.js

@@ -387,6 +387,10 @@ async function main() {
     "# Env: CONDUCTOR_REMOTE_EXEC",
     "# remote_exec: true",
     "",
+    "# Accept `conductor remote cp` file transfers for this host.",
+    "# Env: CONDUCTOR_REMOTE_FILE",
+    "# remote_file: true",
+    "",
     "# Launch each Fire process inside a detached tmux session so it survives a",
     "# daemon restart. Requires tmux on PATH. Env: CONDUCTOR_FIRE_TMUX_MODE",
     "# fire_tmux_mode: false",
@@ -415,8 +419,10 @@ async function main() {
     "# Optional features",
     "# ---------------------------------------------------------------------------",
     "",
-    "# Extra Codex auth.json files to switch between from the app's account",
-    "# picker. The file name (without .json) becomes the account name.",
+    "# Optional: extra Codex auth.json files to switch between from the app's",
+    "# account picker (shown only when more than one is listed). The file name",
+    "# (without .json) becomes the account name. Without this, quota comes from",
+    "# $CODEX_HOME/auth.json (default ~/.codex/auth.json).",
     "# ai_manager:",
     "#   codex:",
     "#     auth_json:",

package/bin/conductor-remote.js

@@ -0,0 +1,93 @@
+#!/usr/bin/env node
+
+/**
+ * conductor remote — act on another daemon's host.
+ *
+ *   conductor remote exec --target ubuntu -- bash -lc "pnpm build 2>&1 | tail -20"
+ *   conductor remote cp ./build.tar.gz ubuntu:/srv/app/build.tar.gz
+ *   conductor remote cp ubuntu:/var/log/conductor.log ./conductor.log
+ *
+ * Verb dispatch is hand-rolled rather than delegated to yargs on purpose:
+ * `exec` has to hand an arbitrary remote argv through untouched, and yargs'
+ * `strict()` would try to interpret the remote command's own flags.
+ */
+
+import path from "node:path";
+import process from "node:process";
+import { fileURLToPath } from "node:url";
+
+import { EXIT } from "../src/remote/client.js";
+import { runRemoteExec, showHelp as showExecHelp } from "../src/remote/exec.js";
+import { runRemoteCp, showHelp as showCpHelp } from "../src/remote/cp.js";
+
+export { EXIT };
+
+const VERBS = new Map([
+  ["exec", runRemoteExec],
+  ["cp", runRemoteCp],
+]);
+
+const isMainModule = (() => {
+  const currentFile = fileURLToPath(import.meta.url);
+  const entryFile = process.argv[1] ? path.resolve(process.argv[1]) : "";
+  return entryFile === currentFile;
+})();
+
+export function showHelp(consoleImpl = console) {
+  consoleImpl.log(`conductor remote - act on another daemon's host
+
+Usage:
+  conductor remote <verb> [options]
+
+Verbs:
+  exec   Run a command on another daemon's host
+  cp     Copy a file to or from another daemon's host
+
+Options:
+  -h, --help   Show this help
+
+Examples:
+  conductor remote exec --target ubuntu --workspace /srv/app -- git log --oneline -5
+  conductor remote cp ./build.tar.gz ubuntu:/srv/app/build.tar.gz
+  conductor remote cp ubuntu:/var/log/conductor.log ./conductor.log
+
+For verb-specific help:
+  conductor remote exec --help
+  conductor remote cp --help
+`);
+}
+
+export async function runRemote(argv, deps = {}) {
+  const consoleImpl = deps.console || console;
+
+  if (argv.length === 0) {
+    showHelp(consoleImpl);
+    return EXIT.CLI_ERROR;
+  }
+
+  const verb = argv[0];
+  if (verb === "--help" || verb === "-h") {
+    showHelp(consoleImpl);
+    return EXIT.OK;
+  }
+
+  const handler = VERBS.get(verb);
+  if (!handler) {
+    // `--help` on a verb belongs to the verb, but a bare flag in the verb slot
+    // is a usage mistake worth naming precisely.
+    consoleImpl.error(`Error: unknown verb '${verb}'`);
+    consoleImpl.error(`Valid verbs: ${[...VERBS.keys()].join(", ")}`);
+    showHelp(consoleImpl);
+    return EXIT.CLI_ERROR;
+  }
+
+  return handler(argv.slice(1), deps);
+}
+
+export { showExecHelp, showCpHelp };
+
+if (isMainModule) {
+  // `process.exitCode` rather than `process.exit()`: writes to a pipe are async,
+  // and exiting outright truncates them. Let the loop drain and end naturally.
+  process.exitCode = await runRemote(process.argv.slice(2));
+}

package/bin/conductor-send-file.js

@@ -182,34 +182,56 @@ export async function sendFileToTask(options) {
   body.set("file", new Blob([fileBuffer], { type: mimeType }), fileName);
 
   const content = typeof options.content === "string" ? options.content.trim() : "";
-  if (content) {
-    body.set("content", content);
-  }
-
   const role = typeof options.role === "string" && options.role.trim()
     ? options.role.trim().toLowerCase()
     : "sdk";
-  body.set("role", role);
+  const authHeaders = {
+    Authorization: `Bearer ${config.agentToken}`,
+    Accept: "application/json",
+  };
 
-  const url = new URL(`/api/tasks/${encodeURIComponent(taskId)}/attachments`, config.backendUrl);
-  const response = await fetchImpl(String(url), {
+  // Phase 1 stages the bytes. This endpoint only stores the file; the row it
+  // creates carries no message and expires within the attachment TTL.
+  const uploadUrl = new URL(`/api/tasks/${encodeURIComponent(taskId)}/attachments`, config.backendUrl);
+  const uploadResponse = await fetchImpl(String(uploadUrl), {
     method: "POST",
-    headers: {
-      Authorization: `Bearer ${config.agentToken}`,
-      Accept: "application/json",
-    },
+    headers: authHeaders,
     body,
   });
 
-  const rawText = await response.text();
-  if (!response.ok) {
-    const details = formatErrorBody(rawText);
-    throw new Error(`Upload failed (${response.status})${details ? `: ${details}` : ""}`);
+  const uploadText = await uploadResponse.text();
+  if (!uploadResponse.ok) {
+    const details = formatErrorBody(uploadText);
+    throw new Error(`Upload failed (${uploadResponse.status})${details ? `: ${details}` : ""}`);
+  }
+  const attachment = (uploadText ? JSON.parse(uploadText) : {}).attachment;
+  if (!attachment || !attachment.id) {
+    throw new Error("Upload succeeded but the server returned no attachment id");
+  }
+
+  // Phase 2 binds the staged file to a message. Without it the attachment
+  // never reaches the chat UI and the janitor deletes it when staging expires.
+  const messageUrl = new URL(`/api/tasks/${encodeURIComponent(taskId)}/messages`, config.backendUrl);
+  const messageResponse = await fetchImpl(String(messageUrl), {
+    method: "POST",
+    headers: { ...authHeaders, "Content-Type": "application/json" },
+    body: JSON.stringify({
+      content: content || `Attached file: ${attachment.name}`,
+      role,
+      attachmentIds: [attachment.id],
+    }),
+  });
+
+  const messageText = await messageResponse.text();
+  if (!messageResponse.ok) {
+    const details = formatErrorBody(messageText);
+    throw new Error(`Attach failed (${messageResponse.status})${details ? `: ${details}` : ""}`);
   }
 
   return {
     taskId,
-    response: rawText ? JSON.parse(rawText) : {},
+    attachment,
+    response: messageText ? JSON.parse(messageText) : {},
   };
 }
 

package/bin/conductor.js

@@ -15,7 +15,7 @@
  *   project  - Manage Conductor projects (list/show/create/...)
  *   issue    - Manage issues (list/show/create/update/start/done)
  *   task     - Manage tasks (create/list/show/send/messages/schedule)
- *   remote-exec - Run a command on another daemon's host
+ *   remote   - Run commands and copy files on another daemon's host
  */
 
 import { fileURLToPath, pathToFileURL } from "node:url";
@@ -51,7 +51,7 @@ export function runConductorCli(args = argv, deps = {}) {
     "project",
     "issue",
     "task",
-    "remote-exec",
+    "remote",
   ];
 
   if (args.length === 0 || args[0] === "--help" || args[0] === "-h") {
@@ -137,7 +137,7 @@ Subcommands:
   project   Manage Conductor projects (list/show/create/...)
   issue     Manage issues (list/show/create/update/start/done)
   task      Manage tasks (create/list/show/send/messages/schedule)
-  remote-exec Run a command on another daemon's host
+  remote    Run commands and copy files on another daemon's host (exec/cp)
 
 Options:
   -h, --help     Show this help message
@@ -162,7 +162,8 @@ Examples:
   conductor task create --title "Refactor module" --prompt "Extract the parser" --backend codex
   conductor task send <task-id> "please add a unit test"
   conductor task schedule create <task-id> "follow up" --delay 10m
-  conductor remote-exec --target ubuntu --workspace /home/duino/ws/holomotion ls .
+  conductor remote exec --target ubuntu --workspace /home/duino/ws/holomotion ls .
+  conductor remote cp ./build.tar.gz ubuntu:/srv/app/build.tar.gz
 
 For subcommand-specific help:
   conductor fire --help
@@ -176,7 +177,7 @@ For subcommand-specific help:
   conductor project --help
   conductor issue --help
   conductor task --help
-  conductor remote-exec --help
+  conductor remote --help
 
 Version: ${pkgJson.version}
 `);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@love-moon/conductor-cli",
-  "version": "0.11.1",
-  "gitCommitId": "c174c7e",
+  "version": "0.12.0",
+  "gitCommitId": "c1047e8",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/lovemoon-ai/conductor.git"
@@ -24,8 +24,8 @@
     "test": "node --test test/*.test.js"
   },
   "dependencies": {
-    "@love-moon/ai-sdk": "0.11.1",
-    "@love-moon/conductor-sdk": "0.11.1",
+    "@love-moon/ai-sdk": "0.12.0",
+    "@love-moon/conductor-sdk": "0.12.0",
     "@github/copilot-sdk": "^0.3.0",
     "chrome-launcher": "^1.2.1",
     "chrome-remote-interface": "^0.33.0",
@@ -38,7 +38,7 @@
   },
   "optionalDependencies": {
     "@roamhq/wrtc": "^0.10.0",
-    "@love-moon/chat-web": "0.11.1"
+    "@love-moon/chat-web": "0.12.0"
   },
   "pnpm": {
     "onlyBuiltDependencies": [