patchcord 0.5.57 → 0.5.59

package/bin/patchcord.mjs

@@ -78,6 +78,7 @@ Usage:
   patchcord whoami --propose "<text>"                     Update your whoami (two-shot: first call asks you to show human, second call applies)
   patchcord agents                                        List every agent (with whoami)
   patchcord agents <name>                                 Show one agent's whoami
+  patchcord upload <file> [--mime <type>] [--as <name>]   Upload a file as a patchcord attachment (prints the storage path)
   patchcord subscribe                                     Start the realtime listener
   patchcord update                                        Update to the latest version
   patchcord --version                                     Show installed version
@@ -390,6 +391,66 @@ if (cmd === "agents") {
   process.exit(0);
 }
 
+// ── upload <file> [--mime <type>] [--as <filename>] ───────────
+// Inline-upload an attachment to patchcord storage. Wraps what would
+// otherwise be a base64-and-curl dance. Prints the storage path on
+// success so the user can pipe it into send_message.
+if (cmd === "upload") {
+  const args = process.argv.slice(3);
+  const filePath = (args[0] || "").trim();
+  if (!filePath) {
+    console.error("Usage: patchcord upload <file> [--mime <type>] [--as <filename>]");
+    process.exit(1);
+  }
+  if (!existsSync(filePath)) {
+    console.error(`file not found: ${filePath}`);
+    process.exit(1);
+  }
+  const mimeIdx = args.indexOf("--mime");
+  const mime = mimeIdx !== -1 ? (args[mimeIdx + 1] || "").trim() : "";
+  const asIdx = args.indexOf("--as");
+  const renameTo = asIdx !== -1 ? (args[asIdx + 1] || "").trim() : "";
+
+  const found = await _resolveBearer();
+  if (!found) {
+    console.error("No patchcord config found in current directory or any parent. Run `npx patchcord@latest` from a project directory first.");
+    process.exit(1);
+  }
+  const { token, baseUrl } = found;
+  if (!isSafeToken(token) || !isSafeUrl(baseUrl)) {
+    console.error(`Invalid patchcord URL or token in config.`);
+    process.exit(1);
+  }
+
+  const { readFileSync, statSync } = await import("fs");
+  const stats = statSync(filePath);
+  if (stats.size === 0) {
+    console.error(`file is empty: ${filePath}`);
+    process.exit(1);
+  }
+  if (stats.size > 25 * 1024 * 1024) {
+    console.error(`file is ${(stats.size / 1024 / 1024).toFixed(1)}MB; max is 25MB for inline upload.`);
+    process.exit(1);
+  }
+
+  const data = readFileSync(filePath);
+  const content_b64 = data.toString("base64");
+  const filename = renameTo || basename(filePath);
+
+  const body = { filename, content_b64 };
+  if (mime) body.mime_type = mime;
+
+  const { status, json, body: respBody } = await _httpJSON(
+    "POST", `${baseUrl}/api/agent/attachment/upload`, token, body
+  );
+  if (status !== "200") {
+    console.error(`✗ HTTP ${status}: ${(json && json.error) || respBody}`);
+    process.exit(1);
+  }
+  console.log(json.path);
+  process.exit(0);
+}
+
 // ── subscribe ─────────────────────────────────────────────────
 // Thin wrapper around scripts/subscribe.mjs so users see a clean
 // "npx patchcord subscribe" in their Claude Code tool log instead
@@ -1929,5 +1990,5 @@ if (cmd === "skill") {
   process.exit(0);
 }
 
-console.error(`Unknown command: ${cmd}. Available: whoami, agents, subscribe, update, --rename, --token, --agent-type, --version, --help`);
+console.error(`Unknown command: ${cmd}. Available: whoami, agents, upload, subscribe, update, --rename, --token, --agent-type, --version, --help`);
 process.exit(1);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "patchcord",
-  "version": "0.5.57",
+  "version": "0.5.59",
   "description": "Cross-machine agent messaging for Claude Code and Codex",
   "author": "ppravdin",
   "license": "MIT",

