@livedesk/hub 0.1.5 → 0.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@livedesk/hub",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "description": "LiveDesk local Hub API and browser frame bridge",
   "type": "module",
   "main": "src/server.js",
@@ -15,10 +15,10 @@
   },
   "dependencies": {
     "@ffmpeg-installer/ffmpeg": "^1.1.0",
+    "@livedesk/runtime-core": "0.1.0",
     "@openai/codex-sdk": "0.145.0",
     "cors": "^2.8.5",
     "express": "^4.21.2",
-    "ffmpeg-static": "^5.3.0",
     "path-to-regexp": "0.1.13",
     "ws": "^8.18.3"
   },

package/src/agents/agent-tool-registry.js

@@ -27,7 +27,7 @@ const readTaskPermission = { ask: 'allow', safeAuto: 'allow', fullAccess: 'allow
 export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   {
     name: 'livedesk.list_devices',
-    description: 'List connected LiveDesk workstations. This is read-only.',
+    description: 'List connected LiveDesk Clients. This is read-only.',
     category: 'read',
     readOnly: true,
     mutating: false,
@@ -40,7 +40,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.get_system_health',
-    description: 'Read system health from the selected workstations.',
+    description: 'Read system health from the selected Clients.',
     category: 'read',
     readOnly: true,
     mutating: false,
@@ -53,7 +53,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.get_gpu_status',
-    description: 'Read GPU status from the selected workstations.',
+    description: 'Read GPU status from the selected Clients.',
     category: 'read',
     readOnly: true,
     mutating: false,
@@ -66,7 +66,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.get_disk_status',
-    description: 'Read disk status from the selected workstations.',
+    description: 'Read disk status from the selected Clients.',
     category: 'read',
     readOnly: true,
     mutating: false,
@@ -79,7 +79,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.list_processes',
-    description: 'Read process status from the selected workstations. Use processName for a named process such as ComfyUI.',
+    description: 'Read process status from the selected Clients. Use processName for a named process such as ComfyUI.',
     category: 'read',
     readOnly: true,
     mutating: false,
@@ -92,7 +92,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.get_service_status',
-    description: 'Read service status from the selected workstations. Use serviceName when the user names a service.',
+    description: 'Read service status from the selected Clients. Use serviceName when the user names a service.',
     category: 'read',
     readOnly: true,
     mutating: false,
@@ -105,7 +105,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.collect_diagnostics',
-    description: 'Collect the existing safe LiveDesk diagnostics payload from the selected workstations.',
+    description: 'Collect the existing safe LiveDesk diagnostics payload from the selected Clients.',
     category: 'read',
     readOnly: true,
     mutating: false,
@@ -118,7 +118,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.control_process',
-    description: 'Stop or restart a named process on the selected workstations. This changes process state.',
+    description: 'Stop or restart a named process on the selected Clients. This changes process state.',
     category: 'processControl',
     readOnly: false,
     mutating: true,
@@ -131,7 +131,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.control_service',
-    description: 'Start, stop, or restart a named service on the selected workstations.',
+    description: 'Start, stop, or restart a named service on the selected Clients.',
     category: 'serviceControl',
     readOnly: false,
     mutating: true,
@@ -157,7 +157,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.close_application',
-    description: 'Close a named application process on the selected workstations.',
+    description: 'Close a named application process on the selected Clients.',
     category: 'applicationControl',
     readOnly: false,
     mutating: true,
@@ -170,7 +170,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.read_file',
-    description: 'Read a bounded UTF-8 text file from the selected workstations, with sensitive credential paths rejected.',
+    description: 'Read a bounded UTF-8 text file from the selected Clients, with sensitive credential paths rejected.',
     category: 'fileRead',
     readOnly: true,
     mutating: false,
@@ -183,7 +183,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.write_file',
-    description: 'Write bounded text content to a file on the selected workstations.',
+    description: 'Write bounded text content to a file on the selected Clients.',
     category: 'fileWrite',
     readOnly: false,
     mutating: true,
@@ -196,7 +196,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.delete_file',
-    description: 'Delete a selected file or explicitly requested directory on the selected workstations.',
+    description: 'Delete a selected file or explicitly requested directory on the selected Clients.',
     category: 'fileDelete',
     readOnly: false,
     mutating: true,
@@ -209,7 +209,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.list_directory',
-    description: 'List bounded file metadata from a directory on the selected workstations.',
+    description: 'List bounded file metadata from a directory on the selected Clients.',
     category: 'fileRead',
     readOnly: true,
     mutating: false,
@@ -222,7 +222,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.run_command',
-    description: 'Run a bounded command through the selected workstation command interpreter. This is an audited high-risk action.',
+    description: 'Run a bounded command through the selected Client command interpreter. This is an audited high-risk action.',
     category: 'shell',
     readOnly: false,
     mutating: true,
@@ -235,7 +235,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.run_script',
-    description: 'Run a bounded script file using the workstation platform interpreter.',
+    description: 'Run a bounded script file using the Client platform interpreter.',
     category: 'script',
     readOnly: false,
     mutating: true,
@@ -261,7 +261,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.get_network_status',
-    description: 'Read bounded network adapter and connectivity status from the selected workstations.',
+    description: 'Read bounded network adapter and connectivity status from the selected Clients.',
     category: 'network',
     readOnly: true,
     mutating: false,
@@ -274,7 +274,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.power_action',
-    description: 'Request an explicit power action on the selected workstations.',
+    description: 'Request an explicit power action on the selected Clients.',
     category: 'systemPower',
     readOnly: false,
     mutating: true,
@@ -287,7 +287,7 @@ export const AGENT_TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: 'livedesk.system_configuration',
-    description: 'Apply one explicitly named system configuration action supported by the workstation agent.',
+    description: 'Apply one explicitly named system configuration action supported by the Client agent.',
     category: 'systemConfiguration',
     readOnly: false,
     mutating: true,

package/src/agents/codex-agent-runtime.js

@@ -472,7 +472,7 @@ export function createCodexAgentRuntime({
       'Never use arbitrary MCP servers, change permissions, forge approvals, request credentials, or invent a tool result.',
       `The Hub has fixed this run to permission mode ${permissionPolicy?.mode || 'ask'} and enforces the policy independently of your instructions.`,
       'Use only the selected connected device IDs below. If the Hub asks for user approval, wait for that approval result and do not work around it.',
-      'You may perform multiple safe read-only checks when the request requires a sequence. For example, find workstations without ComfyUI, then check the service status only on those workstations.',
+      'You may perform multiple safe read-only checks when the request requires a sequence. For example, find Clients without ComfyUI, then check the service status only on those Clients.',
       `Selected device IDs: ${JSON.stringify(deviceIds)}`,
       'Return a concise Korean or English summary grounded only in tool results. Do not invent results.',
       `User request: ${safeText(instruction, 4000)}`
@@ -505,7 +505,7 @@ export function createCodexAgentRuntime({
       const status = await getStatus();
       if (!status.installed) throw new AgentProviderError('codex-sdk-not-installed', 'Codex SDK is not installed.', { status: 503 });
       if (status.authenticated !== 'signed-in') throw new AgentProviderError('codex-auth-required', 'Sign in with the Codex CLI first.', { status: 401 });
-      reportProgress(run, 'preparing-tools', 'Codex is ready. Preparing the LiveDesk tools for this workstation.');
+      reportProgress(run, 'preparing-tools', 'Codex is ready. Preparing the LiveDesk tools for this Client.');
       session = createMcpSession({
         runId: run.runId,
         signal: run.abortController.signal,

package/src/filesystem/shared-folders.js

@@ -139,7 +139,7 @@ export class HubSharedFolders {
           }
         }));
       }
-      folder.message = jobs.length > 0 ? `Syncing ${jobs.length} workstation${jobs.length === 1 ? '' : 's'}` : 'No changed files';
+      folder.message = jobs.length > 0 ? `Syncing ${jobs.length} Client${jobs.length === 1 ? '' : 's'}` : 'No changed files';
       await this.persist();
       return { ok: true, jobs, files: scan.files.length, totalBytes: scan.totalBytes };
     } finally {

package/src/filesystem/transfer-jobs.js

@@ -4,6 +4,7 @@ import { setImmediate as yieldToEventLoop } from 'node:timers/promises';
 
 const CHUNK_BYTES = 512 * 1024;
 const JOB_RETENTION_MS = 10 * 60 * 1000;
+const COMMAND_RESULT_TIMEOUT_MS = 30 * 1000;
 
 function snapshot(job) {
   return {
@@ -28,13 +29,15 @@ function markUpdated(job) {
 }
 
 export class HubTransferJobs {
-  constructor({ filesystem, remoteHub, maxConcurrent = 2 } = {}) {
+  constructor({ filesystem, remoteHub, maxConcurrent = 2, commandResultTimeoutMs = COMMAND_RESULT_TIMEOUT_MS } = {}) {
     this.filesystem = filesystem;
     this.remoteHub = remoteHub;
     this.maxConcurrent = Math.max(1, Number(maxConcurrent) || 2);
+    this.commandResultTimeoutMs = Math.max(1_000, Number(commandResultTimeoutMs) || COMMAND_RESULT_TIMEOUT_MS);
     this.jobs = new Map();
     this.queue = [];
     this.running = 0;
+    this.pendingCommands = new Map();
   }
 
   create({ itemIds = [], deviceIds = [], remoteDirectory = '', files = null, onComplete = null } = {}) {
@@ -90,6 +93,7 @@ export class HubTransferJobs {
     job.state = 'cancelled';
     job.error = 'transfer-cancelled';
     if (job.currentStream) job.currentStream.destroy();
+    this.finishPendingCommands(job.jobId, 'transfer-cancelled');
     markUpdated(job);
     return snapshot(job);
   }
@@ -181,10 +185,13 @@ export class HubTransferJobs {
   }
 
   async sendChunk(job, file, offset, buffer, final, targets) {
-    for (const deviceId of targets) {
-      if (job.cancelled) return;
+    const outcomes = await Promise.all(targets.map(async deviceId => {
+      if (job.cancelled) return { deviceId, ok: false, error: 'transfer-cancelled' };
+      const commandId = crypto.randomUUID();
+      const pending = this.waitForCommandResult(job.jobId, deviceId, commandId);
       const result = this.remoteHub.sendCommand(deviceId, {
         command: 'file.transfer.chunk',
+        commandId,
         payload: {
           transferId: job.jobId,
           remoteDirectory: job.remoteDirectory,
@@ -199,9 +206,87 @@ export class HubTransferJobs {
         }
       });
       if (!result?.ok) {
-        job.failedTargets.add(deviceId);
-        job.targetErrors.set(deviceId, String(result?.error || 'device-not-connected'));
+        this.settlePendingCommand(commandId, {
+          ok: false,
+          error: String(result?.error || 'device-not-connected')
+        });
       }
+      return { deviceId, ...await pending };
+    }));
+
+    for (const outcome of outcomes) {
+      if (outcome.ok || outcome.error === 'transfer-cancelled') continue;
+      job.failedTargets.add(outcome.deviceId);
+      job.targetErrors.set(outcome.deviceId, outcome.error || 'transfer-failed');
+    }
+    if (!job.cancelled && outcomes.length > 0 && outcomes.every(outcome => !outcome.ok)) {
+      throw new Error('all-targets-failed');
+    }
+  }
+
+  waitForCommandResult(jobId, deviceId, commandId) {
+    return new Promise(resolve => {
+      const timer = setTimeout(() => {
+        this.settlePendingCommand(commandId, {
+          ok: false,
+          error: 'file-transfer-ack-timeout'
+        });
+      }, this.commandResultTimeoutMs);
+      timer.unref?.();
+      this.pendingCommands.set(commandId, {
+        jobId,
+        deviceId,
+        timer,
+        resolve
+      });
+    });
+  }
+
+  settlePendingCommand(commandId, outcome) {
+    const pending = this.pendingCommands.get(String(commandId || ''));
+    if (!pending) return false;
+    this.pendingCommands.delete(commandId);
+    clearTimeout(pending.timer);
+    pending.resolve(outcome);
+    return true;
+  }
+
+  finishPendingCommands(jobId, error) {
+    for (const [commandId, pending] of this.pendingCommands) {
+      if (pending.jobId !== jobId) continue;
+      this.settlePendingCommand(commandId, { ok: false, error });
+    }
+  }
+
+  handleRemoteEvent(type, event) {
+    if (type === 'RemoteCommandResult') {
+      const commandId = String(event?.commandId || '');
+      const pending = this.pendingCommands.get(commandId);
+      if (!pending) return;
+      const eventDeviceId = String(event?.deviceId || event?.device?.deviceId || '');
+      if (eventDeviceId && eventDeviceId !== pending.deviceId) return;
+      const result = event?.result;
+      const error = String(
+        event?.error
+        || (result?.ok === false || result?.status === 'failed' || result?.status === 'rejected'
+          ? result?.error || 'file-transfer-rejected'
+          : '')
+      );
+      this.settlePendingCommand(commandId, error
+        ? { ok: false, error }
+        : { ok: true, result });
+      return;
+    }
+
+    if (type !== 'RemoteDeviceDisconnected') return;
+    const deviceId = String(event?.deviceId || event?.device?.deviceId || '');
+    if (!deviceId) return;
+    for (const [commandId, pending] of this.pendingCommands) {
+      if (pending.deviceId !== deviceId) continue;
+      this.settlePendingCommand(commandId, {
+        ok: false,
+        error: 'device-disconnected'
+      });
     }
   }