@yolo-labs/yolobridge 0.16.0 → 0.18.0

package/dist/api-client.js

@@ -268,3 +268,38 @@ async function postEvent(cfg, workspaceId, payload) {
         return undefined;
     }
 }
+/**
+ * Phase 1 of a file share: ask common-api to reserve an asset and hand back a
+ * presigned PUT.
+ *
+ * Note what is NOT sent: no path. Only the basename, the mime type and the
+ * size. The operator's directory layout is not the cloud's business, and the
+ * server has no use for it.
+ *
+ * The tile is chosen SERVER-side from the attachment — there is deliberately no
+ * `tileId` parameter here, because a daemon does not get to pick.
+ */
+export async function presignShare(cfg, workspaceId, attachmentId, meta) {
+    const fetchImpl = cfg.fetchImpl ?? fetch;
+    const res = await fetchImpl(`${base(cfg)}/v1/workspaces/${workspaceId}/yolobridge/attach/${attachmentId}/uploads`, {
+        method: 'POST',
+        headers: { ...authHeaders(cfg), 'Content-Type': 'application/json' },
+        body: JSON.stringify(meta),
+    });
+    if (!res.ok) {
+        const { message, code } = await parseErrorBody(res);
+        throw new YoloBridgeApiError(message, res.status, code);
+    }
+    return (await res.json());
+}
+/** Phase 2: the bytes are in R2; ask the server to verify and seal the asset. */
+export async function finalizeShare(cfg, workspaceId, attachmentId, assetId) {
+    const fetchImpl = cfg.fetchImpl ?? fetch;
+    const res = await fetchImpl(`${base(cfg)}/v1/workspaces/${workspaceId}/yolobridge/attach/${attachmentId}/uploads/${encodeURIComponent(assetId)}/finalize`, { method: 'POST', headers: { ...authHeaders(cfg), 'Content-Type': 'application/json' }, body: '{}' });
+    if (!res.ok) {
+        const { message, code } = await parseErrorBody(res);
+        throw new YoloBridgeApiError(message, res.status, code);
+    }
+    const body = (await res.json());
+    return { assetId: body?.asset?.assetId ?? assetId };
+}

package/dist/cli.js

@@ -22,6 +22,7 @@ import { realpathSync, existsSync, readFileSync } from 'node:fs';
 import { hostname } from 'node:os';
 import { runLogin } from './login-cmd.js';
 import { runAttachFromDisk, pickWorkspaceFromDisk } from './attach-cmd.js';
+import { runShare } from './share-cmd.js';
 import { runDetach } from './detach-cmd.js';
 import { getStatus, formatStatus } from './status-cmd.js';
 import { startLocalAgent, stopLocalAgent, DEFAULT_AGENT_BIN } from './local-agent.js';