package/per-project-skills/codex/SKILL.md

@@ -95,33 +95,30 @@ To message a user outside your namespace, use `@username` as the to_agent. Examp
 
 ## File sharing
 
-Three modes:
-
-**Relay from URL (preferred for public files):**
+**Files on disk → `patchcord upload` (CLI, preferred):**
 ```
-attachment(relay=true, path_or_url="https://example.com/file.md", filename="file.md")
+patchcord upload /path/to/report.md --mime text/markdown
 ```
-Server fetches the URL and stores it. ~50 tokens instead of thousands for the file content.
+Prints the storage path. Pass it to `send_message`. No curl, no base64 in chat. 25MB cap.
 
-**Presigned upload (preferred for local files):**
+**Public URLs → `attachment(relay=true, ...)`:**
 ```
-attachment(upload=true, filename="report.md") -> returns {url, path}
-curl -X PUT -H "Content-Type: text/markdown" --data-binary @/path/to/report.md "<url>"
+attachment(relay=true, path_or_url="https://example.com/file.md", filename="file.md")
 ```
-Then send the `path` to the other agent. No base64, no token waste.
+Server fetches and stores. Use when the file already lives at a public URL.
 
-**Inline base64 (last resort — small generated content only):**
+**Inline base64 last resort:**
 ```
 attachment(upload=true, filename="notes.txt", file_data="<base64>")
 ```
-Base64 adds ~33% overhead and wastes context tokens. Never use this for files on disk — use presigned upload above instead.
+Only if you cannot run shell commands. Wastes context tokens.
 
 **Downloading:**
 ```
 attachment(path_or_url="namespace/agent/timestamp_file.md")
 ```
 
-Send the returned `path` to the other agent in your message so they can download it.
+Always send the storage path (not file content) to the other agent.
 
 ## Rules
 

package/skills/inbox/SKILL.md

@@ -83,34 +83,31 @@ To message a user outside your namespace, use `@username` as the to_agent. Examp
 
 ## File sharing
 
-Three modes, choose based on context:
-
-**Relay from URL (preferred for public files):**
+**Files on disk → `patchcord upload` (CLI, preferred):**
 ```
-attachment(relay=true, path_or_url="https://example.com/file.md", filename="file.md")
+patchcord upload /path/to/report.md --mime text/markdown
 ```
-Server fetches the URL and stores it. You send only a URL string (~50 tokens) instead of the file content (thousands of tokens). Always prefer relay when the file is at a public URL.
+Prints the storage path. Pass that path to `send_message`. No curl, no base64 in chat, no presigned URLs. 25MB cap.
 
-**Presigned upload (preferred for local files):**
+**Public URLs → `attachment(relay=true, ...)`:**
 ```
-attachment(upload=true, filename="report.md") -> returns {url, path}
-curl -X PUT -H "Content-Type: text/markdown" --data-binary @/path/to/report.md "<url>"
+attachment(relay=true, path_or_url="https://example.com/file.md", filename="file.md")
 ```
-Then send the `path` to the other agent. No base64, no token waste.
+Server fetches the URL and stores it. Use when the file already lives at a public URL.
 
-**Inline base64 (last resort — small generated content only):**
+**Web agents (no shell) → inline base64 last resort:**
 ```
 attachment(upload=true, filename="notes.txt", file_data="<base64>")
 ```
-Base64 adds ~33% overhead and wastes context tokens. Never use this for files on disk — use presigned upload above instead.
+Only for agents that cannot run shell commands. Wastes context tokens. Never use if you can run `patchcord upload`.
 
 **Downloading:**
 ```
 attachment(path_or_url="namespace/agent/timestamp_file.md")
 ```
-Use the path from the sender's message.
+Pass the storage path from the sender's message.
 
-Send the returned `path` to the other agent in your message so they can download it.
+Always send the storage path (not the file content) to the other agent.
 
 ## Identity (`patchcord whoami` / `patchcord agents`)