@goodandready/dsh-remote-workspace 0.3.6 → 0.3.8

package/lib/mirror-sync-service.js

@@ -11,6 +11,7 @@ export class MirrorSyncService {
     this.remoteFs = remoteFs;
     this.sshService = sshService;
     this.defaultIgnore = [
+      '.dsh-sync-snapshot.json',
       '.git',
       'node_modules',
       '.worktrees',
@@ -96,7 +97,7 @@ export class MirrorSyncService {
         baseRemoteSize === remoteSize
       ) {
         // Check if local was modified
-        const localContent = fs.readFileSync(localFullPath, 'utf8');
+        const localContent = fs.readFileSync(localFullPath);
         const localHash = this.computeHash(localContent);
         if (localHash === baseHash) {
           // File unchanged on both ends
@@ -107,7 +108,7 @@ export class MirrorSyncService {
       // Download content only when potentially changed
       let remoteContent = '';
       try {
-        remoteContent = await this.remoteFs.readFile(profile, remoteFullPath, 'utf8');
+        remoteContent = await this.remoteFs.readFile(profile, remoteFullPath, null);
       } catch (err) {
         continue; // skip unreadable files
       }
@@ -130,7 +131,7 @@ export class MirrorSyncService {
         updated.push(relPath);
         if (!dryRun) {
           fs.mkdirSync(path.dirname(localFullPath), { recursive: true });
-          fs.writeFileSync(localFullPath, remoteContent, 'utf8');
+          fs.writeFileSync(localFullPath, remoteContent);
         }
       }
 
@@ -195,11 +196,12 @@ export class MirrorSyncService {
     };
 
     const localFiles = walkLocal(localDir, localDir);
+    const createdDirs = new Set(['.', '/', '']);
     for (const rel of localFiles) {
       const localFull = path.join(localDir, rel);
       const remoteFull = path.posix.join(remoteDir, rel.replace(/\\/g, '/'));
       const stat = fs.statSync(localFull);
-      const content = fs.readFileSync(localFull, 'utf8');
+      const content = fs.readFileSync(localFull);
       const localHash = this.computeHash(content);
 
       const baseEntry = snapshot[rel];
@@ -211,8 +213,11 @@ export class MirrorSyncService {
 
       if (!dryRun) {
         const parent = path.posix.dirname(remoteFull);
-        await this.remoteFs.mkdir(profile, parent);
-        await this.remoteFs.writeFile(profile, remoteFull, content, 'utf8');
+        if (!createdDirs.has(parent)) {
+          await this.remoteFs.mkdir(profile, parent);
+          createdDirs.add(parent);
+        }
+        await this.remoteFs.writeFile(profile, remoteFull, content);
       }
       pushed.push(rel);
 

package/lib/remote-fs-service.js

@@ -33,6 +33,10 @@ export class RemoteFsService {
   /**
    * Read directory entries with in-memory TTL cache
    */
+  async list(profile, remotePath, bypassCache = false) {
+    return this.listDir(profile, remotePath, bypassCache);
+  }
+
   async listDir(profile, remotePath, bypassCache = false) {
     const key = this.cacheKey(profile.id, 'list', remotePath);
     if (!bypassCache) {

package/lib/routes/docker.js

@@ -7,6 +7,7 @@ export function registerDockerRoutes(sctx, webServer, { dockerService, healthSer
     path: '/dsh-remote-workspace/health',
     handler: async (req, res) => {
       if (req.method !== 'GET') return writeJson(res, 405, { ok: false, error: 'GET only' });
+      if (!isTrustedSettingsRequest(req)) return writeJson(res, 403, { ok: false, error: 'Forbidden' });
       try {
         const url = new URL(req.url, 'http://localhost');
         const profileId = url.searchParams.get('profileId');
@@ -27,6 +28,7 @@ export function registerDockerRoutes(sctx, webServer, { dockerService, healthSer
     path: '/dsh-remote-workspace/docker/list',
     handler: async (req, res) => {
       if (req.method !== 'GET') return writeJson(res, 405, { ok: false, error: 'GET only' });
+      if (!isTrustedSettingsRequest(req)) return writeJson(res, 403, { ok: false, error: 'Forbidden' });
       try {
         const url = new URL(req.url, 'http://localhost');
         const profileId = url.searchParams.get('profileId');

package/lib/routes/files.js

@@ -9,11 +9,24 @@ export function registerFileRoutes(sctx, webServer, { remoteFs, store, sshServic
       if (req.method !== 'POST') return writeJson(res, 405, { ok: false, error: 'POST only' });
       if (!isTrustedSettingsRequest(req)) return writeJson(res, 403, { ok: false, error: 'Forbidden' });
       try {
-        const { profileId, remotePath } = await readBody(req);
-        const profile = store.getProfile(profileId);
+        const body = await readBody(req);
+        let profile = null;
+        if (body.profileId) {
+          profile = store.getProfile(body.profileId);
+        } else if (typeof body.profile === 'string') {
+          profile = store.getProfile(body.profile);
+        } else if (body.profile && typeof body.profile === 'object' && body.profile.host) {
+          profile = body.profile;
+        } else if (body.profile?.id) {
+          profile = store.getProfile(body.profile.id);
+        } else {
+          profile = store.getActiveProfile();
+        }
         if (!profile) return writeJson(res, 404, { ok: false, error: 'Profile not found' });
-        const items = await remoteFs.list(profile, remotePath || profile.remotePath || '.');
-        writeJson(res, 200, { ok: true, items });
+
+        const targetPath = body.remotePath || body.path || profile.remoteWorkspace || profile.remotePath || '.';
+        const entries = await remoteFs.listDir(profile, targetPath);
+        writeJson(res, 200, { ok: true, entries, items: entries, currentPath: targetPath });
       } catch (err) {
         writeJson(res, 500, { ok: false, error: err.message });
       }

package/lib/routes/helpers.js

@@ -25,14 +25,56 @@ export function readBody(req) {
   });
 }
 
+export function parseCookies(cookieHeader) {
+  const cookies = {};
+  if (typeof cookieHeader !== 'string') return cookies;
+  for (const pair of cookieHeader.split(';')) {
+    const idx = pair.indexOf('=');
+    if (idx !== -1) {
+      const key = pair.slice(0, idx).trim();
+      const val = pair.slice(idx + 1).trim();
+      cookies[key] = decodeURIComponent(val);
+    }
+  }
+  return cookies;
+}
+
 export function isTrustedSettingsRequest(req) {
+  const secFetch = req.headers['sec-fetch-site'];
+  // Fail-closed: unconditionally reject cross-site and cross-origin requests
+  if (secFetch && secFetch !== 'same-origin') {
+    return false;
+  }
+
+  // If origin is present, ensure it matches the request host
+  const origin = req.headers['origin'];
+  const host = req.headers['host'];
+  if (origin && host) {
+    try {
+      const url = new URL(origin);
+      if (url.host !== host) {
+        return false;
+      }
+    } catch {
+      return false;
+    }
+  }
+
   const rawAuth = req.headers['authorization'];
-  if (typeof rawAuth === 'string' && rawAuth.startsWith('Bearer ')) return true;
   const cookieHeader = req.headers['cookie'];
-  if (typeof cookieHeader === 'string' && (cookieHeader.includes('token=') || cookieHeader.includes('dsh_token='))) {
-    return true;
+  const expectedToken = process.env.DSH_AUTH_TOKEN || process.env.DSH_TOKEN;
+
+  if (expectedToken) {
+    if (typeof rawAuth === 'string' && rawAuth.startsWith('Bearer ')) {
+      const token = rawAuth.slice(7).trim();
+      if (token && token === expectedToken) return true;
+    }
+    if (typeof cookieHeader === 'string') {
+      const cookies = parseCookies(cookieHeader);
+      const token = cookies['token'] || cookies['dsh_token'];
+      if (token && token === expectedToken) return true;
+    }
   }
-  const secFetch = req.headers['sec-fetch-site'];
   if (secFetch === 'same-origin') return true;
   const ip = req.socket?.remoteAddress || '';
   if (ip === '127.0.0.1' || ip === '::1' || ip === '::ffff:127.0.0.1') return true;

package/lib/routes/profiles.js

@@ -1,6 +1,15 @@
 import { writeJson, readBody, isTrustedSettingsRequest } from './helpers.js';
 import { SshConfigParser } from '../ssh-config-parser.js';
 
+function maskProfile(p) {
+  if (!p || typeof p !== 'object') return p;
+  const masked = { ...p };
+  if (masked.password) masked.password = '••••••••';
+  if (masked.privateKey) masked.privateKey = '••••••••';
+  if (masked.passphrase) masked.passphrase = '••••••••';
+  return masked;
+}
+
 export function registerProfileRoutes(sctx, webServer, { store, sshService, alertService, tunnelService }) {
   // 1. GET /dsh-remote-workspace/state
   sctx.effect(() => webServer.register({
@@ -8,13 +17,14 @@ export function registerProfileRoutes(sctx, webServer, { store, sshService, aler
     path: '/dsh-remote-workspace/state',
     handler: async (req, res) => {
       if (req.method !== 'GET') return writeJson(res, 405, { ok: false, error: 'GET only' });
+      if (!isTrustedSettingsRequest(req)) return writeJson(res, 403, { ok: false, error: 'Forbidden' });
       try {
         const profiles = store.getProfiles();
         const activeId = store.getActiveId() || (profiles[0]?.id || '');
         const tunnels = tunnelService.listActiveTunnels();
         const autoSync = store.isAutoSyncEnabled ? store.isAutoSyncEnabled() : false;
         const alerts = alertService ? alertService.getAlerts() : [];
-        const sanitized = profiles.map((p) => (store.vault ? store.vault.sanitizeProfile(p) : p));
+        const sanitized = profiles.map((p) => (store.vault ? store.vault.sanitizeProfile(p) : maskProfile(p)));
         writeJson(res, 200, { ok: true, activeId, profiles: sanitized, tunnels, autoSync, alerts });
       } catch (err) {
         writeJson(res, 500, { ok: false, error: err.message });
@@ -36,7 +46,7 @@ export function registerProfileRoutes(sctx, webServer, { store, sshService, aler
         }
         sshService.invalidate(profile.id);
         await store.saveProfile(profile);
-        const sanitized = store.vault ? store.vault.sanitizeProfile(profile) : profile;
+        const sanitized = store.vault ? store.vault.sanitizeProfile(profile) : maskProfile(profile);
         writeJson(res, 200, { ok: true, profile: sanitized });
       } catch (err) {
         writeJson(res, 500, { ok: false, error: err.message });

package/lib/routes/terminal.js

@@ -10,9 +10,9 @@ export function registerTerminalRoutes(sctx, webServer, { sshService, store }) {
       if (!isTrustedSettingsRequest(req)) return writeJson(res, 403, { ok: false, error: 'Forbidden' });
       try {
         const { profileId, cols, rows } = await readBody(req);
-        const profile = store.getProfile(profileId);
+        const profile = profileId ? store.getProfile(profileId) : store.getActiveProfile();
         if (!profile) return writeJson(res, 404, { ok: false, error: 'Profile not found' });
-        const session = await sshService.createTerminalSession(profile, cols || 80, rows || 24);
+        const session = await sshService.createTerminalSession(profile, { cols: cols || 80, rows: rows || 24 });
         writeJson(res, 200, { ok: true, sessionId: session.id });
       } catch (err) {
         writeJson(res, 500, { ok: false, error: err.message });
@@ -50,6 +50,7 @@ export function registerTerminalRoutes(sctx, webServer, { sshService, store }) {
     path: '/dsh-remote-workspace/terminal/stream',
     handler: async (req, res) => {
       if (req.method !== 'GET') return writeJson(res, 405, { ok: false, error: 'GET only' });
+      if (!isTrustedSettingsRequest(req)) return writeJson(res, 403, { ok: false, error: 'Forbidden' });
       try {
         const url = new URL(req.url, 'http://localhost');
         const sessionId = url.searchParams.get('sessionId');

package/lib/routes/tunnels.js

@@ -49,6 +49,7 @@ export function registerTunnelRoutes(sctx, webServer, { tunnelService, store })
     path: '/dsh-remote-workspace/tunnels/telemetry',
     handler: async (req, res) => {
       if (req.method !== 'GET') return writeJson(res, 405, { ok: false, error: 'GET only' });
+      if (!isTrustedSettingsRequest(req)) return writeJson(res, 403, { ok: false, error: 'Forbidden' });
       try {
         writeJson(res, 200, { ok: true, tunnels: tunnelService.listActiveTunnels() });
       } catch (err) {

package/lib/ssh-service.js

@@ -313,6 +313,9 @@ export class SshService {
   }
 
   async createTerminalSession(profile, options = {}) {
+    if (typeof options === 'number') {
+      options = { cols: options, rows: arguments[2] };
+    }
     const conn = await this.getConnection(profile);
     const cols = options.cols || 80;
     const rows = options.rows || 24;

package/lib/tar-sync-service.js

@@ -2,6 +2,11 @@ import { spawn } from 'node:child_process';
 import path from 'node:path';
 import fs from 'node:fs';
 
+export function shellQuote(arg) {
+  if (typeof arg !== 'string') arg = String(arg || '');
+  return "'" + arg.replace(/'/g, "'\\''") + "'";
+}
+
 export class TarSyncService {
   /**
    * @param {import('./ssh-service.js').SshService} sshService
@@ -17,11 +22,13 @@ export class TarSyncService {
    */
   async pullTar(profile, remoteDir, localDir) {
     if (!profile) throw new Error('No profile provided');
+    if (!remoteDir || typeof remoteDir !== 'string') throw new Error('remoteDir must be a non-empty string');
     fs.mkdirSync(localDir, { recursive: true });
 
     const client = await this.sshService.getConnection(profile);
     return new Promise((resolve, reject) => {
-      const remoteCmd = `tar --exclude='.git' --exclude='node_modules' -czf - -C "${remoteDir}" .`;
+      const safeRemoteDir = shellQuote(remoteDir);
+      const remoteCmd = `tar --exclude='.git' --exclude='node_modules' -czf - -C ${safeRemoteDir} .`;
       client.exec(remoteCmd, (err, stream) => {
         if (err) return reject(err);
 
@@ -52,14 +59,16 @@ export class TarSyncService {
    */
   async pushTar(profile, localDir, remoteDir) {
     if (!profile) throw new Error('No profile provided');
+    if (!remoteDir || typeof remoteDir !== 'string') throw new Error('remoteDir must be a non-empty string');
     if (!fs.existsSync(localDir)) throw new Error(`Local dir does not exist: ${localDir}`);
 
+    const safeRemoteDir = shellQuote(remoteDir);
     const client = await this.sshService.getConnection(profile);
     // Ensure remoteDir exists
-    await this.sshService.exec(profile, `mkdir -p "${remoteDir}"`);
+    await this.sshService.exec(profile, `mkdir -p ${safeRemoteDir}`);
 
     return new Promise((resolve, reject) => {
-      const remoteCmd = `tar -xzf - -C "${remoteDir}"`;
+      const remoteCmd = `tar -xzf - -C ${safeRemoteDir}`;
       client.exec(remoteCmd, (err, stream) => {
         if (err) return reject(err);
 

package/lib/updater-service.js

@@ -31,21 +31,9 @@ function isLoopback(value) {
   )
 }
 
-function isPrivateLan(value) {
-  const address = value?.toLowerCase().replace(/^\[|\]$/g, '')
-  if (!address) return false
-  const ipv4 = address.startsWith('::ffff:') ? address.slice(7) : address
-  return (
-    ipv4.startsWith('192.168.') ||
-    ipv4.startsWith('10.') ||
-    /^172\.(1[6-9]|2[0-9]|3[0-1])\./.test(ipv4)
-  )
-}
-
 export function isTrustedUpdateRequest(request) {
   if (header(request, UPDATE_HEADER) !== '1') return false
-  const remote = request.socket?.remoteAddress
-  if (!isLoopback(remote) && !isPrivateLan(remote)) return false
+  if (!isLoopback(request.socket?.remoteAddress)) return false
   const site = header(request, 'sec-fetch-site')
   if (site !== undefined && site !== 'same-origin') return false
   const origin = header(request, 'origin')
@@ -53,11 +41,8 @@ export function isTrustedUpdateRequest(request) {
   if (origin === undefined || host === undefined) return false
   try {
     const url = new URL(origin)
-    return (
-      (url.protocol === 'http:' || url.protocol === 'https:') &&
-      (isLoopback(url.hostname) || isPrivateLan(url.hostname)) &&
-      url.host === host
-    )
+    return (url.protocol === 'http:' || url.protocol === 'https:')
+      && isLoopback(url.hostname) && url.host === host
   } catch (err) { /* best-effort fallback */
     return false
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-remote-workspace",
-  "version": "0.3.6",
+  "version": "0.3.8",
   "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",
@@ -16,7 +16,6 @@
     "README.md",
     "README.ru.md",
     "README.zh.md",
-    "docs/design/DESIGN.md",
     "LICENSE"
   ],
   "scripts": {

package/docs/design/DESIGN.md

@@ -1,91 +0,0 @@
-# DESIGN.md — dsh-remote-workspace
-
-## Product / Purpose
-- **Назначение**: Полнофункциональный корпоративный плагин удалённой разработки для DeepSeek Harness (DSH). Обеспечивает подключение к удалённым серверам по SSH (ключ / пароль), SFTP-навигацию, 3-way синхронизацию локального зеркала, SSH-туннелирование (port forwarding) и компактный набор инструментов для LLM агента (
-emote_exec, 
-emote_fs, 
-emote_sync, 
-emote_tunnel).
-- **Аудитория**: Разработчики и инженеры, использующие DSH для работы с проектами на VPS, удаленных серверах, облачных инстансах и MiniPC.
-- **Статус**: Стабильный релиз v0.3.3 (One-Click Updater, Cross-Plugin API, Clean Tokens, Modular Routes).
-
-## User Surfaces
-- **Web/UI**: Нативная карточка настроек профилей хостов (settings.plugin.item), удаленный SFTP браузер каталогов, чип статуса подключения в шапке сессии (conversation.session.header.utilities).
-- **DSH UI / settings / slots**:
-  - Карточка плагина: settings.plugin.item для @goodandready/dsh-remote-workspace.
-  - Статусный чип сессии: conversation.session.header.utilities (order: 30).
-  - Запрещённые слоты: settings.section (запрещён стандартом authoring, все настройки только внутри settings.plugin.item).
-- **API маршруты**:
-  - GET /dsh-remote-workspace/state: получение профилей, активного хоста и туннелей.
-  - POST /dsh-remote-workspace/profiles/save: сохранение профиля (с валидацией и сбросом старых соединений).
-  - POST /dsh-remote-workspace/profiles/delete: удаление профиля.
-  - POST /dsh-remote-workspace/profiles/active: переключение активного профиля.
-  - POST /dsh-remote-workspace/test: live-проверка связи (Ping, OS, Latency).
-  - POST /dsh-remote-workspace/browse: чтение содержимого удалённой папки (SFTP).
-  - POST /dsh-remote-workspace/sync: запуск двусторонней 3-way синхронизации зеркала (pull/push).
-  - POST /dsh-remote-workspace/tunnels/start: создание локального SSH-туннеля.
-  - POST /dsh-remote-workspace/tunnels/stop: остановка активного SSH-туннеля.
-- **CLI / Model Tools**:
-  - 
-emote_exec: выполнение команд на сервере.
-  - 
-emote_fs: чтение, атомарная запись, stat, mkdir, remove.
-  - 
-emote_sync: запуск 3-way синхронизации с контролем конфликтов.
-  - 
-emote_tunnel: создание/остановка/листинг туннелей.
-
-## Visual Direction (Единый стиль dsh-clinebot)
-- **Атмосфера**: Премиальный инженерный интерфейс уровня VS Code Remote / Linear. Строгая типографика, плотные функциональные карточки, чистые переходы и отсутствие лишних теней.
-- **Токены ядра DSH**:
-  - Фоны: ar(--dsw-alias-bg-layer-3) (карточки), ar(--dsw-alias-bg-layer-2) (поля ввода, вложенные блоки, таблицы), ar(--dsw-alias-bg-layer-1) (сегменты, подложка).
-  - Границы: ar(--dsw-alias-border-l2).
-  - Текст: ar(--dsw-alias-label-primary), ar(--dsw-alias-label-secondary), ar(--dsw-alias-label-dimmed).
-  - Статусы: ar(--dsw-alias-state-success-primary), ar(--dsw-alias-state-warning-primary), ar(--dsw-alias-state-error-primary), ar(--dsw-alias-state-brand-primary).
-- **Шапка карточки**:
-  - Название плагина, подзаголовок, статусный аккордеонный шеврон.
-  - Статусные пилюли (Badges): Host online / Offline, Active Profile, Active Tunnels count.
-
-## Components And States
-1. **Секция 1: Профили серверов (SSH Profiles)**:
-   - Таблица настроенных машин с бейджами типа аутентификации (🔑 key / 🔒 pwd), меткой активного сервера, кнопками «Сделать активным», «Редактировать», «Удалить».
-   - Пустое состояние: ясное сообщение и кнопка + Добавить хост.
-2. **Секция 2: Интерактивный редактор профиля**:
-   - Название, Хост/IP, Порт, Пользователь.
-   - Сегментированный переключатель авторизации (SSH-ключ vs Пароль).
-   - Поля авторизации: путь к приватному ключу, парольная фраза, либо пароль (маскированный).
-   - Директории: Удалённый каталог проекта (с кнопкой вызова SFTP браузера) и **Локальная папка зеркала** (с подсказкой назначения).
-   - Кнопки: «⚡ Проверить связь (Ping)», «Отмена», «Сохранить».
-   - Результат пинга: отображение Latency: X ms | OS: Y или понятный текст ошибки.
-3. **Секция 3: Диагностика активного подключения**:
-   - Четырёхколоночная сетка метрик активного сервера (Хост, Удалённая папка, Метод авторизации, Статус ядра).
-4. **Секция 4: Синхронизация зеркала (Mirror Sync)**:
-   - Кнопки Pull (Remote -> Local) и Push (Local -> Remote) с индикацией процесса, результатом и предупреждением при конфликтах.
-5. **Секция 5: SSH-туннели (Port Forwarding)**:
-   - Таблица активных туннелей (ID, Локальный порт -> Удалённый порт) с кнопкой закрытия.
-   - Форма создания нового туннеля (Local Port, Remote Port, кнопка «Открыть туннель»).
-
-## Security & Reliability
-- Защита всех мутирующих эндпоинтов проверкой isTrustedSettingsRequest(req) (защита от cross-site атак).
-- Безопасное чтение Cordis сервисов через .get() с fallback.
-- Изоляция жизненного цикла стримов в TunnelService (прослушивание сокетных и потоковых ошибок).
-
-### v0.3.3 Client Architecture Modularization (Refs: #14)
-- Client-side code decomposed into modular ES components under `src/client/`:
-  - `src/client/styles.js` — CSS styling with DSH theme variables
-  - `src/client/locales.js` — English and Chinese translation dictionaries
-  - `src/client/env-editor.js` — Remote .env manager tab component
-  - `src/client/docker.js` — Container inspector tab component
-  - `src/client/terminal.js` — Web terminal tab component
-  - `src/client/file-browser.js` — Remote file explorer tab component
-  - `src/client/settings-view.js` — Profiles, tunnels, and sync settings component
-  - `src/client/updater-section.js` — One-click updater section component
-  - `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 }`.