@@ -99,6 +100,8 @@ function printHelp() {
         '                         RESUMES it (same tile) instead of adding a duplicate; --fresh skips',
         '                         that check entirely.',
         '  detach                 Detach the current workspace attachment.',
+        '  share <path>           Share a local file with the attached workspace, so a cloud',
+        '                         agent can see it. Push only — nothing reads your disk remotely.',
         '  status                 Print local login/attach state.',
         '  version                Print the installed yolo-bridge version (also --version, -v).',
         '  --help                 Print this help.',
@@ -547,6 +550,21 @@ function cmdStatus() {
     process.stdout.write(`${formatStatus(getStatus())}\n`);
     return 0;
 }
+async function cmdShare(args) {
+    const rawPath = args[0];
+    if (!rawPath || rawPath.startsWith('-')) {
+        process.stderr.write('yolo-bridge share: a file path is required.\n\n  yolo-bridge share ./cut.mp4\n');
+        return 64;
+    }
+    const result = await runShare(rawPath, { commonApiBaseUrl: apiUrl() });
+    if (!result.ok) {
+        // Every one of these is an operator-actionable condition, not a bug, so it
+        // prints as a sentence with no stack trace.
+        process.stderr.write(`yolo-bridge share: ${result.message}\n`);
+        return 1;
+    }
+    return 0;
+}
 async function cmdWorkspaces() {
     const result = await runListWorkspaces({ commonApiBaseUrl: apiUrl() });
     if (!result.ok) {
@@ -601,6 +619,8 @@ async function main() {
             return cmdAttach(rest);
         case 'detach':
             return cmdDetach();
+        case 'share':
+            return cmdShare(rest);
         case 'status':
             return cmdStatus();
         case 'version':

package/dist/local-agent.js

@@ -412,10 +412,16 @@ export function computeUsedRows(term) {
     }
     return used;
 }
-/** The PTY's grid, which the tile must render at EXACTLY (it cannot be
- *  resized — the human at the keyboard is watching the same PTY), plus how
+/** The PTY's CURRENT grid, which the tile must render at EXACTLY, plus how
  *  much of that grid is in USE.
  *
+ *  The viewer must not resize this grid — the human at the keyboard is
+ *  watching the same PTY, so a cloud-side resize would reach over and reflow
+ *  their real screen. But the human themselves CAN resize it, and when they
+ *  do the ring restarts under a fresh epoch (see `handleLocalResize`), so a
+ *  viewer holding the old geometry re-seeds rather than replaying old bytes
+ *  against a grid that no longer exists.
+ *
  *  ⚠️ `usedRows` IS NOT A SECOND GEOMETRY. It never changes `cols`/`rows` and
  *  the viewer never resizes its terminal to it: a viewer's terminal that is
  *  not exactly `cols`×`rows` cannot replay the daemon's bytes (relative cursor
@@ -679,6 +685,12 @@ export function startLocalAgent(opts = {}) {
     // explicitly to disable stdin piping entirely, distinct from omitting the
     // field (which defaults to wiring up the real `process.stdin`).
     const inStream = 'stdin' in opts ? opts.stdin : process.stdin;
+    // Same defaulting rule as `resolveCols`/`resolveRows`: only reach for the real
+    // `process.stdout` when nothing was injected, so a test with a fake sink does
+    // not silently pick up the CI runner's terminal.
+    const resizeSource = 'resizeSource' in opts
+        ? opts.resizeSource
+        : (opts.stdout === undefined ? process.stdout : undefined);
     const ptyProcess = spawnImpl(agentBin, agentArgs, {
         name: 'xterm-256color',
         cols,
@@ -706,8 +718,14 @@ export function startLocalAgent(opts = {}) {
         writeChain: Promise.resolve(),
         stdin: inStream,
         rawModeEnabled: false,
+        resizeSource,
     };
     current = state;
+    if (resizeSource && typeof resizeSource.on === 'function') {
+        const resizeListener = () => handleLocalResize(state);
+        state.resizeListener = resizeListener;
+        resizeSource.on('resize', resizeListener);
+    }
     ptyProcess.onData((data) => {
         state.lastOutputAt = Date.now();
         outStream.write(data);
@@ -756,7 +774,53 @@ export function startLocalAgent(opts = {}) {
  * exit code 124. With `stdin.pause()` added below, the same repro exits
  * cleanly on its own well under a second — no timeout/kill needed.
  */
+/**
+ * The human at the keyboard resized their terminal window.
+ *
+ * WHY THIS EXISTS: the PTY size used to be treated as fixed at spawn, on the
+ * reasoning that "nothing resizes this PTY, least of all the cloud". The cloud
+ * half of that is right and still enforced — a viewer must never reach over and
+ * reflow the operator's real screen. The other half was simply wrong: the human
+ * watching this PTY through `process.stdout` can drag their window, Node raises
+ * `'resize'` on SIGWINCH, and we ignored it. The child kept rendering to the old
+ * grid, so the operator's own terminal garbled and the tile kept reporting a
+ * geometry that no longer matched anything.
+ *
+ * Everything retained in the ring was captured against a grid that no longer
+ * exists — relative cursor moves and the scroll region are defined against the
+ * REAL grid, so replaying those bytes at the new size garbles rather than
+ * degrades. A fresh epoch is the honest answer: the viewer re-seeds from the
+ * current buffer instead of splicing two geometries together.
+ */
+function handleLocalResize(state) {
+    const src = state.resizeSource;
+    if (!src)
+        return;
+    const cols = src.columns ?? 0;
+    const rows = src.rows ?? 0;
+    // A stdout that has stopped being a TTY (piped, or the window is gone)
+    // reports 0/undefined. Resizing a PTY to zero wedges the child.
+    if (cols < 1 || rows < 1)
+        return;
+    // SIGWINCH also fires for changes that are not a size change. Resizing to the
+    // size we already have would burn an epoch and force a pointless re-seed.
+    if (cols === state.cols && rows === state.rows)
+        return;
+    state.cols = cols;
+    state.rows = rows;
+    state.term.resize(cols, rows);
+    rawRing = newRawRing(rows);
+    try {
+        state.ptyProcess.resize(cols, rows);
+    }
+    catch {
+        // The child is already gone; onExit will clear `current`.
+    }
+}
 function teardownStdio(state) {
+    if (state.resizeSource && state.resizeListener && typeof state.resizeSource.removeListener === 'function') {
+        state.resizeSource.removeListener('resize', state.resizeListener);
+    }
     const { stdin, stdinListener } = state;
     if (stdin && stdinListener && typeof stdin.removeListener === 'function') {
         stdin.removeListener('data', stdinListener);

package/dist/share-cmd.js

@@ -0,0 +1,217 @@
+/**
+ * `yolo-bridge share <path>` — hand a LOCAL file up to the attached workspace
+ * so the cloud orchestrator can see it.
+ *
+ * DIRECTION IS THE SECURITY MODEL. This is the only file path in the daemon,
+ * and it runs because the operator (or their local agent) asked for THIS file.
+ * There is no counterpart that lets the cloud name a path and have the daemon
+ * read it — that would turn a cloud-side prompt injection into a read of the
+ * operator's disk.
+ *
+ * Bytes go straight from this process to R2 via a presigned PUT. They do not
+ * pass through common-api, so a large file is not bounded by any JSON body
+ * limit, and the API never holds the operator's content.
+ */
+import { createReadStream } from 'node:fs';
+import { stat } from 'node:fs/promises';
+import path from 'node:path';
+import { presignShare, finalizeShare, YoloBridgeApiError, } from './api-client.js';
+import { loadAuth, loadAttachment } from './config-store.js';
+/**
+ * Mirrors common-api's `MAX_ASSET_BYTES`. The server is authoritative and
+ * refuses over-cap uploads on its own; this copy exists so a 2 GB video fails
+ * in a second with a readable message instead of after a long upload.
+ *
+ * If the server cap ever moves, the worst this stale copy produces is a local
+ * refusal of something the server would have accepted — a clear message, not a
+ * corrupt upload.
+ */
+export const MAX_SHARE_BYTES = 100 * 1024 * 1024;
+const MIME_BY_EXT = {
+    '.mp4': 'video/mp4', '.mov': 'video/quicktime', '.webm': 'video/webm', '.mkv': 'video/x-matroska',
+    '.png': 'image/png', '.jpg': 'image/jpeg', '.jpeg': 'image/jpeg', '.gif': 'image/gif',
+    '.webp': 'image/webp', '.svg': 'image/svg+xml', '.heic': 'image/heic',
+    '.mp3': 'audio/mpeg', '.wav': 'audio/wav', '.m4a': 'audio/mp4', '.flac': 'audio/flac',
+    '.pdf': 'application/pdf', '.zip': 'application/zip', '.json': 'application/json',
+    '.txt': 'text/plain', '.md': 'text/markdown', '.csv': 'text/csv',
+};
+export function guessMimeType(filename) {
+    return MIME_BY_EXT[path.extname(filename).toLowerCase()] ?? 'application/octet-stream';
+}
+export function formatBytes(n) {
+    if (n < 1024)
+        return `${n} B`;
+    if (n < 1024 * 1024)
+        return `${(n / 1024).toFixed(1)} KB`;
+    if (n < 1024 * 1024 * 1024)
+        return `${(n / (1024 * 1024)).toFixed(1)} MB`;
+    return `${(n / (1024 * 1024 * 1024)).toFixed(2)} GB`;
+}
+/** Raised for a condition the operator can act on. The CLI prints `.message`
+ *  and exits non-zero — no stack trace, because none of these are bugs. */
+export class ShareError extends Error {
+}
+/**
+ * Everything decided from the local filesystem, BEFORE a single byte moves.
+ *
+ * Split out from the upload so the refusals are testable without a network,
+ * and so an over-cap file costs a `stat`, not an upload.
+ */
+export async function inspectLocalFile(rawPath) {
+    const absolutePath = path.resolve(rawPath);
+    let info;
+    try {
+        info = await stat(absolutePath);
+    }
+    catch (err) {
+        const code = err?.code;
+        if (code === 'ENOENT')
+            throw new ShareError(`No such file: ${rawPath}`);
+        if (code === 'EACCES')
+            throw new ShareError(`Permission denied reading ${rawPath}`);
+        throw new ShareError(`Could not read ${rawPath}: ${err.message}`);
+    }
+    if (info.isDirectory()) {
+        throw new ShareError(`${rawPath} is a directory. Share a single file.`);
+    }
+    if (!info.isFile()) {
+        throw new ShareError(`${rawPath} is not a regular file.`);
+    }
+    if (info.size > MAX_SHARE_BYTES) {
+        // Name both numbers: "too large" without the cap leaves the operator
+        // guessing how much to trim.
+        throw new ShareError(`${path.basename(absolutePath)} is ${formatBytes(info.size)}, over the ${formatBytes(MAX_SHARE_BYTES)} limit for a shared file.`);
+    }
+    return {
+        absolutePath,
+        // Only the BASENAME travels. The operator's directory layout is not the
+        // cloud's business.
+        filename: path.basename(absolutePath),
+        size: info.size,
+        mimeType: guessMimeType(absolutePath),
+    };
+}
+/**
+ * Presign → PUT the bytes to R2 → finalize.
+ *
+ * The PUT streams from disk rather than buffering: a 100 MB file must not
+ * become a 100 MB string in this process.
+ */
+export async function shareFile(rawPath, deps) {
+    const write = deps.write ?? ((line) => process.stdout.write(`${line}\n`));
+    const file = await inspectLocalFile(rawPath);
+    // One fetch for all three legs. Taking it from `deps` too means a caller can
+    // inject it once instead of having to remember to put it on `cfg` as well.
+    const cfg = { ...deps.cfg, fetchImpl: deps.cfg.fetchImpl ?? deps.fetchImpl };
+    write(`Sharing ${file.filename} (${formatBytes(file.size)})…`);
+    const presigned = await presignShare(cfg, deps.workspaceId, deps.attachmentId, {
+        filename: file.filename,
+        mimeType: file.mimeType,
+        size: file.size,
+    });
+    const fetchImpl = (deps.fetchImpl ?? deps.cfg.fetchImpl ?? fetch);
+    // Streamed from disk rather than buffered: a 100 MB file must not become a
+    // 100 MB Buffer in this process. The stream is opened lazily, so it is held
+    // in a variable and explicitly destroyed on every failure path — otherwise a
+    // rejected or refused PUT leaks the descriptor.
+    const body = createReadStream(file.absolutePath);
+    // A read failure — the file deleted, truncated or unreadable mid-upload —
+    // arrives as an 'error' EVENT, not a rejected promise. With no listener Node
+    // escalates it to an uncaughtException and takes the process down, which for
+    // a daemon sharing a file the operator just moved is a very poor trade.
+    // Captured here and reported as an ordinary failure instead.
+    let readError;
+    body.on('error', (err) => { readError = err; });
+    let put;
+    try {
+        put = await fetchImpl(presigned.uploadUrl, {
+            method: presigned.method || 'PUT',
+            headers: { ...presigned.headers, 'Content-Length': String(file.size) },
+            body: body,
+            // Node's fetch requires this for a stream body.
+            duplex: 'half',
+        });
+    }
+    catch (err) {
+        body.destroy();
+        throw new ShareError(`Upload failed: ${err?.message || 'network error'}`);
+    }
+    if (!put.ok) {
+        body.destroy();
+        // The presigned URL is short-lived and size-bound; both failure modes are
+        // worth naming rather than surfacing a bare status.
+        throw new ShareError(`Upload failed (HTTP ${put.status}). The link may have expired, or the file changed size while uploading. Try again.`);
+    }
+    if (readError) {
+        throw new ShareError(`Could not read ${file.filename} while uploading: ${readError.message}`);
+    }
+    const finalized = await finalizeShare(cfg, deps.workspaceId, deps.attachmentId, presigned.assetId);
+    write(`Shared ${file.filename} → ${finalized.assetId}`);
+    return finalized;
+}
+/** Turn an API error into something the operator can act on. */
+export function describeShareFailure(err) {
+    if (err instanceof ShareError)
+        return err.message;
+    if (err instanceof YoloBridgeApiError) {
+        if (err.code === 'PAYLOAD_TOO_LARGE')
+            return err.message;
+        if (err.code === 'STORAGE_NOT_CONFIGURED')
+            return 'File sharing is not available on this server.';
+        if (err.code === 'WORKSPACE_LIMIT_REACHED' || err.code === 'LIMIT_REACHED')
+            return err.message;
+        if (err.status === 403)
+            return 'This daemon is not attached to that workspace any more. Re-run `yolo-bridge attach`.';
+        return err.message;
+    }
+    return err?.message || 'Share failed.';
+}
+/**
+ * The disk-backed entry point, mirroring `runDetach`.
+ *
+ * `share` and a running `yolo-bridge attach` are two SEPARATE processes, so
+ * this reads the attachment identity and its workspace-scoped credential from
+ * the config store rather than from daemon memory — the same credential the
+ * running daemon uses, and the only one the upload routes accept (Boundary B).
+ */
+export async function runShare(rawPath, deps) {
+    // Same precondition ordering as detach: `auth.json` is what makes this a
+    // set-up machine, and its absence has a far better remedy to offer than a
+    // 403 would.
+    if (!loadAuth(deps.env, deps.io)) {
+        return { ok: false, reason: 'not-logged-in', message: 'Not logged in — run `yolo-bridge login` first.' };
+    }
+    const attachment = loadAttachment(deps.env, deps.io);
+    if (!attachment) {
+        return { ok: false, reason: 'not-attached', message: 'No active attachment — run `yolo-bridge attach` first.' };
+    }
+    const scopedToken = attachment.scopedToken;
+    if (!scopedToken) {
+        // Nothing on this machine can mint one for an existing attachment, so say
+        // so plainly rather than sending an account token to be refused.
+        return {
+            ok: false,
+            reason: 'no-scoped-credential',
+            message: 'No workspace-scoped credential is stored for this attachment, so files cannot be shared '
+                + 'from this machine. Run `yolo-bridge attach` to reconnect.',
+        };
+    }
+    const cfg = {
+        commonApiBaseUrl: deps.commonApiBaseUrl,
+        accessToken: scopedToken,
+        fetchImpl: deps.fetchImpl,
+    };
+    try {
+        const { assetId } = await shareFile(rawPath, {
+            cfg,
+            workspaceId: attachment.workspaceId,
+            attachmentId: attachment.attachmentId,
+            fetchImpl: deps.fetchImpl,
+            write: deps.write,
+        });
+        return { ok: true, assetId };
+    }
+    catch (err) {
+        return { ok: false, reason: 'error', message: describeShareFailure(err) };
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yolo-labs/yolobridge",
-  "version": "0.16.0",
+  "version": "0.18.0",
   "description": "YoloBridge — local coding-agent daemon that attaches a user's own Claude Code/Codex session to a YOLO Studio workspace as a first-class tile (docs/YOLOBRIDGE_PLAN.md, build-order Phase 5).",
   "license": "MIT",
   "type": "module",