@goodandready/dsh-remote-workspace 0.3.4 → 0.3.6

package/docs/design/DESIGN.md

@@ -83,3 +83,9 @@ emote_tunnel: создание/остановка/листинг туннеле
   - `src/client/plugin-card.js` — Plugin card and status chip components
   - `src/client/entry.js` — Cordis client plugin apply and slot registration
 - Added native builder `scripts/build-client.mjs` hooked into `prepack` and `npm test`.
+
+### v0.3.6 Model Tools JSON Schema Standardization (Refs: #33)
+- Все 9 инструментов (`remote_exec`, `remote_fs`, `remote_sync`, `remote_tunnel`, `remote_docker`, `remote_service`, `remote_transfer`, `remote_diagnose`, `remote_env`) переведены на компиляцию через `defineTool()` из `@deepseek-ai/dsh-tools`.
+- Параметры инструментов компилируются в строгий JSON Schema с корнем `{ type: 'object', properties: ..., required: [...] }`.
+- Предотвращены ошибки валидации схем в OpenAI-совместимых провайдерах (`Invalid schema: schema must be a JSON Schema of 'type: "object"', got 'type: null'`).
+- Схема вывода стандартизирована с явным `{ type: 'object', additionalProperties: true }`.

package/lib/client.js

@@ -2300,7 +2300,25 @@ window.__ModuleLoader__.load({
       }
 
       if (ctx.slots && typeof ctx.slots.inject === 'function') {
-        // Row seat first (the seat the current core renders), legacy seat after it.
+        // Plugin-list seat (plugins.item) first: the seat the current core
+        // (0.1.6-alpha.2) renders as the plugin's own page with its configuration. The
+        // label is a static string on purpose — it is resolved while the page renders,
+        // and a locale lookup there would take the whole client batch down with it.
+        ctx.slots.inject('plugins.item', () =>
+          ctx.slots.register(
+            {
+              name: 'plugins.item',
+              id: ROW_ID,
+              order: 60,
+              label: () => 'Remote Workspace',
+              locale: NS,
+              inject: () => ({ ctx })
+            },
+            (props) => React.createElement(PluginCard, Object.assign({}, props, { ctx }))
+          )
+        )
+
+        // Row seat and the legacy seat stay as fallbacks.
         ctx.slots.inject('plugins.row.config', () =>
           ctx.slots.register(
             {

package/lib/tools.js

@@ -1,15 +1,17 @@
+import { defineTool } from '@deepseek-ai/dsh-tools';
+
 export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunnelService, getActiveProfile, getProfileById, dockerService, diagnoseService, envService, tarSyncService) {
   if (!ctx.tools || typeof ctx.tools.register !== 'function') {
     return;
   }
 
   const genericOutput = {
-    schema: { type: 'object' },
+    schema: { type: 'object', additionalProperties: true },
     render: (_args, value) => ({ type: 'text', content: typeof value === 'string' ? value : JSON.stringify(value, null, 2) })
   };
 
   // 1. remote_exec
-  ctx.tools.register({
+  ctx.tools.register(defineTool({
     name: 'remote_exec',
     description: 'Execute bash/shell command on the active remote workspace host with timeout and ANSI cleaning',
     parameters: {
@@ -41,10 +43,10 @@ export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunne
         signal: res.signal
       };
     }
-  });
+  }));
 
   // 2. remote_fs
-  ctx.tools.register({
+  ctx.tools.register(defineTool({
     name: 'remote_fs',
     description: 'Perform file operations (read, write, stat, list, mkdir, remove) on remote workspace with caching',
     parameters: {
@@ -91,10 +93,10 @@ export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunne
           throw new Error(`Unknown action: ${action}`);
       }
     }
-  });
+  }));
 
   // 3. remote_sync
-  ctx.tools.register({
+  ctx.tools.register(defineTool({
     name: 'remote_sync',
     description: 'Perform 3-way delta or streaming tarball synchronization between local mirror and remote server',
     parameters: {
@@ -125,10 +127,10 @@ export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunne
         return await mirrorSync.push(profile, profile.localMirrorPath, profile.remoteWorkspace, args.force, args.dryRun);
       }
     }
-  });
+  }));
 
   // 4. remote_tunnel
-  ctx.tools.register({
+  ctx.tools.register(defineTool({
     name: 'remote_tunnel',
     description: 'Manage SSH port forwarding tunnels (start, stop, list) with telemetry metrics',
     parameters: {
@@ -157,11 +159,11 @@ export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunne
         return { success: ok };
       }
     }
-  });
+  }));
 
   // 5. remote_docker
   if (dockerService) {
-    ctx.tools.register({
+    ctx.tools.register(defineTool({
       name: 'remote_docker',
       description: 'Manage Docker/Podman containers and Docker Compose on remote host',
       parameters: {
@@ -194,11 +196,11 @@ export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunne
             throw new Error(`Unknown docker action: ${args.action}`);
         }
       }
-    });
+    }));
   }
 
   // 6. remote_service
-  ctx.tools.register({
+  ctx.tools.register(defineTool({
     name: 'remote_service',
     description: 'Inspect and manage Linux systemd or PM2 services on the remote machine',
     parameters: {
@@ -232,10 +234,10 @@ export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunne
         output: res.stdout || res.stderr
       };
     }
-  });
+  }));
 
   // 7. remote_transfer
-  ctx.tools.register({
+  ctx.tools.register(defineTool({
     name: 'remote_transfer',
     description: 'Transfer a file directly between two configured remote servers via in-memory SFTP streams',
     parameters: {
@@ -254,11 +256,11 @@ export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunne
 
       return await sshService.transferFileBetweenHosts(srcProfile, args.sourcePath, destProfile, args.destPath);
     }
-  });
+  }));
 
   // 8. remote_diagnose (New in v0.3.1)
   if (diagnoseService) {
-    ctx.tools.register({
+    ctx.tools.register(defineTool({
       name: 'remote_diagnose',
       description: 'Run targeted system health diagnostics on remote host (ports, disk, memory, oom_killer, service_logs)',
       parameters: {
@@ -278,12 +280,12 @@ export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunne
         if (!profile) throw new Error('No active remote workspace profile selected');
         return await diagnoseService.diagnose(profile, args.category, args.target);
       }
-    });
+    }));
   }
 
   // 9. remote_env (New in v0.3.1)
   if (envService) {
-    ctx.tools.register({
+    ctx.tools.register(defineTool({
       name: 'remote_env',
       description: 'Inspect and atomically update remote .env files preserving comments and structure',
       parameters: {
@@ -320,6 +322,6 @@ export function registerRemoteTools(ctx, sshService, remoteFs, mirrorSync, tunne
           throw new Error(`Unknown env action: ${args.action}`);
         }
       }
-    });
+    }));
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-remote-workspace",
-  "version": "0.3.4",
+  "version": "0.3.6",
   "description": "Enterprise Remote Workspace Plugin for DeepSeek Harness: SSH2 connection pool, SFTP operations, 3-way conflict-aware mirror sync, port forwarding, and native DSH UI.",
   "type": "module",
   "main": "lib/index.js",